Skip to content

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

Merged
TomasVotruba merged 1 commit into
mainfrom
tv-deprecate-rules-summary
Sep 5, 2026
Merged

[ax] Deprecate --rules-summary, suggest --output-format=json#8457
TomasVotruba merged 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
@TomasVotruba
TomasVotruba marked this pull request as ready for review September 5, 2026 11:51
@TomasVotruba
TomasVotruba merged commit fdabc86 into main Sep 5, 2026
43 checks passed
@TomasVotruba
TomasVotruba deleted the tv-deprecate-rules-summary branch September 5, 2026 11:51
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