@diegopetrucci/pi-extensions 0.1.78 → 0.1.79

package/extensions/git-footer/index.ts

@@ -13,10 +13,6 @@ type GitStatusSnapshot = {
 
 type PullRequestSnapshot = {
   number?: number | string;
-  state?: string;
-  isDraft?: boolean;
-  url?: string;
-  title?: string;
 };
 
 type CommandResult = {
@@ -74,7 +70,7 @@ const GIT_STATUS_ARGS = [
   "--porcelain=v2",
   "--branch",
 ] as const;
-const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number,state,isDraft,url,title"] as const;
+const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number"] as const;
 
 function createEmptyGitStatus(): GitStatusSnapshot {
   return {
@@ -202,10 +198,6 @@ function parsePullRequestJson(stdout: string): PullRequestSnapshot | undefined {
   if (typeof record.number === "number" || typeof record.number === "string") {
     snapshot.number = record.number;
   }
-  if (typeof record.state === "string") snapshot.state = record.state;
-  if (typeof record.isDraft === "boolean") snapshot.isDraft = record.isDraft;
-  if (typeof record.url === "string") snapshot.url = record.url;
-  if (typeof record.title === "string") snapshot.title = record.title;
   return snapshot;
 }
 
@@ -228,13 +220,7 @@ function pullRequestSnapshotsEqual(
   left: PullRequestSnapshot | undefined,
   right: PullRequestSnapshot | undefined,
 ): boolean {
-  return (
-    left?.number === right?.number &&
-    left?.state === right?.state &&
-    left?.isDraft === right?.isDraft &&
-    left?.url === right?.url &&
-    left?.title === right?.title
-  );
+  return left?.number === right?.number;
 }
 
 function defaultRunner(

package/extensions/git-footer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-git-footer",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "A TLH-style git status add-on for pi's built-in footer.",
   "keywords": [
     "pi-package",

package/extensions/librarian/index.ts

@@ -766,11 +766,6 @@ async function writeLibrarianPreferences(preferences: LibrarianPreferences): Pro
   await fs.writeFile(configPath, `${JSON.stringify(config, null, 2)}\n`, "utf8");
 }
 
-async function writeCachePreference(preference: CacheMode): Promise<void> {
-  const preferences = await readLibrarianPreferences();
-  await writeLibrarianPreferences({ ...preferences, cacheMode: preference });
-}
-
 function resolveCacheDecision(preference: CacheMode): { enabled: boolean; reason: string } {
   if (preference === "enabled") {
     return { enabled: true, reason: "cache preference enabled; using cached local checkouts" };
@@ -1045,7 +1040,7 @@ function tryCurlyQuotePathVariant(filePath: string): string {
 }
 /**
  * Returns all read path candidates including Pi's resolveReadPathAsync fallback
- * variants. Used only for the read tool guard; grep/find/ls use resolveToCwd only.
+ * variants.
  */
 function getReadPathCandidates(resolved: string): string[] {
   const nfd = tryNFDPathVariant(resolved);
@@ -1058,11 +1053,6 @@ function getReadPathCandidates(resolved: string): string[] {
   ];
 }
 
-function resolveToolPath(cwd: string, rawPath: string): string {
-  const normalized = rawPath.startsWith("@") ? rawPath.slice(1) : rawPath;
-  return path.isAbsolute(normalized) ? path.resolve(normalized) : path.resolve(cwd, normalized);
-}
-
 function getBlockedBashReason(
   command: string,
   options: { workspace: string; cacheRoot: string; cacheEnabled: boolean },
@@ -1165,17 +1155,9 @@ function createLibrarianRuntimeGuardExtension(options: {
             (options.cacheEnabled && (isInside(options.cacheRoot, p) || isInside(realCacheRoot, p)))
           );
         }
-        const resolved = resolveToolPath(options.workspace, input.path);
-        const realPath = await fs.realpath(resolved).catch(() => resolved);
-        if (!isAllowedPath(realPath))
-          return {
-            block: true,
-            reason: `Librarian read is limited to its workspace/cache: ${realPath}`,
-          };
-        // Pi-effective path check (monotonic: block if either resolution is
-        // outside). For the read tool, also check Pi's resolveReadPathAsync
-        // fallback variants so a missing file whose variant resolves outside is
-        // not silently allowed.
+        // For the read tool, also check Pi's resolveReadPathAsync fallback
+        // variants so a missing file whose variant resolves outside is not
+        // silently allowed.
         try {
           const piResolved = resolveToCwd(input.path, options.workspace);
           const candidates = getReadPathCandidates(piResolved);
@@ -1482,7 +1464,6 @@ export const __test__ = {
   parseModelPreference,
   resolveThinkingLevel,
   aggregateAssistantUsage,
-  addAssistantMessageUsage,
   addSessionEventUsage,
   inspectFinalAssistant,
   classifyRunFailure,
@@ -1490,14 +1471,12 @@ export const __test__ = {
   createLibrarianAbortController,
   cleanupExpiredCache,
   cleanupExpiredRepos,
-  cleanupLockOwnershipError,
   tryAcquireCleanupLock,
   heartbeatCleanupLock,
   releaseCleanupLock,
   CLEANUP_LOCK_FILE,
   CLEANUP_LOCK_STALE_MS,
   CLEANUP_LOCK_MAX_LEASE_MS,
-  isCleanupLockLeaseExpired,
 };
 
 export default function librarianExtension(pi: ExtensionAPI) {
@@ -1668,17 +1647,6 @@ export default function librarianExtension(pi: ExtensionAPI) {
       const cacheRoot = getUserCacheReposRoot();
       const configPath = getCacheConfigPath();
 
-      const setPreference = async (mode: CacheMode): Promise<string | undefined> => {
-        cachePreference = mode;
-        try {
-          await writeCachePreference(mode);
-          return undefined;
-        } catch (error) {
-          const message = error instanceof Error ? error.message : String(error);
-          return `Preference changed for this process, but could not save ${configPath}: ${message}`;
-        }
-      };
-
       const formatSetMessage = (mode: CacheMode, warning?: string): string => {
         const main =
           mode === "enabled"
@@ -1688,20 +1656,20 @@ export default function librarianExtension(pi: ExtensionAPI) {
       };
 
       if (action === "on" || action === "enable") {
-        const warning = await setPreference("enabled");
+        const warning = await savePreferences({ ...currentPreferences(), cacheMode: "enabled" });
         notifyCommand(ctx, formatSetMessage("enabled", warning), warning ? "warning" : "info");
         return;
       }
 
       if (action === "off" || action === "disable") {
-        const warning = await setPreference("disabled");
+        const warning = await savePreferences({ ...currentPreferences(), cacheMode: "disabled" });
         notifyCommand(ctx, formatSetMessage("disabled", warning), warning ? "warning" : "info");
         return;
       }
 
       if (action === "toggle") {
         const next = cachePreference === "enabled" ? "disabled" : "enabled";
-        const warning = await setPreference(next);
+        const warning = await savePreferences({ ...currentPreferences(), cacheMode: next });
         notifyCommand(ctx, formatSetMessage(next, warning), warning ? "warning" : "info");
         return;
       }

package/extensions/librarian/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-librarian",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "description": "A pi GitHub research scout with a toggleable local repo checkout cache under the user's OS cache directory.",
   "keywords": [
     "pi-package",

package/extensions/minimal-footer/README.md

@@ -10,7 +10,7 @@ It replaces pi's built-in footer with a cleaner two-line layout that focuses on
 - TLH-style git dirty counts, ahead/behind counts, and optional PR number
 - current repo name
 - current context percentage
-- red `DUMB ZONE` indicator when context usage is above 300k tokens
+- red `DUMB ZONE` indicator when context usage is above 350k tokens
 - current model and thinking level
 - OpenAI Codex usage windows, labeled from the durations reported by OpenAI
 - `xp` marker when Pi experimental features are enabled
@@ -39,7 +39,7 @@ main · +1 ~2 ?3 ↑1 • PR #42
 
 The markers are `!` conflicts, `+` staged, `~` unstaged, `?` untracked, `↑` ahead, and `↓` behind. PR numbers are fetched best-effort with `gh pr view`. Git polling is skipped until Pi reports that the project is trusted, and `git status` is run with fsmonitor disabled.
 
-When context usage is above 300k tokens, the bottom-left line includes a red warning:
+When context usage is above 350k tokens, the bottom-left line includes a red warning:
 
 ```text
 44.1% · DUMB ZONE
@@ -115,7 +115,7 @@ Example:
     "showPercent": true,
     "dumbZone": {
       "enabled": true,
-      "thresholdTokens": 300000,
+      "thresholdTokens": 350000,
       "label": "DUMB ZONE",
       "color": "error"
     }
@@ -231,7 +231,7 @@ Disable git dirty/ahead/PR status:
 
 - **Top left:** current git branch plus dirty/ahead/PR status when available
 - **Top right:** current repo directory name
-- **Bottom left:** current context usage percentage, plus red `DUMB ZONE` above 300k context tokens
+- **Bottom left:** current context usage percentage, plus red `DUMB ZONE` above 350k context tokens
 - **Bottom left on `openai-codex`:** current context usage percentage plus the Codex usage windows reported for the account
 - **Bottom left on `openai` ChatGPT OAuth:** current context usage percentage plus `usage unsupported`; no legacy WHAM request is attempted
 - **Bottom left with `PI_EXPERIMENTAL=1`:** current context usage percentage plus `xp`
@@ -246,7 +246,7 @@ This extension also lives inside the broader [`pi-extensions`](../../README.md)
 - Replaces pi's built-in footer entirely.
 - Uses pi footer data for git branch updates, plus background cached git/gh checks for dirty counts and PR number after project trust is granted.
 - Shows only context percentage, not context window size.
-- Shows `DUMB ZONE` only while context usage is above 300k tokens.
+- Shows `DUMB ZONE` only while context usage is above 350k tokens.
 - Shows the model id rather than a provider-specific display label.
 - Shows `xp` when `PI_EXPERIMENTAL=1`.
 - For `openai-codex`, reads pi's stored OAuth login and fetches usage from ChatGPT's backend usage endpoint. Window labels are inferred from each window's reported duration; missing or unrecognized durations use neutral `usage` labels rather than guessing from window position.

package/extensions/minimal-footer/index.ts

@@ -24,7 +24,7 @@ const DEFAULT_CONFIG: MinimalFooterConfig = {
     showPercent: true,
     dumbZone: {
       enabled: true,
-      thresholdTokens: 300_000,
+      thresholdTokens: 350_000,
       label: "DUMB ZONE",
       color: "error",
     },
@@ -148,10 +148,6 @@ type GitStatusSnapshot = {
 
 type PullRequestSnapshot = {
   number?: number | string;
-  state?: string;
-  isDraft?: boolean;
-  url?: string;
-  title?: string;
 };
 
 type CommandResult = {
@@ -200,7 +196,7 @@ const GIT_STATUS_ARGS = [
   "--porcelain=v2",
   "--branch",
 ] as const;
-const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number,state,isDraft,url,title"] as const;
+const GH_PR_VIEW_ARGS = ["pr", "view", "--json", "number"] as const;
 
 function readConfigFile(path: string): RecursivePartial<MinimalFooterConfig> {
   if (!existsSync(path)) return {};
@@ -468,10 +464,6 @@ function parsePullRequestJson(stdout: string): PullRequestSnapshot | undefined {
   if (typeof record.number === "number" || typeof record.number === "string") {
     snapshot.number = record.number;
   }
-  if (typeof record.state === "string") snapshot.state = record.state;
-  if (typeof record.isDraft === "boolean") snapshot.isDraft = record.isDraft;
-  if (typeof record.url === "string") snapshot.url = record.url;
-  if (typeof record.title === "string") snapshot.title = record.title;
   return snapshot;
 }
 
@@ -494,13 +486,7 @@ function pullRequestSnapshotsEqual(
   left: PullRequestSnapshot | undefined,
   right: PullRequestSnapshot | undefined,
 ): boolean {
-  return (
-    left?.number === right?.number &&
-    left?.state === right?.state &&
-    left?.isDraft === right?.isDraft &&
-    left?.url === right?.url &&
-    left?.title === right?.title
-  );
+  return left?.number === right?.number;
 }
 
 function defaultRunner(

package/extensions/minimal-footer/minimal-footer.example.json

@@ -3,7 +3,7 @@
     "showPercent": true,
     "dumbZone": {
       "enabled": true,
-      "thresholdTokens": 300000,
+      "thresholdTokens": 350000,
       "label": "DUMB ZONE",
       "color": "error"
     }

package/extensions/minimal-footer/openai-usage.ts

@@ -15,7 +15,6 @@ export type UsageAvailability = "unsupported" | "unavailable";
 
 interface WhamUsageWindow {
   limit_window_seconds?: number;
-  reset_at?: number;
   used_percent?: number;
 }
 
@@ -29,7 +28,6 @@ interface WhamUsageResponse {
 export interface UsageWindow {
   usedPercent?: number;
   windowSeconds?: number;
-  resetAt?: number;
 }
 
 export interface UsageSnapshot {
@@ -59,19 +57,12 @@ function normalizeWindowSeconds(value?: number): number | undefined {
   return value;
 }
 
-function normalizeResetAt(value?: number): number | undefined {
-  if (typeof value !== "number" || !Number.isFinite(value)) return undefined;
-  return value * 1000;
-}
-
 function parseUsageWindow(window?: WhamUsageWindow): UsageWindow | undefined {
   if (!window) return undefined;
   const usedPercent = normalizeUsedPercent(window.used_percent);
   const windowSeconds = normalizeWindowSeconds(window.limit_window_seconds);
-  const resetAt = normalizeResetAt(window.reset_at);
-  if (usedPercent === undefined && windowSeconds === undefined && resetAt === undefined)
-    return undefined;
-  return { usedPercent, windowSeconds, resetAt };
+  if (usedPercent === undefined && windowSeconds === undefined) return undefined;
+  return { usedPercent, windowSeconds };
 }
 
 function parseUsageSnapshot(data: WhamUsageResponse): Omit<UsageSnapshot, "fetchedAt"> {

package/extensions/minimal-footer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-minimal-footer",
-  "version": "0.1.24",
+  "version": "0.1.25",
   "description": "A minimal custom footer for pi.",
   "keywords": [
     "pi-package",

package/extensions/oracle/index.ts

@@ -1810,8 +1810,7 @@ function createOracleExtension(pi: ExtensionAPI, deps: OracleExtensionDeps = {})
           onUpdate?.(partial);
         };
 
-        let lastErrorResult: { ok: false; error: string; details: OracleDetails } | undefined;
-        for (let index = 0; index < attempts.length; index++) {
+        for (let index = 0; ; index++) {
           const previous = index > 0 ? attempts[index - 1] : undefined;
           const attempt = previous
             ? withFallbackReason(attempts[index], previous)
@@ -1835,7 +1834,6 @@ function createOracleExtension(pi: ExtensionAPI, deps: OracleExtensionDeps = {})
             });
           }
 
-          lastErrorResult = result;
           const canFallBack = index < attempts.length - 1 && isModelAvailabilityError(result.error);
           if (!canFallBack) {
             return finalizeResult({
@@ -1844,30 +1842,6 @@ function createOracleExtension(pi: ExtensionAPI, deps: OracleExtensionDeps = {})
             });
           }
         }
-
-        return finalizeResult({
-          content: [
-            {
-              type: "text",
-              text: lastErrorResult?.error ?? "Oracle could not select an available model.",
-            },
-          ],
-          details: lastErrorResult?.details ?? {
-            modelRef: "",
-            provider: ctx.model?.provider ?? "unknown",
-            modelId: "",
-            modelName: undefined,
-            thinkingLevel: configuredThinkingLevel ?? DEFAULT_THINKING_LEVEL,
-            autoSelected: true,
-            selectionReason: "No authenticated models were available to run the oracle.",
-            includeBash: params.includeBash ?? false,
-            usage: createEmptyUsage(),
-            stderr: "",
-            exitCode: 1,
-            durationMs: 0,
-            cwd: params.cwd ?? ctx.cwd,
-          },
-        });
       } finally {
         activeRuns.delete(toolCallId);
         updateOracleUi(ctx, activeRuns);

package/extensions/oracle/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-oracle",
-  "version": "0.1.31",
+  "version": "0.1.32",
   "description": "An Amp-style oracle extension for pi that consults the strongest reasoning model on your current provider.",
   "keywords": [
     "pi-package",

package/extensions/permission-gate/index.ts

@@ -77,17 +77,9 @@ function normalizeWindowsShellPathLocal(filePath: string): string {
 }
 
 /**
- * Mirrors Pi's normalizePath (dist/utils/paths.js).
- *
- * Options (all default false/true as noted):
- *   normalizeUnicodeSpaces — replace Unicode space characters with ASCII space.
- *   stripAtPrefix         — strip a leading "@" path sigil.
- *   expandTilde           — expand "~"/"~/" to homedir (default true).
- *
- * On win32, Git Bash / WSL drive-path normalisation always runs (not gated
- * by expandTilde); only the tilde expansion itself is gated by expandTilde.
- * Always resolves "file://" URLs via fileURLToPath; throws for non-local hosts
- * (e.g. file://hostname/…) or invalid URLs.
+ * Mirrors Pi's normalizePath (dist/utils/paths.js). On win32, Git Bash / WSL
+ * drive-path normalisation always runs; only tilde expansion is gated by
+ * expandTilde.
  */
 function normalizePath(
   input: string,
@@ -797,6 +789,28 @@ async function confirmProtectedPathAction(
   );
 }
 
+async function confirmLexicalOrResolvedPath(
+  toolName: "write" | "edit",
+  normalizedPath: NormalizedPath,
+  ctx: ConfirmationContext & { cwd?: string },
+): Promise<GuardDecision> {
+  // Monotonic hardening: also check via Pi-style path resolution against ctx.cwd.
+  // OR logic: protected if EITHER the lexical check OR the Pi-resolved check matches.
+  // When only the Pi-resolved check catches a path, show the resolved absolute path in
+  // the reason so the user understands why the plain-looking path is protected.
+  const lexicalProtected = isProtectedPath(normalizedPath);
+  const piResult = checkPiResolvedPath(normalizedPath.original, ctx.cwd ?? process.cwd());
+  const piAlsoProtected = piResult.status === "protected" || piResult.status === "error";
+  const displaySuffix = lexicalProtected
+    ? undefined
+    : piResult.status === "protected"
+      ? ` (resolves to ${piResult.resolvedPath})`
+      : piResult.status === "error"
+        ? " (path could not be resolved)"
+        : undefined;
+  return confirmProtectedPathAction(toolName, normalizedPath, ctx, piAlsoProtected, displaySuffix);
+}
+
 export default function (pi: ExtensionAPI) {
   pi.on("tool_call", async (event, ctx) => {
     if (event.toolName === "bash" || event.toolName === "powershell") {
@@ -824,60 +838,14 @@ export default function (pi: ExtensionAPI) {
       return undefined;
     }
 
-    if (event.toolName === "write") {
-      const normalizedPath = validateWriteInput(event.input);
-      if (!normalizedPath) {
-        return { block: true, reason: "Malformed write input blocked" };
-      }
-      // Monotonic hardening: also check via Pi-style path resolution against ctx.cwd.
-      // OR logic: protected if EITHER the lexical check OR the Pi-resolved check matches.
-      // When only the Pi-resolved check catches a path, show the resolved absolute path in
-      // the reason so the user understands why the plain-looking path is protected.
-      const lexicalProtected = isProtectedPath(normalizedPath);
-      const piResult = checkPiResolvedPath(normalizedPath.original, ctx.cwd ?? process.cwd());
-      const piAlsoProtected = piResult.status === "protected" || piResult.status === "error";
-      const displaySuffix = lexicalProtected
-        ? undefined
-        : piResult.status === "protected"
-          ? ` (resolves to ${piResult.resolvedPath})`
-          : piResult.status === "error"
-            ? " (path could not be resolved)"
-            : undefined;
-      return confirmProtectedPathAction(
-        "write",
-        normalizedPath,
-        ctx,
-        piAlsoProtected,
-        displaySuffix,
-      );
-    }
-
-    if (event.toolName === "edit") {
-      const normalizedPath = validateEditInput(event.input);
+    if (event.toolName === "write" || event.toolName === "edit") {
+      const toolName = event.toolName;
+      const normalizedPath =
+        toolName === "write" ? validateWriteInput(event.input) : validateEditInput(event.input);
       if (!normalizedPath) {
-        return { block: true, reason: "Malformed edit input blocked" };
+        return { block: true, reason: `Malformed ${toolName} input blocked` };
       }
-      // Monotonic hardening: also check via Pi-style path resolution against ctx.cwd.
-      // OR logic: protected if EITHER the lexical check OR the Pi-resolved check matches.
-      // When only the Pi-resolved check catches a path, show the resolved absolute path in
-      // the reason so the user understands why the plain-looking path is protected.
-      const lexicalProtected = isProtectedPath(normalizedPath);
-      const piResult = checkPiResolvedPath(normalizedPath.original, ctx.cwd ?? process.cwd());
-      const piAlsoProtected = piResult.status === "protected" || piResult.status === "error";
-      const displaySuffix = lexicalProtected
-        ? undefined
-        : piResult.status === "protected"
-          ? ` (resolves to ${piResult.resolvedPath})`
-          : piResult.status === "error"
-            ? " (path could not be resolved)"
-            : undefined;
-      return confirmProtectedPathAction(
-        "edit",
-        normalizedPath,
-        ctx,
-        piAlsoProtected,
-        displaySuffix,
-      );
+      return confirmLexicalOrResolvedPath(toolName, normalizedPath, ctx);
     }
 
     return undefined;

package/extensions/permission-gate/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-permission-gate",
-  "version": "0.1.17",
+  "version": "0.1.18",
   "description": "A pi extension that prompts before dangerous shell commands and protected file writes.",
   "keywords": [
     "pi-package",

package/extensions/review/README.md

@@ -29,7 +29,9 @@ Then reload pi:
 
 ## Notes
 
+- `/review` requires interactive mode (TUI); `/end-review` requires a UI.
 - This fork adds loop fixing, custom review instructions persisted in the session, a blocking-findings parser, and `/end-review` handoff modes (return only, return and summarize, or return and fix findings). Loop fixing cycles until the parser reports no blocking findings: tagged P0–P2 items, or a "needs attention" verdict when no priority tags are present. Commit review is excluded from loop fixing.
+- Loop fixing always uses Empty branch mode and skips the Empty branch / Current session choice.
 - PR review requires `gh` access and a clean working tree for tracked files.
 - If the project is trusted and a `REVIEW_GUIDELINES.md` file exists next to the repo's `<pi-config-dir>` directory, its contents are appended to the review prompt. `<pi-config-dir>` is Pi's runtime config directory name (`CONFIG_DIR_NAME`; `.pi` by default).
 

package/extensions/review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-review",
-  "version": "0.1.15",
+  "version": "0.1.16",
   "description": "A standalone pi extension that adds /review and /end-review commands adapted from mitsuhiko/agent-stuff.",
   "keywords": [
     "pi-package",

package/extensions/triage-comments/index.ts

@@ -272,8 +272,6 @@ type ToolCall = {
   id: string;
   name: string;
   args: unknown;
-  startedAt: number;
-  endedAt?: number;
   isError?: boolean;
 };
 
@@ -736,12 +734,6 @@ function getReadPathCandidates(resolved: string): string[] {
   ];
 }
 
-function resolveToolPath(cwd: string, rawPath: string | undefined): string {
-  const input = rawPath?.trim() || ".";
-  const normalized = input.startsWith("@") ? input.slice(1) : input;
-  return path.isAbsolute(normalized) ? path.resolve(normalized) : path.resolve(cwd, normalized);
-}
-
 async function assertToolPathInsideCwd(
   cwd: string,
   rawPath: unknown,
@@ -754,11 +746,6 @@ async function assertToolPathInsideCwd(
   // where realpath fails and the lexical fallback starts with the alias prefix.
   const root = await fs.realpath(cwd).catch(() => path.resolve(cwd));
   const lexicalCwd = path.resolve(cwd);
-  const resolved = resolveToolPath(cwd, rawPath);
-  const realPath = await fs.realpath(resolved).catch(() => resolved);
-  if (!isInside(root, realPath) && !isInside(lexicalCwd, realPath))
-    return `${toolName} is limited to the local checkout: ${realPath}`;
-  // Pi-effective path check (monotonic: block if either resolution is outside).
   // For the read tool, also check Pi's resolveReadPathAsync fallback variants so
   // a missing file whose variant resolves outside is not silently allowed.
   try {
@@ -2841,7 +2828,6 @@ export default function triageCommentsExtension(pi: ExtensionAPI) {
                 id: event.toolCallId,
                 name: event.toolName,
                 args: event.args,
-                startedAt: Date.now(),
               });
               if (details.toolCalls.length > MAX_TOOL_CALLS_TO_KEEP) {
                 details.toolCalls.splice(0, details.toolCalls.length - MAX_TOOL_CALLS_TO_KEEP);
@@ -2851,7 +2837,6 @@ export default function triageCommentsExtension(pi: ExtensionAPI) {
             case "tool_execution_end": {
               const call = details.toolCalls.find((item) => item.id === event.toolCallId);
               if (call) {
-                call.endedAt = Date.now();
                 call.isError = event.isError;
               }
               emit();

package/extensions/triage-comments/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-triage-comments",
-  "version": "0.1.15",
+  "version": "0.1.16",
   "description": "A pi extension that adds /triage-comments and a read-only triage_comments subagent tool for review-comment triage.",
   "keywords": [
     "pi-package",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegopetrucci/pi-extensions",
-  "version": "0.1.78",
+  "version": "0.1.79",
   "description": "A collection of pi extensions and skills for annotation UIs, context management, workflow audits, contrarian review, review-comment triage, notifications, brrr push alerts, safety guards, GitHub research, repo-local knowledge, todos, tool rendering, model/provider helpers, and Xiaohei-style article illustrations.",
   "keywords": [
     "pi-package",