chore: sync public mirror from internal - #1335
Conversation
|
Resolved 0 review-bot thread(s) on this generated public mirror PR. Review of mirrored code belongs on the mono source PR. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1a1ae01718
ℹ️ 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".
| /// The workspace writing policy and the sender's voice choice, | ||
| /// resolved by the authenticated host. Prompt data only; grants no | ||
| /// authority. Older turns carry none and render no voice. | ||
| #[serde(default, skip_serializing_if = "Option::is_none")] | ||
| voice: Option<crate::TurnVoice>, |
There was a problem hiding this comment.
Render the admitted voice in model requests
When a UserMessage carries this field, it is persisted and rehydrated, but the in-repo dex_loop::Model implementation never reads Context::voice(): packages/dex-host-rs/src/model.rs builds the request from ctx.history() at lines 518-519, and its RequestConfig only uses self.system at lines 103-116. A repository-wide search finds no other voice() consumer, so policy guidance, required/forbidden terms, citation requirements, and tone have no effect on AiRsModel responses. Include the resolved voice in the model prompt/request before treating it as an admitted turn policy.
Useful? React with 👍 / 👎.
Summary
dx-corp/monodx-corp/codeas a generated public mirror of the private source of truthe94970dc115671761bdb96f00e6f47d8e294dac55050b67e1cd94b20955093b1f754fbc8693ae91715file(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@e94970dc115671761bdb96f00e6f47d8e294dac5
Prior destination base: 5050b67
Content SHA-256: eaa908b660a0258d88aec980f5423de6ccfeab1034307347aeac75cf47aae91f
Result: drift_detected; 15 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/compaction.rs
copy/update vendor/dex-loop/src/context.rs
copy/update vendor/dex-loop/src/context/evidence_contract.rs
copy/update vendor/dex-loop/src/event.rs
copy/update vendor/dex-loop/src/lib.rs
copy/update vendor/dex-loop/src/voice.rs
copy/update vendor/dex-loop/tests/action_confirmation.rs
copy/update vendor/dex-loop/tests/authorized_tools.rs
copy/update vendor/dex-loop/tests/client_tool_replay.rs
copy/update vendor/dex-loop/tests/codemode_workflows.rs
copy/update vendor/dex-loop/tests/scenarios.rs
copy/update vendor/dex-loop/tests/sim.rs
copy/update vendor/dex-loop/tests/sim/scenario.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@e94970dc115671761bdb96f00e6f47d8e294dac5, including existing hidden/evaluation surfaces, and keeps public package parity behind the established public-source-provenance gate.