@codacy/verity-cli 0.28.1-experimental.dbd87b1 → 0.28.1-experimental.e910abc

package/bin/verity.js

@@ -13579,6 +13579,13 @@ function reduce(state, events, now) {
         });
         break;
       }
+      case "goal_delivered": {
+        const g = state.goal.find((x) => x.status === "active");
+        if (!g) break;
+        if (g.delivered) break;
+        g.delivered = { at: ev.at, seq: ev.seq, summary: ev.summary };
+        break;
+      }
       case "authored": {
         authoredEvents++;
         const e = byPath.get(ev.path) ?? {
@@ -13693,6 +13700,11 @@ function reduce(state, events, now) {
       }
       case "verdict": {
         state.meta.last_verdict_seq = ev.seq;
+        if (ev.intent_sig) {
+          state.meta.intent_repeat = state.meta.intent_repeat && state.meta.intent_repeat.sig === ev.intent_sig ? { sig: ev.intent_sig, consecutive: state.meta.intent_repeat.consecutive + 1 } : { sig: ev.intent_sig, consecutive: 1 };
+        } else {
+          state.meta.intent_repeat = void 0;
+        }
         state.meta.watermark = {
           sha: ev.head_sha,
           reviewed_hash: ev.watermark_sha,
@@ -13778,7 +13790,12 @@ function compactState(s) {
   const ms = (iso) => Date.parse(iso) || 0;
   return {
     v: 1,
-    g: s.goal.map((g) => [g.seq, ms(g.at), g.hash, g.superseded_by, g.text ?? 0, g.text_len ?? 0, g.source ?? 0, g.status ?? 0, g.repeats ?? 0, g.truncated ? 1 : 0, g.collapsed ? 1 : 0]),
+    // ⚠ APPEND-ONLY POSITIONALLY. Indices 11-13 carry `delivered`; a cache row
+    // written before it existed has length 11 and expands with `delivered`
+    // absent, which is the correct reading of "nothing had been delivered yet".
+    // Inserting rather than appending would silently re-interpret every existing
+    // cached row.
+    g: s.goal.map((g) => [g.seq, ms(g.at), g.hash, g.superseded_by, g.text ?? 0, g.text_len ?? 0, g.source ?? 0, g.status ?? 0, g.repeats ?? 0, g.truncated ? 1 : 0, g.collapsed ? 1 : 0, g.delivered ? ms(g.delivered.at) : 0, g.delivered?.seq ?? 0, g.delivered?.summary ?? 0]),
     a: s.authored?.map((a) => [a.path, a.origin, a.edits, a.hunks, a.adds, a.dels, a.hash_now, a.hash_at_last_verdict, a.first_seq, a.last_seq]) ?? null,
     n: s.not_mine?.map((n) => [n.path, n.reason, ms(n.at), n.head_sha]) ?? null,
     t: s.statements.map((x) => [x.anchor_key, x.file, x.line, x.pattern_id, x.title_hash, x.register, x.line_sha, x.said_at_seq, ms(x.said_at), x.outcome, x.outcome_at ? ms(x.outcome_at) : 0, x.repeats, x.carried ? 1 : 0]),
@@ -13824,7 +13841,14 @@ function expandState(raw) {
         ...x[7] ? { status: x[7] } : {},
         ...x[8] ? { repeats: x[8] } : {},
         ...x[9] ? { truncated: true } : {},
-        ...x[10] ? { collapsed: true } : {}
+        ...x[10] ? { collapsed: true } : {},
+        ...x[11] ? {
+          delivered: {
+            at: iso(x[11]),
+            seq: x[12] ?? 0,
+            summary: x[13] || ""
+          }
+        } : {}
       })),
       authored: decodedAuthored,
       // The cache stores the BOUNDED list, so this is the bounded list too. That
@@ -14038,7 +14062,8 @@ function projectMemory(state, opts) {
       seq: active.seq,
       superseded,
       truncated: active.truncated === true,
-      collapsed: state.meta.collapsed.goal ?? 0
+      collapsed: state.meta.collapsed.goal ?? 0,
+      ...active.delivered && { delivered: active.delivered }
     };
   }
   const ageOf = (seq, carried) => carried ? "carried" : seq > lastVerdictSeq ? "this_turn" : "this_session";
@@ -14466,6 +14491,10 @@ function toStatus(n) {
 }
 function recordVerdict(d, v) {
   const root = repoRoot();
+  if (v.intent?.verdict === "aligned") {
+    const summary = (v.intent.implemented ?? "").trim().slice(0, 400);
+    if (summary) appendEvent(d, { k: "goal_delivered", summary });
+  }
   const lines = /* @__PURE__ */ new Map();
   for (const f of v.findings) {
     if (!f.file || typeof f.line !== "number" || !f.pattern_id) continue;
@@ -14499,9 +14528,15 @@ function recordVerdict(d, v) {
     head_sha: getCurrentCommit(),
     watermark_sha: v.watermarkSha,
     branch: v.branch,
-    decision: v.decision
+    decision: v.decision,
+    ...intentSignature(v.intent) && { intent_sig: intentSignature(v.intent) }
   });
 }
+function intentSignature(intent) {
+  if (!intent?.verdict) return null;
+  if (intent.verdict !== "misaligned" && intent.verdict !== "partial") return null;
+  return `${intent.verdict}:${lineSha(intent.gaps?.[0] ?? "")}`;
+}
 function toRegister(severity) {
   switch (severity) {
     case "critical":
@@ -16287,7 +16322,7 @@ function resolveTaskContext(opts) {
 // src/lib/cli-version.ts
 function cliVersion() {
   try {
-    return true ? "0.28.1-experimental.dbd87b1" : "dev";
+    return true ? "0.28.1-experimental.e910abc" : "dev";
   } catch {
     return "dev";
   }
@@ -16838,9 +16873,16 @@ function buildAgentContext(input) {
   }
   if (input.intentVerdict === "misaligned" || input.intentVerdict === "partial") {
     const gap = input.intentGaps?.[0];
-    lines.push(
-      `- This does not look like the change that was asked for` + (gap ? `: ${gap}` : ".") + " Not blocking \u2014 but check it against what you were asked to do."
-    );
+    const repeat = input.intentRepeat ?? 0;
+    if (repeat > 0) {
+      lines.push(
+        `- Same intent flag as the last ${repeat === 1 ? "turn" : `${repeat} turns`}, on a goal that has not changed. Nothing new here \u2014 do not relay or re-explain it again; either act on it or carry on.`
+      );
+    } else {
+      lines.push(
+        `- This does not look like the change that was asked for` + (gap ? `: ${gap}` : ".") + " Not blocking \u2014 but check it against what you were asked to do."
+      );
+    }
   }
   const agentFindings = (input.findings ?? []).filter((f) => f.scope !== "pre-existing");
   for (const f of agentFindings) {
@@ -17714,10 +17756,11 @@ async function readStopHookStdin() {
     return empty;
   }
 }
-function agentContextFor(response) {
+function agentContextFor(response, intentRepeat = 0) {
   const metadata = response.metadata ?? {};
   const intent = response.intent_alignment ?? {};
   return buildAgentContext({
+    intentRepeat,
     gateDecision: String(response.gate_decision ?? ""),
     findings: response.findings ?? [],
     pendingItems: response.pending_items ?? [],
@@ -18334,6 +18377,7 @@ async function runAnalyze(opts, globals) {
   const sentPaths = codeDelta.files.map((f) => f.path);
   const watermarkHash = sentPaths.length > 0 ? computeContentHash(sentPaths) : contentHash;
   const watermarkIsPartial = !!codeDelta.truncated;
+  let intentRepeatCount = 0;
   if (memorySession) {
     try {
       recordVerdict(memorySession.d, {
@@ -18347,8 +18391,10 @@ async function runAnalyze(opts, globals) {
           pattern_id: f.pattern_id ?? f.rule_id,
           title: f.title,
           severity: f.severity
-        })) ?? []
+        })) ?? [],
+        intent: response.intent_alignment ?? null
       });
+      intentRepeatCount = Math.max(0, (foldDossier(memorySession.d).meta.intent_repeat?.consecutive ?? 1) - 1);
     } catch {
     }
   }
@@ -18538,7 +18584,7 @@ ${YELLOW}${grantNudge.trim()}${NC}
       if (autoSeedNotice) userSummary = `${autoSeedNotice} ${userSummary}`;
       userSummary += loginNudge + grantNudge;
       printJsonCompact(
-        buildHookOutput("PASS", userSummary, agentContextFor(response))
+        buildHookOutput("PASS", userSummary, agentContextFor(response, intentRepeatCount))
       );
       process.exit(0);
       break;
@@ -18552,7 +18598,7 @@ ${YELLOW}${grantNudge.trim()}${NC}
       if (autoSeedNotice) userSummary = `${autoSeedNotice} ${userSummary}`;
       userSummary += loginNudge + grantNudge;
       printJsonCompact(
-        buildHookOutput("WARN", userSummary, agentContextFor(response))
+        buildHookOutput("WARN", userSummary, agentContextFor(response, intentRepeatCount))
       );
       process.exit(0);
       break;
@@ -20354,7 +20400,7 @@ function registerTelemetryCommands(program2) {
 }
 
 // src/cli.ts
-program.name("verity").description("CLI for Verity quality gate service").version("0.28.1-experimental.dbd87b1").option("--token <token>", "Override authentication token").option("--service-url <url>", "Override service URL").option("--verbose", "Log HTTP requests/responses to stderr").hook("preAction", async () => {
+program.name("verity").description("CLI for Verity quality gate service").version("0.28.1-experimental.e910abc").option("--token <token>", "Override authentication token").option("--service-url <url>", "Override service URL").option("--verbose", "Log HTTP requests/responses to stderr").hook("preAction", async () => {
   try {
     await foldLegacyLocalCredential();
   } catch {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/verity-cli",
-  "version": "0.28.1-experimental.dbd87b1",
+  "version": "0.28.1-experimental.e910abc",
   "description": "CLI for Verity quality gate service",
   "homepage": "https://verity.md",
   "bugs": {