[work-qcmz] irssi input: Up/Down history and Tab completion for /quote #61

Merged
keeper merged 1 commit from work-qcmz-preview-irssi-input-recalls-history-with into main 2026-10-08 20:12:52 +00:00
Collaborator

Bead: work-qcmz

In the irssi preview pane, /quote was cleared on Enter and had to be typed again.

  • Up / Down walk past input lines, like irssi. Duplicate lines in a row are stored once. Down past the newest clears the field.
  • Tab completes a partial command (/q → /quote). Tab only stops focus moving when there is something to complete, so keyboard users can still tab out.

Tests: new cases in studio/tests/irc-quote.test.js. All CI tests pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_013cRBa6A8HNzBs1YzqaFxaU

Bead: work-qcmz In the irssi preview pane, `/quote` was cleared on Enter and had to be typed again. - **Up / Down** walk past input lines, like irssi. Duplicate lines in a row are stored once. Down past the newest clears the field. - **Tab** completes a partial command (`/q` → `/quote`). Tab only stops focus moving when there is something to complete, so keyboard users can still tab out. Tests: new cases in `studio/tests/irc-quote.test.js`. All CI tests pass locally. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_013cRBa6A8HNzBs1YzqaFxaU
[work-qcmz] irssi input: Up/Down history and Tab completion for /quote
All checks were successful
Mossfire web app / test (pull_request) Successful in 40s
Mossfire web app / publish (theme.home.dpis.dk, home) (pull_request) Has been skipped
Mossfire web app / publish (tema.uhyre.dk, global) (pull_request) Has been skipped
0cc155f1aa
Up and Down recall past input lines, like irssi.
Tab completes a partial command such as /q to /quote.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013cRBa6A8HNzBs1YzqaFxaU
keeper approved these changes 2026-10-08 20:12:49 +00:00
keeper left a comment

Keeper review (opencode/deepseek-v4.1-flash, second opinion; author isaac).

Routine: static studio preview UI only. No data loss, security, or lockout surface. No repo review:human rule, no label, no human requested.

Checked:

  • Diff: studio/js/scene.js, studio/tests/irc-quote.test.js, studio/README.md.
  • History: Enter trims, stores (consecutive dupes collapsed), resets position; Up/Down clamp at both ends and Down past newest clears; preventDefault only when history exists so caret moves otherwise.
  • Tab: completes /q to /quote, avoids preventDefault when nothing to complete so focus can still move.
  • Tests: full studio suite (20 files) passes locally; CI run #2134 on 0cc155f is green.
Keeper review (opencode/deepseek-v4.1-flash, second opinion; author isaac). Routine: static studio preview UI only. No data loss, security, or lockout surface. No repo review:human rule, no label, no human requested. Checked: - Diff: studio/js/scene.js, studio/tests/irc-quote.test.js, studio/README.md. - History: Enter trims, stores (consecutive dupes collapsed), resets position; Up/Down clamp at both ends and Down past newest clears; preventDefault only when history exists so caret moves otherwise. - Tab: completes /q to /quote, avoids preventDefault when nothing to complete so focus can still move. - Tests: full studio suite (20 files) passes locally; CI run #2134 on 0cc155f is green.
keeper merged commit ec16a908f7 into main 2026-10-08 20:12:52 +00:00
keeper deleted branch work-qcmz-preview-irssi-input-recalls-history-with 2026-10-08 20:12:52 +00:00
Sign in to join this conversation.
No reviewers
No labels
review:human
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
daniel/mossfire!61
No description provided.