Repository navigation
ci: add an optional npm dependency report - #253
Open
PresentJay wants to merge 1 commit into
Open
PresentJay wants to merge 1 commit into
PresentJay wants to merge 1 commit into
Conversation
Author
|
I contribute to zero-shelter. For shx, I am proposing an optional HTML view of npm advisories alongside the existing Node and operating-system compatibility checks. Would you use a report like this when reviewing the dependency tree? I can withdraw it if the separate manual workflow would mostly duplicate your current process. |
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.
shx already has a Node and operating-system compatibility matrix. I tried its committed npm tree separately to see whether a downloadable advisory report would be useful alongside those behavior checks. I contribute to zero-shelter, the tool proposed here, and used AI assistance for this change and validation.
The PR adds only
.github/workflows/dependency-report.yml. Maintainers choose when to run Dependency triage report usingworkflow_dispatch; it requestscontents: read.npm audit --package-lock-only --ignore-scriptssupplies one npm report to pinnedzero-shelter@0.0.10, which produces HTML retained for 7 days. No project dependencies are installed, no project lifecycle scripts or shx commands are run, and no baseline is accepted. Finding exit1and judgement-error exit2fail this manual job. Node 24 is used only for this report job.The stored capture at
3829f83e1498contained 30 raw records, 20 merged findings and 20 outstanding findings. Audit, JSON and HTML commands exited1. YAML, shell syntax and diff checks passed, with manifest and lockfile unchanged. This rewrite includes no fresh scan. Application tests, the compatibility matrix and a hosted manual run were not verified in that validation.The report uses npm advisories only and makes no runtime reachability claim. Audit sends package/version metadata to the configured registry.
npxdownloads and executes third-party CLI code in CI; read-only repository permissions do not remove that execution risk.Does a separate manual advisory report fit how you maintain shx? I can withdraw it if the extra workflow would be redundant.