Repository navigation
docs: re-record README GIFs and screenshots, fix the demo recorder - #172
Merged
Merged
Conversation
added 2 commits
October 27, 2026 11:06
The embedded ssh logs through `-E <fifo>`, and only an "Authenticated to" line there marks a session connected; until then it takes no keystrokes. The fake ssh still printed its handshake to stderr, so every recorded session stayed "connecting" and dropped what the tape typed. It now writes to the -E log, and drops the `--` broadcast puts before the command, which bash's eval rejected. record.py is now executable, so `just record-gifs` runs it, and the navigate tape filters on #db AND #prod, which match two hosts, instead of #ci AND #db, which match none.
Recorded with vt100 0.18 and the fixed demo: sessions connect, typed commands and snippets run, and broadcast exits 0 on every host.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Re-records every README GIF and screenshot after the move to vt100 0.18 (#170), and fixes the demo tooling that the first take showed was broken.
What changed
-E <fifo>, and only anAuthenticated toline in that log marks a session connected; until then SSHub drops all keystrokes. The fakedemo/bin/sshstill printed its handshake to stderr, so every recorded session stayed on "● connecting" and ignored what the tape typed. The committedsnippets.gifshows this. The handshake now goes to the-Elog.ssh host -- <command>. The fake ssh passed the--on to bash'seval, which rejected it, so every host failed with exit 2. It now drops the--, as real ssh does.demo/record.pyis executable. It was never committed with the executable bit, sojust record-gifsfailed with "Permission denied".#db AND #prod. These match the two Cloudflare hosts; the old#ci AND #dbmatched none, so that GIF ended on an empty list.How tested
just record-gifs: every take ran within 1–2% of its scripted timing.uptime,cowsay,uname -a) is present in the recorded output.exit 0on all 5#prodhosts.Written by claude-opus-5-5 (omp) on behalf of the maintainer.