prreviewbuddy 0.36.10 → 0.42.0

package/CHANGELOG.md

@@ -1,5 +1,145 @@
 # Changelog
 
+## 0.42.0
+
+**A review page no longer says "Review up to date" when it isn't.** The status beside **Check for
+updates** used to be worked out once, when the page opened. It now checks again on its own when you
+come back to the tab and once a minute while you're looking at it, so a commit to the branch you're
+reviewing shows as **Code changed** without pressing anything. These checks never contact GitHub and
+are silent: nothing flashes, and a warning you dismissed stays dismissed until something newer
+arrives.
+
+**Pull request pushes show on the page, not just in notifications.** The workspace server already
+checked your pull requests on GitHub every few minutes and told you when one moved. The page itself
+could only see commits your repository had fetched, so it could say "up to date", and **Check for
+updates** could say **No change**, right after a **PR updated** notification. The page now shows
+**PR updated** as soon as the server has seen the push, with **Update review** beside it, whether or
+not anything has been fetched. The server watches the pull requests of reviews that are queued,
+saved, or not yet marked reviewed, so those are the ones this applies to. Once **Update review** finishes, the page and the card on the
+Reviews page stop saying the pull request moved straight away; an unread **PR updated** notification
+clears at the next check a few minutes later, as before. If the author pushes again while the update
+is running, the page says so rather than hiding the newer push.
+
+## 0.41.0
+
+**Your place is kept when a page reloads itself.** The Reviews page reloads whenever something on it
+changes, such as a review finishing or the review queue changing in another tab, and a review
+reloads when its analysis produces something new. Both used to jump back to the top. They now come
+back where you were. Opening a page from a link still starts at the top, and each view of a review
+and each filter of the list keeps its own place.
+
+**Add a selection to the review queue.** In selection mode, the bar has **Add to review queue**,
+which adds every ticked review in the order the list shows them. Reviews already queued keep their
+place. The queue holds fifty reviews; anything that does not fit stays ticked, and the bar says why.
+Reviews deleted since they were queued no longer take up any of those fifty places.
+
+**Save a review from its own page.** The review page's **...** menu has **Save review**, which does
+what the star on the Reviews page does. When the review is saved it reads **Remove from saved**.
+
+**Copy an issue or a question, ready to post.** Every issue and question has a **Copy** button that
+puts it on the clipboard as Markdown. An issue copies its title in bold, its description, then the
+file and line. A question copies the question in bold, why it matters, then the file. Both list any
+unchanged files they rely on. The severity, confidence and theme labels are left out, so it pastes
+straight into a pull request comment.
+
+## 0.40.0
+
+**See which reviewed pull requests have merged.** The workspace server's regular pull request check
+now also reads whether each pull request is open, merged or closed, from GitHub, in the same request
+it already made. A card whose pull request has merged says **Merged** beside its status, and one
+closed without merging says **PR closed**. Neither changes the review's own status: a review stays
+**In progress** until you mark it reviewed, and a reviewed card keeps its **Merged** badge. When a
+pull request you are watching merges or closes you get one notification, **PR merged** or **PR
+closed**, and never a repeat. Pull requests that had already merged before this version are marked
+quietly, without a notification each. A merged pull request is not checked again.
+
+**A PR filter on the Reviews page.** Beside **Status**, **PR** narrows the list to pull requests that
+are **Open**, **Merged** or **Closed**. It combines with Status, so **In progress** with **Merged**
+lists the merged reviews you have not finished. Reviews with no pull request only appear under
+**Any**.
+
+**Select all in selection mode.** The bar at the bottom has **Select all**, which ticks every review
+the search and filters are showing, on every page. To clear out merged reviews: set PR to Merged,
+press Select, **Select all**, then **Mark reviewed**.
+
+## 0.39.3
+
+**Reviews of a branch now notice when the branch moves, not only reviews of a pull request.** The
+workspace server already checked the pull requests behind your queued, saved and in-progress reviews
+every few minutes. It now checks reviews with no pull request too, by looking at the branch they
+follow in your own repository, with no network access. When you commit to that branch, the review's
+card says **Branch updated** and you get a notification, the same way a moved pull request says
+**PR updated**. The card and the review page's **Check for updates** use the same check, so they
+always agree.
+
+**Fetch updates on the Reviews page.** A review that follows a remote branch, such as
+`origin/feature`, only moves when your repository fetches. **Fetch updates** fetches each repository
+once and then checks those reviews straight away, so a teammate's push shows up without opening each
+review. It covers the reviews the page is showing after search and filters. In selection mode,
+**Fetch selected** in the bar at the bottom covers only the reviews you ticked. Nothing is fetched
+unless you press one of them, reviews of a local branch are never fetched for (a fetch cannot move
+them), and your branches, checkout and working tree are never changed. If one repository cannot be
+reached the others still fetch, and the button says so: **Fetched 2 of 3 repositories**. Branches
+your fetch configuration excludes are left out rather than reported as fetched.
+
+**A tidier Reviews toolbar.** The five status buttons are now one **Status** menu, with the same
+choices and the queue count as **Queue (4)**, so search, filters and Fetch updates fit on one row.
+
+## 0.38.8
+
+**Update review replaces the review instead of running on top of it.** While an update runs, the
+middle of the page shows its progress in place of the findings that are about to be replaced. The
+sidebar is dimmed, and Buddy AI keeps its conversation but waits until the new review is ready. The
+page no longer flashes while an update runs: on reviews without a connected pull request it was
+reloading itself every second. When the update finishes the page switches to the new review once. If
+it fails, the previous review comes back unchanged with the reason.
+
+**Nothing you do during an update is lost any more.** Questions ticked, findings dismissed and Buddy
+AI answers given while an update was running could be overwritten when it finished. They are now kept,
+including when the update runs from the terminal and you are working in the browser at the same time.
+
+**The review header separates status from action.** "Review up to date" is now plain status, with a
+**Check for updates** button beside it. A check that finds new commits brings back a Code changed
+banner you had dismissed.
+
+**The "does not read pull request conversation" note can be dismissed.** Dismissing it hides it for
+that host. The reason stays on the PR context page.
+
+**Update results are easier to scan:** **Review updated · 7bc233f · 2 issues fixed · 1 new issue ·
+2 questions answered**.
+
+## 0.38.2
+
+Two gaps in the per-investigation summary added above: a finding revised a second time was left out
+of the count, so a run that regraded something could report "no findings changed"; and findings an
+earlier investigation had added were counted again by every investigation that followed.
+
+## 0.38.1
+
+A second repository investigation could undo the first one's conclusions. Findings an earlier
+investigation had confirmed or refuted went back to plain "Investigated" whenever the new evidence
+had nothing to say about them, while the sentence explaining the original conclusion stayed on the
+card beneath it. A conclusion now survives an investigation that does not bear on it, and only
+evidence that positively changes it can replace it. Each investigation's summary counts what that
+investigation moved, rather than everything concluded so far.
+
+## 0.38.0
+
+**PR Review Buddy notices when a reviewed pull request moves.** The workspace server checks the
+pull requests behind your queued, saved and in-progress reviews every few minutes, using the `gh`
+you are already signed in to, and records one notification when a request gains commits: **PR
+updated - 3 new commits**, linking to the review, where **Update review** is still yours to press.
+Nothing is re-analysed for you, nothing is reordered, and nothing about the review changes. Reviews
+of a local branch or a single commit are not watched, because there is no pull request to watch.
+
+Also fixed, and older than the above: **updating a review could reopen findings and questions an
+earlier update had already settled.** Each update judged the whole list against only the newest
+commits and was never told what the previous update had concluded, so a fix or an answer from two
+commits ago read as "these changes do not address it". Verdicts now travel forward, a resolved
+finding stays resolved, and an answered question keeps its answer - including on reviews already
+left in that state, whose answers are recovered on their next update.
+
+
 ## 0.36.10
 
 **Syntax highlighting in the file viewer.** Code opened in the review is now coloured by language: