@gotgenes/pi-subagents 7.2.7 → 7.2.8

package/CHANGELOG.md

@@ -5,6 +5,18 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [7.2.8](https://github.com/gotgenes/pi-packages/compare/pi-subagents-v7.2.7...pi-subagents-v7.2.8) (2026-05-25)
+
+
+### Performance Improvements
+
+* remove <inherited_system_prompt> wrapper to maximise KV cache reuse ([#180](https://github.com/gotgenes/pi-packages/issues/180)) ([f35e7b1](https://github.com/gotgenes/pi-packages/commit/f35e7b1b4309f91656677932c201d762c4be5cf3))
+
+
+### Documentation
+
+* **retro:** add retro notes for issue [#206](https://github.com/gotgenes/pi-packages/issues/206) ([f439057](https://github.com/gotgenes/pi-packages/commit/f439057bd42edc018df8ddd94783f8a9c89968e0))
+
 ## [7.2.7](https://github.com/gotgenes/pi-packages/compare/pi-subagents-v7.2.6...pi-subagents-v7.2.7) (2026-05-25)
 
 

package/docs/retro/0206-decompose-show-agent-detail.md

@@ -34,3 +34,34 @@ Test count grew from 21 to 33 (+12 new unit tests for the two exported pure func
 - `buildMenuOptions` extracted cleanly with early-return style (no `let menuOptions` intermediate); the refactored function passes all 5 new unit tests and all 21 existing integration tests.
 - `handleEdit`, `handleDelete`, and `handleReset` are closure-internal; they drop the outer `if (file)` guard since the menu only shows those options when `file` is defined.
 - `buildEjectContent` extracted from `ejectAgent` reduces `ejectAgent` to a thin IO function (~10 lines); no behavior change verified by the existing eject integration tests.
+
+## Stage: Final Retrospective (2026-05-25T12:10:00Z)
+
+### Session summary
+
+Completed full lifecycle — Planning, TDD (3 `refactor:` + 1 `docs:` commits, +12 tests), Ship, and Release (`pi-subagents-v7.2.7`) — in a single session with no user corrections.
+
+### Observations
+
+#### What went well
+
+- The three-stage workflow (plan → TDD → ship) executed without any user intervention between stages.
+  Each stage's retro notes provided clean context for the next.
+- Pure-function extraction pattern worked cleanly: `buildMenuOptions` and `buildEjectContent` exported for unit testing; `handleEdit`, `handleDelete`, `handleReset` kept as closure-internal, tested via existing integration tests.
+- The existing 21 integration tests caught no regressions across all 3 refactor commits — strong safety net for mechanical extraction.
+
+#### What caused friction (agent side)
+
+- `other` — Passed `newText: null` to the `Edit` tool during TDD step 1, injecting a literal `null` into `agent-config-editor.ts` and corrupting the file.
+  Self-identified immediately via Biome autoformat failure.
+  Impact: one wasted tool round-trip; recovered by rewriting the file with `Write`.
+- `missing-context` — Used `thinking: "auto"` in a `buildEjectContent` test fixture without checking `ThinkingLevel` is `"minimal" | "low" | "medium" | "high" | "xhigh"`.
+  Self-identified by `pnpm run check` post-TDD gate.
+  Impact: one `--amend` fix, no extra commit.
+- `missing-context` — Plan stated "18 tests" as the baseline count; actual baseline was 21.
+  Impact: none — just an inaccurate number in the plan text.
+
+#### What caused friction (user side)
+
+- No user-side friction observed.
+  The user triggered each stage sequentially without needing to redirect or correct.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotgenes/pi-subagents",
-  "version": "7.2.7",
+  "version": "7.2.8",
   "type": "module",
   "exports": {
     ".": "./src/service.ts"

package/src/session/prompts.ts

@@ -74,12 +74,13 @@ You are operating as a sub-agent invoked to handle a specific task.
       : "";
 
     // Place shared/stable content first so the LLM's KV cache can reuse the
-    // inherited prefix across all subagent invocations. The <active_agent> tag
-    // and env block vary per call and are placed after the cacheable prefix.
+    // inherited prefix across all subagent invocations. The parent prompt is
+    // placed verbatim (no wrapper tag) so it forms an identical byte prefix
+    // with the parent session, maximising KV cache hits. The <active_agent>
+    // tag and env block vary per call and are placed after the cached prefix.
     return (
-      "<inherited_system_prompt>\n" +
       identity +
-      "\n</inherited_system_prompt>\n\n" +
+      "\n\n" +
       bridge +
       "\n\n" +
       activeAgentTag +