portable-agent-layer 0.63.0 → 0.63.2

package/src/targets/cursor/uninstall.ts

@@ -4,15 +4,15 @@
  * Removes PAL skill symlinks.
  */
 
-import { copyFileSync, existsSync, unlinkSync } from "node:fs";
+import { copyFileSync, existsSync } from "node:fs";
 import { resolve } from "node:path";
 import { assets, palPkg, platform } from "../../hooks/lib/paths";
-import { cursorFilename, getSemiStaticSources } from "../../hooks/lib/semi-static";
 import {
   loadCursorHooksTemplate,
   log,
   readJson,
   removeAgentsFromCursor,
+  removePalContextFiles,
   removePalDocs,
   removeSkills,
   removeStatusline,
@@ -73,19 +73,7 @@ if (existsSync(CLI_CONFIG)) {
 removeStatusline("cursor");
 
 // --- Remove ~/.cursor/rules/pal-*.mdc ---
-for (const src of getSemiStaticSources()) {
-  try {
-    unlinkSync(resolve(CURSOR_DIR, "rules", cursorFilename(src)));
-  } catch {
-    /* gone */
-  }
-}
-// Backward compat: remove legacy merged file if present
-try {
-  unlinkSync(resolve(CURSOR_DIR, "rules", "pal-context.mdc"));
-} catch {
-  /* gone */
-}
-log.success("Removed ~/.cursor/rules/pal-*.mdc");
+const removedRules = removePalContextFiles(resolve(CURSOR_DIR, "rules"), ".mdc");
+log.success(`Removed ${removedRules.length} ~/.cursor/rules/pal-*.mdc`);
 
 log.success("Cursor uninstall complete");

package/src/targets/lib.ts

@@ -741,6 +741,23 @@ export function removePalDocs(): void {
 
 const PAL_SKILLS_DIR = resolve(palHome(), "skills");
 
+/**
+ * Run one step of a bulk install, naming it only when it fails.
+ *
+ * Installs handle dozens of skills and agents; a line each buries the paths,
+ * backups and warnings that a reader actually has to act on. Returning false
+ * instead of throwing also keeps one unlinkable skill from aborting the rest.
+ */
+function reportOnlyOnFailure(label: string, install: () => void): boolean {
+  try {
+    install();
+    return true;
+  } catch (e) {
+    log.warn(`Could not install ${label} — ${(e as Error).message}`);
+    return false;
+  }
+}
+
 /**
  * Install PAL skills by symlinking:
  *   ~/.pal/skills/<name> → <repo>/assets/skills/<name>  (source of truth)
@@ -762,16 +779,15 @@ export function copySkills(claudeSkillsDir: string): number {
     const srcDir = resolve(skillsDir, name);
     if (!existsSync(resolve(srcDir, "SKILL.md"))) continue;
 
-    // ~/.pal/skills/<name> → <repo>/assets/skills/<name>
     const palLink = resolve(PAL_SKILLS_DIR, name);
-    ensureSymlink(palLink, srcDir, linkType);
-
-    // ~/.claude/skills/<name> → ~/.pal/skills/<name>
     const claudeLink = resolve(claudeSkillsDir, name);
-    ensureSymlink(claudeLink, palLink, linkType);
-
-    log.info(`Linked skill: ${name}`);
-    count++;
+    const linked = reportOnlyOnFailure(`skill ${name}`, () => {
+      // ~/.pal/skills/<name> → <repo>/assets/skills/<name>
+      ensureSymlink(palLink, srcDir, linkType);
+      // ~/.claude/skills/<name> → ~/.pal/skills/<name>
+      ensureSymlink(claudeLink, palLink, linkType);
+    });
+    if (linked) count++;
   }
 
   // ~/.agents/skills/ → ~/.pal/skills/
@@ -838,6 +854,29 @@ function ensureSymlink(link: string, target: string, type: "dir" | "junction"):
   symlinkSync(target, link, type);
 }
 
+/**
+ * Remove every `pal-*` context file with the given suffix from a directory.
+ *
+ * Globs rather than iterating getSemiStaticSources(): a slug retired from that
+ * registry keeps its already-written file on disk, and a registry-driven delete
+ * can no longer name it — agents then keep loading retired context forever.
+ * Also catches the legacy pre-split filenames without needing a special case.
+ */
+export function removePalContextFiles(dir: string, suffix: string): string[] {
+  if (!existsSync(dir)) return [];
+  const removed: string[] = [];
+  for (const file of readdirSync(dir)) {
+    if (!file.startsWith("pal-") || !file.endsWith(suffix)) continue;
+    try {
+      unlinkSync(resolve(dir, file));
+      removed.push(file);
+    } catch {
+      /* gone or not ours to remove */
+    }
+  }
+  return removed;
+}
+
 /** Remove PAL skill symlinks from ~/.pal/skills/ and ~/.claude/skills/ */
 export function removeSkills(claudeSkillsDir: string): string[] {
   const skillsDir = assets.skills();
@@ -977,14 +1016,16 @@ function installAgents(targetDir: string, platform: AgentPlatform): number {
   let count = 0;
 
   for (const file of readdirSync(agentsDir).filter((f) => f.endsWith(".md"))) {
-    const content = readFileSync(resolve(agentsDir, file), "utf-8");
-    writeFileSync(
-      resolve(targetDir, file),
-      extractAgentForPlatform(content, platform),
-      "utf-8"
-    );
-    log.info(`Installed ${platform} agent: ${file.replace(/\.md$/, "")}`);
-    count++;
+    const name = file.replace(/\.md$/, "");
+    const installed = reportOnlyOnFailure(`${platform} agent ${name}`, () => {
+      const content = readFileSync(resolve(agentsDir, file), "utf-8");
+      writeFileSync(
+        resolve(targetDir, file),
+        extractAgentForPlatform(content, platform),
+        "utf-8"
+      );
+    });
+    if (installed) count++;
   }
   return count;
 }
@@ -1352,7 +1393,6 @@ export function generateSkillIndex(): number {
   const stateDir = resolve(palHome(), "memory", "state");
   mkdirSync(stateDir, { recursive: true });
   writeJson(resolve(stateDir, "skill-index.json"), index);
-  log.info(`Skill index: ${index.totalSkills} skills indexed`);
 
   return index.totalSkills;
 }

package/src/targets/opencode/install.ts

@@ -12,18 +12,9 @@ import {
   writeFileSync,
 } from "node:fs";
 import { resolve } from "node:path";
-import { regenerateIfNeeded } from "../../hooks/lib/claude-md";
 import { palPkg, platform } from "../../hooks/lib/paths";
 import { getSemiStaticSources } from "../../hooks/lib/semi-static";
-import {
-  copyAgentsForOpencode,
-  copyPalDocs,
-  copySkills,
-  countSkills,
-  generateSkillIndex,
-  log,
-  writeJson,
-} from "../lib";
+import { copyAgentsForOpencode, copySkills, countSkills, log, writeJson } from "../lib";
 
 const PKG_ROOT = palPkg();
 const OC_GLOBAL_DIR = platform.opencodeDir();
@@ -55,7 +46,6 @@ if (!existsSync(pkgPath)) {
 
 try {
   Bun.spawnSync(["bun", "install", "--silent"], { cwd: OC_PLUGINS_DIR });
-  log.success("Installed plugin dependencies");
 } catch {
   log.warn(`Could not install plugin deps — run 'bun install' in ${OC_PLUGINS_DIR}`);
 }
@@ -63,22 +53,13 @@ try {
 // --- 3. Install skills into ~/.pal/skills/ ---
 const claudeSkillsDir = resolve(platform.claudeDir(), "skills");
 copySkills(claudeSkillsDir);
-generateSkillIndex();
-log.success("Installed skills to ~/.pal/skills/");
 
 // --- 4. Install agents into ~/.config/opencode/agents/ ---
 const ocAgentsDir = resolve(OC_GLOBAL_DIR, "agents");
-copyAgentsForOpencode(ocAgentsDir);
-
-// --- 5. Copy PAL system docs ---
-const palDocsCount = copyPalDocs();
-log.success(`Installed ${palDocsCount} PAL docs to ~/.pal/docs/`);
-
-// --- 6. Generate ~/.config/opencode/AGENTS.md ---
-regenerateIfNeeded();
-log.success("Generated ~/.config/opencode/AGENTS.md");
+const agentCount = copyAgentsForOpencode(ocAgentsDir);
+log.success(`${countSkills()} skills · ${agentCount} agents → ~/.config/opencode/`);
 
-// --- 7. Add semi-static digest files to instructions[] in config.json ---
+// --- 5. Add semi-static digest files to instructions[] in config.json ---
 const configPath = resolve(OC_GLOBAL_DIR, "config.json");
 const staticFiles = getSemiStaticSources().map((s) => s.path);
 let ocConfig: Record<string, unknown> = {};
@@ -94,11 +75,3 @@ const existingInstructions = Array.isArray(ocConfig.instructions)
   : [];
 ocConfig.instructions = [...new Set([...existingInstructions, ...staticFiles])];
 writeFileSync(configPath, `${JSON.stringify(ocConfig, null, 2)}\n`, "utf-8");
-log.success(
-  `Updated config.json: ${(ocConfig.instructions as string[]).length} instructions`
-);
-
-log.success("opencode installation complete");
-console.log("");
-log.info(`Plugin: ${pluginDst}`);
-log.info(`Skills: ${countSkills()} (native via ~/.pal/skills/)`);