Commit 8267653
committed
fix(examples): export PublicToolbarItemId from superdoc/ui barrel; add pnpm-setup to custom-ui CI
The earlier change in this PR added `type PublicToolbarItemId` to
the consumer-facing `superdoc/ui` re-export list (packages/superdoc/
src/ui.d.ts) but never landed the corresponding source export in the
inner barrel at packages/super-editor/src/ui/index.ts. With
`skipLibCheck: true`, TypeScript silently masked the broken
re-export; consumers writing `import type { PublicToolbarItemId }
from 'superdoc/ui'` would still pass typecheck against the dist but
would fail without skipLibCheck (and would not actually resolve to
the typed union when chased through). Add the missing source export
so the chain is real.
The new custom-ui CI matrix job restores the workspace cache and
invokes the smoke test, but does not run pnpm/action-setup. The
example's package.json predev hook calls `pnpm --filter superdoc
build`, and the playwright config falls back to `pnpm --dir <example>
run dev` when the example has no local node_modules (pnpm-hoisted
layout). Both require pnpm on PATH. Add pnpm setup so the smoke test
can launch its dev server.1 parent fc47f2d commit 8267653
2 files changed
Lines changed: 9 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
211 | 211 | | |
212 | 212 | | |
213 | 213 | | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
214 | 222 | | |
215 | 223 | | |
216 | 224 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
24 | 25 | | |
| |||
0 commit comments