[tmashara]

Fixes

• Blank numeric values: --threshold= and --threshold " " used to read as 0, so the run never failed; now they exit 2 with "--threshold needs a value". --width= and --concurrency= say "needs a value" instead of "got 0".
• Negative numbers: a numeric flag takes a negative value, so the range check names the problem (--threshold -1 gives "expects 0..1, got -1" instead of "needs a value").
• =value on flags that take none: --dry-run=false, --progress=false, --computed=false, --no-recheck=x etc. exit 2 with " takes no value". --dry-run=false used to still dry-run.
• Integer --width: --width 1.5 is rejected at the flag ("--width expects a positive integer"). This matches Config: reject unknown keys, bad widths, non-http origins, blank selectors #120's config widths check.
• Blank selectors: --selector, --left-selector and --right-selector are trimmed, and a blank one is rejected, as with --mask/--hide.
• --sitemap guard: a bare --sitemap together with a route or origin ending in .xml now says "use --sitemap=". Before, the file silently became a route (or the right origin) while the default sitemap was fetched.
• report / merge: they share the main parser's value logic. --x=value works, and a missing value no longer swallows the next flag (--title --markdown f gives "--title needs a value").

All parsers now go through one flags() walker in src/cli.ts. It replaces the inline split/next() and the per-branch blank checks.

Behaviour changes

• --accept " " now says "--accept needs a value" (was "needs a reason").
• report --foo=x says "unexpected argument: --foo" (the flag name, without =x).
• In report/merge, a value starting with - needs the = form: --title=-x. --title -x is rejected.

Testing

typecheck, lint, format:check and npm test pass after the rebase on #120 (388 tests).

• New cases in tests/unit/cli-args.test.ts: blank numerics, fractional width, =value on boolean flags, the sitemap guard (route and origin forms), a blank selector, and = inside an origin and a route still parsing.
• New cases in tests/unit/report.test.ts: report --markdown=<file>, report --title with a missing value, merge --out with a missing value.

🤖 Generated with Claude Code