chore: code-comments sweep slice 3/5 - cmd/coily/ops_* #89

New issue

Open

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

coilysiren commented

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

Owner

Problem

Subset of #334. After the rev: v0.2.8 bump, the agentic-os code-comments hook reports 713 violations across 41 cmd/coily/ops_*.{go, _test.go} files. Biggest slice by volume, may want further subdivision before tackling.

Notable groupings inside this slice:

ops_forgejo_* (issue, label, pr, release, repo, test)
ops_eco_* (mod, test)
ops_factorio_*
ops_audit_* (dashboard, dashboard_test)
ops_gaming.go
ops_git_test.go
ops_lockdown.go / ops_lockdown_test.go
ops_repo.go / ops_repo_test.go
ops_setup.go
ops_systemctl.go / ops_systemd_local_test.go
ops_session.go / ops_session_test.go
ops_pkgmgrs.go
ops_whoami.go
ops_eco.go / ops_eco_test.go
ops_agent_name.go, ops_ci.go, ops_claude_remote_control.go, ops_trello.go, ops_version.go

Directive lines inside this slice (need block-aware fix, not pure truncate):

cmd/coily/ops_eco_mod.go:86 - //nolint:gocyclo,cyclop inside a godoc block
cmd/coily/ops_lockdown.go:21 - //nolint:gosec inside a godoc block
cmd/coily/ops_audit_dashboard.go:22 - //go:embed dashboard.html.tmpl (standalone, fine)

Truncating without preserving the //nolint: directives would re-enable the linter complaints they suppress.

Fix

Recommend splitting into 3-4 sub-slices (e.g. ops_forgejo_*, ops_eco_* + ops_factorio_*, the rest) before running a sweep. Per-block transformation matches #334's rules.

Verify:

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

Refs

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

Ported from coilysiren/coily#340.

**Problem** Subset of #334. After the `rev: v0.2.8` bump, the agentic-os `code-comments` hook reports 713 violations across 41 `cmd/coily/ops_*.{go, _test.go}` files. Biggest slice by volume, may want further subdivision before tackling. Notable groupings inside this slice: - `ops_forgejo_*` (issue, label, pr, release, repo, test) - `ops_eco_*` (mod, test) - `ops_factorio_*` - `ops_audit_*` (dashboard, dashboard_test) - `ops_gaming.go` - `ops_git_test.go` - `ops_lockdown.go` / `ops_lockdown_test.go` - `ops_repo.go` / `ops_repo_test.go` - `ops_setup.go` - `ops_systemctl.go` / `ops_systemd_local_test.go` - `ops_session.go` / `ops_session_test.go` - `ops_pkgmgrs.go` - `ops_whoami.go` - `ops_eco.go` / `ops_eco_test.go` - `ops_agent_name.go`, `ops_ci.go`, `ops_claude_remote_control.go`, `ops_trello.go`, `ops_version.go` **Directive lines inside this slice** (need block-aware fix, not pure truncate): - `cmd/coily/ops_eco_mod.go:86` - `//nolint:gocyclo,cyclop` inside a godoc block - `cmd/coily/ops_lockdown.go:21` - `//nolint:gosec` inside a godoc block - `cmd/coily/ops_audit_dashboard.go:22` - `//go:embed dashboard.html.tmpl` (standalone, fine) Truncating without preserving the `//nolint:` directives would re-enable the linter complaints they suppress. **Fix** Recommend splitting into 3-4 sub-slices (e.g. `ops_forgejo_*`, `ops_eco_* + ops_factorio_*`, the rest) before running a sweep. Per-block transformation matches #334's rules. Verify: ``` pre-commit run code-comments --files cmd/coily/ops_* make build && make test ``` **Refs** - Parent: #334 - Dispatch survey: https://github.com/coilysiren/coily/issues/334#issuecomment-4539785886 --- _Ported from coilysiren/coily#340._

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

No description provided.

Rows
Columns