Sweep pre-existing documentation-layout and code-comments violations #4

Closed
opened 2026-05-26 04:45:22 +00:00 by coilysiren · 1 comment
Owner

Problem

The apply-agentic-os-hooks pre-commit config (untracked locally as of this issue) surfaces ~40 pre-existing violations across the repo:

  • reference/atproto-ecosystem.md over 80-line / 4000-char cap
  • .agents/skills/writing-bluesky/docs/THREADING.md over 4000-char cap
  • .claude/lockdown-deny.sh contiguous-comment blocks of 3-5 lines
  • ~37 more documentation-layout violations not yet enumerated

Until these are resolved, every commit hits the full repo-wide validator set even for unrelated changes.

Plan

  1. Enumerate full violation set with pre-commit run --all-files.
  2. Split oversized docs into multiple files under docs/.
  3. Trim comment blocks in .claude/lockdown-deny.sh (move long context to docs/).
  4. Commit .pre-commit-config.yaml once green.

Context

Filed alongside coilysiren/atmosphere#3 (marker-file commit), which used SKIP=documentation-layout,code-comments to land scoped without sweeping these unrelated violations.

**Problem** The `apply-agentic-os-hooks` pre-commit config (untracked locally as of this issue) surfaces ~40 pre-existing violations across the repo: - `reference/atproto-ecosystem.md` over 80-line / 4000-char cap - `.agents/skills/writing-bluesky/docs/THREADING.md` over 4000-char cap - `.claude/lockdown-deny.sh` contiguous-comment blocks of 3-5 lines - ~37 more documentation-layout violations not yet enumerated Until these are resolved, every commit hits the full repo-wide validator set even for unrelated changes. **Plan** 1. Enumerate full violation set with `pre-commit run --all-files`. 2. Split oversized docs into multiple files under `docs/`. 3. Trim comment blocks in `.claude/lockdown-deny.sh` (move long context to `docs/`). 4. Commit `.pre-commit-config.yaml` once green. **Context** Filed alongside coilysiren/atmosphere#3 (marker-file commit), which used `SKIP=documentation-layout,code-comments` to land scoped without sweeping these unrelated violations.
coilysiren added
P3
and removed
P2
labels 2026-06-04 08:16:39 +00:00
Author
Owner

Backlog burndown 2026-06-17: closing low-priority (P3/P4) to bring the open count to a manageable level. Nothing lost — reopen if this resurfaces. Batch tag: burndown-2026-06.

Backlog burndown 2026-06-17: closing low-priority (P3/P4) to bring the open count to a manageable level. Nothing lost — reopen if this resurfaces. Batch tag: `burndown-2026-06`.
coilysiren 2026-06-17 08:24:21 +00:00
Commenting is not possible because the repository is archived.
No description provided.