feat(ui): Add service aside panel by tab · Pull Request #141 · tab/fuku

tab · 2026-06-02T20:06:51Z

Add Enter keybind to open a side panel with config, env, and health tabs
Switch focus between panels with \ keybind

Add Enter keybind to open a side panel with config, env, and health tabs Switch focus between panels with \ keybind

sentry · 2026-06-02T20:14:23Z

Codecov Report

❌ Patch coverage is 88.88889% with 96 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.57%. Comparing base (10dbb7f) to head (d064c07).

Files with missing lines	Patch %	Lines
internal/app/ui/services/aside.go	89.51%	25 Missing and 12 partials ⚠️
internal/app/ui/components/layout.go	68.60%	23 Missing and 4 partials ⚠️
internal/app/ui/services/update.go	89.41%	8 Missing and 1 partial ⚠️
internal/app/dotenv/parse.go	88.46%	3 Missing and 3 partials ⚠️
internal/app/ui/services/model.go	45.45%	6 Missing ⚠️
internal/app/ui/services/view.go	95.41%	4 Missing and 2 partials ⚠️
internal/app/dotenv/dotenv.go	91.52%	3 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
+ Coverage   89.48%   89.57%   +0.09%     
==========================================
  Files          63       66       +3     
  Lines        5896     6658     +762     
==========================================
+ Hits         5276     5964     +688     
- Misses        487      536      +49     
- Partials      133      158      +25

Files with missing lines	Coverage Δ
internal/app/app.go	`66.03% <100.00%> (ø)`
internal/app/ui/components/theme.go	`100.00% <100.00%> (ø)`
internal/app/ui/services/helpers.go	`100.00% <100.00%> (ø)`
internal/app/ui/services/keys.go	`100.00% <100.00%> (ø)`
internal/config/config.go	`100.00% <ø> (ø)`
internal/app/dotenv/dotenv.go	`91.52% <91.52%> (ø)`
internal/app/dotenv/parse.go	`88.46% <88.46%> (ø)`
internal/app/ui/services/model.go	`70.83% <45.45%> (-1.15%)`	⬇️
internal/app/ui/services/view.go	`96.91% <95.41%> (-1.23%)`	⬇️
internal/app/ui/services/update.go	`80.28% <89.41%> (+5.81%)`	⬆️
... and 2 more

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot

Pull request overview

Adds a new service “info aside” panel to the TUI (opened via enter) that displays per-service configuration, environment (.env) values, and health details, including focus switching between panes and tab navigation. This extends fuku’s interactive UI to surface more service metadata inline while also introducing a new (optional) config surface for env-file display and a new dotenv cache subscriber.

Changes:

Add a split-pane services UI with an aside panel (config/env/health), focus toggle, tab navigation, and rendering/layout optimizations.
Introduce a dotenv bus subscriber + loader to parse/merge .env* files for display (without exporting values to child processes).
Update configuration schema (services.*.env.files) and docs/examples to support the env tab.

Reviewed changes

Copilot reviewed 33 out of 34 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
README.md	Documents the new aside panel and keybindings (needs alignment with actual bindings/behavior).
internal/config/config.go	Extends service config schema with `env.files` for dotenv display.
internal/app/ui/wire/module.go	Wires the dotenv loader into the UI model via FX.
internal/app/ui/wire/module_test.go	Updates UI wiring tests to provide a dotenv mock.
internal/app/ui/services/view.go	Renders split panels and adds caching/conditional UI elements for aside mode.
internal/app/ui/services/view_test.go	Adds coverage for split widths, aside visibility rules, and panel sizing behavior.
internal/app/ui/services/update.go	Adds key handling for opening/closing aside, tab switching, and focus toggling.
internal/app/ui/services/update_test.go	Adds extensive tests for aside open/close, focus/scroll routing, and resize behavior.
internal/app/ui/services/model.go	Stores config + dotenv loader and adds aside state, viewports, and render caches.
internal/app/ui/services/keys.go	Adds bindings for opening/closing aside, tab switching, and focus toggle; adds aside-specific help keymap.
internal/app/ui/services/keys_test.go	Validates the new keybindings and aside help keymap behavior.
internal/app/ui/services/helpers.go	Adds panel width computation, compact layout mode, and viewport recomputation for split view.
internal/app/ui/services/aside.go	Implements aside rendering (tabs, cards, truncation, scroll indicator) including env/health tabs.
internal/app/ui/services/aside_test.go	Adds coverage for aside content rendering, truncation, tab cycling, and env display.
internal/app/ui/components/theme.go	Adds placeholder color/style used by aside empty states.
internal/app/ui/components/styles.go	Adds muted border style and shared content top margin style.
internal/app/ui/components/layout.go	Refactors panel rendering to line-based output, adds border text fitting, compact layout, and metric column support.
internal/app/ui/components/layout_test.go	Updates/extends tests for new layout APIs and border truncation/fitting.
internal/app/ui/components/constants.go	Adds aside sizing constants and compact status width constant.
internal/app/module.go	Registers the dotenv FX module.
internal/app/dotenv/parse.go	Adds `.env` parsing + merge logic (needs security hardening).
internal/app/dotenv/parse_test.go	Tests dotenv parsing and merge ordering/override behavior.
internal/app/dotenv/module.go	Adds FX module and lifecycle hook to run the loader.
internal/app/dotenv/dotenv.go	Implements dotenv loader as a bus subscriber caching merged env entries per service ID.
internal/app/dotenv/dotenv_test.go	Tests loader caching, reload behavior, and event handling.
internal/app/dotenv/dotenv_mock.go	Adds generated gomock for the dotenv Loader interface.
go.mod	Promotes `github.com/charmbracelet/x/ansi` to a direct dependency for ANSI-aware truncation.
fuku.yaml	Updates sample config with `env.files` anchor for the env tab display.
examples/bookstore/auth/.env.test.local	Adds example env file contents for the bookstore auth service.
examples/bookstore/auth/.env.test	Adds example env file contents for the bookstore auth service.
examples/bookstore/auth/.env.development.local	Adds example env file contents for the bookstore auth service.
examples/bookstore/auth/.env.development	Adds example env file contents for the bookstore auth service.
examples/bookstore/auth/.env	Adds example env file contents for the bookstore auth service.
CLAUDE.md	Updates internal project notes to mention the aside panel and service ID conventions.

Files not reviewed (1)

internal/app/dotenv/dotenv_mock.go: Language not supported

- Update documentation to specify that fuku loads its own environment files for process configuration - Clarify that per-service env files are only for display in the TUI info aside - Mention the override mechanism with env.files in configuration

Copilot

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated 7 comments.

Files not reviewed (1)

internal/app/dotenv/dotenv_mock.go: Language not supported

Wrap long .env values (URLs, JWTs, connection strings) to make them visible

Copilot

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated 11 comments.

Files not reviewed (1)

internal/app/dotenv/dotenv_mock.go: Language not supported

- Move close(a.done) to after flushing sentry - Ensures proper shutdown signaling

feat(ui): Add service aside panel

b0e743e

Add Enter keybind to open a side panel with config, env, and health tabs Switch focus between panels with \ keybind

tab requested a review from Copilot June 2, 2026 20:06

tab self-assigned this Jun 2, 2026

Copilot started reviewing on behalf of tab June 2, 2026 20:07 View session

Copilot AI reviewed Jun 2, 2026

View reviewed changes

tab added 2 commits June 2, 2026 23:35

fix(ui): Keep process card stable across restarts

cce40ef

tab requested a review from Copilot June 3, 2026 21:25

Copilot started reviewing on behalf of tab June 3, 2026 21:25 View session

Copilot AI reviewed Jun 3, 2026

View reviewed changes

refactor(ui): Hard-wrap env values in aside panel

c53ec7b

Wrap long .env values (URLs, JWTs, connection strings) to make them visible

tab requested a review from Copilot June 5, 2026 17:58

Copilot started reviewing on behalf of tab June 5, 2026 17:58 View session

Copilot AI reviewed Jun 5, 2026

View reviewed changes

tab added 4 commits June 5, 2026 22:18

docs(dotenv): Update godoc comments

b80e792

docs(ui): Update TUI docs

227dbca

docs(readme): Update README.md

a58ee7a

chore(app): Close app.done channel after execution

d064c07

- Move close(a.done) to after flushing sentry - Ensures proper shutdown signaling

tab merged commit a63b869 into master Jun 5, 2026
10 checks passed

tab deleted the feature/service-info-aside branch June 5, 2026 20:29

Conversation

tab commented Jun 2, 2026

Uh oh!

sentry Bot commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

sentry Bot commented Jun 2, 2026 •

edited

Loading