Skip to content

fix(control-ui): add guarded dashboard shortcuts#84827

Open
BunsDev wants to merge 1 commit into
mainfrom
meow/issue-81946-dashboard-shortcuts
Open

fix(control-ui): add guarded dashboard shortcuts#84827
BunsDev wants to merge 1 commit into
mainfrom
meow/issue-81946-dashboard-shortcuts

Conversation

@BunsDev

@BunsDev BunsDev commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Refreshes the guarded Control UI dashboard shortcuts for #81946 on current origin/main.

  • Adds a centralized dashboard shortcut resolver for /, N, Esc, and existing Cmd/Ctrl+K palette behavior.
  • Keeps shortcut no-ops inside editable targets: inputs, textareas, selects, contenteditable roots, role="textbox", CodeMirror roots, and Monaco roots.
  • Preserves current-main chat session picker and mobile controls behavior, with Escape priority covering session picker before mobile controls.
  • Adds discoverability for the new-messages jump shortcut in the chat UI and a concise Control UI docs note.
  • Keeps the English chat.newMessages.* strings and regenerates locale bundles / .i18n metadata with pnpm ui:i18n:sync.
  • Drops the previous normal-PR CHANGELOG.md churn and unrelated docs/install/installer.md table churn.

Duplicate/source PR #81958 is closed stale-unmerged, so this PR remains the active fix path.

Verification

Behavior addressed: guarded dashboard keyboard shortcuts for Control UI chat/navigation without stealing typing from editable fields.

Real environment tested: local OpenClaw worktree on branch meow/refresh-84827-dashboard-shortcuts, rebased on origin/main 4f26cc9090, pushed as 3bec2bcca39c35ac417f498a23cbc70ba1aeb6a6.

Exact steps or command run after this patch:

  • node scripts/run-vitest.mjs ui/src/ui/navigation.browser.test.ts --reporter=verbose
  • node scripts/run-vitest.mjs ui/src/ui/views/chat.test.ts --reporter=verbose
  • pnpm exec oxfmt --check --threads=1 docs/web/control-ui.md ui/src/i18n/locales/en.ts ui/src/styles/chat/layout.css ui/src/ui/app-keyboard-shortcuts.ts ui/src/ui/app.ts ui/src/ui/navigation.browser.test.ts ui/src/ui/views/chat.test.ts ui/src/ui/views/chat.ts
  • node scripts/run-oxlint.mjs ui/src/ui/app-keyboard-shortcuts.ts
  • pnpm ui:i18n:check
  • pnpm tsgo:test:ui
  • git diff --check origin/main...HEAD
  • git merge-tree --write-tree origin/main HEAD
  • Testbox attempt: tbx_01kspf60p4nt0mn1ty963txesm / run 26555650911, command pnpm check:changed
  • Ambient blocker repro: from pristine origin/main, node scripts/run-tsgo.mjs -p test/tsconfig/tsconfig.core.test.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/core-test-origin-main-recheck.tsbuildinfo

Evidence after fix:

  • navigation.browser.test.ts: 18 passed.
  • chat.test.ts: 59 passed.
  • oxfmt, focused oxlint, ui:i18n:check, tsgo:test:ui, git diff --check, and merge-tree passed locally.
  • ui:i18n:check reported all 18 locales clean, with the expected 3 fallback keys from the regenerated shortcut strings.
  • PR CI on 3bec2bcca39c: check-lint, checks-node-core-ui, check-docs, preflight, security-fast, and workflow sanity checks passed.

Observed result after fix: / focuses/switches to chat, N jumps to new messages only when available, Escape dismisses transient dashboard UI in the expected priority order, Cmd/Ctrl+K still opens the command palette, and printable dashboard shortcuts do nothing from editable/editor targets.

What was not tested: browser smoke was not run. The broad changed gate did not complete: the Testbox wrapper detached with exit 143 and the backing Action was cancelled after it stayed in Run Testbox; the same check-test-types failure currently reproduces on pristine origin/main in unrelated src/agents/minimax-vlm.normalizes-api-key.test.ts and src/agents/tools/pdf-native-providers.test.ts files.

Copilot AI review requested due to automatic review settings May 21, 2026 05:37
@openclaw-barnacle openclaw-barnacle Bot added app: web-ui App: web-ui size: M maintainer Maintainer-authored PR labels May 21, 2026
@clawsweeper

clawsweeper Bot commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Codex review: found issues before merge. Reviewed July 1, 2026, 2:53 AM ET / 06:53 UTC.

Summary
The PR adds guarded Control UI dashboard shortcuts for composer focus, new-message jump, transient UI dismissal, and command-palette behavior, plus new-message shortcut discoverability, i18n output, docs, styles, and focused UI tests.

PR surface: Source +286, Tests +160, Docs +1. Total +447 across 44 files.

Reproducibility: yes. at source level: current main checks open composer details and realtime Talk options before mobile controls, while the PR head's dismiss path closes mobile controls first. I did not run tests because this review is read-only.

Review metrics: 1 noteworthy metric.

  • Escape Dismissal Branches: 2 current-main branches omitted. The PR's state/action path no longer models open composer details or realtime Talk options before closing mobile controls.

Root-cause cluster
Relationship: fixed_by_candidate
Canonical: #81946
Summary: This PR is the active candidate implementation for the canonical guarded Control UI dashboard shortcut issue; the earlier shortcut PR is closed unmerged, and the session-switching shortcut issue overlaps only in guard infrastructure.

Members:

Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything.

Merge readiness
Overall: 🦪 silver shellfish
Proof: 🌊 off-meta tidepool
Patch quality: 🦪 silver shellfish
Result: blocked by patch quality or review findings.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

  • Restore current-main nested Escape dismissal before closing mobile controls.
  • [P2] Clarify the docs sentence so Cmd/Ctrl+K is not claimed to be blocked in editable targets.
  • Refresh the branch against current main and rerun focused Control UI validation.

Mantis proof suggestion
A short browser proof would materially help verify visible Control UI shortcut behavior and editable-target guarding after the Escape-order fix. A maintainer can ask Mantis to capture proof by posting this exact PR comment:

@openclaw-mantis visual task: verify Control UI /, N, Esc, and Cmd/Ctrl+K shortcuts ignore editable fields and preserve nested chat popover dismissal.

Risk before merge

  • [P1] The PR head is currently DIRTY/CONFLICTING against main, so maintainers need a refreshed merge result before landing.
  • [P1] Escape would close the parent mobile controls before open composer detail menus or realtime Talk options, changing current chat dismissal behavior unless maintainers explicitly approve a new contract.
  • [P1] The docs currently claim every listed shortcut is blocked in editable controls even though Cmd/Ctrl+K remains active there.

Maintainer options:

  1. Restore Existing Escape Priority (recommended)
    Preserve current composer-details and realtime Talk option dismissal before closing mobile controls, then rerun focused Control UI checks on a refreshed branch.
  2. Approve A New Escape Contract
    Maintainers can intentionally accept the changed dismissal priority, but the new contract should be explicit in tests and docs before merge.
  3. Replace The Drifted Branch
    If the conflict and generated-output churn are too costly, keep the canonical issue open and carry the useful shortcut design into a fresh narrow branch.

Next step before merge

  • [P2] The remaining action is maintainer/author handling because the PR is protected, dirty/conflicting, and needs shortcut-contract approval after the concrete fixes.

Security
Cleared: No concrete security or supply-chain concern found; the diff is limited to Control UI runtime code, tests, styles, docs, and generated i18n metadata.

Review findings

  • [P2] Preserve Nested Escape Dismissal — ui/src/ui/app.ts:988-990
  • [P3] Clarify The Editable-Target Shortcut Docs — docs/web/control-ui.md:198
Review details

Best possible solution:

Land a refreshed branch that keeps the centralized dispatcher, preserves current-main nested Escape dismissal priority, narrows the docs wording for Cmd/Ctrl+K, and closes the canonical shortcut issue after merge.

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

Yes, at source level: current main checks open composer details and realtime Talk options before mobile controls, while the PR head's dismiss path closes mobile controls first. I did not run tests because this review is read-only.

Is this the best way to solve the issue?

No, not yet: the centralized dispatcher is a maintainable direction, but it needs the existing nested Escape branches, a docs correction, and a current-main refresh before merge.

Full review comments:

  • [P2] Preserve Nested Escape Dismissal — ui/src/ui/app.ts:988-990
    Current main closes open composer details and realtime Talk options before the broader mobile controls popover. This new branch closes the parent mobile controls as soon as chatMobileControlsOpen is true, so restore those nested dismissal branches before this case and cover them in the shortcut tests.
    Confidence: 0.9
  • [P3] Clarify The Editable-Target Shortcut Docs — docs/web/control-ui.md:198
    The resolver returns toggle-palette before the editable-target guard, and the PR test preserves Ctrl+K from a textarea. Narrow this sentence to printable dashboard shortcuts or call out the command-palette exception so the docs match behavior.
    Confidence: 0.86

Overall correctness: patch is incorrect
Overall confidence: 0.9

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against b2355ef6a226.

Label changes

Label justifications:

  • P2: This is a normal-priority Control UI ergonomics PR with limited surface area but concrete pre-merge behavior blockers.
  • merge-risk: 🚨 compatibility: Merging the current head could change existing Escape-key dismissal behavior for nested chat controls before maintainers approve a new contract.
  • rating: 🦪 silver shellfish: Overall readiness is 🦪 silver shellfish; proof is 🌊 off-meta tidepool and patch quality is 🦪 silver shellfish.
  • status: ⏳ waiting on author: ClawSweeper has contributor-facing work open and is waiting for author action. Not applicable: The external-contributor proof gate does not apply because this is a member-authored, maintainer-labeled PR, though the body records focused local UI checks and live PR checks are mostly green.
Evidence reviewed

PR surface:

Source +286, Tests +160, Docs +1. Total +447 across 44 files.

View PR surface stats
Area Files Added Removed Net
Source 41 398 112 +286
Tests 2 160 0 +160
Docs 1 1 0 +1
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 44 559 112 +447

What I checked:

  • Repository policy read: Root AGENTS.md plus scoped UI and docs guides were read fully; protected maintainer handling, generated Control UI i18n rules, and docs accuracy guidance apply to this review. (AGENTS.md:28, b2355ef6a226)
  • Current main Escape contract: Current main handles Escape by closing the chat session picker, then open composer detail menus, then realtime Talk options, and only then the broader mobile controls popover. (ui/src/ui/app.ts:762, b2355ef6a226)
  • PR head Escape regression: The PR head's dismissDashboardTransient path closes chatMobileControlsOpen before modeling the existing composer-details and realtime Talk option branches. (ui/src/ui/app.ts:988, 0aea33d82e6c)
  • Resolver state gap: DashboardShortcutState includes palette, session picker, mobile controls, nav drawer, session notice, sidebar, and onboarding state, but has no state for open composer details or realtime Talk options. (ui/src/ui/app-keyboard-shortcuts.ts:9, 0aea33d82e6c)
  • Docs mismatch: The docs say all listed shortcuts do not fire from editable controls, but the resolver handles Cmd/Ctrl+K before the editable-target guard and the PR test expects Ctrl+K to open the palette from a textarea. Public docs: docs/web/control-ui.md. (docs/web/control-ui.md:198, 0aea33d82e6c)
  • Live PR state: GitHub reports head 0aea33d, mergeable CONFLICTING, mergeStateStatus DIRTY, and labels include maintainer, P2, merge-risk compatibility, and waiting on author. (0aea33d82e6c)

Likely related people:

  • BunsDev: Authored the maintainer-scoped shortcut issue and this PR, and current-main history shows Control UI/chat/docs work on the same surface. (role: active related implementer and feature-history contributor; confidence: high; commits: 96635c7c27e3, 3dd0e8ed6a7c, 52370c59980b; files: ui/src/ui/app.ts, ui/src/ui/views/chat.ts, docs/web/control-ui.md)
  • vincentkoc: Recent app/chat commits touch composer, Talk, rendering, and Control UI behavior adjacent to the shortcut path. (role: recent current-main area contributor; confidence: medium; commits: f5482efbd75e, 9d7fd31dd35b, b3c946999d71; files: ui/src/ui/app.ts, ui/src/ui/views/chat.ts, ui/src/ui/app-render.ts)
  • shakkernerd: Recent navigation browser tests and adjacent UI/chat work cover the test harness and mobile-control behavior this PR changes. (role: adjacent Control UI test contributor; confidence: medium; commits: 862790fbd7f4, 0771a8ab6f87, b80f5c2ea55f; files: ui/src/ui/navigation.browser.test.ts, ui/src/ui/app.ts)
  • steipete: History shows Control UI sidebar shortcuts, composer controls, docs, and chat surface work near this shortcut and discoverability request. (role: adjacent Control UI shortcut and docs contributor; confidence: medium; commits: 84ec0c27bf75, 2b30951b8090, 12a56d4d46dd; files: ui/src/ui/navigation.browser.test.ts, ui/src/styles/chat/layout.css, docs/web/control-ui.md)
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

Copilot AI reviewed May 21, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a centralized, guarded document-level keyboard shortcut dispatcher for the Control UI dashboard, enabling common chat actions without interfering with text entry fields or embedded editors.

Changes:

  • Introduces resolveDashboardShortcutAction() to map key events (/, N, Esc, Cmd/Ctrl+K) to app actions with editable-target guarding.
  • Wires the dispatcher into OpenClawApp and implements focused handlers for toggling palette, focusing composer, scrolling to new messages, and dismissing transient UI.
  • Improves “New messages” discoverability with an on-button <kbd>N</kbd> indicator and adds regression coverage for shortcut dispatch/guarding.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
ui/src/ui/views/chat.ts Adds Escape handling for the search bar input; updates “New messages” button UI/labels to show the N shortcut.
ui/src/ui/views/chat.test.ts Adds a test asserting the new “New messages” shortcut discoverability and preserves click behavior.
ui/src/ui/navigation.browser.test.ts Adds browser-level regression tests for global shortcut routing, editable no-op guarding, and Escape dismissal behavior.
ui/src/ui/app.ts Replaces inline Cmd/Ctrl+K handler with centralized shortcut resolution and adds helper methods for the new actions.
ui/src/ui/app-keyboard-shortcuts.ts New module implementing guarded shortcut-to-action resolution and editable-target detection.
ui/src/styles/chat/layout.css Styles the new <kbd> indicator within the “New messages” button.
CHANGELOG.md Documents the new guarded dashboard shortcuts feature.

Comment thread ui/src/ui/views/chat.ts Outdated
Comment thread ui/src/ui/app.ts Outdated
@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. labels May 21, 2026
@clawsweeper

clawsweeper Bot commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

ClawSweeper PR egg: ✨ hatched 🥚 common Sunspot Crabkin. Rarity: 🥚 common. Trait: finds missing screenshots.

Details

Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Sunspot Crabkin in ClawSweeper.
Hatchability:

  • Merged PRs are hatchable.
  • Open PRs are hatchable when they are status: 👀 ready for maintainer look, status: 🚀 automerge armed, or labeled clawsweeper:automerge.
  • Closed unmerged PRs are hatchable only when one of those hatchable labels is still present in the durable record.

About:

  • Eggs appear after real-behavior proof passes. They are collectible flavor only.
  • Review momentum changes the shell state: follow-up work warms it, re-review makes it wobble, and a clean final review lets it hatch.
  • The hatch is seeded from this repository and PR number, so the same PR keeps the same creature; the reviewed head SHA can only change safe visual details.
  • Rarity is just collectible sparkle: 🥚 common, 🌱 uncommon, 💎 rare, ✨ glimmer, and 🌈 legendary.

@clawsweeper clawsweeper Bot added rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. P2 Normal backlog priority with limited blast radius. merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. and removed rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. labels May 21, 2026
@BunsDev
BunsDev force-pushed the meow/issue-81946-dashboard-shortcuts branch from d8b9c2b to a3464db Compare May 21, 2026 23:46
@openclaw-barnacle openclaw-barnacle Bot added the docs Improvements or additions to documentation label May 21, 2026
@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. and removed rating: 🧂 unranked krab Not merge-ready due to missing proof or serious correctness/safety concerns. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. labels May 21, 2026
@wyf027 wyf027 mentioned this pull request May 26, 2026
Merged
@BunsDev
BunsDev force-pushed the meow/issue-81946-dashboard-shortcuts branch from 350aa01 to c8f37f6 Compare May 28, 2026 04:52
@BunsDev

BunsDev commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Refreshed PR head to c8f37f6 and resolved the dirty merge state. Focused UI proof, i18n check, format, UI typecheck, and merge-tree are clean. Testbox changed gate tbx_01kspemnma51480h6q2bcwjwa3 is blocked by the current-main src/agents/tools/pdf-native-providers.test.ts core-test typecheck failure; the same command reproduces on pristine origin/main.

@clawsweeper

clawsweeper Bot commented May 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels May 28, 2026
@BunsDev
BunsDev force-pushed the meow/issue-81946-dashboard-shortcuts branch from c8f37f6 to 3bec2bc Compare May 28, 2026 05:06
@BunsDev

BunsDev commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper re-review

Final refreshed head is 3bec2bc, rebased on current origin/main 4f26cc9. The dirty merge state is resolved, generated i18n was refreshed through the repo sync command, and the old changelog / installer-doc churn is gone. Focused Control UI tests, UI typecheck, i18n check, format/lint, diff check, and merge-tree are clean.

Current red check is check-test-types in unrelated agent test files: src/agents/minimax-vlm.normalizes-api-key.test.ts and src/agents/tools/pdf-native-providers.test.ts. I reproduced the same failure on pristine origin/main with test/tsconfig/tsconfig.core.test.json, so this is not introduced by the Control UI shortcut diff. Testbox tbx_01kspf60p4nt0mn1ty963txesm detached/stalled before completion and was cancelled; details are in the PR body.

@clawsweeper

clawsweeper Bot commented May 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. and removed rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. labels May 28, 2026
@clawsweeper clawsweeper Bot mentioned this pull request May 28, 2026
Closed
@BunsDev

BunsDev commented Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper review

@clawsweeper clawsweeper Bot added rating: 🌊 off-meta tidepool PR readiness rating does not apply to this item. and removed rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jun 13, 2026
@BunsDev
BunsDev force-pushed the meow/issue-81946-dashboard-shortcuts branch from 3bec2bc to 0aea33d Compare June 13, 2026 05:21
@BunsDev

BunsDev commented Jun 13, 2026

Copy link
Copy Markdown
Contributor Author

@clawsweeper review

@clawsweeper

clawsweeper Bot commented Jun 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

Re-review progress:

@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. and removed rating: 🌊 off-meta tidepool PR readiness rating does not apply to this item. labels Jun 13, 2026
@clawsweeper clawsweeper Bot mentioned this pull request Jun 14, 2026
Open
@BunsDev BunsDev self-assigned this Jun 18, 2026
@clawsweeper clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. and removed rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. labels Jun 20, 2026
@vyctorbrzezowski vyctorbrzezowski mentioned this pull request Jul 8, 2026
Open
@clawsweeper clawsweeper Bot mentioned this pull request Jul 9, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@BunsDev BunsDev

Labels

app: web-ui App: web-ui docs Improvements or additions to documentation maintainer Maintainer-authored PR merge-risk: 🚨 compatibility 🚨 May break existing users, config, migrations, defaults, or upgrade paths. P2 Normal backlog priority with limited blast radius. rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. size: L status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@BunsDev