chore(deps): bump json gem to 2.21.2 (Dependabot #53) - #37
Conversation
Address Dependabot alert #53 by updating the transitive json dependency from 2.20.0 to 2.21.2 via `bundle update json`. Fixes heap-use-after-free in JSON::ResumableParser (GHSA-9hj4-r449-hfvc). Co-authored-by: Vatsal Sanjay | CoMPhy Lab <[email protected]>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (1)
Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📜 Recent review details⏰ Context from checks skipped due to timeout. (2)
🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe ChangesDependency override
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This PR updates dependencies and reports a successful Jekyll build; no actionable merge-blocking risk remains beyond normal CI checks and confirming the security alert is resolved. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches✨ Simplify code
Comment |
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
The npm override `js-yaml: >=4.3.1` resolved to js-yaml 5.x, which removed the default export that markdownlint-cli2 expects. Pin to 4.3.1 to keep security fixes while restoring `npm run lint:md` in CI. Co-authored-by: Vatsal Sanjay | CoMPhy Lab <[email protected]>
Description
Security hygiene PR to resolve Dependabot alert #53 for the transitive Ruby
jsongem inGemfile.lock.Also fixes a pre-existing CI failure on
mainwhere the npmjs-yamlsecurity override (>=4.3.1) resolved to js-yaml 5.x, breakingmarkdownlint-cli2ESM imports duringnpm run lint:md.Type of Change
Changes Made
bundle update jsonto bumpjsonfrom 2.20.0 → 2.21.2js-yamlfrom>=4.3.1to4.3.1so markdownlint-cli2 keeps a default ESM export (js-yaml 5.x removed it)package-lock.jsonaccordinglyjsonis a transitive dependency viafaraday→jekyll-gist→octokit; it is not pinned in the GemfileVulnerability
JSON::ResumableParser#partial_valueaffectingjson2.20.0–2.21.1Major-version judgment
Not required. This is a patch-level bump within the 2.x line (2.20.0 → 2.21.2). No Gemfile changes needed.
Testing Done
bundle update jsoncompleted successfullybundle exec jekyll buildsucceeds locallynpm ci && npm run lint:mdpasses locallybundle exec jekyll serveRelated Issues
Addresses Dependabot alert #53.
Checklist
Additional Notes
CI (
Jekyll site CI) is green on this PR. Do not merge until Dependabot marks alert #53 resolved.Summary by CodeRabbit