Node v24.15.0 nsolid v6.2.4 release

[santigimeno]

Summary by CodeRabbit

• Bug Fixes

  • Fixed HTTP/2 request header handling with improved validation and processing.
  • Resolved a merge error in the library module.

• Chores

  • Updated version marking for release build status.
  • Added changelog entry for version 24.15.0-nsolid-v6.2.4.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 64170c9d-cf6d-4141-a59a-446d5a4ab771

📥 Commits

Reviewing files that changed from the base of the PR and between f397c0b and 48b6bf4.

📒 Files selected for processing (3)

• doc/changelogs/NSOLID_CHANGELOG_V6_NODE_V24.md
• lib/internal/http2/core.js
• src/node_version.h

---

Walkthrough

This PR prepares a new release (v24.15.0-nsolid-v6.2.4) by updating the changelog, fixing an HTTP/2 request header handling issue from a merge conflict, and marking the build as a release version by flipping the NSOLID_VERSION_IS_RELEASE macro.

Changes

Cohort / File(s)	Summary
Release Configuration doc/changelogs/NSOLID_CHANGELOG_V6_NODE_V24.md, src/node_version.h	Changelog entry added for v24.15.0-nsolid-v6.2.4 with commit reference; NSOLID_VERSION_IS_RELEASE flipped from 0 to 1 to mark as release build.
HTTP/2 Request Handler lib/internal/http2/core.js	Refactored ClientHttp2Session.request() to use explicit control flow instead of conditional expression when handling header updates; recomputes headers with stricter validation when context.headersUpdated is true.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• 2025-12-08, Version 24.11.1-nsolid-v6.1.1 'Krypton' #397: Directly related—both PRs flip NSOLID_VERSION_IS_RELEASE from 0 to 1 in src/node_version.h.
• 2025-09-17, Version 22.18.0-nsolid-v6.0.1 'Jod' #365: Directly related—both modify the same macro to mark as release version.
• Node v22.15.1 nsolid v5.7.4 release #332: Related through src/node_version.h modifications, though this PR bumps patch version instead.

Suggested reviewers

• RafaelGSS
• juanarbol

Poem

🐰 A release takes shape with changelog's quill,
HTTP headers merge'd with careful skill,
The version flag waves—we're production-bound,
Six-point-two-point-four, a stable round!
Hop along, Node twenty-four, let's fly! 🚀

🚥 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 directly and clearly describes the main purpose of the pull request: preparing the release of Node v24.15.0 with nsolid v6.2.4, which aligns with all three modified files (changelog entry, version header update, and merge fix).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch node-v24.15.0-nsolid-v6.2.4-release

---

_{Review rate limit: 3/5 reviews remaining, refill in 22 minutes and 15 seconds.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}