Repository navigation
chore: sync public mirror from internal - #1353
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: fe7e7afac6
ℹ️ 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".
| events.push(Event::ContentPolicyRepairRequested { step, feedback }); | ||
| self.emit(ctx, events).await?; | ||
| return Ok(None); |
There was a problem hiding this comment.
Include policy feedback in the repair prompt
When a policy rejection occurs, this immediately retries, but the production AiRsModel::stream builds its request solely from to_ai_messages(ctx.history()) (packages/dex-host-rs/src/model.rs:536), while ModelAttemptAbandoned does not add the rejected completion to history and no code consumes Context::content_policy_repair_guidance. Thus the retry has no model-visible explanation of the violated rule (or the rejected draft) and will commonly repeat the same response until the two-attempt cap fails the turn, despite recording feedback here. Render that guidance into the next model request before retrying.
Useful? React with 👍 / 👎.
Summary
dx-corp/monodx-corp/codeas a generated public mirror of the private source of truth880d24b8d9656db0b8d3c859644bb7fcee1d410c4ee737816a77f0f27974d1aa3dea0465de0e299c10file(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@880d24b8d9656db0b8d3c859644bb7fcee1d410c
Prior destination base: 4ee7378
Content SHA-256: 9f55a740a2489f9028851b0a74b3042b3ce8d15f9486fd7db0cc10bd8d2a1285
Result: drift_detected; 10 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/content_policy.rs
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/src/voice.rs
copy/update vendor/dex-loop/tests/content_policy.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@880d24b8d9656db0b8d3c859644bb7fcee1d410c, including existing hidden/evaluation surfaces, and keeps public package parity behind the established public-source-provenance gate.