chore: code-comments sweep slice 1/5 - Formula/coily.rb + scripts/openapi-to-coily.py #87

Open
opened 2026-05-26 05:22:41 +00:00 by coilysiren · 0 comments
Owner

Problem

Subset of #334. After bumping .pre-commit-config.yaml to rev: v0.2.8, the agentic-os code-comments hook (MAX_CONTIGUOUS=2, 90-char line cap) reports 12 violations in two non-Go files:

  • Formula/coily.rb - 11 violations across multiple multi-line comment blocks
  • scripts/openapi-to-coily.py - 1 violation (3-line block at line 615-617)

Fix

Walk each block. Per #334's rules:

  • Single sentence fitting 2 lines: leave as-is.
  • 3+ line block: shorten to 2 lines and point at docs/<topic>.md if the detail is durable, otherwise drop the prose entirely and let git history carry it.

Verify with:

pre-commit run code-comments --files Formula/coily.rb scripts/openapi-to-coily.py

Why slice

These two files are the only non-Go violators and they don't depend on either gating issue (#335 / #336 / coilysiren/agentic-os#22). Smallest landable slice of #334.

Refs


Ported from coilysiren/coily#338.

**Problem** Subset of #334. After bumping `.pre-commit-config.yaml` to `rev: v0.2.8`, the agentic-os `code-comments` hook (MAX_CONTIGUOUS=2, 90-char line cap) reports 12 violations in two non-Go files: - `Formula/coily.rb` - 11 violations across multiple multi-line comment blocks - `scripts/openapi-to-coily.py` - 1 violation (3-line block at line 615-617) **Fix** Walk each block. Per #334's rules: - Single sentence fitting 2 lines: leave as-is. - 3+ line block: shorten to 2 lines and point at `docs/<topic>.md` if the detail is durable, otherwise drop the prose entirely and let git history carry it. Verify with: ``` pre-commit run code-comments --files Formula/coily.rb scripts/openapi-to-coily.py ``` **Why slice** These two files are the only non-Go violators and they don't depend on either gating issue (#335 / #336 / coilysiren/agentic-os#22). Smallest landable slice of #334. **Refs** - Parent: #334 - Dispatch survey: https://github.com/coilysiren/coily/issues/334#issuecomment-4539785886 --- _Ported from coilysiren/coily#338._
coilysiren added
P4
and removed
P3
labels 2026-05-31 06:59:45 +00:00
Sign in to join this conversation.
No labels
P0
P1
P2
P3
P4
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
coilyco-bridge/coily#87
No description provided.