@codacy/verity-cli 0.36.1-experimental.4892669 → 0.36.1-experimental.634f7d7

package/CHANGELOG.md

@@ -64,6 +64,23 @@ installs are untouched.
   which is the ordinary case for someone whose first Verity install *is* the
   plugin.
 
+## [0.36.1] — 2026-11-13
+
+### 🔎 Reviews at a commit and a push cover more, and say what they did not read
+
+- **A `git push` or a `git commit` with a pipe or a redirection is reviewed.**
+  For example, `git push 2>&1 | tail -5`. Before, Verity could not find the
+  target of such a command, and it allowed the command with no review.
+- **A change made only of large files is reviewed.** A file over the size
+  limit is not sent whole. The agent review now starts for such a change and
+  reads the diff of each file. Before, the change ended with no review and no
+  message.
+- **The result names each large file that Verity did not read in full.** When
+  the agent review received only a part of a file's diff, the notice says
+  "reviewed only in part" and gives the number of characters it received. When
+  nothing read the file, the notice says "NOT reviewed". A PASS then covers
+  only what was read, and says so.
+
 ## [0.36.0] — 2026-11-12
 
 ### 🧠 Curate your project's knowledge from the CLI

package/bin/verity.js

@@ -15454,7 +15454,7 @@ function pinRefusalMessage(verdict, targetUrl) {
 // src/lib/cli-version.ts
 function cliVersion() {
   try {
-    return true ? "0.36.1-experimental.4892669" : "dev";
+    return true ? "0.36.1-experimental.634f7d7" : "dev";
   } catch {
     return "dev";
   }
@@ -42462,8 +42462,26 @@ function oversizedPaths(codeDelta) {
   return codeDelta.excluded.filter((e) => e.reason.startsWith("over-file-size-limit")).map((e) => e.path);
 }
 function unreviewedOversized(codeDelta, agentOutcome) {
-  const concluded = agentOutcome?.kind === "finished" && agentOutcome.response.agent_review.outcome === "concluded";
-  return concluded ? [] : oversizedPaths(codeDelta);
+  return agentConcluded(agentOutcome) ? [] : oversizedPaths(codeDelta);
+}
+function agentConcluded(agentOutcome) {
+  return agentOutcome?.kind === "finished" && agentOutcome.response.agent_review.outcome === "concluded";
+}
+function cutOversizedDiffs(oversized, given, whole) {
+  const shownOf = new Map(given.map((d) => [d.path, d.diff.length]));
+  const totalOf = new Map(whole.map((d) => [d.path, d.diff.length]));
+  const cut = [];
+  for (const path of oversized) {
+    const shown = shownOf.get(path) ?? 0;
+    const total = totalOf.get(path) ?? null;
+    if (total === null || shown < total) cut.push({ path, shown, total });
+  }
+  return cut;
+}
+function partlyReviewedOversized(codeDelta, agentOutcome) {
+  if (!agentConcluded(agentOutcome)) return [];
+  const oversized = new Set(oversizedPaths(codeDelta));
+  return (agentOutcome?.cutDiffs ?? []).filter((d) => oversized.has(d.path));
 }
 var GUARD_BLOCK_CAP = 2;
 var GUARD_ITER_FILE = (0, import_node_path60.join)(VERITY_DIR, ".guard-iteration");
@@ -42769,11 +42787,14 @@ function startAgentReview(input) {
     }
   };
   const sentPaths = input.codeDelta.files.map((f) => f.path);
-  const diffPaths = [...sentPaths, ...oversizedPaths(input.codeDelta)];
+  const oversized = oversizedPaths(input.codeDelta);
+  const diffPaths = [...sentPaths, ...oversized];
   const run4 = async () => {
     const loaded = await loadCodeMap();
     ctx.codeMap = loaded.ok ? loaded.map : null;
-    return runAgentReview({
+    const diffs = await buildAgentDiffs(ctx, diffPaths);
+    const cutDiffs = measureCutDiffs(ctx, oversized, diffs);
+    const outcome = await runAgentReview({
       http: http2,
       executeTools: makeToolExecutor(ctx),
       startedAt: input.hookStartedAt,
@@ -42786,7 +42807,7 @@ function startAgentReview(input) {
         stated_intent: input.statedIntent,
         changed_files: input.files,
         sent_paths: sentPaths,
-        diffs: await buildAgentDiffs(ctx, diffPaths),
+        diffs,
         repo_context: input.requestBody.repo_context,
         static_results: input.requestBody.static_results,
         one_shot_chars: input.codeDelta.files.reduce((n2, f) => n2 + f.content.length, 0),
@@ -42795,9 +42816,16 @@ function startAgentReview(input) {
       oneShot: input.oneShot.then((r) => r.ok ? { ok: true, response: r.data } : { ok: false }),
       sessionId: input.sessionId
     });
+    return { ...outcome, cutDiffs: await cutDiffs };
   };
   return run4().catch((err2) => ({ kind: "failed", reason: String(err2?.message ?? err2) }));
 }
+var WHOLE_DIFF = { perFileChars: Number.MAX_SAFE_INTEGER, totalChars: Number.MAX_SAFE_INTEGER };
+async function measureCutDiffs(ctx, oversized, given) {
+  if (oversized.length === 0) return [];
+  const whole = await buildAgentDiffs(ctx, oversized, WHOLE_DIFF).catch(() => []);
+  return cutOversizedDiffs(oversized, given, whole);
+}
 function formalOffUnverifiedLine(moment, config) {
   if (moment !== "pre-push" || !config.formal_off_unverified) return "";
   const setting = formalSetting(config);
@@ -43108,6 +43136,7 @@ async function runGuard(opts, globals) {
   }
   resetIter(moment);
   const unreviewed = unreviewedOversized(codeDelta, agentOutcome);
+  const partial = partlyReviewedOversized(codeDelta, agentOutcome);
   const notice = verdictNotice({
     decision,
     moment,
@@ -43117,6 +43146,7 @@ async function runGuard(opts, globals) {
     link: link2,
     viewUrl,
     unreviewed,
+    partial,
     sentFiles: codeDelta.total_files,
     narrative: response.assessment?.narrative ?? "",
     serious: seriousFindings(response, witnessed)
@@ -43141,19 +43171,25 @@ ${covDetail}${report}`
     };
   }
   const serious = seriousList(ctx.serious ?? [], verb);
+  const unreviewed = ctx.unreviewed ?? [];
+  const n2 = unreviewed.length;
+  const listed = unreviewed.slice(0, UNREVIEWED_LISTED).map((p) => terminalText(p, 300)).join(", ");
+  const more = n2 > UNREVIEWED_LISTED ? `, and ${n2 - UNREVIEWED_LISTED} more` : "";
+  const why = `${n2} changed file(s) are over the size limit, so they are not sent, and the agent review that reads their diff did not conclude: ${listed}${more}.`;
+  const partial = ctx.partial ?? [];
+  const whyPartial = partialWhy(partial);
   if (decision === "WARN") {
+    const sizeUser = n2 > 0 ? ` \xB7 ${n2} changed file(s) over the size limit NOT reviewed` : partial.length > 0 ? ` \xB7 ${partial.length} changed file(s) over the size limit reviewed only in part` : "";
+    const sizeAgent = `${n2 > 0 ? `NOT reviewed: ${why} Treat those files as unreviewed.
+` : ""}${partial.length > 0 ? `Reviewed only in part: ${whyPartial}
+` : ""}`;
     return {
-      user: `\u26A0 Verity ${moment}: WARN \u2014 ${serious.count ? `${serious.count} high-severity finding(s), not blocking` : "proceeding"} (${covLine})${link2}`,
+      user: `\u26A0 Verity ${moment}: WARN \u2014 ${serious.count ? `${serious.count} high-severity finding(s), not blocking` : "proceeding"} (${covLine})${sizeUser}${link2}`,
       agent: `Verity ${moment} review: WARN (proceeding).
-${serious.text}${covDetail}${report}`
+${sizeAgent}${serious.text}${covDetail}${report}`
     };
   }
-  const unreviewed = ctx.unreviewed ?? [];
-  if (decision === "PASS" && unreviewed.length > 0) {
-    const n2 = unreviewed.length;
-    const listed = unreviewed.slice(0, UNREVIEWED_LISTED).map((p) => terminalText(p, 300)).join(", ");
-    const more = n2 > UNREVIEWED_LISTED ? `, and ${n2 - UNREVIEWED_LISTED} more` : "";
-    const why = `${n2} changed file(s) are over the size limit, so they are not sent, and the agent review that reads their diff did not conclude: ${listed}${more}.`;
+  if (decision === "PASS" && n2 > 0) {
     if ((ctx.sentFiles ?? 0) === 0) {
       return {
         user: `\u26A0 Verity ${moment}: NOT REVIEWED \u2014 ${n2} changed file(s) over the size limit (${covLine})${link2}`,
@@ -43164,6 +43200,13 @@ ${covDetail}${report}`
     return {
       user: `\u26A0 Verity ${moment}: PASS for the files reviewed \u2014 ${n2} changed file(s) over the size limit NOT reviewed (${covLine})${link2}`,
       agent: `Verity ${moment} review: PASS for the files that were sent. NOT reviewed: ${why} Treat those files as unreviewed.
+${serious.text}${covDetail}${report}`
+    };
+  }
+  if (decision === "PASS" && partial.length > 0) {
+    return {
+      user: `\u26A0 Verity ${moment}: PASS for what was read \u2014 ${partial.length} changed file(s) over the size limit reviewed only in part (${covLine})${link2}`,
+      agent: `Verity ${moment} review: PASS for what was read. Reviewed only in part: ${whyPartial}
 ${serious.text}${covDetail}${report}`
     };
   }
@@ -43182,6 +43225,14 @@ ${covDetail}${report}`
 }
 var SERIOUS_LISTED = 8;
 var UNREVIEWED_LISTED = 8;
+function partialWhy(partial) {
+  if (partial.length === 0) return "";
+  const count2 = (chars) => Math.max(0, Math.trunc(chars)).toLocaleString("en-US");
+  const share = (d) => d.total === null ? `${count2(d.shown)} diff characters received, the total was not measured` : `${count2(d.shown)} of ${count2(d.total)} diff characters received`;
+  const listed = partial.slice(0, UNREVIEWED_LISTED).map((d) => `${terminalText(d.path, 300)} (${share(d)})`).join(", ");
+  const more = partial.length > UNREVIEWED_LISTED ? `, and ${partial.length - UNREVIEWED_LISTED} more` : "";
+  return `${partial.length} changed file(s) are over the size limit, so they are not sent, and the agent review did not receive their whole diff: ${listed}${more}. Treat the rest of each file as unreviewed.`;
+}
 function seriousList(findings, verb) {
   if (findings.length === 0) return { count: 0, text: "" };
   const lines2 = [`${findings.length} finding(s) at high or critical severity did not block this ${verb}. Check each one: fix it, or tell the user why it does not apply.`];
@@ -46318,7 +46369,7 @@ async function recordAnswers(ui, momentIds, telemetry, completed) {
       moments: momentIds,
       ...telemetry ? { telemetry } : {},
       // A failed run also clears an earlier stamp: "done" describes this checkout now.
-      init: completed ? { completed_at: (/* @__PURE__ */ new Date()).toISOString(), cli_version: true ? "0.36.1-experimental.4892669" : "dev" } : void 0
+      init: completed ? { completed_at: (/* @__PURE__ */ new Date()).toISOString(), cli_version: true ? "0.36.1-experimental.634f7d7" : "dev" } : void 0
     });
   } catch (err2) {
     ui.rail.line(`Could not record the setup answers: ${terminalText(err2.message, 200)}`, "warn");
@@ -49695,10 +49746,10 @@ function registerLeanCommand(program2) {
 }
 
 // src/cli.ts
-program.name("verity").description("CLI for Verity quality gate service").version("0.36.1-experimental.4892669").option("--token <token>", "Override authentication token").option("--service-url <url>", "Override service URL").option("--verbose", "Log HTTP requests/responses to stderr").hook("preAction", async (_thisCommand, actionCommand) => {
+program.name("verity").description("CLI for Verity quality gate service").version("0.36.1-experimental.634f7d7").option("--token <token>", "Override authentication token").option("--service-url <url>", "Override service URL").option("--verbose", "Log HTTP requests/responses to stderr").hook("preAction", async (_thisCommand, actionCommand) => {
   const isTelemetryHelper = actionCommand.name() === "headers" && actionCommand.parent?.name() === "telemetry";
   if (!isTelemetryHelper) {
-    installStderrLog(actionCommand.name(), process.argv.slice(2), "0.36.1-experimental.4892669");
+    installStderrLog(actionCommand.name(), process.argv.slice(2), "0.36.1-experimental.634f7d7");
   }
   setUserNamedServiceUrl(program.opts().serviceUrl);
   try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/verity-cli",
-  "version": "0.36.1-experimental.4892669",
+  "version": "0.36.1-experimental.634f7d7",
   "description": "CLI for Verity quality gate service",
   "license": "SEE LICENSE IN LICENSE",
   "homepage": "https://verity.md",