Skip to content

Docs pass 2: tighter cuts across all four files (526 → 213 lines)#28

Merged
Xinze-Li-Moqian merged 1 commit into
mainfrom
cleanup-docs-pass2
May 15, 2026
Merged

Docs pass 2: tighter cuts across all four files (526 → 213 lines)#28
Xinze-Li-Moqian merged 1 commit into
mainfrom
cleanup-docs-pass2

Conversation

@Xinze-Li-Moqian
Copy link
Copy Markdown
Contributor

Summary

Second cleanup pass on docs/. After PR #27 cut 901 → 526, this further cuts to 213 (76% off original).

File Pass 1 Pass 2
CONVENTIONS.md 59 45
NAMING_CONVENTION.md 91 84
REFACTOR_PLAYBOOK.md 261 55
SORRY_CATALOG.md 115 29

Biggest cuts

REFACTOR_PLAYBOOK.md (261 → 55):

  • Decision tree as ASCII art (40 lines) → 8-row table.
  • Verifiable-object consolidation 10-step procedure → 5 steps.
  • Each pitfall: one paragraph instead of 5–10 lines.

SORRY_CATALOG.md (115 → 29):

  • Dropped per-file tables. They duplicated docstring repair plans and drifted fast (catalog said total = 32, CI says EXPECTED=42).
  • Kept classification definitions + per-module summary.
  • Source of truth: CI's EXPECTED constant; per-sorry repair plans in each sorry's docstring.

Test plan

  • No file in OpenGALib/ references docs that were removed.
  • Review final docs for any remaining bloat.

REFACTOR_PLAYBOOK.md (261 → 55):
- Decision tree (40 lines of ASCII art) → 8-row table
- Verifiable-object consolidation 10-step procedure → 5 steps + sentence on automation
- Each pitfall: one paragraph instead of 5-10 lines
- Drop separators, sister-doc front matter

SORRY_CATALOG.md (115 → 29):
- Drop per-file tables — they duplicate the per-sorry docstring repair
  plans and drift fast (catalog claimed 32, CI says 42).
- Keep classification definitions + per-module summary table.
- Authoritative count lives in ci.yml; per-sorry repair plans live in
  each sorry's docstring; this file is now an at-a-glance summary only.

NAMING_CONVENTION.md (91 → 84):
- §1 prose tightened, "avoid these engineering suffixes" collapsed
  from 4-line bullet list to one sentence.

CONVENTIONS.md (59 → 45):
- Trim narrative padding, drop horizontal separators between sections,
  collapse a few multi-paragraph items into single paragraphs.
@Xinze-Li-Moqian Xinze-Li-Moqian merged commit 93ac00d into main May 15, 2026
2 checks passed
@Xinze-Li-Moqian Xinze-Li-Moqian deleted the cleanup-docs-pass2 branch May 15, 2026 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant