Repository navigation
Phase 2: adopt shared Verify for CodexUI - #76
Merged
Merged
Conversation
VolkerChristian
marked this pull request as ready for review
October 10, 2026 17:55
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.
Phase 2: shared native CI, preserved web qualification
snode.c@source, thenAISuite@source.x-web: Node 22/24.20.0, checksum-pinned Chrome, frontend tests, profiling, production/browser qualification, failure diagnostics and artifact uploads.Verified locally: native build/install, original/new web-job structural equality except the intentional default-branch checkout, actionlint and whitespace.
Explicitly deferred native failures: 84 CTest cases expose 12 semantic-no-op paint-gate failures and four grouped 10,000-row timeouts; four existing Breeze availability skips. The no-op failure reproduces in the old root model on unchanged sources, and existing default-branch native CI already failed. No test, threshold or production UI code is weakened or changed. The maintainer directed that these fixes wait; full web qualification is not claimed locally.
Source-only decision and coordination
The maintainer explicitly requested no development SDKs and approved pushing/merging this change. Dependencies use only ordered
@sourcebuilds; their default branches and exact commits are read from the GitHub API. No SDK publication, download, release asset prerequisite or caller write permission remains.Control removal PR #4 is merged, and the explicitly approved existing SNodeC/.github:v1.0 tag now points to de1a3aa6d820f42cd141891de415fa9fea122dc3. SNode.C removal PR #268 is merged. Shared control CI is green: 30 tests, catalog and profile drift; all affected workflows pass actionlint and whitespace.
No package workflow/pipeline, secret, licence file, product README, branch name, stable product tag or
Book-*tag is changed. Phase 2 default-branch green gates are not claimed until the new runs finish. CodexUI and AutoTalk test fixes remain explicitly deferred; tests stay enabled.