portable-agent-layer 0.91.2 → 0.91.4

package/package.json

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

package/src/hooks/LoadContext.ts

@@ -57,10 +57,10 @@ try {
     process.exit(0);
   }
 
-  // The reminder is built for one of three targets; every other agent reads the
+  // The reminder is built for one of four targets; every other agent reads the
   // same shape Claude Code does.
   const target: AgentTarget =
-    active === "copilot" || active === "cursor" ? active : "claude";
+    active === "copilot" || active === "cursor" || active === "codex" ? active : "claude";
   const reminder = buildSystemReminder({
     agent: target,
     withoutHandoff: wanted === "without-handoff",

package/src/hooks/lib/context.ts

@@ -13,7 +13,7 @@ import { loadOpinionContext } from "./opinions";
 import { paths, toPath } from "./paths";
 import { loadActiveProjectsContext } from "./projects";
 import { loadRecentNotes } from "./relationship";
-import { loadFailurePatterns } from "./semi-static";
+import { loadFailurePatterns, loadSteeringRules } from "./semi-static";
 import { sessionDir } from "./session-dir";
 import * as settings from "./settings";
 import { readFramePrinciples } from "./wisdom";
@@ -173,10 +173,13 @@ function capSection(text: string, maxChars: number): string {
   return kept.join("\n");
 }
 
-/** Agent targets — determines which context sections are skipped due to native loading. */
+/**
+ * Agent targets — determines which context sections are skipped due to native loading.
+ * Codex has no native include for them, so its reminder carries them.
+ */
 export type AgentTarget = Extract<
   AgentName,
-  "claude" | "opencode" | "cursor" | "copilot"
+  "claude" | "opencode" | "cursor" | "copilot" | "codex"
 >;
 
 /**
@@ -215,6 +218,7 @@ export function buildSystemReminder(
     !skipSemiStatic && settings.isEnabled("opinions") ? loadOpinionContext() : "";
   const selfModel =
     !skipSemiStatic && settings.isEnabled("selfModel") ? loadSelfModel() : "";
+  const steering = skipSemiStatic ? "" : loadSteeringRules();
   const handoff =
     !opts.withoutHandoff && settings.isEnabled("handoff")
       ? loadHandoffContext(sessionDir())
@@ -230,6 +234,7 @@ export function buildSystemReminder(
   if (relationship) parts.push(relationship);
   if (activeProjects) parts.push(activeProjects);
   if (failures) parts.push(failures);
+  if (steering) parts.push(steering);
   if (parts.length === 0) return "";
 
   const now = new Date();

package/src/hooks/lib/semi-static.ts

@@ -104,13 +104,20 @@ export function loadFailurePatterns(): string {
   }
 }
 
+function steeringRulesPath(): string {
+  return resolve(palHome(), "docs", "STEERING_RULES.md");
+}
+
+export function loadSteeringRules(): string {
+  return readFileSafe(steeringRulesPath());
+}
+
 /**
  * All semi-static context sources in load order.
  * Adding one entry here is the only change needed to extend coverage to all consumers.
  */
 export function getSemiStaticSources(): SemiStaticSource[] {
   const memory = paths.memory();
-  const home = palHome();
   return [
     {
       path: resolve(memory, "self-model", "current.md"),
@@ -157,9 +164,9 @@ export function getSemiStaticSources(): SemiStaticSource[] {
       description: "PAL recent failure lessons",
     },
     {
-      path: resolve(home, "docs", "STEERING_RULES.md"),
+      path: steeringRulesPath(),
       writesDigest: false,
-      load: () => readFileSafe(resolve(home, "docs", "STEERING_RULES.md")),
+      load: loadSteeringRules,
       slug: "steering",
       description: "PAL steering rules",
     },

package/src/targets/lib.ts

@@ -989,8 +989,11 @@ export function removePalContextFiles(dir: string, suffix: string): string[] {
   return removed;
 }
 
-/** Remove PAL skill symlinks from ~/.pal/skills/ and ~/.claude/skills/ */
-export function removeSkills(claudeSkillsDir: string): string[] {
+/**
+ * Remove one agent's PAL skill links. ~/.pal/skills/<name> and ~/.agents/skills
+ * are shared by every installed agent, so a per-agent uninstall leaves them.
+ */
+export function removeSkills(agentSkillsDir: string): string[] {
   const skillsDir = assets.skills();
   if (!existsSync(skillsDir)) return [];
 
@@ -998,24 +1001,15 @@ export function removeSkills(claudeSkillsDir: string): string[] {
   for (const name of readdirSync(skillsDir)) {
     if (!existsSync(resolve(skillsDir, name, "SKILL.md"))) continue;
 
-    for (const link of [resolve(palSkillsDir(), name), resolve(claudeSkillsDir, name)]) {
-      try {
-        unlinkSync(link);
-      } catch {
-        /* already gone */
-      }
+    try {
+      unlinkSync(resolve(agentSkillsDir, name));
+    } catch {
+      /* already gone */
     }
     removed.push(name);
     log.detail(`Removed skill: ${name}`);
   }
 
-  // Remove ~/.agents/skills/ → ~/.pal/skills/ symlink
-  try {
-    unlinkSync(resolve(platform.agentsDir(), "skills"));
-  } catch {
-    /* gone */
-  }
-
   return removed;
 }
 

package/src/targets/opencode/uninstall.ts

@@ -7,7 +7,7 @@ import { existsSync, readFileSync, statSync, unlinkSync, writeFileSync } from "n
 import { resolve } from "node:path";
 import { platform } from "../../hooks/lib/paths";
 import { getSemiStaticSources } from "../../hooks/lib/semi-static";
-import { log, removeAgentsFromOpencode, removePalDocs, removeSkills } from "../lib";
+import { log, removeAgentsFromOpencode, removePalDocs } from "../lib";
 import { palHomePattern, removePalHomeAllow } from "./permission";
 
 const OC_GLOBAL_DIR = platform.opencodeDir() || "";
@@ -28,11 +28,6 @@ try {
   log.info("No PAL plugin found");
 }
 
-// --- Remove skills ---
-const removed = removeSkills(resolve(PAL_CLAUDE_DIR, "skills"));
-if (removed.length > 0)
-  log.success(`Removed ${removed.length} skill(s): ${removed.join(", ")}`);
-
 // --- Remove agents ---
 const removedAgents = removeAgentsFromOpencode(resolve(OC_GLOBAL_DIR, "agents"));
 if (removedAgents.length > 0)