Skip to content

Resolve security findings#1094

Merged
AshelyTC merged 4 commits into
mainfrom
ashelytc/security-findings
May 8, 2026
Merged

Resolve security findings#1094
AshelyTC merged 4 commits into
mainfrom
ashelytc/security-findings

Conversation

@AshelyTC
Copy link
Copy Markdown
Contributor

@AshelyTC AshelyTC commented May 8, 2026

Resolves nearly all dependabot alerts, except @typescript-eslint/parser:

It needs more working on since it's a major upgrade and conflicts with peer dependencies.

@AshelyTC AshelyTC marked this pull request as ready for review May 8, 2026 19:02
@AshelyTC AshelyTC requested a review from a team as a code owner May 8, 2026 19:02
Copilot AI review requested due to automatic review settings May 8, 2026 19:02
Copy link
Copy Markdown
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

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

Nice, thanks @AshelyTC !

@AshelyTC AshelyTC merged commit 454943c into main May 8, 2026
9 checks passed
@AshelyTC AshelyTC deleted the ashelytc/security-findings branch May 8, 2026 19:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the action’s npm dependencies/lockfile to address security findings and reduce Dependabot alerts, primarily via patch/minor bumps plus new overrides to force newer transitive dependency versions.

Changes:

  • Bumped several direct dependencies/devDependencies (e.g., ansi-styles, ts-jest, yaml, @vercel/ncc, eslint-plugin-prettier, nodemon, prettier).
  • Added overrides for undici and minimatch to enforce minimum secure versions.
  • Regenerated package-lock.json to reflect updated resolutions and new transitive packages.
Show a summary per file
File Description
package.json Updates direct dependency versions and adds overrides for vulnerable transitive packages.
package-lock.json Lockfile refresh reflecting updated dependency graph and overridden transitive resolutions.

Copilot's findings

  • Files reviewed: 1/2 changed files
  • Comments generated: 1

Comment thread package.json
Comment on lines +66 to +67
"undici": ">=6.24.0",
"minimatch": ">=9.0.7"
mergify Bot added a commit to ArcadeData/arcadedb that referenced this pull request May 10, 2026
…0 [skip ci]

Bumps [actions/dependency-review-action](https://github.com/actions/dependency-review-action) from 4.9.0 to 5.0.0.
Release notes

*Sourced from [actions/dependency-review-action's releases](https://github.com/actions/dependency-review-action/releases).*

> 5.0.0
> -----
>
> This is a new major version of the Dependency Review Action which updates the runtime to node24. This requires a minimum Actions Runner version [v2.327.1](https://github.com/actions/runner/releases/tag/v2.327.1) to run.
>
> What's Changed
> --------------
>
> * Add .github/copilot-instructions.md for Copilot coding agent by [`@​ahpook`](https://github.com/ahpook) in [actions/dependency-review-action#1067](https://redirect.github.com/actions/dependency-review-action/pull/1067)
> * Update Node.js runtime from 20 to 24 by [`@​scottschreckengaust`](https://github.com/scottschreckengaust) in [actions/dependency-review-action#1084](https://redirect.github.com/actions/dependency-review-action/pull/1084)
> * Bump spdx-license-ids from 3.0.20 to 3.0.23 by [`@​mongolyy`](https://github.com/mongolyy) in [actions/dependency-review-action#1091](https://redirect.github.com/actions/dependency-review-action/pull/1091)
> * docs: bump actions/checkout from v4 to v6 in workflow examples by [`@​Marukome0743`](https://github.com/Marukome0743) in [actions/dependency-review-action#1077](https://redirect.github.com/actions/dependency-review-action/pull/1077)
> * fix: patched version display for advisories with non-strict semver ranges (e.g. Maven beta versions) by [`@​tspascoal`](https://github.com/tspascoal) in [actions/dependency-review-action#1076](https://redirect.github.com/actions/dependency-review-action/pull/1076)
> * Resolve security findings by [`@​AshelyTC`](https://github.com/AshelyTC) in [actions/dependency-review-action#1094](https://redirect.github.com/actions/dependency-review-action/pull/1094)
> * v5.0.0 release branch by [`@​ahpook`](https://github.com/ahpook) in [actions/dependency-review-action#1098](https://redirect.github.com/actions/dependency-review-action/pull/1098)
>
> New Contributors
> ----------------
>
> * [`@​scottschreckengaust`](https://github.com/scottschreckengaust) made their first contribution in [actions/dependency-review-action#1084](https://redirect.github.com/actions/dependency-review-action/pull/1084)
> * [`@​mongolyy`](https://github.com/mongolyy) made their first contribution in [actions/dependency-review-action#1091](https://redirect.github.com/actions/dependency-review-action/pull/1091)
> * [`@​Marukome0743`](https://github.com/Marukome0743) made their first contribution in [actions/dependency-review-action#1077](https://redirect.github.com/actions/dependency-review-action/pull/1077)
>
> **Full Changelog**: <actions/dependency-review-action@v4.9.0...v5.0.0>


Commits

* [`a1d282b`](actions/dependency-review-action@a1d282b) Merge pull request [#1098](https://redirect.github.com/actions/dependency-review-action/issues/1098) from actions/ahpook/v5-release
* [`eb6c199`](actions/dependency-review-action@eb6c199) update examples to show [`@​v5`](https://github.com/v5)
* [`3943c2c`](actions/dependency-review-action@3943c2c) v5.0.0 release branch
* [`454943c`](actions/dependency-review-action@454943c) Merge pull request [#1094](https://redirect.github.com/actions/dependency-review-action/issues/1094) from actions/ashelytc/security-findings
* [`6d92a12`](actions/dependency-review-action@6d92a12) revert `@​typescript-eslint/parser` update
* [`a8e5a7e`](actions/dependency-review-action@a8e5a7e) Merge pull request [#1076](https://redirect.github.com/actions/dependency-review-action/issues/1076) from tspascoal/fix-version-matching-for-non-string-s...
* [`b6b7079`](actions/dependency-review-action@b6b7079) update `@​typescript-eslint/parser` to 8.40.0
* [`821a21d`](actions/dependency-review-action@821a21d) update more dependencies
* [`05aaaae`](actions/dependency-review-action@05aaaae) run npm audit fix
* [`55d3e75`](actions/dependency-review-action@55d3e75) Merge pull request [#1077](https://redirect.github.com/actions/dependency-review-action/issues/1077) from Marukome0743/docs/checkout
* Additional commits viewable in [compare view](actions/dependency-review-action@2031cfc...a1d282b)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=actions/dependency-review-action&package-manager=github\_actions&previous-version=4.9.0&new-version=5.0.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
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