Rename ID to JobID in TUI queue view (#211)

## Summary
- Rename the "ID" column header to "JobID" in the TUI queue view for
clarity
- Set minimum column width to 5 so the header doesn't overflow and
misalign rows
- Fix a `gofmt`-mangled comment in `shellEscape` where `''` was
converted to a curly quote
- Run `gofmt` across the codebase and add workflow guidelines to
CLAUDE.md

closes #209 

<img width="996" height="794" alt="image"
src="https://github.com/user-attachments/assets/cdc74374-abd2-41f8-8c9b-517a560ab877"
/>


## Test plan
- [x] Run `roborev tui` and verify the queue view shows "JobID" column
header aligned with data rows
- [x] Verify `go vet ./...` passes cleanly
- [x] Check `internal/daemon/hooks.go` comment on `shellEscape` reads
correctly

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: wesm

CLAUDE.md

@@ -4,6 +4,18 @@
 
 roborev is an automatic code review daemon for git commits. It runs locally, triggered by post-commit hooks, and uses AI agents (Codex, Claude Code) to review commits in parallel.
 
+## General Workflow
+
+When a task involves multiple steps (e.g., implement + commit + PR), complete ALL steps in sequence without stopping. If creating a branch, committing, and opening a PR, finish the entire chain.
+
+## Go Development
+
+After making any Go code changes, always run `go fmt ./...` and `go vet ./...` before committing. Stage ALL resulting changes, including formatting-only files.
+
+## Git Workflow
+
+Always commit after completing each piece of work — do not wait to be asked. When committing changes, always stage ALL modified files (including formatting, generated files, and ancillary changes). Run `git diff` and `git status` before committing to ensure nothing is left unstaged.
+
 ## Architecture
 
 ```
@@ -81,4 +93,4 @@ CLI reads this to find the daemon. If port 7373 is busy, daemon auto-increments.
 - No emojis in code or output (except commit messages)
 - Never amend commits; always create new commits for fixes
 - Never push or pull unless explicitly asked by the user
-- Never change git branches unless explicitly asked by the user
+- **NEVER change git branches without explicit user confirmation**. Always ask before switching, creating, or checking out branches. This is non-negotiable.

cmd/roborev/analyze.go

@@ -13,12 +13,12 @@ import (
 	"strings"
 	"time"
 
-	"github.com/spf13/cobra"
 	"github.com/roborev-dev/roborev/internal/agent"
 	"github.com/roborev-dev/roborev/internal/config"
 	"github.com/roborev-dev/roborev/internal/git"
 	"github.com/roborev-dev/roborev/internal/prompt/analyze"
 	"github.com/roborev-dev/roborev/internal/storage"
+	"github.com/spf13/cobra"
 )
 
 // Maximum time to wait for an analysis job to complete

cmd/roborev/fix.go

@@ -12,11 +12,11 @@ import (
 	"strings"
 	"time"
 
-	"github.com/spf13/cobra"
 	"github.com/roborev-dev/roborev/internal/agent"
 	"github.com/roborev-dev/roborev/internal/config"
 	"github.com/roborev-dev/roborev/internal/git"
 	"github.com/roborev-dev/roborev/internal/storage"
+	"github.com/spf13/cobra"
 )
 
 func fixCmd() *cobra.Command {

cmd/roborev/fix_test.go

@@ -702,8 +702,8 @@ func (a *fakeAgent) Review(ctx context.Context, repoPath, commitSHA, prompt stri
 	return a.reviewFn(ctx, repoPath, commitSHA, prompt, output)
 }
 func (a *fakeAgent) WithReasoning(level agent.ReasoningLevel) agent.Agent { return a }
-func (a *fakeAgent) WithAgentic(agentic bool) agent.Agent                  { return a }
-func (a *fakeAgent) WithModel(model string) agent.Agent                    { return a }
+func (a *fakeAgent) WithAgentic(agentic bool) agent.Agent                 { return a }
+func (a *fakeAgent) WithModel(model string) agent.Agent                   { return a }
 
 func TestFixJobDirectUnbornHead(t *testing.T) {
 	if _, err := exec.LookPath("git"); err != nil {
@@ -1344,11 +1344,11 @@ func TestTruncateString(t *testing.T) {
 		{"hello", 0, ""},
 		{"hello", -1, ""},
 		// Unicode handling: ensure multi-byte characters aren't split
-		{"こんにちは世界", 5, "こん..."},       // Japanese: maxLen=5, output is 2 chars + "..." = 5 runes
-		{"こんにちは", 10, "こんにちは"},          // Japanese: fits within limit
-		{"Hello 世界!", 8, "Hello..."},   // Mixed ASCII and Unicode
-		{"🎉🎊🎁🎄🎅", 3, "🎉🎊🎁"},            // Emoji: exactly 3 runes
-		{"🎉🎊🎁🎄🎅", 4, "🎉..."},            // Emoji: truncate with ellipsis
+		{"こんにちは世界", 5, "こん..."},      // Japanese: maxLen=5, output is 2 chars + "..." = 5 runes
+		{"こんにちは", 10, "こんにちは"},       // Japanese: fits within limit
+		{"Hello 世界!", 8, "Hello..."}, // Mixed ASCII and Unicode
+		{"🎉🎊🎁🎄🎅", 3, "🎉🎊🎁"},          // Emoji: exactly 3 runes
+		{"🎉🎊🎁🎄🎅", 4, "🎉..."},         // Emoji: truncate with ellipsis
 	}
 
 	for _, tt := range tests {

cmd/roborev/list_test.go

@@ -16,12 +16,12 @@ func TestListCommand(t *testing.T) {
 	finished := now.Add(-5 * time.Second)
 	testJobs := []storage.ReviewJob{
 		{
-			ID:        1,
-			GitRef:    "abc1234567890",
-			RepoName:  "myrepo",
-			Agent:     "test",
-			Status:    storage.JobStatusDone,
-			StartedAt: &started,
+			ID:         1,
+			GitRef:     "abc1234567890",
+			RepoName:   "myrepo",
+			Agent:      "test",
+			Status:     storage.JobStatusDone,
+			StartedAt:  &started,
 			FinishedAt: &finished,
 		},
 		{

cmd/roborev/main.go

@@ -13,16 +13,15 @@ import (
 	"os"
 	"os/exec"
 	"os/signal"
-	"sort"
 	"path/filepath"
 	"runtime"
+	"sort"
 	"strconv"
 	"strings"
 	"syscall"
 	"text/tabwriter"
 	"time"
 
-	"github.com/spf13/cobra"
 	"github.com/roborev-dev/roborev/internal/agent"
 	"github.com/roborev-dev/roborev/internal/config"
 	"github.com/roborev-dev/roborev/internal/daemon"
@@ -32,6 +31,7 @@ import (
 	"github.com/roborev-dev/roborev/internal/storage"
 	"github.com/roborev-dev/roborev/internal/update"
 	"github.com/roborev-dev/roborev/internal/version"
+	"github.com/spf13/cobra"
 )
 
 var (
@@ -1280,7 +1280,7 @@ Examples:
 
 			var jobsResp struct {
 				Jobs    []storage.ReviewJob `json:"jobs"`
-				HasMore bool               `json:"has_more"`
+				HasMore bool                `json:"has_more"`
 			}
 			if err := json.NewDecoder(resp.Body).Decode(&jobsResp); err != nil {
 				return fmt.Errorf("failed to parse response: %w", err)
@@ -2087,9 +2087,9 @@ func skillsCmd() *cobra.Command {
 			fmt.Println("Skills:")
 			for _, s := range available {
 				fmt.Printf("\n  %s\n", s.Name)
-			if s.Description != "" {
-				fmt.Printf("  %s\n", s.Description)
-			}
+				if s.Description != "" {
+					fmt.Printf("  %s\n", s.Description)
+				}
 
 				for _, a := range agents {
 					var as *skills.AgentStatus

cmd/roborev/main_test.go

@@ -23,8 +23,8 @@ import (
 	"time"
 
 	"github.com/roborev-dev/roborev/internal/agent"
-	"github.com/roborev-dev/roborev/internal/git"
 	"github.com/roborev-dev/roborev/internal/daemon"
+	"github.com/roborev-dev/roborev/internal/git"
 	"github.com/roborev-dev/roborev/internal/storage"
 	"github.com/roborev-dev/roborev/internal/version"
 )

cmd/roborev/refine.go

@@ -15,13 +15,13 @@ import (
 	"time"
 
 	"github.com/mattn/go-isatty"
-	"github.com/spf13/cobra"
 	"github.com/roborev-dev/roborev/internal/agent"
 	"github.com/roborev-dev/roborev/internal/config"
 	"github.com/roborev-dev/roborev/internal/daemon"
 	"github.com/roborev-dev/roborev/internal/git"
 	"github.com/roborev-dev/roborev/internal/prompt"
 	"github.com/roborev-dev/roborev/internal/storage"
+	"github.com/spf13/cobra"
 )
 
 // postCommitWaitDelay is the delay after creating a commit before checking

cmd/roborev/refine_test.go

@@ -54,14 +54,14 @@ type mockDaemonClient struct {
 	// Track calls for assertions
 	addressedJobIDs []int64
 	addedComments   []addedComment
-	enqueuedReviews  []enqueuedReview
+	enqueuedReviews []enqueuedReview
 
 	// Auto-incrementing review ID counter for WithReview
 	nextReviewID int64
 
 	// Configurable errors for testing error paths
-	markAddressedErr   error
-	getReviewBySHAErr  error
+	markAddressedErr  error
+	getReviewBySHAErr error
 }
 
 type addedComment struct {
@@ -1082,39 +1082,39 @@ func TestWorktreeCleanupBetweenIterations(t *testing.T) {
 
 func TestResolveReasoningWithFast(t *testing.T) {
 	tests := []struct {
-		name                  string
-		reasoning             string
-		fast                  bool
+		name                   string
+		reasoning              string
+		fast                   bool
 		reasoningExplicitlySet bool
-		want                  string
+		want                   string
 	}{
 		{
-			name:                  "fast flag sets reasoning to fast",
-			reasoning:             "",
-			fast:                  true,
+			name:                   "fast flag sets reasoning to fast",
+			reasoning:              "",
+			fast:                   true,
 			reasoningExplicitlySet: false,
-			want:                  "fast",
+			want:                   "fast",
 		},
 		{
-			name:                  "explicit reasoning takes precedence over fast",
-			reasoning:             "thorough",
-			fast:                  true,
+			name:                   "explicit reasoning takes precedence over fast",
+			reasoning:              "thorough",
+			fast:                   true,
 			reasoningExplicitlySet: true,
-			want:                  "thorough",
+			want:                   "thorough",
 		},
 		{
-			name:                  "no fast flag preserves reasoning",
-			reasoning:             "standard",
-			fast:                  false,
+			name:                   "no fast flag preserves reasoning",
+			reasoning:              "standard",
+			fast:                   false,
 			reasoningExplicitlySet: true,
-			want:                  "standard",
+			want:                   "standard",
 		},
 		{
-			name:                  "no flags returns empty",
-			reasoning:             "",
-			fast:                  false,
+			name:                   "no flags returns empty",
+			reasoning:              "",
+			fast:                   false,
 			reasoningExplicitlySet: false,
-			want:                  "",
+			want:                   "",
 		},
 	}
 

cmd/roborev/repo.go

@@ -8,9 +8,9 @@ import (
 	"strings"
 	"text/tabwriter"
 
-	"github.com/spf13/cobra"
 	"github.com/roborev-dev/roborev/internal/git"
 	"github.com/roborev-dev/roborev/internal/storage"
+	"github.com/spf13/cobra"
 )
 
 // resolveRepoIdentifier resolves a path-like identifier to its git repo root.