Distillation

Date: Sep 8, 2026

• 🔴 [enforced-workflow] (21:40) User required agents to always load repo-setup before situation skills; the attempted load failed because repo-setup was unavailable. Available skills were ast-grep, customize-opencode, jj-create-pr, jj-guide, jj-update-pr, jj-workflow, and sentry-cli.
• 🔴 (21:40) Repository instructions in /home/byk/Code/getsentry/cli-pr-1558/AGENTS.md specify the pipeline triage → explore → plan → implement → review → ship; worker is a deprecated alias of implement; target repositories require reading their AGENTS.md / CONTRIBUTING.md first.
• 🔴 (21:41) User required always checking package.json for the latest scripts; commands run from packages/cli or via pnpm --filter sentry run <script> from the repository root.
• 🔴 (21:41) User required all packages to be added to devDependencies, never dependencies, because everything is bundled with esbuild and CI enforces this using pnpm run check:deps.
• 🔴 (21:41) User required package installation with pnpm add -D <package>, always including the -D flag.
• 🔴 (21:41) User required Node.js standard-library node:* APIs and prohibited reintroducing Bun.* globals, bun:test, or bun CLI commands because the project migrated from Bun to Node.js.
• 🔴 (21:41) User required importing buildCommand from src/lib/command.ts rather than directly from @stricli/core; the wrapper adds telemetry, injects --json/--fields, and handles output rendering.
• 🔴 (21:41) User required importing buildRouteMap from src/lib/route-map.ts rather than directly from @stricli/core; its standard aliases are list → ls, view → show, delete → remove, rm, and create → new.
• 🔴 (21:41) User required required positional multi-values to be space-separated variadics without comma splitting, while optional flag multi-values are comma-separated and may also be repeatable.
• 🔴 (21:41) User required all paginated list commands to use the shared bidirectional cursor-stack infrastructure supporting -c next, -c prev, and -c first; "last" is a silent alias for "next".
• 🔴 (21:41) User required using paginationHint() from src/lib/list-command.ts rather than manually assembling navigation arrays.
• 🔴 (21:41) User required list commands whose --limit exceeds API_MAX_PER_PAGE (100) to transparently fetch multiple pages, cap each request with Math.min(flags.limit, API_MAX_PER_PAGE), and stop when the requested limit is met or pages are exhausted.
• 🔴 (21:41) User prohibited passing a per_page value greater than API_MAX_PER_PAGE to the API because the server silently caps it and otherwise causes commands to return fewer items than requested.
• 🔴 (21:41) User required command docs and skill files to be generated through pnpm run generate:docs, which runs generate:command-docs followed by generate:skill; pnpm run check:fragments validates fragment-to-route consistency.
• 🟡 (21:41) Review target state: repository /home/byk/Code/getsentry/cli-pr-1558 was on branch fix/agent-conversation-reference-root-cause at commit bbaa7b3b722b87f1bccfe84063269cbbcabe2741; its upstream origin/pr-1558 was marked [gone].
• 🟡 (21:41) The uncommitted diff contained exactly 14 modified files with 234 insertions(+), 95 deletions(-): apps/cli-docs/src/fragments/commands/agent-conversation.md; packages/cli/script/check-fragments.ts; packages/cli/script/generate-command-docs.ts; packages/cli/script/generate-skill-markdown.ts; packages/cli/script/generate-skill.ts; packages/cli/src/commands/agent-conversation/list.ts; packages/cli/src/commands/agent-conversation/view.ts; packages/cli/src/lib/command.ts; packages/cli/src/lib/introspect.ts; packages/cli/src/lib/list-command.ts; packages/cli/test/commands/agent-conversation/list.test.ts; packages/cli/test/lib/command.test.ts; packages/cli/test/lib/introspect.test.ts; and packages/cli/test/script/generate-skill-markdown.test.ts.