prreviewbuddy 0.73.1 → 0.75.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +95 -0
- package/dist/{delete_review-B0f6rmy_.js → delete_review-BWKGCqwI.js} +6548 -5426
- package/dist/main.js +3 -3
- package/dist/server.js +1018 -175
- package/package.json +1 -1
- package/static/chunk-C0lM8IBt.js +1 -0
- package/static/chunk-egI_P3oB.js +1 -0
- package/static/help.js +1 -1
- package/static/notifications.js +1 -1
- package/static/reviews.js +1 -1
- package/static/settings.js +1 -1
- package/static/shell.css +106 -4
- package/static/sidebar.css +340 -11
- package/static/workspace.js +65 -65
- package/static/chunk-BsU84lX4.js +0 -1
- package/static/chunk-DqEA-C0S.js +0 -1
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,100 @@
|
|
|
1
1
|
# Changelog
|
|
2
2
|
|
|
3
|
+
## 0.75.0
|
|
4
|
+
|
|
5
|
+
**You can change a finding's grade.** An issue's severity and a question's importance are no longer
|
|
6
|
+
fixed by the analysis. Click the grade on a card, pick another, and optionally say why. The card
|
|
7
|
+
shows the grade you hold now and how it moved, for example `major → minor · Reassessed by you`, with
|
|
8
|
+
a History disclosure listing every change, its reason and when it was made. The analysis itself is
|
|
9
|
+
never rewritten, so the original grade is always there to compare against.
|
|
10
|
+
|
|
11
|
+
**Buddy AI can suggest a grade change, and you decide.** Reassess, in a card's Ask Buddy menu, asks
|
|
12
|
+
Buddy AI to judge the finding again. If it thinks the grade should move, the card shows the
|
|
13
|
+
suggestion with its reason and the places it cites, which open their file. Accept it, keep the grade
|
|
14
|
+
you have, or ask more. A suggestion made against an older state of the review, or against a grade
|
|
15
|
+
you have since changed, is shown as out of date rather than applied.
|
|
16
|
+
|
|
17
|
+
**Updates and investigations suggest grade changes instead of making them.** When new commits change
|
|
18
|
+
how serious a finding is, the update suggests the new grade and the update strip says how many it
|
|
19
|
+
suggested. A repository investigation in the extension used to overwrite an issue's severity; it now
|
|
20
|
+
suggests it, and can suggest a question's importance too. A collapsed list of closed findings says
|
|
21
|
+
how many suggestions are waiting in it.
|
|
22
|
+
|
|
23
|
+
**Your grades carry forward.** Prompts for chat, updates and investigations state the grade you hold,
|
|
24
|
+
not the analysis's, so the next round starts from your judgement. Copy includes the change and its
|
|
25
|
+
reason. An update never erases a grade change, and neither does a Look closer run or an
|
|
26
|
+
investigation that finishes while you work.
|
|
27
|
+
|
|
28
|
+
**The extension keeps what you do while a run finishes.** A grade change, dismissal or answer made on
|
|
29
|
+
a review just as an update, investigation or Look closer saves its result is carried to the new
|
|
30
|
+
result, and the sidebar catches up with it. Two Look closer runs finishing together both keep their
|
|
31
|
+
findings. A reply that arrives late never puts an older grade back on screen. (Extension 0.26.0.)
|
|
32
|
+
|
|
33
|
+
## 0.74.1
|
|
34
|
+
|
|
35
|
+
**`review --pr` now works for pull requests from forks.** Before, a pull request whose branch lives on
|
|
36
|
+
the contributor's fork could not be reviewed unless its commit was already in your clone, and the
|
|
37
|
+
error told you to fetch a branch that does not exist on your remote. Now the pull request's own head
|
|
38
|
+
is fetched from the repository it was opened against, so pasting the URL is enough: no fork remote,
|
|
39
|
+
no `gh pr checkout`, no manual fetch. Nothing in your checkout changes, and no branch is created.
|
|
40
|
+
|
|
41
|
+
**It works when your `origin` is your own fork.** The repository the pull request belongs to is found
|
|
42
|
+
among your remotes, or reached by its URL when none of them is it. Only the exact commit GitHub names
|
|
43
|
+
is ever reviewed; a fetch that brings anything else is refused rather than reviewed.
|
|
44
|
+
|
|
45
|
+
**Updating a pull request review follows the pull request, not a branch of the same name.** A fork's
|
|
46
|
+
pull request from its own `main` used to be updated against your repository's `main`. Update now
|
|
47
|
+
asks GitHub where the pull request is and fetches its base too, so a pull request that merged the
|
|
48
|
+
latest base is not shown the base's changes as its own.
|
|
49
|
+
|
|
50
|
+
## 0.74.0
|
|
51
|
+
|
|
52
|
+
**Notifications is now Activity, a history of what happened across your reviews.** Every row says
|
|
53
|
+
more than what kind of event it was. A branch or pull request that moved says where it moved from and
|
|
54
|
+
to, whether it moved locally or on the pull request, and how far behind the review was. A finished
|
|
55
|
+
review says how many issues and unanswered questions it left. An update says what it fixed, what is
|
|
56
|
+
new, what is still open and which questions the new commits answered. A failure says why. Each row
|
|
57
|
+
records what was true when it happened, so an older row never changes its meaning.
|
|
58
|
+
|
|
59
|
+
Each kind of event has its own icon: a branch, a commit, a merge, a check, a refresh, a warning.
|
|
60
|
+
Unread rows keep their dot and tint, so new activity still stands out first. Purple now means only
|
|
61
|
+
unread: pull request merges, closes and reopenings are shown in blue.
|
|
62
|
+
|
|
63
|
+
**A branch that moves twice keeps both moves.** Before, a second move deleted the first row while it
|
|
64
|
+
was still unread, and an update that caught the review up deleted it too. Now every move stays in
|
|
65
|
+
Activity. Only the newest move counts as unread, since that is the only head you can update to, and
|
|
66
|
+
an older one says "replaced by a later move" or "replaced when the review caught up". Neither is
|
|
67
|
+
counted as read: nobody saw them.
|
|
68
|
+
|
|
69
|
+
**PR merge, close and reopen transitions are now recorded in Activity, including changes discovered
|
|
70
|
+
after a review is Done.** A pull request that merged or closed after you marked its review Done, or
|
|
71
|
+
that was noticed by the hourly check or by Check for updates, used to change the card and say
|
|
72
|
+
nothing. It is now in Activity, noting when the review had already been marked Done. A reopened pull
|
|
73
|
+
request is now recorded too. One known gap: a pull request closed, reopened and closed again without
|
|
74
|
+
a new push shows the second close only on its card, not as a second row.
|
|
75
|
+
|
|
76
|
+
**Activity and Settings keep the bell current.** The Activity page adds new rows and quietens rows
|
|
77
|
+
you read elsewhere while it is on screen, and asks again the moment you switch back to it. The bell
|
|
78
|
+
on Settings now updates as well. Toasts stay to one line, with at most one detail.
|
|
79
|
+
|
|
80
|
+
**Activity says how much history it keeps.** It keeps the most recent 200 events for up to thirty
|
|
81
|
+
days. On a busy machine the 200 is reached first, so the page now says both rather than promising
|
|
82
|
+
thirty days.
|
|
83
|
+
|
|
84
|
+
## 0.73.2
|
|
85
|
+
|
|
86
|
+
**Notifications about a review are marked read while that review is on screen.** Until now,
|
|
87
|
+
opening a review marked its notifications read once, when the page loaded, so anything that arrived
|
|
88
|
+
while you stayed on it, such as "Branch updated" after a commit, kept the bell lit long after you
|
|
89
|
+
had seen it. Now a review page marks its own notifications read whenever it is visible: when it
|
|
90
|
+
loads in front of you, when a tab you left in the background becomes visible again, and as new
|
|
91
|
+
ones arrive while you are looking at it. The bell and the rail update at the same moment.
|
|
92
|
+
|
|
93
|
+
A review page that is not visible marks nothing. Opening a review in a background tab, or leaving
|
|
94
|
+
one open behind other tabs, keeps its notifications unread until you switch to it. Notifications
|
|
95
|
+
about other reviews are never marked by a review page. This works on a review opened straight from
|
|
96
|
+
a terminal link, too.
|
|
97
|
+
|
|
3
98
|
## 0.73.1
|
|
4
99
|
|
|
5
100
|
**A review's first update says how long updates take.** While an update runs, the page now
|