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>

Agents - Fix session switch rendering issue and improve picker behavior (#337776) * Agent Host changes for lszomoru/agents/session-switch-rendering-issue-debug * sessions: keep Changes actions stable across session switches Keep the Changes header through workspace metadata hydration, and wait for the active chat folder to be mounted before rendering resource-scoped Git actions. This avoids a transient Merge dropdown on the Create PR button. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Refactor session diff summary computation to use Session Changeset (#335205) * Agent Host changes for lszomoru/sessions-diff-summary * Fix session summary subscription races and nested isolation Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Align changeset summary integration tests with folder semantics Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Refactor the implementation * Merge branch 'main' into lszomoru/sessions-diff-summary * Configure Git identity for session diff checkpoint tests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add symlink setting for Git extension and Agent Host (#338136)

Agents - Adopt ChangesetState.Recomputing in Agent Host (#337545) * agentHost: adopt recomputing changeset status Return cached file lists with Recomputing while refreshing and keep uncached changesets in Computing. Preserve successful empty results, refresh turn subscriptions without awaiting diffs, and align handshake snapshots with subscriber registration. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agentHost: adapt recomputing changesets to chat scopes Preserve folder-owned Branch Changes and chat-owned Turn and Uncommitted Changes across cold subscription restore. Refresh cached states under their new owners, reattach file monitoring, and keep skipped uncommitted refreshes from leaving a transient status behind. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Agent Host changes for lszomoru/agents/adopting-changesetstate-recomputing * agentHost: preserve pending-worktree changeset diffs Keep the source-folder uncommitted diff available while a worktree is pending, and wait for asynchronous changeset initialization before asserting protocol action delivery and replay in tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agentHost: await recomputing changesets in conformance A cached turn can return a Recomputing subscription snapshot while its new diff is still running. Wait for a completion action newer than that snapshot before asserting Ready, including when the cached result is empty. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agentHost: wait for the final changeset snapshot in conformance A Git-state refresh can restart a cached turn recomputation after its first Ready action. Keep subscribing until the returned snapshot is terminal so the no-op turn assertion observes the final result. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add "Sync Changes" action beside branch picker in agents window (#335080) Agent Host changes for lszomoru/optimistic-gazelle

Agents - Implement diff computation strategy with fallback mechanism (#337595) * agentHost: Add selectable changeset diff strategies Preserve auto as the default while allowing callers to choose Git-only or tracked-edit computation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agentHost: Use tracked edits for session, chat, and turn changes Select fileEditTracker at changeset subscription and lifecycle entry points while preserving auto for other changesets. Only reuse tracker-backed incremental baselines so Git and restored results cannot leak into tracked changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Agent Host changes for lszomoru/agents/diffs-computation-fallback-mechanism * agentHost: Fix changeset fallback ordering Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Agents - Implement read-only multi-file diff editors for edit-tracked changesets (#337692) * Agent Host changes for lszomoru/agents/readonly-multi-file-diff-editors * Refactor agent host session files and tests for improved clarity - Update agentHostSessionChangesets.ts and agentHostSessionFiles.ts - Enhance test coverage in agentHostSessionChangesets.test.ts and agentHostSessionFiles.test.ts - Make minor adjustments in session.ts and its corresponding test file

Implement worktree session diff summary using Branch Changes (#335506) Agent Host changes for lszomoru/agents-diff-summary-take-2