feat: async approval contrib (#23690)

codex-rs/ext/extension-api/src/contributors.rs

@@ -134,18 +134,15 @@ pub trait ToolLifecycleContributor: Send + Sync {
     }
 }
 
-/// Future returned by one claimed approval-review contribution.
-pub type ApprovalReviewFuture<'a> =
-    std::pin::Pin<Box<dyn Future<Output = ReviewDecision> + Send + 'a>>;
-
 /// Extension contribution that can claim rendered approval-review prompts.
+#[async_trait::async_trait]
 pub trait ApprovalReviewContributor: Send + Sync {
-    fn contribute<'a>(
+    async fn contribute(
-        &'a self,
+        &self,
-        session_store: &'a ExtensionData,
+        session_store: &ExtensionData,
-        thread_store: &'a ExtensionData,
+        thread_store: &ExtensionData,
-        prompt: &'a str,
+        prompt: &str,
-    ) -> Option<ApprovalReviewFuture<'a>>;
+    ) -> Option<ReviewDecision>;
 }
 
 /// Future returned by one ordered turn-item contribution.

codex-rs/ext/extension-api/src/lib.rs

@@ -18,7 +18,6 @@ pub use codex_tools::ToolPayload;
 pub use codex_tools::ToolSpec;
 pub use codex_tools::parse_tool_input_schema;
 pub use contributors::ApprovalReviewContributor;
-pub use contributors::ApprovalReviewFuture;
 pub use contributors::ConfigContributor;
 pub use contributors::ContextContributor;
 pub use contributors::PromptFragment;

codex-rs/ext/extension-api/src/registry.rs

@@ -1,7 +1,8 @@
 use std::sync::Arc;
 
+use codex_protocol::protocol::ReviewDecision;
+
 use crate::ApprovalReviewContributor;
-use crate::ApprovalReviewFuture;
 use crate::ConfigContributor;
 use crate::ContextContributor;
 use crate::ExtensionData;
@@ -171,15 +172,22 @@ impl<C: Sync> ExtensionRegistry<C> {
 
     /// Claims the first rendered approval-review prompt accepted by an
     /// installed contributor.
-    pub fn approval_review<'a>(
+    pub async fn approval_review(
-        &'a self,
+        &self,
-        session_store: &'a ExtensionData,
+        session_store: &ExtensionData,
-        thread_store: &'a ExtensionData,
+        thread_store: &ExtensionData,
-        prompt: &'a str,
+        prompt: &str,
-    ) -> Option<ApprovalReviewFuture<'a>> {
+    ) -> Option<ReviewDecision> {
-        self.approval_review_contributors
+        for contributor in &self.approval_review_contributors {
-            .iter()
+            if let Some(decision) = contributor
-            .find_map(|contributor| contributor.contribute(session_store, thread_store, prompt))
+                .contribute(session_store, thread_store, prompt)
+                .await
+            {
+                return Some(decision);
+            }
+        }
+
+        None
     }
 
     /// Returns the registered prompt contributors.