Skip to content

app-server: allow remote_control runtime feature override#20047

Merged
euroelessar merged 1 commit intomainfrom
ruslan/app-server-remote-control-enable
Apr 28, 2026
Merged

app-server: allow remote_control runtime feature override#20047
euroelessar merged 1 commit intomainfrom
ruslan/app-server-remote-control-enable

Conversation

@euroelessar
Copy link
Copy Markdown
Collaborator

Why

experimentalFeature/enablement/set lets clients patch process-wide runtime feature enablement, but the app-server handler only accepts feature keys from a local allowlist. remote_control already exists as a feature flag, and this change allows clients to toggle it through the same runtime override RPC as the other supported app-server feature flags.

What Changed

  • Added remote_control to the experimentalFeature/enablement/set allowlist.
  • Added integration coverage proving the RPC accepts remote_control.
  • Updated the app-server RPC README entry so the documented supported keys match the handler.

Verification

  • cargo test -p codex-app-server --test all suite::v2::experimental_feature_list

@euroelessar euroelessar force-pushed the ruslan/app-server-remote-control-enable branch from 9ac5259 to 718fc28 Compare April 28, 2026 18:40
@euroelessar euroelessar marked this pull request as ready for review April 28, 2026 19:08
@euroelessar euroelessar merged commit 1de7a9b into main Apr 28, 2026
25 checks passed
@euroelessar euroelessar deleted the ruslan/app-server-remote-control-enable branch April 28, 2026 20:36
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 28, 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