Repository navigation
ci: add an opt-in npm dependency report - #252
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 and proposed this for Graphlib’s locked development toolchain, since the package has no regular runtime dependencies. Would a manual HTML view of those npm advisories be useful for maintenance, or would it be redundant with how you already review the tooling? |
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.
This offers a manual npm report for Graphlib’s locked development toolchain. Graphlib declares no regular runtime dependencies. I help develop zero-shelter and am trying its report in existing projects to learn whether that extra view is useful.
This adds only Dependency triage report, a separate
workflow_dispatchworkflow. It reads npm audit JSON with--package-lock-only --ignore-scripts, renders it usingzero-shelter@0.0.10, and retains an HTML artifact for seven days. It requestscontents: read, installs no project dependencies and accepts no baseline findings. Finding exit1and judgement-error exit2are both failures.At base
0438a8f5c33d, the recorded 2026-11-02 UTC local check produced 4 raw / 4 merged / 4 outstanding / 0 accepted findings. npm audit and both JSON/HTML commands exited1; a synthetic scanner failure returned2. YAML, shell syntax and whitespace checks passed, and manifest/lockfile hashes were unchanged. This used only npm audit input and has not been rescanned. Graphlib's tests/build and the new hosted manual workflow were not run.npm audit contacts the configured registry, and
npxdownloads and executes third-party code in CI. Read-only permissions limit repository access but do not remove that trust boundary. Reachability and exploitability were not assessed. I used AI assistance for this PR.Would this report be useful for reviewing the development toolchain? I’m happy to withdraw it if another workflow would add little to your current process.