Skip to content

chore(claude): add record demo skill#403

Open
zimeg wants to merge 2 commits intomainfrom
zimeg-chore-maintainers-vhs
Open

chore(claude): add record demo skill#403
zimeg wants to merge 2 commits intomainfrom
zimeg-chore-maintainers-vhs

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Mar 16, 2026

Changelog

N/A - But I am curious if these scripts are useful to docs pages in upcoming builds? 👾 ✨

Summary

This PR adds a claude skill to record demos for PR previews using the vhs package.

Preview

📸 Example of generated "tape" and GIF - the app existed ahead of time which might be useful for intricate setups:

# Demo: slack run + slack app list
Output demos/run-and-app-list.gif

Set Shell "zsh"
Set FontSize 16
Set Width 1200
Set Height 600
Set Theme "Catppuccin Mocha"
Set TypingSpeed 75ms
Set Padding 20

Hide
Type "cd /Users/eden.zimbelman/programming/tools/slack-cli/relaxed-porcupine-80"
Enter
Sleep 500ms
Show

Type "../bin/slack run"
Sleep 500ms
Enter
Sleep 3s
Enter
Sleep 1s
Enter
Sleep 10s
Ctrl+C
Sleep 1s

Type "../bin/slack app list"
Sleep 500ms
Enter
Sleep 3s

run-and-app-list

Requirements

@zimeg zimeg added this to the Next Release milestone Mar 16, 2026
@zimeg zimeg self-assigned this Mar 16, 2026
@zimeg zimeg requested a review from a team as a code owner March 16, 2026 07:04
@zimeg zimeg added code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment labels Mar 16, 2026
@codecov
Copy link

codecov bot commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.90%. Comparing base (35e1743) to head (911840b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #403   +/-   ##
=======================================
  Coverage   67.90%   67.90%           
=======================================
  Files         218      218           
  Lines       18050    18050           
=======================================
+ Hits        12256    12257    +1     
+ Misses       4640     4638    -2     
- Partials     1154     1155    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ Absolutely love this feature! Great idea @zimeg

💬 Asked a question about whether we may want to check these into version control. It can be a future topic though, but would love to hear the discussion.

Comment on lines +40 to +42

# VHS demo recordings
demos/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question(non-blocking): Curious if you considered having our VHS demos checked into version control? Just curious if we'd ever want to re-generate older recordings to show an updated UI.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📸 Such an interesting idea! Without knowing good practices to this I left it for later thinking to be honest-

I'd be so curious to versioning .tape files with the docs 📚

Our hidden docgen command might be extended to require vhs to regenerate all recordings with the latest release perhaps? Would love to learn recommendations to this and thoughts from @lukegalbraithrussell!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants