fix(migrations): scope v0.32.2 dirty-check to targeted sources; surface failed phase detail

- phaseBFenceFacts now queries legacy rows FIRST and dirty-checks only
  the source_ids it will actually write into. Zero fenceable rows (or
  rows scoped to clean sources) no longer fail on an unrelated dirty
  source. Targeted-dirty-source refusal unchanged. Fixes #927.
- apply-migrations now prints each failed phase's name + detail to
  stderr alongside 'reported status=failed', instead of burying the
  actionable message in the ledger. Fixes #921.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

src/commands/apply-migrations.ts

@@ -438,6 +438,13 @@ export async function runApplyMigrations(args: string[]): Promise<void> {
       const result = await m.orchestrator(orchestratorOptsFrom(cli));
       if (result.status === 'failed') {
         console.error(`Migration v${m.version} reported status=failed.`);
+        // Surface each failed phase's detail — the ledger records it, but
+        // the operator needs it on stderr to act (#921).
+        for (const p of result.phases) {
+          if (p.status === 'failed') {
+            console.error(`  phase ${p.name}: ${p.detail ?? '(no detail)'}`);
+          }
+        }
         // Record the attempt as 'partial' (not 'complete') so the cap counts
         // it. Don't let a failed orchestrator look like it never ran.
         try {

src/commands/migrations/v0_32_2.ts

@@ -186,17 +186,6 @@ async function phaseBFenceFacts(
     const localPathById = new Map<string, string | null>();
     for (const s of sources) localPathById.set(s.id, s.local_path);
 
-    // Dirty-tree refusal: check every source's local_path before writing.
-    for (const [id, localPath] of localPathById) {
-      if (localPath && isLocalPathDirty(localPath)) {
-        return {
-          name: 'fence_facts',
-          status: 'failed',
-          detail: `source "${id}" has uncommitted changes in ${localPath}. Commit or stash, then re-run.`,
-        };
-      }
-    }
-
     // Walk legacy rows in (source_id, entity_slug) groups for per-page
     // atomic writes.
     const legacy = await engine.executeRaw<LegacyFactRow>(
@@ -235,6 +224,21 @@ async function phaseBFenceFacts(
       groups.set(key, list);
     }
 
+    // Dirty-tree refusal: check ONLY the sources we are about to write
+    // into. A dirty tree in an unrelated source (or zero fenceable rows
+    // at all) must not block a no-op or a targeted backfill (#927).
+    const targetSourceIds = new Set([...groups.keys()].map(k => k.split('\0')[0]));
+    for (const id of targetSourceIds) {
+      const localPath = localPathById.get(id);
+      if (localPath && isLocalPathDirty(localPath)) {
+        return {
+          name: 'fence_facts',
+          status: 'failed',
+          detail: `source "${id}" has uncommitted changes in ${localPath}. Commit or stash, then re-run.`,
+        };
+      }
+    }
+
     for (const [key, group] of groups) {
       const [sourceId, entitySlug] = key.split('\0');
       const localPath = localPathById.get(sourceId)!;

src/core/ai/recipes/litellm-proxy.ts

@@ -56,36 +56,6 @@ export const litellmProxy: Recipe = {
       cost_per_1m_output_usd: undefined,
       price_last_verified: '2026-06-14',
     },
-    // LiteLLM normalizes Cohere / Voyage / Jina / etc. rerank backends to the
-    // same wire shape gbrain's gateway.rerank() already speaks (the
-    // ZeroEntropy/llama.cpp contract):
-    //   { model, query, documents, top_n } → { results: [{ index, relevance_score }] }
-    // So any rerank model the user registers in their LiteLLM config is
-    // reachable via `gbrain config set search.reranker.model litellm:<model>`
-    // with no request/response adapter — same as embeddings ride the proxy.
-    reranker: {
-      models: [], // user-provided; whatever rerank models the proxy serves
-      // No canonical default — the proxy defines its own model ids. The user
-      // sets search.reranker.model explicitly (mirrors the embedding
-      // touchpoint's user_provided_models contract).
-      default_model: '',
-      // The proxied backend bills (Cohere/Voyage/…); pricing-unknown is the
-      // honest state — same stance as this recipe's embedding/chat
-      // touchpoints and budget-tracker's deliberate litellm exclusion from
-      // the free-provider sets.
-      cost_per_1m_tokens_usd: undefined,
-      price_last_verified: '2026-06-27',
-      max_payload_bytes: 5_000_000,
-      // LEAF path only (matches llama-server-reranker's convention). LiteLLM
-      // serves both `/rerank` and `/v1/rerank`, and LITELLM_BASE_URL may be
-      // set with or without the `/v1` suffix (the setup_hint allows both), so
-      // the leaf form yields a valid route either way:
-      //   http://localhost:4000    + /rerank → /rerank        ✓
-      //   http://localhost:4000/v1 + /rerank → /v1/rerank     ✓
-      // Pinning '/v1/rerank' here would double to /v1/v1/rerank → 404 on
-      // /v1-suffixed bases.
-      path: '/rerank',
-    },
   },
-  setup_hint: 'Run LiteLLM (https://docs.litellm.ai) in front of any provider; set LITELLM_BASE_URL (include the /v1 suffix if your proxy serves the OpenAI route there, e.g. http://localhost:4000/v1) + pass --embedding-model litellm:<model> and --embedding-dimensions <N>. For rerank: register a rerank model in LiteLLM and set search.reranker.model litellm:<model-name>.',
+  setup_hint: 'Run LiteLLM (https://docs.litellm.ai) in front of any provider; set LITELLM_BASE_URL (include the /v1 suffix if your proxy serves the OpenAI route there, e.g. http://localhost:4000/v1) + pass --embedding-model litellm:<model> and --embedding-dimensions <N>.',
 };

test/ai/recipe-litellm-reranker.test.ts

@@ -1,88 +0,0 @@
-/**
- * litellm-proxy reranker touchpoint smoke.
- *
- * Sibling of recipe-llama-server-reranker.test.ts. Pins the reranker
- * touchpoint on the LiteLLM proxy recipe so:
- *  - the touchpoint exists with the LEAF '/rerank' path (LiteLLM serves both
- *    /rerank and /v1/rerank, so the leaf form is valid whether or not the
- *    user's LITELLM_BASE_URL carries the /v1 suffix the setup_hint allows)
- *  - a /v1-suffixed base URL does NOT produce /v1/v1/rerank (the original
- *    community PR pinned '/v1/rerank' which 404s on /v1-suffixed bases)
- *  - models: [] (user-provided; proxy defines the model ids)
- *  - pricing stays undefined (proxy can front a paid provider — same honest
- *    pricing-unknown stance as the embedding/chat touchpoints)
- *
- * The gateway.rerank() URL tests drive the real URL builder via the stubbed
- * transport (same seam as test/ai/rerank.test.ts).
- */
-
-import { describe, expect, test, afterEach } from 'bun:test';
-import { getRecipe } from '../../src/core/ai/recipes/index.ts';
-import {
-  configureGateway,
-  resetGateway,
-  rerank,
-  __setRerankTransportForTests,
-} from '../../src/core/ai/gateway.ts';
-
-afterEach(() => {
-  __setRerankTransportForTests(null);
-  resetGateway();
-});
-
-describe('recipe: litellm reranker touchpoint', () => {
-  test('declares reranker touchpoint with leaf /rerank path', () => {
-    const r = getRecipe('litellm')!;
-    const tp = r.touchpoints.reranker;
-    expect(tp).toBeDefined();
-    expect(tp!.path).toBe('/rerank');
-    expect(tp!.max_payload_bytes).toBe(5_000_000);
-  });
-
-  test('reranker touchpoint uses empty models[] for user-provided model ids', () => {
-    const r = getRecipe('litellm')!;
-    expect(r.touchpoints.reranker!.models).toEqual([]);
-  });
-
-  test('pricing stays undefined — proxy can front a paid provider', () => {
-    const r = getRecipe('litellm')!;
-    expect(r.touchpoints.reranker!.cost_per_1m_tokens_usd).toBeUndefined();
-  });
-
-  test('setup_hint keeps the /v1-suffix guidance AND mentions rerank', () => {
-    const r = getRecipe('litellm')!;
-    expect(r.setup_hint).toMatch(/\/v1 suffix/);
-    expect(r.setup_hint).toMatch(/search\.reranker\.model litellm:/);
-  });
-});
-
-describe('gateway.rerank() URL via litellm recipe', () => {
-  async function capturedRerankUrl(baseUrl?: string): Promise<string> {
-    configureGateway({
-      reranker_model: 'litellm:my-reranker',
-      env: {},
-      ...(baseUrl ? { base_urls: { litellm: baseUrl } } : {}),
-    });
-    let capturedUrl = '';
-    __setRerankTransportForTests(async (url) => {
-      capturedUrl = url;
-      return new Response(
-        JSON.stringify({ results: [{ index: 0, relevance_score: 0.9 }] }),
-        { status: 200, headers: { 'content-type': 'application/json' } },
-      );
-    });
-    await rerank({ query: 'q', documents: ['d'] });
-    return capturedUrl;
-  }
-
-  test('default base (no /v1 suffix) → /rerank', async () => {
-    const url = await capturedRerankUrl();
-    expect(url).toBe('http://localhost:4000/rerank');
-  });
-
-  test('/v1-suffixed base → /v1/rerank, NOT /v1/v1/rerank', async () => {
-    const url = await capturedRerankUrl('http://localhost:4000/v1');
-    expect(url).toBe('http://localhost:4000/v1/rerank');
-    expect(url).not.toContain('/v1/v1/');
-  });
-});

test/apply-migrations.test.ts

@@ -180,3 +180,16 @@ describe('runApplyMigrations exit codes (v0.36.1.x #1062)', () => {
     expect(src).toMatch(/All migrations up to date[\s\S]{0,80}process\.exit\(0\)/);
   });
 });
+
+// #921: a failed orchestrator must print each failed phase's detail to
+// stderr — not just "reported status=failed" — so the operator can act
+// without digging through the ledger.
+describe('failed migration prints phase detail (#921)', () => {
+  test('runner loops result.phases and console.errors failed phase details', async () => {
+    const { readFileSync } = await import('fs');
+    const src = readFileSync('src/commands/apply-migrations.ts', 'utf8');
+    expect(src).toMatch(
+      /reported status=failed[\s\S]{0,400}for \(const p of result\.phases\)[\s\S]{0,200}p\.status === 'failed'[\s\S]{0,200}console\.error\([\s\S]{0,80}p\.name[\s\S]{0,80}p\.detail/,
+    );
+  });
+});

test/migrations-v0_32_2.test.ts

@@ -10,10 +10,11 @@
  * __setTestEngineOverride so we don't need a configured brain.
  */
 
-import { describe, test, expect, beforeAll, afterAll, beforeEach } from 'bun:test';
+import { describe, test, expect, beforeAll, afterAll, beforeEach, afterEach } from 'bun:test';
 import { mkdtempSync, rmSync, existsSync, readFileSync, writeFileSync, mkdirSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
+import { execFileSync } from 'node:child_process';
 
 import { PGLiteEngine } from '../src/core/pglite-engine.ts';
 import { v0_32_2, __setTestEngineOverride, __testing } from '../src/commands/migrations/v0_32_2.ts';
@@ -238,6 +239,52 @@ describe('phaseBFenceFacts — happy path backfill', () => {
   });
 });
 
+describe('phaseBFenceFacts — dirty-tree refusal scoping (#927)', () => {
+  let dirtyDir: string;
+
+  beforeEach(async () => {
+    // A second source whose local_path is a git repo with uncommitted changes.
+    dirtyDir = mkdtempSync(join(tmpdir(), 'mig-v0_32_2-dirty-'));
+    execFileSync('git', ['-C', dirtyDir, 'init', '-q']);
+    writeFileSync(join(dirtyDir, 'uncommitted.md'), 'dirty', 'utf-8');
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    await (engine as any).db.query(
+      `INSERT INTO sources (id, name, local_path) VALUES ('other', 'other', $1)`,
+      [dirtyDir],
+    );
+  });
+
+  afterEach(async () => {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    await (engine as any).db.query(`DELETE FROM sources WHERE id = 'other'`);
+    rmSync(dirtyDir, { recursive: true, force: true });
+  });
+
+  test('no legacy facts at all → complete, dirty unrelated source ignored', async () => {
+    const r = await __testing.phaseBFenceFacts(engine, OPTS);
+    expect(r.status).toBe('complete');
+    expect(r.detail).toContain('scanned=0');
+  });
+
+  test('facts scoped to a clean source fence despite dirty unrelated source', async () => {
+    await seedLegacyFact({ entity_slug: 'people/alice', fact: 'Founded Acme' });
+
+    const r = await __testing.phaseBFenceFacts(engine, OPTS);
+    expect(r.status).toBe('complete');
+    expect(r.detail).toContain('fenced=1');
+    expect(existsSync(join(brainDir, 'people/alice.md'))).toBe(true);
+  });
+
+  test('still refuses when the TARGETED source is dirty', async () => {
+    await seedLegacyFact({ entity_slug: 'people/alice', fact: 'F1', source_id: 'other' });
+
+    const r = await __testing.phaseBFenceFacts(engine, OPTS);
+    expect(r.status).toBe('failed');
+    expect(r.detail).toContain('"other"');
+    expect(r.detail).toContain('uncommitted changes');
+  });
+});
+
 describe('phaseCVerify', () => {
   test('returns complete when fence + DB row counts match', async () => {
     await seedLegacyFact({ entity_slug: 'people/alice', fact: 'F1' });