main is red: documentation-layout rejects all commits (tooling-issue-prioritization 4174 > 4000 cap, regressed by `5f9cd1d`) #169

New issue

Closed

opened 2026-06-04 10:29:32 +00:00 by coilysiren · 0 comments

coilysiren commented

2026-06-04 10:29:32 +00:00

Owner

Problem

main fails the dogfooded documentation-layout pre-commit hook, which is
always_run: true / pass_filenames: false. Because it scans the whole repo on
every commit, any clean commit to this repo is currently rejected:

FAIL: .agents/skills/tooling-issue-prioritization/SKILL.md: 4174 chars exceeds the 4000-char cap.
1 documentation layout violation(s).

Reproduced against clean origin/main (8db332b) via
python -m agentic_os.check_documentation_layout.

Root cause

8fac13f ("dogfood the full catalog pre-commit suite") added local enforcement
of documentation-layout and trimmed the file to 3557 chars (under cap).
5f9cd1d ("express the P1-P4 distribution as even-width bands") grew it back to
4174 chars, 174 over the 4000 cap. It landed anyway - almost certainly committed
on a stale local pre-commit env that had not reinstalled the dogfood hook, so the
whole-repo check did not run. A fresh env (or CI) blocks.

Fix options (owner's judgment)

Trim ~175+ chars of tooling-issue-prioritization/SKILL.md back under 4000.
Split the over-cap material into a docs/*.md walkthrough and link it.
Bump this file's cap via the [tool.agentic-os.*] per-path override if the
content is justified (the pyproject already carries larger-cap opt-ins).

Impact

Blocks every commit to this catalog repo until green, including the doc fix in
coilyco-flight-deck/agentic-os#168. Surfaced while working agentic-os-kai#496
(closes-issue Forgejo-URL rollout).

## Problem `main` fails the dogfooded `documentation-layout` pre-commit hook, which is `always_run: true` / `pass_filenames: false`. Because it scans the whole repo on every commit, **any** clean commit to this repo is currently rejected: ``` FAIL: .agents/skills/tooling-issue-prioritization/SKILL.md: 4174 chars exceeds the 4000-char cap. 1 documentation layout violation(s). ``` Reproduced against clean `origin/main` (`8db332b`) via `python -m agentic_os.check_documentation_layout`. ## Root cause - `8fac13f` ("dogfood the full catalog pre-commit suite") added local enforcement of `documentation-layout` and trimmed the file to 3557 chars (under cap). - `5f9cd1d` ("express the P1-P4 distribution as even-width bands") grew it back to 4174 chars, 174 over the 4000 cap. It landed anyway - almost certainly committed on a stale local pre-commit env that had not reinstalled the dogfood hook, so the whole-repo check did not run. A fresh env (or CI) blocks. ## Fix options (owner's judgment) 1. Trim ~175+ chars of `tooling-issue-prioritization/SKILL.md` back under 4000. 2. Split the over-cap material into a `docs/*.md` walkthrough and link it. 3. Bump this file's cap via the `[tool.agentic-os.*]` per-path override if the content is justified (the pyproject already carries larger-cap opt-ins). ## Impact Blocks every commit to this catalog repo until green, including the doc fix in coilyco-flight-deck/agentic-os#168. Surfaced while working agentic-os-kai#496 (closes-issue Forgejo-URL rollout).

coilysiren added the

label

2026-06-17 07:05:19 +00:00

coilysiren closed this issue

2026-06-17 22:15:57 +00:00