Skip to content

Conversation

@mtorp
Copy link
Contributor

@mtorp mtorp commented Dec 2, 2025

Adds the options --reach-version and --fix-version to override the default Coana version used in the CLI.
Updates the CLI to version 1.1.40.


Note

Adds --reach-version and --fix-version flags to override the @coana-tech/cli version, wiring the value through to Coana execution across reachability and fix flows.

  • CLI Flags
    • socket scan create / socket scan reach: new --reach-version to override @coana-tech/cli version.
    • socket fix: new --fix-version to override @coana-tech/cli version.
  • Execution Layer
    • spawnCoanaDlx in src/utils/dlx.mts accepts coanaVersion and uses it when invoking @coana-tech/cli.
    • Plumb reachVersion/coanaVersion through perform-reachability-analysis, cmd-scan-*.mts, and fix pipeline (cmd-fix.mtshandle-fix.mtscoana-fix.mts).
  • Defaults & CI
    • Include reachVersion (default undefined) in CI and GitHub scan creation configs.
  • Docs/Help & Tests
    • Update help text and snapshots to document new flags.
    • Test utilities normalize Coana version in outputs for stable snapshots.
  • Changelog/Version
    • Bump to 1.1.40 and document new flags in CHANGELOG.md.

Written by Cursor Bugbot for commit aaba0a2. Configure here.

@mtorp mtorp changed the title add --reach-version and --fix-version flags to override the default c… add --reach-version and --fix-version flags to override the default Coana version Dec 2, 2025
@mtorp
Copy link
Contributor Author

mtorp commented Dec 2, 2025

@cursor review

@mtorp mtorp requested a review from jdalton December 2, 2025 07:58
Copy link

@cursor cursor bot left a 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 bugs!


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

@jdalton
Copy link
Contributor

jdalton commented Dec 2, 2025

@mtorp How does this mesh with the process.env['SOCKET_CLI_COANA_LOCAL_PATH'] stuff? Is that more for development and this flag more user-facing? I like the flag, also versioned reachability is kind of a cool thing to have.

@mtorp
Copy link
Contributor Author

mtorp commented Dec 2, 2025

@mtorp How does this mesh with the process.env['SOCKET_CLI_COANA_LOCAL_PATH'] stuff?

Yea, that's a dev only feature.

I think 99% of all users will never use any of these flags, but they are nice to have in case we want to quickly bump the Coana version to fix a bug, without having to wait for a Socket CLI release.

@mtorp mtorp requested a review from jdalton December 2, 2025 15:26
@jdalton jdalton merged commit eeeb240 into v1.x Dec 2, 2025
7 checks passed
@jdalton jdalton deleted the add-coana-version-flags branch December 2, 2025 16:53
jdalton added a commit that referenced this pull request Dec 9, 2025
Ported from v1.x commit eeeb240 (#957)

- Add --fix-version flag to override default @coana-tech/cli version
- Add CoanaDlxOptions type extending DlxOptions with coanaVersion
- Update spawnCoanaDlx to accept and use coanaVersion parameter
- Pass coanaVersion through cmd-fix -> handle-fix -> coana-fix -> spawnCoanaDlx
- Update all spawnCoanaDlx calls in coana-fix.mts to pass coanaVersion
- Update test fixtures to include coanaVersion parameter

Based on PR #957
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.

3 participants