chore: bump @types/node from 22.19.17 to 22.20.3 - #66
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 22.19.17 to 22.20.3. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) --- updated-dependencies: - dependency-name: "@types/node" dependency-version: 22.20.3 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
|
Claude finished @dependabot[bot]'s task in 1m 18s —— View job Dependency review:
|
| Location | Value |
|---|---|
package.json:13 engines.node |
>=22 |
package.json:45 @types/node |
^22.0.0 |
manifest.json:67 compatibility.runtimes.node |
>=22.0.0 |
README.md:18 Requirements |
Node.js ≥ 22 |
.github/workflows/ci.yml:20 matrix |
['22', '24'] |
.github/workflows/update-catalog.yml:27 |
'22' |
(release.yml:26 runs Node 24 deliberately — build/publish only, per CLAUDE.md.)
Risk from type drift — low, and not new
Worth naming precisely since it's the whole reason the majors are pinned: DefinitelyTyped's minor track matches the Node minor it describes, so 22.20.x types describe APIs available on Node 22.20, while the declared floor is 22.0.0. tsc will therefore accept anything added across 22.1–22.20. That gap is inherent to the ^22.0.0 range and already existed at 22.19.17 — this PR widens it by one Node minor, it doesn't create it. DefinitelyTyped doesn't publish a floor-pinned branch, so the practical mitigation stays what it is today: the CI matrix's low end actually runs Node 22.
The realistic blast radius is tiny regardless. Everything shipped from src/ touches only long-stable surface:
process.env/process.exit—src/config.ts:4,11,17,src/index.ts:13typeof globalThis.fetch—src/client/kosli-client.ts:23,67,src/tools/execute-action.ts:5,151node:fs,node:path,node:urlinscripts/only (not inrootDir: "src")
Nothing there has changed shape in the 22.x line. skipLibCheck: true (tsconfig.json:13) also absorbs any churn inside the .d.ts files themselves, so only the surface this code names can bite. strict: true is untouched — the diff is three lines of package-lock.json.
Note on verification
I couldn't run npm ci / npm run build / npm test in this session — npm wasn't in my allowed tools, and the checkout has no node_modules. The review above is from reading the diff and the source, not from a green build. CI's test (node 22) / test (node 24) jobs cover the compile check that matters here; if they pass, that is the real signal and I'd merge on it.
Suggested follow-up
None blocking. The one thing to keep in mind for later: when the Node floor is eventually raised, @types/node should move in the same PR as engines, manifest.json, README.md, and both workflow node-version values — bumping it on its own is what the ignore rule is preventing.
· branch dependabot/npm_and_yarn/types/node-22.20.3
Bumps @types/node from 22.19.17 to 22.20.3.
Commits
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)