Skip to content

refactor!: keep lib helpers out of the public surface - #63

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-9102-react-ui-stop-exporting-lib-internals-from-root-barrel
Sep 27, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-9102-react-ui-stop-exporting-lib-internals-from-root-barrel

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • lib/csv, lib/url, lib/utils, lib/workflow-registry and ui/chat-dock-timing are INTERNAL: absent from the root barrel and the exports map (breaking; CHANGELOG lists the dropped symbols).
  • ui/step-list re-exports StepListItem/StepDisplayStatus and ui/gate-block re-exports GateShellModel/GateKind, so those component props stay nameable.
  • src/index.ts and package.json are regenerated, so they may conflict with feat(ui): primitives for the stage-workspace port #58/feat(chat): optional attachIcon and sendIcon on ChatInput #59; rerun bun run generate to resolve.
  • Deletes the workflow-registry exports nothing imports (LiveRunPhase, LIVE_RUN_PHASE_*, WorkflowListItem*, KindPickerItem). CHANGELOG tells cn users to compose clsx and tailwind-merge.

Verification

typecheck, lint, dep-guard, bun test (268 pass) and bun run build pass.

Closes CL-9102

lib/csv, lib/url, lib/utils, lib/workflow-registry and
ui/chat-dock-timing join INTERNAL, so they leave the root barrel and
the exports map. StepList and GateBlock re-export the prop types they
take from workflow-registry.
@TheGreatAxios
TheGreatAxios force-pushed the cl-9102-react-ui-stop-exporting-lib-internals-from-root-barrel branch from 9763671 to c12c96c Compare September 26, 2026 01:32
@TheGreatAxios TheGreatAxios changed the title Keep lib helpers out of the public surface refactor!: keep lib helpers out of the public surface Sep 26, 2026
@TheGreatAxios
TheGreatAxios added this pull request to stack #74 September 26, 2026 01:57
@TheGreatAxios
TheGreatAxios merged commit ec644d6 into main Sep 27, 2026
1 check 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