fix: release protocol no longer shell-fetches a framework file by path - #32
Merged
Conversation
Output of `codev update`: ignore codev/.update-hashes.json and refresh the CLAUDE.md / AGENTS.md twins from the package. The two files remain byte-identical per the twin rule. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Step 2 told the release runner to `cat codev/protocols/maintain/protocol.md`. Shell reads bypass the four-tier resolver, so that path is absent in fresh installs where MAINTAIN resolves from the package skeleton. Spawn a MAINTAIN builder instead — porch delivers the protocol content inline. Clears the `codev doctor` framework-ref warning. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Clears the
codev doctorframework-ref warning.What
codev/protocols/release/protocol.mdstep 2 told the release runner tocat codev/protocols/maintain/protocol.md. Shell reads bypass the four-tierresolver, so that path does not exist in a fresh install where MAINTAIN
resolves from the package skeleton. Replaced with
afx spawn --protocol maintain,which gets the protocol content delivered inline by porch.
Also folded in the pending
codev updaterefresh (.gitignore,CLAUDE.md,AGENTS.md) that was already sitting uncommitted in the working tree. TheCLAUDE.md / AGENTS.md twins remain byte-identical.
Not included
codev doctoralso lists 74 byte-identical copies undercodev/protocols/,codev/roles/andcodev/consult-types/as "safe to remove". They are notsafe to remove in this repo. That advice targets adopter projects; here the
codev/↔codev-skeleton/mirror is a spec-approved invariant(
spec-1470-parity.test.tscross-tree parity,template-delivery.test.tsmirror parity,
baked-decisions.test.tspaired assertions) and it is how thisrepo dogfoods unreleased protocol changes instead of resolving from the
published package. Deleting them breaks 15 test files.
Worth a follow-up to make the drift check aware that this repo is Codev itself,
so those 74 lines stop reading as actionable drift.
Test
vitest runon the affected suites: 350 passed.🤖 Generated with Claude Code