Remove Codex Apps connector readiness loop
This commit is contained in:
parent
539b8d0dd1
commit
f8738eea51
5 changed files with 15 additions and 16 deletions
|
|
@ -826,6 +826,13 @@ stale MCP tests for the deleted Codex Apps cache/startup snapshot/provenance
|
|||
path were removed. If attribution returns, it belongs in an Epiphany-owned typed
|
||||
adapter receipt, not as universal connector state.
|
||||
|
||||
The Codex Apps readiness flag is gone as well. `AccessibleConnectorsStatus` no
|
||||
longer carries `codex_apps_ready`, the TUI no longer schedules a second forced
|
||||
connector fetch waiting for a deleted app cache, and `connectors_enabled()` is
|
||||
hard-off. Connector listing remains an inert compatibility stub until the whole
|
||||
app connector surface is deleted or replaced by an Epiphany-owned typed MCP
|
||||
adapter.
|
||||
|
||||
That husk has now been cut from `codex-core`. The root core crate no longer
|
||||
exports `plugins`, no longer depends on `codex-core-plugins` or `codex-plugin`,
|
||||
and the core plugin manager / marketplace add-remove-sync modules and tests
|
||||
|
|
|
|||
Binary file not shown.
|
|
@ -343,6 +343,10 @@ current_status:
|
|||
remain. Plugin provenance no longer rides inside every connector/app record:
|
||||
`AppInfo.plugin_display_names`, `ToolInfo.plugin_display_names`, the plugin
|
||||
connector merge helpers, plugin-name search indexing, stale Codex Apps cache
|
||||
tests, and regenerated schema references were deleted. MCP remains JSON-RPC
|
||||
at the edge; user-declared MCP servers are the only runtime MCP authority.
|
||||
tests, and regenerated schema references were deleted. The Codex Apps
|
||||
readiness/cache superstition is also gone from connector prefetch:
|
||||
`AccessibleConnectorsStatus.codex_apps_ready` and the TUI force-refetch loop
|
||||
were removed, and TUI connector activation is hard-off while the core
|
||||
connector list remains an inert compatibility stub. MCP remains JSON-RPC at
|
||||
the edge; user-declared MCP servers are the only runtime MCP authority.
|
||||
next_action: Continue starving coordinator/runtime execution and operator surfaces off app-server JSON-RPC and out of `codex_message_processor.rs`; keep the vestigial Codex organ narrowed to subscription auth/model transport.
|
||||
|
|
|
|||
3
vendor/codex/codex-rs/core/src/connectors.rs
vendored
3
vendor/codex/codex-rs/core/src/connectors.rs
vendored
|
|
@ -8,7 +8,6 @@ use crate::config::Config;
|
|||
#[derive(Debug, Clone)]
|
||||
pub struct AccessibleConnectorsStatus {
|
||||
pub connectors: Vec<AppInfo>,
|
||||
pub codex_apps_ready: bool,
|
||||
}
|
||||
|
||||
pub async fn list_accessible_connectors_from_mcp_tools(
|
||||
|
|
@ -51,7 +50,6 @@ pub async fn list_accessible_connectors_from_mcp_tools_with_options_and_status(
|
|||
) -> anyhow::Result<AccessibleConnectorsStatus> {
|
||||
Ok(AccessibleConnectorsStatus {
|
||||
connectors: Vec::new(),
|
||||
codex_apps_ready: true,
|
||||
})
|
||||
}
|
||||
|
||||
|
|
@ -62,7 +60,6 @@ pub async fn list_accessible_connectors_from_mcp_tools_with_environment_manager(
|
|||
) -> anyhow::Result<AccessibleConnectorsStatus> {
|
||||
Ok(AccessibleConnectorsStatus {
|
||||
connectors: Vec::new(),
|
||||
codex_apps_ready: true,
|
||||
})
|
||||
}
|
||||
|
||||
|
|
|
|||
13
vendor/codex/codex-rs/tui/src/chatwidget.rs
vendored
13
vendor/codex/codex-rs/tui/src/chatwidget.rs
vendored
|
|
@ -61,6 +61,7 @@ use crate::legacy_core::DEFAULT_AGENTS_MD_FILENAME;
|
|||
use crate::legacy_core::config::Config;
|
||||
use crate::legacy_core::config::Constrained;
|
||||
use crate::legacy_core::config::ConstraintResult;
|
||||
use crate::legacy_core::connectors;
|
||||
#[cfg(target_os = "windows")]
|
||||
use crate::legacy_core::windows_sandbox::WindowsSandboxLevelExt;
|
||||
use crate::mention_codec::LinkedMention;
|
||||
|
|
@ -105,7 +106,6 @@ use codex_app_server_protocol::Turn;
|
|||
use codex_app_server_protocol::TurnCompletedNotification;
|
||||
use codex_app_server_protocol::TurnPlanStepStatus;
|
||||
use codex_app_server_protocol::TurnStatus;
|
||||
use crate::legacy_core::connectors;
|
||||
use codex_config::ConfigLayerStackOrdering;
|
||||
use codex_config::types::ApprovalsReviewer;
|
||||
use codex_config::types::Notifications;
|
||||
|
|
@ -7773,8 +7773,6 @@ impl ChatWidget {
|
|||
return;
|
||||
}
|
||||
};
|
||||
let should_schedule_force_refetch =
|
||||
!force_refetch && !accessible_result.codex_apps_ready;
|
||||
let accessible_connectors = accessible_result.connectors;
|
||||
|
||||
app_event_tx.send(AppEvent::ConnectorsLoaded {
|
||||
|
|
@ -7801,12 +7799,6 @@ impl ChatWidget {
|
|||
result,
|
||||
is_final: true,
|
||||
});
|
||||
|
||||
if should_schedule_force_refetch {
|
||||
app_event_tx.send(AppEvent::RefreshConnectors {
|
||||
force_refetch: true,
|
||||
});
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
|
|
@ -10184,7 +10176,7 @@ impl ChatWidget {
|
|||
}
|
||||
|
||||
fn connectors_enabled(&self) -> bool {
|
||||
self.config.features.enabled(Feature::Apps) && self.has_chatgpt_account
|
||||
false
|
||||
}
|
||||
|
||||
fn connectors_for_mentions(&self) -> Option<&[AppInfo]> {
|
||||
|
|
@ -10917,7 +10909,6 @@ impl ChatWidget {
|
|||
self.bottom_pane.set_connectors_snapshot(Some(snapshot));
|
||||
}
|
||||
|
||||
|
||||
pub(crate) fn open_review_popup(&mut self) {
|
||||
let mut items: Vec<SelectionItem> = Vec::new();
|
||||
|
||||
|
|
|
|||
Reference in a new issue