Include branchless jobs in fix, set branch on analyze (#175)

## Summary

- `roborev fix` was not finding analyze jobs because they had no branch
set and the branch filter excluded them
- `roborev analyze` now sets the branch field when enqueuing jobs
- New `WithBranchOrEmpty` storage option includes jobs with empty/NULL
branch, used only by the fix query path via `branch_include_empty=true`
API param
- `WithBranch` remains strict exact-match for all other callers (TUI,
status, etc.)

## Files changed

| File | What |
|------|------|
| `cmd/roborev/analyze.go` | Set `branch` from `git.GetCurrentBranch()`
in `enqueueAnalysisJob` |
| `cmd/roborev/fix.go` | Pass `branch_include_empty=true` in fix query |
| `internal/storage/jobs.go` | Add `WithBranchOrEmpty` option (matches
branch OR empty/NULL) |
| `internal/daemon/server.go` | Support `branch_include_empty=true`
query param |
| `internal/storage/db_test.go` | `TestWithBranchOrEmpty` — strict
excludes branchless, inclusive includes them |

## Test plan

- [x] `go build ./...`
- [x] `go test ./...` — all pass
- [x] `TestWithBranchOrEmpty` — verifies both strict and inclusive
behavior
- [x] Verified: `fix` with branch=main now returns 19 jobs (was 1) for a
repo with branchless analyze jobs

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

---------

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

Author: wesm

cmd/roborev/analyze.go

@@ -437,9 +437,11 @@ func buildOutputPrefix(analysisType string, filePaths []string) string {
 
 // enqueueAnalysisJob sends a job to the daemon
 func enqueueAnalysisJob(repoRoot, prompt, outputPrefix, label string, opts analyzeOptions) (*storage.ReviewJob, error) {
+	branch := git.GetCurrentBranch(repoRoot)
 	reqBody, _ := json.Marshal(map[string]interface{}{
 		"repo_path":     repoRoot,
 		"git_ref":       label, // Use analysis type name as the TUI label
+		"branch":        branch,
 		"agent":         opts.agentName,
 		"model":         opts.model,
 		"reasoning":     opts.reasoning,

cmd/roborev/fix.go

@@ -373,7 +373,7 @@ func queryUnaddressedJobs(repoRoot, branch string) ([]int64, error) {
 	queryURL := fmt.Sprintf("%s/api/jobs?status=done&repo=%s&addressed=false&limit=0",
 		serverAddr, url.QueryEscape(repoRoot))
 	if branch != "" {
-		queryURL += "&branch=" + url.QueryEscape(branch)
+		queryURL += "&branch=" + url.QueryEscape(branch) + "&branch_include_empty=true"
 	}
 
 	resp, err := http.Get(queryURL)

internal/daemon/server.go

@@ -596,7 +596,11 @@ func (s *Server) handleListJobs(w http.ResponseWriter, r *http.Request) {
 		listOpts = append(listOpts, storage.WithGitRef(gitRef))
 	}
 	if branch := r.URL.Query().Get("branch"); branch != "" {
-		listOpts = append(listOpts, storage.WithBranch(branch))
+		if r.URL.Query().Get("branch_include_empty") == "true" {
+			listOpts = append(listOpts, storage.WithBranchOrEmpty(branch))
+		} else {
+			listOpts = append(listOpts, storage.WithBranch(branch))
+		}
 	}
 	if addrStr := r.URL.Query().Get("addressed"); addrStr == "true" || addrStr == "false" {
 		listOpts = append(listOpts, storage.WithAddressed(addrStr == "true"))

internal/storage/db_test.go

@@ -1759,6 +1759,51 @@ func TestListJobsWithBranchAndAddressedFilters(t *testing.T) {
 	})
 }
 
+func TestWithBranchOrEmpty(t *testing.T) {
+	db := openTestDB(t)
+	defer db.Close()
+
+	repo, err := db.GetOrCreateRepo("/tmp/repo-branch-empty")
+	if err != nil {
+		t.Fatalf("GetOrCreateRepo failed: %v", err)
+	}
+
+	// Create jobs: one on "main", one on "feature", one branchless
+	for i, br := range []string{"main", "feature", ""} {
+		sha := fmt.Sprintf("sha-be-%d", i)
+		commit, err := db.GetOrCreateCommit(repo.ID, sha, "Author", "Subject", time.Now())
+		if err != nil {
+			t.Fatalf("GetOrCreateCommit failed: %v", err)
+		}
+		job, err := db.EnqueueJob(repo.ID, commit.ID, sha, br, "codex", "", "")
+		if err != nil {
+			t.Fatalf("EnqueueJob failed: %v", err)
+		}
+		db.ClaimJob("w")
+		db.CompleteJob(job.ID, "codex", "", fmt.Sprintf("output %d", i))
+	}
+
+	t.Run("WithBranch strict excludes branchless", func(t *testing.T) {
+		jobs, err := db.ListJobs("", "", 50, 0, WithBranch("main"))
+		if err != nil {
+			t.Fatalf("ListJobs failed: %v", err)
+		}
+		if len(jobs) != 1 {
+			t.Errorf("Expected 1 job, got %d", len(jobs))
+		}
+	})
+
+	t.Run("WithBranchOrEmpty includes branchless", func(t *testing.T) {
+		jobs, err := db.ListJobs("", "", 50, 0, WithBranchOrEmpty("main"))
+		if err != nil {
+			t.Fatalf("ListJobs failed: %v", err)
+		}
+		if len(jobs) != 2 {
+			t.Errorf("Expected 2 jobs (main + branchless), got %d", len(jobs))
+		}
+	})
+}
+
 func TestReenqueueJob(t *testing.T) {
 	db := openTestDB(t)
 	defer db.Close()

internal/storage/jobs.go

@@ -1098,21 +1098,31 @@ func (db *DB) GetJobRetryCount(jobID int64) (int, error) {
 type ListJobsOption func(*listJobsOptions)
 
 type listJobsOptions struct {
-	gitRef    string
-	branch    string
-	addressed *bool
+	gitRef             string
+	branch             string
+	branchIncludeEmpty bool
+	addressed          *bool
 }
 
 // WithGitRef filters jobs by git ref.
 func WithGitRef(ref string) ListJobsOption {
 	return func(o *listJobsOptions) { o.gitRef = ref }
 }
 
-// WithBranch filters jobs by branch name.
+// WithBranch filters jobs by exact branch name.
 func WithBranch(branch string) ListJobsOption {
 	return func(o *listJobsOptions) { o.branch = branch }
 }
 
+// WithBranchOrEmpty filters jobs by branch name, also including jobs
+// with no branch set (empty string or NULL).
+func WithBranchOrEmpty(branch string) ListJobsOption {
+	return func(o *listJobsOptions) {
+		o.branch = branch
+		o.branchIncludeEmpty = true
+	}
+}
+
 // WithAddressed filters jobs by addressed state (true/false).
 func WithAddressed(addressed bool) ListJobsOption {
 	return func(o *listJobsOptions) { o.addressed = &addressed }
@@ -1151,7 +1161,11 @@ func (db *DB) ListJobs(statusFilter string, repoFilter string, limit, offset int
 		args = append(args, o.gitRef)
 	}
 	if o.branch != "" {
-		conditions = append(conditions, "j.branch = ?")
+		if o.branchIncludeEmpty {
+			conditions = append(conditions, "(j.branch = ? OR j.branch = '' OR j.branch IS NULL)")
+		} else {
+			conditions = append(conditions, "j.branch = ?")
+		}
 		args = append(args, o.branch)
 	}
 	if o.addressed != nil {