Skip to content

Regenerate .rubocop_todo.yml and fix newly-flagged offenses - #1353

Open
apiology wants to merge 2 commits into
castwide:masterfrom
apiology:rubocop_todo_style_cleanup
Open

apiology wants to merge 2 commits into
castwide:masterfrom
apiology:rubocop_todo_style_cleanup

Conversation

@apiology

@apiology apiology commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Claude: This PR was written by Claude Code on behalf of @apiology.

Problem: .rubocop_todo.yml no longer matches what rubocop --auto-gen-config generates — the committed file differs by 112 lines, so its exclusions describe offenses that have since moved or stopped existing. Regenerating it reactivates YARD/CollectionStyle, whose autocorrect rewrites a nested-generic Hash key type into a different type:

# before — a tuple key
# @return [Hash{Array(String, Array<String>) => Pin::Base}]

# after autocorrect — a flat, ambiguous list, not the same type
# @return [Hash{Array, String, Array, String => Pin::Base}]

Accepting that autocorrect would silently change documented types across the codebase rather than reformat them.

Solution: Regenerate .rubocop_todo.yml, fix the Style/StringLiterals and Style/BlockDelimiters offenses it surfaces in spec/source/chain_spec.rb, and disable YARD/CollectionStyle rather than accept its autocorrect — the same finding and fix as #1237. No behavior changes.

Regenerates the RuboCop todo file, which had drifted out of sync
with the current RuboCop/plugin versions (auto-gen-config output
differs by 112 lines from what's currently committed). Fixes the
offenses this surfaces rather than re-excluding them:

- YARD/CollectionStyle: rewrite short-form tuple-key Hash docstrings
  (Hash{Array(String, String) => ...}) to the long form YARD's current
  parser accepts (Hash{Array, String, String => ...}), in
  api_map/constants.rb, api_map/store.rb, and doc_map.rb.
- source_map.rb: silence one YARD/CollectionStyle false positive
  inline rather than widen the todo exclusion.
- spec/source/chain_spec.rb: Style/StringLiterals (single-quoted
  strings) and Style/BlockDelimiters (expect { }.not_to raise_error ->
  expect do end) throughout.

No behavior changes.
@apiology
apiology force-pushed the rubocop_todo_style_cleanup branch from 30ee46d to 009e5ac Compare October 4, 2026 22:02
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.

1 participant