Skip to content

Cut every comment in the CLI to two lines (GRYT-1056) - #35

Merged
sivert-io merged 4 commits into
mainfrom
claude/GRYT-1056-cli-comment-cap
Sep 9, 2026
Merged

Cut every comment in the CLI to two lines (GRYT-1056)#35
sivert-io merged 4 commits into
mainfrom
claude/GRYT-1056-cli-comment-cap

Conversation

@sivert-io

Copy link
Copy Markdown
Member

114 comment blocks in this repository ran past two lines. Every comment under cmd,
internal and .github/workflows is now two lines of prose or fewer.

The diff is comments only, and I checked that rather than saying it. For all 30 changed
files, the source with its comments stripped is identical before and after — go/parser
with comments dropped and go/printer for the Go, a whole-line # filter for the YAML. The
only lines of real code are the new comment-check step in ci.yml.

One thing to look at. internal/app/theme.go opens with a Hallmark stamp, which was
three lines. It is two now, with the same six scores and the same macrostructure, tone,
chrome and contrast fields — just folded onto one fewer line. If the stamp format has to be
exactly three lines, say so and I will carve it out of the check instead.

The check. scripts/check-comment-length.mjs is the same script the client, the server,
mobile, the UI library, the voice engine, the site and the docs run, with .go added to the
file types. It runs in the existing verify job.

Verified: go build ./..., go vet ./..., gofmt -l . clean, and go test ./... green
across all five packages that have tests.

🤖 Generated with Claude Code

sivert-io and others added 4 commits September 8, 2026 23:29
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…n CI (GRYT-1056)

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@sivert-io
sivert-io merged commit a78d97f into main Sep 9, 2026
2 checks passed
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