Skip to content

[codex] Avoid forced directory refresh during plugin install auth checks#25381

Merged
xl-openai merged 1 commit into
mainfrom
xl/plugin-install
May 31, 2026
Merged

[codex] Avoid forced directory refresh during plugin install auth checks#25381
xl-openai merged 1 commit into
mainfrom
xl/plugin-install

Conversation

@xl-openai
Copy link
Copy Markdown
Collaborator

@xl-openai xl-openai commented May 31, 2026

Summary

  • Use normal directory loading for plugin install app metadata so install avoids forced directory refresh while still loading metadata on cold cache.
  • Continue force-refreshing codex_apps tools for auth state.
  • Add regression coverage that pre-warms the directory cache and asserts install returns cached app metadata without extra directory requests.

Validation

  • just fmt
  • git diff --check
  • just test -p codex-app-server plugin_install_returns_apps_needing_auth plugin_install_filters_disallowed_apps_needing_auth (blocked locally: cargo-nextest is not installed)

@xl-openai xl-openai force-pushed the xl/plugin-install branch from 0b242e6 to e91a80d Compare May 31, 2026 07:21
@xl-openai xl-openai changed the title [codex] Avoid directory refresh during plugin install auth checks [codex] Avoid forced directory refresh during plugin install auth checks May 31, 2026
@xl-openai xl-openai force-pushed the xl/plugin-install branch from e91a80d to 80e4e47 Compare May 31, 2026 07:30
@xl-openai xl-openai marked this pull request as ready for review May 31, 2026 07:31
@xl-openai xl-openai requested a review from xli-oai May 31, 2026 07:31
@xl-openai xl-openai merged commit cdde711 into main May 31, 2026
31 checks passed
@xl-openai xl-openai deleted the xl/plugin-install branch May 31, 2026 09:14
@github-actions github-actions Bot locked and limited conversation to collaborators May 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants