scripts/osv-check.sh gates High/Critical on every PR. It is now failing on
main's existing lockfile — no code change required, the advisories were just
published — so every open PR's osv dependency scan job fails, not one of them.
✗ osv: 0 Critical + 3 High vulnerability(ies) — must be resolved before merge.
The three High findings are all transitive:
The remaining 8 are Medium/Low and do not gate: 4 in hono 4.12.32 (fixed in
4.12.34) plus 4 more in undici.
Dependabot #142 does not clear this
#142 bumps hono 4.12.32 → 4.13.1, which covers the four Medium/Low hono
advisories. It does not touch nanoid, js-yaml, or undici — none of them are
direct dependencies — so the gate stays red after it merges.
Suggested fix
pnpm.overrides in the root package.json, matching the existing precedent from
9b6080a (fix(deps): pin jsdom's transitive undici to 7.x so jsdom 29 collects):
"pnpm": {
"overrides": {
"nanoid": "^3.3.18",
"js-yaml": "^4.3.1",
"undici": "^7.29.0"
}
}
Then pnpm install, confirm scripts/osv-check.sh reports 0 High/Critical, and
run pnpm -r test — the existing undici pin exists because jsdom 29 needs 7.x,
so that constraint has to keep holding.
Note the local validate.sh run is warn-only for osv (|| true), so this does
not block committing locally — only CI, and only at the merge gate.
Found by the maintenance loop while landing #144.
scripts/osv-check.shgates High/Critical on every PR. It is now failing onmain's existing lockfile — no code change required, the advisories were justpublished — so every open PR's
osv dependency scanjob fails, not one of them.The three High findings are all transitive:
nanoidjs-yamlundiciThe remaining 8 are Medium/Low and do not gate: 4 in
hono4.12.32 (fixed in4.12.34) plus 4 more in
undici.Dependabot #142 does not clear this
#142 bumps hono 4.12.32 → 4.13.1, which covers the four Medium/Low hono
advisories. It does not touch
nanoid,js-yaml, orundici— none of them aredirect dependencies — so the gate stays red after it merges.
Suggested fix
pnpm.overridesin the rootpackage.json, matching the existing precedent from9b6080a (
fix(deps): pin jsdom's transitive undici to 7.x so jsdom 29 collects):Then
pnpm install, confirmscripts/osv-check.shreports 0 High/Critical, andrun
pnpm -r test— the existingundicipin exists because jsdom 29 needs 7.x,so that constraint has to keep holding.
Note the local
validate.shrun is warn-only for osv (|| true), so this doesnot block committing locally — only CI, and only at the merge gate.
Found by the maintenance loop while landing #144.