Skip to content

Scrub and hide the diff-scan command#350

Merged
pvdz merged 2 commits intomainfrom
diffscan_v2
Mar 4, 2025
Merged

Scrub and hide the diff-scan command#350
pvdz merged 2 commits intomainfrom
diffscan_v2

Conversation

@pvdz
Copy link
Contributor

@pvdz pvdz commented Mar 4, 2025

Updates the diff-scan command;

  • Fix the endpoint being fetched
  • Hide the command
    • In its current form it's not likely to be very useful to users. Additionally, the command was broken up to this point because it was trying to reach an endpoint that didn't exist. We'll probably want to expose this again once we solve the UX around it, and even then, probably under the scan command.
  • Support a --depth flag to control how deep a JSON structure it can print
  • Fixup the --json and `--file flags
  • Support a - value for --file to explicitly dump the raw JSON to stdout
    • This way you can pipe the json to another command like socket diff-scan get | jq
  • Removed the flag for --markdown, I'm not sure it makes sense for this command. Or maybe it does but it would emit a high overview sort of report instead. Either way, that would need some more thinking and work to implement so it's gone for now.
  • Removed the --preview flag.

Add small state notification to the dependencies view.
Add shortcut commands to package.json to ease development.

@pvdz pvdz requested a review from jdalton March 4, 2025 10:42
@pvdz pvdz merged commit 80c1073 into main Mar 4, 2025
15 checks passed
@pvdz pvdz deleted the diffscan_v2 branch March 4, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants