Skip to content

http: reduce raw header name allocations Reuse the values array instead of allocating a second result array. Signed-off-by: Roberto Bianchi <[email protected]> PR-URL: https://github.com/nodejs/node/pull/66488 Reviewed-By: Gürgün Dayıoğlu <[email protected]> Reviewed-By: Tim Perry <[email protected]> - #1

Merged
YouRMum707 merged 15 commits into
actions/tools-update-googletestfrom
main
Oct 31, 2026

Conversation

@YouRMum707

Copy link
Copy Markdown
Owner

No description provided.

lemire and others added 15 commits October 11, 2026 03:36
PR-URL: nodejs#66620
Refs: simdjson/simdjson#2913
Refs: nodejs#66495
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
env.h includes inspector_profiler.h, which included simdjson.h for
the types of two virtual methods and a parser member. As a result,
about 155 files in src/ parsed the amalgamated simdjson header,
which is over 10 MB.

Keep simdjson out of inspector_profiler.h: the profile connections
now say which field of `result` holds the profile, and
WriteProfile() takes the profile as raw JSON. The parser becomes a
local variable, so its buffers are no longer kept alive for the
lifetime of the Environment.

Also drop the unused include from node_task_runner.h, and let the
config file reader rewind its document instead of parsing the whole
file a second time with a second parser.

Now only the eight files that use simdjson include it. On an Apple
M-series machine, a clean rebuild of the src/ objects takes about
15% less CPU time (376 s to 320 s) and 19% less wall time at -j14.

Signed-off-by: Daniel Lemire <[email protected]>
PR-URL: nodejs#66620
Refs: simdjson/simdjson#2913
Refs: nodejs#66495
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
traceCallback() defaults to a frozen context but writes the callback's
result or error to it. Allocate a fresh context when undefined is passed
and subscribers are present, matching traceSync() and tracePromise().
Set the prototype after allocation to avoid dictionary-mode properties.

Assisted-by: pi
Signed-off-by: Matteo Collina <[email protected]>
PR-URL: nodejs#66481
Reviewed-By: Bryan English <[email protected]>
The GHA benchmark never built the addons of benchmark/napi, and its
--debug-node build made benchmark/common.js look for them in
build/Debug. Build them when the napi category runs, and pick the
build type as test/common does.

Refs: nodejs#66395
Signed-off-by: Nigro Simone <[email protected]>
PR-URL: nodejs#66423
Reviewed-By: Antoine du Hamel <[email protected]>
Original commit message:

    fix compilation with GCC 16 on arm64

    Change-Id: I2cbe5f2a68d4a69bac48eea876ff0ed0cb589171
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/8500805
    Reviewed-by: Joyee Cheung <[email protected]>
    Commit-Queue: Joyee Cheung <[email protected]>
    Reviewed-by: Anton Bikineev <[email protected]>
    Cr-Commit-Position: refs/heads/main@{#110517}

Refs: v8/v8@8d14d49
PR-URL: nodejs#66627
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Signed-off-by: Mike McCready <[email protected]>
PR-URL: nodejs#65750
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Stefan Stojanovic <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gürgün Dayıoğlu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ulises Gascón <[email protected]>
Signed-off-by: lazerg <[email protected]>
Assisted-by: Claude Code
PR-URL: nodejs#66565
Fixes: nodejs#66562
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>
Bumps the doc group in /tools/doc with 2 updates: [@doc-kit/generator-react](https://github.com/nodejs/doc-kit/tree/HEAD/packages/react) and [@node-core/doc-kit](https://github.com/nodejs/doc-kit/tree/HEAD/packages/node).

Updates `@doc-kit/generator-react` from 0.4.0 to 0.4.2
- [Release notes](https://github.com/nodejs/doc-kit/releases)
- [Changelog](https://github.com/nodejs/doc-kit/blob/main/packages/react/CHANGELOG.md)
- [Commits](https://github.com/nodejs/doc-kit/commits/@doc-kit/[email protected]/packages/react)

Updates `@node-core/doc-kit` from 2.0.4 to 2.0.5
- [Release notes](https://github.com/nodejs/doc-kit/releases)
- [Changelog](https://github.com/nodejs/doc-kit/blob/main/packages/node/CHANGELOG.md)
- [Commits](https://github.com/nodejs/doc-kit/commits/@node-core/[email protected]/packages/node)

---
updated-dependencies:
- dependency-name: "@doc-kit/generator-react"
  dependency-version: 0.4.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: doc
- dependency-name: "@node-core/doc-kit"
  dependency-version: 2.0.5
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: doc
...

Signed-off-by: dependabot[bot] <[email protected]>
PR-URL: nodejs#66632
Reviewed-By: Aviv Keller <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: nodejs#66497
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ulises Gascón <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Signed-off-by: Antoine du Hamel <[email protected]>
PR-URL: nodejs#66626
Reviewed-By: Ulises Gascón <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
AbortSignal.any() copies the source WeakRefs of a composite input into
the new composite, so nested composites share them. Those WeakRefs were
also used as the unregister tokens for the registrations that prune a
composite's sources once they are collected. When an inner composite
was collected first, its cleanup therefore also unregistered the outer
composite from the same sources. An observed outer composite then
stayed in gcPersistentSignals, together with its listeners, even after
all of its sources were gone and nothing could abort it anymore.

Use each composite's own WeakRef as the unregister token instead, so
collecting one composite only removes its own registrations.

Fixes: nodejs#66612
Refs: nodejs#62367
Assisted-by: Claude Code
Signed-off-by: journaltraces <[email protected]>
PR-URL: nodejs#66617
Reviewed-By: Chemi Atlow <[email protected]>
Reviewed-By: Gürgün Dayıoğlu <[email protected]>
Pulls in nodejs/node-pr-labeler#20, which counts each subsystem once
before applying the four-subsystem limit that collapses labels into
`lib / src`. Until now any PR touching three or more files under a
two-label rule (e.g. `src/quic/`, `src/crypto/`) was collapsed even
though it only touched one subsystem.

Signed-off-by: Soul Lee <[email protected]>
PR-URL: nodejs#66606
Refs: nodejs/node-pr-labeler@dfe5eb3
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Add deps/v8/** to the QUIC workflow's pull request path filter so
V8 updates trigger its build and tests.

Assisted-by: Codex
Signed-off-by: inoway46 <[email protected]>
PR-URL: nodejs#66602
Refs: nodejs#66537
Refs: https://github.com/nodejs/node/actions/runs/37675498427/job/112978479932
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
Signed-off-by: Vas Sudanagunta <[email protected]>
PR-URL: nodejs#66563
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reuse the values array instead of allocating a second result array.

Signed-off-by: Roberto Bianchi <[email protected]>
PR-URL: nodejs#66488
Reviewed-By: Gürgün Dayıoğlu <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
@YouRMum707

Copy link
Copy Markdown
Owner Author

No modifications made by me.

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

Labels

documentation Improvements or additions to documentation

Development

Successfully merging this pull request may close these issues.