Add codespell support with configuration and typo fixes - #2424
Add codespell support with configuration and typo fixes#2424yarikoptic wants to merge 5 commits into
Graphify — looks good
Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).
Details
Graphify reviewed this change.
Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).
Graphify review — findings
This PR adds codespell spell-checking to the project, introducing a new GitHub Actions workflow (.github/workflows/codespell.yml), a pre-commit hook entry, and a [tool.codespell] configuration block in pyproject.toml (with skip paths, ignore-words list, and per-word rationale comments). It applies the resulting spelling corrections to comments, docstrings, and prompt text across extract.py, llm.py, tree_html.py, and dart.py, and adds # codespell:ignore inline directives in test_dedup.py and the plan doc. The changes are confined to documentation/comment text, string content, and tooling config rather than executable logic.
No blocking issues surfaced.
Analysis details — impact, health, verification
Impact & health
Graphify review
Impact — 1796 functions depend on the 463 functions this change touches.
Health — grade A; 10 existing hotspot(s) in the area this change touches (pre-existing, not introduced here):
extract()— 359 callers, 29 callees (high)_rebuild_code()— 68 callers, 49 callees (high)deduplicate_entities()— 49 callers, 21 callees (high)extract_files_direct()— 15 callers, 19 callees (high)build_merge()— 29 callers, 8 callees (high)extract_xaml()— 19 callers, 12 callees (high)dispatch_command()— 2 callers, 110 callees (high)_call_claude_cli()— 26 callers, 8 callees (high)- …and 2 more
Verification — 1796 functions in the blast radius were not formally verified this run (proofs are advisory here).
Gate & verification
graphify gate
PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.
Advisory (not blocking):
- verification_scope: 1656 function(s) in the blast radius were not formally verified this run