portable-agent-layer 0.85.15 → 0.85.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "portable-agent-layer",
-  "version": "0.85.15",
+  "version": "0.85.17",
   "description": "PAL — Portable Agent Layer: persistent personal context for AI coding assistants",
   "type": "module",
   "bin": {

package/src/cli/dependencies.ts

@@ -0,0 +1,11 @@
+import { unresolvedDependencies } from "./doctor/state";
+
+/**
+ * The `bun install` arguments that make PAL's dependencies resolve, or null when
+ * they already do. A published package ships no lockfile, and `bun add -g` has
+ * already installed its dependencies next to it.
+ */
+export function dependencyInstall(pkg: string, repoMode: boolean): string[] | null {
+  if (repoMode) return ["install", "--frozen-lockfile"];
+  return unresolvedDependencies(pkg).length > 0 ? ["install", "--production"] : null;
+}

package/src/cli/doctor/agents.ts

@@ -3,7 +3,7 @@ import { existsSync, readdirSync, readFileSync, statSync } from "node:fs";
 import { isAbsolute, resolve } from "node:path";
 import { palPkg, platform } from "../../hooks/lib/paths";
 import { NO_SESSION_AGENT_MESSAGE } from "../session-agent";
-import { type Finding, failing, optional, passed, warning } from "./finding";
+import { type Finding, type Fix, failing, optional, passed, warning } from "./finding";
 
 export type AgentName = "claude" | "opencode" | "cursor" | "copilot" | "codex";
 
@@ -139,9 +139,10 @@ const LAYOUT: Record<AgentName, AgentLayout> = {
   },
 };
 
-const reinstall = (agent: AgentName) => ({
+const reinstall = (agent: AgentName): Fix => ({
   say: `Reinstall PAL for ${LAYOUT[agent].label}`,
   command: `pal cli install --${agent}`,
+  external: false,
 });
 
 /** Hook-config keys that carry a shell command: cross-platform and per-shell variants. */
@@ -215,6 +216,7 @@ function hookFindings(agent: AgentName, file: string): Finding[] {
         {
           say: `Fix the JSON in ${file}, then reinstall`,
           command: `pal cli install --${agent}`,
+          external: false,
         }
       ),
     ];
@@ -310,6 +312,7 @@ export function agentFindings(agents: AgentName[]): Finding[] {
       : failing("agents-md", "AGENTS.md is missing", {
           say: "Reinstall PAL",
           command: "pal cli install",
+          external: false,
         }),
     ...agents.flatMap(oneAgentFindings),
   ];

package/src/cli/doctor/environment.ts

@@ -83,6 +83,7 @@ export function playwrightFinding(host: BrowserHost): Finding {
         ? "Install it as Ubuntu 24.04, which Playwright supports"
         : "Install it",
       command: chromiumInstallCommand(host),
+      external: false,
     }
   );
 }
@@ -113,15 +114,34 @@ interface PalInstall {
   pal: string | null;
   pkg: string;
   repoMode: boolean;
+  bunBin: string;
+  inBunBin: boolean;
 }
 
 function putPalOnPath(install: PalInstall): Fix {
+  if (install.inBunBin)
+    return {
+      say: `Add ${install.bunBin} to PATH in your shell profile — Bun put pal there`,
+    };
   return install.repoMode
     ? {
         say: "Link the checkout — a shell alias is invisible to agents",
         command: `cd ${install.pkg} && bun link`,
+        external: false,
       }
-    : { say: "Install PAL globally", command: "bun add -g portable-agent-layer" };
+    : {
+        say: "Install PAL globally",
+        command: "bun add -g portable-agent-layer",
+        external: true,
+      };
+}
+
+function bunBin(): string {
+  return resolve(process.env.BUN_INSTALL ?? resolve(homedir(), ".bun"), "bin");
+}
+
+function isInBunBin(): boolean {
+  return ["pal", "pal.exe"].some((name) => existsSync(resolve(bunBin(), name)));
 }
 
 export function palOnPathFinding(install: PalInstall): Finding {
@@ -140,11 +160,12 @@ function rtkInstall(): Fix {
       say: "Download rtk.exe from https://github.com/rtk-ai/rtk/releases and add it to PATH",
     };
   if (process.platform === "darwin")
-    return { say: "Install rtk", command: "brew install rtk" };
+    return { say: "Install rtk", command: "brew install rtk", external: true };
   return {
     say: "Install rtk",
     command:
       "curl -fsSL https://raw.githubusercontent.com/rtk-ai/rtk/refs/heads/master/install.sh | sh",
+    external: true,
   };
 }
 
@@ -161,6 +182,8 @@ export function environmentFindings(rtk: ToolCheck): Finding[] {
       pal: findBinaryOnPath("pal"),
       pkg: palPkg(),
       repoMode: isRepoMode(),
+      bunBin: bunBin(),
+      inBunBin: isInBunBin(),
     }),
     playwrightFinding(thisBrowserHost()),
     rtkFinding(rtk),

package/src/cli/doctor/finding.ts

@@ -6,10 +6,10 @@
 
 export type Severity = "fail" | "warn" | "optional" | "ok";
 
-export interface Fix {
-  say: string;
-  command?: string;
-}
+/** `external`: the command belongs to another tool, so PAL leaves running it to you. */
+export type Fix =
+  | { say: string; command?: undefined }
+  | { say: string; command: string; external: boolean };
 
 export interface Finding {
   id: string;

package/src/cli/doctor/health.ts

@@ -33,10 +33,12 @@ function claudeLoginExpiredFinding(
     return warning(`hook-errors.${group.source}`, title, {
       say: "CLAUDE_CODE_OAUTH_TOKEN is set — if these failures predate it, they clear 24h after the last one; if not, the token is invalid, so create a new one",
       command: "claude setup-token",
+      external: true,
     });
   return failing(`hook-errors.${group.source}`, title, {
     say: "Create a year-long token, then export it as CLAUDE_CODE_OAUTH_TOKEN in your shell profile",
     command: "claude setup-token",
+    external: true,
   });
 }
 
@@ -66,7 +68,7 @@ export function migrationFindings(pending: PendingMigration[]): Finding[] {
     return warning(
       `migration.${migration.id}`,
       `Migration pending: ${migration.description}${detail}`,
-      { say: "Run it", command: "pal cli migrate" }
+      { say: "Run it", command: "pal cli migrate", external: false }
     );
   });
 }
@@ -76,6 +78,7 @@ export function updateFinding(cache: UpdateCache | null): Finding {
   return warning("update", `PAL ${cache.latest} is out (${cache.current} installed)`, {
     say: "Update",
     command: "pal cli update",
+    external: false,
   });
 }
 

package/src/cli/doctor/inference.ts

@@ -34,6 +34,7 @@ export function leakedEnvFindings(env: Env, os: NodeJS.Platform): Finding[] {
           {
             say: "Clear it, and remove it from your shell profile if it is set there",
             command: unsetCommand(name, os),
+            external: true,
           }
         )
       : passed(`env.${name}`, `${name} not set`)

package/src/cli/doctor/state.ts

@@ -95,6 +95,7 @@ function identityFinding(settings: Record<string, unknown>): Finding {
   return warning("identity", "Identity is incomplete — PAL does not know both names", {
     say: "Answer the identity questions",
     command: "pal cli install",
+    external: false,
   });
 }
 
@@ -120,6 +121,7 @@ function settingsFindings(): Finding[] {
       warning("settings.missing", "pal-settings.json is missing", {
         say: "Recreate it",
         command: "pal cli install",
+        external: false,
       }),
     ];
   let settings: unknown;
@@ -165,6 +167,7 @@ function telosFindings(home: string): Finding[] {
       failing("telos.missing", "TELOS is not scaffolded", {
         say: "Scaffold it",
         command: "pal cli init",
+        external: false,
       }),
     ];
   const unanswered = telosStatus(home)
@@ -205,6 +208,7 @@ export function bindingFinding(issue: BindingIssue): Finding {
   return warning(`binding.${project}`, title, {
     say: "Point it at its checkout",
     command: `pal cli project set-path ${project} <path>`,
+    external: false,
   });
 }
 
@@ -250,6 +254,7 @@ function dependencyFinding(): Finding {
         {
           say: "Install them",
           command: "pal cli install",
+          external: false,
         }
       );
 }

package/src/cli/doctor/version-control.ts

@@ -30,22 +30,24 @@ const GH_ON_LINUX: Partial<Record<LinuxFamily, string>> = {
 
 export function gitInstallHint(platform: NodeJS.Platform, osRelease: string): Fix {
   if (platform === "win32")
-    return { say: "Install git", command: "winget install Git.Git" };
-  if (platform === "darwin") return { say: "Install git", command: "brew install git" };
+    return { say: "Install git", command: "winget install Git.Git", external: true };
+  if (platform === "darwin")
+    return { say: "Install git", command: "brew install git", external: true };
   const family = linuxFamily(osRelease);
   return family
-    ? { say: "Install git", command: GIT_ON_LINUX[family] }
+    ? { say: "Install git", command: GIT_ON_LINUX[family], external: true }
     : { say: "Install git with your package manager" };
 }
 
 export function ghInstallHint(platform: NodeJS.Platform, osRelease: string): Fix {
   if (platform === "win32")
-    return { say: "Install gh", command: "winget install GitHub.cli" };
-  if (platform === "darwin") return { say: "Install gh", command: "brew install gh" };
+    return { say: "Install gh", command: "winget install GitHub.cli", external: true };
+  if (platform === "darwin")
+    return { say: "Install gh", command: "brew install gh", external: true };
   const family = linuxFamily(osRelease);
   const command = family ? GH_ON_LINUX[family] : undefined;
   return command
-    ? { say: "Install gh", command }
+    ? { say: "Install gh", command, external: true }
     : { say: `Install gh as described at ${GH_LINUX_DOCS}` };
 }
 
@@ -94,6 +96,7 @@ function ghFinding(host: Host): Finding {
     return warning("gh.logged-out", `${gh.version} is not logged in`, {
       say: "Log in",
       command: "gh auth login",
+      external: true,
     });
   return passed("gh", `${gh.version}, logged in`);
 }

package/src/cli/index.ts

@@ -520,10 +520,16 @@ function targetInstallers(): [keyof Targets, string, () => Promise<unknown>][] {
 async function install(targets: Targets, args: string[]): Promise<number> {
   narrateSteps(args.includes("--verbose"));
   const pkg = palPkg();
-  if (runQuietly("bun", ["install", "--frozen-lockfile"], pkg) !== 0) {
+  const { dependencyInstall } = await import("./dependencies");
+  const { isRepoMode } = await import("../hooks/handlers/update-check");
+  const bunInstall = dependencyInstall(pkg, isRepoMode());
+  if (bunInstall && runQuietly("bun", bunInstall, pkg) !== 0) {
     log.warn("bun install failed — continuing anyway, but hooks may not work");
   }
 
+  const { applyPendingMigrations } = await import("./migrate");
+  for (const line of applyPendingMigrations()) log.info(line);
+
   // Uses `bun x` (not `bunx`) for Windows compatibility — bunx resolves unreliably under cmd.exe.
   if (process.env.PAL_SKIP_BROWSER_INSTALL !== "1") installChromium();
 

package/src/cli/leftovers.ts

@@ -0,0 +1,118 @@
+import { existsSync, lstatSync, readlinkSync, renameSync, unlinkSync } from "node:fs";
+import { resolve } from "node:path";
+import { paths, platform } from "../hooks/lib/paths";
+import { readJson, writeJson } from "../targets/lib";
+
+/** Something an older PAL wrote that the current one neither writes nor reads. */
+export interface Leftover {
+  id: string;
+  description: string;
+  find(): string[];
+  remove(): void;
+}
+
+type Json = Record<string, unknown>;
+
+const claudeSettings = () => resolve(platform.claudeDir(), "settings.json");
+const palSettings = () => resolve(paths.memory(), "pal-settings.json");
+
+function isPathScopedFileToolRule(rule: string): boolean {
+  return /^(?:Grep|Glob)\(/.test(rule);
+}
+
+function claudeAllowRules(): string[] {
+  const settings = readJson<Json>(claudeSettings(), {});
+  const allow = (settings.permissions as { allow?: unknown } | undefined)?.allow;
+  return Array.isArray(allow)
+    ? allow.filter((r): r is string => typeof r === "string")
+    : [];
+}
+
+const claudeFileToolRules: Leftover = {
+  id: "claude-file-tool-rules",
+  description:
+    "Remove Grep()/Glob() allow rules older templates wrote — Claude Code ignores them and warns on every prompt",
+  find: () => claudeAllowRules().filter(isPathScopedFileToolRule),
+  remove() {
+    const settings = readJson<Json>(claudeSettings(), {});
+    const permissions = settings.permissions as { allow: string[] };
+    permissions.allow = permissions.allow.filter(
+      (rule) => !isPathScopedFileToolRule(rule)
+    );
+    writeJson(claudeSettings(), settings);
+  },
+};
+
+function startupFiles(): string[] {
+  const settings = readJson<Json>(palSettings(), {});
+  const files = (settings.loadAtStartup as { files?: unknown } | undefined)?.files;
+  return Array.isArray(files)
+    ? files.filter((f): f is string => typeof f === "string")
+    : [];
+}
+
+const isProjectsFile = (file: string) => file.endsWith("PROJECTS.md");
+
+const startupProjectsFile: Leftover = {
+  id: "startup-projects-md",
+  description:
+    "Stop loading PROJECTS.md at startup — projects moved to the project store",
+  find: () => startupFiles().filter(isProjectsFile),
+  remove() {
+    const settings = readJson<Json>(palSettings(), {});
+    const startup = settings.loadAtStartup as { files: string[] };
+    startup.files = startup.files.filter((file) => !isProjectsFile(file));
+    writeJson(palSettings(), settings);
+  },
+};
+
+const oldOpencodePlugin = () =>
+  resolve(platform.opencodeDir(), "plugins", "pai-plugin.ts");
+
+const opencodePluginOldName: Leftover = {
+  id: "opencode-pai-plugin",
+  description: "Remove the opencode plugin under its old name, pai-plugin.ts",
+  find: () => (existsSync(oldOpencodePlugin()) ? [oldOpencodePlugin()] : []),
+  remove: () => unlinkSync(oldOpencodePlugin()),
+};
+
+const oldCopilotInstructions = () =>
+  resolve(platform.copilotDir(), "copilot-instructions.md");
+
+function linksToAgentsMd(path: string): boolean {
+  try {
+    return lstatSync(path).isSymbolicLink() && readlinkSync(path).includes("AGENTS.md");
+  } catch {
+    return false;
+  }
+}
+
+const copilotInstructionsLink: Leftover = {
+  id: "copilot-instructions-link",
+  description:
+    "Remove the copilot-instructions.md link to AGENTS.md — Copilot now reads PAL's own instruction files",
+  find: () =>
+    linksToAgentsMd(oldCopilotInstructions()) ? [oldCopilotInstructions()] : [],
+  remove: () => unlinkSync(oldCopilotInstructions()),
+};
+
+const debugLog = () => resolve(paths.debug(), "debug.log");
+
+const debugLogPrev: Leftover = {
+  id: "debug-log-prev",
+  description: "Rename the old single rotated debug log, debug.log.prev, to debug.log.1",
+  find: () => (existsSync(`${debugLog()}.prev`) ? [`${debugLog()}.prev`] : []),
+  remove() {
+    const prev = `${debugLog()}.prev`;
+    if (existsSync(`${debugLog()}.1`)) unlinkSync(prev);
+    else renameSync(prev, `${debugLog()}.1`);
+  },
+};
+
+export const LEFTOVERS: Leftover[] = [
+  claudeFileToolRules,
+  startupProjectsFile,
+  opencodePluginOldName,
+  copilotInstructionsLink,
+  debugLogPrev,
+];

package/src/cli/migrate.ts

@@ -1,13 +1,13 @@
 /**
- * pal cli migrate — versioned, non-destructive data migrations.
+ * pal cli migrate — the only place PAL knows about its own past.
  *
  * Each Migration has:
- *   check() — returns whether this migration is needed (safe to call repeatedly)
- *   run()   — applies the migration; NEVER deletes source data
+ *   check() — reads what is on disk, so it is safe to call on every install
+ *   run()   — applies it; user data is moved, never deleted, while leftovers
+ *             PAL itself wrote and no longer reads are removed
  *
- * Add new migrations by appending to MIGRATIONS. Registry is ordered; migrations
- * run in declaration order. Doctor calls checkPendingMigrations() to surface
- * pending work without running anything.
+ * Retiring a setting, hook, file or name means adding a migration that removes
+ * its leftovers. Install runs pending migrations first; the doctor lists them.
  */
 
 import {
@@ -40,6 +40,7 @@ import {
   slugify,
 } from "../tools/knowledge/lib";
 import { readThreads, type Thread, writeThreads } from "../tools/lib/thread";
+import { LEFTOVERS, type Leftover } from "./leftovers";
 
 // ── Types ─────────────────────────────────────────────────────────
 
@@ -715,6 +716,29 @@ const v7RetiredSettingsKeys: Migration = {
   },
 };
 
+function leftoverMigration(leftover: Leftover): Migration {
+  return {
+    id: leftover.id,
+    description: leftover.description,
+    movesSources: true,
+    check() {
+      const found = leftover.find();
+      return { pending: found.length > 0, detail: found.join(", ") || undefined };
+    },
+    run(dryRun = false): MigrationResult {
+      const found = leftover.find();
+      if (found.length === 0) return { migrated: 0, skipped: 0, results: [] };
+      if (!dryRun) leftover.remove();
+      const verb = dryRun ? "would remove" : "removed";
+      return {
+        migrated: found.length,
+        skipped: 0,
+        results: [`${verb} ${found.join(", ")}`],
+      };
+    },
+  };
+}
+
 const MIGRATIONS: Migration[] = [
   v1Projects,
   v2ThreadsToIsc,
@@ -723,6 +747,7 @@ const MIGRATIONS: Migration[] = [
   v5AttributionKeys,
   v6HistorySlugs,
   v7RetiredSettingsKeys,
+  ...LEFTOVERS.map(leftoverMigration),
 ];
 
 // ── Public API ────────────────────────────────────────────────────
@@ -733,6 +758,13 @@ interface PendingMigration {
   detail?: string;
 }
 
+/** Install runs these first, so one install leaves nothing an older PAL wrote. */
+export function applyPendingMigrations(): string[] {
+  return MIGRATIONS.filter((m) => m.check().pending).flatMap((m) =>
+    m.run(false).results.map((line) => `${m.id}: ${line}`)
+  );
+}
+
 /** Returns migrations that have pending work. Used by doctor. */
 export function checkPendingMigrations(): PendingMigration[] {
   return MIGRATIONS.flatMap((m) => {

package/src/hooks/lib/log.ts

@@ -36,22 +36,10 @@ function timestamp(): string {
  * .MAX_ROTATED (oldest). Each rotation shifts .N-1 → .N, drops the oldest,
  * and renames the current log to .1. Total disk footprint bounded at
  * (MAX_ROTATED + 1) * MAX_LOG_SIZE ≈ 300KB.
- *
- * Migrates legacy .prev → .1 on first new rotation so existing history
- * survives the format change.
  */
 function rotateIfNeeded(path: string): void {
   try {
     if (!existsSync(path) || statSync(path).size <= MAX_LOG_SIZE) return;
-    // Backward-compat migration: legacy .prev was the single old rotation file.
-    const legacyPrev = `${path}.prev`;
-    if (existsSync(legacyPrev) && !existsSync(`${path}.1`)) {
-      try {
-        renameSync(legacyPrev, `${path}.1`);
-      } catch {
-        /* ignore */
-      }
-    }
     // Drop the oldest, then shift .N-1 → .N for N from MAX_ROTATED down to 2.
     const oldest = `${path}.${MAX_ROTATED}`;
     if (existsSync(oldest)) {
@@ -99,7 +87,7 @@ function logFilesOldestFirst(): string[] {
     { length: MAX_ROTATED },
     (_, i) => `${path}.${MAX_ROTATED - i}`
   );
-  return [...rotated, `${path}.prev`, path];
+  return [...rotated, path];
 }
 
 function readAllLogs(): string {

package/src/targets/copilot/uninstall.ts

@@ -4,7 +4,7 @@
  * and the VS Code chat.instructionsFilesLocations entry.
  */
 
-import { copyFileSync, existsSync, lstatSync, readlinkSync, unlinkSync } from "node:fs";
+import { copyFileSync, existsSync, unlinkSync } from "node:fs";
 import { resolve } from "node:path";
 import { platform } from "../../hooks/lib/paths";
 import {
@@ -63,22 +63,6 @@ log.success(
   `Removed ${removedInstructions.length} ~/.copilot/instructions/pal-*.instructions.md`
 );
 
-// --- Backward compat: remove old copilot-instructions.md symlink if present ---
-const legacyPath = resolve(COPILOT_DIR, "copilot-instructions.md");
-if (existsSync(legacyPath)) {
-  try {
-    if (
-      lstatSync(legacyPath).isSymbolicLink() &&
-      readlinkSync(legacyPath).includes("AGENTS.md")
-    ) {
-      unlinkSync(legacyPath);
-      log.success("Removed legacy copilot-instructions.md symlink");
-    }
-  } catch {
-    /* ignore */
-  }
-}
-
 // --- Remove ~/.copilot/instructions entry from VS Code settings ---
 const vsSettingsPath = vscodeSettingsFile();
 if (vsSettingsPath && existsSync(vsSettingsPath)) {

package/src/targets/lib.ts

@@ -211,7 +211,7 @@ export function mergeSettings(existing: Settings, template: Settings): Settings
     }
   }
 
-  // Merge permissions.allow (deduplicate), then drop deprecated entries.
+  // Merge permissions.allow (deduplicate)
   if (template.permissions?.allow) {
     result.permissions ??= {};
     result.permissions.allow ??= [];
@@ -221,15 +221,6 @@ export function mergeSettings(existing: Settings, template: Settings): Settings
       }
     }
   }
-  // Strip ineffective Grep(...)/Glob(...) rules left by older templates. Claude Code
-  // governs Grep/Glob via Read(...) rules, so a path-scoped Grep()/Glob() entry never
-  // matches and Claude Code warns about it on every prompt. Read(//*) already covers them.
-  if (result.permissions?.allow) {
-    result.permissions.allow = result.permissions.allow.filter(
-      (perm) => !isIneffectiveFileToolRule(perm)
-    );
-  }
-
   // Merge skillOverrides (object with skill name keys, add if not present)
   if (template.skillOverrides && typeof template.skillOverrides === "object") {
     result.skillOverrides ??= {};
@@ -488,15 +479,6 @@ function isOutdatedPalHook(cmd: string, palCanonical: Set<string>): boolean {
   return palCanonical.has(canonicalPalCmd(cmd)) || runsRemovedPalHook(cmd);
 }
 
-/**
- * True for path-scoped Grep()/Glob() allow rules, which Claude Code cannot honor —
- * it resolves Grep/Glob permission through Read(...) rules, so these never match and
- * trigger a warning on every prompt. Bare "Grep"/"Glob" tool allows are left intact.
- */
-function isIneffectiveFileToolRule(perm: string): boolean {
-  return /^(?:Grep|Glob)\(/.test(perm);
-}
-
 export function loadCodexHooksTemplate(
   templatePath: string,
   pkgRoot: string
@@ -697,21 +679,6 @@ export function scaffoldPalSettings(): void {
     copyFileSync(src, dst);
     log.info("Created pal-settings.json from template");
   }
-
-  // Strip deprecated loadAtStartup.files entries from existing installs.
-  // mergeSettings only adds, never removes — deprecated entries persist indefinitely otherwise.
-  try {
-    const raw = JSON.parse(readFileSync(dst, "utf-8"));
-    const files: string[] = raw?.loadAtStartup?.files ?? [];
-    const cleaned = files.filter((f: string) => !f.endsWith("PROJECTS.md"));
-    if (cleaned.length !== files.length) {
-      raw.loadAtStartup.files = cleaned;
-      writeFileSync(dst, `${JSON.stringify(raw, null, 2)}\n`, "utf-8");
-      log.info("Removed deprecated PROJECTS.md from loadAtStartup.files");
-    }
-  } catch {
-    /* non-fatal — malformed settings left as-is */
-  }
 }
 
 // --- PAL docs (modular context routing files) ---

package/src/targets/opencode/install.ts

@@ -3,14 +3,7 @@
  * Deploys plugin, installs skills, generates AGENTS.md.
  */
 
-import {
-  existsSync,
-  mkdirSync,
-  readFileSync,
-  statSync,
-  unlinkSync,
-  writeFileSync,
-} from "node:fs";
+import { existsSync, mkdirSync, readFileSync, statSync, writeFileSync } from "node:fs";
 import { resolve } from "node:path";
 import { palPkg, platform } from "../../hooks/lib/paths";
 import { getSemiStaticSources } from "../../hooks/lib/semi-static";
@@ -22,11 +15,7 @@ const OC_PLUGINS_DIR = resolve(OC_GLOBAL_DIR, "plugins");
 
 mkdirSync(OC_PLUGINS_DIR, { recursive: true });
 
-// --- 1. Deploy plugin (clean up legacy filename) ---
-const legacyPlugin = resolve(OC_PLUGINS_DIR, "pai-plugin.ts");
-if (existsSync(legacyPlugin)) {
-  unlinkSync(legacyPlugin);
-}
+// --- 1. Deploy plugin ---
 const pluginSrc = resolve(PKG_ROOT, "src", "targets", "opencode", "plugin.ts");
 const pluginDst = resolve(OC_PLUGINS_DIR, "pal-plugin.ts");
 // Embed PKG_ROOT as a hardcoded constant so no env config is needed