Skip to content

chore: refresh deps and require Node 22 - #235

Merged
SutuSebastian merged 1 commit into
mainfrom
chore/upgrade-packages-node22
Aug 24, 2026
Merged

chore: refresh deps and require Node 22#235
SutuSebastian merged 1 commit into
mainfrom
chore/upgrade-packages-node22

Conversation

@SutuSebastian

@SutuSebastian SutuSebastian commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Refresh exact pins: better-sqlite3 13.0.3, oxc-parser 0.147.0, MCP SDK 1.30, oxlint/oxfmt, and related toolchain. Lockfile regenerated on Bun 1.4.0.
  • Published engines.node is now >=22.12.0. better-sqlite3 13 sets engines.node to >=22; keeping Node 20 in Codemap engines would let Node 20 installs fail at the native addon.
  • Heritage extractor skips oxc 0.146+ dummy recoveries of invalid interface extends (A | B) (zero-span ThisExpression + TS2499). Real complex class extends still record an unresolved row.
  • Audit overrides pin patched hono, @hono/node-server 1.19.17 (not the 2.x line), and postcss. CI still ignores the two unfixed image-size highs from omitted Blume.
  • TypeScript stays on 6.0.3. 7.0.2 removes the compiler main export (SyntaxKind is undefined), which breaks TypeDoc 0.28 (peer: 5.x || 6.0.x).

Patch changeset included. Do not hand-edit package.json version — the Version packages PR bumps it after merge.

Test plan

  • bun run build (tsdown target node22.12.0)
  • bun run typecheck
  • bun run test / test:scripts / test:golden / test:agent-eval
  • bun run format:check / lint:ci / dedupe:check
  • CI-equivalent bun audit --ignore=GHSA-w3rx-r6r6-pgpr --ignore=GHSA-5p2g-fcmc-qvqq
  • CI green on this PR
  • After merge, Version packages PR shows 0.11.6 → 0.11.7

Summary by CodeRabbit

  • Compatibility

    • Node.js 22.12 or newer is now required; Node.js 20 is no longer supported.
    • Bun 1.0.31 or newer is required.
  • Bug Fixes

    • Improved handling of invalid interface inheritance syntax during code analysis.
    • Preserved unresolved class inheritance expressions instead of silently dropping them.
  • Documentation

    • Updated runtime and SQLite compatibility guidance.
  • Maintenance

    • Refreshed development and runtime tooling for improved compatibility and security.

better-sqlite3 13 dropped Node 20; keep published engines honest and skip oxc dummy heritage recoveries.
@changeset-bot

changeset-bot Bot commented Aug 24, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e967198

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@stainless-code/codemap Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 98425579-5f89-440c-8f68-605ccfb82a10

📥 Commits

Reviewing files that changed from the base of the PR and between bbebe12 and e967198.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (8)
  • .changeset/refresh-deps-node22.md
  • .github/CONTRIBUTING.md
  • README.md
  • docs/packaging.md
  • package.json
  • src/db.ts
  • src/extractors/heritage.test.ts
  • src/extractors/heritage.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The project now requires Node.js 22.12+, refreshes runtime and development dependencies, and updates SQLite compatibility documentation. Heritage extraction now skips recovered invalid interface expressions and records unresolved class heritage expressions.

Changes

Runtime refresh and heritage extraction

Layer / File(s) Summary
Runtime and dependency refresh
package.json, .changeset/refresh-deps-node22.md, .github/CONTRIBUTING.md, README.md, docs/packaging.md, src/db.ts
The package requires Node.js >=22.12.0. Dependency versions, security overrides, and SQLite compatibility documentation were updated.
Recovered heritage handling
src/extractors/heritage.ts, src/extractors/heritage.test.ts
The extractor skips dummy expressions from invalid interface heritage and records unresolved non-dummy expressions as "(expression)". Tests cover both behaviors.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to e9671

This change refreshes dependencies, requires Node 22.12.0, and updates a localized extractor compatibility case. No actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the dependency refresh and the Node.js 22 requirement, which are the primary changes.
Docstring Coverage ✅ Passed Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/upgrade-packages-node22

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@SutuSebastian
SutuSebastian merged commit edd1c36 into main Aug 24, 2026
25 of 27 checks passed
@SutuSebastian
SutuSebastian deleted the chore/upgrade-packages-node22 branch August 24, 2026 11:59
@github-actions github-actions Bot mentioned this pull request Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant