small: every list that asks for a decision answers j/k, enter, esc and `?` #10717

The asks list now takes j/k/enter, and esc on an opened row goes back to the list with that row focused again. I did not make ? open the help sheet on the first press, and I have no screenshot. I never opened the page in a browser. The new tests read source text and rendered HTML, so the Esc return and the list refocus have not been run in a browser. The focused tests pass, as do the type check, design-lint and the gate script. test/keys.test.ts skipped its ssh-keygen cases here.

What was already there. The shell in src/web/client.ts already binds j/k, o/enter, ? (help sheet) and esc, and it ignores keys while you type in a field. The pulls and issues lists already mark their rows data-nav. src/web/pick.ts is a checkbox filter and binds no list keys, so I left it alone.

What changed

• Asks list: every ask title is now a data-nav link, both the single-ask card and the grouped rows, so j/k/enter work there (src/web/studio/asks_page.ts).
• Esc returns without losing the place: opening a row from a list stores where you came from in sessionStorage, per tab. On that row's page, esc goes back to the list, and the list refocuses the row you opened. It only fires on the page that row opened. It does not fire while you're typing in a field, inside a dialog, or on a page that has its own rows.
• Size: the added client code is under 1 KB gzipped, and a test pins that.
• Tests: test/keys-decide.test.ts covers the asks rows, the issues rows, the Esc return, Esc not firing while typing, and the size limit.

Left out

• ? still lays key hints over the page on the first press and opens the help sheet on the second. I kept that because it is the existing, tested behaviour. The sheet is already built from the keys on the page.
• There is no screenshot in this pull.

Try it: open /asks, press j then enter, then esc. Do the same on an issues list and a pulls list.

🤖 Generated with Claude Code

Closes #10710

The asks list now takes j/k/enter, and esc on an opened row goes back to the list with that row focused again. I did not make ? open the help sheet on the first press, and I have no screenshot. I never opened the page in a browser. The new tests read source text and rendered HTML, so the Esc return and the list refocus have not been run in a browser. The focused tests pass, as do the type check, design-lint and the gate script. test/keys.test.ts skipped its ssh-keygen cases here.

What was already there. The shell in src/web/client.ts already binds j/k, o/enter, ? (help sheet) and esc, and it ignores keys while you type in a field. The pulls and issues lists already mark their rows data-nav. src/web/pick.ts is a checkbox filter and binds no list keys, so I left it alone.

What changed

• Asks list: every ask title is now a data-nav link, both the single-ask card and the grouped rows, so j/k/enter work there (src/web/studio/asks_page.ts).
• Esc returns without losing the place: opening a row from a list stores where you came from in sessionStorage, per tab. On that row's page, esc goes back to the list, and the list refocuses the row you opened. It only fires on the page that row opened. It does not fire while you're typing in a field, inside a dialog, or on a page that has its own rows.
• Size: the added client code is under 1 KB gzipped, and a test pins that.
• Tests: test/keys-decide.test.ts covers the asks rows, the issues rows, the Esc return, Esc not firing while typing, and the size limit.

Left out

• ? still lays key hints over the page on the first press and opens the help sheet on the second. I kept that because it is the existing, tested behaviour. The sheet is already built from the keys on the page.
• There is no screenshot in this pull.

Try it: open /asks, press j then enter, then esc. Do the same on an issues list and a pulls list.

🤖 Generated with Claude Code

Asks, issues and pulls lists now answer j/k, Enter, Esc and ? as the issue asks. I couldn't open a browser here, so box 5 (before/after screenshot) is still undone.

I ran tsc, gate.ts, design-lint.ts, the new DOM tests and the 30 test files that load client.ts; all pass. scripts/fast.ts found the diff too broad to name tests, so I picked those files by hand. The full suite is left to the forge's check.

What changed

• ? (item 1): it opens the one help sheet on the first press. Option still lays the key hints over the page. I updated test/hotkey-hints.test.ts and docs/features/hotkey-hints.md to match.
• Esc order (items 3 and 6): Esc now closes the thing on top before it leaves. If hints are showing, that Esc only clears them. An open <dialog>, an open menu, a field, or an Esc another handler already consumed also keeps it. Only a bare Esc on the opened page returns to the list.
• Stored place (item 7): JSON.parse of the stored place is now guarded in one function, fromPlace(), used by both the refocus on load and leaveToList. A malformed value, or one whose list isn't a same-site path (this check is new), counts as no place. It no longer throws inside the key handler.
• Tests (items 3 and 4): test/keys-decide.test.ts now runs the real shellJs in happy-dom and drives real key events:

- j/k/Enter, and Esc back to the list. - The list refocusing the opened row once. - Esc doing nothing with a dialog, an open menu, hints or a field active. - Malformed and off-site stored places. - ? opening the sheet on the first press. - The issues list the server draws carries data-nav rows.

• Docs (item 2): new docs/features/decision-keys.md.

Where I disagree or couldn't do what was asked

• Item 4, pulls list: it already had data-nav on its rows (changesList in src/web/studio/lists.ts). The old "issues list" test read pages/issues.ts, which is the repository picker, so it proved nothing. The fixture has no open pull to render, so the pulls rows are checked by source pattern only.
• Item 2, terminal screens: I added none. The asks pane (dashboard.ts) and the issues/pulls list (issues.ts, one screen for both kinds) are ListState lists with j/k and Enter, and esc goes back in app.ts. I read that from the code and ran no terminal test.
• Item 5, screenshot: the sandbox can't drive a browser, so the pull has none.

How to try it: open the preview, go to /asks, press j, then Enter, then Esc, and the row you opened is focused again. Press ? once and the sheet opens.

🤖 Generated with Claude Code

Left #10710 open: 1 unticked item remain (A test per behaviour; client script stays small (measured, no growth over 1 KB gzipped beyond what the keys need); design-lint passes; before/after screenshot in the pull.).