portable-agent-layer 0.85.14 → 0.85.16

package/package.json

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

package/src/cli/doctor/environment.ts

@@ -1,3 +1,4 @@
+import { spawnSync } from "node:child_process";
 import { existsSync, readdirSync, readFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, resolve } from "node:path";
@@ -67,22 +68,47 @@ function playwrightPlatformOverride(host: BrowserHost): string {
   return `PLAYWRIGHT_HOST_PLATFORM_OVERRIDE=ubuntu24.04-${host.arch === "arm64" ? "arm64" : "x64"} `;
 }
 
+function chromiumInstallCommand(host: BrowserHost): string {
+  const pinned = host.playwright ? `playwright@${host.playwright.version}` : "playwright";
+  return `${playwrightPlatformOverride(host)}bun x ${pinned} install chromium`;
+}
+
 export function playwrightFinding(host: BrowserHost): Finding {
   if (hasChromium(host)) return passed("playwright", "Playwright Chromium installed");
-  const pinned = host.playwright ? `playwright@${host.playwright.version}` : "playwright";
-  const override = playwrightPlatformOverride(host);
   return warning(
     "playwright.missing",
     "Playwright's Chromium is not installed — create-pdf, consulting-report and screenshots fail",
     {
-      say: override
+      say: playwrightPlatformOverride(host)
         ? "Install it as Ubuntu 24.04, which Playwright supports"
         : "Install it",
-      command: `${override}bun x ${pinned} install chromium`,
+      command: chromiumInstallCommand(host),
     }
   );
 }
 
+function thisBrowserHost(): BrowserHost {
+  return {
+    browsersPath: playwrightBrowsersPath(),
+    playwright: installedPlaywright(),
+    platform: process.platform,
+    arch: process.arch,
+    osRelease: readOsRelease(),
+  };
+}
+
+function runInShell(command: string): boolean {
+  return spawnSync(command, { cwd: palPkg(), shell: true, stdio: "ignore" }).status === 0;
+}
+
+/** A failure is left to the doctor's report that follows, which names the fix. */
+export function installChromium(
+  host: BrowserHost = thisBrowserHost(),
+  run: (command: string) => boolean = runInShell
+): void {
+  if (!hasChromium(host)) run(chromiumInstallCommand(host));
+}
+
 interface PalInstall {
   pal: string | null;
   pkg: string;
@@ -136,13 +162,7 @@ export function environmentFindings(rtk: ToolCheck): Finding[] {
       pkg: palPkg(),
       repoMode: isRepoMode(),
     }),
-    playwrightFinding({
-      browsersPath: playwrightBrowsersPath(),
-      playwright: installedPlaywright(),
-      platform: process.platform,
-      arch: process.arch,
-      osRelease: readOsRelease(),
-    }),
+    playwrightFinding(thisBrowserHost()),
     rtkFinding(rtk),
     ...versionControlFindings(),
   ];

package/src/cli/index.ts

@@ -50,10 +50,10 @@ import { inference, previewInferenceRoute } from "../hooks/lib/inference";
 import { logDebug } from "../hooks/lib/log";
 import { ensureRegistered, writeRegistryEntry } from "../hooks/lib/machine";
 import { palHome, palPkg, paths, platform, toPath } from "../hooks/lib/paths";
-import { telosStatus } from "../hooks/lib/telos-topics";
-import { log } from "../targets/lib";
+import { log, narrateSteps } from "../targets/lib";
 import { builtinToolVerbs, runBuiltinTool } from "./builtin-tools";
 import { type DoctorResult, detectAgents } from "./doctor/agents";
+import { installChromium } from "./doctor/environment";
 import { runDoctor } from "./doctor/run";
 import { findSessionAgent, NO_SESSION_AGENT_MESSAGE } from "./session-agent";
 
@@ -155,7 +155,7 @@ async function runCli(command: string | undefined, args: string[]) {
       break;
     case "install":
       banner();
-      await install(resolveTargets(args));
+      process.exit(await install(resolveTargets(args), args));
       break;
     case "uninstall":
       await uninstall(args);
@@ -265,24 +265,10 @@ function banner() {
   console.log("");
   console.log("  ╔═══════════════════════════════════╗");
   console.log("  ║  PAL — Portable Agent Layer       ║");
-  console.log("  ║  Non-destructive · Modular        ║");
   console.log("  ╚═══════════════════════════════════╝");
   console.log("");
 }
 
-/**
- * Install asks for identity and stops. TELOS is a conversation, not a form: a
- * user who has just installed a tool cannot yet say what they want from it, and
- * answers given under that pressure are worse than none.
- */
-function pointAtOnboarding(): void {
-  const unanswered = telosStatus().filter((topic) => topic.priority && !topic.answered);
-  if (unanswered.length === 0) return;
-  log.info(
-    `PAL knows nothing about you yet (${unanswered.length} topics open). When you have half an hour, ask your agent to onboard you.`
-  );
-}
-
 function showHelp() {
   console.log(`
   Usage:
@@ -290,8 +276,9 @@ function showHelp() {
     pal cli <command> [options]             Admin commands
 
   Admin commands:
-    pal cli init [--claude] [--opencode] [--cursor] [--codex]    Scaffold and install (default: all)
-    pal cli install [--claude] [--opencode] [--cursor] [--codex] Register hooks for targets
+    pal cli init [--claude] [--opencode] [--cursor] [--codex]    Scaffold and install (default: all), then run the doctor
+    pal cli install [--claude] [--opencode] [--cursor] [--codex] Register hooks for targets, then run the doctor
+                                            Add --verbose to either for the step-by-step log
     pal cli uninstall [--claude] [--opencode] [--cursor] [--codex] Remove hooks for targets
     pal cli update                          Update PAL (git pull or npm update)
     pal cli export [path] [--dry-run]       Export state to zip
@@ -486,12 +473,10 @@ async function init(args: string[]) {
   const { scaffoldTelos } = await import("../targets/lib");
 
   banner();
+  narrateSteps(args.includes("--verbose"));
 
   const health = detectAgents();
-  runDoctor([], health);
-  if (!health.hasAgent) {
-    process.exit(1);
-  }
+  if (!health.hasAgent) process.exit(runDoctor([], health));
 
   const home = palHome();
   log.info(`Creating PAL home at ${home}`);
@@ -506,7 +491,7 @@ async function init(args: string[]) {
 
   // Auto-detect available targets
   const targets = resolveTargets(args, health);
-  await install(targets);
+  process.exit(await install(targets, args));
 }
 
 /**
@@ -522,25 +507,28 @@ function runQuietly(cmd: string, args: string[], cwd: string): number | null {
   return r.status;
 }
 
-async function install(targets: Targets) {
-  // Ensure dependencies are installed
+function targetInstallers(): [keyof Targets, string, () => Promise<unknown>][] {
+  return [
+    ["claude", "Claude Code", () => import("../targets/claude/install")],
+    ["opencode", "opencode", () => import("../targets/opencode/install")],
+    ["cursor", "Cursor", () => import("../targets/cursor/install")],
+    ["copilot", "Copilot", () => import("../targets/copilot/install")],
+    ["codex", "Codex", () => import("../targets/codex/install")],
+  ];
+}
+
+async function install(targets: Targets, args: string[]): Promise<number> {
+  narrateSteps(args.includes("--verbose"));
   const pkg = palPkg();
   if (runQuietly("bun", ["install", "--frozen-lockfile"], pkg) !== 0) {
     log.warn("bun install failed — continuing anyway, but hooks may not work");
   }
 
-  // Fetch the Chromium build Playwright uses for PDF rendering (create-pdf skill).
-  // Idempotent — skipped if already cached. Skipped entirely under PAL_SKIP_BROWSER_INSTALL=1
-  // (used by tests to avoid a ~150MB download on every run).
+  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") {
-    const pw = runQuietly("bun", ["x", "playwright", "install", "chromium"], pkg);
-    if (pw !== 0) {
-      log.warn(
-        `playwright install chromium failed (exit ${pw}) — create-pdf and consulting-report skills won't work. Retry manually: bun x playwright install chromium`
-      );
-    }
-  }
+  if (process.env.PAL_SKIP_BROWSER_INSTALL !== "1") installChromium();
 
   // Scaffold TELOS + PAL settings, then prompt for missing identity
   const { scaffoldTelos, scaffoldPalSettings, copyPalDocs, generateSkillIndex } =
@@ -553,7 +541,6 @@ async function install(targets: Targets) {
   await promptIdentity();
   await promptAttribution();
   await promptAutoUpdate();
-  pointAtOnboarding();
 
   // Registers the label loadActor derives, so it travels on the next export.
   const { ensureActorRegistered } = await import("../hooks/lib/actor");
@@ -566,34 +553,10 @@ async function install(targets: Targets) {
   const palDocsCount = copyPalDocs();
   regenerateIfNeeded();
 
-  if (targets.claude) {
-    console.log("━━━ Claude Code ━━━");
-    await import("../targets/claude/install");
-    console.log("");
-  }
-
-  if (targets.opencode) {
-    console.log("━━━ opencode ━━━");
-    await import("../targets/opencode/install");
-    console.log("");
-  }
-
-  if (targets.cursor) {
-    console.log("━━━ Cursor ━━━");
-    await import("../targets/cursor/install");
-    console.log("");
-  }
-
-  if (targets.copilot) {
-    console.log("━━━ Copilot ━━━");
-    await import("../targets/copilot/install");
-    console.log("");
-  }
-
-  if (targets.codex) {
-    console.log("━━━ Codex ━━━");
-    await import("../targets/codex/install");
-    console.log("");
+  for (const [target, label, installTarget] of targetInstallers()) {
+    if (!targets[target]) continue;
+    log.heading(label);
+    await installTarget();
   }
 
   // The rest of the shared work reads what the installers just wrote: the index
@@ -608,7 +571,8 @@ async function install(targets: Targets) {
 
   await refreshControlRoom();
 
-  log.success("Done. Existing config was preserved — only new entries were added.");
+  if (args.includes("--verbose")) console.log("");
+  return runDoctor(args);
 }
 
 /**
@@ -912,7 +876,7 @@ async function update() {
   clearUpdateCache();
 
   log.info("Reinstalling...");
-  await install(resolveTargets([]));
+  process.exit(await install(resolveTargets([]), []));
 }
 
 function cliDebug(args: string[]) {

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

@@ -27,9 +27,21 @@ function runningUnderTest(): boolean {
   return process.env.PAL_TEST_SANDBOX === "1";
 }
 
+let narrating = true;
+
+/** Install's step log is for --verbose; the doctor report that follows carries the result. */
+export function narrateSteps(on: boolean): void {
+  narrating = on;
+}
+
+function narrate(line: string): void {
+  if (narrating) console.log(line);
+}
+
 export const log = {
-  info: (msg: string) => console.log(`\x1b[34m[pal]\x1b[0m ${msg}`),
-  success: (msg: string) => console.log(`\x1b[32m[pal]\x1b[0m ${msg}`),
+  info: (msg: string) => narrate(`\x1b[34m[pal]\x1b[0m ${msg}`),
+  success: (msg: string) => narrate(`\x1b[32m[pal]\x1b[0m ${msg}`),
+  heading: (label: string) => narrate(`\n━━━ ${label} ━━━`),
   warn: (msg: string) => console.log(`\x1b[33m[pal]\x1b[0m ${msg}`),
   error: (msg: string) => console.error(`\x1b[31m[pal]\x1b[0m ${msg}`),
 
@@ -42,7 +54,7 @@ export const log = {
    */
   detail: (msg: string) => {
     if (runningUnderTest()) return;
-    console.log(`\x1b[34m[pal]\x1b[0m ${msg}`);
+    narrate(`\x1b[34m[pal]\x1b[0m ${msg}`);
   },
 };
 
@@ -199,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 ??= [];
@@ -209,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 ??= {};
@@ -476,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
@@ -685,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