Skip to content

[ax] Deprecate --rules-summary, suggest --output-format=json - #8457

Draft
TomasVotruba wants to merge 1 commit into
mainfrom
tv-deprecate-rules-summary
Draft

[ax] Deprecate --rules-summary, suggest --output-format=json#8457
TomasVotruba wants to merge 1 commit into
mainfrom
tv-deprecate-rules-summary

Conversation

@TomasVotruba

@TomasVotruba TomasVotruba commented Sep 4, 2026

Copy link
Copy Markdown
Member

Why

--rules-summary prints a human-only table of "which rule applied how many times". The JSON output format already carries strictly richer per-rule data - every applied rule with its file and line (changes, added in #8448). For agents the table is redundant noise.

Current --rules-summary output:

Rules Summary
-------------

 * SomeRector was applied 3 times
 * OtherRector was applied 1 time

What

Deprecate --rules-summary without breaking it: when used, warn on stderr and point to the JSON format.

[WARNING] The "--rules-summary" option is deprecated and will be removed.
          Use "--output-format=json" instead, which reports every applied rule with its file and line.
  • The option still works (summary still printed) - no behavior change, just a warning.
  • Warning goes to stderr (getErrorStyle), so stdout / JSON stay clean.

Verification

  • rector process --rules-summary -> warning on stderr, summary still shown, stdout clean.
  • without the option -> no warning.

https://claude.ai/code/session_01HpU9jWfDU9D81fTTh1symd

@TomasVotruba
TomasVotruba marked this pull request as draft September 4, 2026 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant