Post-autocorrect repo-wide lint residue report #375

Closed
opened 2026-08-14 20:02:17 +00:00 by jared · 4 comments
Owner

Parent

#368

What to build

A post-autocorrect offense census commented onto #368: what remains per plugin, and whether any remaining Lint/Syntax offenses are real syntax errors versus files that still should not be linted. The remaining debt becomes known rather than assumed.

Acceptance criteria

  • Comment on #368 with per-plugin remaining offense counts
  • Remaining Lint/Syntax offenses each classified: real syntax error vs. candidate for further Exclude
  • No code changes in this ticket

Blocked by

## Parent #368 ## What to build A post-autocorrect offense census commented onto #368: what remains per plugin, and whether any remaining Lint/Syntax offenses are real syntax errors versus files that still should not be linted. The remaining debt becomes known rather than assumed. ## Acceptance criteria - [ ] Comment on #368 with per-plugin remaining offense counts - [ ] Remaining Lint/Syntax offenses each classified: real syntax error vs. candidate for further Exclude - [ ] No code changes in this ticket ## Blocked by - #373 - #374
Author
Owner

Started: post-autocorrect lint residue census.

Started: post-autocorrect lint residue census.
Author
Owner

Repo-wide rubocop census (total=606, unsafe-correctable=280):
root: 75 (61 unsafe) - Style/StringLiterals,FrozenStringLiteralComment,Documentation
cc-architect: 100 (62 unsafe) - Minitest/EmptyLineBeforeAssertion,Sdlc/Dry/DuplicatedStatementSeq
os-adr: 109 (50 unsafe) - FrozenStringLiteralComment,MutableConstant,AbcSize
os-aidd-lint: 238 (66 unsafe) - MethodLength,FrozenStringLiteralComment,Documentation
os-backlog: 9 (0 unsafe) - MethodLength,ClassLength
os-context: 7 (1 unsafe) - FormatStringToken
os-sdlc: 23 (0 unsafe) - MethodLength,LineLength,ClassLength
os-vault: 43 (40 unsafe) - StringLiterals
tests: 2 (0 unsafe) - Lint/Syntax x2

Repo-wide rubocop census (total=606, unsafe-correctable=280): root: 75 (61 unsafe) - Style/StringLiterals,FrozenStringLiteralComment,Documentation cc-architect: 100 (62 unsafe) - Minitest/EmptyLineBeforeAssertion,Sdlc/Dry/DuplicatedStatementSeq os-adr: 109 (50 unsafe) - FrozenStringLiteralComment,MutableConstant,AbcSize os-aidd-lint: 238 (66 unsafe) - MethodLength,FrozenStringLiteralComment,Documentation os-backlog: 9 (0 unsafe) - MethodLength,ClassLength os-context: 7 (1 unsafe) - FormatStringToken os-sdlc: 23 (0 unsafe) - MethodLength,LineLength,ClassLength os-vault: 43 (40 unsafe) - StringLiterals tests: 2 (0 unsafe) - Lint/Syntax x2
Author
Owner

Lint/Syntax: 11 total, all confirmed parser misparses (ruby -c clean on all 4 files) - no real syntax errors. Files: plugins/os-adr/lib/adr/{migration_report,record,repository}.rb (9 offenses), tests/trim_status_table_test.rb (2). Cause: root rubocop config lacks TargetRubyVersion, already ticketed as #379.

Lint/Syntax: 11 total, all confirmed parser misparses (ruby -c clean on all 4 files) - no real syntax errors. Files: plugins/os-adr/lib/adr/{migration_report,record,repository}.rb (9 offenses), tests/trim_status_table_test.rb (2). Cause: root rubocop config lacks TargetRubyVersion, already ticketed as #379.
Author
Owner

Resolution

Done: Posted post-autocorrect repo-wide lint census on #368: 606 total offenses, 280 unsafe-correctable, per-plugin breakdown; all 11 remaining Lint/Syntax confirmed parser misparses via ruby -c (no real syntax errors)

Evidence: Two census comments on #368; misparse root cause already tracked as #379

Follow-ups: #384 autocorrect pass for cc-architect/root/os-vault (blocked by #379); unsafe-only review already ticketed as #383

## Resolution **Done:** Posted post-autocorrect repo-wide lint census on #368: 606 total offenses, 280 unsafe-correctable, per-plugin breakdown; all 11 remaining Lint/Syntax confirmed parser misparses via ruby -c (no real syntax errors) **Evidence:** Two census comments on #368; misparse root cause already tracked as #379 **Follow-ups:** #384 autocorrect pass for cc-architect/root/os-vault (blocked by #379); unsafe-only review already ticketed as #383
jared closed this issue 2026-08-14 20:14:11 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
jared/cc-os#375
No description provided.