Uh oh!
There was an error while loading. Please reload this page.
Add TUI table overrides for high-traffic list commands - #4732
Merged
simonfaltum merged 8 commits intoMar 23, 2026
Merged
Conversation
simonfaltumtemporarily deployed
to
test-trigger-is
March 13, 2026 06:40 — with 
GitHub Actions
Inactive
Collaborator
Commit: 3725d38
20 interesting tests: 9 SKIP, 6 RECOVERED, 5 flaky
Top 32 slowest tests (at least 2 minutes):
|
simonfaltum
marked this pull request as ready for review
March 13, 2026 10:33
simonfaltum
requested review from
andrewnester, anton-107, denik, pietern and shreyas-goenka
as code ownersMarch 13, 2026 10:33
shreyas-goenka
left a comment
Contributor
There was a problem hiding this comment.
Note: This review was posted by Claude (AI assistant). Shreyas will do a separate, more thorough review pass.
Priority: LOW — Clean mechanical PR
This is a straightforward PR that adds TUI table overrides for 5 commands. No bugs or correctness issues found.
Minor Nits
- Unused
listReqvariables: Several overrides declarelistReq(e.g.,listReq := &catalog.ListCatalogsRequest{}) that is never used after flag binding. Consider using_or removing if unneeded. - Consistency: The pattern matches the existing override style in the codebase, which is good.
What looks good
- Clean, consistent pattern across all 5 overrides
- Correct use of
tableview.RegisterConfigfor each command - Proper column selection for each resource type
No blocking issues.
simonfaltumtemporarily deployed
to
test-trigger-is
March 13, 2026 12:41 — with GitHub Actions
Inactive
simonfaltumtemporarily deployed
to
test-trigger-is
March 13, 2026 14:48 — with GitHub Actions
Inactive
simonfaltumtemporarily deployed
to
test-trigger-is
March 13, 2026 15:07 — with GitHub Actions
Inactive
simonfaltumtemporarily deployed
to
test-trigger-is
March 13, 2026 15:07 — with GitHub Actions
Inactive
simonfaltumtemporarily deployed
to
test-trigger-is
March 13, 2026 21:06 — with GitHub Actions
Inactive
simonfaltumtemporarily deployed
to
test-trigger-is
March 14, 2026 21:54 — with GitHub Actions
Inactive
shreyas-goenka
approved these changes
Mar 19, 2026
shreyas-goenka
left a comment
Contributor
There was a problem hiding this comment.
Stamping. I'm assuming the TUI UI is better.
| // sanitizeControlWhitespace replaces newlines and tabs with spaces to prevent | ||
| // corrupting tab-delimited text output. | ||
| func sanitizeControlWhitespace(s string) string { |
Contributor
There was a problem hiding this comment.
This function is also defined in cmd/workspace/pipelines/overrides.go. Consolidate?
Base automatically changed from
simonfaltum/list-tui-overrides to
simonfaltum/list-tui-paginatedMarch 19, 2026 06:43
simonfaltumtemporarily deployed
to
test-trigger-is
March 19, 2026 06:46 — with GitHub Actions
Inactive
Contributor
Suggested reviewersBased on git history of the changed files, these people are best suited to review:
Confidence: high Eligible reviewersBased on CODEOWNERS, these people or teams could also review: @andrewnester, @anton-107, @databricks/eng-apps-devex, @denik, @lennartkats-db, @shreyas-goenka Suggestions based on git history of 37 changed files (11 scored). See CODEOWNERS for path-specific ownership rules. |
simonfaltumtemporarily deployed
to
test-trigger-is
March 20, 2026 07:36 — with GitHub Actions
Inactive
Several list override functions declared a named request parameter (e.g. listReq) that was never used after flag binding. Replace these with blank identifiers to satisfy go vet and make intent clearer. The parameter is kept named in clusters, jobs, pipelines, and workspace overrides where it is actively used for flag binding or search closures.
Previously, RenderIterator and RunPaginated only returned the error from tea.Program.Run(), ignoring any fetch error stored in the model. An API error mid-stream would display an error screen in the TUI but the command would still exit 0. Now both functions inspect the final model via the new Err() accessor and return the fetch error if set. Also documents the destructive MaxWidth truncation behavior on ColumnDef and renderContent.
Pipeline event messages can contain embedded newlines, carriage returns, and tabs that corrupt tab-delimited text output and TUI table rows. Add a `sanitize` template function to cmdio's renderFuncMap and use it in the text template. Also sanitize in the TUI Extract function. Increase MaxWidth from 60 to 200 so diagnostic payloads are not truncated destructively before the actionable part of the error. Co-authored-by: Isaac
Remove the PaginatedModel type alias (FinalModel interface suffices). Remove the duplicate TestPaginatedErrAccessor test that overlaps with TestPaginatedModelErr. Reduce the 5-line MaxWidth truncation comment to a single line. Co-authored-by: Isaac
simonfaltumforce-pushed
the
simonfaltum/list-tui-overrides-2
branch
from
March 22, 2026 07:40
a725914 to
6388a6eComparesimonfaltumtemporarily deployed
to
test-trigger-is
March 22, 2026 07:40 — with GitHub Actions
Inactive
Co-authored-by: Isaac
simonfaltumtemporarily deployed
to
test-trigger-is
March 22, 2026 21:44 — with GitHub Actions
Inactive
Uh oh!
There was an error while loading. Please reload this page.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Why
PR #4731 added curated TUI table overrides for 15 list commands. This follow-up covers 5 additional commands that are among the most frequently used in the CLI, but were missing curated columns.
Changes
Before: these 5 commands used either generic text templates (secrets, cluster-policies) or raw JSON output (lakeview, pipeline events) with no curated TUI table columns.
Now: all 5 register curated TableConfig overrides so they show useful columns in the interactive TUI. Commands that had no text template override (lakeview list, pipelines list-pipeline-events) also get template annotations for the non-interactive fallback.
This PR stacks on #4731. It only adds per-command overrides, no engine changes.
Post-review fixes
\n,\r,\t) in pipeline event messages to prevent table row corruptionPaginatedModeltype alias, useFinalModelinterface insteadTestPaginatedErrAccessortestTest plan
go build ./...make checkspassesmake lintfullpasses (0 issues)secrets list-scopes,lakeview list,pipelines list-pipeline-events