@oh-my-pi/pi-coding-agent 10.6.1 → 11.0.0

package/src/tools/todo-write.ts

@@ -1,10 +1,9 @@
-import { randomUUID } from "node:crypto";
 import path from "node:path";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import { StringEnum } from "@oh-my-pi/pi-ai";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text } from "@oh-my-pi/pi-tui";
-import { logger } from "@oh-my-pi/pi-utils";
+import { logger, Snowflake } from "@oh-my-pi/pi-utils";
 import { Type } from "@sinclair/typebox";
 import chalk from "chalk";
 import { renderPromptTemplate } from "../config/prompt-templates";
@@ -72,7 +71,7 @@ function normalizeTodos(items: Array<{ id?: string; content?: string; status?: s
 			throw new Error("Todo content cannot be empty.");
 		}
 		return {
-			id: item.id && item.id.trim().length > 0 ? item.id : randomUUID(),
+			id: item.id && item.id.trim().length > 0 ? item.id : Snowflake.next(),
 			content,
 			status: normalizeTodoStatus(item.status),
 		};

package/src/utils/frontmatter.ts

@@ -18,7 +18,7 @@ export class FrontmatterError extends Error {
 		error: Error,
 		public readonly source?: unknown,
 	) {
-		super(`Failed to parse YAML frontmatter: ${error.message}`, { cause: error });
+		super(`Failed to parse YAML frontmatter (${source}): ${error.message}`, { cause: error });
 		this.name = "FrontmatterError";
 	}
 

package/src/utils/image-resize.ts

@@ -40,7 +40,7 @@ function pickSmaller(
  * Resize an image to fit within the specified max dimensions and file size.
  * Returns the original image if it already fits within the limits.
  *
- * Uses Photon (Rust/WASM) for image processing. If Photon is not available,
+ * Uses Photon for image processing. If Photon is not available,
  * returns the original image unchanged.
  *
  * Strategy for staying under maxBytes:

package/src/utils/timings.ts

@@ -1,9 +1,10 @@
 /**
  * Central timing instrumentation for startup profiling.
- * Enable with OMP_TIMING=1 or PI_TIMING=1 environment variable.
+ * Enable with PI_TIMING=1 or PI_TIMING=1 environment variable.
  */
+import { $env } from "@oh-my-pi/pi-utils";
 
-const ENABLED = process.env.OMP_TIMING === "1";
+const ENABLED = $env.PI_TIMING === "1";
 const timings: Array<{ label: string; ms: number }> = [];
 let lastTime = Date.now();
 

package/src/web/scrapers/github.ts

@@ -1,4 +1,4 @@
-import { ptree } from "@oh-my-pi/pi-utils";
+import { $env, ptree } from "@oh-my-pi/pi-utils";
 import type { RenderResult, SpecialHandler } from "./types";
 import { finalizeOutput, loadPage } from "./types";
 
@@ -85,7 +85,7 @@ export async function fetchGitHubApi(
 		};
 
 		// Use GITHUB_TOKEN if available
-		const token = process.env.GITHUB_TOKEN || process.env.GH_TOKEN;
+		const token = $env.GITHUB_TOKEN || $env.GH_TOKEN;
 		if (token) {
 			headers.Authorization = `Bearer ${token}`;
 		}

package/src/web/scrapers/utils.ts

@@ -1,8 +1,7 @@
 import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import * as path from "node:path";
-import { ptree } from "@oh-my-pi/pi-utils";
-import { nanoid } from "nanoid";
+import { ptree, Snowflake } from "@oh-my-pi/pi-utils";
 import { ensureTool } from "../../utils/tools-manager";
 
 const MAX_BYTES = 50 * 1024 * 1024; // 50MB for binary files
@@ -42,7 +41,7 @@ export async function convertWithMarkitdown(
 	// Write to temp file with extension hint
 	const ext = extensionHint || ".bin";
 	const tmpDir = os.tmpdir();
-	const tmpFile = path.join(tmpDir, `omp-convert-${nanoid()}${ext}`);
+	const tmpFile = path.join(tmpDir, `omp-convert-${Snowflake.next()}${ext}`);
 
 	if (content.length > MAX_BYTES) {
 		return { content: "", ok: false, error: `content exceeds ${MAX_BYTES} bytes` };

package/src/web/scrapers/youtube.ts

@@ -1,8 +1,7 @@
 import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import path from "node:path";
-import { ptree } from "@oh-my-pi/pi-utils";
-import { nanoid } from "nanoid";
+import { ptree, Snowflake } from "@oh-my-pi/pi-utils";
 import { throwIfAborted } from "../../tools/tool-errors";
 import { ensureTool } from "../../utils/tools-manager";
 import type { RenderResult, SpecialHandler } from "./types";
@@ -197,7 +196,7 @@ export const handleYouTube: SpecialHandler = async (
 
 	// Create temp directory for subtitle download
 	const tmpDir = os.tmpdir();
-	const tmpBase = path.join(tmpDir, `yt-${yt.videoId}-${nanoid()}`);
+	const tmpBase = path.join(tmpDir, `yt-${yt.videoId}-${Snowflake.next()}`);
 
 	try {
 		// Try manual subtitles first (English preferred)

package/src/web/search/auth.ts

@@ -8,8 +8,8 @@
  *   4. ANTHROPIC_API_KEY / ANTHROPIC_BASE_URL fallback
  */
 import * as path from "node:path";
-import { buildAnthropicHeaders as buildProviderAnthropicHeaders, getEnv, getEnvApiKey } from "@oh-my-pi/pi-ai";
-import { logger } from "@oh-my-pi/pi-utils";
+import { buildAnthropicHeaders as buildProviderAnthropicHeaders, getEnvApiKey } from "@oh-my-pi/pi-ai";
+import { $env, logger } from "@oh-my-pi/pi-utils";
 import { getAgentDbPath, getConfigDirPaths } from "../../config";
 import { AgentStorage } from "../../session/agent-storage";
 import type { AuthCredential, AuthCredentialEntry, AuthStorageData } from "../../session/auth-storage";
@@ -17,7 +17,6 @@ import { migrateJsonStorage } from "../../session/storage-migration";
 import type { AnthropicAuthConfig, AnthropicOAuthCredential, ModelsJson } from "./types";
 
 const DEFAULT_BASE_URL = "https://api.anthropic.com";
-export { getEnv };
 
 /**
  * Reads and parses a JSON file safely.
@@ -121,8 +120,8 @@ export async function findAnthropicAuth(): Promise<AnthropicAuthConfig | null> {
 	const configDirs = getConfigDirPaths("", { project: false });
 
 	// 1. Explicit search-specific env vars
-	const searchApiKey = getEnv("ANTHROPIC_SEARCH_API_KEY");
-	const searchBaseUrl = getEnv("ANTHROPIC_SEARCH_BASE_URL");
+	const searchApiKey = $env.ANTHROPIC_SEARCH_API_KEY;
+	const searchBaseUrl = $env.ANTHROPIC_SEARCH_BASE_URL;
 	if (searchApiKey) {
 		return {
 			apiKey: searchApiKey,
@@ -177,7 +176,7 @@ export async function findAnthropicAuth(): Promise<AnthropicAuthConfig | null> {
 
 	// 4. Generic ANTHROPIC_API_KEY fallback
 	const apiKey = getEnvApiKey("anthropic");
-	const baseUrl = getEnv("ANTHROPIC_BASE_URL");
+	const baseUrl = $env.ANTHROPIC_BASE_URL;
 	if (apiKey) {
 		return {
 			apiKey,

package/src/web/search/providers/anthropic.ts

@@ -4,7 +4,8 @@
  * Uses Claude's built-in web_search_20250305 tool to search the web.
  * Returns synthesized answers with citations and source metadata.
  */
-import { applyClaudeToolPrefix, buildAnthropicSystemBlocks, getEnv, stripClaudeToolPrefix } from "@oh-my-pi/pi-ai";
+import { applyClaudeToolPrefix, buildAnthropicSystemBlocks, stripClaudeToolPrefix } from "@oh-my-pi/pi-ai";
+import { $env } from "@oh-my-pi/pi-utils";
 import { buildAnthropicHeaders, buildAnthropicUrl, findAnthropicAuth } from "../../../web/search/auth";
 import type {
 	AnthropicApiResponse,
@@ -42,7 +43,7 @@ export interface AnthropicSearchParams {
  * @returns Model identifier string
  */
 function getModel(): string {
-	return getEnv("ANTHROPIC_SEARCH_MODEL") ?? DEFAULT_MODEL;
+	return $env.ANTHROPIC_SEARCH_MODEL ?? DEFAULT_MODEL;
 }
 
 /**

package/src/utils/terminal-notify.ts

@@ -1,37 +0,0 @@
-export type NotificationProtocol = "bell" | "osc99" | "osc9";
-
-export function detectNotificationProtocol(): NotificationProtocol {
-	const termProgram = process.env.TERM_PROGRAM?.toLowerCase() || "";
-	const term = process.env.TERM?.toLowerCase() || "";
-
-	if (process.env.KITTY_WINDOW_ID || termProgram === "kitty") {
-		return "osc99";
-	}
-
-	if (process.env.GHOSTTY_RESOURCES_DIR || termProgram === "ghostty" || term.includes("ghostty")) {
-		return "osc9";
-	}
-
-	if (process.env.WEZTERM_PANE || termProgram === "wezterm") {
-		return "osc9";
-	}
-
-	if (process.env.ITERM_SESSION_ID || termProgram === "iterm.app") {
-		return "osc9";
-	}
-
-	return "bell";
-}
-
-export function sendNotification(protocol: NotificationProtocol, message: string): void {
-	const payload =
-		protocol === "osc99" ? `\x1b]99;;${message}\x1b\\` : protocol === "osc9" ? `\x1b]9;${message}\x1b\\` : "\x07";
-
-	process.stdout.write(payload);
-}
-
-export function isNotificationSuppressed(): boolean {
-	const value = process.env.OMP_NOTIFICATIONS?.trim().toLowerCase();
-	if (!value) return false;
-	return value === "off" || value === "0" || value === "false";
-}