feat(0.24.0): scaffold Development Philosophy into AGENTS.md/CLAUDE.md on install (BRO-1406)#71
Conversation
…d on install (BRO-1406) Every newly-bootstrapped workspace now inherits an explicit Development Philosophy — four guiding principles, each bound to the primitive that enforces it — so downstream development follows the philosophy by default, cohesively with the user's own project. Previously the scaffold deployed the mechanism (P1–P20) without the intent. The four principles (Think Before Coding · Simplicity First · Surgical Changes · Goal-Driven Execution) are sharpened by Karpathy's LLM-coding- pitfall observations; bstack's edge is the Ritual-vs-Substance rule — each principle is bound to its enforcing primitive (Think→P14+P15, Simplicity→P20, Surgical→P10, Goal-Driven→P11+P19). - AGENTS.md.template: new ## Development Philosophy section (4-principle → enforcing-primitive table + Ritual-vs-Substance rule + extend-it invite), placed before the primitives table so the why frames the how. - CLAUDE.md.template: short philosophy anchor → AGENTS.md#development-philosophy. - references/new-workspace-flow.md: deployed-files note updated. - No bootstrap.sh change (Phase 2 copies templates verbatim). Primitive count unchanged (20); governance-substrate content, not a P-row. Verified: fresh scaffold into a temp repo produces the section in both files with WORKSPACE_NAME substituted. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Warning Review limit reached
More reviews will be available in 50 minutes and 48 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughPR scaffolds a Development Philosophy section into agent and Claude workspace templates, mapping four guiding principles to enforcing primitives. Deployment documentation and release notes are updated accordingly, with VERSION bumped to 0.24.0. ChangesDevelopment Philosophy Template Scaffolding
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related issues
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…dient + de-hype changelog Cross-review (Codex, Strata A) blocked 4/10. Fixes: - drop undefined /simplify ref (not a primitive) - Surgical→Hygiene(P10)+Cross-Review(P20), not P10 alone (P10 is clean-tree, not no-tangential-refactor) - Goal-driven→Empirical(P11)+Orchestrate(P19), drop bare /goal pin (one cube cell, not the enforcer) - column 'Enforced by'→'Backed by' + explicit enforcement-strength-varies note (hard predicate vs judgment gate) - de-hyped CHANGELOG + template intro (drop 'every good engineer'/'proves the demand') Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…elog Cross-review round 2 (Codex) 6/10 BLOCK on two residuals: - (b) Surgical→Hygiene(P10) still overclaimed (P10 = tree cleanliness, not diff scope) → remap to Dep-Chain(P14) bounds-the-surface + Cross-Review(P20) flags- tangential-edits; drop P10 from the row. - (d) CHANGELOG still advocacy copy → tightened to terse release-note prose (dropped 'widely-recognized', star count; rationale reduced to one cf. line). + aligned residual 'enforcing'→'backing' wording (changelog bullet, flow doc). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…tays in Notes) Codex round-3 residual (d): changelog still read as advocacy. Final tightening — intro now states only what shipped + the mapping + what changed; Karpathy provenance preserved factually in the Notes companion-artifact line. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
P20 Cross-Review verdict: ✅ PASS (8/10)Strata A — true cross-vendor (Codex
Net effect: the gate caught that the original diff overclaimed enforcement strength — mapping principles to primitives more tightly than the actual predicates support. Final version adds an explicit enforcement-strength-varies note (hard predicate vs judgment gate), which is both accurate and on-thesis. |
What
Every newly-bootstrapped workspace now inherits an explicit Development Philosophy section in its scaffolded
AGENTS.md(+ a short anchor inCLAUDE.md), so downstream development follows the philosophy by default — cohesively with the user's own project. Previously the scaffold deployed the mechanism (the P1–P20 contract) without the intent.Closes BRO-1406.
Why
The 20 primitives are the how; new repos got no why. The most-starred public CLAUDE.md (
multica-ai/andrej-karpathy-skills, ~168k★) proves enormous demand for four guiding principles — Think Before Coding · Simplicity First · Surgical Changes · Goal-Driven Execution — but leaves them as prose ritual. bstack's edge is the Ritual-vs-Substance rule: each principle is bound to the primitive that makes it machine-checkable (Think→P14+P15, Simplicity→P20, Surgical→P10, Goal→P11+P19).Changes
assets/templates/AGENTS.md.template— new## Development Philosophysection between Self-Meta Definition and the primitives table (the why frames the how): 4-principle → enforcing-primitive table + the binding Ritual-vs-Substance rule + an explicit "extend it with your own principles" invitation.assets/templates/CLAUDE.md.template— short philosophy anchor →AGENTS.md#development-philosophy.references/new-workspace-flow.md— deployed-files note updated.CHANGELOG.md+VERSION→ 0.24.0.Dep-chain
research/entities/tool/karpathy-claude-md-guidelines.md.scripts/bootstrap.shPhase 2 (scaffold_governance_file) copies templates verbatim → no code change needed. Idempotent-never-overwrite → new installs get it; existing workspaces unaffected (backfill is a deferredbstack doctor/repairfollow-up).Verification (P11)
Ran the real scaffold into a temp git repo (
BSTACK_SKIP_SKILLS/RCS=1): bothAGENTS.mdandCLAUDE.mdcontain the section, all 4 principle rows present,{{WORKSPACE_NAME}}substituted (# fresh-repo — Agent Guidelines).Deferred (follow-ups, not in this PR)
bstack doctorlint for the section (makes it contract-enforced; letsrepairbackfill existing workspaces).~/broomva.🤖 Generated with Claude Code
Summary by CodeRabbit
Documentation
Chores