@llblab/pi-actors 0.53.1 → 0.53.2

package/AGENTS.md

@@ -86,7 +86,7 @@ Canonical event:
 {"id":"…","ts":"…","kind":"…","summary":"…","data":{},"level":"info","attention":"notify"}
 ```
 
-Trace is a bounded retained suffix, not an audit archive. Every first-party writer must call `appendRunTraceEvent`; under the canonical token-owned lock it appends within both fixed bounds or atomically retains the newest suffix plus one cumulative warning-only `runtime.trace_compacted` marker. The marker means older history was discarded; terminal/result/execution/artifact files remain authoritative independently. Reads are newline-safe and order equal timestamps by same-source ordinal, fixed source rank, then stable id without exposing ordering metadata. Never write `trace.jsonl` directly. Generic command lifecycle is Trace-only: `command.done` preserves complete bounded execution evidence but never requests or projects any attention value, and Recipe grammar has no command-completion delivery switch. Semantic attention is explicit: `notify` is visible status, `followup` is an ordinary checkpoint hint, and actor-authored `steer` is durable urgent delivery at Pi's next safe boundary. Never infer attention from status or exit code. Admit steer by exact event and Run generation before `deliverAs: "steer"`; acknowledge only its exact model-bound context envelope, and preserve the later root-terminal completion batch. Sequence, parallel, repeat, and import branches remain internal, while each separately launched Run owns a separate generation.
+Trace is a bounded retained suffix, not an audit archive. Every first-party writer must call `appendRunTraceEvent`; under the canonical token-owned lock it appends within both fixed bounds or atomically retains the newest suffix plus one cumulative warning-only `runtime.trace_compacted` marker. The marker means older history was discarded; terminal/result/execution/artifact files remain authoritative independently. Reads are newline-safe and order equal timestamps by same-source ordinal, fixed source rank, then stable id without exposing ordering metadata. Actor Inspector derives stable source-local row sequences before applying its visible-tail bound, including cumulative compaction offsets for canonical Trace, so appends never renumber retained rows. Never write `trace.jsonl` directly. Generic command lifecycle is Trace-only: `command.done` preserves complete bounded execution evidence but never requests or projects any attention value, and Recipe grammar has no command-completion delivery switch. Semantic attention is explicit: `notify` is visible status, `followup` is an ordinary checkpoint hint, and actor-authored `steer` is durable urgent delivery at Pi's next safe boundary. Never infer attention from status or exit code. Admit steer by exact event and Run generation before `deliverAs: "steer"`; acknowledge only its exact model-bound context envelope, and preserve the later root-terminal completion batch. Sequence, parallel, repeat, and import branches remain internal, while each separately launched Run owns a separate generation.
 
 ### Control
 

package/CHANGELOG.md

@@ -4,6 +4,10 @@
 
 ## Unreleased
 
+## 0.53.2: Stable Inspector Trace Sequences
+
+- `Stable Inspector Trace Sequences`: Actor Inspector now numbers Trace rows from stable source-local chronology before applying the 100-row projection bound. Canonical Trace rows include cumulative compaction offsets and owned Pi turns retain their native sequence, so adding `#100` no longer renumbers the previous `#99`; newest-first ordering, source filters, public Trace JSON, and focus-by-id remain unchanged.
+
 ## 0.53.1: Filterable Skills and drift-safe Git installs
 
 - `Filterable Packaged Skills`: Manifest-loaded runtime leaves all six bundled Skills to `pi.skills`, preserving filters and provenance. Auto-discovered checkouts contribute source Skills even when Pi selects compiled code; resolver regressions cover both modes. Unsupported source manifest aliases are removed, and the committed distributive supports git installs with drift-checking atomic builds.

package/dist/lib/inspector-overlay.js

@@ -583,7 +583,8 @@ export class ActorInspectorOverlay {
                 const prefix = this.focus === "list" && index === this.rowIndex
                     ? this.theme.fg("accent", " ▶ ")
                     : "   ";
-                const row = `${prefix}${this.theme.fg("text", `#${items.length - index - 1}`)} ${marker} ${this.theme.fg("muted", item.source)}/${this.theme.fg(item.level === "error" ? "error" : "accent", item.kind)}  ${this.theme.fg("text", item.summary)}`;
+                const sequence = item[TraceProjection.TRACE_ITEM_SEQUENCE] ?? items.length - index - 1;
+                const row = `${prefix}${this.theme.fg("text", `#${sequence}`)} ${marker} ${this.theme.fg("muted", item.source)}/${this.theme.fg(item.level === "error" ? "error" : "accent", item.kind)}  ${this.theme.fg("text", item.summary)}`;
                 return this.focus === "list" && index === this.rowIndex
                     ? this.theme.fg("accent", row)
                     : row;

package/dist/lib/trace-projection.d.ts

@@ -6,9 +6,11 @@
 import { type RunArtifactDeclaration } from "./runs-artifacts.ts";
 export type TraceSource = "lifecycle" | "control" | "process" | "agent" | "artifact" | "runtime";
 export type TraceSourceFilter = TraceSource | "all";
+export declare const TRACE_ITEM_SEQUENCE: unique symbol;
 export interface TraceItem {
     id: string;
     ts: string;
+    [TRACE_ITEM_SEQUENCE]?: number;
     source: TraceSource;
     kind: string;
     summary: string;

package/dist/lib/trace-projection.js

@@ -14,6 +14,7 @@ import { tailFile } from "./runs-status.js";
 import * as RunsTrace from "./runs-trace.js";
 import * as SessionEvidence from "./session-evidence.js";
 import { readJsonFileResilient, } from "./state-readers.js";
+export const TRACE_ITEM_SEQUENCE = Symbol("trace-item-sequence");
 const LIFECYCLE_PREFIXES = ["run.", "command."];
 const SOURCE_LIMIT = 100;
 const SOURCE_RANK = {
@@ -32,7 +33,8 @@ function sourceForEvent(event) {
         ? "lifecycle"
         : "runtime";
 }
-function ordered(item, sourceKey, ordinal) {
+function ordered(item, sourceKey, ordinal, sequence = ordinal) {
+    Object.defineProperty(item, TRACE_ITEM_SEQUENCE, { value: sequence });
     return { item, ordinal, sourceKey };
 }
 function compareTraceItems(left, right) {
@@ -83,6 +85,8 @@ function controlDiagnosticItems(diagnostics) {
 }
 function traceEventItems(stateDir) {
     const read = RunsTrace.readRunTraceJournal(stateDir);
+    const summary = RunsTrace.summarizeRunTraceJournal(read);
+    const sequenceOffset = summary.dropped_events + Math.max(0, summary.compactions_total - 1);
     const retained = read.events.slice(-SOURCE_LIMIT);
     const projected = retained.map(({ event, ordinal }) => ordered({
         id: event.id,
@@ -95,7 +99,7 @@ function traceEventItems(stateDir) {
             ...(event.attention ? { attention: event.attention } : {}),
             ...(event.data !== undefined ? { data: event.data } : {}),
         }),
-    }, "trace", ordinal));
+    }, "trace", ordinal, sequenceOffset + Math.max(0, ordinal - 1)));
     const diagnostics = diagnosticItems(read.diagnostics).map((item, index) => ordered(item, "trace", read.events.length + index + 1));
     const hasMarker = read.events.some(({ event }) => event.kind === "runtime.trace_compacted");
     if (read.omittedPrefixBytes > 0 && !hasMarker) {
@@ -128,7 +132,7 @@ function controlItems(stateDir) {
             summary: `${value.action} ${value.status}`,
             ...(value.status === "failed" ? { level: "error" } : {}),
             detail: value,
-        }, "control", start + index + 1);
+        }, "control", start + index + 1, start + index);
     });
     return [
         ...projected,
@@ -153,7 +157,7 @@ function agentItems(stateDir) {
     });
 }
 function processItems(stateDir) {
-    return ["stdout", "stderr"].flatMap((stream) => {
+    return ["stdout", "stderr"].flatMap((stream, index) => {
         const path = join(stateDir, `${stream}.log`);
         if (!existsSync(path))
             return [];
@@ -169,7 +173,7 @@ function processItems(stateDir) {
                 summary: `${stream} (${stat.size} bytes)`,
                 ...(stream === "stderr" ? { level: "warning" } : {}),
                 detail: redact({ bytes: stat.size, path, tail }),
-            }, `process:${stream}`, 1)];
+            }, `process:${stream}`, 1, index)];
     });
 }
 function resultItems(stateDir) {
@@ -190,7 +194,7 @@ function resultItems(stateDir) {
             summary: failed ? "Run process failed" : "Run process completed",
             ...(failed ? { level: "error" } : {}),
             detail: redact(read.value),
-        }, "process:result", 1), ...diagnosticItems(read.diagnostics).map((item, index) => ordered(item, "process:result", index + 2))];
+        }, "process:result", 1, 2), ...diagnosticItems(read.diagnostics).map((item, index) => ordered(item, "process:result", index + 2))];
 }
 function artifactItems(stateDir, declarations) {
     const manifest = resolveArtifactManifest(declarations);
@@ -211,7 +215,7 @@ function artifactItems(stateDir, declarations) {
             required: artifact.required,
             ...(artifact.size !== undefined ? { size: artifact.size } : {}),
         }),
-    }, "artifact", index + 1));
+    }, "artifact", index + 1, index));
 }
 export function projectRunTrace(stateDir, options = {}) {
     const source = options.source ?? "all";

package/docs/actor-inspector.md

@@ -25,7 +25,7 @@ Captured Recipe evidence belongs to the Run generation and does not change when
 
 Shows the unified bounded Trace projection. Sources include lifecycle/runtime observations, Controls, owned Pi turns, command-log tails, results, artifacts, and diagnostics. The source selector displays only `all` plus sources present in the current projection from `lifecycle`, `control`, `process`, `agent`, `artifact`, and `runtime`. Select a row to open structured detail.
 
-Trace ordering stays deterministic and newest-first: timestamp descending, same-source physical ordinal descending, fixed internal source rank, then stable id. Internal ordinals are never displayed or interpreted as cross-source causality. Row numbers are zero-based chronological identities even though display is newest-first: the oldest visible event is `#0` and the newest carries the highest number. The summary states whether retained history is complete; `runtime.trace_compacted` means older history was discarded and shows bounded cumulative drop evidence. Terminal/result/execution/artifact evidence keeps its own authority. The projection applies path containment and redaction before rendering.
+Trace ordering stays deterministic and newest-first: timestamp descending, same-source physical ordinal descending, fixed internal source rank, then stable id. Row numbers use stable zero-based source sequences rather than positions inside the visible 100-row tail: when `#100` arrives, the existing `#99` remains `#99`. Canonical journal rows reconstruct their absolute sequence from retained ordinals plus cumulative compaction evidence; owned Pi turns keep their native turn sequence. Sequences are source-local and never imply cross-source causality. The summary states whether retained history is complete; `runtime.trace_compacted` means older history was discarded and shows bounded cumulative drop evidence. Terminal/result/execution/artifact evidence keeps its own authority. The projection applies path containment and redaction before rendering.
 
 ## Control
 

package/lib/inspector-overlay.ts

@@ -653,7 +653,8 @@ export class ActorInspectorOverlay {
       const prefix = this.focus === "list" && index === this.rowIndex
         ? this.theme.fg("accent", " ▶ ")
         : "   ";
-      const row = `${prefix}${this.theme.fg("text", `#${items.length - index - 1}`)} ${marker} ${this.theme.fg("muted", item.source)}/${this.theme.fg(item.level === "error" ? "error" : "accent", item.kind)}  ${this.theme.fg("text", item.summary)}`;
+      const sequence = item[TraceProjection.TRACE_ITEM_SEQUENCE] ?? items.length - index - 1;
+      const row = `${prefix}${this.theme.fg("text", `#${sequence}`)} ${marker} ${this.theme.fg("muted", item.source)}/${this.theme.fg(item.level === "error" ? "error" : "accent", item.kind)}  ${this.theme.fg("text", item.summary)}`;
       return this.focus === "list" && index === this.rowIndex
         ? this.theme.fg("accent", row)
         : row;

package/lib/trace-projection.ts

@@ -34,9 +34,12 @@ export type TraceSource =
 
 export type TraceSourceFilter = TraceSource | "all";
 
+export const TRACE_ITEM_SEQUENCE = Symbol("trace-item-sequence");
+
 export interface TraceItem {
   id: string;
   ts: string;
+  [TRACE_ITEM_SEQUENCE]?: number;
   source: TraceSource;
   kind: string;
   summary: string;
@@ -75,7 +78,9 @@ function ordered(
   item: TraceItem,
   sourceKey: string,
   ordinal: number,
+  sequence = ordinal,
 ): OrderedTraceItem {
+  Object.defineProperty(item, TRACE_ITEM_SEQUENCE, { value: sequence });
   return { item, ordinal, sourceKey };
 }
 
@@ -133,6 +138,8 @@ function controlDiagnosticItems(
 
 function traceEventItems(stateDir: string): OrderedTraceItem[] {
   const read = RunsTrace.readRunTraceJournal(stateDir);
+  const summary = RunsTrace.summarizeRunTraceJournal(read);
+  const sequenceOffset = summary.dropped_events + Math.max(0, summary.compactions_total - 1);
   const retained = read.events.slice(-SOURCE_LIMIT);
   const projected = retained.map(({ event, ordinal }) => ordered({
     id: event.id,
@@ -145,7 +152,7 @@ function traceEventItems(stateDir: string): OrderedTraceItem[] {
       ...(event.attention ? { attention: event.attention } : {}),
       ...(event.data !== undefined ? { data: event.data } : {}),
     }),
-  }, "trace", ordinal));
+  }, "trace", ordinal, sequenceOffset + Math.max(0, ordinal - 1)));
   const diagnostics = diagnosticItems(read.diagnostics).map((item, index) =>
     ordered(item, "trace", read.events.length + index + 1)
   );
@@ -183,7 +190,7 @@ function controlItems(stateDir: string): OrderedTraceItem[] {
       summary: `${value.action} ${value.status}`,
       ...(value.status === "failed" ? { level: "error" as const } : {}),
       detail: value,
-    }, "control", start + index + 1);
+    }, "control", start + index + 1, start + index);
   });
   return [
     ...projected,
@@ -212,7 +219,7 @@ function agentItems(stateDir: string): OrderedTraceItem[] {
 }
 
 function processItems(stateDir: string): OrderedTraceItem[] {
-  return ["stdout", "stderr"].flatMap((stream) => {
+  return ["stdout", "stderr"].flatMap((stream, index) => {
     const path = join(stateDir, `${stream}.log`);
     if (!existsSync(path)) return [];
     const stat = statSync(path);
@@ -226,7 +233,7 @@ function processItems(stateDir: string): OrderedTraceItem[] {
       summary: `${stream} (${stat.size} bytes)`,
       ...(stream === "stderr" ? { level: "warning" as const } : {}),
       detail: redact({ bytes: stat.size, path, tail }),
-    }, `process:${stream}`, 1)];
+    }, `process:${stream}`, 1, index)];
   });
 }
 
@@ -250,7 +257,7 @@ function resultItems(stateDir: string): OrderedTraceItem[] {
     summary: failed ? "Run process failed" : "Run process completed",
     ...(failed ? { level: "error" as const } : {}),
     detail: redact(read.value),
-  }, "process:result", 1), ...diagnosticItems(read.diagnostics).map(
+  }, "process:result", 1, 2), ...diagnosticItems(read.diagnostics).map(
     (item, index) => ordered(item, "process:result", index + 2),
   )];
 }
@@ -277,7 +284,7 @@ function artifactItems(
         required: artifact.required,
         ...(artifact.size !== undefined ? { size: artifact.size } : {}),
       }),
-    }, "artifact", index + 1),
+    }, "artifact", index + 1, index),
   );
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llblab/pi-actors",
-  "version": "0.53.1",
+  "version": "0.53.2",
   "private": false,
   "description": "Local Actor Kernel for Pi",
   "keywords": [