Skip to content

fix: use plugin service route for remote uninstall - #27223

Merged
ericning-o merged 1 commit into
mainfrom
codex/fix-remote-plugin-uninstall-route
Jun 9, 2026
Merged

fix: use plugin service route for remote uninstall#27223
ericning-o merged 1 commit into
mainfrom
codex/fix-remote-plugin-uninstall-route

Conversation

@ericning-o

Copy link
Copy Markdown
Contributor

Why

Remote plugin uninstall requests were sent to /plugins/{plugin_id}/uninstall, but the plugin service expects these mutations under /ps/plugins.

What changed

  • Send remote uninstall requests to /ps/plugins/{plugin_id}/uninstall.
  • Update app-server uninstall tests and request assertions to use the corrected route.

Testing

  • just test -p codex-core-plugins (203 passed)
  • Verified against production using the locally built app-server: the uninstall succeeded, and a follow-up plugin list reported installed: false and enabled: false.

@ericning-o
ericning-o merged commit 7a7cee1 into main Jun 9, 2026
31 checks passed
@ericning-o
ericning-o deleted the codex/fix-remote-plugin-uninstall-route branch June 9, 2026 19:52
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 9, 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