chore(deps): cyclonedx-npm 6.0.0 — clears the last remaining npm high - #2373
Merged
Conversation
… npm high) GHSA shell injection via an unsanitised --workspace argument; affected 2.1.0 - 4.2.1. This was the only remaining high-severity npm advisory on development after the php_codesniffer CVE fix. npm audit --package-lock-only on development: before: 0 critical, 1 high after: 0 critical, 0 high Reachability: this repo passes no --workspace, so the advisory was not exploitable here. Bumped anyway because it is cheap and the SBOM step runs this binary directly. Verified (CI is node 20 / npm 10.8.2; local npm 11 prunes entries CI needs, so the lockfile was regenerated with npx npm@10.8.2) - npx npm@10.8.2 ci: rc=0, lockfile md5 identical afterwards. - The bump IS exercised: CI runs 'npx @cyclonedx/cyclonedx-npm', which resolves the local devDependency. That binary now reports 6.0.0, and the EXACT CI command succeeded (rc=0) producing a valid CycloneDX specVersion 1.5 SBOM with 652 components — identical to the 4.2.1 baseline, so the major bump changed the tool without changing the artifact. - npm run build rc=0; npm run lint rc=0. Both baselined on untouched development first. - check:specs rc=0, test:l10n rc=0, jest (npm test) rc=0 - Licence read from the LOCKFILE: cyclonedx-npm 6.0.0 is Apache-2.0. vue3-apexcharts is 1.8.0 MIT, still the pre-proprietary line, untouched.
Contributor
Quality Report — ConductionNL/openregister @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| build | ✅ | ||||
| check-specs | ✅ | ||||
| test-l10n | ✅ | ||||
| composer | ✅ | ✅ 173/173 | |||
| npm | ✅ | ✅ 713/713 | |||
| PHPUnit | ✅ | ||||
| Newman | ✅ | ||||
| Playwright | ✅ | ||||
| Hydra gates | ✅ |
Quality workflow — 2026-08-06 13:35 UTC
Download the full PDF report from the workflow artifacts.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@cyclonedx/cyclonedx-npm4.2.1 → 6.0.0 — shell injection via an unsanitised--workspaceargument (affected2.1.0 - 4.2.1). This was the only remaining high-severity npm advisory ondevelopmentafter the php_codesniffer CVE fix (#2371).npm audit --package-lock-onlyondevelopment:Reachability: this repo passes no
--workspace, so the advisory was not exploitable here. Bumped anyway because it is cheap and the SBOM step runs this binary directly.Verification
CI is node 20 / npm 10.8.2; local npm 11 prunes entries CI requires, so the lockfile was regenerated with
npx npm@10.8.2.npx npm@10.8.2 ci→ rc=0, lockfile md5 identical afterwards.npx @cyclonedx/cyclonedx-npm, which resolves the local devDependency — that binary now reports 6.0.0. The exact CI command succeeded (rc=0), producing a valid CycloneDX specVersion 1.5 SBOM with 652 components — identical to the 4.2.1 baseline, so the major bump changed the tool without changing the artifact.npm run buildrc=0 ·npm run lintrc=0 ·check:specsrc=0 ·test:l10nrc=0 · jest (npm test) rc=0. All baselined on untoucheddevelopmentfirst.vue3-apexchartsis 1.8.0 MIT — still the pre-proprietary line, untouched here.Note:
npm run buildalso rewritesdocs/features.jsonand removes a stalejs/bundle; those were reverted so this PR contains onlypackage.json+package-lock.json.