[Spec 1252] Harvest: 17-file drift reconciliation + knowledge artifacts - #1281
Merged
Conversation
The harvest of Spec 1252 after the pr-gate decision not to merge the full machinery (PR #1278, closed unmerged; charter-level re-scope -- a successor shrinks the surface radically before enforcement is rebuilt fit-for-purpose). RECONCILIATION (the live bug): all 17 drifted codev/ shadow copies synced to the skeleton content they had silently fallen behind. Served SPIR/ASPIR builder prompts regain their Multi-PR Mechanics and Verify Phase sections and the 'Entering verify phase.' notification -- until now, adopters received verify instructions and this repo's builders did not. Architect rulings applied: E1 max_iterations: 3 for spir/aspir (decided on the measured evidence that review rounds run 1-2, mean 1.12); E2 the air/bugfix check cwds live in .codev/config.json porch.checks (Spec #550 mechanism), so the forked protocol.json files sync to the skeleton. Spec-746 pure-addition baselines re-derived for the two synced builder-prompts (frozen-file baselines break on any legitimate edit). KNOWLEDGE ARTIFACTS: the 77-row shadow-tree audit with architect rulings and a harvest note; word-count baseline and after-measurement (the honest -7.0% N1 record with its decomposition); the behavioural baseline (B1 51.88% REQUEST_CHANGES over 160 verdicts, self-excluded, B5 snapshot); the measurement tooling (surface script, behaviour metrics library + CLI + determinism test); the project review telling the true story including the non-merge decision; and the builder thread. The scar-registry concept and the eight ratified rule wordings move into the successor charter -- deferred, not discarded. Full suite from a normal checkout path: green (the three worktree-write-guard failures observed in the tmp-located build worktree are the documented /private/tmp-allowlist platform artifact; the suite is untouched by this change).
waleedkadous
added a commit
that referenced
this pull request
Jul 28, 2026
This was referenced Jul 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this is
The harvest of Spec 1252 (#1252) after the pr-gate decision not to merge the full single-owner machinery (PR #1278, closed unmerged). Waleed's charter-level call: a successor project will shrink the prompt surface radically first (blog-principles judgment rewrite, >50% target, frontier-models-only assumption), and enforcement gets rebuilt fit-for-purpose around the post-shrink surface. This PR lands the live bug-fix and the durable knowledge; the machinery stays on the reference branch and informs the successor.
1. The drift reconciliation (the live bug)
All 17 drifted
codev/shadow copies synced to the skeleton content they had silently fallen behind (the tier-2 copies outrank the installed skeleton in the resolver, so this repo's agents were served the stale versions):Multi-PR MechanicsandVerify Phasesections and the"Entering verify phase."notification — until now, adopters received verify instructions and this repo's builders did not.max_iterations: 3for spir/aspir (decided on the measured evidence that CMAP review rounds run 1–2, mean 1.12); E2 — the air/bugfix checkcwds live in.codev/config.jsonporch.checks(the Spec Language-agnostic porch check commands (af-config.json override) #550 mechanism; main's config already carries them), so the forkedprotocol.jsonfiles sync to the skeleton.2. The knowledge artifacts
codev/resources/1252-shadow-tree-audit.md— the 77-row shadow-copy classification with architect rulings and a harvest notecodev/resources/1252-word-baseline.md+1252-word-after-phase7.md— the always-on measurement record, including the honest −7.0% N1 with its decomposition (which is precisely the evidence behind the shrink-first successor)codev/resources/1252-behavior-baseline.md— behavioural baseline (B1 = 51.88% REQUEST_CHANGES over 160 verdicts, self-excluded; B5 advisory snapshot) — the successor's "before"scripts/measure-prompt-surface.sh,packages/codev/src/lib/prompt-behavior-metrics.ts+ CLI runner + determinism testcodev/reviews/1252-prompt-architecture-single-own.md— the review telling the true story, including the non-merge decisioncodev/state/spir-1252_thread.md— the builder threadExplicitly excluded (stays on
builder/spir-1252for reference)Shadow-tree deletion, scar registry + enforcement, ownership map + completeness machinery, dedup partials, CLAUDE.md restructure, all skeleton edits. The scar-registry concept and the eight ratified rule wordings move into the successor charter — deferred, not discarded.
Tests
Full suite green from a normal checkout path (the three
worktree-write-guardfailures observed in a/private/tmp-located build worktree are that test's documented tmp-allowlist platform artifact; the suite is untouched here). The behaviour-metrics determinism test reproduces the committed baseline numbers against main's corpus.Refs #1252 · Harvested from #1278
🤖 Generated with Claude Code