Simplify DiffStateModel to have one repo

[MaggieShan]

Description

Part of the work needed to support the code review pane and git states on remote environments includes simplifying the contract between the DiffStateModel and CodeReviewView

• This updates the assumption such that a DiffStateModel is responsible/associated with only one repo
• Removes redundant "change repo" logic

Linked Issue

APP-4353

Testing

Verified existing behaviour for code review view locally

• Tested multiple repos in the same tab
• Tested multiple repos in different tabs
• Tested switching branches

Agent Mode

• Warp Agent Mode - This PR was created via Warp's AI Agent Mode

[oz-for-oss]

@MaggieShan

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR simplifies DiffStateModel so each instance is bound to a single repository and moves repository watcher setup into construction.

Concerns

• WorkingDirectoriesModel::get_or_create_diff_state_model now drops otherwise valid repo paths when DetectedRepositories has not already materialized a watched repository, while callers just return on None; this regresses the previous fallback detection behavior.

Verdict

Found: 0 critical, 1 important, 0 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz