Skip to content

feat(devtui): redesign overview tab with setup health report#1157

Merged
Soner (shyim) merged 3 commits into
nextfrom
feat/devtui-overview-setup-health
Jul 3, 2026
Merged

feat(devtui): redesign overview tab with setup health report#1157
Soner (shyim) merged 3 commits into
nextfrom
feat/devtui-overview-setup-health

Conversation

@shyim

Copy link
Copy Markdown
Member

Summary

Redesigns the devtui overview tab into a two-column layout: a readonly project report on the left and a "User action" column with the watchers on the right (narrow terminals fall back to a stacked layout).

  • Access table: Admin Access and Services merged into one table (Service / URL / Username / Password-Auth) with the Shop Admin login as the first row; credential-less services show no auth
  • Setup health report: PHP version and memory limit are read from the PHP runtime the project actually uses (inside the container for docker environments); admin worker and Flow Builder log level are resolved from config/packages for the project's APP_ENV. Checks whose data source is unavailable are omitted rather than shown as failures
  • Background processing moved to the left column, rendered with the same status dots as the health checks (this made tui.CheckboxRow dead code, removed)
  • Clickable links: all URLs and the health check names are OSC 8 hyperlinks in the shared underlined link style; check names open the relevant Shopware docs page (anchors verified). Terminals without OSC 8 support show plain styled text
  • Arrow-key focusing is reduced to the two watchers; services are no longer focus targets

Screenshots

Rendered at 120 columns:

Project details and readonly setup report.                        │  User action
                                                                  │
Shop                                                              │  Watchers
  Version               6.7.11.1                                  │    [ ] Admin         stopped
  Security updates      until 2028-02-28 (66 days left)           │    [ ] Storefront    stopped
  Environment           docker                                    │
  Shop URL              http://127.0.0.1:8000                     │
  Admin URL             http://127.0.0.1:8000/admin               │
──────────────────────────────────────────────────────────────────│
Access                                                            │
  Service      URL                           Username   Password / Auth
  Shop Admin   http://127.0.0.1:8000/admin   admin      shopware  │
  Adminer      http://127.0.0.1:9080         root       root      │
  Mailpit      http://127.0.0.1:55815        -          no auth   │
──────────────────────────────────────────────────────────────────│
Setup health                                                      │
    Check                    Current    Recommended               │
                                                                  │
    Runtime                                                       │
  ● PHP version              8.3.12     >= 8.2                    │
  ● Memory limit             512M       >= 512M                   │
                                                                  │
    Local behavior                                                │
  ● Admin Worker             disabled   disabled                  │
                                                                  │
    Debug (Flow Builder)                                          │
  ● Flow Builder log level   DEBUG      min WARNING               │

Testing

  • Unit tests for the memory-limit parser, each health check's level classification, the Access table (installed / not-installed states), both layouts, and a regression test that OSC 8 sequences stay zero-width so columns remain aligned
  • go test ./..., go vet, and gofmt clean

Split the overview into a two-column layout: a readonly project report on
the left and a "User action" column with the watchers on the right, with a
stacked fallback for narrow terminals.

- Merge Admin Access and Services into one Access table with the Shop
  Admin login as the first row; credential-less services show "no auth"
- Add a Setup health report: PHP version and memory limit read from the
  project's PHP runtime via the executor, admin worker and Flow Builder
  log level resolved from config/packages for the project's APP_ENV
- Move background processing to the left column and render it with the
  same status dots as the health checks
- Make all URLs and health check names clickable OSC 8 hyperlinks in the
  shared underlined link style; check names open the relevant docs page
- Drop arrow-key focusing of services; only the watchers remain focusable

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6784c58de4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "Codex (@codex) review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".

Comment thread internal/devtui/tab_overview.go
Comment thread internal/devtui/tab_overview.go Outdated
- Raise the two-column breakpoint to 110 so the Access table cannot wrap
  into the right column on 100-107 column terminals
- Refresh discovered services and setup-health checks after a config save
  restarts the containers, instead of showing stale runtime values
@shyim Soner (shyim) merged commit 7c9c870 into next Jul 3, 2026
2 checks passed
@shyim Soner (shyim) deleted the feat/devtui-overview-setup-health branch July 3, 2026 11:21
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