@davesheffer/hunch 1.47.1 → 1.48.0

package/dist/synthesis/synthesize.js

@@ -1,10 +1,11 @@
 import { commitMeta, commitDiff, headSha, currentBranch } from "../extractors/git.js";
 import { analyzeDiff } from "../extractors/diff.js";
-import { selectProvider, selectEnsemble, selectVerifier, verifyDecisionSafe, DeterministicProvider } from "./provider.js";
+import { selectProvider, selectEnsemble, selectVerifier, verifyDecisionSafe, DeterministicProvider, UnusableOutputError } from "./provider.js";
 import { decisionId, bugId, constraintId } from "../core/ids.js";
 import { commitCoveredBy } from "../core/dupdetect.js";
 import { pathMatchesGlob } from "../core/glob.js";
 import { draftTripwires, knownRepoDeps } from "./tripwires.js";
+import { prContextFor } from "./prContext.js";
 import { isSubstantive } from "../extractors/languages.js";
 // Subjects that signal mechanical churn. `chore(deps):` is matched as a literal
 // prefix on its own: a word-boundary check after the closing parenthesis would
@@ -124,6 +125,15 @@ export async function syncCommit(store, root, sha, opts = {}) {
                 ? await selectProvider({ root })
                 : new DeterministicProvider();
     const input = { subject: meta.subject, body: meta.body, files: substantiveFiles, diff, analysis };
+    // The merged PR's description often holds the why a squash commit drops (#562). Fetch
+    // it only when a model will read it: never for a local-only capture (no PR text goes
+    // to any provider) and not for the free deterministic draft.
+    // The review thread feeds only the Critic-verified path (gated by a noise audit).
+    if (!localOnly && provider.name !== "deterministic") {
+        const pr = await (opts.prContext ?? prContextFor)(root, { sha: meta.sha, subject: meta.subject }, { thread: wantVerify });
+        if (pr)
+            input.pr = pr;
+    }
     let draft = await draftDecisionSafe(provider, input);
     // The Critic pass: audit the draft against the commit, PRUNE unsupported alternatives
     // (BEFORE they scaffold tripwires below) and consequences, and lower confidence on weak
@@ -137,6 +147,10 @@ export async function syncCommit(store, root, sha, opts = {}) {
     // were reconciled, their agreement, and the verifier's grounding. Rides in `evidence`
     // (no schema change → respects forward-migration invariant con_947c578b2c).
     const synthBits = [`provider=${actualProvider}`];
+    if (input.pr)
+        synthBits.push(`pr=#${input.pr.number}${input.pr.thread ? "+thread" : ""}`);
+    if (draft.fallbackKind)
+        synthBits.push(`fallback=${draft.fallbackKind}`);
     if (draft.samples)
         synthBits.push(`samples=${draft.samples}`);
     if (draft.agreement != null)
@@ -227,7 +241,7 @@ export async function syncCommit(store, root, sha, opts = {}) {
         store.putPrivate("decisions", decision);
     else
         store.json.put("decisions", decision);
-    return { status: "written", decision, provider: actualProvider };
+    return { status: "written", decision, provider: actualProvider, ...(draft.fallbackReason ? { fallbackReason: draft.fallbackReason } : {}) };
 }
 function putBugInHome(store, bug, home) {
     return home === "private" ? store.putPrivate("bugs", bug) : store.json.put("bugs", bug);
@@ -516,9 +530,35 @@ function recentDiffFor(root) {
     const head = headSha(root);
     return head ? commitDiff(head, root, 12_000) : "";
 }
+/** Extra samples when a provider answered but its output held no usable record
+ *  (issue #561): a model that echoed the schema usually answers properly on the
+ *  next sample. Transport failures (timeout, crash, auth) are not retried, since a
+ *  second hang would only double the wait before the same fallback. */
+export const UNUSABLE_OUTPUT_RETRIES = 1;
+async function sampleUntilUsable(draft) {
+    for (let attempt = 0;; attempt++) {
+        try {
+            return await draft();
+        }
+        catch (err) {
+            if (!(err instanceof UnusableOutputError) || attempt >= UNUSABLE_OUTPUT_RETRIES)
+                throw err;
+        }
+    }
+}
+/** Short, print-safe label for the `synth:` evidence line, so a fallback from the
+ *  quiet post-commit hook still leaves a trace `hunch review` can show. */
+function fallbackKindOf(err) {
+    return err instanceof UnusableOutputError ? err.kind : "provider-error";
+}
+function fallbackReasonOf(err) {
+    if (err instanceof UnusableOutputError)
+        return `${err.message}, ${UNUSABLE_OUTPUT_RETRIES + 1} attempts`;
+    return err instanceof Error ? err.message : String(err);
+}
 export async function draftDecisionSafe(provider, input) {
     try {
-        return await provider.draftDecision(input);
+        return await sampleUntilUsable(() => provider.draftDecision(input));
     }
     catch (err) {
         // A provider failure (network, bad creds, CLI crash, unparseable output) must
@@ -527,17 +567,17 @@ export async function draftDecisionSafe(provider, input) {
         // Tell the caller a fallback happened (and why) so it reports the provider that
         // actually drafted the record rather than the one that was selected.
         const fallback = await new DeterministicProvider().draftDecision(input);
-        return { ...fallback, fellBackTo: "deterministic", fallbackReason: err instanceof Error ? err.message : String(err) };
+        return { ...fallback, fellBackTo: "deterministic", fallbackReason: fallbackReasonOf(err), fallbackKind: fallbackKindOf(err) };
     }
 }
 export async function draftBugSafe(provider, input) {
     try {
-        return await provider.draftBug(input);
+        return await sampleUntilUsable(() => provider.draftBug(input));
     }
     catch (err) {
         // Same contract as draftDecisionSafe: surface the fallback and its cause.
         const fallback = await new DeterministicProvider().draftBug(input);
-        return { ...fallback, fellBackTo: "deterministic", fallbackReason: err instanceof Error ? err.message : String(err) };
+        return { ...fallback, fellBackTo: "deterministic", fallbackReason: fallbackReasonOf(err) };
     }
 }
 //# sourceMappingURL=synthesize.js.map

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@davesheffer/hunch",
-  "version": "1.47.1",
+  "version": "1.48.0",
   "mcpName": "io.github.davesheffer/hunch",
-  "license": "Apache-2.0",
+  "license": "FSL-1.1-ALv2",
   "author": "Dave Sheffer <dave.sheffer1@gmail.com>",
   "description": "A shared record for AI agents: decisions, completed work, and commitments, with sources. Git-native engineering memory and a self-hosted state server.",
   "homepage": "https://www.hunchmemory.com",

package/server.json

@@ -7,13 +7,13 @@
     "source": "github"
   },
   "websiteUrl": "https://www.hunchmemory.com",
-  "version": "1.47.1",
+  "version": "1.48.0",
   "packages": [
     {
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "@davesheffer/hunch",
-      "version": "1.47.1",
+      "version": "1.48.0",
       "runtimeHint": "npx",
       "packageArguments": [
         {