Skip to content

feat: let the station's dialog ask about the gallery in birdsbirdbirds and vibe-coding-mad-libs - #42

Merged
theycallmeswift merged 1 commit into
mainfrom
feat/gallery-consent-dialog
Nov 3, 2026
Merged

theycallmeswift merged 1 commit into
mainfrom
feat/gallery-consent-dialog

Conversation

@theycallmeswift

@theycallmeswift theycallmeswift commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

boothstrap 0.4.4 asks the participant in its own dialog before a gallery item is shared, so both gallery packages stop asking in chat. Pi now says what's about to be shared, calls publish_gallery_item, and words its reply by outcome: on shared it says the item is on its way, and on declined it says that's fine and carries on to the finish.

Deploy order

Point the event config at this commit only after every station runs boothstrap ≥ 0.4.4. An older station has no dialog, so on it this change publishes without asking the participant.

Decide

Nothing.

Ready gate

Step Evidence
Tests and lint npm run verify: ok for all 4 packages. npm test: 311 pass, 0 fail. npm run lint: 0 issues in 18 files.
Red run n/a: skill and doc wording only, no code path changed.
Sibling sweep Searched challenges/, docs/, test/ and README.md for gallery, booth screens, asks if and asks whether. Hits in both SKILL.md files, the birdsbirdbirds intro and the vibe-coding-mad-libs README are updated. No test asserts on the chat question.
Docs and specs challenges/birdsbirdbirds/intro.md and challenges/vibe-coding-mad-libs/README.md now say the station asks. Matches boothstrap.sh's MVP challenge spec Gallery section and the station base skill.
Standalone run No Pi playthrough. Each skill's standalone branch is unchanged: without publish_gallery_item it still skips the gallery, so standalone runs never reach the changed text.
Editorial pass Reread against docs/style/markdown.md. Removed the chat question and the yes/no branches.
Skipped tests None.
Secrets, permissions Not touched.

Generated by Claude Code


Autofix with [code]smith
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.

@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e76a7bdb-c8c4-4cb7-a879-2edfd2af6a62
  • Autofix · 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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@theycallmeswift
theycallmeswift marked this pull request as ready for review November 3, 2026 06:38
@theycallmeswift
theycallmeswift merged commit fb39e10 into main Nov 3, 2026
2 checks passed
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