@fitchmultz/pi-subagents 0.44.6 → 0.44.7

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.44.7] - 2026-10-09
+
+### Fixed
+- Keep newer Agents history, including consumed human answers, from being overwritten by an older in-flight page. Recheck the owning session, index, selected attempt, source and published page before applying async history or saved-result reads; preserve requested paging and reading anchors when reloading an invalidated snapshot.
+- Consume refresh notifications received while the latest conversation page is loading, without pulling scrolled readers forward or automatically retrying history errors.
+- Emit the Intercom bridge `session_closed` audit record after broker disconnect completes and the closing session is removed; cancellation still rejects pending and new requests for that identity immediately.
+
 ## [0.44.6] - 2026-10-09
 
 ### Fixed

package/dist/extension/build-info.js

@@ -1 +1 @@
-export const EXTENSION_BUILD = Object.freeze({"version":"0.44.6","sha256":"e9a0bfe1bf57b2b67e3b32c6f449996dadf8bc897fe2577eea3d919158b40466"});
+export const EXTENSION_BUILD = Object.freeze({"version":"0.44.7","sha256":"86c62961d20949874e872f49d9044371c4ae3d0b1b18b9d3b1186c29d1522bb0"});

package/dist/pi-intercom/bridge.js

@@ -295,8 +295,8 @@ class Session {
             await this.disconnect();
             if (this.bridge.sessions.get(this.identity.fingerprint256) === this)
                 this.bridge.sessions.delete(this.identity.fingerprint256);
+            log("session_closed", { ...this.fields(), result: error.code });
         })();
-        log("session_closed", { ...this.fields(), result: error.code });
     }
     receive(from, message) {
         if (this.closed)

package/dist/tui/agent-view.js

@@ -315,7 +315,7 @@ export class AgentViewController {
                 activity: { ...prior.child.activity, ...child.activity } } : child;
             return { key, label: child.identityUnavailable ? "Saved assignment unavailable" : sameAttempt ? agentTaskLabel(child) : prior?.label ?? agentTaskLabel(child), run: view, child: displayChild,
                 model: agentModel(displayChild, child.nativeConfiguration ?? (canonical && sameAttempt && child.sessionFile === prior.child.sessionFile ? prior.page?.configuration : undefined)), history: sameAttempt ? prior.history : [], historyIds: sameAttempt ? prior.historyIds : [],
-                page: sameAttempt ? prior.page : undefined, metadataAt: observed ? prior.metadataAt : undefined, historyLoading: sameAttempt ? prior.historyLoading : false, finalId: sameAttempt ? prior.finalId : undefined, unavailable: child.missingSession && child.state !== "live" ? "Saved conversation unavailable." : view.diagnosis,
+                page: sameAttempt ? prior.page : undefined, historySource: sameAttempt ? prior.historySource : undefined, metadataAt: observed ? prior.metadataAt : undefined, historyLoading: sameAttempt ? prior.historyLoading : false, finalId: sameAttempt ? prior.finalId : undefined, unavailable: child.missingSession && child.state !== "live" ? "Saved conversation unavailable." : view.diagnosis,
                 question: child.identityUnavailable ? undefined : questions.findLast((question) => question.index === child.index && ["awaiting_input", "answer_pending"].includes(question.state)),
                 unread: observed ? prior.unread : Boolean(visit && (child.activity?.lastActivityAt ?? view.updatedAt) > (visit.seenActivityAt ?? 0)), replied: sameAttempt ? prior.replied : false };
         });
@@ -485,7 +485,19 @@ export class AgentViewController {
         const task = this.task(key), generation = this.generation;
         if (!task || !this.live() || task.child.identityUnavailable)
             return;
+        const published = task.page, source = task.historySource, sessionFile = task.child.sessionFile, childState = task.child.state;
         const input = this.historyInput(task), index = await runHistoryIndex(this.state);
+        // Versions can reset on index reopen. Fence the actual owner/read baseline,
+        // including metadata-only source transitions, rather than ordering unrelated versions.
+        const isCurrent = () => {
+            const current = this.task(key);
+            if (!this.live(generation) || this.state.historyIndex !== index || index.failure || !current || current.child.identityUnavailable
+                || current.page !== published || current.historySource !== source || current.child.sessionFile !== sessionFile || current.child.state !== childState)
+                return false;
+            const boundary = this.historyInput(current);
+            return boundary.runId === input.runId && boundary.index === input.index && boundary.leaf === input.leaf
+                && boundary.terminalEntryId === input.terminalEntryId && boundary.endedAt === input.endedAt;
+        };
         const boundary = { runId: task.run.runId, index: task.child.index, terminalEntryId: input.terminalEntryId, endedAt: input.endedAt };
         const marker = typeof input.readThrough === "string" && !input.readThrough.startsWith("result:")
             ? await index.entry({ ...boundary, entryId: input.readThrough.replace(/:(?:\d+|error)$/, "") }) : undefined;
@@ -499,7 +511,7 @@ export class AgentViewController {
         const result = await indexedHistory(index, { ...input, limit: 100, ...paging });
         if (!this.live(generation) || this.task(key)?.run.runId !== task.run.runId)
             return;
-        return { ...result, latestPage: paging.after !== undefined ? !result.page.hasMore : paging.before === undefined && paging.cursor === undefined,
+        return { ...result, isCurrent, latestPage: paging.after !== undefined ? !result.page.hasMore : paging.before === undefined && paging.cursor === undefined,
             readThroughSequence: marker?.sequence ?? (input.readThrough == null ? -1 : undefined) };
     }
     async savedResult(key, id) {
@@ -522,6 +534,7 @@ export class AgentViewController {
     }
     applyMetadata(task, page) {
         const visit = this.visits.get(task.key), delivered = new Map(page.deliveredMessages);
+        task.historySource = JSON.stringify([page.sourceId, page.generation, page.sessionId]);
         if (page.freshness.state !== "catching-up")
             task.metadataAt = task.run.updatedAt;
         task.model = agentModel(task.child, page.configuration);
@@ -1009,6 +1022,7 @@ export class AgentConversation extends Container {
     pageRequest = 0;
     detailRequest = 0;
     loadingPage = false;
+    refreshPending = false;
     pageError;
     latestPage = true;
     readSequence = -1;
@@ -1108,6 +1122,7 @@ export class AgentConversation extends Container {
         if (this.closed)
             return;
         const request = ++this.pageRequest;
+        this.refreshPending = false;
         this.loadingPage = true;
         this.pageError = undefined;
         if (this.task)
@@ -1126,11 +1141,8 @@ export class AgentConversation extends Container {
                     result = reportPage;
             }
             let task = this.task;
-            if (result.readThroughSequence !== undefined)
-                this.readSequence = result.readThroughSequence;
-            this.latestPage = result.latestPage;
             let history = result.history;
-            if ((this.initialPosition || this.latestPage || history.finalId && history.entryIds.includes(history.finalId)) && task.child.state !== "live" && task.child.result && getSingleResultOutput(task.child.result)) {
+            if ((this.initialPosition || result.latestPage || history.finalId && history.entryIds.includes(history.finalId)) && task.child.state !== "live" && task.child.result && getSingleResultOutput(task.child.result)) {
                 const report = await this.controller.savedResult(this.key, `result:${task.run.runId}`);
                 const currentTask = this.task;
                 if (this.closed || request !== this.pageRequest || !currentTask || currentTask.run.runId !== task.run.runId)
@@ -1138,6 +1150,13 @@ export class AgentConversation extends Container {
                 task = currentTask;
                 history = withFinalResult(history, report.text, task.run.runId, task.run.updatedAt);
             }
+            if (!result.isCurrent()) {
+                void this.loadPage(paging, anchor);
+                return;
+            }
+            if (result.readThroughSequence !== undefined)
+                this.readSequence = result.readThroughSequence;
+            this.latestPage = result.latestPage;
             task.history = history.items;
             task.historyIds = history.entryIds;
             task.finalId = history.finalId;
@@ -1159,6 +1178,10 @@ export class AgentConversation extends Container {
                 this.loadingPage = false;
                 if (this.task)
                     this.task.historyLoading = false;
+                if (this.refreshPending && !this.pageError) {
+                    this.refreshPending = false;
+                    this.refresh();
+                }
                 this.tui.requestRender();
             }
         }
@@ -1178,8 +1201,12 @@ export class AgentConversation extends Container {
             this.detail = this.assignment();
         if (!this.initialPosition && !this.scroll.isFollowingEnd)
             this.restoreAnchor = this.anchor();
-        if (!this.loadingPage && this.latestPage && this.scroll.isFollowingEnd && !this.detail && !this.pageError)
-            void this.loadPage();
+        if (this.latestPage && this.scroll.isFollowingEnd && !this.detail && !this.pageError) {
+            if (this.loadingPage)
+                this.refreshPending = true;
+            else
+                void this.loadPage();
+        }
         this.tui.requestRender();
     }
     syncDraft() { if (!this.closed && this.editor.getExpandedText() !== this.visit.draft)

package/docs/intercom-bridge.md

@@ -14,7 +14,7 @@ Optional HTTPS messaging proxy for a remote agent such as **Grok Bot**. Pi-inter
 
 **Delivery/lifecycle:** one broker peer per authorized certificate, registered lazily on the first API call, with a five-minute inactivity lease. Poll inbox to maintain presence. Send returns broker acceptance, not proof a model consumed it. Ask steers the recipient and waits at most two minutes for both the exact recipient ID and exact `replyTo`. One outgoing ask per identity. Revocation/expiry/shutdown/client disconnect terminate waits. Inbox is bounded and volatile; overflow is reported, not silently described as successful receipt. Restarting the helper loses inbox contents and pending replies. Upgrade to a durable inbox journal if restart recovery is required; existing Pi recipients still save their own messages normally.
 
-**Audit:** JSONL metadata on stdout: request/connection identity, certificate fingerprint, operation, peer/message IDs and outcome. Message/attachment bodies and keys are not logged. Protect logs (0700 directory, 0600 files); rotate them with your normal log tooling. Never publish private config/keys or raw inbox/list output as smoke evidence.
+**Audit:** JSONL metadata on stdout: request/connection identity, certificate fingerprint, operation, peer/message IDs and outcome. `session_closed` confirms broker disconnection and removal of the retiring session; cancellation still rejects requests immediately. Message/attachment bodies and keys are not logged. Protect logs (0700 directory, 0600 files); rotate them with your normal log tooling. Never publish private config/keys or raw inbox/list output as smoke evidence.
 
 ## Mac setup
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fitchmultz/pi-subagents",
-  "version": "0.44.6",
+  "version": "0.44.7",
   "description": "Pi extension for subagent orchestration and local session coordination",
   "license": "MIT",
   "type": "module",

package/src/pi-intercom/bridge.ts

@@ -287,8 +287,8 @@ class Session {
       try { await this.connecting; } catch { /* The original request receives the connect failure. */ }
       await this.disconnect();
       if (this.bridge.sessions.get(this.identity.fingerprint256) === this) this.bridge.sessions.delete(this.identity.fingerprint256);
+      log("session_closed", { ...this.fields(), result: error.code });
     })();
-    log("session_closed", { ...this.fields(), result: error.code });
   }
 
   private receive(from: SessionInfo, message: Message): void {

package/src/tui/agent-view.ts

@@ -70,6 +70,7 @@ export interface AgentTask {
 	replied: boolean;
 	page?: HistoryPage;
 	historyLoading?: boolean;
+	historySource?: string;
 	metadataAt?: number;
 }
 
@@ -332,7 +333,7 @@ export class AgentViewController {
 				activity: { ...prior.child.activity, ...child.activity } } : child;
 			return { key, label: child.identityUnavailable ? "Saved assignment unavailable" : sameAttempt ? agentTaskLabel(child) : prior?.label ?? agentTaskLabel(child), run: view, child: displayChild,
 				model: agentModel(displayChild, child.nativeConfiguration ?? (canonical && sameAttempt && child.sessionFile === prior.child.sessionFile ? prior.page?.configuration : undefined)), history: sameAttempt ? prior.history : [], historyIds: sameAttempt ? prior.historyIds : [],
-				page: sameAttempt ? prior.page : undefined, metadataAt: observed ? prior.metadataAt : undefined, historyLoading: sameAttempt ? prior.historyLoading : false, finalId: sameAttempt ? prior.finalId : undefined, unavailable: child.missingSession && child.state !== "live" ? "Saved conversation unavailable." : view.diagnosis,
+				page: sameAttempt ? prior.page : undefined, historySource: sameAttempt ? prior.historySource : undefined, metadataAt: observed ? prior.metadataAt : undefined, historyLoading: sameAttempt ? prior.historyLoading : false, finalId: sameAttempt ? prior.finalId : undefined, unavailable: child.missingSession && child.state !== "live" ? "Saved conversation unavailable." : view.diagnosis,
 				question: child.identityUnavailable ? undefined : questions.findLast((question) => question.index === child.index && ["awaiting_input", "answer_pending"].includes(question.state)),
 				unread: observed ? prior.unread : Boolean(visit && (child.activity?.lastActivityAt ?? view.updatedAt) > (visit.seenActivityAt ?? 0)), replied: sameAttempt ? prior.replied : false };
 		});
@@ -457,10 +458,21 @@ export class AgentViewController {
 		if (this.live(generation)) await this.refresh();
 	}
 
-	async historyPage(key: string, paging: Pick<HistoryPageInput, "before" | "after" | "cursor"> = {}, anchor?: string | null): Promise<{ history: AgentHistory; page: HistoryPage; latestPage: boolean; readThroughSequence?: number } | undefined> {
+	async historyPage(key: string, paging: Pick<HistoryPageInput, "before" | "after" | "cursor"> = {}, anchor?: string | null): Promise<{ history: AgentHistory; page: HistoryPage; latestPage: boolean; readThroughSequence?: number; isCurrent: () => boolean } | undefined> {
 		const task = this.task(key), generation = this.generation;
 		if (!task || !this.live() || task.child.identityUnavailable) return;
+		const published = task.page, source = task.historySource, sessionFile = task.child.sessionFile, childState = task.child.state;
 		const input = this.historyInput(task), index = await runHistoryIndex(this.state);
+		// Versions can reset on index reopen. Fence the actual owner/read baseline,
+		// including metadata-only source transitions, rather than ordering unrelated versions.
+		const isCurrent = () => {
+			const current = this.task(key);
+			if (!this.live(generation) || this.state.historyIndex !== index || index.failure || !current || current.child.identityUnavailable
+				|| current.page !== published || current.historySource !== source || current.child.sessionFile !== sessionFile || current.child.state !== childState) return false;
+			const boundary = this.historyInput(current);
+			return boundary.runId === input.runId && boundary.index === input.index && boundary.leaf === input.leaf
+				&& boundary.terminalEntryId === input.terminalEntryId && boundary.endedAt === input.endedAt;
+		};
 		const boundary = { runId: task.run.runId, index: task.child.index, terminalEntryId: input.terminalEntryId, endedAt: input.endedAt };
 		const marker = typeof input.readThrough === "string" && !input.readThrough.startsWith("result:")
 			? await index.entry({ ...boundary, entryId: input.readThrough.replace(/:(?:\d+|error)$/, "") }) : undefined;
@@ -471,7 +483,7 @@ export class AgentViewController {
 		}
 		const result = await indexedHistory(index, { ...input, limit: 100, ...paging });
 		if (!this.live(generation) || this.task(key)?.run.runId !== task.run.runId) return;
-		return { ...result, latestPage: paging.after !== undefined ? !result.page.hasMore : paging.before === undefined && paging.cursor === undefined,
+		return { ...result, isCurrent, latestPage: paging.after !== undefined ? !result.page.hasMore : paging.before === undefined && paging.cursor === undefined,
 			readThroughSequence: marker?.sequence ?? (input.readThrough == null ? -1 : undefined) };
 	}
 
@@ -494,6 +506,7 @@ export class AgentViewController {
 
 	applyMetadata(task: AgentTask, page: HistoryPage): void {
 		const visit = this.visits.get(task.key), delivered = new Map(page.deliveredMessages);
+		task.historySource = JSON.stringify([page.sourceId, page.generation, page.sessionId]);
 		if (page.freshness.state !== "catching-up") task.metadataAt = task.run.updatedAt;
 		task.model = agentModel(task.child, page.configuration);
 		task.unread = Boolean(page.unreadAfter || visit && (task.child.activity?.lastActivityAt ?? 0) > (visit.seenActivityAt ?? 0));
@@ -865,6 +878,7 @@ export class AgentConversation extends Container {
 	private pageRequest = 0;
 	private detailRequest = 0;
 	private loadingPage = false;
+	private refreshPending = false;
 	private pageError?: string;
 	private latestPage = true;
 	private readSequence = -1;
@@ -953,6 +967,7 @@ export class AgentConversation extends Container {
 	private async loadPage(paging: Pick<HistoryPageInput, "before" | "after" | "cursor"> = {}, anchor?: string | null): Promise<void> {
 		if (this.closed) return;
 		const request = ++this.pageRequest;
+		this.refreshPending = false;
 		this.loadingPage = true; this.pageError = undefined;
 		if (this.task) this.task.historyLoading = true;
 		this.latestPage = paging.before === undefined && paging.after === undefined && paging.cursor === undefined;
@@ -966,16 +981,17 @@ export class AgentConversation extends Container {
 				if (reportPage) result = reportPage;
 			}
 			let task = this.task;
-			if (result.readThroughSequence !== undefined) this.readSequence = result.readThroughSequence;
-			this.latestPage = result.latestPage;
 			let history = result.history;
-			if ((this.initialPosition || this.latestPage || history.finalId && history.entryIds.includes(history.finalId)) && task.child.state !== "live" && task.child.result && getSingleResultOutput(task.child.result)) {
+			if ((this.initialPosition || result.latestPage || history.finalId && history.entryIds.includes(history.finalId)) && task.child.state !== "live" && task.child.result && getSingleResultOutput(task.child.result)) {
 				const report = await this.controller.savedResult(this.key, `result:${task.run.runId}`);
 				const currentTask = this.task;
 				if (this.closed || request !== this.pageRequest || !currentTask || currentTask.run.runId !== task.run.runId) return;
 				task = currentTask;
 				history = withFinalResult(history, report.text, task.run.runId, task.run.updatedAt);
 			}
+			if (!result.isCurrent()) { void this.loadPage(paging, anchor); return; }
+			if (result.readThroughSequence !== undefined) this.readSequence = result.readThroughSequence;
+			this.latestPage = result.latestPage;
 			task.history = history.items; task.historyIds = history.entryIds; task.finalId = history.finalId; task.page = result.page;
 			for (const item of task.history) if (item.id === `result:${task.run.runId}`) item.load = () => this.controller.savedResult(this.key, item.id);
 			this.seenSource ||= result.page.count > 0;
@@ -985,7 +1001,11 @@ export class AgentConversation extends Container {
 		} catch (error) {
 			if (!this.closed && request === this.pageRequest) this.pageError = `History unavailable: ${error instanceof Error ? error.message : String(error)}. F5 retries the selected run.`;
 		} finally {
-			if (!this.closed && request === this.pageRequest) { this.loadingPage = false; if (this.task) this.task.historyLoading = false; this.tui.requestRender(); }
+			if (!this.closed && request === this.pageRequest) {
+				this.loadingPage = false; if (this.task) this.task.historyLoading = false;
+				if (this.refreshPending && !this.pageError) { this.refreshPending = false; this.refresh(); }
+				this.tui.requestRender();
+			}
 		}
 	}
 
@@ -1001,7 +1021,10 @@ export class AgentConversation extends Container {
 		if (this.closed) return;
 		if (this.detail?.id === "assignment" && this.task) this.detail = this.assignment();
 		if (!this.initialPosition && !this.scroll.isFollowingEnd) this.restoreAnchor = this.anchor();
-		if (!this.loadingPage && this.latestPage && this.scroll.isFollowingEnd && !this.detail && !this.pageError) void this.loadPage();
+		if (this.latestPage && this.scroll.isFollowingEnd && !this.detail && !this.pageError) {
+			if (this.loadingPage) this.refreshPending = true;
+			else void this.loadPage();
+		}
 		this.tui.requestRender();
 	}
 	syncDraft(): void { if (!this.closed && this.editor.getExpandedText() !== this.visit.draft) this.editor.setText(this.visit.draft); }