Distillation

🔴 (23:58) User directed: Never return an empty response. 🔴 (23:58) User directed: Always load repo-setup before situation skills. 🔴 (23:58) User directed: Always preserve strict output-mode and binary-safety contracts. 🔴 (23:58) User stated applyPatch() in bspatch.ts ALWAYS computes SHA-256 inline and returns it, with no separate verification step. 🔴 (23:58) User stated buildIssueListCollapse() always starts with ['filtered','unhandled']. 🟡 [requested-review] (23:58) User requested an independent, read-only adversarial correctness review of getsentry/cli PR #1569, pinned to exact base ec83887a16f780f32fba4b7d710bad262dba3a22 and exact head dbee7e9f2a4555e044f74dfebad002cccfa43a1f, in worktree /home/byk/.local/share/opencode/worktree/0e8c2f3bbe145a3280d6fb6ed900a5121f382cf0/toolkit-bridge-upgrade. 🔴 [enforced-read-only] (23:58) User prohibited modifying files, VCS state, or PR state during the review. 🟡 (23:58) User requested inspection of every file changed in base..head, plus relevant surrounding code and tests. 🟡 (23:58) User requested review focus on exact ordered Toolkit-then-legacy source resolution; fallback only for a genuine typed HTTP 404; and reuse of successful responses. 🟡 (23:58) User requested review focus on prefixed CLI release discovery, including draft releases, prereleases, malformed tags, and semantic ordering. 🟡 (23:58) User requested review focus on source affinity across pinned validation, stable/nightly delta download, full download, prefetch, and changelog behavior. 🟡 (23:58) User requested review focus on offline cache behavior, compatibility-wrapper defaults, and updated command fixtures. 🔴 (23:58) User required a substantive structured report in which every finding appears first as PASS, CONCERN, or MUST-FIX, with exact file:line evidence and a deterministic regression for each defect. 🔴 (23:58) User required that a defect-free review explicitly state PASS and list residual test gaps. 🔴 (23:58) User required that a blocked review begin with BLOCKED and include the exact tool/error. 🔴 (23:58) User required the review to end exactly with either MERGE or DO-NOT-MERGE. 🔴 (23:58) User-provided repository listing showed the worktree contains .craft.yml, .git, .github/, .gitignore, .lore.md, .nojekyll, .npmrc, AGENTS.md, apps/, CHANGELOG.md, codemods/, docs/, LICENSE.md, node_modules/, package.json, packages/, pnpm-lock.yaml, pnpm-workspace.yaml, and README.md. 🔴 (23:58) User-provided AGENTS.md identifies the agent as “Jared (Outpost agent),” says autonomous GitHub coding work normally occurs in /workspace/repo, and defines the pipeline as triage → explore → plan → implement → review → ship; worker is a deprecated alias of implement. 🔴 (23:58) User-provided AGENTS.md model roles: primary Jared uses Claude Opus 4.8 for heavy triage/plan/review and xAI Grok 4.3 for light triage/plan/review; explore uses OpenAI gpt-5-mini; implement uses Moonshot kimi-k2.7-code; and ship uses xAI Grok grok-build-0.1. 🔴 (23:58) User-provided AGENTS.md states operator dashboard turns labeled New operator chat or Operator guidance: skip triage and should be treated directly as the task. 🔴 (23:58) User-provided AGENTS.md states long-term project knowledge for the Outpost repository lives in .lore.md when present, and target repositories’ AGENTS.md / CONTRIBUTING.md should be read first. 🔴 (23:58) User-provided .lore.md states sentry api must preserve rawApiRequest() status text; empty textual non-2xx bodies must fall back to HTTP status/request information; JSON mode must expose {status, statusText, body}; binary Uint8Array successes bypass formatters and remain raw; binary errors emit only status/content-type/byte-count summaries and never bytes or JSON coercion; prompts must not block scripted runs or interleave with stdout JSON. 🔴 (23:58) User-provided .lore.md states rawApiRequest() treats every status outside 200–299 as an error, including 304; whitespace-only textual errors fall back to HTTP <status> <statusText> — <method> /api/0/<endpoint>. 🔴 (23:58) User-provided .lore.md states the TRDIFF10 header is 32 bytes: magic, controlLen, diffLen, and newSize, with numeric fields encoded as i64 little-endian; the control block is fully decompressed using zstdDecompressSync, while diff and extra blocks stream through createZstdStreamReader into BufferedStreamReader. 🔴 (23:58) User-provided .lore.md states loadOldBinary() copies to temporary storage using COPYFILE_FICLONE, falling back to a regular copy, before reading into memory; cleanupPatchResources() attempts every cleanup step despite earlier failures; write errors are captured early with writer.on('error') to prevent ERR_UNHANDLED_ERROR on ENOSPC/EIO. 🔴 (23:58) User-provided .lore.md states src/commands/issue/list.ts defines LIFETIME_FIELDS = new Set(['count','userCount','firstSeen','lastSeen']); buildListApiOptions(json, fields) enables collapseLifetime only when json && fields !== undefined && fields.length > 0 && !fields.some(f => LIFETIME_FIELDS.has(f)); human output never collapses lifetime. 🔴 (23:58) User-provided .lore.md states buildIssueListCollapse() conditionally adds 'lifetime' and then 'stats' after its initial ['filtered','unhandled']; ISSUE_DETAIL_COLLAPSE may safely include 'lifetime'; issue-detail output retains count, userCount, firstSeen, and lastSeen. 🔴 (23:58) User-provided .lore.md states IssueViewOutputSchema in src/types/sentry.ts extends SentryIssueSchema with event, org, replayIds, and trace, populated by jsonTransformIssueView, and is wired as schema: IssueViewOutputSchema in view.ts.