@gotgenes/pi-subagents 7.3.2 → 7.4.0

package/CHANGELOG.md

@@ -5,6 +5,19 @@ 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.4.0](https://github.com/gotgenes/pi-packages/compare/pi-subagents-v7.3.2...pi-subagents-v7.4.0) (2026-05-25)
+
+
+### Features
+
+* add model attribution to formatAssistantMessage ([76f2ada](https://github.com/gotgenes/pi-packages/commit/76f2adaa6bad9d1a3b15a3ba208b3ab96e07ecd1))
+* add model attribution to getAgentConversation ([c186c37](https://github.com/gotgenes/pi-packages/commit/c186c370b975e5ef6596d9a3d7719c720a580640))
+
+
+### Documentation
+
+* **retro:** add retro notes for issue [#214](https://github.com/gotgenes/pi-packages/issues/214) ([7e39c96](https://github.com/gotgenes/pi-packages/commit/7e39c96563517661c1c8c7f250402115b232a097))
+
 ## [7.3.2](https://github.com/gotgenes/pi-packages/compare/pi-subagents-v7.3.1...pi-subagents-v7.3.2) (2026-05-25)
 
 

package/docs/retro/0214-convert-remaining-closure-factories-to-classes.md

@@ -38,3 +38,29 @@ Test count held at 913 (57 files) — no new tests needed, no tests removed.
 - Adding the `SpawnOptions` import to `service-adapter.ts` was required for the `spawn` method signature; the plan anticipated this correctly.
 - The `sed -i` command required the macOS `-i ''` form (no in-place backup extension) rather than the GNU `sed -i` form.
 - Dead-code gate (`pnpm fallow dead-code`) passed cleanly from the repo root — no suppression needed.
+
+## Stage: Final Retrospective (2026-05-25T22:00:00Z)
+
+### Session summary
+
+Shipped `pi-subagents-v7.3.2` with 3 refactor commits converting all remaining closure factories to classes.
+All 4 lifecycle stages (plan → TDD → ship → retro) completed in a single day with zero rework and zero deviations from the plan.
+
+### Observations
+
+#### What went well
+
+- Strong precedent from Phase 11 (#195, #196) made this issue zero-friction — the plan, implementation, and test updates all followed an established template.
+- The plan's prediction of a `makeWizard(deps)` helper for `agent-creation-wizard.test.ts` kept the step-2 diff readable by centralizing 14 inline constructor calls.
+- `SubagentsServiceAdapter implements SubagentsService` gave compile-time contract verification, catching any interface drift immediately via `pnpm run check`.
+- The plan correctly anticipated the `SpawnOptions` import need in `service-adapter.ts`.
+
+#### What caused friction (agent side)
+
+- None.
+  This was a textbook mechanical refactoring with no behavioral changes, no edge cases, and no test rework.
+
+#### What caused friction (user side)
+
+- None.
+  The issue was well-scoped with explicit target files and a clear precedent to follow.

package/package.json

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

package/src/lifecycle/agent-runner.ts

@@ -455,8 +455,9 @@ export function getAgentConversation(session: AgentSession): string {
       if (text.trim()) parts.push(`[User]: ${text.trim()}`);
     } else if (msg.role === "assistant") {
       const { textParts, toolNames } = extractAssistantContent(msg.content);
+      const attribution = formatAttribution(msg);
       if (textParts.length > 0)
-        parts.push(`[Assistant]: ${textParts.join("\n")}`);
+        parts.push(`[Assistant${attribution}]: ${textParts.join("\n")}`);
       if (toolNames.length > 0)
         parts.push(`[Tool Calls]:\n${toolNames.map((n) => `  Tool: ${n}`).join("\n")}`);
     } else if (msg.role === "toolResult") {
@@ -468,3 +469,11 @@ export function getAgentConversation(session: AgentSession): string {
 
   return parts.join("\n\n");
 }
+
+/** Build a `(provider/model)` attribution suffix for assistant messages. */
+function formatAttribution(msg: { provider?: string; model?: string }): string {
+  const { provider, model } = msg;
+  if (!provider && !model) return "";
+  if (provider && model) return ` (${provider}/${model})`;
+  return ` (${provider ?? model})`;
+}

package/src/ui/message-formatters.ts

@@ -116,20 +116,31 @@ export function formatToolResult(
   ];
 }
 
+// ── Types ─────────────────────────────────────────────────────────────────────
+
+/** Model/provider attribution for assistant messages. */
+export interface MessageAttribution {
+  provider?: string;
+  model?: string;
+}
+
 // ── formatAssistantMessage ───────────────────────────────────────────────────
 
 /**
  * Format an assistant message into display lines.
  * Always returns at least the [Assistant] header line.
+ * When attribution is provided, renders as `[Assistant (provider/model)]`.
  */
 export function formatAssistantMessage(
   content: { type: string; [key: string]: unknown }[],
   width: number,
   ctx: FormatterContext,
+  attribution?: MessageAttribution,
 ): string[] {
   const { theme, wrapText } = ctx;
   const { textParts, toolNames } = extractAssistantContent(content);
-  const lines: string[] = [theme.bold("[Assistant]")];
+  const label = formatAttributionLabel(attribution);
+  const lines: string[] = [theme.bold(`[Assistant${label}]`)];
   if (textParts.length > 0) {
     lines.push(...wrapText(textParts.join("\n").trim(), width));
   }
@@ -154,10 +165,15 @@ export function formatMessage(
     return formatUserMessage(msg.content as string | unknown[], width, ctx);
   }
   if (msg.role === "assistant") {
+    const attribution: MessageAttribution = {
+      provider: msg.provider as string | undefined,
+      model: msg.model as string | undefined,
+    };
     return formatAssistantMessage(
       msg.content as { type: string; [key: string]: unknown }[],
       width,
       ctx,
+      attribution,
     );
   }
   if (msg.role === "toolResult") {
@@ -168,3 +184,12 @@ export function formatMessage(
   }
   return null;
 }
+
+// ── Helpers ──────────────────────────────────────────────────────────────────
+
+/** Build a `(provider/model)` attribution suffix, or empty string when absent. */
+function formatAttributionLabel(attr?: MessageAttribution): string {
+  if (!attr?.provider && !attr?.model) return "";
+  if (attr.provider && attr.model) return ` (${attr.provider}/${attr.model})`;
+  return ` (${attr.provider ?? attr.model})`;
+}