Skip to content

ci(quality): --ignore-npm-errors on cyclonedx SBOM step - #110

Merged
rubenvdlinde merged 1 commit into
mainfrom
fix/sbom-npm-errors
Jul 25, 2026
Merged

ci(quality): --ignore-npm-errors on cyclonedx SBOM step#110
rubenvdlinde merged 1 commit into
mainfrom
fix/sbom-npm-errors

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

cyclonedx-npm aborts on benign npm ls ELSPROBLEMS (a nested @vueuse/core pulled by @nextcloud/dialogs that mismatches a peer range) even with --package-lock-only. The tree quirk doesn't affect the lockfile-derived SBOM, so tolerate it rather than red the whole quality run for every fleet repo consuming this reusable workflow. Mirrors Codeberg Conduction/.github#70. Fixes OpenRegister's standing Code Quality / SBOM red.

cyclonedx-npm still shells out to `npm ls` internally even with
--package-lock-only, and aborts (ELSPROBLEMS) on benign transitive tree
quirks — e.g. a nested @vueuse/core pulled by @nextcloud/dialogs that
mismatches a peer range. The quirk does not affect the lockfile-derived SBOM,
so tolerate it rather than red the whole quality run for every fleet repo.
@rubenvdlinde
rubenvdlinde merged commit 0b30678 into main Jul 25, 2026
3 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.

1 participant