mirror of
https://github.com/stablyai/orca.git
synced 2026-09-22 08:02:28 +00:00
ci(release-cut): restore the skill-independent version commit (#10460)
#10340 added a ledger-advance step to the release cut, which violates the contract test #9119 added: the cut must not run generate-skill-bundle-manifest or stage resources/skills. Both PRs were green on their own branches and only conflicted once merged, so nothing failed until main had both — main and every open PR have been red since. Revert the two workflow lines. The script's --release implementation stays: it is correct and harmless when unused, and the root fix in #10340 — verify no longer walking git tags — does not depend on the cut step. This leaves #10340 semantically incomplete and that must not be dropped. With the registry seeded from the committed ledger instead of a tag walk, nothing advances the ledger at cut, so each new skill change re-uses the same unreleased tail revision for different bytes; older installs then match no known snapshot and degrade to unrecognized, which reads in the UI as a skill that needs attention and cannot be updated. Follow-up is to reintroduce the advance narrowly — stage only resources/skills/release-mapping.json and narrow the assertion to forbid mutating the content-addressed artifacts while permitting the provenance row.
This commit is contained in:
@@ -521,14 +521,7 @@ jobs:
|
||||
# message and tag name explicitly (avoids npm's `v1.2.3` prefix
|
||||
# assumptions and any lifecycle scripts that would run on bump).
|
||||
npm version "$VERSION" --no-git-tag-version --allow-same-version
|
||||
# Why: release cut is the single authoritative point where the released
|
||||
# skill ledger advances — this version's working-tree bytes become an
|
||||
# immutable released revision so future builds recognize them as an
|
||||
# official snapshot. It is the only place skill artifacts regenerate;
|
||||
# ordinary lint verifies working-tree bytes against the committed ledger
|
||||
# and never walks tags. Uses only Node built-ins, so no install needed.
|
||||
node config/scripts/generate-skill-bundle-manifest.mjs --release "$VERSION"
|
||||
git add package.json resources/skills
|
||||
git add package.json
|
||||
commit_message="release: v$VERSION"
|
||||
if [[ "$EVENT_NAME" == "schedule" ]]; then
|
||||
commit_message="$commit_message [rc-slot:$SLOT]"
|
||||
|
||||
Reference in New Issue
Block a user