Commit Graph

793 Commits

Author SHA1 Message Date
73d42a812a v0.8.0: cl-tty input primitives, on-key keyword dispatch, XDG tangle paths, remove croatoan
- read-raw-byte: sb-unix:unix-read instead of read-char-no-hang
- raw-mode: sb-posix:tcsetattr instead of stty
- read-event: no probe-file /dev/stdin guard
- on-key: accepts &key ctrl alt shift code
- .asd: :croatoan dropped, :cl-tty added
- script: detection fix (empty lisp/ -> XDG)
2026-05-18 13:04:26 -04:00
e04b12c31c v0.8.0: TUI stabilization, command palette reverse-video highlight, hint bar redesign
- ROADMAP: consolidate all TUI work under v0.8.0 (removed premature
  v0.9.0/v0.10.x labels), restored original v0.9.0 eval harness plan
- channel-tui-view.org: Emacs-style reverse-video cursor (swap fg/bg
  instead of drawing █), hint bar now shows F:focus/MCP:count on left
  and token gauge + keybindings on right, sidebar reorganized to show
  GATE TRACE, RULES + BLOCK COUNT, COST, FILES panels
- channel-tui-main.org: command palette selection now uses reverse-video
  highlight (bg-input fg on input-fg bg, matching cursor style), fixed
  cond order so sel-p is checked before cat (all items had :category
  making sel-p unreachable), added session-cost extraction from daemon
- passepartout: export COLORTERM=truecolor for modern backend detection
2026-05-17 15:37:40 -04:00
2fedbbcb3b fix: solid amber █ cursor with finish-output flush
Replaced reverse-video cursor with a solid █ block in :input-prompt
color (amber). Drawn at the insertion point every frame from redraw
and main loop. finish-output ensures the cursor escape reaches the
terminal immediately.
2026-05-16 18:29:49 -04:00
c568ac6842 feat: Emacs-style reverse-video cursor (solid, no blink)
Replaced the software blinking █ cursor with a reverse-video cursor
that swaps foreground and background colors at the insertion point.
Solid at all times — no blink logic, no state tracking, no flicker.
- Removed duplicate cursor-visible-p functions
- Removed software cursor draw from view-input
- Removed terminal cursor style/show from initialize-backend
- position-cursor draws character at cursor with :bg fg + :input-fg bg
2026-05-16 18:22:01 -04:00
aca3f9e314 fix: draw cursor immediately in redraw (not 100ms later)
position-cursor now called at end of redraw so the cursor appears
on the very first frame and after every keypress without a 100ms
delay. Also still called from main loop between sleep for blinking.
2026-05-16 18:08:30 -04:00
5444322bf9 fix: software █ cursor blinking at 2Hz independently of typing
position-cursor runs every frame from main loop (after sleep 0.1),
drawing █ when cursor-visible-p returns T, space when NIL.
This creates a true 2Hz blink that toggles independently of keypresses.
Terminal cursor also set to blinking block as fallback.
2026-05-16 17:56:42 -04:00
f8ae4ac817 fix: terminal cursor instead of software-drawn █
Replaced software cursor (draw-text █ every frame) with native terminal
cursor (position-cursor using cursor-move + cursor-style). Terminal handles
blinking natively at 500ms — no redraw needed for cursor updates.

- position-cursor: computed input insertion point from state, calls
  cursor-move + cursor-style (:block :blink t) + cursor-show.
- Called from main loop every frame after (sleep 0.1), outside
  redraw's begin-sync/end-sync. No flicker.
2026-05-16 17:50:08 -04:00
7eca785b0a fix: TUI startup — bash script debugger-hook and tangled file regen 2026-05-16 17:43:20 -04:00
7e9da0f867 v0.10.5: multi-line expanding input box with software blinking cursor
view-input word-wraps input at prompt-w, expanding the grey panel
upward as needed. Uses software cursor (█) in :input-fg blinking
at 2Hz via get-internal-real-time.
view-chat max-lines adapts to variable panel height via input-panel-top.
Removed terminal cursor (position-cursor, cursor-show, cursor-style).
Dialog minibuffer top now computed from input-panel-top.
2026-05-16 11:01:05 -04:00
bb98b486e4 v0.10.4: spacer rows between messages in history area
Clean implementation: spacer inserted in the rendering loop as an
(incf y) between message blocks, tracked in scroll-fitting loop
via spacer variable. No data structure changes.

Also: fixed premature let close in spacer binding, fixed view-input
closing paren count, and re-applied speaker alignment fixes lost in
revert.
2026-05-16 09:54:34 -04:00
bcab429dd7 Revert "v0.10.4: spacer rows between messages in history area"
This reverts commit 2513466576.
2026-05-16 09:31:52 -04:00
2513466576 v0.10.4: spacer rows between messages in history area
Each message block gets a trailing empty line (no speaker, no text)
to improve visual separation between turns.
2026-05-16 09:24:42 -04:00
f6dbd6dbd0 v0.10.3: input area speaker line — 4 rows get │ in :input-prompt color
Consistent with history area: user │ (amber), agent │ (tan), gate │ (grey),
and the 4-row input box │ (:input-prompt).
2026-05-16 09:14:51 -04:00
bad7686d4e v0.10.2: voice system — all speakers use │, neuro-thinking bg bar, blinking cursor
- Theme: added :agent-border, :thinking-bg, :symbolic-border to all 13
  presets with theme-load fallback for saved themes.
- Agent output now draws │ with :agent-border color (muted tan).
- Neuro-thinking (streaming): draw-rect at column 0 with :thinking-bg
  (dark grey block) instead of a grey │ character. No border text.
- Gate traces: │ with :symbolic-border (was ╎ with :dim).
- Tool calls: │ with tool status color (was ╎).
- Removed > prompt prefix from input line.
- Added position-cursor function: blinking block cursor at insertion
  point, called every frame from the main loop after sleep.
2026-05-16 09:10:39 -04:00
2189745f40 v0.10.1: architectural cleanup — full-frame redraw, explicit bg everywhere, :bg-input fallback
- redraw: always draws all three views (status/chat/input) when any
  dirty flag is set. Dirty flags only gate frame rendering, not
  which parts render. Fixes disappearing input/history.
- Added :bg-input to all 13 presets with #2e2e2e (dark) / #d4d4d4
  (light-amber). theme-load fills missing keys from current preset
  defaults for backward compatibility.
- Removed unused *sidebar-panels* defvar and obsolete contract docs.
- Renamed dim-bg → dim-fg (foreground color, not background).
- All draw-text calls in sidebar and dialog minibuffer now pass
  explicit bg-panel, preventing background leaks.
- render-styled (markdown renderer) passes explicit (theme-color :bg).
- Fix h shadowing in view-chat scroll loop (h → mh).
2026-05-16 09:03:59 -04:00
0a0478f502 v0.10.0: TUI visual overhaul — dark-neutral theme, left-border messages, sidebar auto-show, cl-tty style-reset
- Theme: near-black (#0a0a0a) backgrounds, dark-grey panels (#141414),
  warm amber (#fab283) accent only. New keys: :bg, :bg-panel, :bg-element,
  :text-muted. All 13 presets updated.
- Messages: No background fills (sit on global black). User messages get
  amber left border (│). Agent response has no border (invisible).
  Streaming agent messages get grey left border. Gate traces and tool
  calls use grey ╎ prefix. No label lines, no time separators.
- Sidebar: :sidebar-mode with :auto/:visible/:hidden. Auto-shows at >120
  cols (opencode-style). Width 42 with version + connection dot footer.
- Input: 2-char hpad on each side. Grey panel (2 rows: separator +
  prompt). Hint right-aligned at bottom on black.
- Status bar: empty (clean black line).
- cl-tty backend: draw-text, draw-rect, draw-link, draw-border now use
  \e[22;23;24;25;27m (style-only reset) instead of \e[0m (full reset),
  preserving foreground/background across draw calls.
- Fix: all sidebar text draws pass explicit bg-panel background.
- Fix: hint at h-1 passes explicit (theme-color :bg).
- Fix: sidebar bottom row uses draw-text (no \n) to prevent scroll at h-1.
2026-05-16 08:02:53 -04:00
3bc1977632 fix: Reader error loop from (= nil 27), Swank *standard-output* redirect
- Remove 'code' variable binding (redundant with b). esc-seq now
  starts with (and b (= b 27) ...) so when b is nil (timeout), the
  and short-circuits before (= b 27) can error with 'NIL is not
  of the type NUMBER'.
- Swank prints to *standard-output*, not *error-output*. Bind both
  to string output streams to prevent ';; Swank started' leak.
2026-05-15 16:12:43 -04:00
13b6edab32 fix: nil check in CSI detection (= b2 91), Swank *standard-output* redirect
- (= b2 91) errors when b2 is nil (read-raw-byte timeout). Add
  (and b2 (= b2 91)) to guard against nil.
- Swank writes ';; Swark started at port:...' to *standard-output*,
  not *error-output*. Bind *standard-output* to string stream too.
2026-05-15 16:10:11 -04:00
8d9520a9cb fix: replace cl-tty.input:read-event with direct read-raw-byte + inline CSI detection
cl-tty.input:read-event has bugs (CSI parser timeout causes :escape
to be returned for arrow keys). Replace with direct read-raw-byte
calls that are proven to work for CSI sequences. The inline detection:
- Read first byte with 100ms timeout
- If ESC (27), read two more bytes with 150ms timeout each
- Map 65→:up, 66→:down, 67→:right, 68→:left, etc.
- Other bytes converted via the same cond chain as before
Also re-add resize check (was handled by read-event).

Use handler-case around the reader to prevent any reader errors
from crashing the TUI. Re-add Swank *error-output* redirect.
2026-05-15 15:23:12 -04:00
bd72175d5b fix: use cl-tty.input:read-event for keyboard input (proper CSI handling)
Replace the inline raw byte reader + CSI detection with
cl-tty.input:read-event which uses read-raw-byte (direct fd reads)
and properly parses CSI escape sequences, UTF-8, mouse events, etc.
Also fix: remove extra ) in (t nil) clause that was prematurely
closing the let* binding, causing the if form to receive 4 args.
2026-05-15 13:09:17 -04:00
cc38e67d7c fix: Swank stderr leak, CSI detection with progn wrapper
- Swank: bind *error-output* to string stream to prevent 'Swank started
  at port: 4006.' from leaking to terminal on exit
- CSI detection: wrap inner dotimes in (progn ... t) so the and form
  doesn't short-circuit (dotimes returns nil, breaking the chain)
- Add debug add-msg for CSI detection results
2026-05-15 12:40:07 -04:00
df33e8d6db fix: main loop never closed, disconnect-daemon ran every iteration
The main loop's closing paren was missing — (sleep 0.1)) only closed
sleep and the minibuffer let, but NOT the loop itself. The next form
(progn (disconnect-daemon)) was INSIDE the loop body, called on every
iteration. On first call it added '* Disconnected *' and cleared the
daemon stream, making the TUI permanently disconnected.

Fix: add ) to close the loop. Also:
- Connect-daemon runs synchronously BEFORE with-terminal (3 ports, 6s
  max). If daemon is already running, the TUI starts connected.
- If sync connect fails, background thread retries every 5 seconds.
- start-daemon in background (no blocking wait for daemon startup),
  so TUI appears immediately.
2026-05-15 12:08:40 -04:00
bd1e8a92be fix: terminal stty restore with trap (set -e kills script before restore)
set -e on line 2 causes the bash script to exit immediately when sbcl
returns non-zero, before the stty icanon echo ixon restore runs.
Add trap cleanup EXIT to guarantee terminal restore on any exit path.
2026-05-15 11:43:29 -04:00
9fb4393c9c fix: runtime crash (sb-ext:timeout undefined), replace with listen-based polling
- Remove handler-case + sb-ext:with-timeout 0.1 pattern entirely.
  sb-ext:timeout is a condition class, not a recognized type in the
  compilation environment, causing runtime 'undefined function' crash.
- Replace with dotimes 10 * (listen *standard-input*) + sleep 0.01
  polling loop. Same 0.1s timeout, no condition type dependencies.
- Also fix handler-bind → handler-case in tui-load.lisp so the stack
  unwinds properly (running with-terminal's shutdown-backend cleanup)
  before the crash handler runs, restoring terminal to normal state.
- Fix paren imbalance (off by 1) in the new listen-based reader code.
2026-05-15 11:36:46 -04:00
c1f4ad40d2 fix: disconnect-daemon missing close paren, compilation now succeeds
Major bug: defun disconnect-daemon in channel-tui-main.org was missing
its closing paren. Every form after disconnect-daemon (tui-main, tests,
etc.) was inside the unclosed defun, causing 'end of file' compile errors.
Adding the missing ) fixed all compilation errors.

Also revert handler-case change: keep sb-ext:timeout condition type.
2026-05-15 11:27:57 -04:00
d14ff3a316 fix: daemon port conflict handling, multi-port TUI connect
- start-daemon: handle ADDRESS-IN-USE-ERROR by trying ports 9105-9115
  instead of crashing. Logs which port is used.
- Add *daemon-port* defvar to track actual listening port
- main: wrap start-daemon in handler-case so the daemon doesn't
  crash if all ports are in use
- connect-daemon (TUI): try ports 9105-9115 with 2s timeout each
  instead of retrying the same port 3 times
- Add debug messages for connection success and disconnection timestamp
2026-05-15 10:56:09 -04:00
5924994202 fix: CSI escape detection for arrow keys, fix paren balance
- Add CSI escape sequence detection: when ESC (27) is received, poll
  for up to 20ms for the next bytes to detect arrow/home/end keys
- Use listen+read-char polling (not nested with-timeout) to reliably
  collect multi-byte sequences while keeping standalone ESC responsive
- Fix paren balance in main code block (2 extra opens from nested
  esc-seq forms needed matching closes)
2026-05-15 09:43:03 -04:00
53aa471a51 fix: revert to blocking connect-daemon, daemon connection now reliable
- Revert async connect-daemon thread (bt:make-thread unreliable — errors
  in the thread cause silent failure, no connection, no error message)
- Restore blocking connect-daemon before with-terminal (original pattern
  that was working)
- Revert /reconnect to synchronous call
- Remove stale async thread code and error messages
2026-05-15 09:21:54 -04:00
c148570d4c fix: multi-value backend-size, minibuffer border+width, pre-existing warnings
- backend-size: nested multiple-value-bind/values instead of or+mv-bind
  (or discards secondary values), remove stale env-var pre-check
- Minibuffer: full chat-w width (respects sidebar), horizontal rule
  border, clear filter prompt line to avoid text overlap
- Filter prompt: (or filter "") prevents "NIL" display
- Dirty-flag redraw: skip when dialog-stack is non-nil (minibuffer
  covers the area, prevents flicker)
- Remove 3 unused variables: FOCUS, SENSOR, C (pre-existing warnings)
2026-05-15 08:51:19 -04:00
f56ff4849f fix!: eliminate cat subprocess, use direct stdin reads, fix parens
Cat subprocess (uiop:launch-program '("cat") :input :interactive) was
unreliable — the process would exit immediately in some environments,
breaking ALL keyboard input. Root cause: uiop's :input :interactive mode
opened /dev/tty which failed under specific process-group configurations.

Replace with direct read-char-no-hang on *standard-input*. The bash script
sets stty -icanon -echo -ixon before launching sbcl, so SBCL's stdin is
already in raw mode. No subprocess needed.

Also fixed pre-existing paren imbalance in tui-main (2 extra opens).
2026-05-15 08:33:24 -04:00
3661d00138 fix: cat buffering, dialog filter int-chars, remove double render
- Wrap cat with stdbuf -o0 so keystrokes aren't stuck in cat's 4096-byte
  pipe buffer — text input was invisible until buffer filled
- Dialog filter: (characterp ch) rejects integer char codes from raw event
  dispatch. Accept integerp in range 32-126 and convert via code-char
- Remove initial render (backend-clear + view calls) before main loop.
  Dirty flags already trigger a full sync-wrapped redraw in the first
  iteration, eliminating the pre-loop clear flash
2026-05-14 20:29:50 -04:00
25da9ae685 fix: TUI flicker, bottom-anchored minibuffer, 13 color presets
Issue 1 — flickering during typing/updating:
- Wrap every frame render in DECICM sync (begin-sync/end-sync) so the
  terminal defers rendering until the entire frame is written
- Replace backend-clear (ESC[2J full clear) with draw-rect background
  fill — eliminates visible blank frame between redraws
- These two changes together eliminate all visible tearing/flicker

Issue 2 — bottom-anchored minibuffer (Emacs-style):
- Replace centered overlay dialog with bottom-anchored minibuffer
  that expands upward from the input line
- Unified command menu: Ctrl+P and / both open the same menu with
  all 35+ commands (slash + daemon), dispatch by value type
- Filter prompt at h-3 (same position as normal input),
  options listed above, grows up to 15 lines
- No full-screen dim backdrop — just clear the minibuffer area

Issue 3 — color schemes:
- Add 5 new presets: catppuccin, tokyonight, dracula, gemini, mono
- Total: 13 presets (up from 8)
- Update /theme completion list and help text

Also fixed: pre-existing unbalanced paren in tui-main (missing close)
2026-05-14 19:36:29 -04:00
6d7dd9e1ea fix: clean TUI exit, restore terminal, suppress compiler warnings
passepartout script:
- Add (uiop:quit 0) after tui-main so SBCL exits on Ctrl+Q
- Remove exec to allow stty restore after sbcl subprocess
- Restore icanon echo ixon after TUI exits (terminal stuck raw)

channel-tui-view.org:
- Remove unused fb/h vars from test-sidebar-not-shown-narrow
- Add (declare (ignore w)) to render-styled
- Qualify theme-color as passepartout.channel-tui:theme-color
  (render-styled is in :passepartout package)
- Remove dead :url clause from pick in parse-markdown-spans
  (URLs handled by dedicated branch, not via pick)
- Update literate prose for all changes
2026-05-14 16:25:36 -04:00
e453f9aad9 fix: use global vars for cat subprocess to avoid let* scope crash
Replaced (let* ((cat-proc ...) (tty-in ...)) ...) with global
special variables *cat-proc* and *tty-in* with defvar declarations.
The let* caused 'unbound variable' errors on Ctrl+Q because the
lexical scope didn't extend to terminate-process. Global vars have
indefinite scope and work reliably regardless of paren nesting.
2026-05-14 15:57:37 -04:00
74621cffd2 fix: disable flow control (-ixon) for Ctrl+Q, constrain prompt/hint to chat-w
- Added -ixon to stty so Ctrl+Q (XON byte) isn't swallowed by the
  terminal driver and reaches the TUI as :CTRL-Q
- view-input now truncates the prompt (> prefix + visible text) to
  chat-w - 2 characters, and the hint to chat-w characters, so
  neither extends into the sidebar area
2026-05-14 15:48:34 -04:00
2ce8d9d886 fix: constrain separator/input/status to chat area when sidebar visible
Added chat-w = w - sidebar_width calculation in view-status and
view-input, matching view-chat's existing approach. Also added
chat-w for the separator line drawing in tui-main. This prevents
the prompt, separator, hint, and status bar from extending into
the sidebar area when it's visible.
2026-05-14 15:44:17 -04:00
345f3f397d fix: set stty -icanon -echo in bash script before exec sbcl
uiop:run-program inside SBCL can't access the terminal, so stty
calls from within Lisp fail silently. By running 'stty -icanon -echo'
in the bash script before exec sbcl, the terminal is already in
character-at-a-time mode when the TUI starts, and Ctrl+P/B keys
arrive as individual bytes through the cat pipe.
2026-05-14 15:39:05 -04:00
84ef4c3443 fix: lower sidebar threshold to 60 cols, word-wrap agent messages
- Sidebar threshold lowered from 120 to 60 so it works on 83-col
  terminals
- Agent response text is now word-wrapped through cl-tty.box:word-wrap
  after markdown rendering, preventing text from bleeding past the
  terminal edge
2026-05-14 15:31:00 -04:00
ad5b9669a6 fix: revert w→chat-w replacement outside view-chat
view-status and view-sidebar don't have chat-w bound. Reverted
lines 78 and 236 to use w instead of chat-w.
2026-05-14 15:26:00 -04:00
187ec6e471 fix: constrain chat width when sidebar is visible, Ctrl+B sets all dirty
- Added chat-w = w - sidebar-width in view-chat for all width
  calculations (word-wrap, padding, borders) so text doesn't
  bleed into the sidebar area
- Changed Ctrl+B dirty flags from (list t t nil) to (list t t t)
  so input view also redraws, fixing the toggle-not-turning-off issue
2026-05-14 15:24:43 -04:00
48c2d57c14 fix: restore :key event handler with Ctrl+P/B/Q/L and dialog routing
The key event dispatch was lost during git restores, causing all
:key events (Ctrl+P, Ctrl+B, Enter, etc.) to fall through to
on-key which only handles :enter, :escape, and character insertion.
Added back the case ch with :CTRL-Q/:CTRL-P/:CTRL-B/:CTRL-L
branches and full dialog key routing.
2026-05-14 15:19:54 -04:00
b2f5f1cf1a fix: add stty -icanon for character-at-a-time input
Without -icanon, the terminal driver buffers all input until Enter,
so Ctrl+P/B never arrive as individual key events. With -icanon,
cat reads bytes immediately and pipes them to SBCL. SBCL reads from
the pipe, not fd 0, so there's no fd 0 read block issue.
2026-05-14 15:12:21 -04:00
369a7c93a9 fix: export MY_TERM_COLS/ROWS before exec sbcl 2026-05-14 15:03:00 -04:00
d1359eba1d fix: export MY_TERM_COLS/LINES before exec sbcl
SBCL strips COLUMNS and LINES from the process environment.
Use non-standard names that SBCL doesn't filter. The values
are captured from stty size before SBCL starts.
2026-05-14 14:55:37 -04:00
4006a62e53 fix: remove stty -echo line, sync tangle 2026-05-14 14:42:22 -04:00
a609232589 fix: restructure let* closing so cat-proc stays in scope
sleep had 3 closers (sleep, loop, let*), closing the let* before
terminate-process. Reduced to 2 closers, added let* close after
terminate-process.
2026-05-14 14:35:23 -04:00
e0003a5f3c fix: move nil guard before backend info message
The backend info message showed NIL for height because the nil
guard ran after it. Swap order so the message shows guarded values.
2026-05-14 14:20:28 -04:00
14cdb6c7b4 fix: restore backend info message, remove Connected chat message
Backend dimensions needed to be readable. Connected v0.7.2
was accidentally restored by git checkout — removed again.
2026-05-14 14:18:12 -04:00
d71ccb95c6 fix: guards on resize handler and render loop, O_RDONLY for /dev/tty 2026-05-14 14:10:57 -04:00
55166fc9ff fix: add nil guards on w and h in tui-main before initial render
backend-size can return nil for height (especially when the /dev/tty
ioctl fallback hasn't been compiled in yet). view functions had nil
guards but the direct (- h 4) calls in tui-main's initial render
crashed before reaching them.
2026-05-14 14:04:27 -04:00