{"record":{"id":"e31652604bb41b60","repo":"chenglou/pretext","slug":"invalid-value-for-name-raw-e31652","errorCode":null,"errorMessage":"Invalid value for --${name}: ${raw}","messagePattern":"Invalid value for --(.+?): (.+?)","errorType":"validation","errorClass":"Error","httpStatus":null,"severity":"error","filePath":"scripts/keep-all-check.ts","lineNumber":55,"sourceCode":"  requestId?: string\n  results?: Array<{\n    label: string\n    report: ProbeReport\n  }>\n  message?: string\n}\n\nfunction parseStringFlag(name: string): string | null {\n  const prefix = `--${name}=`\n  const arg = process.argv.find(value => value.startsWith(prefix))\n  return arg === undefined ? null : arg.slice(prefix.length)\n}\n\nfunction parseNumberFlag(name: string, fallback: number): number {\n  const raw = parseStringFlag(name)\n  if (raw === null) return fallback\n  const parsed = Number.parseInt(raw, 10)\n  if (!Number.isFinite(parsed)) throw new Error(`Invalid value for --${name}: ${raw}`)\n  return parsed\n}\n\nfunction parseBrowsers(value: string | null): AutomationBrowserKind[] {\n  const raw = (value ?? 'chrome,safari').trim()\n  if (raw.length === 0) return ['chrome', 'safari']\n\n  const browsers = raw\n    .split(',')\n    .map(part => part.trim().toLowerCase())\n    .filter(Boolean)\n\n  for (const browser of browsers) {\n    if (browser !== 'chrome' && browser !== 'safari' && browser !== 'firefox') {\n      throw new Error(`Unsupported browser ${browser}`)\n    }\n  }\n","sourceCodeStart":37,"sourceCodeEnd":73,"githubUrl":"https://github.com/chenglou/pretext/blob/ac49b09b7d83ede19581fa94a8b892b07d309baf/scripts/keep-all-check.ts#L37-L73","documentation":"parseNumberFlag() in keep-all-check.ts parses a `--<name>=<value>` CLI argument as a base-10 integer and throws when the result is not finite (NaN). This guards the numeric flags (port, timeout) so a malformed value fails fast with a clear message instead of silently becoming NaN downstream.","triggerScenarios":"Invoking `bun run keep-all-check -- --port=abc`, `--port=` (empty), `--timeout=soon`, or any value where Number.parseInt yields NaN. A non-empty string of pure whitespace like `--port=   ` also parses to NaN.","commonSituations":"A typo in a Makefile/npm-script flag, a copied command with a stale placeholder, or a shell variable that expanded to empty.","solutions":["Re-run supplying an integer for the flagged option, e.g. `--port=3210` or `--timeout=60000`.","If you want the default, simply omit the flag rather than passing `--port=`.","Check the exact flag name in the error message and confirm it is one of port/timeout supported by keep-all-check.ts."],"exampleFix":"# before\nbun run keep-all-check -- --port=auto\n\n# after\nbun run keep-all-check -- --port=3210","handlingStrategy":"validation","validationCode":"function readNumberFlag(name: string, fallback: number): number {\n  const prefix = `--${name}=`\n  const arg = process.argv.find(v => v.startsWith(prefix))\n  if (arg === undefined) return fallback\n  const raw = arg.slice(prefix.length)\n  const n = Number.parseInt(raw, 10)\n  if (!Number.isFinite(n)) {\n    console.error(`--${name} expects an integer, got: ${JSON.stringify(raw)}`)\n    process.exit(2)\n  }\n  return n\n}","typeGuard":"const isIntegerString = (v: string): boolean => /^-?\\d+$/.test(v.trim())","tryCatchPattern":null,"preventionTips":["Pass integers explicitly for port/timeout; avoid empty `--flag=`.","Validate CLI args in a single helper so failures are uniform and predictable."],"tags":["cli","argument-parsing"],"backgroundTag":null,"analyzedSha":"ac49b09b7d83ede19581fa94a8b892b07d309baf","analyzedAt":"2026-08-12T17:03:16.263Z","schemaVersion":2},"datasetVersion":"2026-08-12T18:17:37.767Z"}