mirror of
https://github.com/pchuan98/codex.git
synced 2026-07-01 00:31:56 +08:00
xl-openai
679cc08445
## Context The installed-app suggestion expansion added in #24996 reads plugin details for trusted file-backed marketplace candidates because the list response does not include app ids. On Windows-backed WSL mounts, the local `openai-curated` checkout lives under `$CODEX_HOME/.tmp/plugins`, and those per-plugin detail reads can be very slow. Remote curated already has cached app ids, so it does not need the same local filesystem traversal. ## Summary - Keep only the WSL Windows-backed local `openai-curated` checkout on the legacy fallback/configured discovery path. - Preserve installed-app expansion for non-WSL file-backed marketplaces and remote curated. - Add focused tests for the WSL local curated path predicate. ## Test - `just test -p codex-core-plugins discoverable` - `just test -p codex-core plugins::discoverable::tests`
62 lines
2.0 KiB
Rust
62 lines
2.0 KiB
Rust
use std::path::Path;
|
|
|
|
use super::is_wsl_windows_drive_path;
|
|
use super::should_use_legacy_local_curated_discovery_filter;
|
|
use crate::OPENAI_BUNDLED_MARKETPLACE_NAME;
|
|
use crate::OPENAI_CURATED_MARKETPLACE_NAME;
|
|
|
|
#[test]
|
|
fn legacy_local_curated_filter_matches_wsl_windows_backed_curated_checkout() {
|
|
let marketplace_path =
|
|
Path::new("/mnt/c/Users/user/.codex/.tmp/plugins/.agents/plugins/marketplace.json");
|
|
|
|
assert!(should_use_legacy_local_curated_discovery_filter(
|
|
OPENAI_CURATED_MARKETPLACE_NAME,
|
|
marketplace_path,
|
|
));
|
|
}
|
|
|
|
#[test]
|
|
fn legacy_local_curated_filter_does_not_match_native_wsl_curated_checkout() {
|
|
let marketplace_path =
|
|
Path::new("/home/user/.codex/.tmp/plugins/.agents/plugins/marketplace.json");
|
|
|
|
assert!(!should_use_legacy_local_curated_discovery_filter(
|
|
OPENAI_CURATED_MARKETPLACE_NAME,
|
|
marketplace_path,
|
|
));
|
|
}
|
|
|
|
#[test]
|
|
fn legacy_local_curated_filter_does_not_match_other_wsl_marketplaces() {
|
|
let other_marketplace_path = Path::new(
|
|
"/mnt/c/Users/user/.codex/.tmp/marketplaces/other/.agents/plugins/marketplace.json",
|
|
);
|
|
let local_curated_marketplace_path =
|
|
Path::new("/mnt/c/Users/user/.codex/.tmp/plugins/.agents/plugins/marketplace.json");
|
|
|
|
assert!(!should_use_legacy_local_curated_discovery_filter(
|
|
OPENAI_CURATED_MARKETPLACE_NAME,
|
|
other_marketplace_path,
|
|
));
|
|
assert!(!should_use_legacy_local_curated_discovery_filter(
|
|
OPENAI_BUNDLED_MARKETPLACE_NAME,
|
|
local_curated_marketplace_path,
|
|
));
|
|
}
|
|
|
|
#[test]
|
|
fn wsl_windows_drive_path_matches_only_mnt_drive_paths() {
|
|
assert!(is_wsl_windows_drive_path(Path::new(
|
|
"/mnt/c/Users/user/.codex/.tmp/plugins",
|
|
)));
|
|
assert!(is_wsl_windows_drive_path(Path::new("/mnt/Z/tmp")));
|
|
assert!(!is_wsl_windows_drive_path(Path::new("/home/user/.codex")));
|
|
assert!(!is_wsl_windows_drive_path(Path::new(
|
|
"/mnt/codex/Users/user/.codex",
|
|
)));
|
|
assert!(!is_wsl_windows_drive_path(Path::new(
|
|
"/media/c/Users/user/.codex",
|
|
)));
|
|
}
|