fix(security): patch Noema document reader transitives - #2545
seonghobae merged 5 commits into
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough변경 로그는 Changes문서 판독기 전이 의존성 보안 업데이트
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The dependency updates and regression check are in place; no merge-blocking risk remains in the reviewed changes. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Exact-head repair finding — generated lock lacks a source ownerAt exact head I am repairing this at the canonical owner with a RED source-contract assertion, exact npm overrides for |
|
Exact ancestry repair completed without force.
The prior base retarget changed review comparison only; it did not make the exact PR head contain #2531. This ordinary merge fixes that topology. Local merge-tree inspection was conflict-free, added only the 11 canonical #2531 files to the first-parent tree, and passed |
Exact-head independent-review repair receiptPublished exact head The independent review found two substantive issues and both are repaired:
Exact-byte local evidence:
Fresh hosted checks and a qualifying exact-head approval remain required. This comment records repair evidence; it is not approval. |
Exact-head hosted-check revalidationAt exact head
The PR remains open, Ready, and mergeable, but ordinary merge is HOLD until the authenticated exact-head CodeQL verdict and independent approval arrive. No bypass, auto-merge, force update, rebase, or close. |
|
Exact-head readiness correction for Moving this PR to Draft / Proposed preserves every commit and valid delta while the central review/queue prerequisite is repaired. No close, force update, bypass, merge, or stale approval is performed. |
9a4af5e
into
codex/security-baseline-final-20260930
|
Stack preservation update (2026-10-01): parent #2531 now points to This PR is intentionally left open. Its head/base topology should be reconciled only after #2531's fresh exact-head security Checks and independent review establish that the carried owner repair is admissible. |
Problem
The shared Noema document reader lock on protected
mainselected vulnerable transitive releases:fast-uri 3.1.7— GHSA-hrr3-gc8f-f4qj / CVE-2026-86472ip-address 10.7.0— GHSA-j6r3-76f7-8jcv and GHSA-h3mg-xc3c-68pw / CVE-2026-101911 and CVE-2026-101912This PR is the canonical owner repair. Consumer or queue PRs must integrate this owner rather than duplicate or hide the supplier defect.
Change
fast-uri 3.1.8and the first patchedip-address 10.7.1;Exact evidence
Base prerequisite: #2531 at
516471fbe7d4e93a50c7bbba20402447f06f8d8bHead:
9a4af5e438283a31dc05814d6bc2818caee782a3Tree:
f8798f77715c627bfea13523164f769c8121860bTopology:
93433af817dd1bdd752d0491bb0246a61197e980preserves the conflict-free prerequisite integration and all prior Noema repair intent;force=false.RED:
npm audit --package-lock-only --jsonreported two vulnerable packages and the three advisories above.KeyError: 'overrides'.GREEN on the exact published bytes:
npm audit --package-lock-only --audit-level=moderate— 0 vulnerabilitiesnpm ci --ignore-scripts— 108 packages installed6cfad9df7873e018b03f72bd0154e5f32b25f9aa4ec8b87976af1d71cc56336agit diff --check— cleanIntegration
Fresh hosted Checks on exact head
9a4af5e438283a31dc05814d6bc2818caee782a3and a qualifying independent approval are required before ordinary merge. No stale predecessor result is approval evidence. After #2531 and this owner land, affected leaves must be retargeted to this owner or integrate its exact commit lineage.