form-tester 0.3.3 → 0.3.5

package/form-tester.js

@@ -6,7 +6,7 @@ const { spawn, execSync } = require("child_process");
 
 const CONFIG_PATH = path.join(__dirname, "form-tester.config.json");
 const OUTPUT_BASE = path.resolve(__dirname, "output");
-const LOCAL_VERSION = "0.3.3";
+const LOCAL_VERSION = "0.3.5";
 const RECOMMENDED_PERSON = "Uromantisk Direktør";
 
 const PERSONAS = [
@@ -272,7 +272,11 @@ function isPlaywrightCliAvailable() {
 
 function runCommand(command, args, options = {}) {
   return new Promise((resolve) => {
-    const child = spawn(command, args, { stdio: "inherit", ...options });
+    const child = spawn(command, args, {
+      stdio: "inherit",
+      shell: process.platform === "win32",
+      ...options,
+    });
     child.on("error", (err) => {
       console.error(`Failed to launch ${command}: ${err.message}`);
       resolve(1);
@@ -284,9 +288,9 @@ function runCommand(command, args, options = {}) {
 function runPlaywrightCli(args) {
   return new Promise((resolve) => {
     const spec = getPlaywrightCommandSpec();
-    const child = spawn(spec.command, [...spec.args, ...args], {
-      stdio: "inherit",
-    });
+    const spawnOpts = { stdio: "inherit" };
+    if (spec.shell) spawnOpts.shell = true;
+    const child = spawn(spec.command, [...spec.args, ...args], spawnOpts);
     child.on("error", (err) => {
       console.error(`Failed to launch ${spec.command}: ${err.message}`);
       resolve(1);
@@ -298,10 +302,12 @@ function runPlaywrightCli(args) {
 function runPlaywrightCliCapture(args) {
   return new Promise((resolve) => {
     const spec = getPlaywrightCommandSpec();
+    const captureOpts = { stdio: ["ignore", "pipe", "pipe"] };
+    if (spec.shell) captureOpts.shell = true;
     const child = spawn(
       spec.command,
       [...spec.args, ...args],
-      { stdio: ["ignore", "pipe", "pipe"] },
+      captureOpts,
     );
     let stdout = "";
     let stderr = "";
@@ -328,6 +334,7 @@ function getPlaywrightCommandSpec() {
   const command = getPlaywrightCommand();
   const lower = command.toLowerCase();
   if (process.platform === "win32" && (lower.endsWith(".cmd") || lower.endsWith(".ps1"))) {
+    // Try to resolve the actual .js entry point to avoid shell/spawn issues
     const nodeDir = path.dirname(command);
     const cliPath = path.join(
       nodeDir,
@@ -337,16 +344,12 @@ function getPlaywrightCommandSpec() {
       "playwright-cli.js",
     );
     if (fs.existsSync(cliPath)) {
-      return { command: process.execPath, args: [cliPath] };
+      return { command: process.execPath, args: [cliPath], shell: false };
     }
+    // Fallback: run via node process.execPath with the .cmd/.ps1 content
+    return { command, args: [], shell: true };
   }
-  if (lower.endsWith(".ps1")) {
-    return {
-      command: "powershell",
-      args: ["-NoProfile", "-ExecutionPolicy", "Bypass", "-File", command],
-    };
-  }
-  return { command, args: [] };
+  return { command, args: [], shell: false };
 }
 
 function findRepoRoot(startDir) {
@@ -947,36 +950,39 @@ function copyDirSync(src, dest) {
   }
 }
 
-function install(targetDir) {
+function install(targetDir, isGlobal) {
   const pkgDir = __dirname;
   const skillsSrc = path.join(pkgDir, ".claude", "skills");
   const copilotSrc = path.join(pkgDir, ".github", "copilot-instructions.md");
   const configSrc = path.join(pkgDir, "form-tester.config.example.json");
 
   // Copy Claude Code skills
-  const skillsDest = path.join(targetDir, ".claude", "skills");
+  const skillsDest = isGlobal
+    ? path.join(targetDir, "skills")
+    : path.join(targetDir, ".claude", "skills");
   for (const skill of ["form-tester", "playwright-cli"]) {
     const src = path.join(skillsSrc, skill);
     if (fs.existsSync(src)) {
       const dest = path.join(skillsDest, skill);
       copyDirSync(src, dest);
-      console.log(`  Installed .claude/skills/${skill}/`);
+      console.log(`  Installed ${isGlobal ? "~/.claude" : ".claude"}/skills/${skill}/`);
     }
   }
 
-  // Copy Copilot instructions
-  if (fs.existsSync(copilotSrc)) {
-    const copilotDest = path.join(targetDir, ".github", "copilot-instructions.md");
-    fs.mkdirSync(path.dirname(copilotDest), { recursive: true });
-    fs.copyFileSync(copilotSrc, copilotDest);
-    console.log("  Installed .github/copilot-instructions.md");
-  }
+  // Project-only files (skip for global install)
+  if (!isGlobal) {
+    if (fs.existsSync(copilotSrc)) {
+      const copilotDest = path.join(targetDir, ".github", "copilot-instructions.md");
+      fs.mkdirSync(path.dirname(copilotDest), { recursive: true });
+      fs.copyFileSync(copilotSrc, copilotDest);
+      console.log("  Installed .github/copilot-instructions.md");
+    }
 
-  // Copy config example
-  if (fs.existsSync(configSrc)) {
-    const configDest = path.join(targetDir, "form-tester.config.example.json");
-    fs.copyFileSync(configSrc, configDest);
-    console.log("  Installed form-tester.config.example.json");
+    if (fs.existsSync(configSrc)) {
+      const configDest = path.join(targetDir, "form-tester.config.example.json");
+      fs.copyFileSync(configSrc, configDest);
+      console.log("  Installed form-tester.config.example.json");
+    }
   }
 
   // Install playwright-cli globally if not already available
@@ -1009,9 +1015,19 @@ async function main() {
   const args = process.argv.slice(2);
 
   if (args[0] === "install") {
-    const targetDir = args[1] ? path.resolve(args[1]) : process.cwd();
-    console.log(`Installing form-tester skills to ${targetDir} ...\n`);
-    install(targetDir);
+    const isGlobal = args.includes("--global") || args.includes("-g");
+    const remaining = args.slice(1).filter((a) => a !== "--global" && a !== "-g");
+    let targetDir;
+    if (isGlobal) {
+      const home = process.env.HOME || process.env.USERPROFILE;
+      targetDir = path.join(home, ".claude");
+      // Global installs skills directly into ~/.claude/skills/
+      console.log(`Installing form-tester skills globally to ${targetDir} ...\n`);
+    } else {
+      targetDir = remaining[0] ? path.resolve(remaining[0]) : process.cwd();
+      console.log(`Installing form-tester skills to ${targetDir} ...\n`);
+    }
+    install(targetDir, isGlobal);
     process.exit(0);
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "form-tester",
-  "version": "0.3.3",
+  "version": "0.3.5",
   "description": "AI-powered form testing skill for /skjemautfyller forms using Playwright CLI. Works with Claude Code and GitHub Copilot.",
   "main": "form-tester.js",
   "bin": {