sneak/prompts
1
0
Fork 0
Code Issues 6 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Compact REPO_POLICIES.md without losing fidelity or clarity #158

New Issue
Open
opened 2026-11-06 01:18:27 +01:00 by clawbot · 1 comment
clawbot commented 2026-11-06 01:18:27 +01:00
Collaborator
Copy Link
Copy Source

sneak, 2026-11-06 in chat: "i want a PR for me to review that compacts repo_policies without losing fidelity or clarity. it's too long."

Definition of done:

  • prompts/REPO_POLICIES.md on next is rewritten shorter. Every rule, exception, example and condition keeps its meaning, and every one is at least as clear as before. Repetition, history, padding and needless examples go.
  • Short sentences and bulleted lists, no tables; sections keep a clear order, most important first. The pinning rule stays at the top, as the document itself says it is the single most important rule.
  • Anything else in the repo that restates or links to sections of the document (checklists, README.md, anchors) still matches.
  • The PR body says how much shorter it got (lines and words, before and after), and lists every place two passages were merged or a section moved.
  • An independent reviewer checks fidelity rule by rule against the old text before the PR goes to sneak.
  • The PR goes into next, assigned to sneak for his own review. The fleet does not merge it.
  • It starts after #154 and #155 have landed on next, so their changes are kept.
  • make fmt and make check pass. Written on Fable.

Model: opus-5-5

sneak, 2026-11-06 in chat: "i want a PR for me to review that compacts repo_policies without losing fidelity or clarity. it's too long." **Definition of done:** - `prompts/REPO_POLICIES.md` on `next` is rewritten shorter. Every rule, exception, example and condition keeps its meaning, and every one is at least as clear as before. Repetition, history, padding and needless examples go. - Short sentences and bulleted lists, no tables; sections keep a clear order, most important first. The pinning rule stays at the top, as the document itself says it is the single most important rule. - Anything else in the repo that restates or links to sections of the document (checklists, `README.md`, anchors) still matches. - The PR body says how much shorter it got (lines and words, before and after), and lists every place two passages were merged or a section moved. - An independent reviewer checks fidelity rule by rule against the old text before the PR goes to sneak. - The PR goes into `next`, assigned to sneak for his own review. The fleet does not merge it. - It starts after https://git.eeqj.de/sneak/prompts/issues/154 and https://git.eeqj.de/sneak/prompts/issues/155 have landed on `next`, so their changes are kept. - `make fmt` and `make check` pass. Written on Fable. Model: opus-5-5
clawbot commented 2026-11-06 01:58:54 +01:00
Author
Collaborator
Copy Link
Copy Source

PR: #159, into next, labelled needs-review, assigned clawbot for the fidelity review. After that it goes to sneak. Not merged.

  • prompts/REPO_POLICIES.md: 1,145 lines and 8,427 words before, 1,012 lines and 7,709 words after. make fmt and make check pass.
  • Every rule keeps its meaning. The rules sit under headings, pinning first, and each rule that was stated in several places is stated once. The PR body lists every merge, move and removal with the old line numbers.
  • Every command, path, flag and code line of the old text is still there, except the Go and Python make test recipes, which repeated the two Dockerfile examples.

Needs sneak. The cut is 8.5% of the words, because the file held little padding beyond the repetition. Going much shorter means taking content out of the file. The PR body, under "For sneak", gives three ways with sizes and a recommendation: move the two example Dockerfiles into files (about 7,070 words), move the language sections to the style guides (about 4,830 words), or drop the reasons (roughly 6,700 words, not recommended).

Model: fable-5-1

PR: https://git.eeqj.de/sneak/prompts/pulls/159, into `next`, labelled `needs-review`, assigned `clawbot` for the fidelity review. After that it goes to sneak. Not merged. - `prompts/REPO_POLICIES.md`: 1,145 lines and 8,427 words before, 1,012 lines and 7,709 words after. `make fmt` and `make check` pass. - Every rule keeps its meaning. The rules sit under headings, pinning first, and each rule that was stated in several places is stated once. The PR body lists every merge, move and removal with the old line numbers. - Every command, path, flag and code line of the old text is still there, except the Go and Python `make test` recipes, which repeated the two `Dockerfile` examples. **Needs sneak.** The cut is 8.5% of the words, because the file held little padding beyond the repetition. Going much shorter means taking content out of the file. The PR body, under "For sneak", gives three ways with sizes and a recommendation: move the two example `Dockerfile`s into files (about 7,070 words), move the language sections to the style guides (about 4,830 words), or drop the reasons (roughly 6,700 words, not recommended). Model: fable-5-1
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
critical
Extremely important: data loss, a security hole, production down, or basic use impossible
 merge-ready
PR approved and ready to merge
 needs-checks
A check is red
 needs-rebase
PR conflicts with its base branch and must be rebased
 needs-review
PR needs code review
 needs-rework
PR needs changes based on review
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/prompts#158