repo-wide pre-commit hooks fail on pre-existing debt, blocking all commits

coilysiren commented

2026-05-28 02:49:15 +00:00

Owner

The repo-wide pre-commit hooks (from the coilysiren/agentic-os baseline) fail on pre-existing content, so every commit to this repo is currently blocked. Surfaced while landing the catalog dependsOn fix (#14): a one-line .coily/coily.yaml change could not commit because these unrelated hooks scan the whole tree.

catalog-doc-size / documentation-layout

AGENTS.md - 100 lines / 4781 chars (caps: 80 / 4000).
README.md - 84 lines.
docs/FEATURES.md - 4230 chars.
docs/modio.md - 111 lines.
mod/README.md - markdown outside root / docs/ / skill folder (disallowed location).

catalog-trifecta

AGENTS.md missing required H2 headings: Scope, Project shape, Repo boundaries, Commands, Validation, Safety, Cross-repo contracts, Release, Agent rules.

code-comments (51 violations)

Source files with contiguous comment blocks: mod/shell/MockSkillsController.cs, mod/src/Dtos.cs, mod/src/SkillsApiController.cs, src/eco_spec_tracker/{livereload,main,mock_data}.py, tests/test_smoke.py, .claude/lockdown-deny.sh.
Generated build artifacts being linted: mod/**/obj/**/*.AssemblyInfo.cs. These should almost certainly be gitignored rather than comment-groomed - check the .gitignore for a missing obj/ (and likely bin/) entry.

Suggested order

Add obj/ and bin/ to .gitignore, git rm --cached the tracked build artifacts. That clears the bulk of the comment violations.
Split AGENTS.md / README.md / FEATURES.md / modio.md under the size caps; relocate mod/README.md.
Add the trifecta H2 sections to AGENTS.md.
Collapse the remaining real source comment blocks to one line each.

The repo-wide pre-commit hooks (from the coilysiren/agentic-os baseline) fail on pre-existing content, so every commit to this repo is currently blocked. Surfaced while landing the catalog dependsOn fix (#14): a one-line `.coily/coily.yaml` change could not commit because these unrelated hooks scan the whole tree. **catalog-doc-size / documentation-layout** - `AGENTS.md` - 100 lines / 4781 chars (caps: 80 / 4000). - `README.md` - 84 lines. - `docs/FEATURES.md` - 4230 chars. - `docs/modio.md` - 111 lines. - `mod/README.md` - markdown outside root / docs/ / skill folder (disallowed location). **catalog-trifecta** - `AGENTS.md` missing required H2 headings: Scope, Project shape, Repo boundaries, Commands, Validation, Safety, Cross-repo contracts, Release, Agent rules. **code-comments** (51 violations) - Source files with contiguous comment blocks: `mod/shell/MockSkillsController.cs`, `mod/src/Dtos.cs`, `mod/src/SkillsApiController.cs`, `src/eco_spec_tracker/{livereload,main,mock_data}.py`, `tests/test_smoke.py`, `.claude/lockdown-deny.sh`. - Generated build artifacts being linted: `mod/**/obj/**/*.AssemblyInfo.cs`. These should almost certainly be gitignored rather than comment-groomed - check the `.gitignore` for a missing `obj/` (and likely `bin/`) entry. **Suggested order** 1. Add `obj/` and `bin/` to `.gitignore`, `git rm --cached` the tracked build artifacts. That clears the bulk of the comment violations. 2. Split AGENTS.md / README.md / FEATURES.md / modio.md under the size caps; relocate mod/README.md. 3. Add the trifecta H2 sections to AGENTS.md. 4. Collapse the remaining real source comment blocks to one line each.

coilysiren referenced this issue

2026-05-28 13:28:05 +00:00

Deploy verification blocked on DinD->in-cluster-registry push reachability (shared, backend#26) #20

coilysiren commented

2026-05-30 05:43:37 +00:00

Author

Owner

Merged into #12 in the 2026-05-29 backlog burn-down. Same pre-commit doc-size debt as #12 Reopen if it should stand alone.

coilysiren closed this issue

2026-05-30 05:43:37 +00:00

coilysiren added the

P4