@agentprojectcontext/apx 1.15.2 → 1.15.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentprojectcontext/apx",
-  "version": "1.15.2",
+  "version": "1.15.3",
   "description": "APX — unified CLI + daemon for the Agent Project Context (APC) standard.",
   "publishConfig": {
     "access": "public"

package/src/cli/commands/update.js

@@ -4,6 +4,29 @@ import { getLatestVersion } from "../../core/update-check.js";
 
 const PACKAGE_NAME = "@agentprojectcontext/apx";
 
+function isNewer(cur, lat) {
+  const parse = (v) => v.replace(/^v/, "").split(".").map(Number);
+  const [ma, mi, pa] = parse(cur);
+  const [mb, mib, pb] = parse(lat);
+  if (mb > ma) return true;
+  if (mb === ma && mib > mi) return true;
+  if (mb === ma && mib === mi && pb > pa) return true;
+  return false;
+}
+
+function hasPnpmGlobal() {
+  const r = spawnSync("pnpm", ["--version"], { encoding: "utf8", stdio: "pipe" });
+  if (r.status !== 0) return false;
+  // pnpm needs PNPM_HOME configured to manage global packages
+  const check = spawnSync("pnpm", ["root", "-g"], { encoding: "utf8", stdio: "pipe" });
+  return check.status === 0 && !!check.stdout?.trim();
+}
+
+function daemonRunning() {
+  const r = spawnSync("apx", ["daemon", "status", "--json"], { encoding: "utf8", stdio: "pipe" });
+  try { return JSON.parse(r.stdout)?.running === true; } catch { return false; }
+}
+
 export async function cmdUpdate(args, currentVersion) {
   const force = args.flags.force || args.flags.yes || args.flags.y;
 
@@ -15,24 +38,12 @@ export async function cmdUpdate(args, currentVersion) {
     process.exit(1);
   }
 
-  const current = currentVersion;
-
-  function isNewer(cur, lat) {
-    const parse = (v) => v.replace(/^v/, "").split(".").map(Number);
-    const [ma, mi, pa] = parse(cur);
-    const [mb, mib, pb] = parse(lat);
-    if (mb > ma) return true;
-    if (mb === ma && mib > mi) return true;
-    if (mb === ma && mib === mi && pb > pa) return true;
-    return false;
-  }
-
-  if (!isNewer(current, latest)) {
-    console.log(`✅ Already up to date (${current})`);
+  if (!isNewer(currentVersion, latest)) {
+    console.log(`✅ Already up to date (${currentVersion})`);
     return;
   }
 
-  console.log(`\n  Current: ${current}`);
+  console.log(`\n  Current: ${currentVersion}`);
   console.log(`  Latest:  ${latest}`);
 
   if (!force) {
@@ -43,20 +54,41 @@ export async function cmdUpdate(args, currentVersion) {
     }
   }
 
-  console.log(`\nRunning: npm install -g ${PACKAGE_NAME}@${latest}\n`);
-  const result = spawnSync(
-    "npm",
-    ["install", "-g", `${PACKAGE_NAME}@${latest}`],
-    { stdio: "inherit" }
-  );
+  // Stop daemon before replacing the binary so Node doesn't lock files on Windows.
+  const wasDaemonRunning = daemonRunning();
+  if (wasDaemonRunning) {
+    process.stdout.write("\nStopping daemon... ");
+    spawnSync("apx", ["daemon", "stop"], { stdio: "inherit" });
+    console.log("stopped.");
+  }
+
+  // Prefer pnpm global if configured, fall back to npm.
+  const usePnpm = hasPnpmGlobal();
+  const pm = usePnpm ? "pnpm" : "npm";
+  const installArgs = usePnpm
+    ? ["add", "-g", `${PACKAGE_NAME}@${latest}`]
+    : ["install", "-g", `${PACKAGE_NAME}@${latest}`];
+
+  console.log(`\nInstalling ${PACKAGE_NAME}@${latest} via ${pm}...\n`);
+  const result = spawnSync(pm, installArgs, { stdio: "inherit" });
 
   if (result.status !== 0) {
     console.error(`\n❌ Update failed (exit ${result.status})`);
+    if (wasDaemonRunning) {
+      console.log("Restarting daemon with old version...");
+      spawnSync("apx", ["daemon", "start"], { stdio: "inherit" });
+    }
     process.exit(result.status || 1);
   }
 
-  console.log(`\n✅ Updated to ${latest}. Restart any running apx daemon:`);
-  console.log(`   apx daemon stop && apx daemon start`);
+  // Restart daemon with new version.
+  if (wasDaemonRunning) {
+    process.stdout.write("\nStarting daemon... ");
+    spawnSync("apx", ["daemon", "start"], { stdio: "inherit" });
+    console.log("done.");
+  }
+
+  console.log(`\n✅ Updated to ${latest}.`);
 }
 
 function confirm(prompt) {

package/src/core/scaffold.js

@@ -176,7 +176,12 @@ export function installIdeSkills(root, targetIds = null) {
   return results;
 }
 
-// Install bundled APX/APC skills + runtime docs to global ~/.../skills/ dirs.
+// Install bundled APX/APC skills to global ~/.../skills/ dirs.
+// Only apx and apc-context are installed everywhere — they teach IDE tools
+// (Claude Code, Cursor, Codex) about the APX CLI and APC project standard.
+// Runtime CLI skills (claude-code, codex-cli, etc.) are APX-internal; APX
+// loads them from src/daemon/runtime-skills/ at startup and does NOT push
+// them to other tools' global skill dirs.
 // Returns an array of result objects with { dir, skill, status }.
 export function installGlobalSkills() {
   const results = [];
@@ -186,7 +191,8 @@ export function installGlobalSkills() {
   const apcRaw = readBundledSkill("apc-context");
   if (apxRaw) skills.push({ slug: "apx", md: apxRaw });
   if (apcRaw) skills.push({ slug: "apc-context", md: apcRaw });
-  skills.push(...readRuntimeSkillFiles());
+  // Runtime skills (claude-code, codex-cli, opencode-cli, openrouter, …) are
+  // NOT included here — they are APX-only internals, not for IDE consumption.
 
   for (const base of GLOBAL_SKILL_DIRS) {
     for (const { slug, md } of skills) {