Skip to content

Update demos: forge improvements, nix-shell, re-record all#119

Merged
actionshrimp merged 5 commits intomainfrom
dave/demo-update
Feb 25, 2026
Merged

Update demos: forge improvements, nix-shell, re-record all#119
actionshrimp merged 5 commits intomainfrom
dave/demo-update

Conversation

@actionshrimp
Copy link
Owner

Summary

  • Fix vim.NIL crash in parse_checks_summary when PR has no CI checks
  • Improve forge demo repo setup: reuse existing repo/PR instead of delete+recreate, use SSH remote
  • Add review comment creation to forge demo sequence
  • Fix diff viewer demo: move focus from panel to diff buffer before thread navigation
  • Use nix-shell for demo recording dependencies (asciinema_3, asciinema-agg, tmux)
  • Re-record all 3 demos (basics, advanced, forge)

Test plan

  • make demo-forge runs successfully with existing repo
  • make demo records all 3 demos without errors
  • Forge demo shows PR list, detail, diff with threads, and comment creation

Split the single demo recording into three separate demos:
- basics: status view, staging, commit, branch, log, command history
- advanced: native diff viewer, 3-way staging, merge conflict resolution
- forge: GitHub PR list, detail, diff with review threads

Add per-demo Makefile targets (demo-basics, demo-advanced, demo-forge).
Update docs/index.html with three asciinema players and record-demo.sh
to support recording individual or all demos.
- Fix vim.NIL crash in parse_checks_summary when PR has no CI checks
- Reuse existing GitHub demo repo/PR instead of recreating each run
- Use git@github.com SSH remote for pre-authed pushes
- Add review comment creation to forge demo sequence
- Fix diff viewer focus: move from panel to diff buffer before thread nav
Automatically enters nix-shell with asciinema_3, asciinema-agg, and tmux
instead of requiring manual installation.
Show basics, advanced diff, and GitHub integration demos separately
in the README with brief descriptions. Generate preview GIFs for all
3 demos instead of just basics.
Updated basics, advanced, and forge demo recordings.
@actionshrimp actionshrimp merged commit 3fc5294 into main Feb 25, 2026
3 checks passed
@actionshrimp actionshrimp deleted the dave/demo-update branch February 25, 2026 11:56
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.

1 participant