Skip to content

fix(dev-1698): bump browserslist to 4.28.7 - #59

Open
spur-vuln-author[bot] wants to merge 1 commit into
mainfrom
fix/dev-1698
Open

spur-vuln-author[bot] wants to merge 1 commit into
mainfrom
fix/dev-1698

Conversation

@spur-vuln-author

Copy link
Copy Markdown
Contributor

Context

GHSA-73wf-gq98-2v4g: browserslist before 4.28.7 is vulnerable (high severity, ReDoS). Transitive dependency (not declared in any package.json), resolved via pnpm-lock.yaml. No Dependabot PR exists for this finding, so this PR was constructed directly from the advisory's first_patched_version.

Test evidence

No Dependabot PR existed; constructed from first_patched_version 4.28.7. Added a pnpm.overrides entry and regenerated the lockfile with pnpm install --lockfile-only. Confirmed via lockfile inspection that browserslist now resolves to 4.29.3 (previously 4.24.4), satisfying >= 4.28.7. Companion-file scan (Dockerfiles, .nvmrc, go.mod, CI workflow YAML, etc. hardcoding 4.24.4) found no hits.

Risk

Low. Lockfile-only transitive dependency bump within the same major version line. GHSA-73wf-gq98-2v4g / high severity; Tier 1 / 90-day SLA.


@spur-vuln-author spur-vuln-author Bot added the spur-vuln Security vulnerability remediation PR label Sep 30, 2026
@spur-vuln-reviewer

Copy link
Copy Markdown

Escalating: Gate 3 (diff equivalence) mechanically FAILed. gate3-check.sh: GATE3 FAIL target=browserslist manifest=pnpm-lock.yaml pr_version=? required=4.28.7 alerts=115 extra_files=1 state=open. Detail: the helper has no extraction rule for ecosystem=npm manifest basename=pnpm-lock.yaml, so it could not extract the new browserslist version from the diff, and flagged package.json as a file outside {manifest, lockfile} (which is actually the expected direct-dependency file for an npm/pnpm version bump). This looks like a tooling gap in gate3-check.sh (no pnpm-lock.yaml parser) rather than a real diff deviation, but per the review contract a mechanical Gate 3 FAIL is never re-derived or second-guessed by the reviewer. Required version per max open Dependabot alert(s) 115: 4.28.7. Not merged. A human must review this. This reviewer will not act on this PR again. [[spur-vuln-reviewer: escalated diff-mismatch-constructed]]

@spur-vuln-reviewer spur-vuln-reviewer Bot added the hitl Needs human review — automated vuln remediation escalated this label Sep 30, 2026
@spur-vuln-author

Copy link
Copy Markdown
Contributor Author

spur-vuln-remediate: this branch (browserslist bump) has a real merge conflict against current main, confirmed via gh pr update-branch --rebase (Cannot update PR branch due to conflicts) - pnpm-lock.yaml has drifted since this branch was cut, likely from the tinypool/vite/turbo/vitest merges that landed afterward. This sandbox's permission system does not allow the author identity to force-push, merge, or delete/recreate its own branch, so the conflict cannot be resolved from this pass. Needs a human (or a future pass with broader git permissions) to rebase or recreate this branch.

@spur-vuln-author

Copy link
Copy Markdown
Contributor Author

spur-vuln-escalation: failed on 2 consecutive passes. Still DIRTY (real merge conflict against current main) after a fresh gh pr update-branch --rebase attempt this pass. A human-authored consolidated PR #73 already bumps browserslist to 4.29.3 (above this group's 4.28.7 target). Needs a human - tracked in DEV-1698.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hitl Needs human review — automated vuln remediation escalated this spur-vuln Security vulnerability remediation PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants