GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix(report): clarify scan output and package versions by jdalton · Pull Request #1614 · SocketDev/socket-cli

Conversation

@jdalton

@jdalton John-David Dalton (jdalton) commented Oct 12, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Make scan reports easier to read with a health summary and an aligned alert table. Label package versions as Version in Markdown and VERSION in terminal output.

Include tests that check versions, manifests, and URLs across default, version, file, and package folds in both formats. These tests incorporate the work from #1710.

Note

Minor change: Markdown reports now label this field Version instead of Introduced by. JSON output is unchanged.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@jdalton
John-David Dalton (jdalton) force-pushed the fix/scan-report-text-v1x branch from bf05dae to d3acac4 Compare October 27, 2026 20:40
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch from fe7d1a8 to 380118b Compare October 27, 2026 20:42
@jdalton
John-David Dalton (jdalton) force-pushed the fix/scan-report-text-v1x branch from d3acac4 to 8bb04cf Compare November 9, 2026 16:53
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 3 times, most recently from 0c7ab2d to 73b00b7 Compare November 11, 2026 16:41
@jdalton
John-David Dalton (jdalton) changed the base branch from fix/scan-report-text-v1x to v1.x November 11, 2026 16:41
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 2 times, most recently from c8e0f56 to c1d03f3 Compare November 11, 2026 16:55
This was referenced Nov 11, 2026
fix(scan): render readable report text #1605
Closed
fix(scan report): render readable text #1601
Closed
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 2 times, most recently from 24ddac7 to a164eb5 Compare November 11, 2026 17:09
@jdalton John-David Dalton (jdalton) mentioned this pull request Nov 11, 2026
test(report): keep output formats aligned across folds #1710
Merged
@jdalton

This comment was marked as outdated.

Sign in to view
@cursor

cursor Bot commented Nov 12, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes default CLI report output and renames a Markdown table column, which may break consumers parsing Introduced by; behavior of JSON output is unchanged.

Overview
Replaces default socket scan report terminal output with a formatted plain-text report (health summary, settings, aligned alert table with URLs on separate lines) instead of dumping nested Map structures via logger.dir.

Markdown reports now label the third alert column Version (was Introduced by) and URL (was url), backed by shared flattenReportAlerts / table-formatting helpers used by both Markdown and text output. JSON report shape is unchanged.

Adds broad Vitest coverage for plain text rendering (Unicode, column alignment, empty alerts, fold modes, large reports) and updates existing Markdown snapshots.

Reviewed by Cursor Bugbot for commit a164eb5. Configure here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

cursor[bot]
cursor Bot reviewed Nov 12, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit a164eb5. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@jdalton
John-David Dalton (jdalton) enabled auto-merge (squash) November 12, 2026 03:25
@jdalton John-David Dalton (jdalton) changed the title fix(report): label package versions fix(report): clarify scan output and package versions Nov 12, 2026
@jdalton
John-David Dalton (jdalton) requested a review from Vasco Santos (vasco-santos) November 12, 2026 11:36
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch 2 times, most recently from 3e3b0c7 to b9aeceb Compare November 12, 2026 15:27
@jdalton
fix(report): clarify scan output and package versions
cf93334
@jdalton
John-David Dalton (jdalton) force-pushed the jdalton/report-version-v1x branch from b9aeceb to cf93334 Compare November 12, 2026 15:53

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments
@vasco-santos Vasco Santos (vasco-santos) Awaiting requested review from vasco-santos

At least 1 approving review is required to merge this pull request.

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@jdalton
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.