@mjasnikovs/pi-task 0.14.13 → 0.14.15

package/dist/task/question-box.js

@@ -17,8 +17,12 @@
  */
 import { getKeybindings, visibleWidth, wrapTextWithAnsi } from '@earendil-works/pi-tui';
 // ─── Constants ───────────────────────────────────────────────────────────────
-/** Left gutter: two columns reserved for the "▸ " cursor so boxes stay aligned. */
+/** Left gutter: two columns reserved for the cursor bar so boxes stay aligned. */
 const GUTTER = 2;
+/** Cap the dialog at a readable column count. Without this the boxes stretch the
+ *  full terminal width on wide/ultrawide screens — long question lines become
+ *  hard to scan and short answers leave a huge empty right margin inside the box. */
+const MAX_CONTENT_WIDTH = 96;
 const RECOMMENDED_TAG = 'RECOMMENDED';
 /** Smallest box that still fits "┌─ RECOMMENDED ─┐". Below this we drop the tag. */
 const TAG_MIN_BOX_WIDTH = visibleWidth(`┌─ ${RECOMMENDED_TAG} ─┐`);
@@ -49,9 +53,10 @@ function renderCard(card, selected, boxWidth, c) {
     const bodyLines = wrapTextWithAnsi(card.label, innerWidth);
     const body = (bodyLines.length > 0 ? bodyLines : ['']).map(line => borderFn('│ ') + padTo(line, innerWidth) + borderFn(' │'));
     const bottom = borderFn(`└${dash(boxWidth - 2)}┘`);
-    const arrow = selected ? c.arrow('▸') + ' ' : '  ';
-    const pad = '  ';
-    return [arrow + top, ...body.map(b => pad + b), pad + bottom];
+    // A full-height accent bar in the gutter marks the selected card — far more
+    // visible than a single "▸" glyph on the top border alone.
+    const cursor = selected ? c.arrow('▌') + ' ' : '  ';
+    return [cursor + top, ...body.map(b => cursor + b), cursor + bottom];
 }
 /**
  * Lay the whole dialog out for `width` columns: a QUESTION label + wrapped
@@ -59,7 +64,7 @@ function renderCard(card, selected, boxWidth, c) {
  * highlighted with an accent border and "▸" arrow), and a key-hint footer.
  */
 export function renderQuestionBox(width, question, cards, selected, c, hintText = HINT_TEXT) {
-    const W = Math.max(width, 12);
+    const W = Math.min(Math.max(width, 12), MAX_CONTENT_WIDTH);
     const boxWidth = Math.max(6, W - GUTTER);
     const out = [''];
     out.push('  ' + c.questionLabel(QUESTION_LABEL));

package/dist/workers/brave-warning.d.ts

@@ -0,0 +1,10 @@
+/**
+ * One-line startup hint shown when no Brave Search key is configured.
+ *
+ * pi-task's web-search worker needs BRAVE_SEARCH_API_KEY (or BRAVE_API_KEY).
+ * When neither is set we surface a single, unobtrusive widget line on session
+ * start so the user knows search is disabled — it never blocks work and clears
+ * itself on the first interaction (any keystroke).
+ */
+import type { ExtensionAPI } from '@earendil-works/pi-coding-agent';
+export declare function registerBraveKeyWarning(pi: ExtensionAPI): void;

package/dist/workers/brave-warning.js

@@ -0,0 +1,46 @@
+/**
+ * One-line startup hint shown when no Brave Search key is configured.
+ *
+ * pi-task's web-search worker needs BRAVE_SEARCH_API_KEY (or BRAVE_API_KEY).
+ * When neither is set we surface a single, unobtrusive widget line on session
+ * start so the user knows search is disabled — it never blocks work and clears
+ * itself on the first interaction (any keystroke).
+ */
+const WIDGET_KEY = 'pi-task-brave-warning';
+const WARNING = '⚠ pi-task: BRAVE_SEARCH_API_KEY not set — web search is disabled. '
+    + 'Get a free key at https://api.search.brave.com/app/keys';
+/** Mirrors the lookup in search-core so the hint matches what the worker reads. */
+function hasBraveKey() {
+    return Boolean(process.env.BRAVE_SEARCH_API_KEY ?? process.env.BRAVE_API_KEY);
+}
+export function registerBraveKeyWarning(pi) {
+    pi.on('session_start', (_event, ctx) => {
+        // Terminal-only hint: needs an interactive TUI to render and to catch the
+        // keystroke that dismisses it. Skip when a key is already present.
+        if (ctx.mode !== 'tui' || hasBraveKey())
+            return;
+        let unsubscribe = null;
+        const clear = () => {
+            try {
+                ctx.ui.setWidget(WIDGET_KEY, undefined);
+            }
+            catch {
+                /* stale ctx after a session switch — nothing to clear */
+            }
+            unsubscribe?.();
+            unsubscribe = null;
+        };
+        try {
+            ctx.ui.setWidget(WIDGET_KEY, [ctx.ui.theme.fg('warning', WARNING)]);
+        }
+        catch {
+            return;
+        }
+        // Disappear on any interaction — the first raw keystroke clears it.
+        // Returning undefined leaves the input untouched (we only observe it).
+        unsubscribe = ctx.ui.onTerminalInput(() => {
+            clear();
+            return undefined;
+        });
+    });
+}

package/dist/workers/index.js

@@ -2,9 +2,11 @@ import { registerPiWorker } from './pi-worker.js';
 import { registerPiWorkerSearch } from './pi-worker-search.js';
 import { registerPiWorkerFetch } from './pi-worker-fetch.js';
 import { registerPiWorkerDocs } from './pi-worker-docs.js';
+import { registerBraveKeyWarning } from './brave-warning.js';
 export function registerWorkers(pi) {
     registerPiWorker(pi);
     registerPiWorkerSearch(pi);
     registerPiWorkerFetch(pi);
     registerPiWorkerDocs(pi);
+    registerBraveKeyWarning(pi);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.14.13",
+    "version": "0.14.15",
     "description": "Deterministic spec-orchestration for local models, with a bundled real-time remote web view and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",