form-tester 0.4.2 → 0.5.0

package/README.md

@@ -32,7 +32,9 @@ Edit `form-tester.config.json` and set your `pnr`.
 
 ## Usage
 
-### Non-interactive (recommended for AI agents)
+### Default: non-interactive (`--auto`)
+
+AI agents (Claude Code, Copilot) will use `--auto` mode by default. No prompts, just runs the test:
 
 ```bash
 form-tester test <url> --auto
@@ -43,6 +45,8 @@ Persona IDs: `ung-mann`, `gravid-kvinne`, `eldre-kvinne`, `kronisk-syk-mann`
 
 ### Interactive CLI
 
+For manual use without an AI agent:
+
 ```bash
 form-tester
 ```
@@ -51,11 +55,55 @@ Commands: `/setup`, `/update`, `/version`, `/people`, `/test {url}`, `/save {lab
 
 ### Claude Code
 
-The skill is automatically detected when you open the project. Use the `/form-tester` skill.
+After `form-tester install`, the skill is automatically detected. Use the `/form-tester` skill or just ask Claude to test a form.
+
+If the skill isn't showing up:
+1. Make sure `.claude/skills/form-tester/` exists in your project (run `form-tester install`)
+2. Restart Claude Code or start a new conversation
 
 ### GitHub Copilot
 
-Copilot reads instructions from `.github/copilot-instructions.md` automatically.
+After `form-tester install`, Copilot reads `.github/copilot-instructions.md` and `.claude/skills/` automatically.
+
+If Copilot doesn't recognize the skill:
+1. Make sure `.claude/skills/form-tester/` exists in your project (run `form-tester install`)
+2. Run `/skills` in the Copilot CLI to reload skills, or restart the session
+
+## Skip permission prompts
+
+By default, AI agents will ask permission for every shell command. To run without interruptions, pre-allow the relevant tools.
+
+### Claude Code
+
+Add to your project's `.claude/settings.local.json` (or global `~/.claude/settings.json`):
+
+```json
+{
+  "permissions": {
+    "allow": [
+      "Skill(form-tester)",
+      "Bash(form-tester:*)",
+      "Bash(playwright-cli:*)"
+    ]
+  }
+}
+```
+
+Or use the Claude Code CLI:
+```bash
+claude config add permissions.allow "Skill(form-tester)"
+claude config add permissions.allow "Bash(form-tester:*)"
+claude config add permissions.allow "Bash(playwright-cli:*)"
+```
+
+### GitHub Copilot
+
+In Copilot CLI, use auto-approve mode:
+```bash
+copilot --auto-approve
+```
+
+Or approve the tool categories when first prompted and select "Always allow".
 
 ## Test Output
 

package/form-tester.js

@@ -6,7 +6,7 @@ const { spawn, execSync } = require("child_process");
 
 const CONFIG_PATH = path.join(process.cwd(), "form-tester.config.json");
 const OUTPUT_BASE = path.resolve(process.cwd(), "output");
-const LOCAL_VERSION = "0.4.2";
+const LOCAL_VERSION = "0.5.0";
 const RECOMMENDED_PERSON = "Uromantisk Direktør";
 
 const PERSONAS = [
@@ -430,7 +430,7 @@ function printNextSteps(outputDir, dokumenterUrl) {
   );
   console.log(`- Or use: /save step`);
   console.log(
-    `- Screenshot before submit: playwright-cli screenshot --filename "${path.join(outputDir, "before_submit.png")}"`,
+    `- Screenshot before submit: playwright-cli screenshot --filename "${path.join(outputDir, "before_submit.png")}" --full-page`,
   );
   console.log("- Submit only when validation errors are cleared.");
   console.log(
@@ -702,8 +702,8 @@ async function saveArtifacts(config, label) {
   const timestamp = new Date().toISOString().replace(/[:.]/g, "-");
   const base = path.join(config.lastRunDir, `${safeLabel}_${timestamp}`);
   await runPlaywrightCli(["snapshot", "--filename", `${base}.yml`]);
-  await runPlaywrightCli(["screenshot", "--filename", `${base}.png`]);
-  console.log(`Saved: ${base}.yml and ${base}.png`);
+  await runPlaywrightCli(["screenshot", "--filename", `${base}.png`, "--full-page"]);
+  console.log(`Saved: ${base}.yml and ${base}.png (full-page)`);
 }
 
 async function handleSetup() {
@@ -862,6 +862,7 @@ async function handleTest(url, config) {
     "screenshot",
     "--filename",
     path.join(outputDir, "page_open.png"),
+    "--full-page",
   ]);
 
   await promptPersonSelection(config);
@@ -879,6 +880,10 @@ async function handleTest(url, config) {
 }
 
 async function handleTestAuto(url, config, flags) {
+  const v = flags.verbosity || "normal";
+  const log = (msg) => { if (v !== "silent") console.log(msg); };
+  const verbose = (msg) => { if (v === "verbose") console.log(msg); };
+
   // Resolve PNR
   const pnr = flags.pnr || config.pnr;
   if (!pnr) {
@@ -888,6 +893,7 @@ async function handleTestAuto(url, config, flags) {
   const fullUrl = extractPnrFromUrl(url) ? url : setPnrOnUrl(url, pnr);
   config.pnr = pnr;
   saveConfig(config);
+  verbose(`URL: ${fullUrl}`);
 
   // Resolve persona
   let personaChoice;
@@ -895,14 +901,14 @@ async function handleTestAuto(url, config, flags) {
   if (personaId) {
     const found = getPersonaById(personaId);
     if (found) {
-      console.log(`Persona: ${found.name} — ${found.description}`);
+      log(`Persona: ${found.name} — ${found.description}`);
       personaChoice = { type: "preset", persona: found };
     } else {
-      console.log(`Unknown persona "${personaId}", using Noen.`);
+      log(`Unknown persona "${personaId}", using Noen.`);
       personaChoice = { type: "noen", persona: null };
     }
   } else {
-    console.log("Persona: Noen — nøytrale svar (auto)");
+    log("Persona: Noen — nøytrale svar (auto)");
     personaChoice = { type: "noen", persona: null };
   }
 
@@ -910,7 +916,7 @@ async function handleTestAuto(url, config, flags) {
   const scenarioChoice = flags.scenario
     ? { type: "custom", description: flags.scenario }
     : { type: "default", description: "Standard test" };
-  console.log(`Scenario: ${scenarioChoice.description}`);
+  log(`Scenario: ${scenarioChoice.description}`);
 
   // Create output directory
   const formId = sanitizeSegment(extractFormId(fullUrl));
@@ -932,16 +938,18 @@ async function handleTestAuto(url, config, flags) {
   }
   fs.writeFileSync(path.join(outputDir, "scenario.json"), JSON.stringify(scenarioChoice, null, 2));
 
-  // Open and snapshot
-  console.log("Opening form with Playwright CLI...");
+  // Open and take initial full-page screenshot
+  log("Opening form with Playwright CLI...");
   await runPlaywrightCli(["open", fullUrl]);
   await runPlaywrightCli(["snapshot", "--filename", path.join(outputDir, "page_open.yml")]);
-  await runPlaywrightCli(["screenshot", "--filename", path.join(outputDir, "page_open.png")]);
+  await runPlaywrightCli(["screenshot", "--filename", path.join(outputDir, "page_open.png"), "--full-page"]);
+  log("Saved: page_open.yml + page_open.png (full-page)");
 
   // Auto-select person (try recommended, then first available)
   let options = extractPersonsFromSnapshotFile(path.join(outputDir, "page_open.yml"));
   if (!options.length) {
     for (let attempt = 0; attempt < 3; attempt++) {
+      verbose(`Scanning for person options (attempt ${attempt + 1})...`);
       await sleep(1500);
       options = await fetchPersonOptions();
       if (options.length) break;
@@ -950,17 +958,31 @@ async function handleTestAuto(url, config, flags) {
   if (options.length) {
     options = prioritizeRecommended(options, RECOMMENDED_PERSON);
     const chosen = options[0];
-    console.log(`Auto-selected person: ${chosen}`);
+    log(`Auto-selected person: ${chosen}`);
     config.lastPerson = chosen;
     saveConfig(config);
   }
 
+  // Take form loaded screenshot after person selection
+  await runPlaywrightCli(["snapshot", "--filename", path.join(outputDir, "form_loaded.yml")]);
+  await runPlaywrightCli(["screenshot", "--filename", path.join(outputDir, "form_loaded.png"), "--full-page"]);
+  log("Saved: form_loaded.yml + form_loaded.png (full-page)");
+
   const dokumenterUrl = resolveDokumenterUrl(config);
+
+  // Always print output folder (even in silent mode)
   console.log(`Output folder: ${outputDir}`);
-  if (dokumenterUrl) {
-    console.log(`Dokumenter URL: ${dokumenterUrl}`);
+
+  if (v !== "silent") {
+    if (dokumenterUrl) {
+      console.log(`Dokumenter URL: ${dokumenterUrl}`);
+    }
+    console.log("");
+    console.log("IMPORTANT: All screenshots MUST use --full-page to capture the entire page.");
+    console.log("Example: playwright-cli screenshot --filename \"path/to/file.png\" --full-page");
+    console.log("");
+    printNextSteps(outputDir, dokumenterUrl || "/dokumenter?pnr={PNR}");
   }
-  printNextSteps(outputDir, dokumenterUrl || "/dokumenter?pnr={PNR}");
 }
 
 async function handleCommand(line, config) {
@@ -1120,13 +1142,17 @@ async function main() {
     const config = loadConfig();
     const url = args.find((a) => a.startsWith("http"));
     if (!url) {
-      console.error("Usage: form-tester test <url> --auto [--pnr <pnr>] [--persona <id>] [--scenario <text>]");
+      console.error("Usage: form-tester test <url> --auto [--pnr <pnr>] [--persona <id>] [--scenario <text>] [--silent|--verbose]");
       process.exit(1);
     }
-    const pnrFlag = args[args.indexOf("--pnr") + 1];
-    const personaFlag = args[args.indexOf("--persona") + 1];
-    const scenarioFlag = args[args.indexOf("--scenario") + 1];
-    await handleTestAuto(url, config, { pnr: pnrFlag, persona: personaFlag, scenario: scenarioFlag });
+    const flagVal = (flag) => args.includes(flag) ? args[args.indexOf(flag) + 1] : undefined;
+    const verbosity = args.includes("--silent") ? "silent" : args.includes("--verbose") ? "verbose" : "normal";
+    await handleTestAuto(url, config, {
+      pnr: flagVal("--pnr"),
+      persona: flagVal("--persona"),
+      scenario: flagVal("--scenario"),
+      verbosity,
+    });
     process.exit(0);
   }
 

package/package.json

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