Release v1.1.1: dorian init starter is load-bearing (golden path now blocks)#16
Conversation
In v1.1.0 the scaffolded starter claim was non-load-bearing, so a later break folded the warrant to DEGRADED (exit 3) — which the scaffolded `fail_on: revoked` Action does not block on, letting a first broken promise silently ship. Both starter branches (config-value and the path: fallback) are now load-bearing, so a break folds to REVOKED (exit 4) and the default Action blocks the PR — the golden path now demonstrates the gate it advertises. Sealing is unaffected (the starter still seals green; load_bearing only changes the fold on a later break). Adds a unit assertion for both branches and an end-to-end break -> REVOKED -> exit 4 test. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Bump 1.1.0 -> 1.1.1 (a scaffold default only; no breaking changes). README's
Getting-started section now states the load-bearing starter blocks on a break
(REVOKED, exit 4); action refs move to v1.1.1. Corrects a residual overclaim in
BENCHMARK_CURRENT.md ("removal of accidentally-packaged duplicate source files" ->
the `… 2.py` dups were untracked local artifacts, never tracked or shipped) and
re-stamps it at 1.1.1 (figures unchanged; no checker/binding/fold code touched).
New CHANGELOG entry and docs/releases/v1.1.1.md.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughVersion bump to 1.1.1. The Changesv1.1.1 Golden-path polish
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 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 docstrings
🧪 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 |
Summary
Follow-up to v1.1.0. Makes the
dorian initstarter claim load-bearing, so the golden path demonstrates the gate it advertises. No breaking changes — a single scaffold default; warrant format, checker grammar, exit codes, and trust semantics are unchanged.Why
The v1.1.0 public-install smoke test (install from PyPI →
init→verify→ break →revalidate) revealed the starter claim wasload_bearing: false, so a broken starter promise folded to DEGRADED (exit 3) — which the scaffoldedfail_on: revokedAction does not block on. So a first-time user's first broken promise silently shipped — the opposite of the headline. Now the starter is load-bearing in both branches (config-value and thepath:fallback): a break folds to REVOKED (exit 4) and the default Action blocks the PR. This also matches the scaffolded note, which already said "load-bearing facts."Changes
src/dorian/init.py— starter claimload_bearing: truein both_starterbranches.tests/test_init.py— +2 tests: starter is load-bearing (both branches), and end-to-end break → REVOKED → exit 4.__init__, READMElatest:+ action refs,action/README.md,BENCHMARK_CURRENTstamp).BENCHMARK_CURRENT.md(the… 2.pydups were untracked local artifacts — never tracked or shipped, not "accidentally-packaged").CHANGELOGentry +docs/releases/v1.1.1.md.What dorian is / is not
Verifies explicit, checkable claims with deterministic, token-free checks; re-checks across PRs. Not a runtime control plane, MCP gateway, sandbox, or proof of arbitrary correctness.
--deny-exec/--deny-shellandchecker_trust: baseare fail-closed policies, not sandboxes. One documented real catch (encode/httpx) — not broad validation.Tests & verification
uv run pytest— 885 passeduv run ruff check+ruff format --check— cleanuv build+twine check— wheel + sdist PASS, no… 2.py, 33 modulesload_bearingbranches had flipped); fixed + covered before this PR.Security
No change to the security boundary.
dorian initstill writes files only (no execution, repo-confined, atomic,--force/--dry-run/--json); the starter remains a read-only C3 checker. Already-sealed warrants are unaffected.🤖 Generated with Claude Code
Summary by CodeRabbit
Changed
dorian initstarter claims are now load-bearing; breaking sealed claims results in exit code 4 (REVOKED) and blocks PRs via default GitHub Actions settings.Documentation
Tests