chore: sync public mirror from internal - #1302
Conversation
|
Resolved 0 review-bot thread(s) on this generated public mirror PR. Review of mirrored code belongs on the mono source PR. |
![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: b8c20fa2cc
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| // Policy asked for approval: no human is asked. The call is | ||
| // granted at once and the receipt goes to the log before the | ||
| // effect; the pending wave runs first so effects keep order. | ||
| if let (None, Verdict::NeedsApproval { approval, summary }) = (decision, verdict) { |
There was a problem hiding this comment.
This makes every NeedsApproval verdict emit AutoApproved, including ApprovalMode::Interactive, but the unchanged scenarios integration tests still require the old behavior: headless_turn_auto_approves_an_ask_gated_tool_and_records_the_audit_pair expects ApprovalRequested plus ApprovalDecided, interactive_turn_still_parks_on_the_same_ask_gated_tool expects Exit::Parked, and the client-tool case expects the same old pair. Those assertions now fail whenever cargo test --test scenarios can run; update or remove the obsolete mode-specific expectations along with this behavior change.
Useful? React with 👍 / 👎.
Summary
dx-corp/monodx-corp/codeas a generated public mirror of the private source of truthfac55f7217fbc6e14c155ecfc00aed5c8c31adcae9728f9fd83065904e209c56238a89291741bd748file(s) to copy/update and0stale file(s) to delete0sync-holdlabel to this PR; the sync workflow skips every push while it is setSource-of-truth status
Projection: deixic-code
Source: dx-corp/mono@fac55f7217fbc6e14c155ecfc00aed5c8c31adca
Prior destination base: e9728f9
Content SHA-256: 40c32f0bd494617b1e7297f7daf817ddd6861452f132690578f73452179f2900
Result: drift_detected; 8 changed, 0 deleted
Destination-owned content is preserved. Destination CI is a separate health signal.
copy/update .repository-projection.json
copy/update vendor/dex-loop/src/context.rs
copy/update vendor/dex-loop/src/engine.rs
copy/update vendor/dex-loop/src/event.rs
copy/update vendor/dex-loop/src/lib.rs
copy/update vendor/dex-loop/tests/scenarios.rs
copy/update vendor/dex-loop/tests/sim/invariants.rs
copy/update vendor/dex-loop/tests/support/mod.rs
Drift sample
Public-only commits since last generated sync
Validation
Test Plan
maestro-sync-public-release-mirrorworkflow inpublic-treemoderequire-internal-prconfirms internal source PR lineageStaged Rollout
dx-corp/mono@fac55f7217fbc6e14c155ecfc00aed5c8c31adca, including existing hidden/evaluation surfaces, and keeps public package parity behind the established public-source-provenance gate.