Skip to content

Add browser and tool profile quick settings#80609

Merged
clawsweeper[bot] merged 1 commit into
openclaw:mainfrom
jeffjhunter:jjh/quick-settings-browser-tools
May 11, 2026
Merged

Add browser and tool profile quick settings#80609
clawsweeper[bot] merged 1 commit into
openclaw:mainfrom
jeffjhunter:jjh/quick-settings-browser-tools

Conversation

@jeffjhunter

@jeffjhunter jeffjhunter commented May 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds Quick Settings controls for:

  • Browser enabled
  • Tool profile

Also fixes the narrow-card Tool profile layout by stacking the label above the segmented control.

Screenshot

Quick Settings browser and tool profile controls

Real behavior proof

  • Behavior or issue addressed: Quick Settings now exposes Browser enabled and Tool profile controls, and the Tool profile row no longer overlaps/truncates in the narrow Security card.
  • Real environment tested: Local OpenClaw gateway v2026.5.10-beta.3 on Linux, dev Control UI served from http://localhost:5173, gateway on ws://127.0.0.1:18789.
  • Exact steps or command run after this patch: Opened the dev Control UI, navigated to Config -> Quick Settings, changed Browser enabled, changed Tool profile, saved, and observed the gateway restart/reconnect path.
  • Evidence after fix: Screenshot embedded above, plus redacted runtime log excerpt from the local gateway:
2026-05-11T09:44:25.938Z [ws] res ok config.set
2026-05-11T09:44:25.984Z [reload] config change detected; evaluating reload (meta.lastTouchedAt, browser.enabled)
2026-05-11T09:44:26.200Z [reload] config change requires gateway restart (browser.enabled)
2026-05-11T09:44:26.215Z [gateway] received SIGUSR1; restarting
2026-05-11T09:44:41.639Z [gateway] http server listening
2026-05-11T09:44:42.885Z [browser/server] Browser control listening on http://127.0.0.1:18791/ (auth=token)
2026-05-11T09:44:42.998Z [gateway] ready
  • Observed result after fix: Quick Settings saved successfully via config.set; changing Browser enabled triggered the required gateway restart; the gateway came back ready; browser control came back listening; post-save config state had plugins.entries.browser.enabled=true and tools.profile=full.
  • What was not tested: No additional gaps.

Automated verification

npm --prefix ui test -- src/ui/views/config-quick.test.ts

Result: 12 passed

@openclaw-barnacle openclaw-barnacle Bot added app: web-ui App: web-ui size: S triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels May 11, 2026
@clawsweeper

clawsweeper Bot commented May 11, 2026

Copy link
Copy Markdown
Contributor

Codex review: passed.

Summary
The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.

Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Real behavior proof
Sufficient (logs): Sufficient: the PR includes an after-fix UI screenshot plus redacted gateway logs showing save, restart, browser control recovery, and ready state for the changed behavior.

Next step before merge
No repair lane is needed; this automerge-opted PR has no narrow code defect for ClawSweeper to fix.

Security
Cleared: The diff only exposes existing operator config paths in the UI and adds tests plus a PNG proof asset; it does not change dependencies, workflows, secrets handling, downloads, or executable scripts.

Review details

Best possible solution:

Land this through the exact-head automerge and CI gates, preserving the existing config contracts and focused UI coverage.

Do we have a high-confidence way to reproduce the issue?

Not applicable; this PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Is this the best way to solve the issue?

Yes; wiring Quick Settings rows to the existing browser.enabled and tools.profile config paths is the narrow maintainable solution. A centralized source for tool profile labels would be a future cleanup, not a blocker for the current fixed option set.

What I checked:

Likely related people:

  • Shakker: Current-main blame and symbol history connect Shakker to the Quick Settings Security card, extractQuickSettingsSecurity, the browser config schema, and tool profile catalog that this PR extends. (role: recent area contributor; confidence: medium; commits: 497fed0e50bf, 62ace89c2333; files: ui/src/ui/views/config-quick.ts, ui/src/ui/app-render.ts, src/config/zod-schema.ts)

Remaining risk / open question:

  • I did not run the UI test command or live gateway in this read-only review; runtime validation is contributor-supplied and source-backed.
  • The tool profile option list is duplicated in the Quick Settings view, but it matches the current schema and catalog values.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 62ace89c2333.

@jeffjhunter jeffjhunter force-pushed the jjh/quick-settings-browser-tools branch 2 times, most recently from 2457581 to 847f018 Compare May 11, 2026 09:40
@openclaw-barnacle openclaw-barnacle Bot added the docs Improvements or additions to documentation label May 11, 2026
@jeffjhunter

jeffjhunter commented May 11, 2026

Copy link
Copy Markdown
Contributor Author

Summary

Adds Quick Settings controls for:

  • Browser enabled
  • Tool profile

Also fixes the narrow-card Tool profile layout by stacking the label above the segmented control.

Screenshot

Quick Settings browser and tool profile controls

Runtime Evidence

Verified manually in the dev Control UI at http://localhost:5173:

  • Changed Browser enabled from Quick Settings.
  • Changed/saved Tool profile from Quick Settings.
  • Save issued config.set.
  • Gateway detected browser.enabled changed, performed the required restart, and came back ready.
  • Post-save config state: plugins.entries.browser.enabled=true, tools.profile=full.

Sanitized gateway log excerpt:

2026-05-11T09:44:25.938Z [ws] res ok config.set
2026-05-11T09:44:25.984Z [reload] config change detected; evaluating reload (meta.lastTouchedAt, browser.enabled)
2026-05-11T09:44:26.200Z [reload] config change requires gateway restart (browser.enabled)
2026-05-11T09:44:26.215Z [gateway] received SIGUSR1; restarting
2026-05-11T09:44:41.639Z [gateway] http server listening
2026-05-11T09:44:42.885Z [browser/server] Browser control listening on http://127.0.0.1:18791/ (auth=token)
2026-05-11T09:44:42.998Z [gateway] ready

Automated Verification

npm --prefix ui test -- src/ui/views/config-quick.test.ts

Result: 12 passed

Re-review progress:

@openclaw-barnacle openclaw-barnacle Bot added proof: supplied External PR includes structured after-fix real behavior proof. and removed triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. labels May 11, 2026
@jeffjhunter jeffjhunter force-pushed the jjh/quick-settings-browser-tools branch from 847f018 to 06adff1 Compare May 11, 2026 09:51
@steipete

Copy link
Copy Markdown
Contributor

@clawsweeper automerge

@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge labels May 11, 2026
@clawsweeper

clawsweeper Bot commented May 11, 2026

Copy link
Copy Markdown
Contributor

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=06adff19fda5de6aef4f43b4dc35944425393f22)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-11T10:06:01Z
Merge commit: 3247b091d6ff

What merged:

  • The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
  • Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:

  • No ClawSweeper repair was needed after automerge opt-in.

The automerge loop is complete.

Automerge progress:

  • 2026-05-11 09:58:17 UTC review queued 06adff19fda5 (queued)
  • 2026-05-11 10:05:47 UTC review passed 06adff19fda5 (structured ClawSweeper verdict: pass (sha=06adff19fda5de6aef4f43b4dc35944425393...)
  • 2026-05-11 10:06:03 UTC merged 06adff19fda5 (merged by ClawSweeper automerge)

@clawsweeper clawsweeper Bot merged commit 3247b09 into openclaw:main May 11, 2026
87 of 90 checks passed
greench-ai pushed a commit to greench-ai/nexisclaw that referenced this pull request May 12, 2026
Summary:
- The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
- Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:
- No ClawSweeper repair was needed after automerge opt-in.

Validation:
- ClawSweeper review passed for head 06adff19fda5de6aef4f43b4dc35944425393f22.
- Required merge gates passed before the squash merge.

Prepared head SHA: 06adff19fda5de6aef4f43b4dc35944425393f22
Review: openclaw/openclaw#80609 (comment)

Co-authored-by: Jeff J Hunter <support@aipersonamethod.com>
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
Summary:
- The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
- Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:
- No ClawSweeper repair was needed after automerge opt-in.

Validation:
- ClawSweeper review passed for head 06adff19fda5de6aef4f43b4dc35944425393f22.
- Required merge gates passed before the squash merge.

Prepared head SHA: 06adff19fda5de6aef4f43b4dc35944425393f22
Review: openclaw/openclaw#80609 (comment)

Co-authored-by: Jeff J Hunter <support@aipersonamethod.com>
markfietje pushed a commit to markfietje/openclaw that referenced this pull request May 20, 2026
Summary:
- The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
- Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:
- No ClawSweeper repair was needed after automerge opt-in.

Validation:
- ClawSweeper review passed for head 06adff19fda5de6aef4f43b4dc35944425393f22.
- Required merge gates passed before the squash merge.

Prepared head SHA: 06adff19fda5de6aef4f43b4dc35944425393f22
Review: openclaw/openclaw#80609 (comment)

Co-authored-by: Jeff J Hunter <support@aipersonamethod.com>
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
Summary:
- The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
- Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:
- No ClawSweeper repair was needed after automerge opt-in.

Validation:
- ClawSweeper review passed for head 06adff1.
- Required merge gates passed before the squash merge.

Prepared head SHA: 06adff1
Review: openclaw#80609 (comment)

Co-authored-by: Jeff J Hunter <support@aipersonamethod.com>
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
Summary:
- The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
- Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:
- No ClawSweeper repair was needed after automerge opt-in.

Validation:
- ClawSweeper review passed for head 06adff1.
- Required merge gates passed before the squash merge.

Prepared head SHA: 06adff1
Review: openclaw#80609 (comment)

Co-authored-by: Jeff J Hunter <support@aipersonamethod.com>
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
Summary:
- The PR adds Browser enabled and Tool profile controls to Control UI Quick Settings, stacks the tool profile row for narrow cards, and adds focused UI tests plus a screenshot asset.
- Reproducibility: not applicable. This PR adds a Control UI quick-settings capability rather than fixing a reported reproducible bug. Source inspection and the supplied screenshot/log proof cover the changed behavior.

Automerge notes:
- No ClawSweeper repair was needed after automerge opt-in.

Validation:
- ClawSweeper review passed for head 06adff1.
- Required merge gates passed before the squash merge.

Prepared head SHA: 06adff1
Review: openclaw#80609 (comment)

Co-authored-by: Jeff J Hunter <support@aipersonamethod.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app: web-ui App: web-ui clawsweeper:automerge Maintainer opted this PR into bounded ClawSweeper-reviewed automerge docs Improvements or additions to documentation proof: sufficient ClawSweeper judged the real behavior proof convincing. proof: supplied External PR includes structured after-fix real behavior proof. size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants