lacuna-cli 0.1.4 → 0.1.6

package/README.md

@@ -67,10 +67,16 @@ Interactive setup wizard. Configures your model, test runner, source directory,
 
 Works from any subdirectory — lacuna finds the project root automatically.
 
-For **React and Next.js** projects, `lacuna init` also:
+For **React** projects, `lacuna init` also:
 - Installs `@testing-library/react`, `@testing-library/jest-dom`, `@testing-library/user-event`, and `jsdom`
-- Creates `vitest.config.ts` with the correct `environment: 'jsdom'` and `@/` alias (read from your `tsconfig.json`)
-- Creates a setup file pre-loaded with global `vi.mock()` calls for `next/navigation`, `next/headers`, and `next/cache` so individual tests don't need to mock them
+- Creates `vitest.config.ts` with `environment: 'jsdom'` and `restoreMocks: true` + `clearMocks: true`
+- Creates a setup file with `@testing-library/jest-dom` and `beforeEach`/`afterEach` mock cleanup hooks
+
+For **Next.js** projects, the setup is different — Next.js and plain React are not compatible:
+- Installs the same testing-library packages but does **not** add `environment: 'jsdom'` to vitest config (Next.js manages its own environment)
+- Adds `@/` alias (from your `tsconfig.json`) and a `server-only` stub alias to vitest config
+- Creates a setup file with global `vi.mock()` calls for `next/navigation`, `next/headers`, `next/cache`, `next/image`, and `next/font` — these are Next.js-specific mocks that would crash a plain React project
+- If test dependencies are found in `node_modules` but not declared in `package.json` (e.g. from a different branch), lacuna will prompt you to add them — undeclared deps break CI on a fresh checkout
 
 ```bash
 lacuna init
@@ -108,7 +114,7 @@ lacuna generate --format json --output report.json
 ```
 
 ### `lacuna fix`
-Finds all failing tests and repairs them using AI — without rewriting them from scratch. Sends each failing file along with its error output and source code to the model, which surgically fixes what's broken and retries until it passes.
+Finds all failing tests and repairs them using AI. Sends each failing file along with its error output and source code to the model, which surgically fixes what's broken and retries until it passes. If all fix retries fail, lacuna automatically deletes the broken test and regenerates it from the source file — a clean slate rather than another round of patchwork.
 
 ```bash
 lacuna fix
@@ -117,15 +123,19 @@ lacuna fix --file src/utils/math.test.ts   # fix a single test file (skips full
 lacuna fix --dry-run                       # preview fixes without writing
 lacuna fix --verbose                       # live code panel as model writes each fix
 lacuna fix --fresh                         # re-run the suite even if cache is recent
+lacuna fix --no-regenerate-on-failure      # disable the regen fallback (fix only, no delete)
+lacuna fix --fix-polluters                 # also handle tests that pass alone but fail in suite
 ```
 
-Unlike `lacuna generate`, which creates new tests, `lacuna fix` operates on existing failing tests. It preserves all test logic and only changes what is necessary to make the suite pass.
+Unlike `lacuna generate`, which creates new tests, `lacuna fix` operates on existing failing tests and preserves all test logic where possible.
+
+**Regeneration fallback (default on):** When fix retries are exhausted, lacuna deletes the test file and regenerates it from scratch using the generate path. This works better than continued repair for structurally broken tests — the AI starts with a clean conversation and full source context instead of carrying forward a chain of failed hypotheses. Use `--no-regenerate-on-failure` to disable this and get fix-only behaviour.
 
-If all retries fail or the model oscillates (identical output detected), the original file is restored automatically. Your test suite is always left in a coherent state.
+**Passing in isolation, failing in suite (`--fix-polluters`):** Some tests pass when run alone but fail in the full suite. `--fix-polluters` handles these in two phases: (1) bisect the test suite to find if another file is leaking state (e.g. an uncleaned mock or global), and fix the polluter; (2) if no polluter can be isolated (the test has an internal spy lifecycle bug), delete and regenerate the victim file directly.
 
-If a fix attempt breaks an import (causing 0 tests to be collected) or reduces the number of passing tests, lacuna detects the regression and tells the model exactly what the original failure was — so it doesn't waste further iterations trying to recover from the wrong problem.
+If all retries fail or the model oscillates (identical output detected), the original file is restored automatically. If a fix attempt breaks an import or reduces passing tests, lacuna detects the regression and anchors the next retry to the original error.
 
-When `--file` is given, lacuna skips the full suite and runs only the target file — much faster for iterating on a single broken test. Without `--file`, the failing-files list is cached for 30 minutes. After a fix run, the cache is updated to contain only the files that are still failing — so re-running `lacuna fix` immediately picks up exactly where the last run left off. Once all files are fixed, the cache is cleared so the next run does a clean suite scan.
+When `--file` is given, lacuna skips the full suite and runs only the target file. Without `--file`, the failing-files list is cached for 30 minutes. After a fix run, the cache is updated to contain only the still-failing files — so re-running immediately picks up where the last run left off.
 
 ### `lacuna run`
 Runs your test suite and reports coverage. No AI involved.
@@ -142,9 +152,10 @@ Created by `lacuna init`. All fields are optional with sensible defaults.
 
 ```json
 {
-  "provider": "anthropic",
-  "model": "claude-sonnet-4-6",
-  "apiKeyEnv": "ANTHROPIC_API_KEY",
+  "provider": "openai-compatible",
+  "model": "deepseek-chat",
+  "baseURL": "https://api.deepseek.com/v1",
+  "apiKeyEnv": "DEEPSEEK_API_KEY",
   "testRunner": "jest",
   "coverageFormat": "lcov",
   "coverageDir": "coverage",
@@ -159,14 +170,14 @@ Created by `lacuna init`. All fields are optional with sensible defaults.
 
 | Field | Default | Description |
 |---|---|---|
-| `provider` | `anthropic` | `anthropic` or `openai-compatible` |
-| `model` | `claude-sonnet-4-6` | Model name |
-| `apiKeyEnv` | `ANTHROPIC_API_KEY` | Env var holding your API key |
-| `baseURL` | — | Required for `openai-compatible` provider |
+| `provider` | `openai-compatible` | `anthropic` or `openai-compatible` |
+| `model` | `deepseek-chat` | Model name |
+| `apiKeyEnv` | `DEEPSEEK_API_KEY` | Env var holding your API key |
+| `baseURL` | `https://api.deepseek.com/v1` | API base URL — required for `openai-compatible` provider |
 | `testRunner` | auto-detect | `jest` \| `vitest` \| `pytest` \| `mocha` \| `go-test` |
 | `coverageFormat` | `lcov` | `lcov` \| `json-summary` |
 | `coverageDir` | `coverage` | Where your test runner writes coverage |
-| `sourceDir` | `src` | Root directory of source files — set during `lacuna init` |
+| `sourceDir` | `"src"` | Source directory to scan. Accepts a string or an array — `["src", "lib", "utils"]` scans multiple directories |
 | `threshold` | `80` | Minimum line coverage % to pass |
 | `maxIterations` | `3` | How many times to retry a failing generated test |
 | `coverageTimeout` | `300` | Seconds before the test suite is killed (prevents hanging on open handles) |
@@ -183,9 +194,10 @@ Lacuna works with any AI model — local or cloud.
 
 | Preset | Model | API key env | Notes |
 |---|---|---|---|
-| Claude (default) | `claude-sonnet-4-6` | `ANTHROPIC_API_KEY` | Best for code |
+| **DeepSeek (default)** | `deepseek-chat` | `DEEPSEEK_API_KEY` | Best value — fast, cheap, no rate-limit issues |
+| DeepSeek R1 | `deepseek-reasoner` | `DEEPSEEK_API_KEY` | Reasoning model |
+| Claude Sonnet | `claude-sonnet-4-6` | `ANTHROPIC_API_KEY` | High quality |
 | Claude Opus | `claude-opus-4-7` | `ANTHROPIC_API_KEY` | Most capable |
-| DeepSeek | `deepseek-chat` | `DEEPSEEK_API_KEY` | Very cost-effective |
 | DeepSeek R1 | `deepseek-reasoner` | `DEEPSEEK_API_KEY` | Reasoning model |
 | GPT-4o | `gpt-4o` | `OPENAI_API_KEY` | |
 | Groq | `llama-3.3-70b-versatile` | `GROQ_API_KEY` | Fast, free tier |
@@ -255,7 +267,25 @@ beforeEach(() => {
 lacuna generate
 ```
 
-Every generated test will import from `src/test/mocks.ts` instead of creating its own `vi.fn()` calls. If a test needs a mock that doesn't exist yet, Claude will add it to the mocks file and import it — keeping everything centralized.
+Every generated test will import from `src/test/mocks.ts` instead of creating its own `vi.fn()` calls. If a test needs a mock that doesn't exist yet, lacuna will add it to the mocks file and import it — keeping everything centralized.
+
+### How lacuna reads and updates the shared mock file
+
+Lacuna parses your mock file before every prompt and builds a **mock inventory** — a structured table of every `vi.mock()` call, its line number, and all the keys it exports:
+
+```
+'react-router-dom'  → useNavigate, useParams
+'axios'             → get, post, put, delete
+```
+
+This inventory is injected into every prompt so the AI knows exactly which modules are already mocked and what they export. When the AI needs to add a new export (e.g. a new API client method), it updates the **existing** `vi.mock()` block with the full list of old + new exports — never appending a second block for the same module.
+
+**Structure rules the AI follows:**
+- Module already in inventory → update the existing block (never a duplicate)
+- New mock variable → declare near same-domain exports, reset in existing `beforeEach`
+- New module → append at end before the final `beforeEach`
+
+For fix prompts, the mock file is **compressed** before sending (multi-line React component stub bodies are collapsed to `vi.fn()`) to reduce token cost while preserving all `vi.mock()` blocks and factory functions. For generate prompts, only the inventory and exports list are sent — the raw file is skipped entirely since the AI only needs to know what to import, not re-read every implementation.
 
 ---
 
@@ -298,8 +328,8 @@ jobs:
         with:
           threshold: 80
           workers: 2                    # parallel workers — increase for larger repos
-          model: claude-sonnet-4-6      # any lacuna preset or model name
-          anthropic-api-key: ${{ secrets.ANTHROPIC_API_KEY }}
+          model: deepseek               # default — cost-effective, no rate-limit issues
+          deepseek-api-key: ${{ secrets.DEEPSEEK_API_KEY }}
 
       # Runs even when lacuna exits with code 1 (below threshold) so generated
       # tests are never lost. Skips the commit if nothing was written.

package/dist/agent/fix-loop.d.ts

@@ -9,11 +9,16 @@ export interface FixOptions {
     targetFile?: string;
     workers?: number;
     fresh?: boolean;
+    regenerateOnFailure?: boolean;
+    fixPolluters?: boolean;
     log: (msg: string) => void;
 }
 export interface FixResult {
     filesProcessed: number;
     filesFixed: number;
+    filesAlreadyPassing: number;
+    pollutersFixed: number;
+    victimsRegenerated: number;
     errors: string[];
 }
 export declare function runFixLoop(options: FixOptions): Promise<FixResult>;

package/dist/agent/fix-loop.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fix-loop.d.ts","sourceRoot":"","sources":["../../src/agent/fix-loop.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAA;AACpD,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAA;AAe7D,MAAM,WAAW,UAAU;IACzB,MAAM,EAAE,YAAY,CAAA;IACpB,GAAG,EAAE,mBAAmB,CAAA;IACxB,GAAG,EAAE,MAAM,CAAA;IACX,MAAM,EAAE,OAAO,CAAA;IACf,OAAO,EAAE,OAAO,CAAA;IAChB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,OAAO,CAAC,EAAE,MAAM,CAAA;IAChB,KAAK,CAAC,EAAE,OAAO,CAAA;IACf,GAAG,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAA;CAC3B;AAoCD,MAAM,WAAW,SAAS;IACxB,cAAc,EAAE,MAAM,CAAA;IACtB,UAAU,EAAE,MAAM,CAAA;IAClB,MAAM,EAAE,MAAM,EAAE,CAAA;CACjB;AA6UD,wBAAsB,UAAU,CAAC,OAAO,EAAE,UAAU,GAAG,OAAO,CAAC,SAAS,CAAC,CAgIxE"}
+{"version":3,"file":"fix-loop.d.ts","sourceRoot":"","sources":["../../src/agent/fix-loop.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAA;AACpD,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAA;AAiB7D,MAAM,WAAW,UAAU;IACzB,MAAM,EAAE,YAAY,CAAA;IACpB,GAAG,EAAE,mBAAmB,CAAA;IACxB,GAAG,EAAE,MAAM,CAAA;IACX,MAAM,EAAE,OAAO,CAAA;IACf,OAAO,EAAE,OAAO,CAAA;IAChB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,OAAO,CAAC,EAAE,MAAM,CAAA;IAChB,KAAK,CAAC,EAAE,OAAO,CAAA;IACf,mBAAmB,CAAC,EAAE,OAAO,CAAA;IAC7B,YAAY,CAAC,EAAE,OAAO,CAAA;IACtB,GAAG,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAA;CAC3B;AAoCD,MAAM,WAAW,SAAS;IACxB,cAAc,EAAE,MAAM,CAAA;IACtB,UAAU,EAAE,MAAM,CAAA;IAClB,mBAAmB,EAAE,MAAM,CAAA;IAC3B,cAAc,EAAE,MAAM,CAAA;IACtB,kBAAkB,EAAE,MAAM,CAAA;IAC1B,MAAM,EAAE,MAAM,EAAE,CAAA;CACjB;AA6nBD,wBAAsB,UAAU,CAAC,OAAO,EAAE,UAAU,GAAG,OAAO,CAAC,SAAS,CAAC,CAwJxE"}