wip: feat: add TUI w/ bubbletea support - #790
Draft
floreks wants to merge 22 commits into
Draft
Conversation
floreks
requested review from
maciaszczykm,
michaeljguarino and
zreigz
as code owners
July 29, 2026 06:21
floreks
marked this pull request as draft
July 29, 2026 06:21
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Contributor
Greptile SummaryAdds a Bubble Tea terminal interface and supporting authentication/access infrastructure.
Confidence Score: 2/5The credential rollback and stale-cursor panic need to be fixed before merging; reverse panel navigation also behaves incorrectly. Failed profile updates can remove an existing credential, and loading a shorter profile list leaves stale cursors that panic when the user confirms a selection. Files Needing Attention: pkg/bridge/access/access.go, tui/screens/access/model.go
|
| Filename | Overview |
|---|---|
| pkg/bridge/access/access.go | Adds access-profile orchestration, but failed saves can delete credentials belonging to existing profiles. |
| pkg/bridge/access/local.go | Adds owner-only atomic registry persistence and one-time migration of legacy app and console profiles. |
| pkg/bridge/credentials.go | Adds keyring-backed credential storage with a permission-restricted filesystem fallback. |
| pkg/bridge/auth.go | Moves device login, identity resolution, impersonation, and access-token exchange behind a context-aware service. |
| pkg/common/common.go | Migrates existing login flows onto the bridge authentication service while retaining legacy persistence behavior. |
| tui/screens/access/model.go | Implements access-screen state handling, but reloads can leave invalid cursors that panic on confirmation and reverse panel navigation moves in the wrong direction. |
| tui/app/model.go | Adds top-level TUI routing, sizing, and quit/cancellation handling. |
| cmd/command/tui/tui.go | Wires production welcome and access dependencies into the new explicit TUI command. |
Reviews (1): Last reviewed commit: "feat: initialize Plural CLI authenticati..." | Re-trigger Greptile
- Add `assets.go` to embed terminal cell logo for CLI - Implement core authentication types and interfaces in `bridge/types.go` - Develop CLI state reading logic in `bridge/welcome/local.go` - Add unit tests for `welcome` package functionality - Create access management infrastructure in `bridge/access/local.go` - Include diagnostics view tests in `tui/screens/diagnostics` - Establish access management and tests in `bridge/access/access.go` - Add TUI models for diagnostics and access screens
zreigz
force-pushed
the
feat/bubbletea-rewrite
branch
from
July 30, 2026 10:11
29c8786 to
786b8ae
Compare
- Add AI and workbenches as new sections in TUI and implement navigation logic - Integrate agents and workbenches bridges in TUI app model - Implement list, get, and job functionalities for console workbenches - Update unit tests to cover new AI routes and navigation capabilities
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
assets.goto embed terminal cell logo for CLIbridge/types.gobridge/welcome/local.gowelcomepackage functionalitybridge/access/local.gotui/screens/diagnosticsbridge/access/access.goSummary
Labels
Test Plan
Checklist