feat(tui): Command palette fuzzy-find any action with Ctrl+P or : prefix by jaeko44 · Pull Request #438 · virtengine/bosun

jaeko44 · 2026-03-25T18:19:55Z

Task-ID: 2316d321-5b65-4768-be5d-952e1c5cdec8\n\nAutomated PR for task 2316d321-5b65-4768-be5d-952e1c5cdec8

github-actions · 2026-03-25T18:20:16Z

Bosun CI signal: Bosun-created PR currently has failing checks.

Automation scope: Bosun-created
Workflow: CI
Conclusion: failure
Run: https://github.com/virtengine/bosun/actions/runs/23598714939
Head SHA: 0c8f8bb
Failed jobs: Build + Tests

Copilot

Pull request overview

Adds a TUI command palette that can fuzzy-find and execute actions, intended to open via Ctrl+P or : and remember recent commands across runs.

Changes:

Introduces command-palette helpers to build/rank actions and persist a small recent-history file.
Wires a new Ink CommandPalette component into the TUI app, including workflow template fetching and action execution.
Adds unit + integration test coverage for palette ranking, history, and basic execution.

Reviewed changes

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

Show a summary per file

File	Description
ui/tui/fuzzy.js	Adds a small re-export shim for fuzzy scoring helpers.
tui/lib/command-palette.mjs	New helpers for palette actions, fuzzy ranking, and persisted history.
tui/app.mjs	Integrates palette open/close, action execution, workflow fetching, and recent-history wiring.
tui/CommandPalette.js	New Ink UI component for querying and selecting actions.
tests/tui/command-palette.test.mjs	Unit tests for ranking and history helpers.
tests/tui/command-palette.integration.test.mjs	Ink integration test for opening palette and executing an action.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-25T18:26:08Z

tui/app.mjs

+  const handleInput = useCallback((input, key) => {
+    if (key?.ctrl && String(input || "") === "p") {
+      setPaletteOpen(true);
+      return;
+    }


Ctrl+P detection is checking input === "p", but Ink typically reports Ctrl key combos as a control character in input (e.g. Ctrl+P is \u0010) with the letter in key.name. This will likely prevent the palette from opening in real terminals. Prefer checking key.ctrl && key.name === "p" (or matching the control sequence) instead of comparing input to "p".

Copilot · 2026-03-25T18:26:08Z

tui/app.mjs

+    if (screenInputLocked) {
+      if (key?.ctrl && String(input || "") === "p") {
+        setPaletteOpen(true);
+      }
+      return;


Same Ctrl+P handling issue as above inside the screenInputLocked branch: comparing input to "p" may not match Ctrl+P. Use key.name / key.sequence so the palette can still open while inputs are captured by a screen.

Copilot · 2026-03-25T18:26:08Z

tui/app.mjs

+async function fetchJson(url, init = {}) {
+  const response = await fetch(url, init);
+  const payload = await response.json().catch(() => ({}));
+  if (!response.ok || payload?.ok === false) {
+    throw new Error(payload?.error || `Request failed: ${response.status}`);
+  }
+  return payload;
+}


fetchJson duplicates logic that already exists in TuiWsBridge.requestJson() (including https/ws protocol mapping and Bearer token injection). Using hard-coded http://${host}:${port} here can break when the UI server is https/wss or requires the token header. Consider routing these requests through bridge.requestJson() (with a fallback for injected mock clients) instead of direct fetch().

Copilot · 2026-03-25T18:26:08Z

tui/app.mjs

+      } else if (action.type === "config") {
+        if (action.id.startsWith("config:refresh:")) {
+          setRefreshCountdownSec(Number(action.id.split(":").at(-1) || 2));
+        }


Config palette actions currently only call setRefreshCountdownSec(...), which changes the displayed countdown but doesn’t update the underlying refreshMs value used by resetRefreshCountdown() (so it can revert on the next stats tick). Either make refresh interval configurable via state and use it consistently, or remove these config actions until they have an effect.

Suggested change

} else if (action.type === "config") {

if (action.id.startsWith("config:refresh:")) {

setRefreshCountdownSec(Number(action.id.split(":").at(-1) || 2));

}

Copilot · 2026-03-25T18:26:09Z

ui/tui/fuzzy.js

@@ -0,0 +1 @@
+export { scoreFuzzyMatch, rankFuzzyMatches } from "../../tui/lib/fuzzy-score.mjs";


This file re-exports fuzzy helpers but isn’t referenced anywhere in the repo (no imports found). If it’s not needed, consider removing it; if it is intended as a shared entrypoint, add the corresponding import(s) or documentation so it’s clear why it exists.

Copilot · 2026-03-25T18:26:09Z

tui/lib/command-palette.mjs

+  for (const task of Array.isArray(tasks) ? tasks : []) {
+    const taskId = String(task?.id || "").trim();
+    if (!taskId) continue;
+    for (const action of TASK_ACTIONS) {
+      actions.push(actionWithSearch({
+        id: `task:${action.key}:${taskId}`,
+        type: "task",
+        command: action.key,
+        icon: action.icon,
+        label: action.key === "create" ? `${action.label}` : `${action.label} ${task.title || taskId}`,
+        shortcut: action.shortcut,
+        context: action.key === "create" ? "Tasks" : `Task ${taskId}`,
+        payload: { taskId, task },
+        keywords: [task.title, task.status, taskId],
+      }));
+    }


TASK_ACTIONS includes create, and the per-task loop generates a "Create task" action for every existing task (task:create:<taskId>). This produces many duplicate create entries (plus the separate task:create:new action) and can crowd out the result list. Consider removing create from the per-task loop (handle create as a single global action) so only update/delete are task-scoped.

Copilot · 2026-03-25T18:26:09Z

tui/lib/command-palette.mjs

+  { key: "config:connectOnly:on", label: "Enable connect-only mode", icon: "⚙", context: "Config" },
+  { key: "config:connectOnly:off", label: "Disable connect-only mode", icon: "⚙", context: "Config" },


CONFIG_ACTIONS defines connect-only toggles, but executePaletteAction in tui/app.mjs doesn’t implement any behavior for config:connectOnly:on/off. Either implement these actions end-to-end or remove them from the palette so users don’t select a no-op command.

Suggested change

{ key: "config:connectOnly:on", label: "Enable connect-only mode", icon: "⚙", context: "Config" },

{ key: "config:connectOnly:off", label: "Disable connect-only mode", icon: "⚙", context: "Config" },

github-actions · 2026-03-25T23:08:11Z

Bosun PR classification: Bosun-created.
This PR is tracked by Bosun attachment automation.

PR class label: bosun-pr-bosun-created
Attach label: bosun-attached (yes, because Bosun-created PRs remain attached regardless of human PR attach mode)
Attach mode policy: all
Trusted author: no
Bosun-created label present: yes
Automation scope: Eligible for Bosun repair and merge automation (Bosun-created PR).
CI failure signal label: bosun-needs-fix
CI failure marker:
Trigger: pull_request_target / synchronize

…uzzy-find-any-action-w

Co-authored-by: bosun-ve[bot] <262908237+bosun-ve[bot]@users.noreply.github.com>

jaeko44 added the bosun-attached Bosun PR attachment marker label Mar 25, 2026

Copilot AI review requested due to automatic review settings March 25, 2026 18:19

Copilot started reviewing on behalf of jaeko44 March 25, 2026 18:20 View session

github-actions bot added the bosun-needs-fix Attached PR with failing CI that Bosun should pick up for repair label Mar 25, 2026

Copilot AI reviewed Mar 25, 2026

View reviewed changes

jaeko44 added 2 commits March 26, 2026 10:07

chore: auto-commit agent work (2316d321-5b6)

c8ab938

chore: auto-commit agent work (2316d321-5b6)

63dae12

jaeko44 force-pushed the task/2316d3215b65-feat-tui-command-palette-fuzzy-find-any-action-w branch from 8ef73c2 to 63dae12 Compare March 25, 2026 23:08

github-actions bot added the bosun-pr-public PR observed by Bosun but not trusted for high-risk automation label Mar 25, 2026

Merge branch 'main' into task/2316d3215b65-feat-tui-command-palette-f…

0490027

…uzzy-find-any-action-w

jaeko44 added bosun-pr-bosun-created PR created by Bosun and eligible for Bosun automation and removed bosun-pr-public PR observed by Bosun but not trusted for high-risk automation labels Mar 26, 2026

jaeko44 and others added 4 commits March 26, 2026 22:46

Merge branch 'main' into task/2316d3215b65-feat-tui-command-palette-f…

d35e049

…uzzy-find-any-action-w

Merge remote-tracking branch 'origin/main' into HEAD

0f9448b

Co-authored-by: bosun-ve[bot] <262908237+bosun-ve[bot]@users.noreply.github.com>

Merge remote-tracking branch 'origin/main' into HEAD

a07d755

Co-authored-by: bosun-ve[bot] <262908237+bosun-ve[bot]@users.noreply.github.com>

Merge remote-tracking branch 'origin/main' into HEAD

0c8f8bb

Co-authored-by: bosun-ve[bot] <262908237+bosun-ve[bot]@users.noreply.github.com>

jaeko44 added bosun-needs-fix Attached PR with failing CI that Bosun should pick up for repair and removed bosun-needs-fix Attached PR with failing CI that Bosun should pick up for repair labels Mar 26, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(tui): Command palette fuzzy-find any action with Ctrl+P or : prefix#438

feat(tui): Command palette fuzzy-find any action with Ctrl+P or : prefix#438
jaeko44 wants to merge 7 commits intomainfrom
task/2316d3215b65-feat-tui-command-palette-fuzzy-find-any-action-w

jaeko44 commented Mar 25, 2026

Uh oh!

github-actions bot commented Mar 25, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

Copilot AI Mar 25, 2026

Uh oh!

github-actions bot commented Mar 25, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1 @@
		export { scoreFuzzyMatch, rankFuzzyMatches } from "../../tui/lib/fuzzy-score.mjs";

		{ key: "config:connectOnly:on", label: "Enable connect-only mode", icon: "⚙", context: "Config" },
		{ key: "config:connectOnly:off", label: "Disable connect-only mode", icon: "⚙", context: "Config" },

Conversation

jaeko44 commented Mar 25, 2026

Uh oh!

github-actions bot commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions bot commented Mar 25, 2026 •

edited

Loading

github-actions bot commented Mar 25, 2026 •

edited

Loading