portable-agent-layer 0.85.16 → 0.85.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "portable-agent-layer",
-  "version": "0.85.16",
+  "version": "0.85.18",
   "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,7 +520,10 @@ 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");
   }