@rosenfan/matrix 0.1.0 → 0.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rosenfan/matrix",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Install and manage the Matrix workflow for Matt Pocock's agent skills.",
   "license": "MIT",
   "type": "module",

package/src/catalog.js

@@ -11,7 +11,7 @@ export const MATRIX_SKILLS = [
   "matrix-archive", "matrix-hotfix", "matrix-tweak", "matrix-status", "matrix-claude"
 ];
 export const MATT_SKILLS = [
-  "grill-with-docs", "domain-modeling", "research", "wayfinder", "prototype",
+  "grill-with-docs", "grilling", "domain-modeling", "research", "wayfinder", "prototype",
   "codebase-design", "implement", "tdd", "code-review", "diagnosing-bugs",
   "resolving-merge-conflicts", "improve-codebase-architecture"
 ];

package/src/matt.js

@@ -1,10 +1,21 @@
 import { execFileSync } from "node:child_process";
+import fs from "node:fs";
+import path from "node:path";
 import { PLATFORMS, MATT_SKILLS } from "./catalog.js";
 
+export function buildProcessInvocation({ command, args, platform = process.platform, nodeExecutable = process.execPath, npmCliPath }) {
+  if (platform !== "win32" || command !== "npx") return { executable: command, args, shell: false };
+  const candidates = [npmCliPath, process.env.npm_execpath, path.join(path.dirname(nodeExecutable), "node_modules", "npm", "bin", "npm-cli.js")].filter(Boolean);
+  const cli = candidates.find((candidate) => npmCliPath || fs.existsSync(candidate));
+  if (!cli) throw new Error("npm CLI was not found beside the Node runtime.");
+  const [yes, packageSpec, ...commandArgs] = args;
+  if (yes !== "--yes" || !packageSpec) throw new Error("Windows npx invocation must use fixed npm exec arguments.");
+  return { executable: nodeExecutable, args: [cli, "exec", yes, packageSpec, "--", ...commandArgs], shell: false };
+}
+
 export function defaultProcessRunner({ command, args, cwd, timeout }) {
-  const executable = process.platform === "win32" && command === "npx" ? "npx.cmd" : command;
-  // The only shell-enabled case is the Windows npm shim; all inputs are catalog values.
-  return execFileSync(executable, args, { cwd, stdio: "inherit", timeout, shell: process.platform === "win32" && command === "npx" });
+  const invocation = buildProcessInvocation({ command, args });
+  return execFileSync(invocation.executable, invocation.args, { cwd, stdio: "inherit", timeout, shell: invocation.shell });
 }
 
 export function createMattAdapter({ run = defaultProcessRunner } = {}) {