Migrate consumers to @workflowbuilder/ui (2/3: consumers)#45
Open
librowski wants to merge 2 commits into
Open
Conversation
This was referenced Jun 24, 2026
librowski
requested review from
lukasz-jazwa,
piotrblaszczyk and
szymon-t-sc
as code owners
The SDK bundles @workflowbuilder/ui and @base-ui/react into its dist, but
declared them as runtime dependencies - so a published @workflowbuilder/sdk
would 404 on @workflowbuilder/ui (not published) at npm install. Move both to
devDependencies (they are inlined, consumers don't install them).
Add @workflowbuilder/ui to the dts plugin's bundledPackages so its types are
inlined into dist/index.d.ts instead of leaking unresolvable
import('@workflowbuilder/ui') references into the public type surface. Verified:
no real ui/base-ui imports remain in dist/index.d.ts.
Remove the obsolete docs/overflow-ui.md (described the old external
@synergycodes/overflow-ui local-dev flow this migration replaces).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part 2 of 3. Stacked on #44 (base branch
librowski/ui-package) - review/merge that first. This PR's diff is only the consumer migration.Summary
Switches
sdk/demo/ai-studiooff the published@synergycodes/overflow-ui@1.0.0-beta.27(MUI) onto the in-repo@workflowbuilder/ui(Base UI) added in Part 1, and fixes the resulting API breaks.Changes
@synergycodes/overflow-ui→@workflowbuilder/ui; dependency →workspace:*;@base-ui/reactadded to sdk/demo/ai-studio.LOCAL_OVERFLOW_UIvite aliases, theoverflow-ui-cssalias, and thedev:localscript. SDKindex.cssnow pulls@workflowbuilder/ui/{tokens,index}.css.Switch.onChange(checked, event)and DatePickerdate-fnsvalueFormat. Menu / Select / IconSwitch / Tooltip were already compatible.@workflowbuilder/ui.@workflowbuilder/sdkincluded.Verification
pnpm installclean;build,build:lib,build:ai-studio,build:docsgreen;typecheck,lint,test(sdk 184 + execution-core 78) green.1.4.1pin (Part 1) is what keeps the modal backdrop fade working - 1.6.0 regressed it.@synergycodes/overflow-uifrom npm or the old local dist path.