clone/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-04-24 06:35:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

update messaging

Browse Source
This commit is contained in:
kevin zhao
2025-12-01 19:38:54 -05:00
parent 7200d7fca3
commit c26056924e
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
codex-rs/core/src/exec_policy.rs
View File

@@ -187,7 +187,7 @@ pub(crate) async fn create_approval_requirement_for_command(
reason: FORBIDDEN_REASON.to_string(),
},
Decision::Prompt => {
let prompt_reason = prompt_reason_for(&evaluation);
let prompt_reason = derive_prompt_reason(&evaluation);
if matches!(approval_policy, AskForApproval::Never) {
ApprovalRequirement::Forbidden {
reason: PROMPT_CONFLICT_REASON.to_string(),
@@ -204,11 +204,9 @@ pub(crate) async fn create_approval_requirement_for_command(
},
}
}
/// Determine the prompt reason: only rule-based prompts surface the execpolicy
/// message. Heuristics-only prompts leave the reason empty so callers can inject
/// contextual messaging (e.g. sandbox escalation justifications) when needed.
fn prompt_reason_for(evaluation: &Evaluation) -> Option<String> {
/// Only return PROMPT_REASON when an execpolicy rule drove the prompt decision
fn derive_prompt_reason(evaluation: &Evaluation) -> Option<String> {
evaluation.matched_rules.iter().find_map(|rule_match| {
if !matches!(rule_match, RuleMatch::HeuristicsRuleMatch { .. })
&& rule_match.decision() == Decision::Prompt