Repository navigation
ci: move the codefly pin to 0.1.180 so CI reads the migrated declarations (Refs #952) - #1035
antoinetoussaint-byte wants to merge 1 commit into
Conversation
f528a97 to
1336c26
Compare
… v0.14.0 (Refs #952) #953 migrated every endpoint declaration to the reach/addressing split core v0.14.0 introduced. The pin was 0.1.179, whose CLI vendors core v0.8.1 — checked at the tag, not inferred — so it predates the split entirely and reads none of the new spelling. 0.1.180 is the first release vendoring v0.14.0. The installer's default and ci-performance.test.mjs's DEFAULT_CODEFLY_VERSION move together, because that test exists to keep the two in sync: a pin moved in one place and not the other is the failure it was written to catch, and it caught this. Pin-only, and deliberately a fresh branch rather than a rebase of the earlier one: that branch predated #953 and carried 440 files, so rebasing it onto the merged declarations would have replayed work already on main. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1336c26 to
9323dea
Compare
|
Closing in favour of #1031, which the saas-starter session owns and has verified locally. I opened this without checking what that session already had in flight, which was my mistake: it had both the pin rebuild and the release commit (#1037) prepared. Two pin PRs against one repo is exactly the duplication the one-objective-one-PR rule exists to prevent. For the record, this branch was also failing for a real reason worth keeping: at the 0.1.180 pin, |
Pull request was closed
#953 migrated every endpoint declaration to the reach/addressing split core v0.14.0 introduced. The CI pin was still
0.1.171, which vendors a core that refuses the new spelling — so the declarations onmainand the CLI that reads them disagree, and nothing can be released until they agree.Why this is its own PR and not part of #953.
codefly ci plan --base <main>loads the base revision's workspace with the head's CLI. A single PR carrying both the declarations and the pin can therefore never be green: the new CLI reads the old declarations from the base. The order is forced — declarations first against the old pin, then the pin alone.Why a fresh branch rather than rebasing #1031. That branch predates #953 and carries 440 files; rebasing it onto the merged declarations replays work already on
mainand conflicts immediately. This takes onlyscripts/ci/install-codefly.sh, which is the whole of the change. #1031 should be closed in favour of this.Next: a release carrying the migrated declarations, which is the last pin a composing platform workspace needs before its deployment render stops refusing.