@saifymatteo/pi-extensions 0.1.4 → 0.1.6

package/packages/pi-codegraph/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-codegraph",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -11,6 +11,7 @@
     "src",
     "examples",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-compact-tools/CHANGELOG.md

@@ -0,0 +1,24 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.5]
+
+### Fixed
+
+- Keep new-request thinking and activity out of earlier tool cards, including steering messages received before session history is updated.
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-compact-tools/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-compact-tools",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Groups consecutive tool calls into compact expandable transcript cards.",
   "type": "module",
   "license": "AGPL-3.0-only",
@@ -11,6 +11,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-compact-tools/src/index.ts

@@ -455,7 +455,7 @@ function nestedSkills(result: SkillResult, toolName: string | undefined): string
 // Anything visible between two calls (user/custom message, assistant text)
 // starts a new group; tool results and thinking do not. A call seen twice
 // (the in-flight copy of an already persisted message) keeps its first group.
-function buildGroups(messages: Msg[]): { groupOf: Map<string, Group>; last?: Group } {
+function buildGroups(messages: Msg[]): { groupOf: Map<string, Group>; last?: Group; active?: Group } {
 	const groupOf = new Map<string, Group>();
 	let current: Group | undefined;
 	let last: Group | undefined;
@@ -488,7 +488,9 @@ function buildGroups(messages: Msg[]): { groupOf: Map<string, Group>; last?: Gro
 			last = current;
 		}
 	}
-	return { groupOf, last };
+	// Historical cards remain addressable, but only an unbroken trailing run
+	// can own live activity. A user message or assistant text ends that run.
+	return { groupOf, last, active: current };
 }
 
 // [verb, singular, plural]; tools sharing verb+noun are counted together.
@@ -680,9 +682,9 @@ function markToolContexts(): void {
 export default function (pi: ExtensionAPI) {
 	let retired = false;
 	let sessionManager: ExtensionContext["sessionManager"] | undefined;
-	let live: Msg | undefined; // in-flight assistant message, maybe not persisted yet
+	let live: Msg | undefined; // latest user boundary or in-flight assistant, maybe not persisted yet
 	let running = false;
-	let cache: { key: string; groupOf: Map<string, Group>; last?: Group } | undefined;
+	let cache: (ReturnType<typeof buildGroups> & { key: string }) | undefined;
 	const invalidators = new Map<string, () => void>(); // toolCallId -> row redraw
 	// Native expanded belongs to Ctrl+O; pointer disclosure is local to the card.
 	const openHeads = new Set<string>();
@@ -705,7 +707,7 @@ export default function (pi: ExtensionAPI) {
 	}
 
 	function syncActivity() {
-		const head = running && !latestThought(live) ? groups().last?.head : undefined;
+		const head = running && !latestThought(live) ? groups().active?.head : undefined;
 		if (!head || !invalidators.has(head)) { stopActivity(); return; }
 		if (activityTimer !== undefined) return;
 		// One timer for the current parent, never one per row or historical card.
@@ -713,7 +715,7 @@ export default function (pi: ExtensionAPI) {
 			syncActivity();
 			if (activityTimer === undefined) return;
 			activityFrame = (activityFrame + 1) % activityFrames.length;
-			invalidators.get(groups().last!.head)?.();
+			invalidators.get(groups().active!.head)?.();
 		}, 160);
 	}
 
@@ -829,7 +831,15 @@ export default function (pi: ExtensionAPI) {
 		if (event.message.role === "assistant") {
 			live = event.message as Msg;
 			syncActivity();
-		} else refresh();
+		} else {
+			if (event.message.role === "user") {
+				// message_start precedes persistence: do not wait for getBranch()
+				// to contain this boundary, especially for steering mid-run.
+				live = event.message as Msg;
+				cache = undefined;
+			}
+			refresh();
+		}
 	});
 	pi.on("message_update", (event) => {
 		if (event.message.role !== "assistant") return;
@@ -848,7 +858,7 @@ export default function (pi: ExtensionAPI) {
 		}
 		// nf-fa-caret_down and nf-fa-caret_right
 		let text = theme.fg("warning", theme.bold(`${openHeads.has(group.head) ? "\uf0d7" : "\uf0da"} ${summarize(names, images) || "Skill instructions"}`));
-		if (running && group === groups().last) {
+		if (running && group === groups().active) {
 			const thought = latestThought(live);
 			// nf-md-thought_bubble_outline
 			text += theme.fg("dim", thought ? ` \udb81\udff7 ${thought}` : ` ${activityFrames[activityFrame]}`);

package/packages/pi-compact-tools/test/behavior.test.mjs

@@ -0,0 +1,131 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import * as tui from '@earendil-works/pi-tui';
+import { loadExtension } from '../../../test/load-extension.mjs';
+
+const theme = { fg: (_token, text) => text, bold: text => text };
+const assistant = content => ({ role: 'assistant', content });
+const call = id => ({ type: 'toolCall', id, name: 'bash', arguments: { command: id } });
+const user = text => ({ role: 'user', content: text });
+
+function harness(t) {
+  const hooks = {};
+  const branch = [];
+  let resolver;
+  let activity;
+  const invalidated = [];
+  const extension = loadExtension(new URL('../src/index.ts', import.meta.url), {
+    '@earendil-works/pi-coding-agent': {
+      AssistantMessageComponent: class { updateContent() {} },
+      ToolExecutionComponent: class {},
+      keyHint: (_key, label) => `Ctrl+O ${label}`,
+    },
+    '@earendil-works/pi-tui': tui,
+  }, {
+    // No wall-clock animation needed to check card ownership.
+    setInterval: fn => { activity = fn; return 1; },
+    clearInterval: () => { activity = undefined; },
+  });
+  extension.default({
+    on: (name, handler) => { hooks[name] = handler; },
+    registerToolRenderer: fn => { resolver = fn; },
+    getSettings: () => ({}),
+  });
+  const emit = (name, message) => hooks[name]?.({ message });
+  hooks.session_start({}, { sessionManager: {
+    getSessionId: () => 'test',
+    getLeafId: () => branch.at(-1)?.id,
+    getBranch: () => branch,
+  } });
+  t.after(() => emit('session_shutdown'));
+  const persist = message => branch.push({ type: 'message', id: String(branch.length), message });
+  const render = id => resolver('bash', () => undefined).renderCall({ command: id }, theme, {
+    toolCallId: id, [Symbol.for('compact-tools.tuiContext')]: true,
+    expanded: false, isPartial: false, invalidate: () => invalidated.push(id),
+  }).render(90).join('\n');
+  return { emit, persist, render, invalidated, tick: () => activity?.() };
+}
+
+test('new request thinking does not appear in the previous tool card', t => {
+  const h = harness(t);
+  h.persist(user('first request'));
+  h.persist(assistant([call('old')]));
+  h.emit('agent_end');
+  const before = h.render('old');
+  assert.match(before, /Ran 1 command/);
+
+  h.emit('agent_start');
+  h.emit('message_start', user('second request'));
+  h.persist(user('second request'));
+  h.emit('message_start', assistant([]));
+  h.emit('message_update', assistant([{ type: 'thinking', thinking: 'NEW REQUEST THOUGHT' }]));
+  assert.equal(h.render('old'), before, 'previous card must not show new request activity');
+});
+
+test('new tool calls stay below the user boundary, live and persisted', t => {
+  const h = harness(t);
+  h.persist(user('first request'));
+  h.persist(assistant([call('old')]));
+  const before = h.render('old');
+  h.emit('agent_start');
+  h.emit('message_start', user('second request'));
+  h.persist(user('second request'));
+  h.emit('message_start', assistant([]));
+  h.emit('message_update', assistant([call('new')]));
+  assert.match(h.render('new'), /Ran 1 command/);
+  assert.equal(h.render('old'), before);
+  h.persist(assistant([call('new')]));
+  assert.match(h.render('new'), /Ran 1 command/, 'persisted/live duplicate must not count twice');
+  h.emit('agent_end');
+  assert.match(h.render('new'), /Ran 1 command/);
+  assert.equal(h.render('old'), before);
+});
+
+test('a steering user boundary stops previous card activity before persistence', t => {
+  const h = harness(t);
+  h.persist(user('first request'));
+  h.persist(assistant([call('old')]));
+  const before = h.render('old');
+  h.emit('agent_start');
+  h.emit('message_start', assistant([{ type: 'thinking', thinking: 'OLD THOUGHT' }]));
+  assert.match(h.render('old'), /OLD THOUGHT/);
+  h.emit('message_start', user('steering request'));
+  assert.equal(h.render('old'), before);
+  h.invalidated.length = 0;
+  h.tick();
+  assert.equal(h.invalidated.length, 0, 'old card animation stops at the pending boundary');
+  h.persist(user('steering request'));
+  h.emit('message_start', assistant([]));
+  h.emit('message_update', assistant([call('new')]));
+  assert.match(h.render('new'), /Ran 1 command/);
+  assert.equal(h.render('old'), before);
+  h.invalidated.length = 0;
+  h.tick();
+  assert.deepEqual(h.invalidated, ['new'], 'animation now belongs to the new card');
+});
+
+test('assistant text ends the active card while thinking alone preserves it', t => {
+  const h = harness(t);
+  h.persist(user('request'));
+  h.persist(assistant([call('first')]));
+  const before = h.render('first');
+  h.emit('agent_start');
+  h.emit('message_start', assistant([]));
+  h.emit('message_update', assistant([{ type: 'thinking', thinking: 'SAME REQUEST' }]));
+  assert.match(h.render('first'), /SAME REQUEST/);
+  h.emit('message_update', assistant([
+    { type: 'thinking', thinking: 'SAME REQUEST' },
+    { type: 'text', text: 'Visible response' },
+  ]));
+  assert.equal(h.render('first'), before);
+});
+
+test('consecutive tool-only assistant messages still share one card', t => {
+  const h = harness(t);
+  h.persist(user('request'));
+  h.persist(assistant([call('first')]));
+  h.emit('message_start', assistant([]));
+  h.emit('message_update', assistant([call('second')]));
+  assert.match(h.render('first'), /Ran 2 commands/);
+  assert.equal(h.render('second'), '');
+});

package/packages/pi-decisions/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-decisions/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-decisions",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -11,6 +11,7 @@
     "src",
     "examples",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-load-pi-dotenv/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-load-pi-dotenv/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-load-pi-dotenv",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Loads environment variables from the Pi agent directory without replacing existing environment values.",
   "type": "module",
   "license": "AGPL-3.0-only",
@@ -11,6 +11,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-model-go/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-model-go/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-model-go",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-prestart-working/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-prestart-working/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-prestart-working",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Show a labelled Working spinner while before_agent_start hooks run.",
   "type": "module",
   "license": "AGPL-3.0-only",
@@ -11,6 +11,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-root-guard/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-root-guard/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-root-guard",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Blocks selected write/edit and simple bash writes into Pi and Claude configuration roots.",
   "type": "module",
   "license": "AGPL-3.0-only",
@@ -11,6 +11,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-session-finder/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-session-finder/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-session-finder",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-session-id/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-session-id/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-session-id",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-session-nav/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-session-nav/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-session-nav",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-session-rename/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-session-rename/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-session-rename",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-session-retention/CHANGELOG.md

@@ -0,0 +1,25 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.5]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.4]
+
+### Fixed
+
+- Schedule automatic maintenance outside the awaited session-start handler; defer launch while the agent is busy.
+- Offer `/session-cleanup` for first approval without interrupting startup, retaining store locking, protected sessions, and shutdown draining.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-session-retention/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-session-retention",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],

package/packages/pi-skill-usage-log/CHANGELOG.md

@@ -0,0 +1,18 @@
+# Changelog
+
+Release notes are recorded from 0.1.3 onward.
+
+## [Unreleased]
+
+## [0.1.4]
+
+### Added
+
+- Include package release notes in Pi 1.5.0's Extension Updates.
+
+## [0.1.3]
+
+### Fixed
+
+- Classify the package as an extension in the Pi package gallery.
+- Show the package screenshot as its gallery preview.

package/packages/pi-skill-usage-log/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saifymatteo/pi-skill-usage-log",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "license": "AGPL-3.0-only",
   "type": "module",
   "keywords": [
@@ -10,6 +10,7 @@
   "files": [
     "src",
     "README.md",
+    "CHANGELOG.md",
     "NOTICE.md",
     "LICENSE"
   ],