repos / zmx

fix: timing issue with first attach and sending terminal state

On first attach we don't want to send the terminal state but there
appears to be a timing issue where sometimes we already have pty output
before the Init handler can complete.

docs: update changelog

fix: correct cursor position when reattaching to session

Serialize terminal state BEFORE resizing the terminal in handleInit.
Previously, the resize was done first, which caused cursor reflow and
moved the cursor position. The shell's SIGWINCH-triggered prompt redraw
would then run after our snapshot was sent, leaving the cursor at the
wrong position (end of line instead of after the prompt).

By capturing the terminal state before resize, we preserve the correct
cursor position from when the user detached.

docs: update detach key

refactor: input shortcut handling

Another code clarity change that abstracts handling the shortcuts for
detach

refactor: daemonLoop and terminal serialization

Code cleanup to make the daemonLoop and ghostty interaction easier to
understand

feat: ctrl+b + d to detach

With support to add more commands

fix: reset terminal modes on detach

Closes: https://github.com/neurosnap/zmx/issues/26

chore: update ghostty-vt

Use XDG_RUNTIME_DIR for the socket directory if defined

XDG_RUNTIME_DIR[0] is a file path for user-specific runtime files, like
sockets.

It will typically be defined on Linux. I can't say for sure on BSDs, and
I know for certain it is not defined on macOS.

Link: https://specifications.freedesktop.org/basedir/latest/ [0]
Signed-off-by: Tristan Partin <tristan@partin.io>

revert: don't force pty resize

This seems to be causing issues with multiple clients connected, instead
we will require clients to force resize themselves for now

chore: publish v0.1.1

docs(changelog): v0.1.1

feat(list): sort by session name

fix: send SIGWINCH to PTY on re-attach

TIOCSWINSZ alone doesn't notify the shell when terminal size is set.
Now we explicitly signal the foreground process group after setting
the window size, fixing display issues on re-attach.

chore: remove dummy integration tests

fix(getTerminalSize): return default size if cols and rows are 0

The issue is that `getTerminalSize` can return 0x0 even when ioctl(TIOCGWINSZ) succeeds. The ghostty terminal library panics when initialized with zero dimensions.

The fix in `getTerminalSize` now validates that both rows and cols are positive before using them, otherwise falling back to the default 24x80.

Closes: https://github.com/neurosnap/zmx/issues/25

add zsh shell prompt doc (#24)

fix: flake.nix only supports linux atm

`ghostty` does not support packaging nix on mac which I think means we will not be able to support it:

- https://github.com/ghostty-org/ghostty/blob/cf06417b7dfbd0daeb58a9143f9b6ee194cbce26/nix/package.nix#L49
- https://github.com/ghostty-org/ghostty#2824

When our flake tries to build on macOS, it pulls ghostty as a Zig dependency, and ghostty's build.zig calls into apple-sdk/build.zig which needs the Darwin SDK — but ghostty's Nix infrastructure hasn't set that up.

feat: homebrew install

docs: changelog spec ref

docs: update binaries

chore: v0.1.0

docs: readme

docs: readme

docs: added tmux to comparison table

fix: remove mac flag

feat: add version command

chore: build.zig version build option

refactor!: postfix uid for socket dir

- first we look for TMPDIR env var or else we default to `/tmp`
- then we look for ZMX_DIR env var or else we create a `/tmp/zmx-{uid}` folder

Since this is a breaking change, you can kill all of your previous
sessions by using `ZMX_DIR=/tmp/zmx zmx kill {sesh}`

Closes: https://github.com/neurosnap/zmx/issues/15

BREAKING CHANGE: this moves sockets to a new location which means none
of your previous sessions will be available

docs: comparison table

docs: index site content

feat: docs site

fix(attach): cannot attach to session within session

Closes: https://github.com/neurosnap/zmx/issues/20

docs(readme): fish prompt file loc

chore(release): add shasum files as well

fix(release): strip mac metadata

fix(release): strip apple xattrs

docs: links for binaries

add nix flake (#5)

fix(macos): release build

chore: upload script

feat: add release build

fix: support kitty keyboard ctrl+\ with event flags

References: https://github.com/neurosnap/zmx/issues/10

chore: add kitty graphics protocol example script

refactor: when daemon is unresponsive, remove it

This commit does a better job removing a unix socket file when
the daemon is unresponsive, likely indicating that the process has
already been killed.

We accomplish this by creating a probSession fn that will connect to the
unix socket, send the Info event, and timeout after 1 second.  After
that point we remove the unix socket file.

fix: send SIGTERM to pty child process before shutting down

The issue was that when `zmx kill` was called, the daemon would call `shutdown()` and exit the loop, but then get stuck at `waitpid(daemon.pid, 0)` waiting for the PTY child shell to exit. Since the shell was never killed, the daemon would hang forever, preventing the defer block from running that deletes the socket file.

The fix sends SIGTERM to the PTY child process before shutting down, allowing the shell to exit gracefully so waitpid returns and the socket cleanup defer runs.

References: https://github.com/neurosnap/zmx/issues/10

fix: print help text with unknown commands

We should print the help text when possible and refrain from using the
logger when the user provides invalid input since that gets sent to our
log files.

Closes: https://github.com/neurosnap/zmx/issues/11

fix: ghostty should not send color pallet on re-attach

When `zmx` starts a session, `ghostty-vt.Terminal.init()` uses Ghostty's default palette, not foot's palette. On reattach, it dumps all 256 colors from ghostty's defaults, which may differ from foot's defaults.

References: https://github.com/neurosnap/zmx/issues/7

feat: increase max scrollback

Closes: https://github.com/neurosnap/zmx/issues/9

Bump minimum Zig version (#6)

fix: dont send ghostty snapshot on first attach

chore: clear main screen and set cursor to home on first attach

fix: dont use alt screen on attach

The problem is that the client switches to the alternate screen buffer on attach, which doesn't have scrollback. When ghostty replays content, it goes into the alt screen where scrollback doesn't accumulate.

The alternate screen buffer (\x1b[?1049h) never has scrollback - that's by design in terminals.

To get scrollback working properly we now stay on main screen and clear on detach.

docs: readme

docs: readme

docs: readme

docs: readme

docs: readme

docs: readme

docs: readme

chore: license

docs: readme

chore: logo

chore: logo

docs: logo

chore: update logo

docs: logo

docs: readme

docs: readme

feat: shorthand aliases for commands

docs: readme

docs: readme

feat: render terminal snapshot to client on re-attach

This change uses libghostty-vt to restore the previous state of the
terminal when a client re-attaches to a session.

How it works:

- user creates session `zmx attach term`
- user interacts with terminal stdin
- stdin gets sent to pty via daemon
- daemon sends pty output to client *and* ghostty-vt
- ghostty-vt holds terminal state and scrollback
- user disconnects
- user re-attaches to session
- ghostty-vt sends terminal snapshot to client stdout

In this way, ghostty-vt doesn't sit in the middle of an active terminal
session, it simply receives all the same data the client receives so it
can re-hydrate clients that connect to the session.

chore: cleanup build files

docs: clarify detach cmd

chore: move code around

docs: readme

docs: readme

fix: discard any unread input when restoring terminal settings on detach

feat: handle resize events

fix: free socket_path and the clients array backing storage before the daemon process exits

fix: close the server socket and delete the socket file if spawnPty fails

fix: close the PTY master fd when the daemon exits

fix: client socket fd ensure the socket is closed when the function exits

fix: ensure the fd is closed if initUnix, bind, or listen fails

fix: reap the pty child process and prevent zombies

fix: use relative path since we are already inside the socket_dir

fix: properly allocate NUL-terminated copies of each argument

the child process will exec (replacing memory) or exit, so no cleanup needed.

feat: provide attach with a command

docs: readme

docs(readme): shell prompt section

refactor: cleanup logs

feat: on re-attach send SIGWINCH and in-band resize event

feat: send logs to a file with rotation

fix: cpu spike from unhandled events in our daemon and client loops

docs: readme

feat: ctrl+\ detach client

docs: readme

feat: list command

chore(docs): readme

daemon-per-session (#4)

This is a ground-up rewrite of this tool.  We have dramatically simplified how this tool works.  Instead of having a single daemon that manages all sessions, we have a daemon-per-session.

We employ a simple architecture: `fork() -> setsid() on child -> forkpty() -> attach on parent`

This has a bunch of benefits, one of which is we can safely remove libxev and use simple `poll()` loops for daemon and clients.

feat: attach will start in the cwd of client

shore: fix cursor error

fix: filter our client input queries from echoing to stdout

chore: centralize winsize change logic

feat: in-band resize window events

https://gist.github.com/rockorager/e695fb2924d36b2bcf1fff4a3704bd83

fix: flush buffer so prompt always shows after cmd

fix: properly clear terminal after detach

fix: reattach rendering

fix: restore errors

docs: render

chore: tests

feat: rendering colors on reattach

chore: cursor position and escape sequences

chore: Wide/Double-Width Character Handling

refactor: move render terminal snapshot to separate module

chore(reattach): row iteration

chore: checkpoint

fix: need clrf

perf: increase buffer sizes

chore: fmt

feat: restore terminal state

feat: restore modes

feat: restore terminal modes

chore: reset

refactor: renderTerminalSnapshot

refactor: renderTerminalSnapshot

docs: fmt

refactor: renderTerminalSnapshot

refactor: increase pty buffer sizes

This improves performance when writing to ghostty-vt

chore: remove pty json handling

chore: suppress ghostty-vt warnings

refactor: remove sanitized buffer for now

refactor(daemon): we dont need to buffer pty read for partial utf8

ghostty already handles this for us.

fix: replace to device attribute queries within daemon

chore: rm scrollback buffer

ghostty handles this for us

refactor: revert env var

chore: forward more env vars from daemon to children ptys

fix: forward TERM to pty

fix: change detach key to ctrl+b d like tmux

fix: move DA query handling to proper location

fix(attach): when daemon shuts down don't print EOF msg in attach

fix(attach): carriage return required in raw mode

fix: respond to Device Attribute queries to prevent fish shell warning

Fish shell sends a Primary Device Attribute (DA1) query (ESC [ c) when
it starts to detect terminal capabilities. Previously, zmx forwarded
this query to the PTY but never sent a response back to the client,
causing fish to wait 2 seconds and display a terminal compatibility
warning.

This commit intercepts DA queries in handlePtyInput() before they reach
the PTY and responds directly to the client with VT220 capabilities:
- Primary DA (ESC [ c): VT220 with color (22) and clipboard (52)
- Secondary DA (ESC [ > c): Terminal version info

The response format matches ghostty's deviceAttributes implementation.

Fixes the "fish could not read response to Primary Device Attribute
query" warning on first attach.

fix(attach): use-after-free bug

docs(protocol): pty binary format

fix(attach): pty_out needs to use binary protocol

refactor(attach): use xev.Stream for stdout writes

refactor: use binary format for pty_in and pty_out

fix: vt.resize on attach

fix: filter out characters ghostty cannot handle

refactor: resize ghostty term on attach

support macOS and FreeBSD PTY functions (#1)

Replace Linux-specific PTY headers with platform-specific imports
to enable cross-platform compatibility. On macOS, use util.h for
openpty(). On FreeBSD, use libutil.h for openpty(). On Linux and
other platforms, continue using pty.h.

Remove unused utmp.h include that was never referenced in the code.

This change allows the daemon to compile and run on macOS and
FreeBSD in addition to Linux without requiring platform-specific
preprocessor directives or build configurations.

Amp-Thread-ID: https://ampcode.com/threads/T-15fd58fa-5673-40c5-87e1-3ea0057599ca

Co-authored-by: Amp <amp@ampcode.com>

refactor: move detach key to Config struct

fix: segfault on daemon after existing pty

fix: memory leak

refactor: use new protocol module

refactor: move unix ipc protocol to separate module

refactor: test

fix: use xev for pty writes

chore: pass config struct to all subcommands

feat: config toml

feat: systemd unit file

docs: agents+readme

fix: detach shortcut

fix: alt-screen

fix: more cleanup

feat: version

fix: allow zig optimization

fix: help text

fix: initial attach clear terminal

feat: terminal restore works!

fix: restore session

fix: ghostty impl

docs: readme

docs: readme

docs: readme

feat: init