Skip to content

docs: make the demo tape portable - #16

Merged
Steel-tech merged 1 commit into
mainfrom
docs/tape-portable
Aug 10, 2026
Merged

docs: make the demo tape portable#16
Steel-tech merged 1 commit into
mainfrom
docs/tape-portable

Conversation

@Steel-tech

@Steel-tech Steel-tech commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

The recording tape hardcoded /Users/vics/..., which is unreproducible on anyone else's machine and a personal detail to publish. Caught while auditing the repo ahead of making it public.

Now uses $HOME for the cargo bin path, and BULLPEN_HOME to isolate the demo store rather than overriding HOME — which is what BULLPEN_HOME is for, and didn't exist when this tape was written.

No effect on the committed GIF; this only matters when re-recording with vhs docs/media/bullpen-agents.tape.


Compound Engineering
Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the demo command to use a portable Cargo binary path.
    • Configured the demo to run with an isolated temporary home directory.

The recording tape hardcoded an absolute home path, which is both
unreproducible on anyone else's machine and a personal detail to publish.

Uses $HOME for the cargo bin path, and BULLPEN_HOME rather than overriding
HOME to isolate the demo store — which is what BULLPEN_HOME exists for and
did not when this tape was written.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PfAfAujueuZ3rDTiL9apx3
@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c3016c3a-1698-4699-a013-27f2c90b94b3

📥 Commits

Reviewing files that changed from the base of the PR and between 2d445fb and e0de22d.

📒 Files selected for processing (1)
  • docs/media/bullpen-agents.tape

📝 Walkthrough

Walkthrough

The demo shell command now resolves Cargo from $HOME and stores Bullpen state under /tmp/bpdemo/home.

Changes

Bullpen demo setup

Layer / File(s) Summary
Portable demo environment
docs/media/bullpen-agents.tape
The terminal setup command replaces the hardcoded Cargo path and HOME assignment with $HOME/.cargo/bin and BULLPEN_HOME=/tmp/bpdemo/home.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: making the demo tape portable.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/tape-portable

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

@Steel-tech

Copy link
Copy Markdown
Contributor Author

🤖 Lab Code Review (draft opinion)

No diff provided to review.

@Steel-tech
Steel-tech merged commit 580ac50 into main Aug 10, 2026
5 checks passed
@Steel-tech
Steel-tech deleted the docs/tape-portable branch August 10, 2026 03:35
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