@davesheffer/hunch 1.20.0-rc.1 → 1.20.0-rc.2

package/dist/synthesis/synthesize.js

@@ -121,10 +121,15 @@ export async function syncCommit(store, root, sha, opts = {}) {
     // grounding. No-ops when no assistant CLI is available; never raises trust (dec_9a2f2fe72a).
     if (wantVerify)
         draft = await verifyDecisionSafe(await selectVerifier({ root }), input, draft);
+    // The draft's own drafter, not merely the one selected — draftDecisionSafe sets
+    // fellBackTo when the selected provider threw and it silently substituted the
+    // deterministic heuristic. Every telemetry/return below must report THIS, since
+    // backfill's LLM-vs-heuristic tally reads it directly off the return value.
+    const actualProvider = draft.fellBackTo ?? provider.name;
     // Advisory synthesis telemetry for `hunch review` — which provider ran, how many drafts
     // were reconciled, their agreement, and the verifier's grounding. Rides in `evidence`
     // (no schema change → respects forward-migration invariant con_947c578b2c).
-    const synthBits = [`provider=${provider.name}`];
+    const synthBits = [`provider=${actualProvider}`];
     if (draft.samples)
         synthBits.push(`samples=${draft.samples}`);
     if (draft.agreement != null)
@@ -215,7 +220,7 @@ export async function syncCommit(store, root, sha, opts = {}) {
         store.putPrivate("decisions", decision);
     else
         store.json.put("decisions", decision);
-    return { status: "written", decision, provider: provider.name };
+    return { status: "written", decision, provider: actualProvider };
 }
 function putBugInHome(store, bug, home) {
     return home === "private" ? store.putPrivate("bugs", bug) : store.json.put("bugs", bug);
@@ -244,6 +249,9 @@ export async function recordFailure(store, root, failure, opts = {}) {
         suspects: suspects.map((s) => `${s.name} @ ${s.file}`),
     };
     const draft = await draftBugSafe(provider, input);
+    // The draft's own drafter, not merely the one selected — see the identical
+    // comment in syncCommit above.
+    const actualProvider = draft.fellBackTo ?? provider.name;
     // Seed the id from the test id (stable), not the LLM title — one bug per test.
     const id = bugId(failure.test);
     // recurrence = a DIFFERENT prior bug with a similar symptom (not this same one).
@@ -284,7 +292,7 @@ export async function recordFailure(store, root, failure, opts = {}) {
         putBugInHome(store, bug, home);
     }
     raiseFragility(store, affectedFiles, home);
-    return { status: "written", bug, constraint, provider: provider.name, touchedHomes: [home] };
+    return { status: "written", bug, constraint, provider: actualProvider, touchedHomes: [home] };
 }
 /** Orchestrate one `hunch test` run into graph writes: capture each failing test
  *  as a Bug (recordFailure → suspects / recurrence / Constraint promotion), and
@@ -506,19 +514,26 @@ export async function draftDecisionSafe(provider, input) {
     try {
         return await provider.draftDecision(input);
     }
-    catch {
+    catch (error) {
         // A provider failure (network, bad creds, CLI crash, unparseable output) must
         // never abort the learning loop — fall back to the deterministic heuristic
         // draft, which is honestly labeled ("inferred") rather than a hollow llm_draft.
-        return new DeterministicProvider().draftDecision(input);
+        // fellBackTo/fallbackReason tell the caller what ACTUALLY drafted this, not
+        // merely what was selected — see syncCommit, which must report this instead
+        // of the originally-selected provider's name.
+        const draft = await new DeterministicProvider().draftDecision(input);
+        return { ...draft, fellBackTo: "deterministic", fallbackReason: error instanceof Error ? error.message : String(error) };
     }
 }
 export async function draftBugSafe(provider, input) {
     try {
         return await provider.draftBug(input);
     }
-    catch {
-        return new DeterministicProvider().draftBug(input);
+    catch (error) {
+        // See draftDecisionSafe above — the same fallback contract applies to bugs,
+        // and recordFailure must report the actual (fallback) provider likewise.
+        const draft = await new DeterministicProvider().draftBug(input);
+        return { ...draft, fellBackTo: "deterministic", fallbackReason: error instanceof Error ? error.message : String(error) };
     }
 }
 //# sourceMappingURL=synthesize.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@davesheffer/hunch",
-  "version": "1.20.0-rc.1",
+  "version": "1.20.0-rc.2",
   "mcpName": "io.github.davesheffer/hunch",
   "license": "Apache-2.0",
   "author": "Dave Sheffer <dave.sheffer1@gmail.com>",

package/server.json

@@ -7,13 +7,13 @@
     "source": "github"
   },
   "websiteUrl": "https://hunch-pi.vercel.app",
-  "version": "1.20.0-rc.1",
+  "version": "1.20.0-rc.2",
   "packages": [
     {
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "@davesheffer/hunch",
-      "version": "1.20.0-rc.1",
+      "version": "1.20.0-rc.2",
       "runtimeHint": "npx",
       "packageArguments": [
         {