@oh-my-pi/pi-coding-agent 12.9.0 → 12.10.0

package/CHANGELOG.md

@@ -2,6 +2,46 @@
 
 ## [Unreleased]
 
+## [12.10.0] - 2026-02-18
+
+### Breaking Changes
+
+- Changed keyless provider auth sentinel from `"<no-auth>"` to `kNoAuth` (`"N/A"`) for `ModelRegistry.getApiKey()` and `ModelRegistry.getApiKeyForProvider()`
+
+### Added
+
+- Added `--no-rules` CLI flag to disable rules discovery and loading
+- Added `sessionDir` option to RpcClientOptions for specifying agent session directory
+- Added `Symbol.dispose` method to RpcClient for resource cleanup support
+- Added `rules` option to CreateAgentSessionOptions for explicit rule configuration
+- Added `sessionDir` option to RpcClientOptions for specifying agent session directory
+- Added `Symbol.dispose` method to RpcClient for resource cleanup support
+- Added `condition` and `scope` fields to rule frontmatter for advanced TTSR matching and stream filtering
+- Added `ttsr.interruptMode` setting to control when TTSR rules interrupt mid-stream vs inject warnings after completion
+- Added support for loading rules, prompts, commands, context files (AGENTS.md), and system prompts (SYSTEM.md) from ~/.agent/ directory (with fallback to ~/.agents/)
+- Added scoped stream buffering for TTSR matching to isolate prose, thinking, and tool argument streams
+- Added file-path-aware TTSR scope matching for tool calls with glob patterns (e.g., `tool:edit(*.ts)`)
+- Added legacy field support: `ttsr_trigger` and `ttsrTrigger` are accepted as fallback for `condition`
+
+### Changed
+
+- Changed TTSR injection tracking to record all turns where rules were injected (instead of only the last turn) to support repeat-after-gap mode across resumed sessions
+- Changed TTSR injection messages to use custom message type with metadata instead of synthetic user messages for better session tracking
+- Changed TTSR rule injection to persist injected rule names in session state for restoration when resuming sessions
+- Changed model discovery to automatically discover built-in provider models (Anthropic, OpenAI, Groq, Cerebras, Xai, Mistral, OpenCode, OpenRouter, Vercel AI Gateway, Kimi Code, GitHub Copilot, Google, Cursor, Google Antigravity, Google Gemini CLI, OpenAI Codex) when credentials are configured
+- Changed `getModel()` and `getModels()` imports to `getBundledModel()` and `getBundledModels()` across test utilities
+- Changed TTSR rule matching from single `ttsrTrigger` regex to multiple `condition` patterns with scope filtering
+- Changed TTSR buffer management to use per-stream-key buffers instead of a single global buffer
+- Changed rule discovery to use unified `buildRuleFromMarkdown` helper across all providers (builtin, cline, cursor, windsurf, agents)
+- Changed TTSR injection to defer warnings until stream completion when `interruptMode` is not `always`
+- Changed `TtsrManager.addRule()` to return boolean indicating successful registration instead of void
+
+### Fixed
+
+- Fixed TTSR repeat-after-gap mode to correctly calculate gaps when rules are restored from previous sessions
+- Fixed TTSR matching to respect tool-specific scope filters, preventing cross-tool rule contamination
+- Fixed path normalization in TTSR glob matching to handle both relative and absolute path variants
+
 ## [12.9.0] - 2026-02-17
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.9.0",
+	"version": "12.10.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"bin": {
@@ -84,12 +84,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.9.0",
-		"@oh-my-pi/pi-agent-core": "12.9.0",
-		"@oh-my-pi/pi-ai": "12.9.0",
-		"@oh-my-pi/pi-natives": "12.9.0",
-		"@oh-my-pi/pi-tui": "12.9.0",
-		"@oh-my-pi/pi-utils": "12.9.0",
+		"@oh-my-pi/omp-stats": "12.10.0",
+		"@oh-my-pi/pi-agent-core": "12.10.0",
+		"@oh-my-pi/pi-ai": "12.10.0",
+		"@oh-my-pi/pi-natives": "12.10.0",
+		"@oh-my-pi/pi-tui": "12.10.0",
+		"@oh-my-pi/pi-utils": "12.10.0",
 		"@sinclair/typebox": "^0.34.48",
 		"@xterm/headless": "^6.0.0",
 		"ajv": "^8.18.0",

package/src/capability/rule.ts

@@ -7,15 +7,23 @@
 import { defineCapability } from ".";
 import type { SourceMeta } from "./types";
 
+const CONDITION_GLOB_SCOPE_TOOLS = ["edit", "write"] as const;
+
 /**
- * Parsed frontmatter from MDC rule files (Cursor format).
+ * Parsed frontmatter from rule files.
  */
 export interface RuleFrontmatter {
 	description?: string;
 	globs?: string[];
 	alwaysApply?: boolean;
-	/** Regex pattern that triggers time-traveling rule injection */
-	ttsr_trigger?: string;
+	/** New key for TTSR match conditions. */
+	condition?: string | string[];
+	/** New key for TTSR stream scope. */
+	scope?: string | string[];
+	/** Legacy key accepted for backward compatibility with existing rules. */
+	ttsr_trigger?: string | string[];
+	/** Legacy camelCase key accepted for backward compatibility with existing rules. */
+	ttsrTrigger?: string | string[];
 	[key: string]: unknown;
 }
 
@@ -35,12 +43,172 @@ export interface Rule {
 	alwaysApply?: boolean;
 	/** Description (for agent-requested rules) */
 	description?: string;
-	/** Regex pattern that triggers time-traveling rule injection */
-	ttsrTrigger?: string;
+	/** Regex condition(s) that can trigger TTSR interruption. */
+	condition?: string[];
+	/** Optional stream scope tokens (for example: text, thinking, tool:edit(*.ts)). */
+	scope?: string[];
 	/** Source metadata */
 	_source: SourceMeta;
 }
 
+function normalizeRuleField(value: unknown): string[] | undefined {
+	if (typeof value === "string") {
+		const token = value.trim();
+		return token.length > 0 ? [token] : undefined;
+	}
+	if (!Array.isArray(value)) {
+		return undefined;
+	}
+
+	const tokens = value
+		.filter((item): item is string => typeof item === "string")
+		.map(item => item.trim())
+		.filter(item => item.length > 0);
+	if (tokens.length === 0) {
+		return undefined;
+	}
+
+	return Array.from(new Set(tokens));
+}
+
+function splitScopeTokens(value: string): string[] {
+	const tokens: string[] = [];
+	let current = "";
+	let parenDepth = 0;
+	let bracketDepth = 0;
+	let braceDepth = 0;
+	let quote: '"' | "'" | undefined;
+	for (let i = 0; i < value.length; i++) {
+		const char = value[i];
+		if (quote) {
+			current += char;
+			if (char === quote && value[i - 1] !== "\\") {
+				quote = undefined;
+			}
+			continue;
+		}
+		if (char === '"' || char === "'") {
+			quote = char;
+			current += char;
+			continue;
+		}
+		if (char === "(") {
+			parenDepth++;
+			current += char;
+			continue;
+		}
+		if (char === ")") {
+			parenDepth = Math.max(0, parenDepth - 1);
+			current += char;
+			continue;
+		}
+		if (char === "[") {
+			bracketDepth++;
+			current += char;
+			continue;
+		}
+		if (char === "]") {
+			bracketDepth = Math.max(0, bracketDepth - 1);
+			current += char;
+			continue;
+		}
+		if (char === "{") {
+			braceDepth++;
+			current += char;
+			continue;
+		}
+		if (char === "}") {
+			braceDepth = Math.max(0, braceDepth - 1);
+			current += char;
+			continue;
+		}
+		if (char === "," && parenDepth === 0 && bracketDepth === 0 && braceDepth === 0) {
+			const token = current.trim();
+			if (token.length > 0) {
+				tokens.push(token);
+			}
+			current = "";
+			continue;
+		}
+		current += char;
+	}
+
+	const tail = current.trim();
+	if (tail.length > 0) {
+		tokens.push(tail);
+	}
+
+	return tokens;
+}
+function normalizeScopeField(value: unknown): string[] | undefined {
+	const normalized = normalizeRuleField(value);
+	if (!normalized) {
+		return undefined;
+	}
+
+	const tokens = normalized.flatMap(splitScopeTokens).filter(item => item.length > 0);
+	if (tokens.length === 0) {
+		return undefined;
+	}
+	return Array.from(new Set(tokens));
+}
+/**
+ * Heuristic for condition shorthand that looks like a file glob (for example `*.rs`).
+ */
+function isLikelyFileGlob(value: string): boolean {
+	const token = value.trim();
+	if (token.length === 0) {
+		return false;
+	}
+	if (/[\\^$+|()]/.test(token)) {
+		return false;
+	}
+	if (!/[?*[\]{}]/.test(token)) {
+		return false;
+	}
+	if (token.includes("/")) {
+		return true;
+	}
+	return /^\*\.[^\s/]+$/.test(token);
+}
+
+/**
+ * Parse `condition` + `scope` from rule frontmatter.
+ *
+ * - `condition` accepts string or string[]
+ * - `scope` accepts string or string[]
+ * - legacy `ttsr_trigger` / `ttsrTrigger` are accepted as a `condition` fallback
+ * - condition tokens that look like file globs become scope shorthands:
+ *   `*.rs` => `tool:edit(*.rs)`, `tool:write(*.rs)` and a catch-all condition `.*`
+ */
+export function parseRuleConditionAndScope(frontmatter: RuleFrontmatter): Pick<Rule, "condition" | "scope"> {
+	const rawCondition = frontmatter.condition ?? frontmatter.ttsr_trigger ?? frontmatter.ttsrTrigger;
+	const parsedCondition = normalizeRuleField(rawCondition);
+	const parsedScope = normalizeScopeField(frontmatter.scope);
+
+	const inferredScope: string[] = [];
+	const condition: string[] = [];
+	for (const token of parsedCondition ?? []) {
+		if (isLikelyFileGlob(token)) {
+			for (const toolName of CONDITION_GLOB_SCOPE_TOOLS) {
+				inferredScope.push(`tool:${toolName}(${token})`);
+			}
+			continue;
+		}
+		condition.push(token);
+	}
+
+	if (condition.length === 0 && inferredScope.length > 0) {
+		condition.push(".*");
+	}
+
+	const scope = [...(parsedScope ?? []), ...inferredScope];
+	return {
+		condition: condition.length > 0 ? Array.from(new Set(condition)) : undefined,
+		scope: scope.length > 0 ? Array.from(new Set(scope)) : undefined,
+	};
+}
+
 export const ruleCapability = defineCapability<Rule>({
 	id: "rules",
 	displayName: "Rules",

package/src/cli/args.ts

@@ -40,6 +40,7 @@ export interface Args {
 	export?: string;
 	noSkills?: boolean;
 	skills?: string[];
+	noRules?: boolean;
 	listModels?: string | true;
 	noTitle?: boolean;
 	messages: string[];
@@ -150,6 +151,8 @@ export function parseArgs(args: string[], extensionFlags?: Map<string, { type: "
 			result.noExtensions = true;
 		} else if (arg === "--no-skills") {
 			result.noSkills = true;
+		} else if (arg === "--no-rules") {
+			result.noRules = true;
 		} else if (arg === "--no-title") {
 			result.noTitle = true;
 		} else if (arg === "--skills" && i + 1 < args.length) {

package/src/cli/update-cli.ts

@@ -6,7 +6,6 @@
  */
 import { execSync, spawnSync } from "node:child_process";
 import * as fs from "node:fs";
-import * as path from "node:path";
 import { pipeline } from "node:stream/promises";
 import { isEnoent } from "@oh-my-pi/pi-utils";
 import { APP_NAME, VERSION } from "@oh-my-pi/pi-utils/dirs";
@@ -80,7 +79,7 @@ async function getLatestRelease(): Promise<ReleaseInfo> {
 	return {
 		tag,
 		version,
-		assets: [makeAsset(getBinaryName()), ...getNativeAddonNames().map(makeAsset)],
+		assets: [makeAsset(getBinaryName())],
 	};
 }
 
@@ -142,27 +141,6 @@ function getBinaryName(): string {
 	return `${APP_NAME}-${os}-${archName}`;
 }
 
-/**
- * Get native addon names for this platform, ordered by preference.
- */
-function getNativeAddonNames(): string[] {
-	const platform = process.platform;
-	const arch = process.arch;
-	if (!["linux", "darwin", "win32"].includes(platform)) {
-		throw new Error(`Unsupported platform: ${platform}`);
-	}
-	if (!["x64", "arm64"].includes(arch)) {
-		throw new Error(`Unsupported architecture: ${arch}`);
-	}
-
-	const baseName = `pi_natives.${platform}-${arch}.node`;
-	if (arch !== "x64") {
-		return [baseName];
-	}
-
-	return [`pi_natives.${platform}-${arch}-modern.node`, `pi_natives.${platform}-${arch}-baseline.node`];
-}
-
 /**
  * Update via bun package manager.
  */
@@ -200,44 +178,13 @@ async function updateViaBun(expectedVersion: string): Promise<void> {
  */
 async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 	const binaryName = getBinaryName();
-	const nativeAddonNames = getNativeAddonNames();
 	const asset = release.assets.find(a => a.name === binaryName);
 	if (!asset) {
 		throw new Error(`No binary found for ${binaryName}`);
 	}
 	const execPath = process.execPath;
-	const execDir = path.dirname(execPath);
 	const tempPath = `${execPath}.new`;
 	const backupPath = `${execPath}.bak`;
-	const nativeDownloads: Array<{ name: string; tempPath: string; finalPath: string }> = [];
-
-	const downloadNativeAsset = async (name: string, required: boolean): Promise<boolean> => {
-		const nativeAsset = release.assets.find(assetEntry => assetEntry.name === name);
-		if (!nativeAsset) {
-			if (required) throw new Error(`No native addon found for ${name}`);
-			return false;
-		}
-
-		console.log(chalk.dim(`Downloading ${name}…`));
-		try {
-			const nativeResponse = await fetch(nativeAsset.url, { redirect: "follow" });
-			if (!nativeResponse.ok || !nativeResponse.body) {
-				if (required) throw new Error(`Native addon download failed for ${name}: ${nativeResponse.statusText}`);
-				return false;
-			}
-
-			const nativeFinalPath = path.join(execDir, name);
-			const nativeTempPath = `${nativeFinalPath}.new`;
-			const nativeFileStream = fs.createWriteStream(nativeTempPath, { mode: 0o755 });
-			await pipeline(nativeResponse.body, nativeFileStream);
-			nativeDownloads.push({ name, tempPath: nativeTempPath, finalPath: nativeFinalPath });
-			return true;
-		} catch (err) {
-			if (required) throw err;
-			return false;
-		}
-	};
-
 	console.log(chalk.dim(`Downloading ${binaryName}…`));
 
 	// Download binary to temp file
@@ -247,9 +194,6 @@ async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 	}
 	const fileStream = fs.createWriteStream(tempPath, { mode: 0o755 });
 	await pipeline(response.body, fileStream);
-	for (const nativeAddonName of nativeAddonNames) {
-		await downloadNativeAsset(nativeAddonName, true);
-	}
 	// Replace current binary
 	console.log(chalk.dim("Installing update..."));
 	try {
@@ -262,11 +206,7 @@ async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 		await fs.promises.rename(tempPath, execPath);
 		await fs.promises.unlink(backupPath);
 
-		for (const nativeDownload of nativeDownloads) {
-			await fs.promises.rename(nativeDownload.tempPath, nativeDownload.finalPath);
-		}
 		console.log(chalk.green(`\n${theme.status.success} Updated to ${release.version}`));
-		console.log(chalk.dim(`Installed ${nativeDownloads.length} native addon file(s)`));
 		console.log(chalk.dim(`Restart ${APP_NAME} to use the new version`));
 	} catch (err) {
 		if (fs.existsSync(backupPath) && !fs.existsSync(execPath)) {
@@ -275,11 +215,6 @@ async function updateViaBinary(release: ReleaseInfo): Promise<void> {
 		if (fs.existsSync(tempPath)) {
 			await fs.promises.unlink(tempPath);
 		}
-		for (const nativeDownload of nativeDownloads) {
-			if (fs.existsSync(nativeDownload.tempPath)) {
-				await fs.promises.unlink(nativeDownload.tempPath);
-			}
-		}
 		throw err;
 	}
 }

package/src/commands/launch.ts

@@ -105,6 +105,9 @@ export default class Index extends Command {
 		skills: Flags.string({
 			description: "Comma-separated glob patterns to filter skills (e.g., git-*,docker)",
 		}),
+		"no-rules": Flags.boolean({
+			description: "Disable rules discovery and loading",
+		}),
 		export: Flags.string({
 			description: "Export session file to HTML and exit",
 		}),