Skip to content

docs(readme): refresh loadout demo with current CLI syntax - #1095

Merged
twitchyliquid64 merged 1 commit into
mainfrom
docs/refresh-loadout-demo-cli-syntax
Jul 30, 2026
Merged

docs(readme): refresh loadout demo with current CLI syntax#1095
twitchyliquid64 merged 1 commit into
mainfrom
docs/refresh-loadout-demo-cli-syntax

Conversation

@norrietaylor

@norrietaylor norrietaylor commented Jul 30, 2026

Copy link
Copy Markdown
Member

The README's asciinema demo (docs/public/loadout-demo.castloadout-demo.gif) showed two command spellings that no longer exist. Both verified against main:

Was Now Why
min activate --loadout dev --attach . min session activate --loadout dev --attach . No top-level activate; it lives under the session noun (crates/minimal/src/lib.rs Command enum, docs/reference/cli-min.md)
min run build / test / shell (MOTD table) mip run build / test / shell min has no run subcommand — mip run <task> is the task runner (crates/mip/src/main.rs, docs/reference/tasks.md). min run <task> exists only as a wire form the daemon's exec channel accepts, not as something you type in a session shell

Cast mechanics

The inserted session keystrokes continue the recording's existing 6-step typing-jitter cycle (0.057, 0.037, 0.065, 0.039, 0.060, 0.045), with every later event shifted by +0.408s, so the file stays cycle-consistent. Runtime 15.014s → 15.422s.

MOTD column padding is byte-preserved (min run build and mip run build are the same width), so the task table stays aligned.

GIF

Re-rendered with agg --line-height 1.3 — the parameters that reproduce the committed asset's exact 886x541 geometry. Rendered frame checked visually: layout, palette, and logo unchanged.

Out of scope, flagging

docs/reference/cli-min.md documents min add <--runtime|--build|--task <TASK>>, omitting --session, which the binary does accept. The README's min add --session git gh claude-code is correct; the reference doc is the stale one. Not touched here.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the terminal loadout demo recording with revised command text and task examples.
    • Refined recording timing and positioning while preserving the existing later actions and displayed tool paths.

Note

Refresh loadout demo recording with current CLI syntax

Updates loadout-demo.cast and loadout-demo.gif to reflect current CLI syntax. The recorded command now uses the session activate prefix, and task label prefixes in the Tasks listing changed from min to mip for build, test, and shell entries.

Macroscope summarized 0b730ab.

The README's asciinema demo showed two command spellings that no longer
exist:

- `min activate` — activate lives under the `session` noun; there is no
  top-level `activate` in the `min` CLI.
- `min run <task>` in the MOTD task table — `min` has no `run`
  subcommand. `mip run <task>` is the task runner. (`min run <task>` is
  only a wire form the daemon's exec channel accepts, not something you
  type in a session shell.)

The inserted ` session` keystrokes continue the recording's existing
6-step typing-jitter cycle, with every later event shifted by +0.408s, so
the cast stays cycle-consistent (runtime 15.014s -> 15.422s). MOTD column
padding is byte-preserved, keeping the task table aligned.

GIF re-rendered with `agg --line-height 1.3`, which reproduces the
committed asset's 886x541 geometry.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The terminal loadout demo recording updates its initial command sequence, changes task entries from min run to mip run, and revises later timing and positioning while preserving the displayed interactive actions.

Changes

Loadout demo recording

Layer / File(s) Summary
Update demo commands and playback
docs/public/loadout-demo.cast
The cast updates the initial prompt sequence, changes build/test/shell entries to mip run, and adjusts later timing and positioning values while retaining the existing interactive content.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: msample

Poem

I hop through the cast with a bright little cheer,
mip run commands now sparkle clear.
The timings are tuned, the prompts neatly align,
While vim and tool paths continue to shine.
A bunny-approved demo, recorded just right!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is concise, conventional, and accurately summarizes the main change to the loadout demo syntax.
Description check ✅ Passed The description covers the change and rationale well, but it omits the template's explicit Testing and Checklist sections.

Comment @coderabbitai help to get the list of available commands.

@twitchyliquid64
twitchyliquid64 merged commit 7665248 into main Jul 30, 2026
29 checks passed
@twitchyliquid64
twitchyliquid64 deleted the docs/refresh-loadout-demo-cli-syntax branch July 30, 2026 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants