Skip to content

Commit 0f647b1

Browse files
wesmclaude
andcommitted
Fix review findings: --addr normalization, roborev-tui references
- Normalize --addr flag to add http:// prefix if missing - Update init message to say 'roborev tui' not 'roborev-tui' - Remove roborev-tui from .gitignore (binary no longer exists) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent c76ce4a commit 0f647b1

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
# Binaries (in root directory only)
22
/roborev
33
/roborevd
4-
/roborev-tui
54
*.exe
65
/dist/
76

cmd/roborev/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ roborev enqueue --sha HEAD 2>/dev/null &
206206
fmt.Println("Commands:")
207207
fmt.Println(" roborev status - view queue and daemon status")
208208
fmt.Println(" roborev show HEAD - view review for a commit")
209-
fmt.Println(" roborev-tui - interactive terminal UI")
209+
fmt.Println(" roborev tui - interactive terminal UI")
210210

211211
return nil
212212
},

cmd/roborev/tui.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -449,6 +449,8 @@ func tuiCmd() *cobra.Command {
449449
RunE: func(cmd *cobra.Command, args []string) error {
450450
if addr == "" {
451451
addr = getDaemonAddr()
452+
} else if !strings.HasPrefix(addr, "http://") && !strings.HasPrefix(addr, "https://") {
453+
addr = "http://" + addr
452454
}
453455
p := tea.NewProgram(newTuiModel(addr), tea.WithAltScreen())
454456
if _, err := p.Run(); err != nil {

0 commit comments

Comments
 (0)