Skip to content

Simplify graph subcommand CLI#36

Merged
nikolay-e merged 2 commits into
mainfrom
claude/fix-smart-mode-selection-v5lP0
Apr 4, 2026
Merged

Simplify graph subcommand CLI#36
nikolay-e merged 2 commits into
mainfrom
claude/fix-smart-mode-selection-v5lP0

Conversation

@nikolay-e
Copy link
Copy Markdown
Owner

Summary

  • Remove redundant/bloated flags from treemapper graph: --mermaid, --cycles, --hotspots, --metrics, --impact, --blast-radius, --edge-types
  • --summary now automatically includes cycles, hotspots (top 10), and coupling metrics
  • Only 3 flags remain: --format, --level, --summary
  • Net deletion: ~150 lines across CLI, handler, and tests

Test plan

  • treemapper graph . outputs mermaid graph (default)
  • treemapper graph . -f json outputs JSON
  • treemapper graph . --summary includes nodes/edges stats, cycles, hotspots, and metrics
  • Old flags (--mermaid, --cycles, etc.) are properly rejected
  • pytest tests/test_graph.py passes

https://claude.ai/code/session_013MyvbfbFgB4vAtRn9NgEUD

claude added 2 commits April 4, 2026 13:17
Remove --mermaid (duplicate of --format mermaid), --cycles, --hotspots,
--metrics as standalone flags. The --summary flag now automatically
includes cycles, hotspots (top 10), and coupling metrics. Keep --impact
and --blast-radius as separate flags since they require a FILE argument.

https://claude.ai/code/session_013MyvbfbFgB4vAtRn9NgEUD
Further simplification: graph subcommand now has only --format, --level,
and --summary. Removed point analysis flags (--impact, --blast-radius)
and --edge-types filter. The underlying analytics functions remain in the
codebase for use by --diff mode.

https://claude.ai/code/session_013MyvbfbFgB4vAtRn9NgEUD
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 4, 2026

@nikolay-e nikolay-e merged commit 799b594 into main Apr 4, 2026
9 of 11 checks passed
@nikolay-e nikolay-e deleted the claude/fix-smart-mode-selection-v5lP0 branch April 4, 2026 14:08
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