Cover workspace and index mutation attribution

[svarlamov]

• Adds end-to-end attribution coverage for add, rm, restore, branch, checkout, and switch modes.
• Extends the integration repository helper with stdin, shell-wrapper, target-repository, and deterministic daemon synchronization support.

[devin-ai-integration]

Devin Review found 1 new potential issue.

View 5 additional findings in Devin Review.

[devin-ai-integration]

🟡 Cleanup of discarded AI edits after a file restore still aborts when the repository's current position cannot be read

The repository's current position is looked up unconditionally (repo.head()?.target()? at src/daemon.rs:1599) even when the position captured at command time is already known, so a restore whose position is known is still abandoned with an error whenever the current position cannot be resolved.

Impact: Discarded AI edits can stay attached to restored files (and the operation reports an error) even though all the information needed to clean them up was already available.

Eager evaluation of the `unwrap_or` fallback defeats the new event-derived head

The PR threads the head captured by the RestorePaths semantic event (src/daemon.rs:8019-8023, produced in src/daemon/analyzers/workspace.rs:30-32) into apply_checkout_switch_working_log_side_effect so the async side effect no longer depends on live repository state. However Option::unwrap_or evaluates its argument eagerly, so repo.head()?.target()? runs on every restore with pathspecs, even when restore_head is Some:

• Any error from repo.head() / Reference::target() (src/git/repository.rs:832-854, which falls back to spawning git rev-parse) propagates out of the function with ?, so the pruning is skipped and a side-effect error is recorded even though the known head made the lookup unnecessary. This is reachable in the intended async scenario: HEAD may have become unborn (e.g. a later checkout --orphan) by the time the restore side effect runs.
• It also performs an avoidable ref read / git rev-parse spawn per restore command.

The pre-existing analogue at src/daemon.rs:7422-7425 has the same shape, but there the value is genuinely needed as a fallback.

Suggested change

	let head = restore_head
	.filter(|head| !head.is_empty())
	.map(ToOwned::to_owned)
	.unwrap_or(repo.head()?.target()?);
	let head = match restore_head.filter(|head| !head.is_empty()) {
	Some(head) => head.to_string(),
	None => repo.head()?.target()?,
	};

---

Was this helpful? React with 👍 or 👎 to provide feedback.