form-tester 0.8.1 → 0.8.3

package/.claude/skills/form-tester/INSTRUCTIONS.md

@@ -16,14 +16,23 @@ Test modes:
 form-tester test <url> --auto
 form-tester test <url> --auto --pnr 12345 --persona ung-mann
 
-# Human mode — prompts for persona, scenario, person selection:
+# Human mode — user chooses persona and scenario:
+# Step 1: Run without flags to see available personas:
 form-tester test <url> --human
+# Step 2: Ask the user which persona and scenario they want.
+# Step 3: Re-run with their choices:
+form-tester test <url> --human --persona ung-mann --scenario "test validation"
 
 # Full interactive CLI:
 form-tester
 ```
 
-When the user asks for human/interactive mode, use `--human`. Otherwise default to `--auto`.
+When the user asks for human/interactive mode, use `--human`:
+1. First run `form-tester test <url> --human` (no flags) to get the persona list.
+2. Show the personas to the user and ask them to choose.
+3. Ask the user for a test scenario (or use "" for standard).
+4. Re-run with their choices: `form-tester test <url> --human --persona <id> --scenario "<text>"`
+Otherwise default to `--auto`.
 
 Commands:
 ```
@@ -69,27 +78,33 @@ Dokumenter verification (only when modal confirms storage):
 1. Navigate to `/dokumenter?pnr={PNR}` and select the same person used during form fill.
 2. The document list loads sorted newest first. The first entry should match the form title.
 3. Click "Se detaljer" on the first document, then click "Åpne dokumentet".
-4. After clicking "Åpne dokumentet", the document may open in a dialog, a new tab, or inline. FIRST determine the format before capturing:
-
-   Step A — Detect format:
-   Run `form-tester exec snapshot` and examine the result.
-   - If you see an iframe/embed/object with a src containing `.pdf`, `blob:`, or a PDF viewer → it's a PDF.
-   - If you see rendered HTML content (headings, paragraphs, form data) → it's HTML.
-   - If `--full-page` screenshot times out → it's almost certainly a PDF viewer. Do NOT retry the screenshot. Switch to PDF download.
-
-   Step B — PDF documents (viewer in dialog/modal/iframe/new tab):
-   Do NOT screenshot PDFs — it will timeout or produce garbage. Download the file:
-   1. Extract PDF URL from the iframe/embed/object:
-      `form-tester exec eval "document.querySelector('iframe')?.src || document.querySelector('embed')?.src || document.querySelector('object')?.data"`
-   2. If the result is a URL, download it with run-code:
-      `form-tester exec run-code "async page => { const url = await page.evaluate(() => document.querySelector('iframe')?.src || document.querySelector('embed')?.src || document.querySelector('object')?.data); if (url) { const resp = await page.request.get(url); require('fs').writeFileSync('$OUTPUT_DIR/document.pdf', await resp.body()); } }"`
-   3. Or if the PDF viewer has a download button, click it.
-   4. Record in test_results.txt: document type PDF, downloaded to document.pdf.
-
-   Step C — HTML documents:
-   Take a FULL-PAGE screenshot (`form-tester exec screenshot --filename "$OUTPUT_DIR/document_screenshot.png" --full-page`).
+4. After clicking "Åpne dokumentet", determine the document format:
+
+   How to detect format:
+   - Take a snapshot: `form-tester exec snapshot`
+   - If snapshot shows a link with href containing `/pdf/` or `blob:` → PDF
+   - If `--full-page` screenshot times out → PDF (do NOT retry, switch to download)
+   - If snapshot shows rendered HTML content (headings, paragraphs, form data) → HTML
+
+   PDF documents — DOWNLOAD, do NOT screenshot:
+   IMPORTANT: `run-code` does NOT have access to `require` or `fs`. Do NOT use `require('fs')`. Use Playwright's download API instead.
+
+   Find the PDF download link in the snapshot (look for `a[href*="/pdf/"]` or "Last ned" link), then download using the Playwright download event:
+   ```
+   form-tester exec run-code "async page => { const link = page.locator('a[href*=\"/pdf/\"]').first(); const [download] = await Promise.all([ page.waitForEvent('download'), link.click() ]); await download.saveAs('$OUTPUT_DIR/document.pdf'); }"
+   ```
+   If there is no direct PDF link but a "Last ned" button:
+   ```
+   form-tester exec run-code "async page => { const [download] = await Promise.all([ page.waitForEvent('download'), page.getByRole('link', { name: 'Last ned' }).click() ]); await download.saveAs('$OUTPUT_DIR/document.pdf'); }"
+   ```
+   Verify the download: check that document.pdf exists in the output folder.
+
+   HTML documents — SCREENSHOT full page:
+   ```
+   form-tester exec screenshot --filename "$OUTPUT_DIR/document_screenshot.png" --full-page
+   ```
    Also save raw HTML: `form-tester exec eval "document.documentElement.outerHTML"` → save to document.html.
 
-   Step D — XML/other: Note type in test_results.txt, skip capture.
+   XML/other: Note type in test_results.txt, skip capture.
 
 5. Include the document verification results in test_results.txt (document title, whether it matched the form h1, document type: HTML/PDF/XML).

package/.claude/skills/form-tester/SKILL.md

@@ -20,8 +20,9 @@ form-tester install --global   # or install to ~/.claude/skills/
 form-tester test <url> --auto
 form-tester test <url> --auto --pnr 12345 --persona ung-mann --scenario "test validation"
 
-# Interactive mode (prompts for persona, scenario, etc.):
-form-tester test <url> --human
+# Human mode (user picks persona and scenario):
+form-tester test <url> --human                                          # lists personas, ask user
+form-tester test <url> --human --persona ung-mann --scenario "test X"   # run with user's choices
 
 # Full interactive CLI:
 form-tester

package/.github/copilot-instructions.md

@@ -17,8 +17,9 @@ form-tester install
 form-tester test <url> --auto
 form-tester test <url> --auto --pnr 12345 --persona ung-mann --scenario "test validation"
 
-# Human mode — prompts for persona, scenario, person selection:
-form-tester test <url> --human
+# Human mode — user chooses persona and scenario:
+form-tester test <url> --human                                          # lists personas
+form-tester test <url> --human --persona ung-mann --scenario "test X"   # run with choices
 
 # Full interactive CLI:
 form-tester
@@ -26,7 +27,11 @@ form-tester
 
 Persona IDs: `ung-mann`, `gravid-kvinne`, `eldre-kvinne`, `kronisk-syk-mann`. Defaults to "noen" if omitted.
 
-When the user asks for `--human` mode, use that flag. Otherwise default to `--auto`.
+When the user asks for `--human` mode:
+1. Run `form-tester test <url> --human` to get persona list.
+2. Ask the user to pick a persona and scenario.
+3. Re-run: `form-tester test <url> --human --persona <id> --scenario "<text>"` (use `""` for standard test).
+Otherwise default to `--auto`.
 
 ## Commands
 
@@ -82,13 +87,18 @@ After submission, read the modal text:
 - If it mentions Dokumenter storage -> navigate to `/dokumenter?pnr={PNR}`, verify the document appears.
 - If it does NOT mention Dokumenter -> skip verification, note in test_results.txt.
 
-Document capture — FIRST detect the format by running `form-tester exec snapshot`:
+Document capture — detect format via `form-tester exec snapshot`:
 
-**PDF documents** (iframe/embed with .pdf or blob: URL, or screenshot times out): Do NOT screenshot PDFs. Download instead:
-1. Extract URL: `form-tester exec eval "document.querySelector('iframe')?.src || document.querySelector('embed')?.src || document.querySelector('object')?.data"`
-2. Download: `form-tester exec run-code "async page => { const url = await page.evaluate(() => document.querySelector('iframe')?.src || document.querySelector('embed')?.src || document.querySelector('object')?.data); if (url) { const resp = await page.request.get(url); require('fs').writeFileSync('OUTPUT_DIR/document.pdf', await resp.body()); } }"`
-3. Or click the download button in the PDF viewer if available.
-4. If `--full-page` screenshot times out, it's a PDF — switch to download, don't retry screenshot.
+**PDF documents** (link with href containing `/pdf/`, or screenshot times out):
+Do NOT screenshot PDFs. Do NOT use `require('fs')` in run-code (it doesn't exist there).
+Download using Playwright's download event:
+```
+form-tester exec run-code "async page => { const link = page.locator('a[href*=\"/pdf/\"]').first(); const [download] = await Promise.all([ page.waitForEvent('download'), link.click() ]); await download.saveAs('$OUTPUT_DIR/document.pdf'); }"
+```
+Or if there's a "Last ned" button:
+```
+form-tester exec run-code "async page => { const [download] = await Promise.all([ page.waitForEvent('download'), page.getByRole('link', { name: 'Last ned' }).click() ]); await download.saveAs('$OUTPUT_DIR/document.pdf'); }"
+```
 
 **HTML documents**: `form-tester exec screenshot --filename "..." --full-page`. Also save raw HTML.
 

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.8.1";
+const LOCAL_VERSION = "0.8.3";
 const RECOMMENDED_PERSON = "Uromantisk Direktør";
 
 // Recording — persisted to disk so `form-tester exec` can append across processes
@@ -1321,10 +1321,31 @@ async function main() {
     const config = loadConfig();
     const url = args.find((a) => a.startsWith("http"));
     if (!url) {
-      console.error("Usage: form-tester test <url> --human");
+      console.error("Usage: form-tester test <url> --human --persona <id> --scenario \"<text>\"");
       process.exit(1);
     }
-    await handleTest(url, config);
+    const flagVal = (flag) => args.includes(flag) ? args[args.indexOf(flag) + 1] : undefined;
+    const personaFlag = flagVal("--persona");
+    const scenarioFlag = flagVal("--scenario");
+
+    // If persona or scenario missing, print choices and exit so the AI can ask the user
+    if (!personaFlag || scenarioFlag === undefined) {
+      console.log("HUMAN MODE: Ask the user to choose persona and scenario, then re-run with flags.\n");
+      console.log("Available personas:");
+      console.log(formatPersonaList());
+      console.log("\nPersona IDs: ung-mann, gravid-kvinne, eldre-kvinne, kronisk-syk-mann, noen");
+      console.log("\nRe-run with: form-tester test <url> --human --persona <id> --scenario \"<description>\"");
+      console.log("Use --scenario \"\" for standard test.");
+      process.exit(0);
+    }
+
+    // Run with user's choices
+    await handleTestAuto(url, config, {
+      pnr: flagVal("--pnr"),
+      persona: personaFlag,
+      scenario: scenarioFlag || undefined,
+      verbosity: "normal",
+    });
     process.exit(0);
   }
 

package/package.json

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