Merge feature/yagni-ladder into develop
This commit is contained in:
@@ -518,3 +518,4 @@ rules:
|
|||||||
|
|
||||||
## 2026-09-27
|
## 2026-09-27
|
||||||
- bugfix/gitignore-diagram-allowlist merged into develop on user go, `gitflow finish` → facd26d, pushed, copies removed by the lib. Day 2026-09-25 lot fully on develop: BDR-101 default profile, full +4 gstack skills, gitignore allowlist. No working branch anywhere; `skills/diagram` ignored.
|
- bugfix/gitignore-diagram-allowlist merged into develop on user go, `gitflow finish` → facd26d, pushed, copies removed by the lib. Day 2026-09-25 lot fully on develop: BDR-101 default profile, full +4 gstack skills, gitignore allowlist. No working branch anywhere; `skills/diagram` ignored.
|
||||||
|
- 6-repo review, case 1 (ponytail + chisle, token-economy layer): rejected as plugins. Ponytail 146.7k stars, injects ~600 tok at SessionStart + every SubagentStart; chisle 566 stars, PostToolUse `updatedToolOutput` rewrite unverified on native tools, prose rules collide with writing-style.md; rtk already covers input axis (chisle bench: dedup 0 hit on rtk-filtered corpus); caveman purge precedent v3.5.0. Borrowed the ordered YAGNI ladder + `shortcut:` marker into CLAUDE.global.md § Code style, user go. feature/yagni-ladder UNMERGED.
|
||||||
|
|||||||
@@ -1,5 +1,15 @@
|
|||||||
# TODO
|
# TODO
|
||||||
|
|
||||||
|
## 2026-09-27 — YAGNI ladder + shortcut marker in doctrine (feature/yagni-ladder)
|
||||||
|
Case 1 of the 6-repo review (ponytail, chisle). Both rejected as plugins: per-turn
|
||||||
|
and per-subagent injection, prose rules colliding with writing-style.md, caveman
|
||||||
|
precedent (purged v3.5.0), rtk already covers the input axis. One net gain, the
|
||||||
|
ordered decision ladder, borrowed into CLAUDE.global.md § Code style plus a
|
||||||
|
`shortcut:` marker convention. 6 lines, 287 → 293, budget 320.
|
||||||
|
- [x] L1 doctrine edit, doctrine-citers census, banner budget.
|
||||||
|
UNMERGED — human gate. Cases 2-5 (agent-skills, ui-skills, reticle, OmniRoute)
|
||||||
|
follow one by one.
|
||||||
|
|
||||||
## 2026-09-25 — full profile +4 gstack web/doc skills (bugfix/full-profile-web-doc-skills)
|
## 2026-09-25 — full profile +4 gstack web/doc skills (bugfix/full-profile-web-doc-skills)
|
||||||
User go after the "why is gstack off under full?" answer (it was not: unapplied
|
User go after the "why is gstack off under full?" answer (it was not: unapplied
|
||||||
default). `scrape`, `skillify`, `diagram`, `make-pdf` join full.profile; the rest
|
default). `scrape`, `skillify`, `diagram`, `make-pdf` join full.profile; the rest
|
||||||
|
|||||||
@@ -126,6 +126,14 @@ Format follows [Keep a Changelog](https://keepachangelog.com/).
|
|||||||
seeded like a real tree (gstack off, nothing linked).
|
seeded like a real tree (gstack off, nothing linked).
|
||||||
|
|
||||||
### Changed
|
### Changed
|
||||||
|
- **CLAUDE.global.md § Code style** — the ordered YAGNI decision ladder
|
||||||
|
(not needed → reuse → stdlib → platform → installed dependency → one line
|
||||||
|
→ the minimum that works, after understanding the problem) and a
|
||||||
|
`shortcut:` comment convention for assumed shortcuts, harvested into
|
||||||
|
TODO.md. Six lines, 287 → 293 of the 320 budget. Case 1 of the 6-repo
|
||||||
|
review: ponytail and chisle rejected as plugins (per-turn and
|
||||||
|
per-subagent injection, prose rules colliding with writing-style.md,
|
||||||
|
the caveman purge precedent, rtk already covering the input axis).
|
||||||
- **Default profile = `full`.** With no selection (`.active-profile`
|
- **Default profile = `full`.** With no selection (`.active-profile`
|
||||||
absent, empty, or the legacy `none`), `full` is in force: statusline,
|
absent, empty, or the legacy `none`), `full` is in force: statusline,
|
||||||
`profile.sh current`, `gstack off` and `reset` all resolve it the same
|
`profile.sh current`, `gstack off` and `reset` all resolve it the same
|
||||||
|
|||||||
@@ -9,6 +9,12 @@ Apply unless repo-specific instructions override.
|
|||||||
- One responsibility per function/method.
|
- One responsibility per function/method.
|
||||||
- Preserve existing behavior unless asked.
|
- Preserve existing behavior unless asked.
|
||||||
- Scope changes to task — no unrelated edits.
|
- Scope changes to task — no unrelated edits.
|
||||||
|
- Before writing code, stop at the first rung that applies: not needed
|
||||||
|
(YAGNI) → reuse what the codebase has → stdlib → platform/runtime
|
||||||
|
feature → dependency already installed → one line → the minimum that
|
||||||
|
works. Runs after understanding the problem, never instead of it.
|
||||||
|
- Assumed shortcut → `shortcut:` comment at the site (limit + upgrade
|
||||||
|
path), harvested into TODO.md so "later" stays findable.
|
||||||
|
|
||||||
## Limits (adapt to language)
|
## Limits (adapt to language)
|
||||||
- Max 25 logic lines/function (executable statements; comments and
|
- Max 25 logic lines/function (executable statements; comments and
|
||||||
|
|||||||
Reference in New Issue
Block a user