Skip to content

chore: resolve open dependabot security alerts - #425

Merged
jonathannorris merged 1 commit into
mainfrom
chore/dependabot-alerts-2
Aug 27, 2026
Merged

jonathannorris merged 1 commit into
mainfrom
chore/dependabot-alerts-2

Conversation

@jonathannorris

Copy link
Copy Markdown
Member

Summary

Resolves the open Dependabot alerts that have a lockfile-level fix available. Two alerts are deferred (unresolvable without a larger migration) and one has no patched version at all — see notes below.

Dependabot Alerts Resolved

Alert Package Severity Fix
#243 js-yaml high Overrode js-yaml@>=4.0.0 <4.3.1 to ^4.3.1 (nx/eslint/cosmiconfig pin 4.3.0)
#247 js-yaml high Overrode js-yaml@>=3.0.0 <3.15.1 to ^3.15.1 (splitio 10.x/istanbul/markdown-toc pin 3.15.0)

Deferred / Unresolvable

Verified locally: build, lint, and test all pass with npm install --legacy-peer-deps --ignore-scripts (matching CI's install flags).

- js-yaml <4.3.1 -> ^4.3.1 via override (nx/eslint/cosmiconfig pin 4.3.0) (high, alert #243)
- js-yaml <3.15.1 -> ^3.15.1 via override (splitio 10.x/istanbul/markdown-toc pin 3.15.0) (high, alert #247)

Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
@jonathannorris
jonathannorris marked this pull request as ready for review August 17, 2026 14:31
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

No new commits to review since the last review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 080d8fd4-2b6d-4978-8722-6bee9158a29a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 080d8fd4-2b6d-4978-8722-6bee9158a29a

📥 Commits

Reviewing files that changed from the base of the PR and between 4814e6a and 470195c.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The package configuration replaces one js-yaml override with separate overrides for vulnerable ranges below 4.3.1 and 3.15.1.

Changes

Dependency override update

Layer / File(s) Summary
js-yaml version range overrides
package.json
The js-yaml override now covers versions below 4.3.1 and 3.15.1 with separate ranges.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 47019

This change updates vulnerable dependency resolutions in the package manifests and lockfile, with build, lint, and test checks reported as passing; no actionable merge-blocking risk remains beyond normal review and checks.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title clearly identifies the security-alert resolution, which matches the primary changes to dependency overrides.
Description check ✅ Passed The description accurately explains the resolved js-yaml alerts, deferred alerts, and validation results.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jonathannorris

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@jonathannorris

Copy link
Copy Markdown
Member Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.


Your included review limit is currently reached under our Fair Usage Limits Policy. This review may still proceed through usage-based billing if eligible. Your next included review will be available in 57 minutes.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Already reviewed.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@jonathannorris
jonathannorris requested review from aepfli, askpt, beeme1mr and toddbaert and a balanced review from Copilot August 26, 2026 15:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jonathannorris
jonathannorris added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit 6ddba35 Aug 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants