Distillation

Date: Aug 24, 2026

• 🔴 (15:08) User stated Craft package management must always use pnpm; npm and yarn must never be used.
• 🔴 (15:08) User stated Craft uses Volta-managed Node.js v22.12.0 and dependencies must be installed with pnpm install --frozen-lockfile.
• 🔴 (15:08) User stated development commands are pnpm build (builds dist/craft), pnpm test (runs tests), pnpm lint (runs ESLint), and pnpm fix (auto-fixes lint); manual testing uses pnpm build && ./dist/craft.
• 🔴 (15:08) User stated the project package is @sentry/craft, at version 2.29.0-dev.0, with executable bins craft and sentry-craft both mapped to dist/craft.
• 🔴 (15:08) User stated Craft is TypeScript throughout; formatting uses Prettier 3.x with single quotes and no arrow parentheses via .prettierrc.yml; linting uses ESLint 9.x flat config in eslint.config.mjs with typescript-eslint; unused variables prefixed with _ are permitted.
• 🔴 (15:08) User stated project structure includes src/__mocks__/, src/__tests__/, src/artifact_providers/, src/commands/, src/schemas/, src/status_providers/, src/targets/, src/types/, src/utils/, src/config.ts, src/index.ts, src/logger.ts, and bundled executable dist/craft.
• 🔴 (15:08) User stated tests use Vitest; test files reside in src/__tests__/ with *.test.ts names; mocks use vi.fn(), vi.mock(), and vi.spyOn().
• 🔴 (15:08) User stated the main branch is master; CI tests Node.js 20 and 22; Craft releases itself using its own tooling.
• 🔴 (15:08) User stated project configuration is .craft.yml at repository root and its schema is defined in src/schemas/.
• 🔴 (15:08) User stated Craft dry-run mode uses centralized proxies rather than per-function isDryRun() checks: Git via getGitClient() from src/utils/git.ts or createGitClient(directory); GitHub via getGitHubClient() from src/utils/githubApi.ts; file writes via safeFs from src/utils/dryRun.ts; other actions via safeExec() or safeExecSync() from src/utils/dryRun.ts.
• 🔴 (15:08) User stated ESLint blocks direct simple-git imports via no-restricted-imports and direct new Octokit() via no-restricted-syntax; wrapper modules requiring raw simple-git must precede the import with // eslint-disable-next-line no-restricted-imports -- This is the wrapper module.
• 🔴 (15:08) User stated new destructive operations must use dry-run-aware clients: getGitClient()/createGitClient() for Git, getGitHubClient() for GitHub APIs, safeFs.writeFile(), safeFs.unlink(), etc. for file writes, and safeExec(action, description) for other actions. Explicit isDryRun() checks remain appropriate for tools with their own dry-run flags (such as dart pub publish --dry-run in the pubDev target), mock-data needs, and UX optimizations such as skipped sleep timers.
• 🟡 (15:08) Assistant identified 2.29.0-dev.0 as the active development version and planned to change the version gate to 2.29.0, make version comparison robust to build metadata, and replace a custom argv scan with node:util.parseArgs plus token validation.