Streamline plugin, apps, and skills handlers

[pakrym-oai]

Why

The plugin, app, and skills handlers had a lot of repeated send_error/return branches that made the success path hard to scan. This slice keeps behavior the same while moving fallible steps into local response-producing helpers, so the request boundary can send one result.

What Changed

• Converted plugin list/install/uninstall handlers in codex-rs/app-server/src/codex_message_processor/plugins.rs to return Result<*Response, JSONRPCErrorError> from helper methods and call send_result once.
• Added local error-mapping helpers for plugin install/uninstall and marketplace failures.
• Applied the same mechanical shape to app list, skills list/config, and marketplace add/remove/upgrade handlers in codex-rs/app-server/src/codex_message_processor.rs.

Verification

• cargo check -p codex-app-server
• cargo test -p codex-app-server --test all v2::app_list -- --test-threads=1
• cargo test -p codex-app-server --test all v2::plugin_ -- --test-threads=1
• cargo test -p codex-app-server --test all v2::skills_list -- --test-threads=1

[chatgpt-codex-connector]

💡 Codex Review

codex/codex-rs/app-server/src/codex_message_processor.rs

Line 3115 in 13c6797

self.send_internal_error(

Restore send_internal_error helper to compile app-server

This commit removes async fn send_internal_error(...) but leaves many calls like self.send_internal_error(...) intact. As a result, codex-app-server no longer type-checks (method not found on CodexMessageProcessor) and the build is blocked. Re-introduce the helper (or replace all remaining callsites) before merging.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".