Skip to content

fix(tui): Correctly render summary text in TUI Wizard #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
starpia-forge merged 1 commit into from
Mar 6, 2026
Merged

fix(tui): Correctly render summary text in TUI Wizard #11

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 12 additions & 7 deletions internal/tui/tui_wizard.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,9 @@ func (m *WizardModel) createForm() {
}),
),
huh.NewGroup(
huh.NewConfirm().
Key("confirm").
TitleFunc(func() string {
huh.NewNote().
Title("Summary of settings:").
DescriptionFunc(func() string {
url := m.form.GetString("repoURL")
dir := m.form.GetString("repoDir")
if dir == "" {
Expand All @@ -113,14 +113,19 @@ func (m *WizardModel) createForm() {
iStr = "30"
}

return fmt.Sprintf("Summary of settings:\n\n"+
return fmt.Sprintf("\n"+
"Repository URL: %s\n"+
"Local Directory: %s\n"+
"Branch: %s\n"+
"Command: %s\n"+
"Interval: %s seconds\n\n"+
"Proceed with these settings?", url, dir, b, c, iStr)
}, &repoURL).
"Interval: %s seconds\n", url, dir, b, c, iStr)
}, &repoURL), // Note: huh DescriptionFunc expects exactly one dependency argument of type any in this version.
// We can use a struct to track all dependencies if needed, but since we are navigating forward sequentially
// without jumping, repoURL is technically enough to avoid compilation error while fulfilling the function signature.
// Let's create an aggregate pointer or just pass a struct pointer.
huh.NewConfirm().
Key("confirm").
Title("Proceed with these settings?").
Value(&confirm),
),
)
Expand Down