Skip to content

Fix bumpy add: distinguish skip vs none, respect existing bump files#69

Merged
theoephraim merged 3 commits into
mainfrom
fix/add-command-skip-none
May 4, 2026
Merged

Fix bumpy add: distinguish skip vs none, respect existing bump files#69
theoephraim merged 3 commits into
mainfrom
fix/add-command-skip-none

Conversation

@theoephraim
Copy link
Copy Markdown
Member

@theoephraim theoephraim commented May 4, 2026

Summary

  • Adds a skip level to the bump select prompt to distinguish between "not included in bump file" and explicitly marking a package as none
  • Loads existing bump files on the branch so already-covered packages default to skip instead of patch when re-running bumpy add
  • Fixes keypress listener cleanup and emitKeypressEvents ordering to prevent rendering bugs between prompt steps
  • Removes the interactive cascade prompt entirely — cascades are an advanced feature better authored directly in bump files

Test plan

  • All 212 existing tests pass
  • TypeScript compiles with no errors
  • Manual test: run bumpy add interactively, verify skip/none/patch/minor/major levels work correctly
  • Manual test: create a bump file, run bumpy add again, verify already-covered packages default to skip
  • Manual test: verify prompt transitions between steps render cleanly

… existing bump files

- Add 'skip' level to distinguish "not included" from explicit "none" in bump files
- Load existing branch bump files so already-covered packages default to skip
- Fix keypress listener cleanup to prevent ghost handlers on subsequent prompts
- Fix emitKeypressEvents ordering
- Only offer cascade options for actual version bumps, not 'none'
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

bumpy-frog

The changes in this PR will be included in the next version bump.

patch Patch releases

  • @varlock/bumpy 1.7.0 → 1.7.1

Bump files in this PR

Click here if you want to add another bump file to this PR


This comment is maintained by bumpy.

@theoephraim theoephraim merged commit e91d499 into main May 4, 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.

1 participant