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

cleanup forbidden messaging

Browse Source
This commit is contained in:
kevin zhao
2025-12-01 19:29:52 -05:00
parent 696a543fec
commit 57ba6f7de1
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -190,7 +190,7 @@ pub(crate) async fn create_approval_requirement_for_command(
reason: FORBIDDEN_REASON.to_string(),
},
Decision::Prompt => {
let prompt_reason = derive_prompt_reason(&evaluation);
let prompt_reason = prompt_reason_for(&evaluation);
if matches!(approval_policy, AskForApproval::Never) {
ApprovalRequirement::Forbidden {
reason: PROMPT_CONFLICT_REASON.to_string(),
@@ -208,8 +208,10 @@ pub(crate) async fn create_approval_requirement_for_command(
}
}
/// Only return PROMPT_REASON when an execpolicy rule drove the prompt decision
fn derive_prompt_reason(evaluation: &Evaluation) -> Option<String> {
/// 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> {
evaluation.matched_rules.iter().find_map(|rule_match| {
if !matches!(rule_match, RuleMatch::HeuristicsRuleMatch { .. })
&& rule_match.decision() == Decision::Prompt