Distillation

Date: Sep 9, 2026

• 🔴 (18:40) User stated the injected --log-level flag is always stripped; flags in ALWAYS_STRIP are always removed before command execution, so the command never sees them.
• 🔴 (18:40) User stated --fields is always injected when output: { human: ... }, regardless of whether a command-defined json flag causes the injected JSON_FLAG to be skipped.
• 🔴 (18:40) User directed contributors to always check package.json for the latest scripts; commands run from packages/cli or via pnpm --filter sentry run <script> from the repository root.
• 🔴 (18:40) User directed that packages always be added with pnpm add -D <package>, including the -D flag.
• 🔴 (18:40) User stated command functions use async *func() generators, yield new CommandOutput(data), and return { hint }; output.human receives the same data object serialized to JSON, avoiding divergent data paths.
• 🔴 (18:40) User directed commands not to define their own json flag and not to use stdout.write() or if (flags.json) branching because the wrapper auto-injects --json and --fields and handles output.
• 🔴 (18:40) User stated create commands should import DRY_RUN_FLAG and DRY_RUN_ALIASES for consistent dry-run support.
• 🔴 (18:40) User stated list commands have 3 abstraction levels and contributors should prefer the highest abstraction level that fits the use case.
• 🔴 (18:40) User stated all command documentation and skill files are generated with pnpm run generate:docs, which runs generate:command-docs followed by generate:skill; generation also runs automatically during dev, build, typecheck, and test.
• 🔴 (18:40) User directed reuse of existing utilities: duration formatting from src/lib/formatters/time-utils.ts via formatDurationCompact or formatDurationVerbose; hex ID handling via validateHexId, tryNormalizeHexId, or normalizeHexId in src/lib/hex-id.ts; relative time via formatRelativeTime; table/Markdown output from src/lib/formatters/; pagination from src/lib/db/pagination.ts and src/lib/list-command.ts; errors from src/lib/errors.ts rather than ad-hoc error types; and search query construction from src/lib/search-query.ts and src/lib/arg-parsing.ts.
• 🔴 (18:40) User directed that new explore datasets verify the filtering mechanism expected by the underlying API and implement dataset-specific behavior in resolveDatasetConfig; sort, query, fetch, and field validation should remain centralized there rather than using scattered if (dataset === ...) checks in the func body.
• 🔴 (18:40) User identified property-based testing candidates: 1. parsing functions such as DSNs, issue IDs, and aliases; 2. encoding/decoding with round-trip invariants; 3. symmetric operations where a op b = b op a; 4. idempotent operations where f(f(x)) = f(x); 5. validation functions where valid inputs are accepted and invalid inputs rejected.
• 🔴 [requested-tests] (18:40) User directed that when property tests are added for a function with existing unit tests, redundant unit tests must be removed and the unit-test file must receive a header comment identifying which invariants live in the property-test file.
• 🟡 (18:40) Repository documentation described CLI capabilities: DSN auto-detection across .env files and JavaScript, Python, Go, Java, Ruby, and PHP source; project-root detection via VCS, language, and build markers; directory-name inference using bidirectional word-boundary matching; multi-region fan-out to us.sentry.io and de.sentry.io; monorepo short aliases; Seer AI issue explain and issue plan; and OAuth Device Flow authentication without browser redirects.
• 🟡 (18:40) Repository guidance showed parseSlashSeparatedArg and parseOrgProjectArg imported from ../../lib/arg-parsing.js.
• 🟡 (18:40) Repository guidance showed upsert("users", { id: 1, name: "Bob", created_at: now }, ["id"], { excludeFromUpdate: ["created_at"] }) as the pattern for excluding created_at from updates.
• 🟡 (18:40) Repository guidance identified silently swallowing optional-data fetch errors with try { data = await fetchOptionalData(); } catch { return []; } as wrong because the error vanishes silently.
• 🟡 (18:40) Relevant repository instruction files found: /home/byk/Code/getsentry/cli-pr-1558/AGENTS.md, /home/byk/Code/getsentry/cli-pr-1558/packages/cli/CONTRIBUTING.md, and /home/byk/Code/getsentry/cli-pr-1558/packages/cli/AGENTS.md.
• 🟢 (18:41) A subsequent file search returned No files found.