Skip to content

fix: split type-only and runtime exports#518

Merged
43081j merged 2 commits intobombshell-dev:mainfrom
jeffrey-takuma:fix/prompts-export-split
Apr 26, 2026
Merged

fix: split type-only and runtime exports#518
43081j merged 2 commits intobombshell-dev:mainfrom
jeffrey-takuma:fix/prompts-export-split

Conversation

@jeffrey-takuma
Copy link
Copy Markdown
Contributor

Summary

Splits the mixed type/value re-export from @clack/core into separate type-only and runtime exports.

This avoids runtime exports such as isCancel, settings, and updateSettings being dropped by some TypeScript emitters or bundlers.

Closes #470

Testing

  • pnpm build
  • pnpm types

No tests were added because this only separates type-only and runtime re-exports.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 26, 2026

🦋 Changeset detected

Latest commit: 8310833

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

This PR includes changesets to release 1 package
Name Type
@clack/prompts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 26, 2026

commit: 8310833

@gameroman
Copy link
Copy Markdown

Would it make sense to add a biome lint rule for this?

@43081j 43081j merged commit 5b897a7 into bombshell-dev:main Apr 26, 2026
7 checks passed
@43081j
Copy link
Copy Markdown
Collaborator

43081j commented Apr 26, 2026

i'd rather us not have to maintain custom rules. but if this ends up existing inside biome, i'd be up for turning it on

@gameroman
Copy link
Copy Markdown

Pretty sure there is already a builtin rule for that

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.

isCancel runtime export regression in @clack/prompts

3 participants