Codex Flight Suite: Consolidated Agent Flight and Metric Models - #4437
Codex Flight Suite: Consolidated Agent Flight and Metric Models#4437dieterolson wants to merge 2 commits into
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 91ea71b931
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| src/chat | ||
| src/contracts.py | ||
| src/python/src/utils | ||
| src/shared | ||
| src/sidekick |
There was a problem hiding this comment.
When the repository's full pytest suite runs, tests/ops/test_cross_repo_python_integration.py::test_each_downstream_declares_its_required_sparse_scope fails because REQUIRED_SPARSE_PATHS still expects the former roots (chat, contracts.py, python/src/utils, shared, and sidekick), while this workflow now declares their src/-prefixed replacements. Update the contract expectations alongside these paths so the workflow change does not leave the suite permanently red.
Useful? React with 👍 / 👎.
Closing as an exact duplicate — this branch does not contain the work its title advertisesVerified with git on 2026-08-13 against 1. #4437 and #4439 are the same snapshot. 2. Neither branch contains any flight / metric-model work. That is #4429's palette/bolt suite squashed onto the branch, plus a lint commit. There is no flight-model or metric-model source change on this branch at all. 3. Everything here is already in #4438. The palette/bolt content lands in #4438. Queued workflow runs on this branch are being cancelled to free the runner fleet (584 queued runs vs 1 executing at the time of this triage).
|
Pull request was closed
…tion (#4438) Consolidates the Palette & Bolt suite that originated in #4429 (squash 91ea71b, itself covering merges of #4313, #4314, #4419, #4420) plus a movement-optimizer motion-view extraction, with Python formatting corrected to the CI-pinned ruff==0.14.10 and a Python 3.10 regression reverted. Substance: - Palette: form semantics, useId() label binding and role="alert" in RotationConverter.tsx and ODESolverCalculator.tsx. - Bolt: single-pass CSV string building in chartSnapshot.ts, Array.from removal in explorer/csv.ts, fewer intermediate allocations for SVG paths in LinePlot/ScatterPlot/SpectrumPlot.tsx. - refactor(movement-optimizer): extract MotionViewMixin into src/movement_optimizer/gui/motion_view.py. - fix(ci): correct sparse-checkout paths in cross-repo-python-integration.yml. - fix(tests): restore timezone.utc in test_action_audit.py; a pre-commit ruff pass had moved a noqa off its line, letting UP017 rewrite the import to datetime.UTC, which does not exist on Python 3.10. Supersedes, all closed as containing nothing this branch lacks: - #4429 — the original palette/bolt branch; quality-gate red at Format Check because its own pre-commit commits re-wrapped 86 files with a ruff that disagrees with the pin. Proven a strict semantic subset of this tree by normalising both sides with one ruff format pass. - #4437 and #4439 — byte-identical trees to each other (7fc2ce4), each only the #4429 squash plus a lint commit, despite advertising flight and rate-of-closure work. Palette/Bolt work items carried: #4310, #4311, #4313, #4314, #4418, #4419, #4420, #4421. None of the 59 still-open agent PRs named in the three closed PRs' descriptions are in this diff; they remain open and unconsolidated. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 participant
Consolidates agent work (2 PRs) authored by
@d-sorgcodexagent[bot]: