Skip to content

fix(ui): replace Manual RPC text input with sorted method dropdown#14967

Merged
BunsDev merged 4 commits intoopenclaw:mainfrom
rixau:fix/debug-rpc-dropdown
Mar 10, 2026
Merged

fix(ui): replace Manual RPC text input with sorted method dropdown#14967
BunsDev merged 4 commits intoopenclaw:mainfrom
rixau:fix/debug-rpc-dropdown

Conversation

@rixau
Copy link
Copy Markdown
Contributor

@rixau rixau commented Feb 12, 2026

Summary

  • Replace the free-text method input in Debug > Manual RPC with a <select> dropdown populated from hello.features.methods
  • Stack the method and params fields vertically instead of side-by-side (stack instead of form-grid)
  • Add a disabled "Select a method..." placeholder when no method is chosen

Why

The gateway exposes a known, finite set of RPC methods (~80+ base methods plus channel plugin methods). These are already sent to the client in the WebSocket hello message via features.methods. The current free-text input requires users to guess or look up method names in source code, which defeats the purpose of a debug panel.

The side-by-side layout also causes the text input to sit far below its label on narrower viewports.

Changes

  • ui/src/ui/views/debug.ts — Add methods: string[] prop, replace <input> with <select>, change form-grid to stack
  • ui/src/ui/app-render.ts — Pass sorted hello.features.methods array to renderDebug

Test plan

  • pnpm build passes
  • pnpm check passes (format + types + lint)
  • pnpm test — 882 passed
  • Tested locally: dropdown renders all gateway methods alphabetically, RPC calls work correctly

AI Disclosure

  • AI-assisted (Claude)
  • Lightly tested — verified build/check/test pass and visually confirmed in local Control UI
  • I understand what the code does: wires hello.features.methods into a <select> element and changes the CSS layout class

Greptile Overview

Greptile Summary

This PR updates the Debug � Manual RPC panel to use a method dropdown populated from the gateway hello payload (hello.features.methods), and adjusts the layout from a side-by-side grid to a vertical stack. The method list is passed down from ui/src/ui/app-render.ts into renderDebug in ui/src/ui/views/debug.ts, sorted client-side.

Confidence Score: 4/5

  • This PR is close to safe to merge, with one UI state management issue to address.
  • Changes are small and localized, but the new mixes a controlled .value binding with explicit selected attributes on options, which can cause the displayed selection to desync from state on rerender. ui/src/ui/views/debug.ts Last reviewed commit: 7c5b68b (2/5) Greptile learns from your feedback when you react with thumbs up/down!

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@openclaw-barnacle
Copy link
Copy Markdown

This pull request has been automatically marked as stale due to inactivity.
Please add updates or it will be closed.

@openclaw-barnacle openclaw-barnacle bot added stale Marked as stale due to inactivity and removed stale Marked as stale due to inactivity labels Feb 21, 2026
@BunsDev BunsDev self-assigned this Mar 10, 2026
@openclaw-barnacle openclaw-barnacle bot added agents Agent runtime and tooling size: S and removed size: XS labels Mar 10, 2026
@BunsDev BunsDev force-pushed the fix/debug-rpc-dropdown branch from 6cd764e to d481eb7 Compare March 10, 2026 06:06
@openclaw-barnacle openclaw-barnacle bot added size: XS and removed agents Agent runtime and tooling size: S labels Mar 10, 2026
@BunsDev BunsDev force-pushed the fix/debug-rpc-dropdown branch from d481eb7 to c8c9b0e Compare March 10, 2026 06:13
rixau and others added 4 commits March 10, 2026 01:23
The method field was a free-text input despite methods being a known
finite set sent via the hello message. Replace with a <select> dropdown
populated from hello.features.methods and stack the form fields
vertically for better layout.
Remove ?selected and selected attributes from <option> elements
to avoid conflicting with .value binding on the <select>.
@BunsDev BunsDev force-pushed the fix/debug-rpc-dropdown branch from c8c9b0e to 1bb49b2 Compare March 10, 2026 06:30
@BunsDev BunsDev merged commit 9d403fd into openclaw:main Mar 10, 2026
12 checks passed
@BunsDev
Copy link
Copy Markdown
Member

BunsDev commented Mar 10, 2026

Merged via squash.

Thanks @rixau!

mukhtharcm pushed a commit to hnykda/openclaw that referenced this pull request Mar 10, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
mrosmarin added a commit to mrosmarin/openclaw that referenced this pull request Mar 10, 2026
* main: (43 commits)
  docs: add openclaw#42173 to CHANGELOG — strip leaked model control tokens (openclaw#42216)
  Agents: align onPayload callback and OAuth imports
  docs: add Tengji (George) Zhang to maintainer table (openclaw#42190)
  fix: strip leaked model control tokens from user-facing text (openclaw#42173)
  Changelog: add unreleased March 9 entries
  chore: add .dev-state to .gitignore (openclaw#41848)
  fix(agents): avoid duplicate same-provider cooldown probes in fallback runs (openclaw#41711)
  fix(mattermost): preserve markdown formatting and native tables (openclaw#18655)
  feat(acp): add resumeSessionId to sessions_spawn for ACP session resume (openclaw#41847)
  ACPX: bump bundled acpx to 0.1.16 (openclaw#41975)
  mattermost: fix DM media upload for unprefixed user IDs (openclaw#29925)
  fix(msteams): use General channel conversation ID as team key for Bot Framework compatibility (openclaw#41838)
  fix(mattermost): read replyTo param in plugin handleAction send (openclaw#41176)
  fix(sandbox): pass real workspace to sessions_spawn when workspaceAccess is ro (openclaw#40757)
  fix(ui): replace Manual RPC text input with sorted method dropdown (openclaw#14967)
  CI: select Swift 6.2 toolchain for CodeQL (openclaw#41787)
  fix(agents): forward memory flush write path (openclaw#41761)
  fix(telegram): move network fallback to resolver-scoped dispatchers (openclaw#40740)
  fix(security): harden replaceMarkers() to catch space/underscore boundary marker variants (openclaw#35983)
  fix(web-search): recover OpenRouter Perplexity citations from message annotations (openclaw#40881)
  ...
Moshiii pushed a commit to Moshiii/openclaw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
Moshiii pushed a commit to Moshiii/openclaw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
frankekn pushed a commit to MoerAI/openclaw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
frankekn pushed a commit to Effet/openclaw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
frankekn pushed a commit to ImLukeF/openclaw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
dominicnunez pushed a commit to dominicnunez/openclaw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
dhoman pushed a commit to dhoman/chrono-claw that referenced this pull request Mar 11, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
ahelpercn pushed a commit to ahelpercn/openclaw that referenced this pull request Mar 12, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
Ruijie-Ysp pushed a commit to Ruijie-Ysp/clawdbot that referenced this pull request Mar 12, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
qipyle pushed a commit to qipyle/openclaw that referenced this pull request Mar 12, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
senw-developers pushed a commit to senw-developers/va-openclaw that referenced this pull request Mar 17, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
V-Gutierrez pushed a commit to V-Gutierrez/openclaw-vendor that referenced this pull request Mar 17, 2026
…penclaw#14967)

Merged via squash.

Prepared head SHA: 1bb49b2
Co-authored-by: rixau <[email protected]>
Co-authored-by: BunsDev <[email protected]>
Reviewed-by: @BunsDev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants