Workflow: Phase 1 - Foundation (Issues #11763, #11764, #11776) #8327

Closed
opened 2026-02-16 18:09:41 -05:00 by yindo · 3 comments
Owner

Originally created by @randomm on GitHub (Feb 2, 2026).

Originally assigned to: @thdxr on GitHub.

Workflow Tracking

This issue tracks the Phase 1 implementation workflow for the Ink TUI rewrite.

Issues in Phase 1

Issue Title Status PR
#11763 Setup Ink dependencies Implemented Pending
#11776 Theming system 🔄 Next -
#11764 App component & state 🔄 Next -

Workflow Steps

  1. Research completed (Ink patterns, lite TUI analysis)
  2. GitHub issues exist
  3. Feature branch: feature/oclite-ink-rewrite
  4. 🔄 TDD implementation
  5. PR creation
  6. Code review
  7. Squash merge

PR Strategy

Combining #11763, #11776, #11764 into single PR since:

Branch

feature/oclite-ink-rewrite - commit ab97fd9dd (#11763 complete)

Part of Epic #11762

Originally created by @randomm on GitHub (Feb 2, 2026). Originally assigned to: @thdxr on GitHub. ## Workflow Tracking This issue tracks the Phase 1 implementation workflow for the Ink TUI rewrite. ### Issues in Phase 1 | Issue | Title | Status | PR | |-------|-------|--------|-----| | #11763 | Setup Ink dependencies | ✅ Implemented | Pending | | #11776 | Theming system | 🔄 Next | - | | #11764 | App component & state | 🔄 Next | - | ### Workflow Steps 1. ✅ Research completed (Ink patterns, lite TUI analysis) 2. ✅ GitHub issues exist 3. ✅ Feature branch: `feature/oclite-ink-rewrite` 4. 🔄 TDD implementation 5. ⏳ PR creation 6. ⏳ Code review 7. ⏳ Squash merge ### PR Strategy Combining #11763, #11776, #11764 into single PR since: - All are foundational setup - Combined < 500 lines - Same epic #11762 ### Branch `feature/oclite-ink-rewrite` - commit `ab97fd9dd` (#11763 complete) Part of Epic #11762
yindo added the opentui label 2026-02-16 18:09:41 -05:00
yindo closed this issue 2026-02-16 18:09:41 -05:00
Author
Owner

@randomm commented on GitHub (Feb 2, 2026):

Status Update

In Progress

  • 🔄 Fixing Ink React hook error (critical - blocks PR)
  • 🔄 Theme system implemented, verifying tests
  • 🔄 App state types TDD started

Code Review Feedback (#11763)

  • CHANGES_REQUESTED - Invalid hook call error
  • Root cause: Bun JSX transformation conflict
  • Fix in progress: Using Babel React preset for cli/ink/ files

Commits on Branch

  • ab97fd9dd - Initial Ink setup (pending fix)

Next Steps

  1. Complete critical fix
  2. Re-request code review
  3. Continue App component implementation
  4. Create PR once review passes
@randomm commented on GitHub (Feb 2, 2026): ## Status Update ### In Progress - 🔄 Fixing Ink React hook error (critical - blocks PR) - 🔄 Theme system implemented, verifying tests - 🔄 App state types TDD started ### Code Review Feedback (#11763) - **CHANGES_REQUESTED** - Invalid hook call error - Root cause: Bun JSX transformation conflict - Fix in progress: Using Babel React preset for cli/ink/ files ### Commits on Branch - `ab97fd9dd` - Initial Ink setup (pending fix) ### Next Steps 1. Complete critical fix 2. Re-request code review 3. Continue App component implementation 4. Create PR once review passes
Author
Owner

@randomm commented on GitHub (Feb 2, 2026):

Progress Update

Phase 1 Complete

Commit: 3db1fb3e4 (code review fixes in progress)

Phase 2 In Progress 🔄

  • #11765 StreamingProse - implementing (simplified)
  • #11769 TextInput - done, adversarial review
  • #11770 SelectMenu - done, adversarial review
  • #11774 StatusBar - done

Remaining

Current Blockers

  • Fixing typecheck/test failures before re-review
@randomm commented on GitHub (Feb 2, 2026): ## Progress Update ### Phase 1 Complete ✅ - #11763 Setup Ink dependencies - #11764 App component & state - #11766 ToolDisplay - #11767 TaskDisplay - #11776 Theme system **Commit:** `3db1fb3e4` (code review fixes in progress) ### Phase 2 In Progress 🔄 - #11765 StreamingProse - implementing (simplified) - #11769 TextInput - done, adversarial review - #11770 SelectMenu - done, adversarial review - #11774 StatusBar - done ### Remaining - #11768 Static integration - #11771 useSDKEvents hook - #11772 useStreaming hook - #11773 Slash commands ### Current Blockers - Fixing typecheck/test failures before re-review
Author
Owner

@randomm commented on GitHub (Feb 2, 2026):

Final Status

All Commits

SHA Description
ab97fd9dd Initial Ink setup
3db1fb3e4 Phase 1 foundation
08c44170f Code review fixes
d6eab3cf2 Phase 2 components
4345bb18e Test fix + MessageList

Issues Completed

Test Coverage

  • 90+ Ink tests passing
  • Typecheck clean

Awaiting

  • Final code review approval
  • PR creation and merge
@randomm commented on GitHub (Feb 2, 2026): ## Final Status ### All Commits | SHA | Description | |-----|-------------| | `ab97fd9dd` | Initial Ink setup | | `3db1fb3e4` | Phase 1 foundation | | `08c44170f` | Code review fixes | | `d6eab3cf2` | Phase 2 components | | `4345bb18e` | Test fix + MessageList | ### Issues Completed - ✅ #11763 Setup Ink dependencies - ✅ #11764 App component & state - ✅ #11765 StreamingProse - ✅ #11766 ToolDisplay - ✅ #11767 TaskDisplay - ✅ #11768 Static/MessageList integration - ✅ #11769 TextInput/InputLine - ✅ #11770 SelectMenu - ✅ #11774 StatusBar - ✅ #11776 Theme system ### Test Coverage - 90+ Ink tests passing - Typecheck clean ### Awaiting - Final code review approval - PR creation and merge
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#8327