chore: code-comments sweep slice 4/5 - cmd/coily core (.go, non-test, non-ops/gh) #90

New issue

Open

opened 2026-05-26 05:22:46 +00:00 by coilysiren · 0 comments

coilysiren commented

2026-05-26 05:22:46 +00:00

Owner

Problem

Subset of #334. 470 violations across 17 cmd/coily/*.go files that aren't gh_*, ops_*, or *_test.go. The entry-point + core-flow surface:

cmd/coily/brew.go
cmd/coily/cfg.go
cmd/coily/channel.go
cmd/coily/denial_reason.go
cmd/coily/dispatch.go
cmd/coily/git_passthrough.go
cmd/coily/hook.go
cmd/coily/lint.go
cmd/coily/lockdown_driver.go
cmd/coily/main.go
cmd/coily/mcporter_rewrite.go
cmd/coily/passthroughs.go
cmd/coily/runner.go (if present)
cmd/coily/runtime.go
cmd/coily/scoop.go
cmd/coily/ssh_passthrough.go
cmd/coily/upgrade.go

Gating

cmd/coily/lockdown_driver.go ships coilyRenderHookScript, the generator that produces .claude/lockdown-deny.sh with the 5-line header that re-dirties the file on every coily lockdown --apply. #335 / #336 need to land first so the sweep here also fixes the generator, otherwise next coily lockdown --apply re-violates.

Fix

Per #334's rules. No //nolint: or //go: directives inside violation blocks in this slice (the //go:embed and //nolint: are all in ops_*), so the transformation is mechanical truncate-to-first-2-lines with grammar preservation.

Verify:

pre-commit run code-comments --files cmd/coily/*.go
make build && make test

Refs

Parent: #334
Blocked by: #335 / #336
Dispatch survey: https://github.com/coilysiren/coily/issues/334#issuecomment-4539785886

Ported from coilysiren/coily#341.

**Problem** Subset of #334. 470 violations across 17 `cmd/coily/*.go` files that aren't `gh_*`, `ops_*`, or `*_test.go`. The entry-point + core-flow surface: - `cmd/coily/brew.go` - `cmd/coily/cfg.go` - `cmd/coily/channel.go` - `cmd/coily/denial_reason.go` - `cmd/coily/dispatch.go` - `cmd/coily/git_passthrough.go` - `cmd/coily/hook.go` - `cmd/coily/lint.go` - `cmd/coily/lockdown_driver.go` - `cmd/coily/main.go` - `cmd/coily/mcporter_rewrite.go` - `cmd/coily/passthroughs.go` - `cmd/coily/runner.go` (if present) - `cmd/coily/runtime.go` - `cmd/coily/scoop.go` - `cmd/coily/ssh_passthrough.go` - `cmd/coily/upgrade.go` **Gating** `cmd/coily/lockdown_driver.go` ships `coilyRenderHookScript`, the generator that produces `.claude/lockdown-deny.sh` with the 5-line header that re-dirties the file on every `coily lockdown --apply`. **#335 / #336 need to land first** so the sweep here also fixes the generator, otherwise next `coily lockdown --apply` re-violates. **Fix** Per #334's rules. No `//nolint:` or `//go:` directives inside violation blocks in this slice (the `//go:embed` and `//nolint:` are all in `ops_*`), so the transformation is mechanical truncate-to-first-2-lines with grammar preservation. Verify: ``` pre-commit run code-comments --files cmd/coily/*.go make build && make test ``` **Refs** - Parent: #334 - Blocked by: #335 / #336 - Dispatch survey: https://github.com/coilysiren/coily/issues/334#issuecomment-4539785886 --- _Ported from coilysiren/coily#341._

coilysiren added the

label

2026-05-31 01:52:40 +00:00

coilysiren added

and removed

labels

2026-05-31 06:59:44 +00:00

No labels

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#90

No description provided.

Rows
Columns