fix: Goal sidebar rendering with the OpenTUI function API

[tienbac2314]

Summary

Rewrites the Goal TUI entrypoint from TSX/JSX to the @opentui/solid function API because the published raw TSX entrypoint does not render in OpenCode 1.18.3 on the tested Windows 11 and Ubuntu 24.04 installations.

The server plugin and /goal command already load correctly. This change keeps the existing sidebar data, lifecycle actions, cache behavior, and command-palette integration while avoiding the failing JSX renderer path.

Related issue

Closes #14

Changes

• Replace JSX rendering with createElement, insert, and setProp.
• Preserve Goal states and fields: objective, status, elapsed time, tokens, auto-continues, checkpoint, latest status, and stop reason.
• Preserve refresh, history, pause, resume, and clear actions.
• Preserve session tool-output scanning and in-memory/KV snapshot caching.
• Request a TUI render once per second so elapsed time and state changes refresh.
• Rename the TUI source entrypoint from src/tui.tsx to src/tui.ts and update package exports, published files, tests, and README documentation.

Verification

Tested the local package with OpenCode 1.18.3 on Windows 11. The vanilla 0.1.24 package loaded /goal but did not show the Goal block; this function-API version rendered it on the same installation.

Commands run:

bun run lint
# passed

bun run typecheck
# passed

bun test test/tui.test.ts test/package.test.ts
# 8 passed, 0 failed

bun run build
# passed; server.js built successfully

bun run pack:dry-run
# passed; package contains dist/server.js and src/tui.ts

Full bun test result on Windows:

63 passed, 1 failed

The only failure is the existing POSIX file-mode assertion in test/state.test.ts (0o666 received instead of 0o600 on Windows). All TUI and package tests pass.

Visual comparison:

Vanilla 0.1.24: Goal block absent

Function-API rewrite: Goal block visible

Checklist

• bun test passes (new behavior has regression coverage)
• bun run lint passes
• bun run typecheck passes
• bun run build passes and dist/server.js is committed if server code changed
• README/docs updated if behavior or options changed

Full bun test is unchecked because of the Windows-only file-permission assertion documented above. The focused TUI/package suite passes.

Note: merging to main automatically publishes a new patch release to npm.

[danyel117]

Thanks for the detailed reproduction and for validating the function-API approach on Windows and Ubuntu. The direction looks valuable and appears to address the missing sidebar from #14.

I found one functional regression that should be fixed before merge:

• In src/tui.ts, GoalSidebar computes elapsed once when the slot output is constructed. The one-second interval only calls renderer.requestRender(), which repaints the existing renderables but does not rerun liveTimeUsedSeconds() or update the text node. As a result, an active goal’s displayed elapsed time remains frozen until another reactive state change reconstructs the slot output.

Please make the elapsed text reactive (for example, with a signal/accessor insertion) or explicitly update the elapsed text node on each timer tick. It would also be good to add a render-level regression test that mounts an active goal and verifies that the visible elapsed time advances after a tick. The current sidebar registration test uses an empty session, so it does not render or inspect the goal tree and cannot catch this behavior.

Local validation otherwise passed for me:

• bun run typecheck
• bun run lint
• bun test (64 passed)
• bun run build
• bun run pack:dry-run
• packed --omit=dev install and TUI entrypoint import

Once the live timer is fixed and covered, I think this should be ready for re-review.