@oh-my-pi/pi-coding-agent 3.4.1337 → 3.5.1337

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 ## [Unreleased]
 
+## [3.5.1337] - 2026-01-03
+
+### Added
+
+- Added session header and footer output in text mode showing version, model, provider, thinking level, and session ID
+- Added Extension Control Center dashboard accessible via `/extensions` command for unified management of all providers and extensions
+- Added ability to enable/disable individual extensions with persistent settings
+- Added three-column dashboard layout with sidebar tree, extension list, and inspector panel
+- Added fuzzy search filtering for extensions in the dashboard
+- Added keyboard navigation with Tab to cycle panes, j/k for navigation, Space to toggle, Enter to expand/collapse
+
+### Changed
+
+- Redesigned Extension Control Center from 3-column layout to tabbed interface with horizontal provider tabs and 2-column grid
+- Replaced sidebar tree navigation with provider tabs using TAB/Shift+TAB cycling
+
+### Fixed
+
+- Fixed title generation flag not resetting when starting a new session
+
 ## [3.4.1337] - 2026-01-03
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "3.4.1337",
+	"version": "3.5.1337",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -39,9 +39,9 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-agent-core": "3.4.1337",
-		"@oh-my-pi/pi-ai": "3.4.1337",
-		"@oh-my-pi/pi-tui": "3.4.1337",
+		"@oh-my-pi/pi-agent-core": "3.5.1337",
+		"@oh-my-pi/pi-ai": "3.5.1337",
+		"@oh-my-pi/pi-tui": "3.5.1337",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",
 		"chalk": "^5.5.0",

package/src/core/sdk.ts

@@ -79,8 +79,10 @@ import {
 	createLsTool,
 	createReadOnlyTools,
 	createReadTool,
+	createRulebookTool,
 	createWriteTool,
 	editTool,
+	filterRulebookRules,
 	findTool,
 	grepTool,
 	lsTool,
@@ -604,7 +606,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 	const skills = options.skills ?? discoverSkills(cwd, agentDir, settingsManager.getSkillsSettings());
 	time("discoverSkills");
 
-	// Discover TTSR rules
+	// Discover rules
 	const ttsrManager = createTtsrManager(settingsManager.getTtsrSettings());
 	const rulesResult = loadCapability<Rule>(ruleCapability.id, { cwd });
 	for (const rule of rulesResult.items) {
@@ -614,6 +616,10 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 	}
 	time("discoverTtsrRules");
 
+	// Filter rules for the rulebook (non-TTSR, non-alwaysApply, with descriptions)
+	const rulebookRules = filterRulebookRules(rulesResult.items);
+	time("filterRulebookRules");
+
 	const contextFiles = options.contextFiles ?? discoverContextFiles(cwd, agentDir);
 	time("discoverContextFiles");
 
@@ -757,6 +763,11 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 
 	let allToolsArray: Tool[] = [...builtInTools, ...wrappedCustomTools];
 
+	// Add rulebook tool if there are rules with descriptions (always enabled, regardless of --tools)
+	if (rulebookRules.length > 0) {
+		allToolsArray.push(createRulebookTool(rulebookRules));
+	}
+
 	// Filter out hidden tools unless explicitly requested
 	if (options.explicitTools) {
 		const explicitSet = new Set(options.explicitTools);
@@ -781,6 +792,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 		cwd,
 		skills,
 		contextFiles,
+		rulebookRules,
 	});
 	time("buildSystemPrompt");
 
@@ -791,6 +803,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 			cwd,
 			skills,
 			contextFiles,
+			rulebookRules,
 			customPrompt: options.systemPrompt,
 		});
 	} else {

package/src/core/settings-manager.ts

@@ -105,6 +105,7 @@ export interface Settings {
 	edit?: EditSettings;
 	ttsr?: TtsrSettings;
 	disabledProviders?: string[]; // Discovery provider IDs that are disabled
+	disabledExtensions?: string[]; // Individual extension IDs that are disabled (e.g., "skill:commit")
 }
 
 /** Deep merge settings: project/overrides take precedence, nested objects merge recursively */
@@ -594,6 +595,38 @@ export class SettingsManager {
 		this.save();
 	}
 
+	getDisabledExtensions(): string[] {
+		return [...(this.settings.disabledExtensions ?? [])];
+	}
+
+	setDisabledExtensions(extensionIds: string[]): void {
+		this.globalSettings.disabledExtensions = extensionIds;
+		this.save();
+	}
+
+	isExtensionEnabled(extensionId: string): boolean {
+		return !(this.settings.disabledExtensions ?? []).includes(extensionId);
+	}
+
+	enableExtension(extensionId: string): void {
+		const disabled = this.globalSettings.disabledExtensions ?? [];
+		const index = disabled.indexOf(extensionId);
+		if (index !== -1) {
+			disabled.splice(index, 1);
+			this.globalSettings.disabledExtensions = disabled;
+			this.save();
+		}
+	}
+
+	disableExtension(extensionId: string): void {
+		const disabled = this.globalSettings.disabledExtensions ?? [];
+		if (!disabled.includes(extensionId)) {
+			disabled.push(extensionId);
+			this.globalSettings.disabledExtensions = disabled;
+			this.save();
+		}
+	}
+
 	getTtsrSettings(): TtsrSettings {
 		return this.settings.ttsr ?? {};
 	}

package/src/core/system-prompt.ts

@@ -5,12 +5,14 @@
 import { existsSync, readFileSync } from "node:fs";
 import chalk from "chalk";
 import { contextFileCapability } from "../capability/context-file";
+import type { Rule } from "../capability/rule";
 import { systemPromptCapability } from "../capability/system-prompt";
 import { getDocsPath, getExamplesPath, getReadmePath } from "../config";
 import { type ContextFile, loadSync, type SystemPrompt as SystemPromptFile } from "../discovery/index";
 import type { SkillsSettings } from "./settings-manager";
 import { formatSkillsForPrompt, loadSkills, type Skill } from "./skills";
 import type { ToolName } from "./tools/index";
+import { formatRulesForPrompt } from "./tools/rulebook";
 
 /**
  * Execute a git command synchronously and return stdout or null on failure.
@@ -238,6 +240,8 @@ export interface BuildSystemPromptOptions {
 	contextFiles?: Array<{ path: string; content: string; depth?: number }>;
 	/** Pre-loaded skills (skips discovery if provided). */
 	skills?: Skill[];
+	/** Pre-loaded rulebook rules (rules with descriptions, excluding TTSR and always-apply). */
+	rulebookRules?: Rule[];
 }
 
 /** Build the system prompt with tools, guidelines, and context */
@@ -250,6 +254,7 @@ export function buildSystemPrompt(options: BuildSystemPromptOptions = {}): strin
 		cwd,
 		contextFiles: providedContextFiles,
 		skills: providedSkills,
+		rulebookRules,
 	} = options;
 	const resolvedCwd = cwd ?? process.cwd();
 	const resolvedCustomPrompt = resolvePromptInput(customPrompt, "system prompt");
@@ -310,6 +315,11 @@ export function buildSystemPrompt(options: BuildSystemPromptOptions = {}): strin
 			prompt += formatSkillsForPrompt(skills);
 		}
 
+		// Append rules section (always enabled when rules exist)
+		if (rulebookRules && rulebookRules.length > 0) {
+			prompt += formatRulesForPrompt(rulebookRules);
+		}
+
 		// Add date/time and working directory last
 		prompt += `\nCurrent date and time: ${dateTime}`;
 		prompt += `\nCurrent working directory: ${resolvedCwd}`;
@@ -419,6 +429,11 @@ Documentation:
 		prompt += formatSkillsForPrompt(skills);
 	}
 
+	// Append rules section (always enabled when rules exist)
+	if (rulebookRules && rulebookRules.length > 0) {
+		prompt += formatRulesForPrompt(rulebookRules);
+	}
+
 	// Add date/time and working directory last
 	prompt += `\nCurrent date and time: ${dateTime}`;
 	prompt += `\nCurrent working directory: ${resolvedCwd}`;

package/src/core/title-generator.ts

@@ -4,6 +4,7 @@
 
 import type { Model } from "@oh-my-pi/pi-ai";
 import { completeSimple } from "@oh-my-pi/pi-ai";
+import { logger } from "./logger";
 import type { ModelRegistry } from "./model-registry";
 import { findSmolModel } from "./model-resolver";
 
@@ -43,21 +44,35 @@ export async function generateSessionTitle(
 	savedSmolModel?: string,
 ): Promise<string | null> {
 	const model = await findTitleModel(registry, savedSmolModel);
-	if (!model) return null;
+	if (!model) {
+		logger.debug("title-generator: no smol model found");
+		return null;
+	}
 
 	const apiKey = await registry.getApiKey(model);
-	if (!apiKey) return null;
+	if (!apiKey) {
+		logger.debug("title-generator: no API key for model", { provider: model.provider, id: model.id });
+		return null;
+	}
 
 	// Truncate message if too long
 	const truncatedMessage =
 		firstMessage.length > MAX_INPUT_CHARS ? `${firstMessage.slice(0, MAX_INPUT_CHARS)}...` : firstMessage;
 
+	const request = {
+		model: `${model.provider}/${model.id}`,
+		systemPrompt: TITLE_SYSTEM_PROMPT,
+		userMessage: `<user-message>\n${truncatedMessage}\n</user-message>`,
+		maxTokens: 30,
+	};
+	logger.debug("title-generator: request", request);
+
 	try {
 		const response = await completeSimple(
 			model,
 			{
-				systemPrompt: TITLE_SYSTEM_PROMPT,
-				messages: [{ role: "user", content: truncatedMessage, timestamp: Date.now() }],
+				systemPrompt: request.systemPrompt,
+				messages: [{ role: "user", content: request.userMessage, timestamp: Date.now() }],
 			},
 			{
 				apiKey,
@@ -74,13 +89,20 @@ export async function generateSessionTitle(
 		}
 		title = title.trim();
 
-		if (!title || title.length > 60) {
+		logger.debug("title-generator: response", {
+			title,
+			usage: response.usage,
+			stopReason: response.stopReason,
+		});
+
+		if (!title) {
 			return null;
 		}
 
 		// Clean up: remove quotes, trailing punctuation
 		return title.replace(/^["']|["']$/g, "").replace(/[.!?]$/, "");
-	} catch {
+	} catch (err) {
+		logger.debug("title-generator: error", { error: err instanceof Error ? err.message : String(err) });
 		return null;
 	}
 }

package/src/core/tools/index.ts

@@ -24,6 +24,12 @@ export { createNotebookTool, type NotebookToolDetails, notebookTool } from "./no
 export { createOutputTool, type OutputToolDetails, outputTool } from "./output";
 export { createReadTool, type ReadToolDetails, readTool } from "./read";
 export { createReportFindingTool, createSubmitReviewTool, reportFindingTool, submitReviewTool } from "./review";
+export {
+	createRulebookTool,
+	filterRulebookRules,
+	formatRulesForPrompt,
+	type RulebookToolDetails,
+} from "./rulebook";
 export { BUNDLED_AGENTS, createTaskTool, taskTool } from "./task/index";
 export type { TruncationResult } from "./truncate";
 export { createWebFetchTool, type WebFetchToolDetails, webFetchCustomTool, webFetchTool } from "./web-fetch";

package/src/core/tools/rulebook.ts

@@ -0,0 +1,124 @@
+/**
+ * Rulebook Tool
+ *
+ * Allows the agent to fetch full content of rules that have descriptions.
+ * Rules are listed in the system prompt with name + description; this tool
+ * retrieves the complete rule content on demand.
+ */
+
+import type { AgentTool } from "@oh-my-pi/pi-agent-core";
+import { Type } from "@sinclair/typebox";
+import type { Rule } from "../../capability/rule";
+
+export interface RulebookToolDetails {
+	type: "rulebook";
+	ruleName: string;
+	found: boolean;
+	content?: string;
+}
+
+const rulebookSchema = Type.Object({
+	name: Type.String({ description: "The name of the rule to fetch" }),
+});
+
+/**
+ * Create a rulebook tool with access to discovered rules.
+ * @param rules - Array of discovered rules (non-TTSR rules with descriptions)
+ */
+export function createRulebookTool(rules: Rule[]): AgentTool<typeof rulebookSchema> {
+	// Build lookup map for O(1) access
+	const ruleMap = new Map<string, Rule>();
+	for (const rule of rules) {
+		ruleMap.set(rule.name, rule);
+	}
+
+	const ruleNames = rules.map((r) => r.name);
+
+	return {
+		name: "rulebook",
+		label: "Rulebook",
+		description: `Fetch the full content of a project rule by name. Use this when a rule listed in <available_rules> is relevant to your current task. Available: ${ruleNames.join(", ") || "(none)"}`,
+		parameters: rulebookSchema,
+		execute: async (_toolCallId: string, { name }: { name: string }) => {
+			const rule = ruleMap.get(name);
+
+			if (!rule) {
+				const available = ruleNames.join(", ");
+				return {
+					content: [{ type: "text", text: `Rule "${name}" not found. Available rules: ${available || "(none)"}` }],
+					details: {
+						type: "rulebook",
+						ruleName: name,
+						found: false,
+					} satisfies RulebookToolDetails,
+				};
+			}
+
+			return {
+				content: [{ type: "text", text: `# Rule: ${rule.name}\n\n${rule.content}` }],
+				details: {
+					type: "rulebook",
+					ruleName: name,
+					found: true,
+					content: rule.content,
+				} satisfies RulebookToolDetails,
+			};
+		},
+	};
+}
+
+/**
+ * Filter rules to only those suitable for the rulebook (have descriptions, no TTSR trigger).
+ */
+export function filterRulebookRules(rules: Rule[]): Rule[] {
+	return rules.filter((rule) => {
+		// Exclude TTSR rules (handled separately by streaming)
+		if (rule.ttsrTrigger) return false;
+		// Exclude always-apply rules (already in context)
+		if (rule.alwaysApply) return false;
+		// Must have a description for agent to know when to fetch
+		if (!rule.description) return false;
+		return true;
+	});
+}
+
+/**
+ * Format rules for inclusion in the system prompt.
+ * Lists rule names and descriptions so the agent knows what's available.
+ */
+export function formatRulesForPrompt(rules: Rule[]): string {
+	if (rules.length === 0) {
+		return "";
+	}
+
+	const lines = [
+		"\n\n## Available Rules",
+		"",
+		"The following project rules are available. Use the `rulebook` tool to fetch a rule's full content when it's relevant to your task.",
+		"",
+		"<available_rules>",
+	];
+
+	for (const rule of rules) {
+		lines.push("  <rule>");
+		lines.push(`    <name>${escapeXml(rule.name)}</name>`);
+		lines.push(`    <description>${escapeXml(rule.description || "")}</description>`);
+		if (rule.globs && rule.globs.length > 0) {
+			lines.push(`    <globs>${escapeXml(rule.globs.join(", "))}</globs>`);
+		}
+		lines.push("  </rule>");
+	}
+
+	lines.push("</available_rules>");
+
+	return lines.join("\n");
+}
+
+function escapeXml(str: string): string {
+	return str
+		.replace(/&/g, "&amp;")
+		.replace(/</g, "&lt;")
+		.replace(/>/g, "&gt;")
+		.replace(/"/g, "&quot;")
+		.replace(/'/g, "&apos;");
+}