@codacy/verity-cli 0.28.1-experimental.89ee5db → 0.28.1-experimental.902dbd9

package/bin/verity.js

@@ -10386,7 +10386,7 @@ function projectPath(relativePath) {
   return (0, import_node_path.join)(repoRoot(), relativePath);
 }
 var MAX_DELTA_BYTES = 194560;
-var MAX_FILES = 20;
+var MAX_FILES = 40;
 var MAX_FILE_BYTES = 51200;
 var DEBOUNCE_SECONDS = 30;
 var MAX_SPEC_FILES = 10;
@@ -13654,6 +13654,79 @@ function shouldSkipForBareAck(input) {
   return input.canSeeTurnAuthorship;
 }
 
+// src/lib/pending-repeat.ts
+var STOP = /* @__PURE__ */ new Set([
+  "the",
+  "and",
+  "that",
+  "this",
+  "with",
+  "from",
+  "have",
+  "been",
+  "were",
+  "what",
+  "when",
+  "which",
+  "their",
+  "there",
+  "these",
+  "those",
+  "would",
+  "could",
+  "should",
+  "must",
+  "will",
+  "also",
+  "just",
+  "only",
+  "into",
+  "over",
+  "than",
+  "then",
+  "them",
+  "some",
+  "such",
+  "more",
+  "most",
+  "other",
+  "about",
+  "after",
+  "before",
+  "since",
+  "because",
+  "while",
+  "where",
+  "whether",
+  "ensure",
+  "confirm",
+  "verify",
+  "check"
+]);
+function pendingTokens(text) {
+  if (!text || typeof text !== "string") return [];
+  const out = /* @__PURE__ */ new Set();
+  for (const raw of text.toLowerCase().split(/[^a-z0-9]+/)) {
+    if (raw.length <= 3) continue;
+    if (STOP.has(raw)) continue;
+    out.add(raw);
+  }
+  return [...out].sort();
+}
+var REPEAT_THRESHOLD = 0.3;
+function overlapCoefficient(a, b) {
+  if (a.length === 0 || b.length === 0) return 0;
+  const setB = new Set(b);
+  let shared = 0;
+  for (const t of a) if (setB.has(t)) shared++;
+  return shared / Math.min(a.length, b.length);
+}
+function isRepeatOfAny(text, priorFingerprints, threshold = REPEAT_THRESHOLD) {
+  const tokens = pendingTokens(text);
+  if (tokens.length === 0) return false;
+  return priorFingerprints.some((prior) => overlapCoefficient(tokens, prior) >= threshold);
+}
+
 // src/lib/dossier.ts
 var import_node_fs9 = require("node:fs");
 var import_node_crypto4 = require("node:crypto");
@@ -13662,6 +13735,7 @@ var MAX_LINE_BYTES = 4096;
 var MAX_GOAL_CHARS = 2e3;
 var GOAL_KEEP = 8;
 var GOAL_TOTAL_CAP = 32;
+var RECENT_PENDING_CAP = 20;
 var HASH_WIDTH = 16;
 var AUTHORED_CAP = 300;
 var NOT_MINE_CAP = 300;
@@ -14030,6 +14104,10 @@ function reduce(state, events, now) {
           consecutiveIdle: ev.idle === false ? 0 : (state.meta.channel?.consecutiveIdle ?? 0) + 1
         };
         state.meta.last_adjudication = ev.intent_verdict ? { verdict: ev.intent_verdict, score: ev.intent_score ?? null, at: ev.at, decision: ev.decision } : void 0;
+        if (Array.isArray(ev.pending_sigs) && ev.pending_sigs.length > 0) {
+          const prior = state.meta.recent_pending_sigs ?? [];
+          state.meta.recent_pending_sigs = [...prior, ...ev.pending_sigs].slice(-RECENT_PENDING_CAP);
+        }
         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 {
@@ -14886,6 +14964,13 @@ function recordVerdict(d, v) {
     branch: v.branch,
     decision: v.decision,
     ...sig && { intent_sig: sig },
+    // Fingerprints of the pending items this verdict delivered, so the NEXT turn
+    // can tell a repeat from a new requirement. Only recorded when the channel
+    // actually spoke — a silenced turn delivered nothing, so nothing was "said
+    // before" and labelling the next turn's items as repeats would be a lie.
+    ...v.emitted === true && v.pendingTexts && v.pendingTexts.length > 0 && {
+      pending_sigs: v.pendingTexts.slice(0, 8).map((t) => pendingTokens(t).slice(0, 16))
+    },
     emitted: v.emitted === true,
     idle: v.idle !== false,
     ...v.intent?.verdict && { intent_verdict: v.intent.verdict },
@@ -15542,6 +15627,34 @@ ${addedLines}`,
   }
   return { diffs, has_baseline: true };
 }
+function absorbIntoBaseline(paths, sessionId) {
+  const baseline = readBaseline(sessionId);
+  if (!baseline || paths.length === 0) return 0;
+  const dir = sessionDir(sessionKey(baseline.session_id));
+  let adopted = 0;
+  const dirty = new Set(baseline.dirty_paths);
+  for (const p of paths) {
+    try {
+      const content = safeReadForMirror(projectPath(p));
+      if (content === null) continue;
+      const dest = mirrorPath(dir, p);
+      (0, import_node_fs13.mkdirSync)((0, import_node_path14.dirname)(dest), { recursive: true });
+      (0, import_node_fs13.writeFileSync)(dest, content);
+      dirty.add(p);
+      adopted++;
+    } catch {
+    }
+  }
+  if (adopted === 0) return 0;
+  try {
+    const updated = { ...baseline, dirty_paths: [...dirty] };
+    (0, import_node_fs13.writeFileSync)(manifestPath(dir), JSON.stringify(updated));
+    preImageCache.delete(baseline);
+  } catch {
+    return 0;
+  }
+  return adopted;
+}
 function changedSinceBaseline(repoRelPath, baseline) {
   const pre = preImage(repoRelPath, baseline);
   let current;
@@ -16700,7 +16813,7 @@ function resolveTaskContext(opts) {
 // src/lib/cli-version.ts
 function cliVersion() {
   try {
-    return true ? "0.28.1-experimental.89ee5db" : "dev";
+    return true ? "0.28.1-experimental.902dbd9" : "dev";
   } catch {
     return "dev";
   }
@@ -17374,9 +17487,13 @@ function buildAgentContext(input) {
   }
   for (const p of input.pendingItems ?? []) {
     if (lines.length >= MAX_AGENT_ITEMS) break;
+    if (p.pattern_id === "intent-misalignment") continue;
     const text = p.description ?? p.title ?? p.reason;
     if (!text) continue;
-    lines.push(renderItem("", text, p.pattern_id, p.file, p.line));
+    const seenBefore = isRepeatOfAny(text, input.priorPendingFingerprints ?? []);
+    lines.push(
+      renderItem("", text, p.pattern_id, p.file, p.line) + (seenBefore ? "\n    (raised earlier this session and still open \u2014 do not re-explain it; act on it or carry on)" : "")
+    );
   }
   if (lines.length === 0) return null;
   const body = `${REPORT_PREFIX}
@@ -17637,9 +17754,11 @@ var MAX_SUMMARY_BYTES = 4096;
 var HOME = process.env.HOME ?? "";
 async function extractActionSummary(transcriptPath) {
   try {
-    const lines = readTurnLines(transcriptPath);
-    if (!lines || lines.length === 0) return null;
-    return buildSummary(lines);
+    const read = readTurnLines(transcriptPath);
+    if (!read || read.lines.length === 0) return null;
+    const summary = buildSummary(read.lines);
+    if (summary) summary.transcript_windowed = read.window;
+    return summary;
   } catch {
     return null;
   }
@@ -17653,9 +17772,11 @@ function readTurnLines(transcriptPath) {
   }
   if (size === 0) return null;
   let raw;
+  let windowed = false;
   if (size <= SMALL_FILE_BYTES) {
     raw = (0, import_node_fs22.readFileSync)(transcriptPath, "utf-8");
   } else {
+    windowed = true;
     const buf = Buffer.alloc(Math.min(MAX_READ_BYTES, size));
     const fd = require("node:fs").openSync(transcriptPath, "r");
     try {
@@ -17673,17 +17794,22 @@ function readTurnLines(transcriptPath) {
   const allLines = raw.split("\n").filter((l) => l.trim().length > 0);
   if (allLines.length === 0) return null;
   let turnStart = 0;
+  let boundaryFound = false;
   for (let i = allLines.length - 1; i >= 0; i--) {
     try {
       const parsed = JSON.parse(allLines[i]);
       if (parsed.type === "user" && isRealUserMessage(parsed)) {
         turnStart = i;
+        boundaryFound = true;
         break;
       }
     } catch {
     }
   }
-  return allLines.slice(turnStart);
+  return {
+    lines: allLines.slice(turnStart),
+    window: !windowed ? "whole" : boundaryFound ? "windowed" : "orphaned"
+  };
 }
 function isRealUserMessage(parsed) {
   const message = parsed.message;
@@ -18241,11 +18367,12 @@ async function readStopHookStdin() {
     return empty;
   }
 }
-function agentContextFor(response, intentRepeat = 0) {
+function agentContextFor(response, intentRepeat = 0, priorPendingFingerprints = []) {
   const metadata = response.metadata ?? {};
   const intent = response.intent_alignment ?? {};
   return buildAgentContext({
     intentRepeat,
+    priorPendingFingerprints,
     gateDecision: String(response.gate_decision ?? ""),
     findings: response.findings ?? [],
     pendingItems: response.pending_items ?? [],
@@ -18381,13 +18508,21 @@ async function runAnalyze(opts, globals) {
   const conversation = await readAndClearConversationBuffer(baselineSessionId);
   const specs = discoverSpecs();
   const plans = discoverPlans();
+  const agentAuthoredCodeThisTurn = !!(actionSummary && (actionSummary.files_edited.length > 0 || actionSummary.files_created.length > 0));
+  const turnAuthoredCode = agentAuthoredCodeThisTurn || !!baseline && allForReview.some((f) => changedSinceBaseline(f, baseline));
+  const canSeeTurnAuthorship = !!actionSummary || !!baseline;
   const latestPrompt = conversation?.prompts?.[conversation.prompts.length - 1]?.prompt ?? "";
   if (/^\s*\/verity-/i.test(latestPrompt)) {
+    const setupAuthored = [
+      ...actionSummary?.files_edited ?? [],
+      ...actionSummary?.files_created ?? []
+    ];
+    if (setupAuthored.length > 0) {
+      const adopted = absorbIntoBaseline(setupAuthored, baselineSessionId);
+      logEvent("baseline_absorbed", { skip: "verity-command", offered: setupAuthored.length, adopted });
+    }
     await passAndExit("Verity command \u2014 skipping analysis", "verity-command");
   }
-  const agentAuthoredCodeThisTurn = !!(actionSummary && (actionSummary.files_edited.length > 0 || actionSummary.files_created.length > 0));
-  const turnAuthoredCode = agentAuthoredCodeThisTurn || !!baseline && allForReview.some((f) => changedSinceBaseline(f, baseline));
-  const canSeeTurnAuthorship = !!actionSummary || !!baseline;
   if (shouldSkipForBareAck({ prompt: latestPrompt, turnAuthoredCode, canSeeTurnAuthorship })) {
     await passAndExit("Bare acknowledgment \u2014 skipping analysis", "bare-acknowledgment");
   }
@@ -18437,7 +18572,11 @@ async function runAnalyze(opts, globals) {
     );
   }
   if (analysisMode === "skip") {
-    await passAndExit("Skip mode \u2014 no code work to analyze", "skip-mode");
+    await passAndExit(
+      "Skip mode \u2014 no code work to analyze",
+      "skip-mode",
+      turnAuthoredCode ? "capacity" : void 0
+    );
   }
   let staticResults = {
     tool: "@codacy/analysis-cli",
@@ -18709,7 +18848,30 @@ async function runAnalyze(opts, globals) {
     hasUserPrompt: (conversation?.prompts?.length ?? 0) > 0,
     isTTY: process.stdout.isTTY === true
   });
+  const excludedByReason = {};
+  for (const e of codeDelta.excluded ?? []) {
+    excludedByReason[e.reason] = (excludedByReason[e.reason] ?? 0) + 1;
+  }
+  const coverageTelemetry = {
+    // git's whole answer, before ANY narrowing. The number that has never been sent.
+    changed_all: allChanged.length,
+    analyzable: analyzable.length,
+    reviewable: reviewable.length,
+    security: securityFiles.length,
+    // after the allowlist, before authorship scoping and the caps
+    for_review: allForReview.length,
+    // what actually reaches the reviewer
+    sent: codeDelta.files.length,
+    // the two silent narrowings, counted separately so they can be told apart
+    capped_out: actionSummary?.capped_out?.length ?? 0,
+    excluded: (codeDelta.excluded ?? []).length,
+    excluded_by_reason: excludedByReason,
+    // was the transcript itself truncated? The 256 KB window means "this turn"
+    // can quietly mean "the last 256 KB of it".
+    transcript_windowed: actionSummary?.transcript_windowed ?? null
+  };
   const requestBody = {
+    coverage_telemetry: coverageTelemetry,
     static_results: staticResults,
     code_delta: codeDelta,
     changed_files: allForReview,
@@ -18795,6 +18957,54 @@ async function runAnalyze(opts, globals) {
         // replaced a population floor with ≈35% power that was sub-integer for
         // three-quarters of the fleet.
         conservation: foldConservation,
+        // ⚠ VRT-52 — RECORDED, NOT APPLIED. The number nobody has.
+        //
+        // The whole "task-scoped delta" design space rests on an assumption that
+        // has been observed exactly ONCE: that delta files routinely belong to
+        // earlier work. Three designs were built on it and all three were killed
+        // adversarially — two by measurement — so before another is attempted,
+        // measure the base rate.
+        //
+        // `authored_under_earlier_goal` counts delta paths whose LAST authorship
+        // event precedes the seq of the goal now in force. Both numbers come from
+        // the same append-only counter (`nextSeq`), so the comparison is exact.
+        //
+        // Keyed on the GOAL, deliberately, not on the task id. The task classifier
+        // reported `is_new_task` on two consecutive turns of one task 25 seconds
+        // apart, so a task-keyed number would measure its unreliability rather
+        // than the phenomenon. And this only became meaningful once `recordGoal`
+        // stopped letting a bare "ok" supersede the goal — before that the seq
+        // advanced every turn and this would have degenerated to "not edited this
+        // turn", which is the exact mistake that sank one of the three designs.
+        //
+        // Changes no payload the reviewer sees, no narrowing, no verdict.
+        vrt52: (() => {
+          const goalSeq = memory?.projection.goal?.seq;
+          if (goalSeq === void 0 || !memorySession) return { known: false };
+          const lastSeq2 = new Map(
+            foldDossier(memorySession.d).authored_all.map((a) => [a.path, a.last_seq])
+          );
+          let earlier = 0;
+          let unknown = 0;
+          for (const f of codeDelta.files) {
+            const seen = lastSeq2.get(f.path);
+            if (seen === void 0) unknown++;
+            else if (seen < goalSeq) earlier++;
+          }
+          return {
+            known: true,
+            goal_seq: goalSeq,
+            delta: codeDelta.files.length,
+            // Files this delta carries that were last written under an EARLIER
+            // instruction. If this stays near zero, VRT-52's code half is
+            // unnecessary and should be closed saying so.
+            authored_under_earlier_goal: earlier,
+            // Delta files the dossier has no authorship record for at all —
+            // pre-existing tree state, or an authorship channel the fold cannot
+            // see. Reported separately so a blind spot is never counted as a zero.
+            no_authorship_record: unknown
+          };
+        })(),
         // P2 — RECORDED, NOT APPLIED. What this run WOULD have reviewed if it
         // narrowed to the within-session increment: what changed since the last
         // VERDICT rather than since task start.
@@ -19041,6 +19251,13 @@ async function runAnalyze(opts, globals) {
     });
   }
   let intentRepeatCount = 0;
+  const priorPendingFingerprints = memorySession ? (() => {
+    try {
+      return foldDossier(memorySession.d).meta.recent_pending_sigs ?? [];
+    } catch {
+      return [];
+    }
+  })() : [];
   if (memorySession) {
     try {
       recordVerdict(memorySession.d, {
@@ -19063,7 +19280,10 @@ async function runAnalyze(opts, globals) {
         // What next turn reads as `emittedLast`. A suppressed turn did not
         // speak, so it cannot be the cause of the turn after it — which is what
         // keeps this from becoming a permanent gag.
-        emitted: !silenced
+        emitted: !silenced,
+        // Fingerprinted for the NEXT turn's repeat check. Reviewer pending items
+        // carry no `pattern_id`, so their content is the only available key.
+        pendingTexts: (response.pending_items ?? []).map((p) => String(p.description ?? p.title ?? p.reason ?? "")).filter(Boolean)
       });
       intentRepeatCount = Math.max(0, (foldDossier(memorySession.d).meta.intent_repeat?.consecutive ?? 1) - 1);
     } catch {
@@ -19282,7 +19502,7 @@ ${YELLOW}${grantNudge.trim()}${NC}
         // Subject to the SAME cycle cut as PASS/WARN. Suppressing here is safe:
         // the findings themselves are rendered above by the blocking renderer,
         // so what the cut removes is the repeated commentary, never the defect.
-        agentContext: silenced ? null : agentContextFor(response, intentRepeatCount),
+        agentContext: silenced ? null : agentContextFor(response, intentRepeatCount, priorPendingFingerprints),
         // The coverage note is silenced with it — half a channel is still a channel.
         silenced: !!silenced,
         openElsewhere
@@ -19303,7 +19523,7 @@ ${YELLOW}${grantNudge.trim()}${NC}
         changed: skipCoverageChanged,
         coverage: reviewCoverage,
         userSummary,
-        agentContext: silenced ? null : agentContextFor(response, intentRepeatCount),
+        agentContext: silenced ? null : agentContextFor(response, intentRepeatCount, priorPendingFingerprints),
         // The coverage note is silenced with it — half a channel is still a channel.
         silenced: !!silenced,
         openElsewhere
@@ -19323,7 +19543,7 @@ ${YELLOW}${grantNudge.trim()}${NC}
         changed: skipCoverageChanged,
         coverage: reviewCoverage,
         userSummary,
-        agentContext: silenced ? null : agentContextFor(response, intentRepeatCount),
+        agentContext: silenced ? null : agentContextFor(response, intentRepeatCount, priorPendingFingerprints),
         // The coverage note is silenced with it — half a channel is still a channel.
         silenced: !!silenced,
         openElsewhere
@@ -21127,7 +21347,7 @@ function registerTelemetryCommands(program2) {
 }
 
 // src/cli.ts
-program.name("verity").description("CLI for Verity quality gate service").version("0.28.1-experimental.89ee5db").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.902dbd9").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.89ee5db",
+  "version": "0.28.1-experimental.902dbd9",
   "description": "CLI for Verity quality gate service",
   "homepage": "https://verity.md",
   "bugs": {