[ci] Project only executable cell selectors - #3590
Conversation
97e206f to
5527487
Compare
1c25593 to
1632fd7
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gbsj7gxuv5hpi22zhup5w3fajtjjxnldj #3590 +/- ##
====================================================================
Coverage ? 91.85%
====================================================================
Files ? 20
Lines ? 6093
Branches ? 0
====================================================================
Hits ? 5597
Misses ? 496
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1632fd7 to
ed2ac06
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
7fd0482 to
943b0c0
Compare
c370bbe to
256a24f
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
943b0c0 to
99b8cd8
Compare
256a24f to
1c5f2a6
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. Nice work! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1c5f2a6 to
c8420e6
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
239ae55 to
85ecd5e
Compare
858d976 to
7bce93a
Compare
85ecd5e to
f47a484
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
7bce93a to
e71c94e
Compare
ab1aedd to
e7cd2ac
Compare
e71c94e to
5f450b6
Compare
The compact Actions matrices still duplicated derived feature arguments,
target behavior, and Miri flags. The typed executor deliberately
resolves those details again from checked policy, so carrying both
representations would create a second contract which could drift.
Emit only the complete selectors accepted by each executor. Keep the
richer semantics in the human-facing plan artifact, where they remain
useful for review without becoming execution input.
Add exact compact-schema and output-size assertions so future transport
growth requires deliberate review.
Tests: cargo +1.93.1 test -p zc --all-targets --offline
cargo +1.93.1 clippy -p zc --all-targets --offline --
-D warnings
*Authored by an agent, posting via joshlf's account*
gherrit-pr-id: Gkr4fd4vz5vegygalblsmjjgpdq3eulsx
e7cd2ac to
b3c77b4
Compare
5f450b6 to
834050c
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. You're on a roll. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
2 participants
The compact Actions matrices still duplicated derived feature arguments,
target behavior, and Miri flags. The typed executor deliberately
resolves those details again from checked policy, so carrying both
representations would create a second contract which could drift.
Emit only the complete selectors accepted by each executor. Keep the
richer semantics in the human-facing plan artifact, where they remain
useful for review without becoming execution input.
Add exact compact-schema and output-size assertions so future transport
growth requires deliberate review.
Tests: cargo +1.93.1 test -p zc --all-targets --offline
cargo +1.93.1 clippy -p zc --all-targets --offline --
-D warnings
Authored by an agent, posting via joshlf's account
Latest Update: v29 — Compare vs v28
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Gkr4fd4vz5vegygalblsmjjgpdq3eulsx && git checkout -b pr-Gkr4fd4vz5vegygalblsmjjgpdq3eulsx FETCH_HEADCheckout
git fetch origin refs/heads/Gkr4fd4vz5vegygalblsmjjgpdq3eulsx && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gkr4fd4vz5vegygalblsmjjgpdq3eulsx && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.