fix: progress tests pass on Bun by injecting context directly

Replace vi.mock-based context override with direct ctx parameter
injection in createStoreProgress/createRestoreProgress. The vi.mock
for getCliContext was not being applied in Bun's module loader,
causing pipe-mode detection to return a no-op tracker.

Also adds CLAUDE.md with zero-tolerance test policy.

Author: flyingrobots

CLAUDE.md

@@ -0,0 +1,31 @@
+# CLAUDE.md — Project Instructions
+
+## Quality Policy
+
+**Zero tolerance for failing tests.** Pre-existing failures are not acceptable. If any test fails on any supported runtime (Node, Bun, Deno), it must be fixed before proceeding. Never skip, ignore, or defer a failing test.
+
+## Supported Runtimes
+
+All code must pass on all three runtimes:
+- Node.js 22.x (primary)
+- Bun (via Docker: `docker compose run --build --rm test-bun`)
+- Deno (via Docker: `docker compose run --build --rm test-deno`)
+
+## Test Commands
+
+- `npm test` — unit tests (Node)
+- `npx eslint .` — lint (0 errors required)
+- `docker compose run --build --rm test-bun bunx vitest run test/unit` — Bun unit tests
+- `docker compose run --build --rm test-deno deno run -A npm:vitest run test/unit` — Deno unit tests
+- `docker compose run --build --rm test-bun bunx vitest run test/integration` — Bun integration tests
+- `docker compose run --build --rm test-deno deno run -A npm:vitest run test/integration` — Deno integration tests
+
+## Release Checklist
+
+Before tagging a release, ALL of the following must pass:
+1. `npx eslint .` — 0 errors
+2. `npm test` — all tests pass (Node)
+3. Bun unit + integration tests pass
+4. Deno unit + integration tests pass
+5. `npm pack --dry-run` — clean
+6. `npx jsr publish --dry-run --allow-dirty` — clean

bin/ui/progress.js

@@ -32,12 +32,12 @@ function formatBytes(bytes) {
  * @param {boolean} [options.quiet] - Suppress all progress output.
  * @returns {{ attach(observer: { on(event: string, fn: Function): void, removeListener(event: string, fn: Function): void }): void, detach(): void }}
  */
-export function createStoreProgress({ filePath, chunkSize, quiet, fileSize: providedSize }) {
+export function createStoreProgress({ filePath, chunkSize, quiet, fileSize: providedSize, ctx: providedCtx }) {
   if (quiet) {
     return { attach() {}, detach() {} };
   }
 
-  const ctx = getCliContext();
+  const ctx = providedCtx || getCliContext();
   if (ctx.mode === 'pipe') {
     return { attach() {}, detach() {} };
   }
@@ -60,12 +60,12 @@ export function createStoreProgress({ filePath, chunkSize, quiet, fileSize: prov
  * @param {boolean} [options.quiet] - Suppress all progress output.
  * @returns {{ attach(observer: { on(event: string, fn: Function): void, removeListener(event: string, fn: Function): void }): void, detach(): void }}
  */
-export function createRestoreProgress({ totalChunks, quiet }) {
+export function createRestoreProgress({ totalChunks, quiet, ctx: providedCtx }) {
   if (quiet || totalChunks === 0) {
     return { attach() {}, detach() {} };
   }
 
-  const ctx = getCliContext();
+  const ctx = providedCtx || getCliContext();
   if (ctx.mode === 'pipe') {
     return { attach() {}, detach() {} };
   }

test/unit/cli/progress.test.js

@@ -1,18 +1,14 @@
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import EventEmitterObserver from '../../../src/infrastructure/adapters/EventEmitterObserver.js';
+import { createStoreProgress, createRestoreProgress } from '../../../bin/ui/progress.js';
 import { makeCtx } from './_testContext.js';
 
-vi.mock('../../../bin/ui/context.js', () => ({
-  getCliContext: () => makeCtx('static'),
-}));
-
-const { createStoreProgress, createRestoreProgress } = await import('../../../bin/ui/progress.js');
-
 const FILE_SIZE = 5 * 256 * 1024;
+const ctx = makeCtx('static');
 
 describe('createStoreProgress', () => {
   it('returns no-op when quiet is true', () => {
-    const p = createStoreProgress({ filePath: 'test.bin', chunkSize: 256 * 1024, quiet: true });
+    const p = createStoreProgress({ filePath: 'test.bin', chunkSize: 256 * 1024, quiet: true, ctx });
     const emitter = new EventEmitterObserver();
     p.attach(emitter);
     emitter.metric('chunk', { action: 'stored', index: 0, size: 256 * 1024 });
@@ -21,7 +17,7 @@ describe('createStoreProgress', () => {
   });
 
   it('attaches and detaches from EventEmitter', () => {
-    const p = createStoreProgress({ filePath: 'test.bin', chunkSize: 256 * 1024, quiet: false, fileSize: FILE_SIZE });
+    const p = createStoreProgress({ filePath: 'test.bin', chunkSize: 256 * 1024, quiet: false, fileSize: FILE_SIZE, ctx });
     const emitter = new EventEmitterObserver();
     p.attach(emitter);
     expect(emitter.listenerCount('chunk:stored')).toBe(1);
@@ -30,7 +26,7 @@ describe('createStoreProgress', () => {
   });
 
   it('tracks chunk events without throwing', () => {
-    const p = createStoreProgress({ filePath: 'test.bin', chunkSize: 256 * 1024, quiet: false, fileSize: FILE_SIZE });
+    const p = createStoreProgress({ filePath: 'test.bin', chunkSize: 256 * 1024, quiet: false, fileSize: FILE_SIZE, ctx });
     const emitter = new EventEmitterObserver();
     p.attach(emitter);
     for (let i = 0; i < 5; i++) {
@@ -43,23 +39,23 @@ describe('createStoreProgress', () => {
 
 describe('createRestoreProgress', () => {
   it('returns no-op when quiet is true', () => {
-    const p = createRestoreProgress({ totalChunks: 5, quiet: true });
+    const p = createRestoreProgress({ totalChunks: 5, quiet: true, ctx });
     const emitter = new EventEmitterObserver();
     p.attach(emitter);
     p.detach();
     expect(emitter.listenerCount('chunk:restored')).toBe(0);
   });
 
   it('returns no-op for 0-chunk manifests', () => {
-    const p = createRestoreProgress({ totalChunks: 0, quiet: false });
+    const p = createRestoreProgress({ totalChunks: 0, quiet: false, ctx });
     const emitter = new EventEmitterObserver();
     p.attach(emitter);
     p.detach();
     expect(emitter.listenerCount('chunk:restored')).toBe(0);
   });
 
   it('attaches and detaches from EventEmitter', () => {
-    const p = createRestoreProgress({ totalChunks: 3, quiet: false });
+    const p = createRestoreProgress({ totalChunks: 3, quiet: false, ctx });
     const emitter = new EventEmitterObserver();
     p.attach(emitter);
     expect(emitter.listenerCount('chunk:restored')).toBe(1);