docs: badges reflect the real install path, not an empty npm package - #112
Merged
Conversation
askalf
enabled auto-merge (squash)
July 31, 2026 02:36
sprayberry-reviewer
approved these changes
Jul 31, 2026
sprayberry-reviewer
left a comment
Collaborator
There was a problem hiding this comment.
Deterministic approval — no LLM reviewed this diff.
Classified low-risk: docs-only (1 file(s)).
This repository requires an approving review before auto-merge can proceed, so this
approval exists to satisfy that gate without spending a full Code Reviewer execution
on a diff whose risk is provable from the file list alone. This approval satisfies ONLY
the review requirement — the merge still separately requires green CI, which this sweep
does not and cannot short-circuit.
Anything outside the docs allowlist, any major dependency bump, any non-Dependabot
dependency change, and any unknown diff keep the full reviewer path.
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.
The README said "Not yet on npm" three lines below two badges that pointed at
npmjs.com — and
@askalf/redstampcurrently resolves to a 0.0.1 placeholder.So the top of our most-visible security repo rendered
npm v0.0.1while theproject is at v0.7.2, and the downloads badge read from the same empty package.
Swapped both for badges that reflect how redstamp is actually installed:
npm version→github/v/release(tracks the real version)downloads→ a sigstore-signed-release badge (matches the documentedtarball +
gh attestation verifyinstall path)No text changes — "Not yet on npm" is still accurate and now the badges agree
with it. Nothing to re-verify beyond the rendered badge row.