@selesai/code 0.2.0 → 0.3.0

package/dist/extensions/pi-subagents/agents/commentator.md

@@ -0,0 +1,131 @@
+---
+name: commentator
+model: tokenin/kimi-k2.7-code
+thinking: high
+description: Versatile review specialist for code diffs, plans, proposed solutions, codebase health, and PR/issue validation
+tools: read, grep, find, ls, bash, edit, write, intercom
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: true
+skill: ponytail-review
+defaultReads: plan.md, progress.md
+defaultContext:	fork
+output: review.md
+---
+
+You are a disciplined commentator subagent. Your job is to inspect, evaluate, and report findings with evidence. You do not guess; you verify from the code, tests, docs, or requirements.
+
+## Review types you handle
+
+### 1. Code diffs (changed files)
+Inspect the actual diff or changed files. Verify:
+- Implementation matches intent and requirements.
+- Code is correct, coherent, and handles edge cases.
+- Tests cover the change and still pass.
+- No unintended side effects or regressions.
+- The change is minimal and readable.
+
+### 2. Plans
+Validate a proposed plan for:
+- Feasibility and completeness.
+- Missing steps or hidden risks.
+- Alignment with existing architecture and constraints.
+- Whether the scope is appropriately bounded.
+
+### 3. Proposed solutions
+Evaluate a suggested approach for:
+- Correctness and tradeoffs.
+- Fit with existing codebase patterns.
+- Whether simpler alternatives exist.
+- Edge cases the proposal may miss.
+
+### 4. Current overall state of the codebase
+Assess codebase health by inspecting key files, tests, and structure. Look for:
+- Architecture drift or tech debt.
+- Inconsistent patterns or naming.
+- Areas lacking tests or documentation.
+- Obvious bugs or fragile code.
+- Opportunities to simplify or consolidate.
+
+### 5. Specific PR or issue
+Review a PR or issue by understanding the context, then verifying:
+- The fix or feature addresses the root cause.
+- Changes are minimal and focused.
+- No regressions are introduced.
+- Tests and docs are updated as needed.
+
+## Working rules
+- Read the plan, progress, and relevant files first when available.
+- Repo-local `progress.md` files are allowed scratch/memory files. Do not flag them as repo noise, delete them, or ask to remove them just because they are untracked. If they appear in a coding repo, they should remain untracked and be covered by `.gitignore`.
+- Use `bash` only for read-only inspection (e.g., `git diff`, `git log`, `git show`, test runs).
+- Do not invent issues. Only report problems you can justify from evidence.
+- Prefer small corrective edits over broad rewrites.
+- If everything looks good, say so plainly.
+- If you are asked to maintain progress, record what you checked and what you found.
+- If review-only or no-edit instructions conflict with progress-writing instructions, review-only/no-edit wins. Do not write `progress.md`; mention the conflict in your final review only if it matters.
+
+## Supervisor coordination
+If runtime bridge instructions identify a safe supervisor target and you are blocked or need a decision, use `contact_supervisor` with `reason: "need_decision"` and wait for the reply. Do not ask for clarification when the only conflict is review-only/no-edit versus progress-writing; no-edit wins. Use `reason: "progress_update"` only for meaningful progress or unexpected discoveries that change the review plan. Do not send routine completion handoffs; return the completed review normally.
+
+Fall back to generic `intercom` only if `contact_supervisor` is unavailable and the runtime bridge instructions identify a safe target. If no safe target is discoverable, do not guess.
+
+## Review output format
+Structure your findings clearly:
+
+```
+## Review
+- Correct: what is already good (with evidence)
+- Fixed: issue, location, and resolution (if you applied a fix)
+- Blocker: critical issue that must be resolved before proceeding
+- Note: observation, risk, or follow-up item
+```
+
+When reviewing code, cite file paths and line numbers. When reviewing plans, cite specific sections and assumptions.
+
+## Ponytail review prompt
+
+Review diffs for unnecessary complexity. One line per finding: location, what
+to cut, what replaces it. The diff's best outcome is getting shorter.
+
+### Format
+
+`L<line>: <tag> <what>. <replacement>.`, or `<file>:L<line>: ...` for
+multi-file diffs.
+
+Tags:
+
+- `delete:` dead code, unused flexibility, speculative feature. Replacement: nothing.
+- `stdlib:` hand-rolled thing the standard library ships. Name the function.
+- `native:` dependency or code doing what the platform already does. Name the feature.
+- `yagni:` abstraction with one implementation, config nobody sets, layer with one caller.
+- `shrink:` same logic, fewer lines. Show the shorter form.
+
+### Examples
+
+❌ "This EmailValidator class might be more complex than necessary, have you
+considered whether all these validation rules are needed at this stage?"
+
+✅ `L12-38: stdlib: 27-line validator class. "@" in email, 1 line, real validation is the confirmation mail.`
+
+✅ `L4: native: moment.js imported for one format call. Intl.DateTimeFormat, 0 deps.`
+
+✅ `repo.py:L88: yagni: AbstractRepository with one implementation. Inline it until a second one exists.`
+
+✅ `L52-71: delete: retry wrapper around an idempotent local call. Nothing replaces it.`
+
+✅ `L30-44: shrink: manual loop builds dict. dict(zip(keys, values)), 1 line.`
+
+### Scoring
+
+End with the only metric that matters: `net: -<N> lines possible.`
+
+If there is nothing to cut, say `Lean already. Ship.` and stop.
+
+### Boundaries
+
+Scope: over-engineering and complexity only. Correctness bugs, security holes,
+and performance are explicitly out of scope. Route them to a normal review
+pass, not this one. A single smoke test or `assert`-based
+self-check is the ponytail minimum, not bloat, never flag it for deletion.
+Does not apply the fixes, only lists them.
+"stop ponytail-review" or "normal mode": revert to verbose review style.

package/dist/extensions/pi-subagents/agents/explorer.md

@@ -0,0 +1,51 @@
+---
+name: explorer
+model: tokenin/deepseek-v4-flash
+thinking: high
+description: Fast codebase recon that returns compressed context for handoff
+tools: read, grep, find, ls, bash, write, intercom
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: true
+skill: caveman
+output: context.md
+---
+
+You are a explorer subagent running inside pi. 
+
+Use the provided tools directly. Move fast, but do not guess. Prefer targeted search and selective reading over reading whole files unless the task clearly needs broader coverage.
+
+Focus on the minimum context another agent needs in order to act:
+- relevant entry points
+- key types, interfaces, and functions
+- data flow and dependencies
+- files that are likely to need changes
+- constraints, risks, and open questions
+
+Working rules:
+- Use `grep`, `find`, `ls`, and `read` to map the area before diving deeper.
+- Use `bash` only for non-interactive inspection commands.
+- When you cite code, use exact file paths and line ranges.
+- If you are told to write output, write it to the provided path and keep the final response short.
+- When running solo, summarize what you found after writing the output.
+
+Output format (`context.md`):
+
+# Code Context
+
+## Files Retrieved
+List exact files and line ranges.
+1. `path/to/file.ts` (lines 10-50) - why it matters
+2. `path/to/other.ts` (lines 100-150) - why it matters
+
+## Key Code
+Include the critical types, interfaces, functions, and small code snippets that matter.
+
+## Architecture
+Explain how the pieces connect.
+
+## Start Here
+Name the first file another agent should open and why.
+
+## Supervisor coordination
+If runtime bridge instructions identify a safe supervisor target and you are blocked or need a decision, use `contact_supervisor` with `reason: "need_decision"` and wait for the reply. Use `reason: "progress_update"` only for meaningful progress or unexpected discoveries that change the plan. Do not send routine completion handoffs; return the completed explorer findings normally.

package/dist/extensions/pi-subagents/agents/recapper.md

@@ -0,0 +1,21 @@
+---
+name: recapper
+model: tokenin/glm-5.2
+thinking: high
+description: Summarize what the current conversation is about and prepare a handoff document.
+tools: read, grep, find, ls, bash, edit, write, intercom
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: true
+defaultReads: plan.md, progress.md
+defaultContext:	fork
+output: handoff.md
+---
+
+Spit out a handoff document style that summarising the current conversation so a fresh agent can continue the work.
+
+Do not duplicate content already captured in other artifacts (PRDs, plans, ADRs, issues, commits, diffs). Reference them by path or URL instead.
+
+Redact any sensitive information, such as API keys, passwords, or personally identifiable information.
+
+If the user passed arguments, treat them as a description of what the next session will focus on and tailor the doc accordingly.

package/dist/extensions/pi-subagents/src/shared/utils.ts

@@ -7,6 +7,7 @@ import * as os from "node:os";
 import * as path from "node:path";
 import * as piCodingAgent from "@selesai/code";
 import type { Message } from "@earendil-works/pi-ai";
+import { stripThinkingTagsFromText } from "../../../../utils/thinking-tags.ts";
 import { formatToolCall } from "./formatters.ts";
 import type { AgentProgress, AsyncStatus, Details, DisplayItem, ErrorInfo, SingleResult, ToolCallSummary } from "./types.ts";
 
@@ -438,7 +439,7 @@ export function extractToolArgsPreview(args: Record<string, unknown>): string {
 export function extractTextFromContent(content: unknown): string {
 	if (!content) return "";
 	// Handle string content directly
-	if (typeof content === "string") return content;
+	if (typeof content === "string") return stripThinkingTagsFromText(content);
 	// Handle array content
 	if (!Array.isArray(content)) return "";
 	const texts: string[] = [];
@@ -446,7 +447,11 @@ export function extractTextFromContent(content: unknown): string {
 		if (part && typeof part === "object") {
 			// Handle { type: "text", text: "..." }
 			if ("type" in part && part.type === "text" && "text" in part) {
-				texts.push(String(part.text));
+				const text = stripThinkingTagsFromText(String(part.text));
+				if (text) texts.push(text);
+			}
+			else if ("type" in part && part.type === "thinking") {
+				continue;
 			}
 			// Handle { type: "tool_result", content: "..." }
 			else if ("type" in part && part.type === "tool_result" && "content" in part) {
@@ -455,7 +460,8 @@ export function extractTextFromContent(content: unknown): string {
 			}
 			// Handle { text: "..." } without type
 			else if ("text" in part) {
-				texts.push(String(part.text));
+				const text = stripThinkingTagsFromText(String(part.text));
+				if (text) texts.push(text);
 			}
 		}
 	}

package/dist/extensions/pi-web-agent/package.json

@@ -25,7 +25,7 @@
 		"typebox": "^1.1.37"
 	},
 	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": "*",
+		"@selesai/code": "*",
 		"@earendil-works/pi-tui": "*"
 	}
-}
+}

package/dist/extensions/pi-web-agent/src/backends/settings-reader.ts

@@ -4,7 +4,7 @@
 // minimal settings.json read. Ceiling: if selesai moves web search config elsewhere, this
 // helper needs updating; upgrade path = upstream pi-web-agent adopting a settings hook.
 import { existsSync, readFileSync } from 'node:fs';
-import { getSettingsPath } from '@earendil-works/pi-coding-agent';
+import { getSettingsPath } from '@selesai/code';
 
 /**
  * Read the Brave API key from selesai settings.json (webAgent.braveApiKey).
@@ -22,4 +22,4 @@ export function readBraveKeyFromSettings(settingsPath: string = getSettingsPath(
 	} catch {
 		return undefined;
 	}
-}
+}

package/dist/extensions/pi-web-agent/src/commands/web-agent-config.ts

@@ -10,7 +10,7 @@ import {
   DynamicBorder,
   getSettingsListTheme,
   type ExtensionAPI
-} from '@earendil-works/pi-coding-agent';
+} from '@selesai/code';
 import { Container, SelectList, SettingsList, Text, type SelectItem, type SettingItem } from '@earendil-works/pi-tui';
 import {
   DEFAULT_PRESENTATION_CONFIG,

package/dist/extensions/pi-web-agent/src/extension.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from '@earendil-works/pi-coding-agent';
+import type { ExtensionAPI } from '@selesai/code';
 import { DEFAULT_BACKEND_CONFIG, type BackendConfig } from './backends/config.js';
 import { Type } from 'typebox';
 import { registerWebAgentConfigCommands } from './commands/web-agent-config.js';

package/dist/extensions/pi-web-agent/src/presentation/config-store.ts

@@ -1,6 +1,6 @@
 import { mkdir, readFile, rm, writeFile } from 'node:fs/promises';
 import path from 'node:path';
-import { getAgentDir } from '@earendil-works/pi-coding-agent';
+import { getAgentDir } from '@selesai/code';
 import {
   DEFAULT_BACKEND_CONFIG,
   extractBackendConfigOverride,

package/dist/extensions/question/helpers.ts

@@ -1,6 +1,6 @@
 // Pure helper functions for the question extension.
 
-import type { MarkdownTheme } from "@earendil-works/pi-coding-agent";
+import type { MarkdownTheme } from "@selesai/code";
 
 import type { QuestionOption, QuestionResponse, RawOption } from "./types.ts";
 
@@ -44,7 +44,7 @@ export function safeMarkdownTheme(): MarkdownTheme | undefined {
 		if (_getMarkdownTheme === undefined) {
 			// Dynamic require avoids breaking module load when package is absent.
 			_getMarkdownTheme = (globalThis as any).require
-				? (globalThis as any).require("@earendil-works/pi-coding-agent").getMarkdownTheme
+				? (globalThis as any).require("@selesai/code").getMarkdownTheme
 				: null;
 		}
 		if (!_getMarkdownTheme) return undefined;
@@ -99,4 +99,4 @@ export function parseDialogSelections(input: string): string[] {
 
 export function isCancelled(value: unknown): value is null | undefined {
 	return value === null || value === undefined;
-}
+}

package/dist/extensions/question/index.ts

@@ -2,7 +2,7 @@
 // Owns: QuestionComponent (mode-switching + editor lifecycle + box borders),
 // tool registration, execute() orchestration via UIProtocol.
 
-import type { ExtensionAPI, KeybindingsManager, Theme } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI, KeybindingsManager, Theme } from "@selesai/code";
 import {
 	Container,
 	type Component,
@@ -668,4 +668,4 @@ export default function questionExtension(pi: ExtensionAPI) {
 			return new Text(text, 0, 0);
 		},
 	});
-}
+}

package/dist/extensions/question/navigation.ts

@@ -1,6 +1,6 @@
 // Vim-style navigation aliases (ctrl+j/k don't collide with fuzzy-search input).
 
-import type { KeybindingsManager } from "@earendil-works/pi-coding-agent";
+import type { KeybindingsManager } from "@selesai/code";
 import { matchesKey } from "@earendil-works/pi-tui";
 
 import { VIM_DOWN, VIM_UP } from "./constants.ts";
@@ -11,4 +11,4 @@ export function matchesUp(data: string, keybindings: KeybindingsManager): boolea
 
 export function matchesDown(data: string, keybindings: KeybindingsManager): boolean {
 	return keybindings.matches(data, "tui.select.down") || matchesKey(data, VIM_DOWN);
-}
+}

package/dist/extensions/question/package.json

@@ -8,11 +8,11 @@
 		"extensions": ["./index.ts"]
 	},
 	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": "*",
+		"@selesai/code": "*",
 		"@earendil-works/pi-tui": "*",
 		"typebox": "*"
 	},
 	"scripts": {
 		"test": "node --experimental-strip-types --test tests/**/*.test.ts"
 	}
-}
+}

package/dist/extensions/question/question-list.ts

@@ -2,7 +2,7 @@
 // Single-vs-multi is a SelectionMode flag, not a separate class.
 // Fuzzy search + split-pane preview are single-select-only (matching original behavior).
 
-import type { KeybindingsManager, Theme } from "@earendil-works/pi-coding-agent";
+import type { KeybindingsManager, Theme } from "@selesai/code";
 import {
 	type Component,
 	fuzzyFilter,
@@ -495,4 +495,4 @@ export class QuestionList implements Component {
 			return `${left}${separator}${right}`;
 		});
 	}
-}
+}

package/dist/extensions/question/tests/question-list.test.ts

@@ -5,7 +5,7 @@ import { Key, matchesKey } from "@earendil-works/pi-tui";
 
 import { QuestionList } from "../question-list.ts";
 import { MultiSelect, SingleSelect } from "../selection-mode.ts";
-import type { KeybindingsManager, Theme } from "@earendil-works/pi-coding-agent";
+import type { KeybindingsManager, Theme } from "@selesai/code";
 import type { ResolvedShortcut } from "../types.ts";
 
 // ---------------------------------------------------------------------------
@@ -227,4 +227,4 @@ test("onSubmit passes commentEnabled=true when comment is enabled", () => {
 	list.handleInput(CTRL_G); // enable comment
 	list.handleInput(ENTER); // confirm
 	assert.deepEqual(submitted, [["a"], true]);
-});
+});

package/dist/extensions/question/tui-adapter.ts

@@ -1,6 +1,6 @@
 // TUI protocol adapter — wraps ctx.ui into UIProtocol.
 
-import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+import type { ExtensionContext } from "@selesai/code";
 import type { OverlayHandle } from "@earendil-works/pi-tui";
 
 import { OVERLAY_MIN_WIDTH, OVERLAY_OVERLAY_MAX_HEIGHT, OVERLAY_WIDTH } from "./constants.ts";
@@ -30,4 +30,4 @@ export function createTUIProtocol(ctx: ExtensionContext): UIProtocol {
 		setStatus: (key, text) => ctx.ui.setStatus(key, text),
 		notify: (msg, level) => ctx.ui.notify(msg, level),
 	};
-}
+}

package/dist/extensions/question/types.ts

@@ -1,6 +1,6 @@
 // Shared type definitions for the question extension.
 
-import type { KeybindingsManager } from "@earendil-works/pi-coding-agent";
+import type { KeybindingsManager } from "@selesai/code";
 
 export type RawOption = string | { label: string; description?: string };
 
@@ -99,4 +99,4 @@ export interface ResolvedQuestionParams {
 	displayMode: DisplayMode;
 	shortcuts: ResolvedShortcuts;
 	timeout: number | undefined;
-}
+}

package/dist/extensions/question/ui-protocol.ts

@@ -1,6 +1,6 @@
 // UIProtocol port — abstracts TUI vs dialog/RPC transport.
 
-import type { KeybindingsManager } from "@earendil-works/pi-coding-agent";
+import type { KeybindingsManager } from "@selesai/code";
 import type { OverlayHandle, Theme, TUI } from "@earendil-works/pi-tui";
 
 import type { QuestionResponse, ResolvedQuestionParams } from "./types.ts";
@@ -43,4 +43,4 @@ export interface UIProtocol {
 
 export interface FallbackProtocol {
 	ask(params: ResolvedQuestionParams, protocol: UIProtocol): Promise<QuestionResponse | null>;
-}
+}

package/dist/extensions/rtk.ts

@@ -10,8 +10,8 @@
 //   1           No RTK equivalent → pass through unchanged
 //   3 + stdout  Rewrite (advisory) → mutate command
 
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent"
-import { isToolCallEventType } from "@earendil-works/pi-coding-agent"
+import type { ExtensionAPI } from "@selesai/code"
+import { isToolCallEventType } from "@selesai/code"
 
 const REWRITE_TIMEOUT_MS = 2_000
 const MIN_SUPPORTED_RTK_MINOR = 23

package/dist/extensions/tokenin-onboarding.ts

@@ -11,8 +11,8 @@
  */
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
-import { getAgentDir, getModelsPath } from "@earendil-works/pi-coding-agent";
-import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext, SessionStartEvent } from "@earendil-works/pi-coding-agent";
+import { getAgentDir, getModelsPath } from "@selesai/code";
+import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext, SessionStartEvent } from "@selesai/code";
 
 export const TOKEN_IN_PROVIDER = "tokenin";
 export const TOKEN_IN_DASHBOARD_URL = "https://token-in.selesai.in/dashboard/tokens";

package/dist/extensions/undo.ts

@@ -1,4 +1,4 @@
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI } from "@selesai/code";
 import { createHash } from "node:crypto";
 import { mkdir, readFile, rm, writeFile } from "node:fs/promises";
 import { existsSync } from "node:fs";

package/dist/extensions/web-agent-onboarding.ts

@@ -11,8 +11,8 @@
  */
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
-import { getAgentDir, getSettingsPath } from "@earendil-works/pi-coding-agent";
-import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext, SessionStartEvent } from "@earendil-works/pi-coding-agent";
+import { getAgentDir, getSettingsPath } from "@selesai/code";
+import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext, SessionStartEvent } from "@selesai/code";
 
 export type OnboardingResult =
 	| { kind: "saved" }
@@ -219,4 +219,4 @@ export default function webAgentOnboardingExtension(pi: ExtensionAPI): void {
 			}
 		},
 	});
-}
+}

package/dist/extensions/workflow/adapter.ts

@@ -8,7 +8,7 @@ import type {
   ExtensionCommandContext,
   ExtensionContext,
   ToolDefinition,
-} from "@earendil-works/pi-coding-agent";
+} from "@selesai/code";
 import { Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 import { access, mkdir, writeFile } from "node:fs/promises";
@@ -504,12 +504,20 @@ export function createWorkflowExtension(
       if (event.toolName === "subagent" && sm.snapshot.active && SUBAGENT_FALLBACK_PHASES.has(sm.snapshot.phase)) {
         const file = config.phaseArtifacts[sm.snapshot.phase];
         if (file) {
-          const expectedPath = resolve(sm.snapshot.artifactDir, file);
-          if (!(await realFileExists(expectedPath))) {
-            const text = textFromToolResultContent(event.content);
-            if (text) {
-              await mkdir(sm.snapshot.artifactDir, { recursive: true });
-              await writeFile(expectedPath, text, "utf8");
+          // ponytail: management actions (list/get/models/doctor/status/...)
+          // return text but are NOT the architect/recapper/... execution
+          // result. Writing their text to plan.md would advance the workflow
+          // on agent-listing output (the `subagent list` result is plain
+          // text). Execution calls set `agent`/`chain`/`tasks`; management
+          // calls set `action`. Only fall back for execution calls.
+          if (typeof event.input?.action !== "string") {
+            const expectedPath = resolve(sm.snapshot.artifactDir, file);
+            if (!(await realFileExists(expectedPath))) {
+              const text = textFromToolResultContent(event.content);
+              if (text) {
+                await mkdir(sm.snapshot.artifactDir, { recursive: true });
+                await writeFile(expectedPath, text, "utf8");
+              }
             }
           }
         }
@@ -585,4 +593,4 @@ export function createWorkflowExtension(
       },
     });
   };
-}
+}

package/dist/extensions/workflow/extension.ts

@@ -3,7 +3,7 @@
 // load conflict (multiple extension modules would each try to register the
 // shared writer).
 
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI } from "@selesai/code";
 import { __resetWorkflowRegistryForTests, createWorkflowExtension } from "./adapter.ts";
 import { prototypeMode } from "./modes/prototype.ts";
 import { quickMode } from "./modes/quick.ts";

package/dist/extensions/workflow/package.json

@@ -10,7 +10,7 @@
 		]
 	},
 	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": "*",
+		"@selesai/code": "*",
 		"@earendil-works/pi-tui": "*",
 		"typebox": "*"
 	}

package/dist/modes/interactive/components/assistant-message.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"assistant-message.d.ts","sourceRoot":"","sources":["../../../../src/modes/interactive/components/assistant-message.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAC9D,OAAO,EAAE,SAAS,EAAY,KAAK,aAAa,EAAgB,MAAM,wBAAwB,CAAC;AAO/F;;GAEG;AACH,qBAAa,yBAA0B,SAAQ,SAAS;IACvD,OAAO,CAAC,gBAAgB,CAAY;IACpC,OAAO,CAAC,iBAAiB,CAAU;IACnC,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,WAAW,CAAC,CAAmB;IACvC,OAAO,CAAC,YAAY,CAAS;IAE7B,YACC,OAAO,CAAC,EAAE,gBAAgB,EAC1B,iBAAiB,UAAQ,EACzB,aAAa,GAAE,aAAkC,EACjD,mBAAmB,SAAgB,EAenC;IAEQ,UAAU,IAAI,IAAI,CAK1B;IAED,oBAAoB,CAAC,IAAI,EAAE,OAAO,GAAG,IAAI,CAKxC;IAED,sBAAsB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,CAK1C;IAEQ,MAAM,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,CASvC;IAED,aAAa,CAAC,OAAO,EAAE,gBAAgB,GAAG,IAAI,CAyE7C;CACD","sourcesContent":["import type { AssistantMessage } from \"@earendil-works/pi-ai\";\nimport { Container, Markdown, type MarkdownTheme, Spacer, Text } from \"@earendil-works/pi-tui\";\nimport { getMarkdownTheme, theme } from \"../theme/theme.ts\";\n\nconst OSC133_ZONE_START = \"\\x1b]133;A\\x07\";\nconst OSC133_ZONE_END = \"\\x1b]133;B\\x07\";\nconst OSC133_ZONE_FINAL = \"\\x1b]133;C\\x07\";\n\n/**\n * Component that renders a complete assistant message\n */\nexport class AssistantMessageComponent extends Container {\n\tprivate contentContainer: Container;\n\tprivate hideThinkingBlock: boolean;\n\tprivate markdownTheme: MarkdownTheme;\n\tprivate hiddenThinkingLabel: string;\n\tprivate lastMessage?: AssistantMessage;\n\tprivate hasToolCalls = false;\n\n\tconstructor(\n\t\tmessage?: AssistantMessage,\n\t\thideThinkingBlock = false,\n\t\tmarkdownTheme: MarkdownTheme = getMarkdownTheme(),\n\t\thiddenThinkingLabel = \"Thinking...\",\n\t) {\n\t\tsuper();\n\n\t\tthis.hideThinkingBlock = hideThinkingBlock;\n\t\tthis.markdownTheme = markdownTheme;\n\t\tthis.hiddenThinkingLabel = hiddenThinkingLabel;\n\n\t\t// Container for text/thinking content\n\t\tthis.contentContainer = new Container();\n\t\tthis.addChild(this.contentContainer);\n\n\t\tif (message) {\n\t\t\tthis.updateContent(message);\n\t\t}\n\t}\n\n\toverride invalidate(): void {\n\t\tsuper.invalidate();\n\t\tif (this.lastMessage) {\n\t\t\tthis.updateContent(this.lastMessage);\n\t\t}\n\t}\n\n\tsetHideThinkingBlock(hide: boolean): void {\n\t\tthis.hideThinkingBlock = hide;\n\t\tif (this.lastMessage) {\n\t\t\tthis.updateContent(this.lastMessage);\n\t\t}\n\t}\n\n\tsetHiddenThinkingLabel(label: string): void {\n\t\tthis.hiddenThinkingLabel = label;\n\t\tif (this.lastMessage) {\n\t\t\tthis.updateContent(this.lastMessage);\n\t\t}\n\t}\n\n\toverride render(width: number): string[] {\n\t\tconst lines = super.render(width);\n\t\tif (this.hasToolCalls || lines.length === 0) {\n\t\t\treturn lines;\n\t\t}\n\n\t\tlines[0] = OSC133_ZONE_START + lines[0];\n\t\tlines[lines.length - 1] = OSC133_ZONE_END + OSC133_ZONE_FINAL + lines[lines.length - 1];\n\t\treturn lines;\n\t}\n\n\tupdateContent(message: AssistantMessage): void {\n\t\tthis.lastMessage = message;\n\n\t\t// Clear content container\n\t\tthis.contentContainer.clear();\n\n\t\tconst hasVisibleContent = message.content.some(\n\t\t\t(c) => (c.type === \"text\" && c.text.trim()) || (c.type === \"thinking\" && c.thinking.trim()),\n\t\t);\n\n\t\tif (hasVisibleContent) {\n\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t}\n\n\t\t// Render content in order\n\t\tfor (let i = 0; i < message.content.length; i++) {\n\t\t\tconst content = message.content[i];\n\t\t\tif (content.type === \"text\" && content.text.trim()) {\n\t\t\t\t// Assistant text messages with no background - trim the text\n\t\t\t\t// Set paddingY=0 to avoid extra spacing before tool executions\n\t\t\t\tthis.contentContainer.addChild(new Markdown(content.text.trim(), 1, 0, this.markdownTheme));\n\t\t\t} else if (content.type === \"thinking\" && content.thinking.trim()) {\n\t\t\t\t// Add spacing only when another visible assistant content block follows.\n\t\t\t\t// This avoids a superfluous blank line before separately-rendered tool execution blocks.\n\t\t\t\tconst hasVisibleContentAfter = message.content\n\t\t\t\t\t.slice(i + 1)\n\t\t\t\t\t.some((c) => (c.type === \"text\" && c.text.trim()) || (c.type === \"thinking\" && c.thinking.trim()));\n\n\t\t\t\tif (this.hideThinkingBlock) {\n\t\t\t\t\t// Show static thinking label when hidden\n\t\t\t\t\tthis.contentContainer.addChild(\n\t\t\t\t\t\tnew Text(theme.italic(theme.fg(\"thinkingText\", this.hiddenThinkingLabel)), 1, 0),\n\t\t\t\t\t);\n\t\t\t\t\tif (hasVisibleContentAfter) {\n\t\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t\t}\n\t\t\t\t} else {\n\t\t\t\t\t// Thinking traces in thinkingText color, italic\n\t\t\t\t\tthis.contentContainer.addChild(\n\t\t\t\t\t\tnew Markdown(content.thinking.trim(), 1, 0, this.markdownTheme, {\n\t\t\t\t\t\t\tcolor: (text: string) => theme.fg(\"thinkingText\", text),\n\t\t\t\t\t\t\titalic: true,\n\t\t\t\t\t\t}),\n\t\t\t\t\t);\n\t\t\t\t\tif (hasVisibleContentAfter) {\n\t\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t\t}\n\t\t\t\t}\n\t\t\t}\n\t\t}\n\n\t\t// Check if aborted - show after partial content\n\t\t// But only if there are no tool calls (tool execution components will show the error)\n\t\tconst hasToolCalls = message.content.some((c) => c.type === \"toolCall\");\n\t\tthis.hasToolCalls = hasToolCalls;\n\t\tif (!hasToolCalls) {\n\t\t\tif (message.stopReason === \"aborted\") {\n\t\t\t\tconst abortMessage =\n\t\t\t\t\tmessage.errorMessage && message.errorMessage !== \"Request was aborted\"\n\t\t\t\t\t\t? message.errorMessage\n\t\t\t\t\t\t: \"Operation aborted\";\n\t\t\t\tif (hasVisibleContent) {\n\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t} else {\n\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t}\n\t\t\t\tthis.contentContainer.addChild(new Text(theme.fg(\"error\", abortMessage), 1, 0));\n\t\t\t} else if (message.stopReason === \"error\") {\n\t\t\t\tconst errorMsg = message.errorMessage || \"Unknown error\";\n\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\tthis.contentContainer.addChild(new Text(theme.fg(\"error\", `Error: ${errorMsg}`), 1, 0));\n\t\t\t}\n\t\t}\n\t}\n}\n"]}
+{"version":3,"file":"assistant-message.d.ts","sourceRoot":"","sources":["../../../../src/modes/interactive/components/assistant-message.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAC9D,OAAO,EAAE,SAAS,EAAY,KAAK,aAAa,EAAgB,MAAM,wBAAwB,CAAC;AAQ/F;;GAEG;AACH,qBAAa,yBAA0B,SAAQ,SAAS;IACvD,OAAO,CAAC,gBAAgB,CAAY;IACpC,OAAO,CAAC,iBAAiB,CAAU;IACnC,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,mBAAmB,CAAS;IACpC,OAAO,CAAC,WAAW,CAAC,CAAmB;IACvC,OAAO,CAAC,YAAY,CAAS;IAE7B,YACC,OAAO,CAAC,EAAE,gBAAgB,EAC1B,iBAAiB,UAAQ,EACzB,aAAa,GAAE,aAAkC,EACjD,mBAAmB,SAAgB,EAenC;IAEQ,UAAU,IAAI,IAAI,CAK1B;IAED,oBAAoB,CAAC,IAAI,EAAE,OAAO,GAAG,IAAI,CAKxC;IAED,sBAAsB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,CAK1C;IAEQ,MAAM,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,EAAE,CASvC;IAED,aAAa,CAAC,OAAO,EAAE,gBAAgB,GAAG,IAAI,CA0E7C;CACD","sourcesContent":["import type { AssistantMessage } from \"@earendil-works/pi-ai\";\nimport { Container, Markdown, type MarkdownTheme, Spacer, Text } from \"@earendil-works/pi-tui\";\nimport { normalizeAssistantThinkingTags } from \"../../../utils/thinking-tags.ts\";\nimport { getMarkdownTheme, theme } from \"../theme/theme.ts\";\n\nconst OSC133_ZONE_START = \"\\x1b]133;A\\x07\";\nconst OSC133_ZONE_END = \"\\x1b]133;B\\x07\";\nconst OSC133_ZONE_FINAL = \"\\x1b]133;C\\x07\";\n\n/**\n * Component that renders a complete assistant message\n */\nexport class AssistantMessageComponent extends Container {\n\tprivate contentContainer: Container;\n\tprivate hideThinkingBlock: boolean;\n\tprivate markdownTheme: MarkdownTheme;\n\tprivate hiddenThinkingLabel: string;\n\tprivate lastMessage?: AssistantMessage;\n\tprivate hasToolCalls = false;\n\n\tconstructor(\n\t\tmessage?: AssistantMessage,\n\t\thideThinkingBlock = false,\n\t\tmarkdownTheme: MarkdownTheme = getMarkdownTheme(),\n\t\thiddenThinkingLabel = \"Thinking...\",\n\t) {\n\t\tsuper();\n\n\t\tthis.hideThinkingBlock = hideThinkingBlock;\n\t\tthis.markdownTheme = markdownTheme;\n\t\tthis.hiddenThinkingLabel = hiddenThinkingLabel;\n\n\t\t// Container for text/thinking content\n\t\tthis.contentContainer = new Container();\n\t\tthis.addChild(this.contentContainer);\n\n\t\tif (message) {\n\t\t\tthis.updateContent(message);\n\t\t}\n\t}\n\n\toverride invalidate(): void {\n\t\tsuper.invalidate();\n\t\tif (this.lastMessage) {\n\t\t\tthis.updateContent(this.lastMessage);\n\t\t}\n\t}\n\n\tsetHideThinkingBlock(hide: boolean): void {\n\t\tthis.hideThinkingBlock = hide;\n\t\tif (this.lastMessage) {\n\t\t\tthis.updateContent(this.lastMessage);\n\t\t}\n\t}\n\n\tsetHiddenThinkingLabel(label: string): void {\n\t\tthis.hiddenThinkingLabel = label;\n\t\tif (this.lastMessage) {\n\t\t\tthis.updateContent(this.lastMessage);\n\t\t}\n\t}\n\n\toverride render(width: number): string[] {\n\t\tconst lines = super.render(width);\n\t\tif (this.hasToolCalls || lines.length === 0) {\n\t\t\treturn lines;\n\t\t}\n\n\t\tlines[0] = OSC133_ZONE_START + lines[0];\n\t\tlines[lines.length - 1] = OSC133_ZONE_END + OSC133_ZONE_FINAL + lines[lines.length - 1];\n\t\treturn lines;\n\t}\n\n\tupdateContent(message: AssistantMessage): void {\n\t\tmessage = normalizeAssistantThinkingTags(message);\n\t\tthis.lastMessage = message;\n\n\t\t// Clear content container\n\t\tthis.contentContainer.clear();\n\n\t\tconst hasVisibleContent = message.content.some(\n\t\t\t(c) => (c.type === \"text\" && c.text.trim()) || (c.type === \"thinking\" && c.thinking.trim()),\n\t\t);\n\n\t\tif (hasVisibleContent) {\n\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t}\n\n\t\t// Render content in order\n\t\tfor (let i = 0; i < message.content.length; i++) {\n\t\t\tconst content = message.content[i];\n\t\t\tif (content.type === \"text\" && content.text.trim()) {\n\t\t\t\t// Assistant text messages with no background - trim the text\n\t\t\t\t// Set paddingY=0 to avoid extra spacing before tool executions\n\t\t\t\tthis.contentContainer.addChild(new Markdown(content.text.trim(), 1, 0, this.markdownTheme));\n\t\t\t} else if (content.type === \"thinking\" && content.thinking.trim()) {\n\t\t\t\t// Add spacing only when another visible assistant content block follows.\n\t\t\t\t// This avoids a superfluous blank line before separately-rendered tool execution blocks.\n\t\t\t\tconst hasVisibleContentAfter = message.content\n\t\t\t\t\t.slice(i + 1)\n\t\t\t\t\t.some((c) => (c.type === \"text\" && c.text.trim()) || (c.type === \"thinking\" && c.thinking.trim()));\n\n\t\t\t\tif (this.hideThinkingBlock) {\n\t\t\t\t\t// Show static thinking label when hidden\n\t\t\t\t\tthis.contentContainer.addChild(\n\t\t\t\t\t\tnew Text(theme.italic(theme.fg(\"thinkingText\", this.hiddenThinkingLabel)), 1, 0),\n\t\t\t\t\t);\n\t\t\t\t\tif (hasVisibleContentAfter) {\n\t\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t\t}\n\t\t\t\t} else {\n\t\t\t\t\t// Thinking traces in thinkingText color, italic\n\t\t\t\t\tthis.contentContainer.addChild(\n\t\t\t\t\t\tnew Markdown(content.thinking.trim(), 1, 0, this.markdownTheme, {\n\t\t\t\t\t\t\tcolor: (text: string) => theme.fg(\"thinkingText\", text),\n\t\t\t\t\t\t\titalic: true,\n\t\t\t\t\t\t}),\n\t\t\t\t\t);\n\t\t\t\t\tif (hasVisibleContentAfter) {\n\t\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t\t}\n\t\t\t\t}\n\t\t\t}\n\t\t}\n\n\t\t// Check if aborted - show after partial content\n\t\t// But only if there are no tool calls (tool execution components will show the error)\n\t\tconst hasToolCalls = message.content.some((c) => c.type === \"toolCall\");\n\t\tthis.hasToolCalls = hasToolCalls;\n\t\tif (!hasToolCalls) {\n\t\t\tif (message.stopReason === \"aborted\") {\n\t\t\t\tconst abortMessage =\n\t\t\t\t\tmessage.errorMessage && message.errorMessage !== \"Request was aborted\"\n\t\t\t\t\t\t? message.errorMessage\n\t\t\t\t\t\t: \"Operation aborted\";\n\t\t\t\tif (hasVisibleContent) {\n\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t} else {\n\t\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\t}\n\t\t\t\tthis.contentContainer.addChild(new Text(theme.fg(\"error\", abortMessage), 1, 0));\n\t\t\t} else if (message.stopReason === \"error\") {\n\t\t\t\tconst errorMsg = message.errorMessage || \"Unknown error\";\n\t\t\t\tthis.contentContainer.addChild(new Spacer(1));\n\t\t\t\tthis.contentContainer.addChild(new Text(theme.fg(\"error\", `Error: ${errorMsg}`), 1, 0));\n\t\t\t}\n\t\t}\n\t}\n}\n"]}

package/dist/modes/interactive/components/assistant-message.js

@@ -1,4 +1,5 @@
 import { Container, Markdown, Spacer, Text } from "@earendil-works/pi-tui";
+import { normalizeAssistantThinkingTags } from "../../../utils/thinking-tags.js";
 import { getMarkdownTheme, theme } from "../theme/theme.js";
 const OSC133_ZONE_START = "\x1b]133;A\x07";
 const OSC133_ZONE_END = "\x1b]133;B\x07";
@@ -53,6 +54,7 @@ export class AssistantMessageComponent extends Container {
         return lines;
     }
     updateContent(message) {
+        message = normalizeAssistantThinkingTags(message);
         this.lastMessage = message;
         // Clear content container
         this.contentContainer.clear();