Skip to content

docs: correct remaining reachability CLI reference gaps (v2.4.6)#228

Merged
Martin Torp (mtorp) merged 1 commit into
mainfrom
martin/cli-docs-update
Jun 4, 2026
Merged

docs: correct remaining reachability CLI reference gaps (v2.4.6)#228
Martin Torp (mtorp) merged 1 commit into
mainfrom
martin/cli-docs-update

Conversation

@mtorp
Copy link
Copy Markdown
Contributor

@mtorp Martin Torp (mtorp) commented Jun 3, 2026

What

Layers the remaining Python CLI reachability reference corrections onto current main (v2.4.5 → 2.4.6), bringing the reachability documentation in line with the current implementation. Keeps the stable #reachability-analysis anchor in docs/cli-reference.md.

The headline reachability flag work already landed on main separately — #226 (canonical --reach-analysis-timeout / --reach-analysis-memory-limit names, coana-delegated defaults) and #227 (unified --exclude-paths, deprecate --reach-exclude-paths). This PR fixes only what those left stale or missing in docs/cli-reference.md.

Changes (docs/cli-reference.md)

  1. Requirements — document the uv + Enterprise-plan prerequisites the CLI enforces before running reachability (exits with code 3 if unmet), and clarify that per-ecosystem build toolchains (JDK / .NET / Go / a compatible Python interpreter) are checked by the analysis engine at runtime, not pre-checked by the CLI.
  2. --reach-min-severity — corrected the documented values to info, low, moderate, high, critical (was the inaccurate low, medium, high, critical).
  3. Documented 4 previously-undocumented flags: --reach-enable-analysis-splitting, --reach-detailed-analysis-log-file, --reach-lazy-mode, --reach-use-only-pregenerated-sboms.
  4. --only-facts-file — fixed misleading wording (submits only the facts file when creating the full scan; no pre-existing scan required).
  5. --reach — noted it creates a tier-1 full-application scan (scan_type=socket_tier1).

Other files (tooling churn, no code changes)

  • CHANGELOG.md2.4.6 entry (docs-only).
  • socketsecurity/__init__.py, pyproject.toml, uv.lock — patch bump 2.4.5 → 2.4.6, mandated by the repo's sync-version pre-commit hook (every commit forces a patch bump + lockfile refresh).

Notes

  • Draft intentionally — held for batched review/merge.
  • Python⇄Node reachability parity reconciled with the Node CLI maintainer; path-exclusion is now converged (--exclude-paths in both CLIs).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

🚀 Preview package published!

Install with:

pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple socketsecurity==2.4.6.dev2

Docker image: socketdev/cli:pr-228

Reachability-reference fixes layered on current main (v2.4.5):

- Document the uv + Enterprise-plan prerequisites the CLI enforces before
  running reachability (exit 3), and that per-ecosystem build toolchains are
  the analysis engine's runtime check, not a CLI pre-check.
- Correct --reach-min-severity values to info/low/moderate/high/critical.
- Document --reach-enable-analysis-splitting, --reach-detailed-analysis-log-file,
  --reach-lazy-mode, --reach-use-only-pregenerated-sboms.
- Clarify --only-facts-file submits only the facts file when creating the full
  scan (no pre-existing scan required).
- Note --reach creates a tier-1 full-application scan (scan_type=socket_tier1).

Docs-only; the version bump + uv.lock are mandated by the sync-version hook.
@mtorp Martin Torp (mtorp) changed the title docs: align reachability CLI reference with v2.4.2 implementation docs: correct remaining reachability CLI reference gaps (v2.4.6) Jun 4, 2026
@mtorp Martin Torp (mtorp) marked this pull request as ready for review June 4, 2026 06:48
@mtorp Martin Torp (mtorp) requested a review from a team as a code owner June 4, 2026 06:48
@mtorp Martin Torp (mtorp) deployed to socket-firewall June 4, 2026 06:48 — with GitHub Actions Active
@mtorp Martin Torp (mtorp) merged commit 405fdc9 into main Jun 4, 2026
38 checks passed
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