Skip to content

Update to node 24.22.1 as well as bump the deps - #410

Open
Bohreromir wants to merge 7 commits into
mainfrom
chore/node-24.22.1
Open

Bohreromir wants to merge 7 commits into
mainfrom
chore/node-24.22.1

Conversation

@Bohreromir

Copy link
Copy Markdown
Member

Description

Update to node 24.22.1 as well as bump the deps

Docs

Release note

Checklist

Author

  • AI review done, findings addressed
  • Tests added or updated
  • Changelog updated (if the repo keeps one)

Reviewer

  • Useful comments and logging
  • Docs added or updated per the style guide

Acceptance

  • Check for release notes and docs

Node.js 24.22.1 fixes the Brotli decoder buffer overflow (Brotli
1.0.3-1.2.0). Node's fetch decodes Content-Encoding: br responses from
upstream services automatically.

Bump dependencies to the latest versions within their semver ranges.
Held back:
- better-auth / @better-auth/drizzle-adapter on ~1.6.33: 1.7.0 drops
  genericOAuthClient from better-auth/client/plugins
- tar-stream on 3.2.0: 3.2.1+ ships its own typings that conflict with
  @types/tar-stream and archive.ts

Unpin @smithy/node-http-handler so it dedupes with the AWS SDK's copy.
Only breaking change splits audioWorklet globals out of browser; not used here.
Requires Prettier >= 3.7 (we have 3.9). No formatting changes.
9.0 drops Node 20 and replaces Express with plain Node HTTP handlers. The E2E mock only reads req.headers, which is unchanged.
TypeScript 7 is not an option yet: it ships no compiler API, and svelte-check, svelte2tsx, typescript-eslint and SvelteKit 2 all cap their peer range at 6.
3.2.1 ships its own typings, so drop @types/tar-stream and adapt archive.ts:
Header instead of Headers, the entry stream type from ExtractEvents, inferred
entry handler parameters, and an explicit Buffer cast for streamx data
events. No runtime changes between 3.2.0 and 3.2.2.
0.56 reorganised the package exports, so the deep esm/ imports no longer
resolve. Switch the editor worker and German NLS bundle to the public
export paths; the NLS bundle now ships types, so drop the ts-expect-error.

Monaco 0.57 pins DOMPurify to 3.4.15. Raise the override to ^3.4.17 for the
latest sanitisation fixes and correct the stale note.
@github-actions

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 64.72% 8706 / 13451
🔵 Statements 63.83% 11709 / 18342
🔵 Functions 66.1% 2787 / 4216
🔵 Branches 54.93% 4999 / 9100
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/lib/components/editor/MonacoEditor.svelte 0% 0% 0% 0% 26-323
src/lib/components/editor/TextEditor.svelte 68.93% 51.18% 84% 77.41% 41, 116, 117, 118, 120, 121, 122, 123, 124, 125, 126, 127, 128, 129, 130, 131, 132, 133, 134, 138, 164, 169, 194, 210-216, 224-236
src/lib/server/storage/archive.ts 80.04% 64.83% 96.87% 81.06% 119-122, 134-138, 150-161, 181, 191, 206-207, 216, 256, 266, 280-290, 378, 416, 468, 474, 477-484, 516, 540, 544-546, 557-559, 563-568, 573-580, 632, 659-661, 670-671, 680-681, 718, 747-749, 758-759, 768-769, 819, 892-896, 959, 968, 1039, 1068-1072, 1088-1098, 1109
Generated in workflow #1152 for commit f8c2c95 by the Vitest Coverage Report Action

@Bohreromir Bohreromir self-assigned this Oct 26, 2026
@Bohreromir
Bohreromir marked this pull request as ready for review October 26, 2026 15:13
@Bohreromir Bohreromir moved this to Development: Waiting for Review in Stackable Engineering Oct 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Waiting for Review

Development

Successfully merging this pull request may close these issues.

1 participant