fix(package): correct mislabeled header in deep score markdown#1430
Open
John-David Dalton (jdalton) wants to merge 1 commit into
Open
fix(package): correct mislabeled header in deep score markdown#1430John-David Dalton (jdalton) wants to merge 1 commit into
John-David Dalton (jdalton) wants to merge 1 commit into
Conversation
Collaborator
Author
|
CI note (both red checks are pre-existing, not from this PR):
|
John-David Dalton (jdalton)
force-pushed
the
fix/1356-deep-score-header
branch
3 times, most recently
from
July 25, 2026 16:17
b889740 to
730b409
Compare
The transitive section of `socket package score --markdown` emitted `### Capabilities` twice. The first was mislabeled: its body lists the lowest-scoring package per score category, not capabilities. Rename that header to `### Lowest Scoring Package Per Category` so the two sections are distinct and correctly labeled. Snapshots updated. Fixes #1356
John-David Dalton (jdalton)
force-pushed
the
fix/1356-deep-score-header
branch
from
July 25, 2026 17:42
730b409 to
9f202cf
Compare
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.
What
socket package score --markdownprinted### Capabilitiestwice in the Transitive Package Results section, back to back. The first one was mislabeled — its body is the lowest-scoring package per score category, not capabilities.This renames that first header to
### Lowest Scoring Package Per Category, so the two sections are distinct and each header matches its body.Fixes #1356.
Why
The duplicated heading is confusing in the rendered report (and produces duplicate anchors). The mislabel also means the report claimed a "Capabilities" section that actually described low-scoring packages.
Change
Before / after (transitive section)
Before:
After:
src/commands/package/output-purls-deep-score.mts— rename the header string.Testing
vitest run test/unit/commands/package/— 144 passed.pnpm run lint <file>— passed.tsc -p tsconfig.json --noEmit— no errors in the changed file.One unrelated pre-existing failure (
test/unit/constants.test.mts > has correct path properties) is present on a cleanorigin/maintoo (environment-dependent path assertion); not touched by this change.Note
Low Risk
String-only markdown heading change with snapshot test updates; no scoring or API behavior changes.
Overview
Fixes duplicate
### Capabilitiesheadings in the transitive section ofsocket package score --markdownoutput.The block that lists lowest-scoring packages per score category (after Deep Score) is now titled
### Lowest Scoring Package Per Category, so it no longer shares a heading with the real capabilities list. Unit test inline snapshots were updated to match.Reviewed by Cursor Bugbot for commit c25647a. Configure here.