pi-developer-messages 0.3.2 → 0.3.6

package/README.md

@@ -1,5 +1,7 @@
 # pi-developer-messages
 
+[Pi gallery](https://pi.dev/packages/pi-developer-messages)
+
 `pi-developer-messages` is a Pi extension that builds the provider-ready prompt
 envelope for each turn. It keeps three kinds of instructions in their intended
 roles: provider (system) instructions, developer messages, and project context
@@ -159,8 +161,9 @@ pi-developer-messages.auditEntries = ["developer", "context-user"]
 
 Set it to `[]`, `["developer"]`, or `["context-user"]` to reduce what is
 persisted. Audit entries are stored as one `pi-developer-messages/group` custom
-entry per envelope (skipped when identical to the latest group) and rendered as
-collapsible rows. They only affect the transcript; the model request is
+entry containing only new or changed instructions, rendered as collapsible rows.
+Each instruction is compared with its latest audit on the current session branch,
+including after resume. Changes to one instruction do not repeat the others. They only affect the transcript; the model request is
 unchanged.
 
 ## Troubleshooting
@@ -194,4 +197,3 @@ unchanged.
 Source: https://github.com/luan/agents, directory
 harnesses/pi/agent/packages/pi-developer-messages. Run `bun run typecheck` and
 `bun test test` in that directory.
-

package/node_modules/pi-libtui/README.md

@@ -1,5 +1,7 @@
 # pi-libtui
 
+[Pi gallery](https://pi.dev/packages/pi-libtui)
+
 Shared terminal UI for Pi extensions: layouts, split panes, dialogs, pickers,
 selection actions, semantic colors, icons, cursors, syntax highlighting,
 animated tool surfaces, streamed output, diffs, terminal projection, and the

package/node_modules/pi-xsettings/README.md

@@ -1,5 +1,7 @@
 # pi-xsettings
 
+[Pi gallery](https://pi.dev/packages/pi-xsettings)
+
 `pi-xsettings` is a settings host for Pi extensions. It adds the
 `/xsettings` editor to Pi's interactive TUI, persists Pi and extension settings
 in one `xsettings.toml` file, exposes a UI-free SDK

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-developer-messages",
-  "version": "0.3.2",
+  "version": "0.3.6",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/audit-entries.ts

@@ -58,9 +58,10 @@ export function publishPromptAuditEntries(
 		for (const message of envelope.contextualUserMessages)
 			auditEntries.push({ role: "user", id: message.id, content: message.content });
 	}
-	if (auditEntries.length === 0 || isCurrentAuditGroup(entries, auditEntries)) return;
+	const changedEntries = auditEntries.filter((item) => !isCurrentAuditEntry(entries, item));
+	if (changedEntries.length === 0) return;
 	try {
-		pi.appendEntry(PROMPT_AUDIT_GROUP_ENTRY_TYPE, { entries: auditEntries });
+		pi.appendEntry(PROMPT_AUDIT_GROUP_ENTRY_TYPE, { entries: changedEntries });
 	} catch {
 		// Audit persistence must not change the model request.
 	}
@@ -84,24 +85,20 @@ function isPromptAuditType(customType: string | undefined): boolean {
 	);
 }
 
-function isCurrentAuditGroup(entries: readonly SessionEntryLike[], expected: readonly PromptAuditData[]): boolean {
+function isCurrentAuditEntry(entries: readonly SessionEntryLike[], expected: PromptAuditData): boolean {
 	for (let index = entries.length - 1; index >= 0; index -= 1) {
 		const entry = entries[index];
-		if (entry?.type !== "custom" || entry.customType !== PROMPT_AUDIT_GROUP_ENTRY_TYPE) continue;
-		const data = promptAuditGroupData(entry.data);
-		if (!data) continue;
-		return (
-			data.entries.length === expected.length &&
-			data.entries.every((item, itemIndex) => sameAuditData(item, expected[itemIndex]))
-		);
+		if (entry?.type !== "custom" || !isPromptAuditType(entry.customType)) continue;
+		const candidates =
+			entry.customType === PROMPT_AUDIT_GROUP_ENTRY_TYPE
+				? (promptAuditGroupData(entry.data)?.entries ?? [])
+				: [promptAuditData(entry.data)];
+		const previous = candidates.find((item) => item?.role === expected.role && item.id === expected.id);
+		if (previous) return previous.content === expected.content;
 	}
 	return false;
 }
 
-function sameAuditData(left: PromptAuditData, right: PromptAuditData | undefined): boolean {
-	return right !== undefined && left.role === right.role && left.id === right.id && left.content === right.content;
-}
-
 function renderAuditRow(data: PromptAuditData, theme: Theme, expanded: boolean): Component {
 	const body = new MarkdownText({ theme, text: data.content });
 	return new ToolActivity({

package/test/extension.test.ts

@@ -383,6 +383,39 @@ function stripAnsi(value: string): string {
 	return value.replace(/\x1B\[[0-?]*[ -/]*[@-~]/g, "");
 }
 
+test("audits only new or changed instructions across turns and reloads", () => {
+	const harness = extensionHarness();
+	const beforeStart = harness.handlers.get("before_agent_start");
+	if (!beforeStart) throw new Error("before_agent_start was not registered");
+	const context = harness.context("audit-deltas");
+	const event = startEvent("Pi base", "/repo", [{ path: "/repo/AGENTS.md", content: "Rules." }]);
+	registerDeveloperMessageContribution({ id: "skills", content: "Catalogue A." });
+	beforeStart(event, context);
+	const removeAnnotation = registerDeveloperMessageContribution({ id: "annotation", content: "Annotation guidance." });
+	beforeStart(event, context);
+	removeAnnotation();
+	beforeStart(event, context);
+	registerDeveloperMessageContribution({ id: "annotation", content: "Annotation guidance." });
+	beforeStart(event, context);
+	registerDeveloperMessageContribution({ id: "skills", content: "Catalogue B." });
+	beforeStart(event, context);
+
+	expect(harness.auditEntries.map(({ data }) => data.entries)).toEqual([
+		[
+			{ role: "developer", id: "skills", content: "Catalogue A." },
+			{ role: "developer", id: "environment", content: expect.stringContaining("<environment_context>") },
+			{ role: "user", id: "agents-md", content: expect.stringContaining("Rules.") },
+		],
+		[{ role: "developer", id: "annotation", content: "Annotation guidance." }],
+		[{ role: "developer", id: "skills", content: "Catalogue B." }],
+	]);
+	const resumed = extensionHarness();
+	const resumedStart = resumed.handlers.get("before_agent_start");
+	if (!resumedStart) throw new Error("before_agent_start was not registered");
+	resumedStart(event, context);
+	expect(resumed.auditEntries).toEqual([]);
+});
+
 test("records a prompt value again after it changes away and back", () => {
 	const harness = extensionHarness();
 	const beforeStart = harness.handlers.get("before_agent_start");