Skip to content

fix(core): remove AskableContextImpl from public exports#154

Merged
vamgan merged 2 commits intomainfrom
fix/138-remove-context-impl-export
Apr 10, 2026
Merged

fix(core): remove AskableContextImpl from public exports#154
vamgan merged 2 commits intomainfrom
fix/138-remove-context-impl-export

Conversation

@vamgan
Copy link
Copy Markdown
Collaborator

@vamgan vamgan commented Apr 10, 2026

Summary

  • Removes AskableContextImpl from the public re-exports in @askable-ui/core
  • The implementation class was never intended to be part of the public API; consumers should use the createAskableContext factory function instead
  • The internal import used by createAskableContext is unchanged

Closes #138

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 10, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@askable-ui/core@154
npm i https://pkg.pr.new/@askable-ui/react@154
npm i https://pkg.pr.new/@askable-ui/svelte@154
npm i https://pkg.pr.new/@askable-ui/vue@154

commit: 1686c73

@vamgan vamgan merged commit 7426a50 into main Apr 10, 2026
6 checks passed
@vamgan vamgan deleted the fix/138-remove-context-impl-export branch April 10, 2026 21:13
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.

Bug: AskableContextImpl leaks implementation class in public API

1 participant