Skip to content

Youngjin/UI next#123

Open
yjp20 wants to merge 22 commits into
mainfrom
youngjin/ui-next
Open

Youngjin/UI next#123
yjp20 wants to merge 22 commits into
mainfrom
youngjin/ui-next

Conversation

@yjp20
Copy link
Copy Markdown
Member

@yjp20 yjp20 commented Mar 16, 2026

No description provided.

@yjp20 yjp20 force-pushed the youngjin/ui-next branch 4 times, most recently from a7b74d9 to e8d6426 Compare March 17, 2026 05:35
stainless-app Bot and others added 18 commits March 17, 2026 15:24
@stainless-app
fix: only set client options when the corresponding CLI flag or env v…
64c31cf 
…ar is explicitly set
@stainless-app
chore(internal): version bump
d6f4985
@yjp20
fix: fill project property more uniformly
23b8227
@yjp20
refactor: auto-set Before hook on all subcommands via traversal
73a12cd
@yjp20
feat: add git Show and CurrentBranch helpers
7427170
@yjp20
feat(components/diagnostics): rewrite diagnostics view with Rust-styl…
c094b0f 
…e formatting
@yjp20
feat(components/build): rewrite build pipeline view as single-line wi…
270e85b 
…th header
@yjp20
feat(cmd/dev): refactor dev command to use compare builds
b7cc41a
@yjp20
feat(cmd/build): use build component view for builds list
0c0c4ee
@yjp20
feat(cmd/builddiagnostic): use diagnostics component for diagnostics …
407ce31 
…list
@yjp20
fix: read check step conclusion from top-level field
e16abc3 
The deprecated nested completed.conclusion field was being read
instead of the top-level conclusion field on CheckStepUnion.
@yjp20
feat(components/dev): remove config section and waiting message from …
83b7995 
…preview
@yjp20
feat(components/build): single newline after header, show download path
cba8ce9
@yjp20
feat(cmd/build): add spacing between header and targets in builds list
63df786
@yjp20
fix(components/dev): preserve preview content on quit
fcb4d88 
Render the full view even when there's an error instead of
short-circuiting, so content persists after ctrl+c.
@yjp20
refactor: don't use deprecated .Completed property
397114c
@yjp20
fix: make download error display better
31a38dc
@yjp20
chore: automatically generate demo gifs
a057170
@yjp20 yjp20 force-pushed the youngjin/ui-next branch from e8d6426 to a057170 Compare March 19, 2026 02:11
@yjp20 yjp20 force-pushed the youngjin/ui-next branch from a3aebe9 to 48c573e Compare March 19, 2026 02:18
@Karen86Tonoyan
Copy link
Copy Markdown

Karen86Tonoyan commented May 30, 2026

  • Podziel PR na mniejsze części:

    • UI Components
    • Build/Dev Commands
    • Git Helpers
    • Refactoring

  • Dodać testy regresyjne dla:

    • CurrentBranch
    • Show
    • builddiagnostic
    • compare builds

  • Usunąć duplikację logiki renderowania między components/build i components/dev.

  • Zweryfikować obsługę błędów dla wszystkich nowych wywołań Git.

  • Dodać ochronę przed nil/null przy odczycie conclusion i statusów checków.

  • Ujednolicić nazewnictwo metod oraz struktur pomiędzy komponentami Build i Diagnostics.

  • Ograniczyć odpowiedzialność komendy dev do orkiestracji, a logikę porównywania buildów przenieść do warstwy serwisowej.

  • Sprawdzić zgodność z terminalami o małej szerokości oraz poprawność zawijania nowych widoków jednoliniowych.

  • Dodać benchmark renderowania dla nowych komponentów TUI.

  • Zweryfikować kompatybilność zmian z istniejącymi skryptami CI korzystającymi z poprzednich formatów wyjścia.

  • Rozważyć feature flag dla nowego interfejsu UI przed pełnym wdrożeniem.

  • Dodać dokumentację nowych helperów Git wraz z przykładami użycia.

  • Upewnić się, że wszystkie błędy pobierania zawierają kod błędu, przyczynę oraz sugerowane działanie użytkownika.

  • Wymusić spójny format odstępów i nagłówków we wszystkich widokach CLI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yjp20 @Karen86Tonoyan @bruce-hill