guito 1.0.10 → 1.0.12

package/CHANGELOG.md

@@ -4,6 +4,41 @@ All notable changes to Guito are documented in this file. The project follows [S
 
 ## [Unreleased]
 
+## [1.0.12] - 2026-10-15
+
+### Changed
+
+- Azure DevOps is now detected from the repository's remotes instead of configured. Guito derives the project from `origin` (or the first other remote pointing to Azure DevOps) and creates pull requests against that remote; the settings dialog shows the detected project. The `guito.azureDevOpsUrl` VS Code setting and the Server URL field in the settings dialog are removed.
+
+### Added
+
+- Pull request **Comments** tab now shows code context on inline threads: a focused snippet around the commented line with a **Show code context** loader and an expandable full-file diff, so reviewers see what a comment refers to without leaving the tab.
+- Azure-generated comment activity now reads as plain language: vote entries like "changed status to 10" render as "approved" (with suggestions / waiting for author / rejected mapped the same way) and named status changes use friendly labels like Resolved, with the raw text kept as a tooltip. System entries in an inline thread's diff view render as a quiet one-line note instead of a comment with avatar and markdown, again with the raw text as a tooltip.
+- Filter pull request comments by thread status (with counts), location (General vs Inline), and entry kind (comments vs activity), with a **Clear** reset and a distinct "No comments match these filters" empty state.
+- Mark automated review findings as fixed from the PR dialog's **Review** tab with a per-finding **I fixed it** button. Fixed points move to **Already handled** with a green **Fixed** label and are never raised again by future reviews, like dismissed ones. A **Dim handled** toggle dims the handled section via opacity so pending points stand out; it is on by default and remembered per browser.
+- Decline to review a pull request from the PR dialog's **Review** menu (**Decline to review** / **Resume review**). Reviewers who declined appear in their own **Declined** section of the dialog's reviewer list, and the side panel marks pull requests you declined with a **Declined** badge.
+
+### Fixed
+
+- Right-clicking a branch badge in the commit history no longer offers the wrong actions for local branches whose names contain a slash (e.g. `feature/topic`). Badges are now resolved against the loaded branch list instead of guessing from the name, so **Checkout Branch...** opens the local switch dialog and **Delete Branch...** deletes the local branch instead of failing against a misread remote.
+- Detached `HEAD` and symbolic `origin/HEAD` badges no longer offer **Rename**, **Delete** or **Pull** actions that Git would refuse, and **Unselect in Branches Dropdown** now removes only that branch from the history filter (instead of clearing the whole selection) and stays disabled when the branch is not selected.
+
+## [1.0.11] - 2026-10-12
+
+### Added
+
+- Keyboard shortcuts for the commit history. **Ctrl+F** focuses the commit search box, **Ctrl+J** selects and scrolls to the current branch's commit (or the checked-out commit when HEAD is detached), and **Ctrl+G** opens a **Go to Commit** dialog. Type a row number to jump to that row of the list as currently shown, or a commit hash (at least 4 characters) to jump to that commit. History that isn't loaded yet is fetched as needed, and the dialog explains when a row is out of range, a hash is ambiguous or unknown, or the commit is hidden by the current branch or search filter. On macOS, Cmd works in place of Ctrl. The shortcuts do nothing while a dialog is open.
+- **Code review of uncommitted changes.** A new **Code review** button sits after the stash buttons on the uncommitted changes panel. It has Claude Code review everything not yet committed: staged, unstaged and untracked files. While the review runs, the button turns into **Stop review**. When it finishes, the results open in a new tab: a VS Code editor tab inside the extension, or a browser tab in standalone Guito. Each finding shows its severity and comment and links to the file and line. Its **Auto fix** button has Claude Code fix that one problem in the working tree. Claude Code may only read and edit files; it cannot run commands or use git. The fix shows up as an ordinary uncommitted edit, and **Stop** cancels a running fix. Reviews and fixes use the configured review model, effort, instructions and timeout (`guito.aiReview.*`). The last review is kept per worktree in `.git/guito-working-review.json`.
+
+### Changed
+
+- Ref pills in the commit history now match the graph. Each branch pill takes the color of its graph line; the current branch stays filled, now in its line color instead of a fixed blue. Local and remote branches and tags share one pill style. Remote branches get a cloud icon and tags keep the tag icon, and tags use a soft yellow that no graph line uses. To keep that yellow unique, the graph's two yellowish line colors were replaced with violet and silver. Prefer a quieter look? The new **Gray branch and tag pills** setting (`guito.grayRefBadges` in VS Code, off by default) draws every branch and tag pill in one neutral gray, with the current branch filled.
+- The close buttons on the uncommitted changes panel, commit detail, settings, pull request, diff and merge conflict views, and the error banner's dismiss button now share one look and behavior, matching the commit panel's close button.
+
+### Removed
+
+- Stashes no longer appear in the commit history. The "Show stashes" setting (`guito.showStashes` in VS Code) is gone, and the latest stash commit is no longer drawn in the graph; stashes are listed only in the repository panel.
+
 ## [1.0.10] - 2026-10-10
 
 ### Added

package/README.md

@@ -97,9 +97,9 @@ The settings dialog also manages preferences that have no VS Code equivalent: th
 | -------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------- |
 | `guito.autoReload`               | Reload commits and the working tree automatically when the repository changes outside Guito, for example from a terminal. On by default.                 |
 | `guito.showGraph`                | Show the Git graph beside the commit history. On by default.                                                                                            |
-| `guito.showStashes`              | Show stash entries above the commit history. Off by default.                                                                                            |
 | `guito.showTags`                 | Show tag badges attached to commits. On by default.                                                                                                     |
 | `guito.showRemoteBranches`       | Show remote branches in the repository panel and the branch pills on commits. On by default.                                                            |
+| `guito.grayRefBadges`            | Show branch and tag pills in a single neutral gray instead of graph lane colors. Off by default.                                                        |
 | `guito.fileListView`             | How changed-file lists are shown: `flat` (default) lists every file with its full path, `tree` groups them into collapsible folders.                    |
 | `guito.refListView`              | How branches and tags are shown in the repository panel: `flat` (default) lists full names, `tree` groups them into collapsible namespace folders.      |
 | `guito.sidePanelSectionsExpanded`| Open the repository panel with all sections expanded. On by default.                                                                                    |
@@ -111,9 +111,10 @@ The settings dialog also manages preferences that have no VS Code equivalent: th
 
 ### Azure DevOps
 
+Guito detects Azure DevOps from the repository's remotes: when `origin` (or, failing that, any other remote) has a URL like `https://server/DefaultCollection/Project/_git/Repo`, pull request creation, the pull request panel, and the automated reviewer are enabled for that project. ssh remotes are reached over https on the same host. Requests authenticate with Windows integrated auth.
+
 | Setting                        | Description                                                                                                                                                          |
 | ------------------------------ | -------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| `guito.azureDevOpsUrl`         | Base URL of your on-prem Azure DevOps Server, for example `https://server/DefaultCollection`. Enables pull request creation and the automated reviewer; requests authenticate with Windows integrated auth. |
 | `guito.prBranchNameTemplate`   | Branch name template used when Guito creates a new branch for a pull request. Defaults to `pr/${randomstring}`. Variables: `${username}`, `${randomstring}`, `${branch}`, `${targetbranch}`, `${title}`, `${repository}`, `${date}` (UTC `YYYY-MM-DD`), `${time}` (UTC `HHmmss`), and `${timestamp}`. |
 | `guito.allowMerge`             | Offer the "No fast-forward (merge commit)" and "Semi-linear merge" pull request completion strategies. On by default.                                                |
 

package/bin/ai-review.js

@@ -2,7 +2,7 @@ import { spawn } from 'node:child_process';
 import { createHash } from 'node:crypto';
 import { homedir } from 'node:os';
 import { join } from 'node:path';
-const SEVERITIES = ['blocker', 'concern', 'suggestion', 'nit'];
+export const SEVERITIES = ['blocker', 'concern', 'suggestion', 'nit'];
 /** Effort levels Claude Code accepts for --effort; '' leaves it to Claude Code. */
 export const AI_REVIEW_EFFORTS = ['', 'low', 'medium', 'high', 'xhigh', 'max'];
 /** How much a drafted commit message says beyond its subject line. */
@@ -267,7 +267,7 @@ export function parseModelReview(raw) {
     };
 }
 /** Identity of a finding across re-reviews: same place, same point. */
-function fingerprint(file, title) {
+export function fingerprint(file, title) {
     const normalized = title
         .toLowerCase()
         .replace(/[^a-z0-9]+/g, ' ')
@@ -275,7 +275,7 @@ function fingerprint(file, title) {
     return createHash('sha1').update(`${file}\u0000${normalized}`).digest('hex').slice(0, 16);
 }
 /** Renders one file's diff with the line numbers the model must cite. */
-function renderFileDiff(diff, budget) {
+export function renderFileDiff(diff, budget) {
     const header = `--- FILE: ${diff.path}${diff.oldPath && diff.oldPath !== diff.path ? ` (renamed from ${diff.oldPath})` : ''} [${diff.status}] ---`;
     if (diff.binary)
         return `${header}\n(binary file, not reviewable)\n`;
@@ -320,7 +320,7 @@ function renderThreads(threads) {
     })
         .join('\n');
 }
-const REVIEW_CONTRACT = `Reply with one JSON object and nothing else:
+export const REVIEW_CONTRACT = `Reply with one JSON object and nothing else:
 {"summary": "<one or two sentences on the change as a whole>",
  "findings": [
    {"file": "<path exactly as given in the diff header, or \\"\\" for the change as a whole>",
@@ -346,8 +346,9 @@ export function buildReviewPrompt(input) {
     const incremental = Boolean(sinceCommit);
     const perFile = Math.max(2000, Math.floor(input.maxDiffChars / Math.max(diffs.length, 1)));
     const body = diffs.map((diff) => renderFileDiff(diff, perFile)).join('\n');
-    const raised = previous.filter((finding) => finding.status !== 'dismissed');
+    const raised = previous.filter((finding) => finding.status === 'pending' || finding.status === 'posted');
     const dismissed = previous.filter((finding) => finding.status === 'dismissed');
+    const fixed = previous.filter((finding) => finding.status === 'fixed');
     const sections = [
         'You are reviewing an Azure DevOps pull request as a careful senior engineer on this team.',
         '',
@@ -377,6 +378,13 @@ export function buildReviewPrompt(input) {
                 .join('\n')
             : '  None.',
         '',
+        'POINTS THE USER FIXED (never raise these again):',
+        fixed.length
+            ? fixed
+                .map((finding) => `  - ${finding.file || 'pull request'} — ${finding.title}`)
+                .join('\n')
+            : '  None.',
+        '',
         'EXISTING COMMENT THREADS ON THE PULL REQUEST:',
         renderThreads(threads),
         '',
@@ -755,6 +763,22 @@ export function createAiReviewer(deps, initialConfig = DEFAULT_AI_REVIEW_CONFIG,
                 return next;
             });
         },
+        async fix(pullRequestId, findingIds) {
+            const wanted = new Set(findingIds);
+            return mutateStore((store) => {
+                const current = store.pullRequests[String(pullRequestId)];
+                if (!current)
+                    throw new Error(`Pull request ${pullRequestId} has not been reviewed yet.`);
+                const next = {
+                    ...current,
+                    findings: current.findings.map((finding) => finding.status === 'pending' && wanted.has(finding.id)
+                        ? { ...finding, status: 'fixed', resolvedAt: now().toISOString() }
+                        : finding),
+                };
+                store.pullRequests[String(pullRequestId)] = next;
+                return next;
+            });
+        },
         async forget(pullRequestId) {
             await mutateStore((store) => {
                 delete store.pullRequests[String(pullRequestId)];