Skip to content

feat: add mypy command with grouped error output#109

Open
Rmohid wants to merge 1 commit intortk-ai:masterfrom
Rmohid:001-mypy-cmd
Open

feat: add mypy command with grouped error output#109
Rmohid wants to merge 1 commit intortk-ai:masterfrom
Rmohid:001-mypy-cmd

Conversation

@Rmohid
Copy link

@Rmohid Rmohid commented Feb 13, 2026

Summary

  • Add rtk mypy command that parses mypy type checker output, groups errors by file (sorted by error count), shows top error codes summary, and preserves note continuation lines (~80% token reduction)
  • Add discovery registry pattern and hook rewrites for mypy and python -m mypy
  • 11 new tests (324 total), all passing

Test plan

  • All 9 mypy_cmd unit tests pass (grouped output, column numbers, top codes, single code, every error shown, note continuation, fileless errors, no errors, no file limit)
  • Both registry tests pass (test_classify_mypy, test_classify_python_m_mypy)
  • test_patterns_rules_length_match passes (PATTERNS/RULES aligned at 22)
  • Full pre-commit gate: cargo fmt --all --check && cargo clippy --all-targets && cargo test

🤖 Generated with Claude Code

Add rtk mypy module that parses mypy type checker output, groups errors
by file and error code, and produces compact summaries. Includes
discovery registry pattern, hook rewrites, and 11 new tests (324 total).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pszymkowiak
Copy link
Collaborator

Rust code ok, 4 items to address:

  1. Remove specs/ — 533 lines of process docs (spec, plan, tasks, research, checklists). Should not be in the repo.
  2. Clean CLAUDE.md — remove the "Active Technologies" and "Recent Changes" sections added at the bottom. Keep only the module table entry.
  3. Duplicate rtk lint mypy — already merged in feat(python): add lint dispatcher + universal format command #100. Either remove the duplicate parser in lint_cmd.rs or have it delegate to mypy_cmd.
  4. Rebase needed — mergeStateStatus: UNSTABLE.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants