Repository navigation
Consolidate capability qualification and camera/workbench experiments - #3
Merged
Merged
Conversation
…-macos-prerelease
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
Footwork's capability qualification and independently developed camera, desk inbox and physical workbench now share one reviewed integration branch. This brings both requested work streams onto main while retaining their experiment records and qualification limits.
The integration repairs two fresh-checkout problems: physical controller sockets now use a short, private directory independent of checkout length, and frozen onboarding packets verify their wheel against the requested immutable source commit. CI now exercises the camera, inbox, workbench and pinned MuJoCo simulation suites in addition to the existing Python, Node, Rust and wheel checks.
The original macOS capability candidate remains byte-identical: ZIP
8dc81c724bf617a9f3aa4833f250d5124b9e8c5e9a1c504c67d634080f44fd1f, qualified product445a348ff31243c2f3d90baae9ce209d89bd7eac, release sourced9315593886c17196db477db969f21481fc10e8d. A newly built wheel from this branch includes experimental artifact checking and is a separate artifact. The prerelease will use the exact qualified candidate.Validation:
jevdual/results/consolidation-20261014/. The remaining fail-closed limits are explicit.The initial failed fresh-checkout runs and Linux CI run are retained. Linux CI initially failed only because synthetic AX tests imported the macOS-only framework; an explicit Linux fake now preserves those tests and refuses unconfigured actions. A bounded publication audit found only metadata for the live camera/desk lane and controlled simulation screenshots; private note/camera inputs remain excluded. Private run evidence and the unrelated untracked draft are preserved separately before checkout retirement.
Qualification remains narrow: guest permission reset, installed dependency checks and actual agent execution are separate evidence lanes. Simulation does not qualify physical hardware. This PR does not claim another Mac, a never-granted OS, arbitrary target discovery, Screen Recording or saved-file persistence. README-only Codex adoption on the existing Mac follows landing, with the VM stopped and the requested memory headroom.
Merge with a true merge commit so the qualified product and release-source commits remain reachable from main. The final remediation head is
881f27d; merge requires its current CI and inspection of every review thread.A second Linux-only failure came from a test clock modifying the shared standard-library time module. The test now uses an isolated clock; its new regression failed before the change and all 57 native fixture tests pass afterward. Runtime/helper bytes and the qualified package are unchanged.