@diegopetrucci/pi-extensions 0.1.74 → 0.1.78

package/extensions/claude-fast/index.ts

@@ -1,293 +1,35 @@
-import { existsSync, readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-import {
-	CONFIG_DIR_NAME,
-	getAgentDir,
-	type ExtensionAPI,
-	type ExtensionContext,
-} from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 const EXTENSION_ID = "claude-fast";
-const PROVIDER_ID = "anthropic";
-const API_ID = "anthropic-messages";
-const FAST_SPEED = "fast";
 const FAST_BETA = "fast-mode-2026-02-01";
-const CLAUDE_CODE_OAUTH_BETAS = ["claude-code-20250219", "oauth-2025-04-20"];
-const SUPPORTED_MODELS = new Set(["claude-opus-4-8", "claude-opus-5"]);
-
-const DEFAULT_CONFIG: ClaudeFastConfig = {
-	enabled: false,
-	showStatus: true,
-};
-
-type FastOverride = "auto" | "on" | "off";
-
-type ClaudeFastConfig = {
-	/** Default Fast-mode state when there is no session override. */
-	enabled: boolean;
-	/** Show a compact `fast` status when Fast mode is active for the current model. */
-	showStatus: boolean;
-};
-
-type SessionState = {
-	config: ClaudeFastConfig;
-	override: FastOverride;
-	lastInjectedAt?: number;
-	lastInjectedModel?: string;
-};
-
-type ProjectConfigContext = {
-	cwd: string;
-	isProjectTrusted?: () => boolean;
-};
-
-type RecursivePartial<T> = {
-	[P in keyof T]?: T[P] extends object ? RecursivePartial<T[P]> : T[P];
-};
-
-type PayloadRecord = Record<string, unknown>;
-
-type Eligibility = {
-	eligible: boolean;
-	modelKey: string;
-	reason?: string;
-};
-
-type HeaderModel = {
-	headers?: Record<string, string>;
-};
-
-function readConfigFile(path: string): RecursivePartial<ClaudeFastConfig> {
-	if (!existsSync(path)) return {};
-
-	try {
-		const parsed = JSON.parse(readFileSync(path, "utf-8"));
-		return isPayloadRecord(parsed) ? (parsed as RecursivePartial<ClaudeFastConfig>) : {};
-	} catch (error) {
-		console.error(`Warning: Could not parse ${path}: ${error}`);
-		return {};
-	}
-}
-
-function normalizeBoolean(value: unknown, fallback: boolean): boolean {
-	return typeof value === "boolean" ? value : fallback;
-}
-
-function mergeConfig(
-	base: ClaudeFastConfig,
-	overrides: RecursivePartial<ClaudeFastConfig>,
-): ClaudeFastConfig {
-	return {
-		enabled: normalizeBoolean(overrides.enabled, base.enabled),
-		showStatus: normalizeBoolean(overrides.showStatus, base.showStatus),
-	};
-}
-
-function canReadProjectConfig(ctx: ProjectConfigContext): boolean {
-	return typeof ctx.isProjectTrusted === "function" && ctx.isProjectTrusted();
-}
-
-function findProjectConfigPath(cwd: string): string {
-	let current = cwd;
-	while (true) {
-		const candidate = join(current, CONFIG_DIR_NAME, "claude-fast.json");
-		if (existsSync(candidate)) return candidate;
-
-		const parent = dirname(current);
-		if (parent === current) return join(cwd, CONFIG_DIR_NAME, "claude-fast.json");
-		current = parent;
-	}
-}
-
-function loadConfig(ctx: ProjectConfigContext): ClaudeFastConfig {
-	const globalConfig = readConfigFile(join(getAgentDir(), "extensions", "claude-fast.json"));
-	const projectConfig = canReadProjectConfig(ctx) ? readConfigFile(findProjectConfigPath(ctx.cwd)) : {};
-	return mergeConfig(mergeConfig(DEFAULT_CONFIG, globalConfig), projectConfig);
-}
-
-function isPayloadRecord(payload: unknown): payload is PayloadRecord {
-	return typeof payload === "object" && payload !== null && !Array.isArray(payload);
-}
-
-function modelKey(ctx: ExtensionContext): string {
-	const model = ctx.model;
-	return model ? `${model.provider}/${model.id}` : "no-model";
-}
-
-function isFastEnabled(state: SessionState): boolean {
-	if (state.override === "on") return true;
-	if (state.override === "off") return false;
-	return state.config.enabled;
-}
-
-function describeMode(state: SessionState): string {
-	if (state.override === "on") return "on (session override)";
-	if (state.override === "off") return "off (session override)";
-	return state.config.enabled ? "on (config default)" : "off (config default)";
-}
-
-function getEligibility(ctx: ExtensionContext): Eligibility {
-	const model = ctx.model;
-	if (!model) {
-		return { eligible: false, modelKey: "no-model", reason: "no model is selected" };
-	}
-
-	const key = `${model.provider}/${model.id}`;
-	if (model.provider !== PROVIDER_ID) {
-		return {
-			eligible: false,
-			modelKey: key,
-			reason: `current provider is ${model.provider}, not ${PROVIDER_ID}`,
-		};
-	}
-
-	if (model.api !== API_ID) {
-		return {
-			eligible: false,
-			modelKey: key,
-			reason: `current API is ${model.api}, not ${API_ID}`,
-		};
-	}
-
-	if (!SUPPORTED_MODELS.has(model.id)) {
-		return {
-			eligible: false,
-			modelKey: key,
-			reason: "Fast mode is only enabled for Claude Opus 4.8 and Claude Opus 5",
-		};
-	}
-
-	return { eligible: true, modelKey: key };
-}
-
-function splitBetaHeader(value: string | undefined): string[] {
-	return (value ?? "")
-		.split(",")
-		.map((part) => part.trim())
-		.filter(Boolean);
-}
-
-function syncModelBetaHeader(ctx: ExtensionContext, state: SessionState): void {
-	const model = ctx.model as (typeof ctx.model & HeaderModel) | undefined;
-	if (!model || model.provider !== PROVIDER_ID || model.api !== API_ID) return;
-
-	const shouldEnable = isFastEnabled(state) && getEligibility(ctx).eligible;
-	const headers = { ...(model.headers ?? {}) };
-	const existing = splitBetaHeader(headers["anthropic-beta"] ?? headers["Anthropic-Beta"]);
-	const requiredBase = ctx.modelRegistry.isUsingOAuth(model) ? CLAUDE_CODE_OAUTH_BETAS : [];
-	const next = shouldEnable
-		? Array.from(new Set([...existing, ...requiredBase, FAST_BETA]))
-		: existing.filter((beta) => beta !== FAST_BETA);
-
-	delete headers["Anthropic-Beta"];
-	if (next.length > 0) headers["anthropic-beta"] = next.join(",");
-	else delete headers["anthropic-beta"];
-	model.headers = headers;
-}
-
-function updateStatus(ctx: ExtensionContext, state: SessionState): void {
-	syncModelBetaHeader(ctx, state);
-	if (!ctx.hasUI) return;
-	if (!state.config.showStatus) {
-		ctx.ui.setStatus(EXTENSION_ID, undefined);
-		return;
-	}
-
-	const eligibility = getEligibility(ctx);
-	ctx.ui.setStatus(
-		EXTENSION_ID,
-		isFastEnabled(state) && eligibility.eligible ? "fast" : undefined,
-	);
-}
-
-function getStatusMessage(ctx: ExtensionContext, state: SessionState): string {
-	const enabled = isFastEnabled(state);
-	const eligibility = getEligibility(ctx);
-	const active = enabled && eligibility.eligible;
-	const injected = state.lastInjectedAt
-		? ` Last injected for ${state.lastInjectedModel ?? "unknown model"} ${Math.max(0, Math.round((Date.now() - state.lastInjectedAt) / 1000))}s ago.`
-		: "";
-
-	if (active) {
-		return `Claude Fast mode is ${describeMode(state)} and active for ${eligibility.modelKey}; requests will use speed=${FAST_SPEED}.${injected}`;
-	}
-
-	if (enabled) {
-		return `Claude Fast mode is ${describeMode(state)}, but inactive for ${eligibility.modelKey}: ${eligibility.reason}.${injected}`;
-	}
-
-	return `Claude Fast mode is ${describeMode(state)}. Current model: ${eligibility.modelKey}.${injected}`;
-}
-
-function injectFastSpeed(
-	payload: unknown,
-	ctx: ExtensionContext,
-	state: SessionState,
-): PayloadRecord | undefined {
-	if (!isFastEnabled(state)) return undefined;
-	if (!getEligibility(ctx).eligible) return undefined;
-	if (!isPayloadRecord(payload)) return undefined;
-	if (payload.model !== ctx.model?.id) return undefined;
-	if ("speed" in payload) return undefined;
-
-	state.lastInjectedAt = Date.now();
-	state.lastInjectedModel = modelKey(ctx);
-	return {
-		...payload,
-		speed: FAST_SPEED,
-	};
-}
-
-export default function claudeFastExtension(pi: ExtensionAPI) {
-	const states = new WeakMap<object, SessionState>();
-
-	function getState(ctx: ExtensionContext): SessionState {
-		let state = states.get(ctx.sessionManager);
-		if (!state) {
-			state = {
-				config: loadConfig(ctx),
-				override: "auto",
-			};
-			states.set(ctx.sessionManager, state);
-		}
-		return state;
-	}
-
-	pi.on("session_start", (_event, ctx) => {
-		const state: SessionState = {
-			config: loadConfig(ctx),
-			override: "auto",
-		};
-		states.set(ctx.sessionManager, state);
-		updateStatus(ctx, state);
-	});
-
-	pi.on("model_select", (_event, ctx) => {
-		updateStatus(ctx, getState(ctx));
-	});
-
-	pi.on("before_provider_request", (event, ctx) => {
-		const state = getState(ctx);
-		const nextPayload = injectFastSpeed(event.payload, ctx, state);
-		updateStatus(ctx, state);
-		return nextPayload;
-	});
-
-	pi.registerCommand("claude-fast", {
-		description: "Toggle Claude Fast mode for supported Anthropic Claude Opus models",
-		getArgumentCompletions: () => null,
-		handler: async (args, ctx) => {
-			const state = getState(ctx);
-			const action = args.trim();
-
-			if (!action) {
-				state.override = isFastEnabled(state) ? "off" : "on";
-				updateStatus(ctx, state);
-				ctx.ui.notify(getStatusMessage(ctx, state), "info");
-				return;
-			}
-
-			ctx.ui.notify("Usage: /claude-fast", "warning");
-		},
-	});
+const DEPRECATION_NOTICE =
+  "claude-fast is deprecated and no longer changes provider requests. Use the unified fast extension (/fast) instead.";
+
+/**
+ * Deprecated leftover of the provider-specific Fast stack.
+ * The collection loads extensions/fast only. This package keeps the published
+ * npm name working. It does not read config, register commands, or inject Fast
+ * fields. Startup only clears leftover footer status and the Fast beta header,
+ * which /reload otherwise keeps from the previous extension runtime.
+ */
+function stripFastBeta(headers: Record<string, string> | undefined): void {
+  if (!headers) return;
+  for (const key of Object.keys(headers)) {
+    if (key.toLowerCase() !== "anthropic-beta") continue;
+    const next = headers[key]
+      .split(",")
+      .map((part) => part.trim())
+      .filter((part) => part && part !== FAST_BETA);
+    if (next.length === 0) delete headers[key];
+    else headers[key] = next.join(",");
+  }
+}
+
+export default function claudeFastExtension(pi: ExtensionAPI): void {
+  pi.on("session_start", (_event, ctx) => {
+    stripFastBeta(ctx.model?.headers);
+    if (!ctx.hasUI) return;
+    ctx.ui.setStatus(EXTENSION_ID, undefined);
+    ctx.ui.notify(DEPRECATION_NOTICE, "warning");
+  });
 }

package/extensions/claude-fast/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@diegopetrucci/pi-claude-fast",
-  "version": "0.1.15",
-  "description": "A pi extension that enables Anthropic Claude Fast mode for supported Claude Opus models by injecting speed=fast.",
+  "version": "0.1.16",
+  "description": "Deprecated standalone leftover. Does not change provider requests; use @diegopetrucci/pi-fast.",
   "keywords": [
     "pi-package",
     "pi",
@@ -17,7 +17,6 @@
   },
   "files": [
     "index.ts",
-    "claude-fast.example.json",
     "README.md",
     ".pi-fleet-tested-version"
   ],

package/extensions/code-reviewer/README.md

@@ -1,10 +1,8 @@
 # code-reviewer
 
-Adds a read-only `code_reviewer` subagent tool that adapts The Last Harness [`code-reviewer`](https://github.com/diegopetrucci/the-last-harness/blob/main/agents/subagents/code-reviewer.md) prompt into a standalone pi extension.
+Adds a `code_reviewer` subagent tool that adapts The Last Harness [`code-reviewer`](https://github.com/diegopetrucci/the-last-harness/blob/main/agents/subagents/code-reviewer.md) prompt into a standalone pi extension.
 
-Use it when you want an isolated second-pass review of a proposed change against the local checkout. The tool stays read-only, checks ticket fit and scope first, then looks for diff mismatches, correctness bugs, security/safety issues, unnecessary complexity, and missing validation.
-
-This package is adapted from the TLH code-reviewer workflow for use as a standalone pi extension.
+Use it when you want an isolated second-pass review of a proposed change against the local checkout. The subagent uses read-only built-in tools plus any configured MCP tools, checks ticket fit and scope first, then looks for diff mismatches, correctness bugs, security/safety issues, unnecessary complexity, and missing validation.
 
 ## Install
 
@@ -46,7 +44,8 @@ When called, it launches an isolated in-memory child agent with:
 - an auto-selected review model that prefers contrarian-style opposite-provider and opposite-family candidates before same-provider fallbacks;
 - automatic model candidates constrained to the current Pi session's model scope when that scope is non-empty;
 - requested thinking taken from the active session when available, otherwise defaulting to `high` for reasoning models and `off` for non-reasoning models, then clamped to the selected model's supported level;
-- read-only tools only: `read`, `grep`, `find`, `ls`, and guarded `bash`;
+- read-only built-in tools: `read`, `grep`, `find`, `ls`, and guarded `bash`;
+- configured MCP tools (run without confirmation in the subagent; may have side effects; use for evidence gathering only; requires host Pi >=1.0.4; older hosts degrade to no MCP, no error); the global `mcp.json` always applies; a project `.pi/mcp.json` is loaded only when the host session has already trusted the project;
 - a local-checkout path guard for file inspection;
 - a bash guard that allows only direct read-only `git`, `gh`, or `pwd` invocations.
 
@@ -59,12 +58,13 @@ The review prompt prioritizes:
 5. simplicity and maintainability
 6. tests and validation gaps
 
-The final output is concise and includes a verdict, findings, validation notes, a scope check, and run details that show the final selected model and effective thinking level.
+The model report is concise and includes a verdict, findings, validation notes, and a scope check. The tool appends a run-details suffix with the final selected model, effective thinking level, turn count, tool calls, duration, working directory, and tools used.
 
 ## Read-only guarantees
 
 - The subagent is explicitly instructed not to implement changes.
 - Runtime guards block write/edit tools, shell control operators, pipelines, redirection, path traversal outside the checkout, mutating `git`/`gh` commands, `npm`/publish commands, and other filesystem mutation.
+- Runtime guards apply to built-in tools only; configured MCP tools pass the guard, run without confirmation, and are limited only by prompt guidance.
 - Built-in file-inspection tools are preferred over shell commands for local files.
 
 ## Example