@prjct.app/pi-signals 0.2.0 → 0.2.1

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.2.1 (2026-10-13)
+
+- The `seen` list in `evidenceTool` is now a cell holding an immutable array. Each `execute` builds a new array with the additions appended instead of pushing in place; `contains` reads from the cell.
+
 ## 0.2.0 (2026-10-09)
 
 - `/signals` opens the shared docked panel, like the other prjct extensions. It lists what needs a fix first, then queued, running, flaky and resolved issues, each with component, status and sightings. An issue's detail shows the triage decision, recent observations and runs. The Daily triage row shows schedule, recent days, repositories and the repair setting. `r` queues an issue for triage again. Outside the TUI the same list prints as text instead of JSON.

package/dist/cli.mjs

@@ -834,7 +834,7 @@ var init_schema = __esm({
 // src/evidence.ts
 import { Type as Type2 } from "typebox";
 function evidenceTool(source) {
-  const seen = [];
+  const seen = { items: [] };
   const tool = {
     name: "signal_evidence",
     label: "Read original signal evidence",
@@ -844,14 +844,16 @@ function evidenceTool(source) {
       const input = raw, offset = input.offset ?? 0, limit = input.limit ?? 1;
       const rows = typeof source === "function" ? source(offset, limit + 1) : source.slice(offset, offset + limit + 1);
       const items = await protectOutboundData(rows.slice(0, limit));
+      const additions = [];
       for (const item of items) {
         const payload = item.payload;
-        seen.push(payload?.body ?? JSON.stringify(item));
+        additions.push(payload?.body ?? JSON.stringify(item));
       }
+      if (additions.length) seen.items = [...seen.items, ...additions];
       return { content: [{ type: "text", text: JSON.stringify({ items, nextOffset: rows.length > limit ? offset + limit : null }) }], details: {} };
     }
   };
-  return { tool, contains: (quote2) => seen.some((text2) => text2.includes(quote2)) };
+  return { tool, contains: (quote2) => seen.items.some((text2) => text2.includes(quote2)) };
 }
 var init_evidence = __esm({
   "src/evidence.ts"() {
@@ -953,10 +955,10 @@ ${JSON.stringify({ issue })}`,
     const source = resolve(path, file), destination = resolve(baselinePath, file);
     if (!source.startsWith(path + sep2) || !destination.startsWith(baselinePath + sep2)) return blocked("Regression file escaped the worktree.");
     for (const root of [path, baselinePath]) {
-      let current = root;
+      const cursor = { current: root };
       for (const part of file.split("/")) {
-        current = join6(current, part);
-        const stat = await lstat(current).catch(() => void 0);
+        cursor.current = join6(cursor.current, part);
+        const stat = await lstat(cursor.current).catch(() => void 0);
         if (stat?.isSymbolicLink()) return blocked("Regression paths must not contain symlinks.");
       }
     }
@@ -973,7 +975,7 @@ ${JSON.stringify({ issue })}`,
   if (await protectOutboundData(diff) !== diff) return blocked("The candidate diff contains data caught by outbound protection; publication is blocked.");
   await must(path, ["git", "commit", "-m", `fix: ${issue.title}`], signal);
   const head = await must(path, ["git", "rev-parse", "HEAD"], signal);
-  for (let sample = 0; sample < 3; sample++) {
+  for (const _sample of [0, 1, 2]) {
     const before = await command(baselinePath, fix.regressionCommand, signal);
     checks.push(before);
     if (before.code === 0 || !before.output.includes(fix.failurePattern)) return blocked("The unchanged baseline did not consistently reproduce the claimed failure.");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prjct.app/pi-signals",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Local failure signals, daily agent-owned triage and verified repair candidates for Pi.",
   "type": "module",
   "license": "MIT",
@@ -51,7 +51,7 @@
   },
   "scripts": {
     "check": "tsc --noEmit",
-    "test": "node --import ./tests/isolate.mjs --import tsx --test --test-concurrency=2 tests/*.test.ts",
+    "test": "export PI_CODING_AGENT_DIR=\"$(mktemp -d)\" PRJCT_HOME=\"$(mktemp -d)\" && node --import ./tests/isolate.mjs --import tsx --test --test-concurrency=2 tests/*.test.ts",
     "check:package": "npm pack --dry-run --ignore-scripts",
     "build": "node scripts/build.mjs",
     "prepack": "npm run build",

package/src/evidence.ts

@@ -5,7 +5,8 @@ import { protectOutboundData } from '@prjct.app/pi-secrets/privacy';
 export type EvidenceSource = readonly unknown[] | ((offset: number, limit: number) => unknown[]);
 /** Keep full observations in storage; the deciding agent chooses which pages to read. */
 export function evidenceTool(source: EvidenceSource) {
-  const seen: string[] = [];
+  // The one cell that changes: it holds the current list, which is always replaced, never edited.
+  const seen: { items: readonly string[] } = { items: [] };
   const tool: ToolDefinition = {
     name: 'signal_evidence', label: 'Read original signal evidence',
     description: 'Read original issue observations by page. Nothing is discarded; follow nextOffset to inspect all sightings. Select a smaller limit for large reports.',
@@ -14,12 +15,14 @@ export function evidenceTool(source: EvidenceSource) {
       const input = raw as { offset?: number; limit?: number }, offset = input.offset ?? 0, limit = input.limit ?? 1;
       const rows = typeof source === 'function' ? source(offset, limit + 1) : source.slice(offset, offset + limit + 1);
       const items = await protectOutboundData(rows.slice(0, limit));
+      const additions: string[] = [];
       for (const item of items) {
         const payload = (item as { payload?: { body?: string } }).payload;
-        seen.push(payload?.body ?? JSON.stringify(item));
+        additions.push(payload?.body ?? JSON.stringify(item));
       }
+      if (additions.length) seen.items = [...seen.items, ...additions];
       return { content: [{ type: 'text', text: JSON.stringify({ items, nextOffset: rows.length > limit ? offset + limit : null }) }], details: {} };
     },
   };
-  return { tool, contains: (quote: string) => seen.some(text => text.includes(quote)) };
+  return { tool, contains: (quote: string) => seen.items.some(text => text.includes(quote)) };
 }

package/src/repair.ts

@@ -43,10 +43,10 @@ export async function repair(issue: LocalIssue, repo: Repository, path: string,
     if (!source.startsWith(path + sep) || !destination.startsWith(baselinePath + sep)) return blocked('Regression file escaped the worktree.');
     // Refuse symlinks throughout either path; do not copy arbitrary external files.
     for (const root of [path, baselinePath]) {
-      let current = root;
+      const cursor = { current: root };
       for (const part of file.split('/')) {
-        current = join(current, part);
-        const stat = await lstat(current).catch(() => undefined);
+        cursor.current = join(cursor.current, part);
+        const stat = await lstat(cursor.current).catch(() => undefined);
         if (stat?.isSymbolicLink()) return blocked('Regression paths must not contain symlinks.');
       }
     }
@@ -62,7 +62,7 @@ export async function repair(issue: LocalIssue, repo: Repository, path: string,
   if (await protectOutboundData(diff) !== diff) return blocked('The candidate diff contains data caught by outbound protection; publication is blocked.');
   await must(path, ['git', 'commit', '-m', `fix: ${issue.title}`], signal);
   const head = await must(path, ['git', 'rev-parse', 'HEAD'], signal);
-  for (let sample = 0; sample < 3; sample++) {
+  for (const _sample of [0, 1, 2]) {
     const before = await command(baselinePath, fix.regressionCommand, signal); checks.push(before);
     if (before.code === 0 || !before.output.includes(fix.failurePattern)) return blocked('The unchanged baseline did not consistently reproduce the claimed failure.');
     const after = await command(path, fix.regressionCommand, signal); checks.push(after);