@quandev104/pi-style 0.1.5 → 0.1.7

package/extension-src/pi-style/pi/index.ts

@@ -1,6 +1,14 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import type { StatusSnapshot } from "../domain/status.js";
 import { closeActiveBatch } from "../features/tools/boxed/batch.js";
+import {
+	beginAgentRun,
+	finishAgentRun,
+	invalidateTurnMembers,
+	rebuildTurnRegistryFromEntries,
+	registerTurnFromMessage,
+} from "../features/tools/boxed/turn-summary.js";
+import { requestToolPresentationRender } from "../features/tools/index.js";
 import { registerPiStyleCommand } from "./commands.js";
 import { type CompatibilityTestHooks, createPiStyleSessionCoordinator } from "./session-coordinator.js";
 import { usageFromSession } from "./session-usage.js";
@@ -40,8 +48,9 @@ export function __setCompatibilityTestHooks(hooks: CompatibilityTestHooks): () =
 
 /** Thin Pi adapter: register flags, commands, and forward lifecycle events. */
 export default function piStyleExtension(pi: ExtensionAPI): void {
-	// The core/message/tool surfaces are default-on (safe: fingerprint-certified against
-	// exact Pi 0.83.0, fail-closed elsewhere, conflict-preserving). The OFF switch is the
+	// The core/message/tool surfaces are default-on (identity-certified per surface by
+	// name/arity/source fingerprint, graceful native fallback for any surface whose
+	// runtime identity is not recorded, conflict-preserving). The OFF switch is the
 	// product gate `compatibility.allowCorePatches: false` (or `enabled: false`) in config.
 	for (const [name, description] of [
 		["pi-style-core-patches", "Enable pi-style message/tool core patches"],
@@ -65,7 +74,12 @@ export default function piStyleExtension(pi: ExtensionAPI): void {
 		}
 		await coordinator.start(event, ctx);
 	});
-	pi.on("agent_start", () => coordinator.app.runtime.current?.dismissStartup());
+	pi.on("agent_start", () => {
+		coordinator.app.runtime.current?.dismissStartup();
+		// Turn summary (ADR 0007): a summary group spans the whole agent run
+		// (user request → agent_end), not pi's per-message turn_end.
+		beginAgentRun();
+	});
 	pi.on("input", (event, _ctx) => {
 		coordinator.app.runtime.current?.dismissStartup();
 		// Bare `!`/`!!` submit guard: Pi treats `!`-prefixed input as a direct bash
@@ -110,9 +124,30 @@ export default function piStyleExtension(pi: ExtensionAPI): void {
 	// message/turn boundaries, mirroring Pi's native footer; per-chunk updates
 	// stay usage-free to keep streaming cheap.
 	pi.on("message_end", (_event, ctx) => coordinator.app.update({ ...usagePatch(ctx) }, "coalesced"));
-	pi.on("turn_end", (_event, ctx) => coordinator.app.update({ ...usagePatch(ctx) }, "deferred"));
+	pi.on("turn_end", (event, ctx) => {
+		// Append the finalized assistant message's tool batch to the current run.
+		registerTurnFromMessage(event.message, event.toolResults);
+		coordinator.app.update({ ...usagePatch(ctx) }, "deferred");
+	});
+	pi.on("agent_end", () => {
+		// The run is complete: collapse its tool blocks into one summary line.
+		// Pi only re-invokes the tool renderer selectors from updateDisplay(), so
+		// the captured per-block invalidate callbacks force the collapse and the
+		// captured Tui repaints. Interrupted runs (a call without a result) stay
+		// expanded.
+		const run = finishAgentRun();
+		if (run) {
+			invalidateTurnMembers(run);
+			requestToolPresentationRender();
+		}
+	});
 	pi.on("agent_settled", (_event, ctx) => coordinator.app.update({ ...usagePatch(ctx) }, "coalesced"));
-	pi.on("session_tree", (_event, ctx) => coordinator.app.update({ ...usagePatch(ctx) }, "deferred"));
+	pi.on("session_tree", (_event, ctx) => {
+		// Rebuild the turn registry from session content so restored/branched
+		// history renders collapsed consistently (no in-process turn_end events).
+		rebuildTurnRegistryFromEntries(ctx.sessionManager.getEntries());
+		coordinator.app.update({ ...usagePatch(ctx) }, "deferred");
+	});
 	pi.on("session_compact", (_event, ctx) => coordinator.app.update({ ...usagePatch(ctx) }, "deferred"));
 	pi.on("tool_result", (event, ctx) => {
 		if (["write", "edit", "bash"].includes(event.toolName)) {

package/extension-src/pi-style/pi/session-coordinator.ts

@@ -13,6 +13,7 @@ import {
 	stopAllElapsedTickers,
 	type ToolsRenderConfig,
 } from "../features/tools/boxed/session-config.js";
+import { rebuildTurnRegistryFromEntries, resetTurnRegistry } from "../features/tools/boxed/turn-summary.js";
 import { createCompatibilityCoordinator } from "./compatibility-coordinator.js";
 import {
 	type CompatibilityCleanupResult,
@@ -175,6 +176,11 @@ export function createPiStyleSessionCoordinator(pi: ExtensionAPI, hooks: Compati
 			resetBatchRegistry();
 			resetGrepRegistry();
 			resetBashTreeRegistry();
+			// Turn summaries (ADR 0007): rebuild the registry from session content so
+			// restored/forked history renders collapsed before the first render pass
+			// (deterministic; no in-process turn_end events needed).
+			resetTurnRegistry();
+			rebuildTurnRegistryFromEntries(ctx.sessionManager.getEntries());
 			// Stop any 1s elapsed re-render ticker left by a tool that was still
 			// running when the session ended.
 			stopAllElapsedTickers();
@@ -247,6 +253,7 @@ export function createPiStyleSessionCoordinator(pi: ExtensionAPI, hooks: Compati
 			resetBatchRegistry();
 			resetGrepRegistry();
 			resetBashTreeRegistry();
+			resetTurnRegistry();
 			stopAllElapsedTickers();
 			app.sessionShutdown();
 			// Tier C prototype patches stay installed across session switches. Pi renders

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quandev104/pi-style",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "A native-layout, cohesive visual style package for Pi.",
   "license": "MIT",
   "type": "module",
@@ -42,18 +42,18 @@
     "check": "npm run typecheck && npm run lint && npm run depcruise && npm run test && npm run build && npm run package:smoke"
   },
   "peerDependencies": {
-    "@earendil-works/pi-agent-core": ">=0.83.0 <0.84.0",
-    "@earendil-works/pi-ai": ">=0.83.0 <0.84.0",
-    "@earendil-works/pi-coding-agent": ">=0.83.0 <0.84.0",
-    "@earendil-works/pi-tui": ">=0.83.0 <0.84.0",
+    "@earendil-works/pi-agent-core": ">=0.83.0 <0.85.0",
+    "@earendil-works/pi-ai": ">=0.83.0 <0.85.0",
+    "@earendil-works/pi-coding-agent": ">=0.83.0 <0.85.0",
+    "@earendil-works/pi-tui": ">=0.83.0 <0.85.0",
     "typebox": ">=1.3.9 <2.0.0"
   },
   "devDependencies": {
     "@biomejs/biome": "^2.5.0",
-    "@earendil-works/pi-agent-core": "^0.83.0",
-    "@earendil-works/pi-ai": "^0.83.0",
-    "@earendil-works/pi-coding-agent": "^0.83.0",
-    "@earendil-works/pi-tui": "^0.83.0",
+    "@earendil-works/pi-agent-core": "^0.84.0",
+    "@earendil-works/pi-ai": "^0.84.0",
+    "@earendil-works/pi-coding-agent": "^0.84.0",
+    "@earendil-works/pi-tui": "^0.84.0",
     "@types/node": "^24.0.0",
     "dependency-cruiser": "^17.4.3",
     "git-cliff": "^2.13.1",