Repository navigation
ci: add an opt-in npm dependency report - #111
Closed
PresentJay wants to merge 1 commit into
Closed
PresentJay wants to merge 1 commit into
PresentJay wants to merge 1 commit into
Conversation
Author
|
I work on zero-shelter and am trying its manual report for basic-auth’s locked development toolchain. This is an npm advisory view alongside the existing CodeQL and Scorecard checks, rather than a test of authentication behavior. Would it be useful in your maintenance workflow, or mostly duplicate your current review? |
Member
|
Not at this point in time, thanks! I don't think it'd be something I run, and it'd be an extra thing to maintain. |
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.
basic-auth already has CodeQL, Scorecard and release checks. I contribute to zero-shelter and proposed a trial of its manual HTML report for the package's locked development toolchain; the manifest declares no regular runtime dependencies.
The sole changed file is
.github/workflows/dependency-report.yml. It runs only throughworkflow_dispatch, requestscontents: read, and uses SHA-pinned official actions plus pinnedzero-shelter@0.0.10.npm audit --package-lock-only --ignore-scriptssupplies the input; generated HTML is retained for 7 days. Finding exit1and scanner/judgement-error exit2both fail the job. It installs no project dependencies, accepts no baseline and changes no existing gate or publishing behavior.A saved capture from 2 November 2026 at
301fe415bd5cproduced 17 raw / 17 merged / 17 outstanding findings, 0 accepted. npm audit and JSON/HTML exited1; a failed-scanner control exited2. YAML, Bash and whitespace checks passed, and manifest/lockfile hashes were unchanged. These are dated results, not a new scan. Application test/build checks and the new hosted manual workflow remain untested.Only npm audit is used; authentication correctness and code reachability are untested. npm audit sends dependency metadata to the configured registry, and npx downloads and executes the third-party CLI in CI. I used AI assistance to prepare this work and description.
Would a manual report fit your current maintenance workflow? I'm happy to withdraw it if it duplicates what you already use.