feat(results): ⌘A in a raw result pane selects just that text

In TSV/JSON output mode the result is a plain <div>, so ⌘A fell through to the
browser's whole-page select-all. Make the raw (.raw-text-view) and JSON
(.json-view) panes focusable (tabindex=0) and, when ⌘/Ctrl+A fires with focus
inside one, select that node's contents via Selection.selectAllChildren so it
can be copied. Focus elsewhere (e.g. the editor textarea) still falls through to
the native select-all, so ⌘A in the SQL area keeps selecting the whole query.

Verified in-browser: ⌘A in a focused TSV pane selects exactly the TSV text; ⌘A
in the editor is not intercepted. 325 tests pass; shortcuts/results at 100%.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QennTvGKAtJZrv9EpQagef

Author: 2 people

src/styles.css

@@ -348,6 +348,8 @@ body {
   color: var(--fg); white-space: pre;
   background: var(--bg);
 }
+/* Focusable (tabindex) so a click scopes ⌘A to the pane's text; no focus ring. */
+.raw-text-view:focus, .json-view:focus { outline: none; }
 
 /* ------------ chart placeholder ------------ */
 .chart-view {

src/ui/results.js

@@ -33,7 +33,7 @@ export function renderResults(app) {
   } else if (r.error) {
     inner.appendChild(h('div', { class: 'results-error' }, r.error));
   } else if (r.rawText != null) {
-    inner.appendChild(h('div', { class: 'raw-text-view' }, r.rawText));
+    inner.appendChild(h('div', { class: 'raw-text-view', tabindex: '0' }, r.rawText));
   } else if (r.rows.length === 0) {
     inner.appendChild(h('div', { class: 'placeholder' }, h('div', null, 'Query returned 0 rows.')));
   } else if (app.state.resultView === 'json') {
@@ -87,7 +87,7 @@ export function renderJson(r) {
     r.columns.forEach((c, i) => { o[c.name] = row[i]; });
     return o;
   });
-  return h('div', { class: 'json-view' }, JSON.stringify(arr, null, 2));
+  return h('div', { class: 'json-view', tabindex: '0' }, JSON.stringify(arr, null, 2));
 }
 
 export function renderTable(app, r) {

src/ui/shortcuts.js

@@ -73,6 +73,17 @@ export function handleKeydown(e, app) {
     app.actions.toggleSaved();
     return 'toggleSaved';
   }
+  if (mod && e.key.toLowerCase() === 'a') {
+    // Inside a raw result pane (TSV / JSON output), select just that text so it
+    // can be copied — not the whole page. Elsewhere (e.g. the editor textarea)
+    // fall through to the browser's native select-all.
+    const t = e.target;
+    const box = t && t.closest && t.closest('.raw-text-view, .json-view');
+    if (!box) return null;
+    e.preventDefault();
+    box.ownerDocument.defaultView.getSelection().selectAllChildren(box);
+    return 'selectAll';
+  }
   if (e.key === '?' && !mod) {
     const t = e.target;
     if (t && (t.tagName === 'INPUT' || t.tagName === 'TEXTAREA' || t.isContentEditable)) return null;

tests/unit/shortcuts.test.js

@@ -89,4 +89,27 @@ describe('handleKeydown', () => {
     expect(handleKeydown(ev({ key: 'x' }), app)).toBeNull();
     expect(handleKeydown(ev({ key: '?', target: null }), makeApp())).toBe('shortcuts');
   });
+
+  it('⌘A inside a raw result pane selects just that text', () => {
+    const app = makeApp();
+    const box = document.createElement('div');
+    box.className = 'raw-text-view';
+    box.textContent = 'a\tb\nc\td';
+    document.body.appendChild(box);
+    const e = ev({ metaKey: true, key: 'a', target: box });
+    expect(handleKeydown(e, app)).toBe('selectAll');
+    expect(e.preventDefault).toHaveBeenCalled();
+    expect(box.ownerDocument.defaultView.getSelection().toString()).toBe('a\tb\nc\td');
+  });
+
+  it('⌘A elsewhere falls through to the native select-all', () => {
+    const app = makeApp();
+    // editor textarea (no raw-pane ancestor) → not handled
+    const ta = document.createElement('textarea');
+    const e = ev({ metaKey: true, key: 'A', target: ta });
+    expect(handleKeydown(e, app)).toBeNull();
+    expect(e.preventDefault).not.toHaveBeenCalled();
+    // no target at all
+    expect(handleKeydown(ev({ metaKey: true, key: 'a', target: null }), app)).toBeNull();
+  });
 });