@bigking67/pi-67 0.10.0 → 0.10.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [0.10.1]
+
+- Falls back from `npm` to `npm.cmd` on Windows when checking npm registry state
+  or running `pi-67 self-update`, avoiding `spawnSync npm ENOENT` in PowerShell
+  environments where only the command shim is directly spawnable.
+- Clarifies that global install is the normal daily path and `npx @latest` is
+  the always-fresh one-shot validation or recovery path.
+
 ## [0.10.0]
 
 - Initial npm manager CLI for pi-67.

package/README.md

@@ -55,6 +55,10 @@ If the local manager may be stale, run the latest npm package for one repair:
 npx -y @bigking67/pi-67@latest update --repair
 ```
 
+Use `npm install -g @bigking67/pi-67` for normal daily operation. Use `npx -y
+@bigking67/pi-67@latest ...` when you want a zero-install, always-fresh one-shot
+check or repair before trusting the globally installed manager.
+
 ## Safety defaults
 
 `pi-67 update` preserves local runtime choices:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bigking67/pi-67",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "description": "Cross-platform manager CLI for the pi-67 Pi Coding Agent distribution.",
   "type": "module",
   "bin": {

package/scripts/check.mjs

@@ -4,6 +4,7 @@ import path from "node:path";
 import { spawnSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import { npmPublishTargetStatus } from "../src/lib/npm-registry.mjs";
+import { commandCandidatesForPlatform } from "../src/lib/shell-runner.mjs";
 import { readExtensionRegistry, validateExtensionRegistry } from "../src/lib/extension-registry.mjs";
 import { buildPlanDecisions, classifyGitShort } from "../src/lib/update-plan.mjs";
 import {
@@ -26,6 +27,7 @@ for (const file of files.filter((item) => item.endsWith(".json"))) {
   JSON.parse(fs.readFileSync(file, "utf8"));
 }
 runPublishTargetSelfTests();
+runShellRunnerSelfTests();
 runExtensionRegistrySelfTests();
 runUpdatePlanSelfTests();
 runUpdateSafetySelfTests();
@@ -92,6 +94,21 @@ function runPublishTargetSelfTests() {
   );
 }
 
+function runShellRunnerSelfTests() {
+  assert(
+    JSON.stringify(commandCandidatesForPlatform("npm", "win32")) === JSON.stringify(["npm", "npm.cmd"]),
+    "Windows npm execution must fall back to npm.cmd",
+  );
+  assert(
+    JSON.stringify(commandCandidatesForPlatform("git", "win32")) === JSON.stringify(["git"]),
+    "Windows non-npm execution should keep the requested command",
+  );
+  assert(
+    JSON.stringify(commandCandidatesForPlatform("npm", "darwin")) === JSON.stringify(["npm"]),
+    "POSIX npm execution should keep the requested command",
+  );
+}
+
 function runExtensionRegistrySelfTests() {
   const registry = readExtensionRegistry(path.join(root, "src", "data", "extension-registry.json"));
   const manifest = extensionRegistryTestManifest();

package/src/lib/shell-runner.mjs

@@ -7,7 +7,7 @@ export function runCommand(command, args = [], options = {}) {
     info(`DRY-RUN (${cwd}) ${[command, ...args].join(" ")}`);
     return { status: 0, stdout: "", stderr: "" };
   }
-  const result = spawnSync(command, args, {
+  const { result } = spawnWithFallback(command, args, {
     cwd,
     stdio: options.stdio || "inherit",
     env: { ...process.env, ...(options.env || {}) },
@@ -24,7 +24,7 @@ export function runCommand(command, args = [], options = {}) {
 }
 
 export function captureCommand(command, args = [], options = {}) {
-  const result = spawnSync(command, args, {
+  const { result } = spawnWithFallback(command, args, {
     cwd: options.cwd || process.cwd(),
     env: { ...process.env, ...(options.env || {}) },
     encoding: "utf8",
@@ -38,3 +38,24 @@ export function captureCommand(command, args = [], options = {}) {
     error: result.error ? result.error.message : "",
   };
 }
+
+export function commandCandidatesForPlatform(command, platform = process.platform) {
+  if (platform === "win32" && command === "npm") {
+    return ["npm", "npm.cmd"];
+  }
+  return [command];
+}
+
+function spawnWithFallback(command, args, options) {
+  let lastResult;
+  for (const candidate of commandCandidatesForPlatform(command)) {
+    const result = spawnSync(candidate, args, options);
+    lastResult = result;
+    if (!isCommandNotFound(result)) return { command: candidate, result };
+  }
+  return { command, result: lastResult };
+}
+
+function isCommandNotFound(result) {
+  return result?.error?.code === "ENOENT";
+}