Coder/deepseek-harness
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): reject partially parsed concurrency limits

Browse Source 
Number.parseInt accepts a numeric prefix, so values like 1.5 or 8junk
silently ran an unintended worker count. Require the full string to
round-trip (same pattern as run-gates' positiveIntArg).
This commit is contained in:
Chinesezjc
2026-07-22 15:30:53 +08:00
parent becc33d7fb
commit 0580bc9068
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
scripts/verify-built-package-invariants.mjs
View File

@@ -33,7 +33,7 @@ function probeConcurrency(total) {
const raw = process.env[CONCURRENCY_ENV]
if (raw !== undefined && raw !== '') {
const parsed = Number.parseInt(raw, 10)
if (!Number.isSafeInteger(parsed) || parsed < 1) {
if (!Number.isSafeInteger(parsed) || parsed < 1 || String(parsed) !== raw) {
throw new Error(`verify-built-package-invariants: ${CONCURRENCY_ENV} must be a positive integer, got ${JSON.stringify(raw)}.`)
}
return Math.min(total, parsed)