From b405a8c8128eac692a592095f9cf8039f0095da8 Mon Sep 17 00:00:00 2001
From: kevin zhao <zhao@openai.com>
Date: Mon, 10 Nov 2025 17:55:17 -0800
Subject: [PATCH] refactor parse_pattern to use iterator chain

---
 codex-rs/execpolicy2/src/parser.rs | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/codex-rs/execpolicy2/src/parser.rs b/codex-rs/execpolicy2/src/parser.rs
index f3f6409831..c72806e9d6 100644
--- a/codex-rs/execpolicy2/src/parser.rs
+++ b/codex-rs/execpolicy2/src/parser.rs
@@ -84,10 +84,11 @@ impl PolicyBuilder {
 }
 
 fn parse_pattern<'v>(pattern: UnpackList<Value<'v>>) -> Result<Vec<PatternToken>> {
-    let mut tokens = Vec::new();
-    for item in pattern.items {
-        tokens.push(parse_pattern_token(item)?);
-    }
+    let tokens: Vec<PatternToken> = pattern
+        .items
+        .into_iter()
+        .map(parse_pattern_token)
+        .collect::<Result<_>>()?;
     if tokens.is_empty() {
         return Err(Error::InvalidPattern("pattern cannot be empty".to_string()));
     }