Skip to content

fix(config): the TUI draws every built-in pet and plays its idle animation - #35

Merged
NovusEdge merged 1 commit into
mainfrom
fix/config-tui-pets
Oct 8, 2026
Merged

NovusEdge merged 1 commit into
mainfrom
fix/config-tui-pets

Conversation

@NovusEdge

@NovusEdge NovusEdge commented Oct 8, 2026 •

Copy link
Copy Markdown
Owner

The /glowup config window carried one still frame of Clawd and skipped the pet column for any other pet, so the robot and the egg were blank.

  • hooks/packexport.ts exports each built-in pet's idle animation (Clawd, shiny Clawd, robot, egg) into packs.json, replacing the single Clawd frame. The installer binary grows by about 350 KB.
  • The config TUI plays the animation at its 100 ms tick, widens the pet column to the pet (the robot is 32 cells), and holds the first frame under reduced motion.
  • The egg always shows its uncracked stage there: the crack stage lives in the session's store, which the installer cannot read.

Testing: just ci passes. New tests cover the export, frame timing, the column width per pet, and the animation with and without reduced motion. The built TUI ran in tmux at 130×40: the robot is drawn and moves between captures, and the egg is drawn.

Summary by CodeRabbit

  • Bug Fixes
    • /glowup config now previews Clawd, shiny Clawd, the robot, and the egg with their idle animations.
    • Pets remain still when reduced motion is enabled. The egg appears in its uncracked stage.

…ation

The config window carried one still frame of Clawd and skipped the pet
column for any other pet, so the robot and the egg were blank. packs.json
now holds each built-in pet's idle frames; the column follows the pet's
width and holds the first frame under reduced motion.

Signed-off-by: NovusEdge <novusedge0@gmail.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 451e11cb-12b1-4041-8126-431a1d72684e
📥 Commits

Reviewing files that changed from the base of the PR and between 3f96701 and b2c6c6d.

📒 Files selected for processing (11)
  • .docket/ledger.jsonl
  • CHANGELOG.md
  • hooks/packexport.ts
  • installer/internal/configtui/view.go
  • installer/internal/configtui/view_test.go
  • installer/internal/packs/packs.go
  • installer/internal/packs/packs.json
  • installer/internal/packs/packs_test.go
  • installer/internal/tui/clawd.go
  • installer/internal/tui/tui_test.go
  • test/packexport.test.ts
 ________________________________________
< CI is red. I'm also red. We match now. >
 ----------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@NovusEdge
NovusEdge merged commit a30a06a into main Oct 8, 2026
6 of 7 checks passed
@NovusEdge
NovusEdge deleted the fix/config-tui-pets branch October 8, 2026 22:27
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