Ladislau Szomoru

Agents - add review/unreview operation to the multi-file diff editor (#324218) * Review changes (P1): add overlayPathIntoTree git primitive Add `IAgentHostGitService.overlayPathIntoTree(base, path, source)` which builds a new tree equal to `base` with a single repo-relative path replaced by (or removed to match) the content in `source`, using a throwaway GIT_INDEX_FILE so the user's real index is untouched. This is the file-level building block for the upcoming reviewed-ref based "review changes" feature: mark reviewed = overlay from the working-tree snapshot; unmark = overlay from the baseline tree. Includes the `parseSingleLsTreeEntry` helper and real-git integration tests covering modified, added, and removed paths, plus the non-git no-op case. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Review changes (P2): extract shared Branch Changes baseline resolver Extract the Branch Changes baseline resolution so the changeset service and the upcoming review service cannot drift on which baseline they use: - Add `IAgentHostGitService.resolveBranchBaselineCommit(workingDirectory, baseBranch?)` which resolves the merge-base commit-ish the Branch Changes diff is anchored on (origin/<base> preference, HEAD fallback, empty-tree fallback). `computeSessionFileDiffs` now delegates to the same private `_resolveBranchMergeBaseCommit` helper -- pure refactor, no behavior change. - Add the pure `resolveDiffBaseBranchName(persisted, gitState)` helper for the base-branch-name precedence and use it from the changeset service. Adds unit tests for both pure helpers and real-git integration tests for `resolveBranchBaselineCommit`; existing `computeSessionFileDiffs` tests still pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Review changes (P3): add reviewed-ref review service core Add `IAgentHostReviewService` and its node implementation, tracking which files in a session's Branch Changes the user has reviewed as a session-private synthetic git ref (`refs/agents/<sid>/reviewed`) whose tree snapshots the reviewed content. - `markFileReviewed` overlays the file's current working-tree content into the reviewed tree; `unmarkFileReviewed` resets it to the baseline content. Each action appends a parent-chained commit (first parent = baseline commit) and advances the ref; no-op mutations are skipped so the chain doesn't grow. - `getReviewedPaths` derives review status from git trees: reviewed = Branch Changes files whose reviewed-tree content matches the working tree. Editing a reviewed file therefore auto-unreviews it. - Mutations/reads are serialized per session; the reviewed ref is deleted on session data disposal. Operations no-op when not in a git work tree (a future milestone adds the non-git DB fallback). Also adds the `diffTreePaths` git primitive used to compute review status, and wires the service into the agent-host node and server entry points. Covered by real-git integration tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Review changes (P4): wire the review operation handler + provider Add `AgentHostReviewFileOperationHandler` (mark/unmark, one instance per direction) and `AgentHostReviewOperationContribution`, advertising the `mark-as-reviewed` / `mark-as-unreviewed` resource-scoped operations for the Branch Changes changeset and delegating to `IAgentHostReviewService`. The handler resolves the session's working directory and base branch (via the shared `resolveDiffBaseBranchName`) so review status is keyed on the same baseline the Branch Changes diff uses. Plumb the review service into `AgentService`'s inner DI (mirroring the checkpoint service, defaulting to `NULL_REVIEW_SERVICE`) and register the contribution. Covered by handler unit tests using a fake review service. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Review changes (P5): surface per-file review status on the Branch changeset Populate `ChangesetFile._meta.reviewed` for the Branch Changes changeset so the client can render reviewed/unreviewed affordances per file: - Inject `IAgentHostReviewService` into the changeset service and, when publishing the Branch changeset, look up the reviewed repo-relative paths and set `_meta.reviewed` per file (keyed by the file id's repo-relative path). - Extract the base-branch resolution into a shared `_resolveBranchBaseBranch` helper reused by the diff computation and the review-status lookup. - After a mark/unmark, the operation handler triggers `refreshBranchChangeset` so the updated `_meta.reviewed` is republished (the working tree is unchanged by a review toggle, so no file-watcher event would otherwise fire). Adds a reusable `createNoopChangesetService` test helper, updates the handler test to assert the refresh, and threads the new dependency through existing agent-host test fakes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Review changes (P6): copy the reviewed ref on session fork Add `IAgentHostReviewService.copyReviewedRef(source, target, workingDirectory)` which points the forked session's reviewed ref at the same commit as the source's, so a fork starts with the parent's review progress. No-op when the source has no reviewed ref or the directory is not a git work tree. Wire it into the Copilot session-fork path (best-effort, after the forked session's database is copied), and inject the review service into CopilotAgent. Covered by a real-git integration test. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * More refactoring/cleanup * More changes to get things working * Fix the actions in the toolbar * Collapse multi-file diff item that is reviewed * Remove precondition * Fix tests + pull request feedback * Fix hygiene * Skip integration tests for now * Make the UI snappier --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Agents - enable switching branches for folder sessions (#334354) * Initial implementation of the operation * Fix condition to check session state and draft usage Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Refactor operation key and lane handling Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

AgentHost - extract changeset subscriptions into a separate service (#321906) * WIP - initial refactoring * More refactoring * Extract changeset subscriptions into a separate service * Pull request feedback

Agent Host - add "Discard Changes" action to the "Uncommitted Changes" changeset (#321594) * WIP - scaffold the discard changes action * Wire up the event * Pull request feedback * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Fix compilation error * Fix integration tests --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

AgentHost - store pull request information on the server (#322607) * Initial implementation * Split summary/session _meta information * Pull request feedback * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

AgentHost - adopt the concept of reviewed from the protocol (#325310) * AgentHost - adopt the concept of reviewed file from the agent host protocol

Agents - limit creating a session from a pull request in the same repository, but ensure that the worktree is set up correctly (#331962) * Agents - limit creating a session from a pull request in the same repository, but ensure that the worktree is set up correctly * Remove tests related to code paths that were removed

AgentHost - add multi-root support for the checkpoint service (#328389) Initial implementation

Agent - show "Changes" tab for new session (#327189) * Show the Changes tab on the new session page * Show the uncommitted changes changeset * Attempt to fix git status integration test * Missed a file

Agents - handle errors during branch checkout (#334452)