Allow multiple clones and fix job_id address API

[wesm]

Summary

This PR fixes two related issues and improves sync compatibility:

1. Allow multiple clones of the same repository (#131)

Previously, registering a second local clone of the same remote repository failed with:

UNIQUE constraint failed: repos.identity (2067)

Fix: Changed repos.identity index from UNIQUE to non-unique. Multiple local clones (e.g., ~/project-main and ~/project-feature) can now coexist with the same identity.

2. Fix roborev address command to use job_id (residual from e956d59)

The address command and API were updated to use job_id instead of review_id, but the daemon client was missed. This caused the refine command and other callers to fail with 400/404 errors.

Fix: Updated daemon.Client.MarkReviewAddressed to send job_id instead of review_id, and updated all callers in refine.go to pass review.JobID.

3. Handle duplicate identities in PostgreSQL sync

With multiple clones now allowed, the sync system needed updates to handle ambiguous identity matches.

Example scenario - Two developers collaborating:

Alice's local SQLite (~/.roborev/roborev.db):

repos table:
| id | root_path              | name   | identity                        |
|----|------------------------|--------|---------------------------------|
| 1  | /home/alice/myapp      | myapp  | git@github.com:org/myapp.git    |

Bob's local SQLite (~/.roborev/roborev.db):

repos table:
| id | root_path              | name   | identity                        |
|----|------------------------|--------|---------------------------------|
| 1  | /home/bob/projects/app | app    | git@github.com:org/myapp.git    |

Shared PostgreSQL (after both sync):

repos table:
| id | root_path              | name   | identity                        | machine_id |
|----|------------------------|--------|---------------------------------|------------|
| 1  | /home/alice/myapp      | myapp  | git@github.com:org/myapp.git    | alice-mac  |
| 2  | /home/bob/projects/app | app    | git@github.com:org/myapp.git    | bob-linux  |

When Alice pulls Bob's jobs from Postgres:

• GetOrCreateRepoByIdentity("git@github.com:org/myapp.git") is called
• Alice has exactly 1 local repo with that identity → returns her repo (id=1)
• Bob's synced jobs appear under Alice's local repo ✓

Example scenario - One developer with multiple clones:

repos table (local SQLite):
| id | root_path                | name         | identity                        |
|----|--------------------------|--------------|--------------------------------|
| 1  | /home/alice/myapp-main   | myapp-main   | git@github.com:org/myapp.git   |
| 2  | /home/alice/myapp-feature| myapp-feature| git@github.com:org/myapp.git   |

When syncing:

• GetOrCreateRepoByIdentity("git@github.com:org/myapp.git") is called
• Alice has 2 local repos with that identity → ambiguous!
• Creates a placeholder repo:

repos table (after sync):
| id | root_path                      | name   | identity                        |
|----|--------------------------------|--------|---------------------------------|
| 1  | /home/alice/myapp-main         | myapp-main   | git@github.com:org/myapp.git |
| 2  | /home/alice/myapp-feature      | myapp-feature| git@github.com:org/myapp.git |
| 3  | git@github.com:org/myapp.git   | myapp        | git@github.com:org/myapp.git |  ← placeholder

Synced jobs attach to the placeholder (id=3), avoiding ambiguity about which clone they belong to.

Selection logic:

1. If exactly 1 local repo has the identity → use it (always preferred)
2. If a placeholder exists → use it
3. If 0 or 2+ local repos → create placeholder

4. Repo display name extraction

Added ExtractRepoNameFromIdentity() to derive human-readable names:

• git@github.com:org/repo.git → repo
• https://github.com/org/my-project.git → my-project

Config override available via sync.repo_names in config.toml.

Test plan

• TestRepoIdentity/multiple_clones_with_same_identity_allowed - verifies duplicate identities work
• TestUniqueIndexMigration - verifies old UNIQUE indexes are migrated
• TestGetOrCreateRepoByIdentity/* - covers all sync selection scenarios
• TestExtractRepoNameFromIdentity - covers name extraction edge cases
• TestSyncConfigGetRepoDisplayName - covers config override functionality
• TestHTTPClientMarkReviewAddressed - verifies job_id is sent
• TestMarkReviewAddressedByJobID - verifies storage method
• All existing tests pass

Commits

1. e6eb6e1 - Allow multiple clones (drop UNIQUE constraint, add migration)
2. c4bbaec - Fix daemon client to send job_id instead of review_id
3. 69d2b39 - Handle duplicate identities in sync with placeholder repos
4. 96adb97 - Address review findings (nil checks, tests, empty identity handling)
5. d3ba7b5 - Prefer single local repo over existing placeholder

🤖 Generated with Claude Code