codeceipt 0.1.1 → 0.1.2

package/codeceipt.mjs

@@ -20787,12 +20787,14 @@ var itemSchema = external_exports.object({
 }).strict();
 var QUESTION_RULE = "Ask a question only when something cannot be built without guessing what the user wants (for example 'make it better'): start that line with '? ' and say what needs clarifying. A request that names what to do and where is clear (for example 'Add a comment at the top of the main page'): never ask about wording, text, style or details the agent can reasonably choose.";
 var ITEM_RULE = "Each item is an outcome the user would check (what is true once it is done), never a step to get there (such as finding, opening or reading a file) and never a general rule about how to write the code (such as keeping the existing code unchanged or following its style). A constraint the user asked for (for example 'passwords are never logged') is an outcome. A short request with one purpose is exactly one item, in the user's words.";
+var SPLIT_RULE = "A request that asks for several things, joined by 'and', 'then', ';' or a new sentence, is one item per thing: each quoted name to check and each change to make is its own item. Never merge several things into one item.";
 var DATA_RULE = "File paths, quoted text, code, package names and tool names in the request are details of a normal request: list its items as usual; never refuse and never turn the whole request into a question.";
 var POSTPONE_RULE = "When the user says to leave part of the request for later or to skip it for now, still list that item, starting the line with '~ ' (postponed); never drop it.";
 var CHECKLIST_SYSTEM = [
   "You split a user's software change request into a checklist. Answer only with lines, at most 10.",
   "Each line is one concrete, separately checkable item: a page, each distinct element, each behaviour.",
   ITEM_RULE,
+  SPLIT_RULE,
   "Start a clear item with '- '.",
   QUESTION_RULE,
   POSTPONE_RULE,
@@ -21286,7 +21288,9 @@ var findingList = (value) => list(value).flatMap((raw) => {
 });
 var either = (s, key3, legacy) => s[key3] !== void 0 ? s[key3] : s[legacy];
 var entry = (raw) => typeof raw === "string" ? { text: raw } : obj(raw);
-function receiptView(summary2) {
+var ITEM_ID = /^[\w-]{1,64}$/;
+var itemIdOf = (raw) => typeof raw === "string" && ITEM_ID.test(raw) ? raw : void 0;
+function receiptView(summary2, closedByOwner = /* @__PURE__ */ new Set()) {
   const s = obj(summary2);
   const noChanges = s.no_changes === true;
   const untested = new Set(
@@ -21305,11 +21309,15 @@ function receiptView(summary2) {
       if (text === "" || typeof status !== "string" || !STATUSES.has(status)) return [];
       const legacyUntested = status === "not_verified" && typeof i.id === "string" && untested.has(i.id);
       const shown2 = legacyUntested ? "done_untested" : noChanges && status === "missed" ? "not_done" : status;
+      const itemId = itemIdOf(i.id);
+      const closed = itemId !== void 0 && closedByOwner.has(itemId);
       return [
         {
           text,
-          status: shown2,
-          evidence: list(i.evidence).map(safePath).filter(Boolean).slice(0, 5)
+          status: closed ? "done" : shown2,
+          evidence: list(i.evidence).map(safePath).filter(Boolean).slice(0, 5),
+          ...itemId !== void 0 ? { itemId } : {},
+          ...closed ? { closedByOwner: true } : {}
         }
       ];
     }),
@@ -21956,6 +21964,15 @@ async function verifiedBlobs(git, root, blobs, snapshot) {
   }
   return out;
 }
+async function verifiedStartTexts(git, root, blobs, snapshot) {
+  const out = {};
+  const ids = await verifiedBlobs(git, root, blobs, snapshot);
+  for (const [path3, id] of Object.entries(ids)) {
+    const text = await git(["cat-file", "blob", id], root).catch(() => null);
+    if (text !== null && text.length <= MAX_SCAN_BYTES) out[path3] = text;
+  }
+  return out;
+}
 async function blobOf(git, root, file2) {
   const abs = await inside(root, file2);
   const st = abs ? await lstat2(abs).catch(() => null) : null;
@@ -22238,24 +22255,36 @@ async function beforeText(git, root, base, file2) {
   const text = await git(["show", `${base}:./${file2}`], root).catch(() => null);
   return text !== null && text.length <= MAX_SCAN_BYTES ? text : null;
 }
-async function localIntegrity(git, root, base, files) {
+async function localIntegrity(git, root, base, files, startTexts = {}) {
   const relevant = files.filter((f) => isTestFile(f) || isTestConfig(f)).slice(0, MAX_FILES2);
   const changes = [];
+  const earlier = [];
   for (const path3 of relevant) {
     const [before, after] = await Promise.all([
       base ? beforeText(git, root, base, path3) : Promise.resolve(null),
       afterText(root, path3)
     ]);
+    const start = Object.hasOwn(startTexts, path3) ? startTexts[path3] : void 0;
+    if (start !== void 0) {
+      changes.push({ path: path3, before: start, after });
+      earlier.push({ path: path3, before, after: start });
+      continue;
+    }
     if (before === null && after === null) continue;
     changes.push({ path: path3, before, after });
   }
-  const flags = checkTestIntegrity(changes).sort((a, b) => PRIORITY.indexOf(a.kind) - PRIORITY.indexOf(b.kind)).map((f) => ({
+  const clean3 = (list2) => list2.sort((a, b) => PRIORITY.indexOf(a.kind) - PRIORITY.indexOf(b.kind)).map((f) => ({
     ...f,
     file: safePath(f.file),
     names: f.names.map((n) => safeDisplay(n, 200)).filter(Boolean),
     count: Math.min(f.count, MAX_COUNT2)
   }));
-  return { flags: flags.slice(0, MAX_FLAGS), omitted: Math.max(0, flags.length - MAX_FLAGS) };
+  const flags = clean3(checkTestIntegrity(changes));
+  return {
+    flags: flags.slice(0, MAX_FLAGS),
+    omitted: Math.max(0, flags.length - MAX_FLAGS),
+    beforeSession: clean3(checkTestIntegrity(earlier))
+  };
 }
 async function passingTests(root, reportPath) {
   if (!reportPath) return null;
@@ -22366,6 +22395,13 @@ var head = (git, root) => git(["rev-parse", "--verify", "-q", "HEAD"], root).the
 var count = (v) => Array.isArray(v) ? v.length : 0;
 var number4 = (v) => typeof v === "number" && Number.isSafeInteger(v) && v > 0 ? Math.min(v, 1e6) : 0;
 var plural2 = (n, one, many = `${one}s`) => `${n} ${n === 1 ? one : many}`;
+var PRE_TEST_LABEL = {
+  skipped_tests: "skipped test",
+  deleted_tests: "deleted test",
+  deleted_test_file: "deleted test file",
+  removed_assertions: "removed assertion",
+  test_config_changed: "changed test config"
+};
 function receiptSummary(receipt, config2, receiptId, filesChanged) {
   const grouped = Array.isArray(receipt.files_by_feature) ? receipt.files_by_feature.reduce(
     (n, g) => n + count(typeof g === "object" && g !== null ? g.files : null),
@@ -22380,7 +22416,14 @@ function receiptSummary(receipt, config2, receiptId, filesChanged) {
   const untested = status("done_untested");
   const done = status("done") + untested;
   const partial2 = status("partial");
-  const missed = status("missed") + partial2;
+  const missed = status("missed");
+  const notVerified = status("not_verified");
+  const states = [
+    done > 0 ? `${done} done` : "",
+    partial2 > 0 ? `${partial2} partly done` : "",
+    missed > 0 ? `${missed} missed` : "",
+    notVerified > 0 ? `${notVerified} not verified` : ""
+  ].filter(Boolean);
   const postponed = status("postponed");
   const present = status("already_present");
   const questions = status("question");
@@ -22416,7 +22459,7 @@ function receiptSummary(receipt, config2, receiptId, filesChanged) {
     receipt.no_request === true ? `Codeceipt receipt: ${changed ? `${changed}. ` : ""}${NO_REQUEST_NOTE}` : receipt.no_changes === true ? (
       // Real-test F: a request but no changed file still gets a receipt.
       `Codeceipt receipt: No files changed; ${status("missed")} not done${present > 0 ? `, ${present} already present` : ""}.`
-    ) : `Codeceipt receipt: ${changed ? `${changed}; ` : ""}${done} done, ${missed} missed${partial2 > 0 ? ` (${partial2} partly done)` : ""}.`,
+    ) : `Codeceipt receipt: ${changed ? `${changed}; ` : ""}${states.length > 0 ? states.join(", ") : "no items"}.`,
     `Checks needing attention: ${attention.length > 0 ? attention.join(", ") : "none"}.`,
     ...extra.length > 0 ? [`Also listed: ${extra.join(", ")}.`] : []
   ];
@@ -22591,11 +22634,18 @@ async function sessionEnd(_agent, deps, opts = {}) {
   const session = new Set(files);
   const sessionFindings = scan?.findings.filter((f) => session.has(f.file)) ?? [];
   const beforeFindings = scan?.findings.filter((f) => !session.has(f.file)) ?? [];
+  const startTexts = snapshot ? await verifiedStartTexts(
+    deps.git,
+    root,
+    Object.fromEntries(Object.entries(blobs).filter(([path3]) => preExisting.includes(path3))),
+    snapshot
+  ).catch(() => ({})) : {};
   const integrity = await localIntegrity(
     deps.git,
     root,
     start ?? (commit ? "HEAD" : null),
-    files
+    files,
+    startTexts
   ).catch(() => null);
   const passingAfter = await passingTests(root, brandEnv(deps.env ?? {}, "TEST_REPORT"));
   const summaries = await changeSummaries(
@@ -22748,6 +22798,13 @@ async function sessionEnd(_agent, deps, opts = {}) {
         `${WARN} ${testWarnings} sign(s) that tests were removed or weakened; see the receipt.`
       );
     }
+    const earlier = integrity?.beforeSession ?? [];
+    if (earlier.length > 0) {
+      const items = earlier.slice(0, MAX_WARNED_ITEMS).map((f) => `${plural2(f.count, PRE_TEST_LABEL[f.kind] ?? "test change")} in ${f.file}`);
+      warnings.push(
+        `${WARN} already modified before this session, not counted: ${items.join("; ")}.`
+      );
+    }
     if (scan && scan.findings.length + scan.omitted > 0) {
       warnings.push(
         `${WARN} ${scan.findings.length + scan.omitted} possible secret(s) in the changed files; see the receipt.`
@@ -23809,7 +23866,7 @@ var STOP_HOOK = hook(BRAND, "stop");
 var LEGACY_START_HOOK = hook(LEGACY, "session-start");
 var LEGACY_END_HOOK = hook(LEGACY, "session-end");
 var NODE_PATHS = '"$PATH:/usr/local/bin:/opt/homebrew/bin:$HOME/.volta/bin:$HOME/.npm-global/bin:$HOME/.local/bin"';
-var CLI_VERSION = "0.1.1";
+var CLI_VERSION = "0.1.2";
 var wrap = (args, version2) => `PATH=${NODE_PATHS}; if command -v codeceipt >/dev/null 2>&1; then codeceipt ${args}; else npx -y codeceipt@${version2} ${args}; fi`;
 var shellHook = (bare) => wrap(bare.replace(/^codeceipt /, ""), CLI_VERSION);
 var isWrapped = (command, bare) => {
@@ -24728,6 +24785,46 @@ async function runMcpProxy(lines, config2, fetchImpl, write2) {
   }
 }
 
+// src/owner-states.ts
+var PLAIN_ID = /^[\w-]{1,64}$/;
+var MAX_IDS = 500;
+function plainIds(value) {
+  const out = /* @__PURE__ */ new Set();
+  if (!Array.isArray(value)) return out;
+  for (const id of value.slice(0, MAX_IDS)) {
+    if (typeof id === "string" && PLAIN_ID.test(id)) out.add(id);
+  }
+  return out;
+}
+function applyOwnerStates(receipt, closedIds, dismissedIds) {
+  if (!Array.isArray(receipt.items)) return receipt;
+  const closed = plainIds(closedIds);
+  const dismissed = plainIds(dismissedIds);
+  if (closed.size === 0 && dismissed.size === 0) return receipt;
+  const items = receipt.items.map((item) => {
+    if (typeof item !== "object" || item === null) return item;
+    const id = item.id;
+    if (typeof id !== "string") return item;
+    if (dismissed.has(id)) return { ...item, status: "dismissed" };
+    if (closed.has(id)) return { ...item, status: "closed_by_owner" };
+    return item;
+  });
+  return { ...receipt, items };
+}
+function ownerStateLines(receipt) {
+  if (!Array.isArray(receipt.items)) return [];
+  const lines = [];
+  for (const item of receipt.items) {
+    if (typeof item !== "object" || item === null) continue;
+    const { status, text } = item;
+    const shown2 = typeof text === "string" ? safeDisplay(text, 200) : "";
+    if (shown2 === "") continue;
+    if (status === "closed_by_owner") lines.push(`Closed by you: ${shown2}`);
+    if (status === "dismissed") lines.push(`Dismissed: ${shown2}`);
+  }
+  return lines.slice(0, 30);
+}
+
 // src/repo-link.ts
 import {
   constants as constants3,
@@ -25426,25 +25523,24 @@ async function runLast(deps, path3, evidence = false) {
       "Saved server address must be https: run `npx codeceipt` again."
     );
   }
-  const answer = await request(
-    deps.fetch,
-    config2,
-    "GET",
-    `/api/hooks/${encodeURIComponent(config2.projectId)}/last-receipt`
-  );
-  const receipt = answer.receipt;
-  if (typeof receipt !== "object" || receipt === null) {
+  const answer = await request(deps.fetch, config2, "GET", `/api/hooks/${encodeURIComponent(config2.projectId)}/last-receipt`);
+  const stored = answer.receipt;
+  if (typeof stored !== "object" || stored === null) {
     deps.stdout("No receipt yet: make a change with your agent and end the session.");
     return EXIT.ok;
   }
+  const receipt = applyOwnerStates(
+    stored,
+    answer.owner_closed_ids,
+    answer.dismissed_ids
+  );
   const id = typeof answer.receipt_id === "string" && /^[\w-]{1,64}$/.test(answer.receipt_id) ? answer.receipt_id : null;
   const at = typeof answer.created_at === "string" && /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}/.test(answer.created_at) ? ` (${answer.created_at.slice(0, 16).replace("T", " ")} UTC)` : "";
   deps.stdout(`Latest receipt${at}:`);
-  for (const line2 of receiptSummary(receipt, config2, id))
-    deps.stdout(line2);
+  for (const line2 of receiptSummary(receipt, config2, id)) deps.stdout(line2);
+  for (const line2 of ownerStateLines(receipt)) deps.stdout(line2);
   for (const line2 of attentionLines(receipt)) deps.stdout(line2);
-  if (evidence)
-    for (const line2 of evidenceLines(receipt)) deps.stdout(line2);
+  if (evidence) for (const line2 of evidenceLines(receipt)) deps.stdout(line2);
   return EXIT.ok;
 }
 var OWN_COMMAND = /^!?\s*(npx\s+)?codeceipt\b/i;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceipt",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "A receipt for every AI change.",
   "keywords": [
     "claude-code",