From a0d305aae84ec9c567fffaab76e2fb8d3e67ce72 Mon Sep 17 00:00:00 2001
From: xli-oai <xli@openai.com>
Date: Thu, 14 May 2026 17:42:30 -0700
Subject: [PATCH] Clarify installed plugin response helpers

---
 .../app-server/src/request_processors/plugins.rs    | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/codex-rs/app-server/src/request_processors/plugins.rs b/codex-rs/app-server/src/request_processors/plugins.rs
index b264e5a16e..6bb33d674a 100644
--- a/codex-rs/app-server/src/request_processors/plugins.rs
+++ b/codex-rs/app-server/src/request_processors/plugins.rs
@@ -122,7 +122,7 @@ fn share_context_for_source(
     }
 }
 
-fn configured_marketplace_to_info(
+fn plugin_marketplace_entry_from_configured_marketplace(
     marketplace: codex_core_plugins::ConfiguredMarketplace,
     shared_plugin_ids_by_local_path: &std::collections::BTreeMap<AbsolutePathBuf, String>,
 ) -> PluginMarketplaceEntry {
@@ -136,13 +136,16 @@ fn configured_marketplace_to_info(
             .plugins
             .into_iter()
             .map(|plugin| {
-                configured_marketplace_plugin_to_info(plugin, shared_plugin_ids_by_local_path)
+                plugin_summary_from_configured_marketplace_plugin(
+                    plugin,
+                    shared_plugin_ids_by_local_path,
+                )
             })
             .collect(),
     }
 }
 
-fn configured_marketplace_plugin_to_info(
+fn plugin_summary_from_configured_marketplace_plugin(
     plugin: codex_core_plugins::ConfiguredMarketplacePlugin,
     shared_plugin_ids_by_local_path: &std::collections::BTreeMap<AbsolutePathBuf, String>,
 ) -> PluginSummary {
@@ -765,7 +768,7 @@ impl PluginRequestProcessor {
                     .marketplaces
                     .into_iter()
                     .map(|marketplace| {
-                        configured_marketplace_to_info(
+                        plugin_marketplace_entry_from_configured_marketplace(
                             marketplace,
                             &shared_plugin_ids_by_local_path,
                         )
@@ -826,7 +829,7 @@ impl PluginRequestProcessor {
                                                 .contains(&plugin.name)
                                     })
                                     .map(|plugin| {
-                                        configured_marketplace_plugin_to_info(
+                                        plugin_summary_from_configured_marketplace_plugin(
                                             plugin,
                                             &shared_plugin_ids_by_local_path_for_suggestions,
                                         )