qa-browser checks 5 of 18 pages by default and captures no console errors #227

New Issue

Closed

opened 2026-09-10 04:13:34 -05:00 by null · 1 comment

null commented

2026-09-10 04:13:34 -05:00

Owner

Seen at a25077d (v0.9.5).

What is true now. scripts/qa-browser.mjs defaults --url to production (line 60) and --paths to /, /about, /services, /contact and /support (line 73): 5 of the 18 pages, and none of the service or industry pages this batch changes. It records no console or page errors, so a hydration failure or a thrown effect is invisible to it.

What to do. Default the paths to the target's sitemap.xml, and exit 2 if it cannot be read (nothing checked is not a pass). Capture console errors and page errors per page, as findings. Fix docs/qa/ClaudeQAPlan.md:85 ("five widths"; it is four) and docs/TOOLS.md to match.

Verify: node scripts/qa-browser.mjs --url http://localhost:3001 checks all 18 sitemap pages, and reports a finding when a page logs a console error.

Seen at a25077d (v0.9.5). **What is true now.** `scripts/qa-browser.mjs` defaults `--url` to production (line 60) and `--paths` to `/`, `/about`, `/services`, `/contact` and `/support` (line 73): 5 of the 18 pages, and none of the service or industry pages this batch changes. It records no console or page errors, so a hydration failure or a thrown effect is invisible to it. **What to do.** Default the paths to the target's `sitemap.xml`, and exit 2 if it cannot be read (nothing checked is not a pass). Capture console errors and page errors per page, as findings. Fix `docs/qa/ClaudeQAPlan.md:85` ("five widths"; it is four) and `docs/TOOLS.md` to match. Verify: `node scripts/qa-browser.mjs --url http://localhost:3001` checks all 18 sitemap pages, and reports a finding when a page logs a console error.

null added this to the Batch 18: Guards and landmines found building Batch 17 milestone 2026-09-10 04:13:34 -05:00

null added the

bug

infra

labels 2026-09-10 04:13:34 -05:00

null referenced this issue from a commit

2026-09-10 04:53:57 -05:00

fix(qa): the browser check measured 5 of 18 pages and heard no errors

null closed this issue

2026-09-10 04:53:57 -05:00

null commented

2026-09-10 04:54:07 -05:00

Author

Owner

Done in 7cd556c. scripts/qa-browser.mjs now defaults its path list to the target's own sitemap (18 pages instead of 5 hand-typed ones, and it exits 2 if it cannot read the sitemap) and captures page errors and console errors as findings. Proven: a console.error added to About produced '/about @390: the page logged an error', and a clean build reports nothing across all 18 sitemap pages. docs/qa/ClaudeQAPlan.md corrected from 'five pages, five widths' to every page in the sitemap at four widths.

Done in 7cd556c. scripts/qa-browser.mjs now defaults its path list to the target's own sitemap (18 pages instead of 5 hand-typed ones, and it exits 2 if it cannot read the sitemap) and captures page errors and console errors as findings. Proven: a console.error added to About produced '/about @390: the page logged an error', and a clean build reports nothing across all 18 sitemap pages. docs/qa/ClaudeQAPlan.md corrected from 'five pages, five widths' to every page in the sitemap at four widths.