fix #423 - bug: Enable Dependabot security updates - #424
Conversation
Signed-off-by: Fabrizio Antonangeli <fabrizio.antonangeli@gmail.com>
Signed-off-by: Fabrizio Antonangeli <fabrizio.antonangeli@gmail.com>
✅ Deploy Preview for openworkflow-editor canceled.
|
There was a problem hiding this comment.
🟡 Changes recommended
Bound the fast-uri overrides to the supported 3.x release line and consolidate the selectors.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
This pull request configures grouped Dependabot security updates and applies dependency overrides to address vulnerable transitive packages.
Changes:
- Groups npm and GitHub Actions security updates.
- Adds pnpm dependency overrides.
- Updates the lockfile with revised resolutions.
File summaries
| File | Summary |
|---|---|
pnpm-workspace.yaml |
Adds overrides, but the fast-uri ranges must remain bounded to major version 3. |
pnpm-lock.yaml |
Records the resulting dependency resolutions. |
.github/dependabot.yml |
Configures grouped security updates. |
Review details
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
Suppressed comments (1)
pnpm-workspace.yaml:77
- The unbounded replacement range resolves this override to
nanoid@6.0.1in the lockfile, even though the consuming PostCSS toolchain was on the 3.x line. This is a major-version substitution outside the consumer's semver contract and can break the Vite/PostCSS build; constrain the security override to the patched 3.x range instead of allowing the latest major.
nanoid@<3.3.18: ">=3.3.18"
- Files reviewed: 2/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
…pull/424/changes#r4004233053 Signed-off-by: Fabrizio Antonangeli <fabrizio.antonangeli@gmail.com>
Signed-off-by: Fabrizio Antonangeli <fabrizio.antonangeli@gmail.com>
There was a problem hiding this comment.
🟡 Changes recommended
The critical fast-uri override still permits vulnerable versions, so its range must be corrected before approval.
Get a fresh assessment by requesting another Copilot review.
Review details
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
- Files reviewed: 2/3 changed files
- Comments generated: 1
- Review effort level: Lite
Signed-off-by: Fabrizio Antonangeli <fabrizio.antonangeli@gmail.com>
There was a problem hiding this comment.
🟢 Approval recommended
No unresolved blocking issues were identified in the reviewed changes.
Review details
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
- Files reviewed: 2/3 changed files
- Comments generated: 0 new
- Review effort level: Lite
|
Hi @ricardozanini we would need you to check the configuration in the PR description as we don't have permissions to access the security settings |
|
@fantonangeli I haven't changed anything and I remember turning this on once I created the repo:
|
@ricardozanini Thanks for checking. We should have everything we need now |
17131c9
into
open-workflow-specification:main

Closes #423
❗ Maintainer action required @ricardozanini
Please verify that Dependabot security updates are enabled for this repository:
Describe the Bug
The repository currently has several open security alerts, including high-severity vulnerabilities.
The current Dependabot configuration only groups regular version updates.
Steps to reproduce
How to test:
pnpm audit