@gotgenes/pi-subagents 6.16.3 → 6.17.1

package/docs/retro/0147-inject-wrap-text-into-conversation-viewer.md

@@ -0,0 +1,90 @@
+---
+issue: 147
+issue_title: "Inject text wrapping into ConversationViewer (Phase 9, Step O)"
+---
+
+# Retro: #147 — Inject text wrapping into ConversationViewer (Phase 9, Step O)
+
+## Stage: Planning (2026-05-23T00:00:00Z)
+
+### Session summary
+
+Read the issue, loaded package-pi-subagents, code-design, testing, and markdown-conventions skills.
+Explored `src/ui/conversation-viewer.ts`, `src/ui/agent-menu.ts`, `test/conversation-viewer.test.ts`, and the Phase 9 architecture roadmap.
+Wrote and committed the plan at `packages/pi-subagents/docs/plans/0147-inject-wrap-text-into-conversation-viewer.md`.
+
+### Observations
+
+- The change is tightly scoped: two source files (`conversation-viewer.ts`, `agent-menu.ts`) and one test file.
+- `wrapTextWithAnsi` is called in exactly four places inside `buildContentLines` — all in the same private method, making the replacement straightforward.
+- The only production call site for `new ConversationViewer({…})` is `viewAgentConversation` in `agent-menu.ts`.
+  `wrapTextWithAnsi` is added as a static import there and passed as `wrapText` — no threading through `AgentMenuDeps` needed.
+- All `new ConversationViewer({…})` calls in the test file are inline (no shared factory helper), so every call site needs the new `wrapText` field added.
+  Grep confirms the count: 11+ calls, all in `test/conversation-viewer.test.ts`.
+- The plan uses 2 TDD cycles: Cycle 1 adds the field and updates all call sites (with the `vi.mock` still present for safety); Cycle 2 removes the mock and converts dynamic `await import()` to static imports.
+  This ordering avoids a large simultaneous change and gives the suite a stable intermediate state.
+- The "mock is intercepting wrapTextWithAnsi" test is deleted in Cycle 1 (it verified the mock mechanism, not production behavior).
+- No exported API symbols are removed; `wrapText` is a new required field on `ConversationViewerOptions`, which is a breaking change only for external constructors of `ConversationViewer` — confirmed none exist outside this package.
+
+## Stage: Implementation — TDD (2026-05-23T11:36:00Z)
+
+### Session summary
+
+Completed both TDD cycles from the plan.
+Cycle 1 added `wrapText` to `ConversationViewerOptions`, destructured options in the constructor, replaced all four `wrapTextWithAnsi` calls with `this.wrapText`, updated `agent-menu.ts` to import and pass `wrapTextWithAnsi`, and updated all 16 test constructor call sites (11 render-width-safety + 5 safety-net) while keeping the `vi.mock` shim in place.
+Cycle 2 removed the `vi.mock` block, `wrapOverride`, and `beforeEach` reset, then converted the dynamic `await import()` calls to ordinary static imports.
+Test count: 806 → 805 (deleted the mock-mechanism sentinel test).
+Full suite 50 files, 805 tests, all green.
+
+### Observations
+
+- The plan said the render-width-safety constructor calls numbered "11+" — the actual count was 16 total (11 render-width-safety + 5 safety-net), all in `test/conversation-viewer.test.ts`.
+  No external call sites existed.
+- `wrapTextWithAnsi` needed to be added to the dynamic import in Cycle 1 (`const { visibleWidth, wrapTextWithAnsi } = await import(...)`) because the render-width-safety tests reference it by name.
+  The plan didn't call this out explicitly — minor omission.
+- Used a Python script (inline via bash) to make the 17 constructor-call edits rather than 17 separate Edit-tool calls.
+  The safety-net tests each had a different stub character (`X`, `Y`, `Z`, `B`, `W`) which required a regex capture group to preserve.
+  The script worked on the first attempt.
+- Cycle 2 was a single Edit call replacing the entire mock block + the two dynamic imports + `beforeEach`.
+  The autoformatter then cleaned up import ordering automatically.
+- Architecture doc updated: smells table row struck-through and Step O marked ✓.
+
+## Stage: Final Retrospective (2026-05-23T11:45:00Z)
+
+### Session summary
+
+Planned, implemented, shipped, and released issue #147 across a single session chain.
+Two TDD cycles completed cleanly; CI passed; issue closed; `pi-subagents-v6.17.0` released.
+Test count: 806 → 805 (deleted mock-mechanism sentinel test).
+
+### Observations
+
+#### What went well
+
+- Python script for bulk constructor edits worked first-try on 17 call sites with per-test regex capture groups.
+  This is now a proven pattern — #116 and #147 both used it successfully for `ConversationViewer` constructor changes.
+- Two-cycle TDD approach (Cycle 1: add DI with mock still present; Cycle 2: remove mock) gave a stable intermediate state and caught the missing `wrapTextWithAnsi` import before proceeding.
+- Scope was tight: 3 files changed, 2 commits of substance, no rework on the production code.
+
+#### What caused friction (agent side)
+
+1. `instruction-violation` — Failed to load the `colgrep` skill during planning.
+   Constructed the path by pattern (`.pi/skills/colgrep/SKILL.md`) instead of using the `<location>` listed in the `<available_skills>` block (`packages/pi-colgrep/skills/colgrep/SKILL.md`).
+   Got ENOENT and silently moved on.
+   Impact: user-caught; required a follow-up exchange to load the skill and re-review the plan.
+   No rework needed — the plan was already correct.
+2. `wrong-abstraction` — Cycle 2 Edit replaced the entire `vi.mock` block + dynamic imports as one chunk, leaving the `ConversationViewer` static import stranded after `const testRegistry` instead of at the top with other imports.
+   Impact: user-caught ("Wait, we have a dynamic import?"); required a follow-up edit and amend into the retro commit.
+3. `missing-context` — Plan stated "11+" constructor call sites; actual count was 16 (11 render-width-safety + 5 safety-net).
+   The grep output during planning showed all 16 but the count wasn't verified.
+   Impact: no rework — the Python script handled all 17 (16 test + 1 production) regardless.
+
+#### What caused friction (user side)
+
+- No friction identified — user interventions were timely and precise.
+
+### Changes made
+
+1. Appended this final retrospective entry to `packages/pi-subagents/docs/retro/0147-inject-wrap-text-into-conversation-viewer.md`.
+2. Separated `colgrep` skill loading into its own bullet in `.pi/prompts/plan-issue.md` to reduce pattern-matching shortcuts.
+3. Replaced deterministic step in `.pi/prompts/retro.md` with a "Sync with remote" section matching the other prompts.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotgenes/pi-subagents",
-  "version": "6.16.3",
+  "version": "6.17.1",
   "exports": {
     ".": "./src/service.ts"
   },

package/src/agent-manager.ts

@@ -9,17 +9,17 @@
 import { randomUUID } from "node:crypto";
 import type { Model } from "@earendil-works/pi-ai";
 import type { AgentSession } from "@earendil-works/pi-coding-agent";
-import { AgentRecord } from "./agent-record.js";
-import type { AgentRunner } from "./agent-runner.js";
-import { AgentTypeRegistry } from "./agent-types.js";
-import { debugLog } from "./debug.js";
-import { NotificationState } from "./notification-state.js";
-import type { ParentSnapshot } from "./parent-snapshot.js";
-import { subscribeRecordObserver } from "./record-observer.js";
-import type { RunConfig } from "./runtime.js";
-import type { AgentInvocation, IsolationMode, ShellExec, SubagentType, ThinkingLevel } from "./types.js";
-import type { WorktreeManager } from "./worktree.js";
-import { WorktreeState } from "./worktree-state.js";
+import { AgentRecord } from "./agent-record";
+import type { AgentRunner } from "./agent-runner";
+import { AgentTypeRegistry } from "./agent-types";
+import { debugLog } from "./debug";
+import { NotificationState } from "./notification-state";
+import type { ParentSnapshot } from "./parent-snapshot";
+import { subscribeRecordObserver } from "./record-observer";
+import type { RunConfig } from "./runtime";
+import type { AgentInvocation, IsolationMode, ShellExec, SubagentType, ThinkingLevel } from "./types";
+import type { WorktreeManager } from "./worktree";
+import { WorktreeState } from "./worktree-state";
 
 export type CompactionInfo = { reason: "manual" | "threshold" | "overflow"; tokensBefore: number };
 

package/src/agent-record.ts

@@ -13,12 +13,12 @@
  */
 
 import type { AgentSession } from "@earendil-works/pi-coding-agent";
-import type { ExecutionState } from "./execution-state.js";
-import type { NotificationState } from "./notification-state.js";
-import type { AgentInvocation, SubagentType } from "./types.js";
-import type { LifetimeUsage } from "./usage.js";
-import { addUsage } from "./usage.js";
-import type { WorktreeState } from "./worktree-state.js";
+import type { ExecutionState } from "./execution-state";
+import type { NotificationState } from "./notification-state";
+import type { AgentInvocation, SubagentType } from "./types";
+import type { LifetimeUsage } from "./usage";
+import { addUsage } from "./usage";
+import type { WorktreeState } from "./worktree-state";
 
 export type AgentRecordStatus =
 	| "queued"

package/src/agent-runner.ts

@@ -8,12 +8,12 @@ import {
   type AgentSessionEvent,
   type SettingsManager,
 } from "@earendil-works/pi-coding-agent";
-import type { AgentConfigLookup } from "./agent-types.js";
-import { extractText } from "./context.js";
-import type { EnvInfo } from "./env.js";
-import type { ParentSnapshot } from "./parent-snapshot.js";
-import { type AssemblerIO, assembleSessionConfig } from "./session-config.js";
-import type { ShellExec, SubagentType, ThinkingLevel } from "./types.js";
+import type { AgentConfigLookup } from "./agent-types";
+import { extractText } from "./context";
+import type { EnvInfo } from "./env";
+import type { ParentSnapshot } from "./parent-snapshot";
+import { type AssemblerIO, assembleSessionConfig } from "./session-config";
+import type { ShellExec, SubagentType, ThinkingLevel } from "./types";
 
 /** Names of tools registered by this extension that subagents must NOT inherit. */
 const EXCLUDED_TOOL_NAMES = ["Agent", "get_subagent_result", "steer_subagent"];

package/src/agent-types.ts

@@ -5,8 +5,8 @@
  * User agents override defaults with the same name. Disabled agents are kept but excluded from spawning.
  */
 
-import { DEFAULT_AGENTS } from "./default-agents.js";
-import type { AgentConfig } from "./types.js";
+import { DEFAULT_AGENTS } from "./default-agents";
+import type { AgentConfig } from "./types";
 
 // ── AgentConfigLookup interface ──────────────────────────────────────────────
 

package/src/custom-agents.ts

@@ -5,9 +5,9 @@
 import { existsSync, readdirSync, readFileSync } from "node:fs";
 import { basename, join } from "node:path";
 import { getAgentDir, parseFrontmatter } from "@earendil-works/pi-coding-agent";
-import { BUILTIN_TOOL_NAMES } from "./agent-types.js";
-import { debugLog } from "./debug.js";
-import type { AgentConfig, MemoryScope, ThinkingLevel } from "./types.js";
+import { BUILTIN_TOOL_NAMES } from "./agent-types";
+import { debugLog } from "./debug";
+import type { AgentConfig, MemoryScope, ThinkingLevel } from "./types";
 
 /**
  * Scan for custom agent .md files from multiple locations.

package/src/default-agents.ts

@@ -4,7 +4,7 @@
  * These are always available but can be overridden by user .md files with the same name.
  */
 
-import type { AgentConfig } from "./types.js";
+import type { AgentConfig } from "./types";
 
 const READ_ONLY_TOOLS = ["read", "bash", "grep", "find", "ls"];
 

package/src/env.ts

@@ -2,8 +2,8 @@
  * env.ts — Detect environment info (git, platform) for subagent system prompts.
  */
 
-import { debugLog } from "./debug.js";
-import type { ShellExec } from "./types.js";
+import { debugLog } from "./debug";
+import type { ShellExec } from "./types";
 
 export interface EnvInfo {
   isGitRepo: boolean;

package/src/handlers/index.ts

@@ -1,2 +1,2 @@
-export { SessionLifecycleHandler } from "./lifecycle.js";
-export { ToolStartHandler } from "./tool-start.js";
+export { SessionLifecycleHandler } from "./lifecycle";
+export { ToolStartHandler } from "./tool-start";

package/src/index.ts

@@ -20,35 +20,35 @@ import {
   SettingsManager as SdkSettingsManager,
   SessionManager,
 } from "@earendil-works/pi-coding-agent";
-import { AgentManager, type AgentManagerObserver } from "./agent-manager.js";
-import { createAgentRunner, getAgentConversation, type RunnerIO, steerAgent } from "./agent-runner.js";
-import { AgentTypeRegistry } from "./agent-types.js";
-import { loadCustomAgents } from "./custom-agents.js";
-import { detectEnv } from "./env.js";
-import { SessionLifecycleHandler, ToolStartHandler } from "./handlers/index.js";
-import { buildMemoryBlock, buildReadOnlyMemoryBlock } from "./memory.js";
-import { type ModelRegistry, resolveModel } from "./model-resolver.js";
-import { buildEventData, type NotificationDetails, NotificationManager } from "./notification.js";
-import { buildParentSnapshot } from "./parent-snapshot.js";
-import { buildAgentPrompt } from "./prompts.js";
-import { createNotificationRenderer } from "./renderer.js";
-import { createSubagentRuntime } from "./runtime.js";
-import { publishSubagentsService, unpublishSubagentsService } from "./service.js";
-import { createSubagentsService } from "./service-adapter.js";
-import { deriveSubagentSessionDir } from "./session-dir.js";
-import { SettingsManager } from "./settings.js";
-import { preloadSkills } from "./skill-loader.js";
-import { createAgentTool } from "./tools/agent-tool.js";
-import { createGetResultTool } from "./tools/get-result-tool.js";
-import { getModelLabelFromConfig } from "./tools/helpers.js";
-import { createSteerTool } from "./tools/steer-tool.js";
-import { FsAgentFileOps } from "./ui/agent-file-ops.js";
-import { createAgentsMenuHandler } from "./ui/agent-menu.js";
+import { AgentManager, type AgentManagerObserver } from "./agent-manager";
+import { createAgentRunner, getAgentConversation, type RunnerIO, steerAgent } from "./agent-runner";
+import { AgentTypeRegistry } from "./agent-types";
+import { loadCustomAgents } from "./custom-agents";
+import { detectEnv } from "./env";
+import { SessionLifecycleHandler, ToolStartHandler } from "./handlers/index";
+import { buildMemoryBlock, buildReadOnlyMemoryBlock } from "./memory";
+import { type ModelRegistry, resolveModel } from "./model-resolver";
+import { buildEventData, type NotificationDetails, NotificationManager } from "./notification";
+import { buildParentSnapshot } from "./parent-snapshot";
+import { buildAgentPrompt } from "./prompts";
+import { createNotificationRenderer } from "./renderer";
+import { createSubagentRuntime } from "./runtime";
+import { publishSubagentsService, unpublishSubagentsService } from "./service";
+import { createSubagentsService } from "./service-adapter";
+import { deriveSubagentSessionDir } from "./session-dir";
+import { SettingsManager } from "./settings";
+import { preloadSkills } from "./skill-loader";
+import { createAgentTool } from "./tools/agent-tool";
+import { createGetResultTool } from "./tools/get-result-tool";
+import { getModelLabelFromConfig } from "./tools/helpers";
+import { createSteerTool } from "./tools/steer-tool";
+import { FsAgentFileOps } from "./ui/agent-file-ops";
+import { createAgentsMenuHandler } from "./ui/agent-menu";
 import {
   AgentWidget,
   type UICtx,
-} from "./ui/agent-widget.js";
-import { GitWorktreeManager } from "./worktree.js";
+} from "./ui/agent-widget";
+import { GitWorktreeManager } from "./worktree";
 
 export default function (pi: ExtensionAPI) {
   // ---- Register custom notification renderer ----

package/src/invocation-config.ts

@@ -1,4 +1,4 @@
-import type { AgentConfig, IsolationMode, ThinkingLevel } from "./types.js";
+import type { AgentConfig, IsolationMode, ThinkingLevel } from "./types";
 
 interface AgentInvocationParams {
   model?: string;

package/src/memory.ts

@@ -10,8 +10,8 @@
 import { existsSync, lstatSync, mkdirSync, readFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { join, } from "node:path";
-import { debugLog } from "./debug.js";
-import type { MemoryScope } from "./types.js";
+import { debugLog } from "./debug";
+import type { MemoryScope } from "./types";
 
 /** Maximum lines to read from MEMORY.md */
 const MAX_MEMORY_LINES = 200;

package/src/notification.ts

@@ -1,7 +1,7 @@
-import { debugLog } from "./debug.js";
-import type { AgentRecord } from "./types.js";
-import type { AgentActivityTracker } from "./ui/agent-activity-tracker.js";
-import { getLifetimeTotal, getSessionContextPercent } from "./usage.js";
+import { debugLog } from "./debug";
+import type { AgentRecord } from "./types";
+import type { AgentActivityTracker } from "./ui/agent-activity-tracker";
+import { getLifetimeTotal, getSessionContextPercent } from "./usage";
 
 /** Details attached to custom notification messages for visual rendering. */
 export interface NotificationDetails {

package/src/parent-snapshot.ts

@@ -3,7 +3,7 @@
  */
 
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { buildParentContext } from "./context.js";
+import { buildParentContext } from "./context";
 
 /**
  * Plain data snapshot of the parent session state captured at spawn time.

package/src/prompts.ts

@@ -2,8 +2,8 @@
  * prompts.ts — System prompt builder for agents.
  */
 
-import type { EnvInfo } from "./env.js";
-import type { AgentPromptConfig } from "./types.js";
+import type { EnvInfo } from "./env";
+import type { AgentPromptConfig } from "./types";
 
 /** Extra sections to inject into the system prompt (memory, skills, etc.). */
 export interface PromptExtras {

package/src/record-observer.ts

@@ -5,8 +5,8 @@
  * and resume() with a single direct subscription.
  */
 
-import type { CompactionInfo } from "./agent-manager.js";
-import type { AgentRecord } from "./agent-record.js";
+import type { CompactionInfo } from "./agent-manager";
+import type { AgentRecord } from "./agent-record";
 
 /** Narrow session interface — only the subscribe method needed by the observer. */
 interface SubscribableSession {

package/src/renderer.ts

@@ -1,6 +1,6 @@
 import { Text } from "@earendil-works/pi-tui";
-import type { NotificationDetails } from "./notification.js";
-import { formatMs, formatTokens, formatTurns } from "./ui/display.js";
+import type { NotificationDetails } from "./notification";
+import { formatMs, formatTokens, formatTurns } from "./ui/display";
 
 /** Narrow theme interface — only the methods the renderer actually calls. */
 interface RendererTheme {

package/src/runtime.ts

@@ -6,8 +6,8 @@
  * Follows the same pattern as pi-permission-system's ExtensionRuntime.
  */
 
-import type { AgentActivityTracker } from "./ui/agent-activity-tracker.js";
-import type { UICtx } from "./ui/agent-widget.js";
+import type { AgentActivityTracker } from "./ui/agent-activity-tracker";
+import type { UICtx } from "./ui/agent-widget";
 
 /**
  * Narrow widget interface consumed by SubagentRuntime delegation methods.

package/src/service-adapter.ts

@@ -6,11 +6,11 @@
  */
 
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
-import type { ModelRegistry } from "./model-resolver.js";
-import type { ParentSnapshot } from "./parent-snapshot.js";
-import { buildParentSnapshot } from "./parent-snapshot.js";
-import type { SubagentRecord, SubagentsService } from "./service.js";
-import type { AgentRecord } from "./types.js";
+import type { ModelRegistry } from "./model-resolver";
+import type { ParentSnapshot } from "./parent-snapshot";
+import { buildParentSnapshot } from "./parent-snapshot";
+import type { SubagentRecord, SubagentsService } from "./service";
+import type { AgentRecord } from "./types";
 
 /** Narrow interface for the AgentManager — avoids coupling to the concrete class. */
 export interface AgentManagerLike {

package/src/service.ts

@@ -9,7 +9,7 @@
  *   svc?.spawn("Explore", "Check for stale TODOs");
  */
 
-import type { LifetimeUsage } from "./usage.js";
+import type { LifetimeUsage } from "./usage";
 
 export type { LifetimeUsage };
 

package/src/session-config.ts

@@ -14,16 +14,16 @@ import {
   type AgentConfigLookup,
   getMemoryToolNames,
   getReadOnlyMemoryToolNames,
-} from "./agent-types.js";
-import type { EnvInfo } from "./env.js";
-import type { PromptExtras } from "./prompts.js";
-import type { PreloadedSkill } from "./skill-loader.js";
+} from "./agent-types";
+import type { EnvInfo } from "./env";
+import type { PromptExtras } from "./prompts";
+import type { PreloadedSkill } from "./skill-loader";
 import type {
   AgentPromptConfig,
   MemoryScope,
   SubagentType,
   ThinkingLevel,
-} from "./types.js";
+} from "./types";
 
 // ── Public interfaces ────────────────────────────────────────────────────────
 

package/src/skill-loader.ts

@@ -23,8 +23,8 @@ import { existsSync, readdirSync } from "node:fs";
 import { homedir } from "node:os";
 import { join } from "node:path";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { debugLog } from "./debug.js";
-import { isSymlink, isUnsafeName, safeReadFile } from "./memory.js";
+import { debugLog } from "./debug";
+import { isSymlink, isUnsafeName, safeReadFile } from "./memory";
 
 export interface PreloadedSkill {
   name: string;

package/src/tools/agent-tool.ts

@@ -1,24 +1,24 @@
 import type { AgentToolResult } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
 import { Type } from "@sinclair/typebox";
-import type { AgentSpawnConfig } from "../agent-manager.js";
-import { AgentTypeRegistry } from "../agent-types.js";
-import type { ParentSnapshot } from "../parent-snapshot.js";
+import type { AgentSpawnConfig } from "../agent-manager";
+import { AgentTypeRegistry } from "../agent-types";
+import type { ParentSnapshot } from "../parent-snapshot";
 
-import type { AgentRecord } from "../types.js";
-import { AgentActivityTracker } from "../ui/agent-activity-tracker.js";
-import { type UICtx } from "../ui/agent-widget.js";
+import type { AgentRecord } from "../types";
+import { AgentActivityTracker } from "../ui/agent-activity-tracker";
+import { type UICtx } from "../ui/agent-widget";
 import {
   type AgentDetails,
   formatMs,
   formatTurns,
   getDisplayName,
   SPINNER,
-} from "../ui/display.js";
-import { spawnBackground } from "./background-spawner.js";
-import { runForeground } from "./foreground-runner.js";
-import { buildDetails, buildTypeListText, textResult } from "./helpers.js";
-import { type ModelInfo, resolveSpawnConfig } from "./spawn-config.js";
+} from "../ui/display";
+import { spawnBackground } from "./background-spawner";
+import { runForeground } from "./foreground-runner";
+import { buildDetails, buildTypeListText, textResult } from "./helpers";
+import { type ModelInfo, resolveSpawnConfig } from "./spawn-config";
 
 // ---- Deps interface ----
 

package/src/tools/background-spawner.ts

@@ -1,11 +1,11 @@
-import type { AgentSpawnConfig } from "../agent-manager.js";
-import type { ParentSnapshot } from "../parent-snapshot.js";
-import type { AgentRecord } from "../types.js";
-import { AgentActivityTracker } from "../ui/agent-activity-tracker.js";
-import { subscribeUIObserver } from "../ui/ui-observer.js";
-import type { AgentActivityAccess } from "./agent-tool.js";
-import { textResult } from "./helpers.js";
-import type { ResolvedSpawnConfig } from "./spawn-config.js";
+import type { AgentSpawnConfig } from "../agent-manager";
+import type { ParentSnapshot } from "../parent-snapshot";
+import type { AgentRecord } from "../types";
+import { AgentActivityTracker } from "../ui/agent-activity-tracker";
+import { subscribeUIObserver } from "../ui/ui-observer";
+import type { AgentActivityAccess } from "./agent-tool";
+import { textResult } from "./helpers";
+import type { ResolvedSpawnConfig } from "./spawn-config";
 
 /** Narrow manager interface for the background spawner. */
 export interface BackgroundManagerDeps {

package/src/tools/foreground-runner.ts

@@ -1,23 +1,23 @@
 import type { AgentToolResult } from "@earendil-works/pi-coding-agent";
-import type { AgentSpawnConfig } from "../agent-manager.js";
-import type { ParentSnapshot } from "../parent-snapshot.js";
-import type { AgentRecord } from "../types.js";
-import { AgentActivityTracker } from "../ui/agent-activity-tracker.js";
+import type { AgentSpawnConfig } from "../agent-manager";
+import type { ParentSnapshot } from "../parent-snapshot";
+import type { AgentRecord } from "../types";
+import { AgentActivityTracker } from "../ui/agent-activity-tracker";
 import {
   type AgentDetails,
   describeActivity,
   formatMs,
   SPINNER,
-} from "../ui/display.js";
-import { subscribeUIObserver } from "../ui/ui-observer.js";
-import type { AgentActivityAccess } from "./agent-tool.js";
+} from "../ui/display";
+import { subscribeUIObserver } from "../ui/ui-observer";
+import type { AgentActivityAccess } from "./agent-tool";
 import {
   buildDetails,
   formatLifetimeTokens,
   getStatusNote,
   textResult,
-} from "./helpers.js";
-import type { ResolvedSpawnConfig } from "./spawn-config.js";
+} from "./helpers";
+import type { ResolvedSpawnConfig } from "./spawn-config";
 
 /** Narrow manager interface for the foreground runner. */
 export interface ForegroundManagerDeps {

package/src/tools/get-result-tool.ts

@@ -1,10 +1,10 @@
 import type { AgentSession } from "@earendil-works/pi-coding-agent";
 import { Type } from "@sinclair/typebox";
-import type { AgentConfigLookup } from "../agent-types.js";
-import type { AgentRecord } from "../types.js";
-import { formatDuration, getDisplayName } from "../ui/display.js";
-import { getSessionContextPercent } from "../usage.js";
-import { formatLifetimeTokens, textResult } from "./helpers.js";
+import type { AgentConfigLookup } from "../agent-types";
+import type { AgentRecord } from "../types";
+import { formatDuration, getDisplayName } from "../ui/display";
+import { getSessionContextPercent } from "../usage";
+import { formatLifetimeTokens, textResult } from "./helpers";
 
 /** Create the get_subagent_result tool definition (without Pi SDK wrapper). */
 export function createGetResultTool(

package/src/tools/helpers.ts

@@ -1,7 +1,7 @@
-import type { AgentConfigLookup } from "../agent-types.js";
-import { AgentActivityTracker } from "../ui/agent-activity-tracker.js";
-import { type AgentDetails, formatTokens } from "../ui/display.js";
-import { getLifetimeTotal, type LifetimeUsage } from "../usage.js";
+import type { AgentConfigLookup } from "../agent-types";
+import { AgentActivityTracker } from "../ui/agent-activity-tracker";
+import { type AgentDetails, formatTokens } from "../ui/display";
+import { getLifetimeTotal, type LifetimeUsage } from "../usage";
 
 /** Parenthetical status note for completed agent result text. */
 export function getStatusNote(status: string): string {

package/src/tools/spawn-config.ts

@@ -7,17 +7,17 @@
  */
 
 import type { Model } from "@earendil-works/pi-ai";
-import { normalizeMaxTurns } from "../agent-runner.js";
-import type { AgentTypeRegistry } from "../agent-types.js";
-import { resolveAgentInvocationConfig } from "../invocation-config.js";
-import { resolveInvocationModel } from "../model-resolver.js";
-import type { AgentInvocation, IsolationMode, SubagentType, ThinkingLevel } from "../types.js";
+import { normalizeMaxTurns } from "../agent-runner";
+import type { AgentTypeRegistry } from "../agent-types";
+import { resolveAgentInvocationConfig } from "../invocation-config";
+import { resolveInvocationModel } from "../model-resolver";
+import type { AgentInvocation, IsolationMode, SubagentType, ThinkingLevel } from "../types";
 import {
   type AgentDetails,
   buildInvocationTags,
   getDisplayName,
   getPromptModeLabel,
-} from "../ui/display.js";
+} from "../ui/display";
 
 /** Model info extracted from the parent session context. */
 export interface ModelInfo {

package/src/tools/steer-tool.ts

@@ -1,8 +1,8 @@
 import type { AgentSession } from "@earendil-works/pi-coding-agent";
 import { Type } from "@sinclair/typebox";
-import type { AgentRecord } from "../types.js";
-import { getSessionContextPercent } from "../usage.js";
-import { formatLifetimeTokens, textResult } from "./helpers.js";
+import type { AgentRecord } from "../types";
+import { getSessionContextPercent } from "../usage";
+import { formatLifetimeTokens, textResult } from "./helpers";
 
 /** Create the steer_subagent tool definition (without Pi SDK wrapper). */
 export function createSteerTool(

package/src/types.ts

@@ -5,7 +5,7 @@
 import type { ThinkingLevel } from "@earendil-works/pi-ai";
 
 
-export { AgentRecord } from "./agent-record.js";
+export { AgentRecord } from "./agent-record";
 export type { ThinkingLevel };
 
 /** Agent type: any string name (built-in defaults or user-defined). */

package/src/ui/agent-activity-tracker.ts

@@ -5,7 +5,7 @@
  * in `ui-observer.ts`. Callers use named transition methods; readers use read-only accessors.
  */
 
-import type { SessionLike } from "../usage.js";
+import type { SessionLike } from "../usage";
 
 /** Per-agent live activity state with explicit transition methods and read-only accessors. */
 export class AgentActivityTracker {