Skip to content

fix(release): repair beta6 validation regressions#105034

Merged
steipete merged 7 commits into
release/2026.7.1from
codex/beta6-chat-send-isolation
Jul 12, 2026
Merged

fix(release): repair beta6 validation regressions#105034
steipete merged 7 commits into
release/2026.7.1from
codex/beta6-chat-send-isolation

Conversation

@steipete

@steipete steipete commented Jul 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What Problem This Solves

The beta6 candidate had three deterministic release-validation regressions:

  • 29 Control UI chat-send failures from a shared Vitest settings-module mock leak.
  • two Codex context-engine failures because hashed dynamic-tool fingerprints did not accept the immediately preceding legacy representation.
  • the native Codex runtime was rejected for the canonical codex/* provider route.

Why This Change Was Made

  • Give the chat-send fixture the real host settings contract instead of relying on a worker-global module mock.
  • Pass the already-supported legacy fingerprint into the thread-bootstrap compatibility decision.
  • Backport the canonical main fix from db39fe807202b259cf1269eae560c213e0c2afef, centralizing provider/runtime compatibility while preserving rejection for unsupported pairs.

User Impact

Beta6 retains native Codex runtime switching for both codex/* and openai/* routes. Existing pre-hash Codex thread bindings resume without unnecessary context reprojection. Control UI behavior is unchanged; its suite is deterministic under shared-worker ordering.

Evidence

  • Failed candidate UI job: https://github.com/openclaw/openclaw/actions/runs/29181488371/job/86619939595
  • Failed plugin prerelease job: https://github.com/openclaw/openclaw/actions/runs/29181487775/job/86619931025
  • Failed release-checks native runtime job: https://github.com/openclaw/openclaw/actions/runs/29181487474/job/86620171884
  • Blacksmith Testbox tbx_01kxadxjrn0dayyqn5assy8ck8: focused chat-send 99/99 and full Control UI 143 files / 2,331 tests passed.
  • Blacksmith Testbox tbx_01kxaee72ew2mdqgrcxccpy6fk: runtime/directive 328/328 and Codex context-engine 29/29 passed.
  • Blacksmith Testbox tbx_01kxagj2q2aygpzw0hxhhegx07: full TUI PTY real-backend file 8/8 passed after registering terminalizing validation-abort cases last.
  • Blacksmith Testbox tbx_01kxah6wq8h2v9q22c6qnp6dme: full TUI PTY real-backend file 8/8 passed after recognizing the Gateway's transient active-parent response in the existing bounded /new retry loop.
  • Same Testbox: package-acceptance workflow contract file 48/48 passed after restoring the matrix lookup helper omitted by the beta6 feature backport.
  • git diff --check passed.
  • Fresh branch autoreview: clean, no accepted/actionable findings.

Risk

The runtime change is an exact canonical backport already landed and validated on main. The fingerprint change uses the existing compatibility API and only admits the prior representation for the same next tool set. The TUI changes preserve all cases, follow main's canonical terminalizing-test ordering, and keep success mandatory after bounded busy-state retries. The package helper is the exact test contract required by the existing matrix assertions. No config, secret, or migration surface changes.

Current review state

Awaiting exact-head hosted PR CI. After merge, rerun the three failed lanes and full beta release validation on the new release/2026.7.1 head.

@openclaw-barnacle openclaw-barnacle Bot added app: web-ui App: web-ui size: XS maintainer Maintainer-authored PR labels Jul 12, 2026
@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. labels Jul 12, 2026
@clawsweeper

clawsweeper Bot commented Jul 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Codex review: stale review; fresh review needed.

Summary
The latest durable ClawSweeper review was for head ac050147b1dde6b3703b4df4a29bc6e2700327b1, but the PR head is now 2541cd6771d0413a1d340a4440b97288ce79b9d6. Its old verdict and PR readiness labels are no longer current.

Next step
Run or wait for a fresh ClawSweeper review on the current PR head.

Review history (2 earlier review cycles)
  • reviewed 2026-07-12T06:02:28.387Z sha 359cfca :: needs real behavior proof before merge. :: [P2] Declare applySettings on the test host
  • reviewed 2026-07-12T06:34:05.625Z sha b34869e :: needs maintainer review before merge. :: none

@openclaw-barnacle openclaw-barnacle Bot added agents Agent runtime and tooling extensions: codex size: S and removed size: XS labels Jul 12, 2026
@steipete steipete changed the title test(ui): isolate chat send settings fix(release): repair beta6 validation regressions Jul 12, 2026
@steipete steipete mentioned this pull request Jul 12, 2026
Merged
@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. and removed rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. labels Jul 12, 2026
@clawsweeper clawsweeper Bot removed proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jul 12, 2026
@steipete

steipete commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

Land-ready proof for exact head 2541cd6771d0413a1d340a4440b97288ce79b9d6:

  • Blacksmith Testbox tbx_01kxadxjrn0dayyqn5assy8ck8: focused chat-send 99/99 and full Control UI 2,331/2,331 passed.
  • Blacksmith Testbox tbx_01kxaee72ew2mdqgrcxccpy6fk: runtime/directive 328/328 and Codex context-engine 29/29 passed.
  • Blacksmith Testbox tbx_01kxah6wq8h2v9q22c6qnp6dme: TUI PTY real-backend 8/8 and package-acceptance workflow 48/48 passed.
  • pnpm check:test-types: passed on Testbox after the typed chat-host correction.
  • Fresh autoreview after every accepted change: clean, no accepted/actionable findings.
  • Exact-head hosted PR CI: green.

Known proof gap: the three original release-validation child failures and full beta validation intentionally rerun after merge against the new canonical release/2026.7.1 head.

@steipete
steipete merged commit 577c7e4 into release/2026.7.1 Jul 12, 2026
104 checks passed
@steipete
steipete deleted the codex/beta6-chat-send-isolation branch July 12, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

agents Agent runtime and tooling app: web-ui App: web-ui extensions: codex maintainer Maintainer-authored PR P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. size: M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@steipete