Skip to content

fix: publish tui runtime dependencies#11

Open
fengjikui wants to merge 1 commit into
prevalentWare:mainfrom
fengjikui:codex/tui-runtime-dependencies
Open

fix: publish tui runtime dependencies#11
fengjikui wants to merge 1 commit into
prevalentWare:mainfrom
fengjikui:codex/tui-runtime-dependencies

Conversation

@fengjikui

Copy link
Copy Markdown

What changed

  • Moves @opentui/solid and solid-js from devDependencies to dependencies.
  • Adds a package manifest regression test for the published TUI entrypoint runtime imports.
  • Updates bun.lock to match the package dependency classification.

Why

The package publishes src/tui.tsx through the ./tui export. That source file uses the OpenTUI Solid JSX runtime, so consumers that install the published package without dev dependencies need @opentui/solid and solid-js available at runtime.

I reproduced the install-path failure with a packed tarball before this change:

pkg=$(npm pack --silent)
tmp=$(mktemp -d)
cp "$pkg" "$tmp/"
(cd "$tmp" && npm init -y >/dev/null && npm install --omit=dev "./$pkg" >/dev/null && bun -e 'import("@prevalentware/opencode-goal-plugin/tui")')

Before the fix, that failed with:

Cannot find module '@opentui/solid/jsx-dev-runtime' from .../node_modules/@prevalentware/opencode-goal-plugin/src/tui.tsx

After the fix, the same clean packed install/import completes with import ok.

Validation

  • bun test
  • bun run lint
  • bun run typecheck
  • bun run build
  • bun run pack:dry-run
  • clean packed install with npm install --omit=dev plus bun -e 'import("@prevalentware/opencode-goal-plugin/tui")'

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