[ci] Require execution parity before planning - #3598
Conversation
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! 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". |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gvwvbb4qvqvk2nt4mdo3fp333lmrtmydn #3598 +/- ##
====================================================================
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:
|
d206edd to
483b00c
Compare
01b16a9 to
4fc813f
Compare
b58c1c7 to
03e0996
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Breezy! 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". |
662029c to
c719e07
Compare
03e0996 to
c7f935d
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
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". |
c7f935d to
a708660
Compare
c719e07 to
25c4bbd
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. What shall we delve into next? 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". |
25c4bbd to
e2d9f7c
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. Keep it up! 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". |
e2d9f7c to
bad5c53
Compare
5e627d1 to
802a4a7
Compare
bad5c53 to
2238a9b
Compare
802a4a7 to
04663fb
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. Breezy! 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". |
fa4256b to
5ceb1a6
Compare
31916bc to
0ae7201
Compare
The typed command model helps only if every caller checks it before a plan can escape. Run its full legacy-parity proof while constructing CiInputs, after policy, inventory, workflow, and baselines are trusted. This centralizes the temporary CLI-level gate added with the command model. Validation is now all-or-nothing: new planners and local commands cannot omit the execution audit or consume a model which has drifted from independently captured behavior. Tests: cargo test -p zc --all-targets --offline Tests: cargo clippy -p zc --all-targets --offline -- -D warnings *Authored by an agent, posting via joshlf's account* gherrit-pr-id: Gnp4lv6jrqbhnp5pyd2625o7jwdguulmz
5ceb1a6 to
dfc8b9d
Compare
0ae7201 to
b8f5ab8
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. Breezy! 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 typed command model helps only if every caller checks it before a
plan can escape. Run its full legacy-parity proof while constructing
CiInputs, after policy, inventory, workflow, and baselines are trusted.
This centralizes the temporary CLI-level gate added with the command
model. Validation is now all-or-nothing: new planners and local commands
cannot omit the execution audit or consume a model which has drifted
from independently captured behavior.
Tests: cargo test -p zc --all-targets --offline
Tests: cargo 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/Gnp4lv6jrqbhnp5pyd2625o7jwdguulmz && git checkout -b pr-Gnp4lv6jrqbhnp5pyd2625o7jwdguulmz FETCH_HEADCheckout
git fetch origin refs/heads/Gnp4lv6jrqbhnp5pyd2625o7jwdguulmz && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gnp4lv6jrqbhnp5pyd2625o7jwdguulmz && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.