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>

Agent Host - adopt changesets breaking changes (#320636) * Bring over the latest version of the protocol * Phase 1 & Phase 2 * Phase 3 * Phase 4 * Phase 5 * Fix diff information in the sessions list * Wire-up the resolving the changesets when the session is active * More fixes following the branch * Attach the changesets for a materialized session * Manually fix agent host protocol file * 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 errors * Skip/fix some tests --------- 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>

Agents - refactor GitHub models and polling logic (#313261) * Add GitHub PR review threads model * Cache GitHub PR review thread models * Use PR review threads model in code review * Keep GitHub PR model refresh lightweight * Complete PR review threads validation checklist * Saving my work * Limit PR review polling to active session * Refactor CodeReviewService to only poll the model for the active session * Add GitHub PR polling lifecycle * Refactor the logic that polls * Use pull request head SHA for CI checks * More fixes in * Fix bug + tests * More test cleanup

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 - make changesets contributable (#316583) * WIP - initial implementation * Move changesets to only be present in ISession * Use factory to create the changesets * More cleanup * Pull request feedback * Remove disposable

Agents - more changesets cleanup (#320831)

Agents - wire up changeset operations (#321777) * WIP - initial scaffolding * WIP - saving my work * Wire up progress indication for operations * Fix tests * Pull request feedback

AgentHost - support `git.worktreeIncludeFiles` setting (#325118) * WIP - initial implementation * Revert changes that are not needed * Refactor the calculation of paths to be copied * Fix test * Add back the test that was removed by mistake

Initial implementation (#314957)

AgentHost - adopt the ChangesetContentChanged action (#323170) * AgentHost - switch over to ChangesetContentChanged action * Revert the change to the tests * Fix integration test

Agents - improve GitHub models polling logic (#313457) Keep GitHub polling refreshes fresh after mutations

AgentHost - refactor git state + changeset refresh (#323320) * WIP - initial implementation * Replace the old method with the new one * More cleanup * Pull request feedback * Adjust code after the merge * Include git state in the session list

AgentHost - do not persist uncommitted changes in the database (#321207) * Phase 1 * Phase 2 * Phase 3 * Phase 4 * Fix tests * Refresh branch changes when git state changes * Pull request feedback

Agents - add checkpoints based changesets for agent host sessions (#319793) * WIP - initial implementation * Refactor things * Use "baseline" as a special token for a turnId in order to be able to compute the "All Changes" changeset * Pull request feedback

AgentHost - add changeset operation to sync changes (#322385) * Refactor git service methods * Add sync changes operation * Pull request feedback * Fix tests

AgentHost - add operations to create pull requests with various auto-merge options (#323081) * AgentHost - add operations to create pull requests with various auto-merge options * 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 hygiene --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

AgentHost - refactor how operations are refreshed (#321837) * Refactor how we track changeset subscriptions * Refactor how we update changeset operations