Repository navigation
Resume native releases and add Experiment and review controls - #2186
Merged
Merged
Conversation
A release-* branch pusher can amend an exact-bump commit message. Discovery treated that trailer as authority to drop another live cut, publish the planted version, and close the other pull request. Two live cuts now stay blocked. A recut in the current process still records the version it replaced.
This was referenced Nov 14, 2026
Closed
Merged
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.
Summary
Adds
--with-experimentto the CLI, admin-scoped start API and Console.--reviewis a reasoned CLI-only parity exclusion. Published Releases stay final.Closes #2159
Evidence
Execution evidence
cargo test --test release native::native_pass_quietly_skips_human_blocked_superseded_cuts -- --exact --nocaptureexited 101:superseded release 0.2.0 requires human review;0 passed; 1 failed.After: the same command passes:
native_pass_quietly_skips_human_blocked_superseded_cuts ... ok;1 passed; 0 failed. The retained regression covers draft and foreign-commit blockers, quiet repeated Passes, preserved refs and deduplicated Activity log entries.node --test --test-name-pattern='Release start requires admin' console/tests/start.test.cjsfailed withlocator.check: Timeout 5000ms exceededwaiting forRun release Experiment.After: the same command passed; the full Start browser file also passed all 16 tests, including desktop and mobile requests.
cargo clippy --all-targets --locked -- -D warnings: passed.cargo build --locked. Full Rust validation passed in all four CI partitions on exact head53cbcda3(CI run). Following the instruction to reuse successful CI, the duplicate local Rust run was stopped after CI completed; its selector exit reflects that deliberate interruption. No local assertion failure was reported before cancellation.Test seams: the release CLI against real Git and fake GitHub/Harness executables; the admin start HTTP interface; the Console Start form and confirmation flow. No tests target private release helpers.
Visual evidence
Console Start Release run — 1440×900
Console Start Release run — 390×844
Expanded release manual — 1440×900
Expanded release manual — 390×844
Expanded How it works release flow — 1440×900
Expanded How it works release flow — 390×844
Expanded architecture release flow — 1440×900
Expanded architecture release flow — 390×844
Review coverage
Standards: 3 findings, all addressed. Spec: 1 finding, reproduced with the reviewer's exact regression and fixed. Both independent reviewers read all 31 changed files, including generated documentation hunks. Changed files left unread: none. Generated-output exclusions: none.
Unaddressed findings
Standards
None. Typed release PR observations, named unfinished-release fields and shared PR identity interpretation address all three design findings.
Spec
None. Superseded release blockers now return a quiet Pass skip; the reviewer's regression is retained.
Security
None.
Security review outcome
No unaddressed introduced findings.
Merge Danger
Door: one-way
Running the release command can publish permanent artifacts. Recovery only moves a tag after verifying that its GitHub Release is unpublished, and uses expected-ref leases.
Blast Radius: releases
Built with codex codex-cli 0.167.1 · gpt-6.1-sol · xhigh