takomi 2.1.13 → 2.1.15

package/.pi/extensions/takomi-context-manager/diagnostics-tools.ts

@@ -1,28 +1,28 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
-import { Type } from "typebox";
-import type { ContextManagerState } from "./state";
-import { renderReport } from "./diagnostics";
-
-export function registerDiagnostics(pi: ExtensionAPI, state: ContextManagerState): void {
-  pi.registerTool({
-    name: "context_report",
-    label: "Context Report",
-    description: "Show takomi-context-manager diagnostics for prompt rewriting, candidates, policies, gates, and progressive skill loading.",
-    promptSnippet: "Show context manager diagnostics and prompt composition decisions",
-    parameters: Type.Object({ verbose: Type.Optional(Type.Boolean({ description: "Include skill and policy indexes in the report" })) }),
-    async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
-      state.report.cwd = ctx.cwd;
-      state.report.toolCalls.contextReport += 1;
-      return { content: [{ type: "text", text: renderReport(state, params.verbose ?? false) }], details: state.report };
-    },
-  });
-
-  pi.registerCommand("context-report", {
-    description: "Show takomi-context-manager diagnostics",
-    handler: async (_args, ctx) => {
-      state.report.cwd = ctx.cwd;
-      state.report.toolCalls.contextReport += 1;
-      ctx.ui.notify(renderReport(state, true), "info");
-    },
-  });
-}
+import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import { Type } from "typebox";
+import type { ContextManagerState } from "./state";
+import { renderReport } from "./diagnostics";
+
+export function registerDiagnostics(pi: ExtensionAPI, state: ContextManagerState): void {
+  pi.registerTool({
+    name: "context_report",
+    label: "Context Report",
+    description: "Show takomi-context-manager diagnostics for prompt rewriting, candidates, policies, gates, and progressive skill loading.",
+    promptSnippet: "Show context manager diagnostics and prompt composition decisions",
+    parameters: Type.Object({ verbose: Type.Optional(Type.Boolean({ description: "Include skill and policy indexes in the report" })) }),
+    async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
+      state.report.cwd = ctx.cwd;
+      state.report.toolCalls.contextReport += 1;
+      return { content: [{ type: "text", text: renderReport(state, params.verbose ?? false) }], details: state.report };
+    },
+  });
+
+  pi.registerCommand("context-report", {
+    description: "Show takomi-context-manager diagnostics",
+    handler: async (_args, ctx) => {
+      state.report.cwd = ctx.cwd;
+      state.report.toolCalls.contextReport += 1;
+      ctx.ui.notify(renderReport(state, true), "info");
+    },
+  });
+}

package/.pi/extensions/takomi-context-manager/diagnostics.ts

@@ -1,55 +1,55 @@
-import type { ContextManagerState } from "./state";
-import { sortedSkills } from "./skill-registry";
-import { syncReportLedger } from "./state";
-import { renderDuplicateExtensionGuidance } from "./extension-conflicts";
-
-export function renderReport(state: ContextManagerState, verbose = false): string {
-  syncReportLedger(state);
-  const report = state.report;
-  const reduction = report.promptRewrite.originalLength > 0
-    ? Math.round((1 - report.promptRewrite.rewrittenLength / report.promptRewrite.originalLength) * 1000) / 10
-    : 0;
-  const lines = [
-    "Takomi Context Manager Report",
-    `- Timestamp: ${report.timestamp}`,
-    `- CWD: ${report.cwd || "(unknown)"}`,
-    "- Scope: latest context-manager prompt rewrite and context-manager tool state only; this is not a full Pi session transcript.",
-    `- Skill count discovered: ${report.skillCount}`,
-    `- Prompt rewrite attempted: ${report.promptRewrite.attempted ? "yes" : "no"}`,
-    `- Prompt changed: ${report.promptRewrite.changed ? "yes" : "no"}`,
-    `- Original prompt length observed by context manager: ${report.promptRewrite.originalLength} chars`,
-    `- Rewritten prompt length returned by context manager: ${report.promptRewrite.rewrittenLength} chars`,
-    `- Context-manager prompt reduction: ${reduction}%`,
-    `- Sections compressed/removed by context manager: ${report.promptRewrite.removedSections.join(", ") || "none"}`,
-    `- Skills loaded through skill_load: ${report.loadedByTool.join(", ") || "none"}`,
-    `- Policies loaded through policy_load/gates: ${report.loadedPolicies.join(", ") || "none"}`,
-    `- Context-manager tracked read files: ${report.readFiles.length}`,
-    `- Context-manager tracked edited files: ${report.editedFiles.length}`,
-    `- Context-manager tracked written files: ${report.writtenFiles.length}`,
-    `- Blocked context-manager gated actions: ${report.blockedActions.length}`,
-    `- Duplicate Takomi extension tool-name warnings: ${report.duplicateExtensionWarnings.length}`,
-    `- Context-manager tool calls: skill_index=${report.toolCalls.skillIndex}, skill_manifest=${report.toolCalls.skillManifest}, skill_load=${report.toolCalls.skillLoad}, policy_manifest=${report.toolCalls.policyManifest}, policy_load=${report.toolCalls.policyLoad}, context_report=${report.toolCalls.contextReport}`,
-    `- Rewrite warnings: ${report.promptRewrite.warnings.join("; ") || "none"}`,
-    "- Note: native Pi read/edit/write/bash calls are not counted here unless a context-manager tool explicitly records them.",
-  ];
-  if (report.candidates.length > 0) {
-    lines.push("- Candidates:");
-    for (const candidate of report.candidates) {
-      lines.push(`  - ${candidate.name} (${candidate.confidence}, ${candidate.score}): ${candidate.reasons.join("; ")}`);
-    }
-  } else {
-    lines.push("- Candidates: none");
-  }
-  if (report.duplicateExtensionWarnings.length > 0 || verbose) {
-    lines.push("", "Extension Conflict Diagnostics:", ...renderDuplicateExtensionGuidance(report.duplicateExtensionWarnings));
-  }
-  if (verbose) {
-    lines.push("", "Skill Index:", ...sortedSkills(state.skills).map((skill) => `- ${skill.name}`));
-    lines.push("", "Policy Packs:", ...[...state.policies.values()].map((policy) => `- ${policy.name}: ${policy.description}`));
-    if (report.blockedActions.length > 0) {
-      lines.push("", "Blocked Actions:");
-      for (const action of report.blockedActions.slice(-10)) lines.push(`- ${action.timestamp} ${action.toolName}: ${action.reason}`);
-    }
-  }
-  return lines.join("\n");
-}
+import type { ContextManagerState } from "./state";
+import { sortedSkills } from "./skill-registry";
+import { syncReportLedger } from "./state";
+import { renderDuplicateExtensionGuidance } from "./extension-conflicts";
+
+export function renderReport(state: ContextManagerState, verbose = false): string {
+  syncReportLedger(state);
+  const report = state.report;
+  const reduction = report.promptRewrite.originalLength > 0
+    ? Math.round((1 - report.promptRewrite.rewrittenLength / report.promptRewrite.originalLength) * 1000) / 10
+    : 0;
+  const lines = [
+    "Takomi Context Manager Report",
+    `- Timestamp: ${report.timestamp}`,
+    `- CWD: ${report.cwd || "(unknown)"}`,
+    "- Scope: latest context-manager prompt rewrite and context-manager tool state only; this is not a full Pi session transcript.",
+    `- Skill count discovered: ${report.skillCount}`,
+    `- Prompt rewrite attempted: ${report.promptRewrite.attempted ? "yes" : "no"}`,
+    `- Prompt changed: ${report.promptRewrite.changed ? "yes" : "no"}`,
+    `- Original prompt length observed by context manager: ${report.promptRewrite.originalLength} chars`,
+    `- Rewritten prompt length returned by context manager: ${report.promptRewrite.rewrittenLength} chars`,
+    `- Context-manager prompt reduction: ${reduction}%`,
+    `- Sections compressed/removed by context manager: ${report.promptRewrite.removedSections.join(", ") || "none"}`,
+    `- Skills loaded through skill_load: ${report.loadedByTool.join(", ") || "none"}`,
+    `- Policies loaded through policy_load/gates: ${report.loadedPolicies.join(", ") || "none"}`,
+    `- Context-manager tracked read files: ${report.readFiles.length}`,
+    `- Context-manager tracked edited files: ${report.editedFiles.length}`,
+    `- Context-manager tracked written files: ${report.writtenFiles.length}`,
+    `- Blocked context-manager gated actions: ${report.blockedActions.length}`,
+    `- Duplicate Takomi extension tool-name warnings: ${report.duplicateExtensionWarnings.length}`,
+    `- Context-manager tool calls: skill_index=${report.toolCalls.skillIndex}, skill_manifest=${report.toolCalls.skillManifest}, skill_load=${report.toolCalls.skillLoad}, policy_manifest=${report.toolCalls.policyManifest}, policy_load=${report.toolCalls.policyLoad}, context_report=${report.toolCalls.contextReport}`,
+    `- Rewrite warnings: ${report.promptRewrite.warnings.join("; ") || "none"}`,
+    "- Note: native Pi read/edit/write/bash calls are not counted here unless a context-manager tool explicitly records them.",
+  ];
+  if (report.candidates.length > 0) {
+    lines.push("- Candidates:");
+    for (const candidate of report.candidates) {
+      lines.push(`  - ${candidate.name} (${candidate.confidence}, ${candidate.score}): ${candidate.reasons.join("; ")}`);
+    }
+  } else {
+    lines.push("- Candidates: none");
+  }
+  if (report.duplicateExtensionWarnings.length > 0 || verbose) {
+    lines.push("", "Extension Conflict Diagnostics:", ...renderDuplicateExtensionGuidance(report.duplicateExtensionWarnings));
+  }
+  if (verbose) {
+    lines.push("", "Skill Index:", ...sortedSkills(state.skills).map((skill) => `- ${skill.name}`));
+    lines.push("", "Policy Packs:", ...[...state.policies.values()].map((policy) => `- ${policy.name}: ${policy.description}`));
+    if (report.blockedActions.length > 0) {
+      lines.push("", "Blocked Actions:");
+      for (const action of report.blockedActions.slice(-10)) lines.push(`- ${action.timestamp} ${action.toolName}: ${action.reason}`);
+    }
+  }
+  return lines.join("\n");
+}

package/.pi/extensions/takomi-context-manager/extension-conflicts.ts

@@ -1,56 +1,56 @@
-import { access } from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-import type { ContextManagerState } from "./state";
-
-type KnownTakomiExtension = {
-  toolName: string;
-  relativePath: string;
-};
-
-const KNOWN_TAKOMI_EXTENSIONS: KnownTakomiExtension[] = [
-  { toolName: "takomi_workflow", relativePath: path.join("takomi-runtime", "index.ts") },
-  { toolName: "takomi_board", relativePath: path.join("takomi-runtime", "index.ts") },
-  { toolName: "takomi_subagent", relativePath: path.join("takomi-subagents", "index.ts") },
-];
-
-async function exists(filePath: string): Promise<boolean> {
-  try {
-    await access(filePath);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-function homePiExtensionsRoot(): string {
-  return path.join(os.homedir(), ".pi", "agent", "extensions");
-}
-
-export async function detectDuplicateTakomiExtensions(cwd: string): Promise<Array<{ toolName: string; paths: string[] }>> {
-  const globalRoot = homePiExtensionsRoot();
-  const projectRoot = path.resolve(cwd, ".pi", "extensions");
-  const warnings: Array<{ toolName: string; paths: string[] }> = [];
-
-  for (const known of KNOWN_TAKOMI_EXTENSIONS) {
-    const globalPath = path.join(globalRoot, known.relativePath);
-    const projectPath = path.join(projectRoot, known.relativePath);
-    if (await exists(globalPath) && await exists(projectPath)) {
-      warnings.push({ toolName: known.toolName, paths: [globalPath, projectPath] });
-    }
-  }
-
-  return warnings;
-}
-
-export function renderDuplicateExtensionGuidance(warnings: Array<{ toolName: string; paths: string[] }>): string[] {
-  if (warnings.length === 0) return ["- Duplicate Takomi extensions: none detected"];
-  const lines = ["- Duplicate Takomi extensions detected:"];
-  for (const warning of warnings) {
-    lines.push(`  - ${warning.toolName}`);
-    for (const filePath of warning.paths) lines.push(`    - ${filePath}`);
-  }
-  lines.push("- Recommended dev command: use scripts/pi-dev.ps1, which starts Pi with --no-extensions and explicit project-local Takomi extensions.");
-  lines.push("- For global Pi sessions, disable/remove one duplicate source or prefer explicit extension loading to avoid tool registration conflicts.");
-  return lines;
-}
+import { access } from "node:fs/promises";
+import os from "node:os";
+import path from "node:path";
+import type { ContextManagerState } from "./state";
+
+type KnownTakomiExtension = {
+  toolName: string;
+  relativePath: string;
+};
+
+const KNOWN_TAKOMI_EXTENSIONS: KnownTakomiExtension[] = [
+  { toolName: "takomi_workflow", relativePath: path.join("takomi-runtime", "index.ts") },
+  { toolName: "takomi_board", relativePath: path.join("takomi-runtime", "index.ts") },
+  { toolName: "takomi_subagent", relativePath: path.join("takomi-subagents", "index.ts") },
+];
+
+async function exists(filePath: string): Promise<boolean> {
+  try {
+    await access(filePath);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+function homePiExtensionsRoot(): string {
+  return path.join(os.homedir(), ".pi", "agent", "extensions");
+}
+
+export async function detectDuplicateTakomiExtensions(cwd: string): Promise<Array<{ toolName: string; paths: string[] }>> {
+  const globalRoot = homePiExtensionsRoot();
+  const projectRoot = path.resolve(cwd, ".pi", "extensions");
+  const warnings: Array<{ toolName: string; paths: string[] }> = [];
+
+  for (const known of KNOWN_TAKOMI_EXTENSIONS) {
+    const globalPath = path.join(globalRoot, known.relativePath);
+    const projectPath = path.join(projectRoot, known.relativePath);
+    if (await exists(globalPath) && await exists(projectPath)) {
+      warnings.push({ toolName: known.toolName, paths: [globalPath, projectPath] });
+    }
+  }
+
+  return warnings;
+}
+
+export function renderDuplicateExtensionGuidance(warnings: Array<{ toolName: string; paths: string[] }>): string[] {
+  if (warnings.length === 0) return ["- Duplicate Takomi extensions: none detected"];
+  const lines = ["- Duplicate Takomi extensions detected:"];
+  for (const warning of warnings) {
+    lines.push(`  - ${warning.toolName}`);
+    for (const filePath of warning.paths) lines.push(`    - ${filePath}`);
+  }
+  lines.push("- Recommended dev command: use scripts/pi-dev.ps1, which starts Pi with --no-extensions and explicit project-local Takomi extensions.");
+  lines.push("- For global Pi sessions, disable/remove one duplicate source or prefer explicit extension loading to avoid tool registration conflicts.");
+  return lines;
+}

package/.pi/extensions/takomi-context-manager/index.ts

@@ -1,56 +1,56 @@
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
-import { loadConfig, DEFAULT_CONFIG } from "./config";
-import { createState } from "./state";
-import { collectSkillsFromOptions, collectSkillsFromXml, mergeSkills } from "./skill-registry";
-import { discoverPolicies } from "./policy-registry";
-import { findCandidates } from "./context-router";
-import { rewritePrompt } from "./prompt-rewriter";
-import { registerSkillTools } from "./skill-tools";
-import { registerPolicyTools } from "./policy-tools";
-import { registerDiagnostics } from "./diagnostics-tools";
-import { installPrerequisiteGates } from "./prerequisite-gates";
-import { installModelPolicyGate } from "./model-policy-gate";
-import { detectDuplicateTakomiExtensions } from "./extension-conflicts";
-import type { ContextManagerConfig } from "./types";
-
-export default function takomiContextManager(pi: ExtensionAPI) {
-  const state = createState();
-  let config: ContextManagerConfig = DEFAULT_CONFIG;
-
-  registerSkillTools(pi, state);
-  registerPolicyTools(pi, state);
-  registerDiagnostics(pi, state);
-  installPrerequisiteGates(pi, state, () => config);
-  installModelPolicyGate(pi, state);
-
-  pi.on("before_agent_start", async (event, ctx) => {
-    config = await loadConfig(ctx.cwd);
-    state.policies = await discoverPolicies(ctx.cwd, config);
-    const duplicateExtensionWarnings = await detectDuplicateTakomiExtensions(ctx.cwd);
-    const optionSkills = collectSkillsFromOptions(event.systemPromptOptions);
-    const xmlSkills = collectSkillsFromXml(event.systemPrompt);
-    state.skills = mergeSkills([...optionSkills, ...xmlSkills]);
-
-    const candidates = findCandidates(event.prompt, state.skills, config);
-    const rewrite = rewritePrompt(event.systemPrompt, state.skills, candidates, config);
-    state.report = {
-      ...state.report,
-      timestamp: new Date().toISOString(),
-      cwd: ctx.cwd,
-      userPrompt: event.prompt,
-      skillCount: state.skills.size,
-      candidates,
-      duplicateExtensionWarnings,
-      promptRewrite: {
-        attempted: true,
-        changed: rewrite.changed,
-        originalLength: event.systemPrompt.length,
-        rewrittenLength: rewrite.prompt.length,
-        removedSections: rewrite.removedSections,
-        warnings: rewrite.warnings,
-      },
-    };
-
-    return { systemPrompt: rewrite.prompt };
-  });
-}
+import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import { loadConfig, DEFAULT_CONFIG } from "./config";
+import { createState } from "./state";
+import { collectSkillsFromOptions, collectSkillsFromXml, mergeSkills } from "./skill-registry";
+import { discoverPolicies } from "./policy-registry";
+import { findCandidates } from "./context-router";
+import { rewritePrompt } from "./prompt-rewriter";
+import { registerSkillTools } from "./skill-tools";
+import { registerPolicyTools } from "./policy-tools";
+import { registerDiagnostics } from "./diagnostics-tools";
+import { installPrerequisiteGates } from "./prerequisite-gates";
+import { installModelPolicyGate } from "./model-policy-gate";
+import { detectDuplicateTakomiExtensions } from "./extension-conflicts";
+import type { ContextManagerConfig } from "./types";
+
+export default function takomiContextManager(pi: ExtensionAPI) {
+  const state = createState();
+  let config: ContextManagerConfig = DEFAULT_CONFIG;
+
+  registerSkillTools(pi, state);
+  registerPolicyTools(pi, state);
+  registerDiagnostics(pi, state);
+  installPrerequisiteGates(pi, state, () => config);
+  installModelPolicyGate(pi, state);
+
+  pi.on("before_agent_start", async (event, ctx) => {
+    config = await loadConfig(ctx.cwd);
+    state.policies = await discoverPolicies(ctx.cwd, config);
+    const duplicateExtensionWarnings = await detectDuplicateTakomiExtensions(ctx.cwd);
+    const optionSkills = collectSkillsFromOptions(event.systemPromptOptions);
+    const xmlSkills = collectSkillsFromXml(event.systemPrompt);
+    state.skills = mergeSkills([...optionSkills, ...xmlSkills]);
+
+    const candidates = findCandidates(event.prompt, state.skills, config);
+    const rewrite = rewritePrompt(event.systemPrompt, state.skills, candidates, config);
+    state.report = {
+      ...state.report,
+      timestamp: new Date().toISOString(),
+      cwd: ctx.cwd,
+      userPrompt: event.prompt,
+      skillCount: state.skills.size,
+      candidates,
+      duplicateExtensionWarnings,
+      promptRewrite: {
+        attempted: true,
+        changed: rewrite.changed,
+        originalLength: event.systemPrompt.length,
+        rewrittenLength: rewrite.prompt.length,
+        removedSections: rewrite.removedSections,
+        warnings: rewrite.warnings,
+      },
+    };
+
+    return { systemPrompt: rewrite.prompt };
+  });
+}