prreviewbuddy 0.50.1 → 0.53.1
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 +120 -0
- package/dist/{delete_review-BIXqUnpx.js → delete_review-DMJXD3bf.js} +322 -119
- package/dist/main.js +7 -6
- package/dist/server.js +536 -126
- package/package.json +1 -1
- package/static/{chunk-BeL0x-wt.js → chunk-B5CvMZtt.js} +1 -1
- package/static/chunk-BeqgAIrQ.js +1 -1
- package/static/chunk-CofqJ-Qn.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 +237 -75
- package/static/workspace.js +73 -65
- package/static/chunk-vvGSmhmc.js +0 -1
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,125 @@
|
|
|
1
1
|
# Changelog
|
|
2
2
|
|
|
3
|
+
## 0.53.1
|
|
4
|
+
|
|
5
|
+
**The running KISS review has its normal layout back.** While a KISS review ran, its progress
|
|
6
|
+
message sat in a rounded grey pill with the text pressed against its edges. It borrowed the styling
|
|
7
|
+
of the new recommendation status labels, which shared its name.
|
|
8
|
+
|
|
9
|
+
## 0.53.0
|
|
10
|
+
|
|
11
|
+
**KISS recommendations are decisions now.** Each recommendation carries its status and two choices
|
|
12
|
+
worded for whose code it is. Reviewing your own code, you choose **Plan to simplify** or **Keep as
|
|
13
|
+
implemented**. Reviewing someone else's, you choose **Raise with author** or **Leave it**. Decided
|
|
14
|
+
recommendations move into a **Decided** list below the open ones, in the order the analysis ranked
|
|
15
|
+
them. Press the chosen button again to take a decision back. Decisions are kept across reloads, and
|
|
16
|
+
the KISS rail counts what is still undecided. If a decision can't be saved, the card goes back as it
|
|
17
|
+
was and offers **Try again**.
|
|
18
|
+
|
|
19
|
+
- **Copy recommendations** (reviewing someone else's code): copies the recommendations you chose to
|
|
20
|
+
raise as Markdown ready to paste, with each one's title, why it matters, the simpler approach, where
|
|
21
|
+
to start and the draft comment. It is disabled until you choose one, while a decision is still
|
|
22
|
+
being saved, and whenever the branch has moved since the run. If a rerun fails, the earlier review
|
|
23
|
+
stays on the page and is checked against the branch by its own commit, so it is marked stale when
|
|
24
|
+
the branch has moved past it.
|
|
25
|
+
- Each card also has **Copy**, **Ask Buddy**, **Open file** and, where the run wrote one, **Copy
|
|
26
|
+
draft comment**.
|
|
27
|
+
- **Whose code is this?** is now part of running KISS, not part of the result. A finished review
|
|
28
|
+
shows its perspective in the header, beside **Run again**, which opens the choice with the last
|
|
29
|
+
answer already selected. The stale-branch notice offers the same **Run again**, and **Not now**
|
|
30
|
+
now folds that notice to one line instead of removing it.
|
|
31
|
+
- **Complexity worth keeping** is collapsed on the KISS summary. New runs give each item a one-line
|
|
32
|
+
reason beside its title.
|
|
33
|
+
- Fixed: a review finishing in the background could undo something you changed on the page while it
|
|
34
|
+
ran, such as a dismissed notice or an answered question.
|
|
35
|
+
|
|
36
|
+
## 0.52.3
|
|
37
|
+
|
|
38
|
+
**Selected cards keep their rounded corners.** In select mode, a ticked card's tint now fills the
|
|
39
|
+
card's own rounded shape instead of a square behind it.
|
|
40
|
+
|
|
41
|
+
## 0.52.2
|
|
42
|
+
|
|
43
|
+
**Continue reviewing is gone from the Reviews page.** The banner opened whatever was first in your
|
|
44
|
+
review queue, which the Queue view already shows as its first card. The queue itself is unchanged:
|
|
45
|
+
the **Queue** filter lists your queued reviews in order, marking one Reviewed takes it out, and the
|
|
46
|
+
card menus and select mode still add and remove reviews.
|
|
47
|
+
|
|
48
|
+
## 0.52.1
|
|
49
|
+
|
|
50
|
+
**Fetching remote updates moved to where it applies.** A fetch only refreshes reviews that follow a
|
|
51
|
+
remote branch, such as `origin/feature`. Pull request reviews are checked automatically, and a fetch
|
|
52
|
+
cannot move a local branch. So the **Fetch updates** button is gone from the Reviews toolbar, where
|
|
53
|
+
it read as the way the list checks for updates and sat disabled for most people. Instead:
|
|
54
|
+
|
|
55
|
+
- A card that follows a remote branch has **Fetch remote updates** in its `...` menu.
|
|
56
|
+
- A review that follows one has **Fetch remote updates** in its `...` menu, beside Check for
|
|
57
|
+
updates. Check for updates still only reads what is on your machine. After the fetch the review
|
|
58
|
+
checks again, so the freshness banner reflects what came in.
|
|
59
|
+
- In select mode, **Fetch selected** appears only when the selection includes a review that can be
|
|
60
|
+
fetched.
|
|
61
|
+
|
|
62
|
+
What a fetch did, such as "Fetched 1 repository", now appears as a notice in the corner.
|
|
63
|
+
|
|
64
|
+
## 0.52.0
|
|
65
|
+
|
|
66
|
+
**Saved is now Pinned, and pinned reviews have their own section.** The Reviews page lists
|
|
67
|
+
**Pinned**, **Active**, **Waiting** and **Done** in that order, each under a heading with its count.
|
|
68
|
+
Pinning a review moves it into Pinned straight away, and unpinning moves it back. A pinned review
|
|
69
|
+
keeps its lifecycle: a pinned Waiting or Done review says so on its card, and a Waiting one that
|
|
70
|
+
changes says "Changed while waiting" there instead of jumping to Active. Pinning no longer changes
|
|
71
|
+
the order. Every section, Pinned included, is sorted by what last happened to each change.
|
|
72
|
+
|
|
73
|
+
The Status filter no longer has a Saved option, since the Pinned section does that job. A link to
|
|
74
|
+
the old Saved view opens the whole list. On a review's own page, the menu item is now **Pin review**
|
|
75
|
+
or **Unpin review**. Pinned reviews are still kept for ninety days.
|
|
76
|
+
|
|
77
|
+
Under the **Queue** filter the list stays in the order you queued it: a pinned review keeps its
|
|
78
|
+
place in the queue and shows its star. Select mode acts on reviews top to bottom as they are shown,
|
|
79
|
+
so **Add to review queue** queues them in that order. The star cannot be pressed again until the
|
|
80
|
+
last press has been saved, so quick clicks can no longer leave it showing the opposite of what was
|
|
81
|
+
stored.
|
|
82
|
+
|
|
83
|
+
## 0.51.2
|
|
84
|
+
|
|
85
|
+
**The review queue counts what it shows.** When a queued branch was analysed again, the queue kept
|
|
86
|
+
pointing at the older review. The Queue count included it, the Queue view had no card for it,
|
|
87
|
+
Continue reviewing opened the older review, and the card's menu could not take it out. A new review
|
|
88
|
+
of a queued branch now takes that branch's place in the queue. Opening the Reviews page repairs a
|
|
89
|
+
queue that already has older reviews in it, and a branch queued twice keeps its earlier place.
|
|
90
|
+
|
|
91
|
+
## 0.51.1
|
|
92
|
+
|
|
93
|
+
**The workspace fits a phone-width window.** Below about 600px the review's top bar ran off the
|
|
94
|
+
right edge and the whole page scrolled sideways; below about 450px the thin bar above it did too.
|
|
95
|
+
The review bar now wraps onto a second row (what the review is of, then its facts and controls),
|
|
96
|
+
and the thin bar keeps every control and the version by tightening its spacing and showing Send
|
|
97
|
+
feedback as an icon.
|
|
98
|
+
|
|
99
|
+
Buddy AI, opened in a narrow window, now covers exactly the review below the bars instead of
|
|
100
|
+
assuming how tall they are, so it no longer sits over a banner such as the freshness notice. The
|
|
101
|
+
review's `...` menu now opens over the Buddy AI tab rather than under it.
|
|
102
|
+
|
|
103
|
+
## 0.51.0
|
|
104
|
+
|
|
105
|
+
**KISS recommendations read in seconds.** Each card now leads with the point: a sentence or two on
|
|
106
|
+
what the change adds and why the requirements do not need it, then the simpler approach, a short
|
|
107
|
+
line of what acting on it gains, and where to start. The full reasoning (the evidence, why the
|
|
108
|
+
requirements do not call for it, the trade-offs and the other files involved) is one click away
|
|
109
|
+
under **Evidence & trade-offs**. A trade-off that could change your decision is shown on the card
|
|
110
|
+
itself.
|
|
111
|
+
|
|
112
|
+
Titles now say what to do rather than naming the idea, such as "Remove key readback until hardware
|
|
113
|
+
supports it" instead of "Defer device-reported key support".
|
|
114
|
+
|
|
115
|
+
A recommendation is shown only when it comes with a summary, a simpler approach, the evidence and
|
|
116
|
+
reasoning behind it, and a file that exists in the reviewed commit. One that falls short, including
|
|
117
|
+
one that points only at files that do not exist, is no longer shown as a card. The review says how many were held back, both in the workspace and
|
|
118
|
+
in the terminal. If every recommendation came back incomplete, the review says the analysis was
|
|
119
|
+
incomplete rather than calling the change appropriately simple.
|
|
120
|
+
|
|
121
|
+
KISS reviews you already have keep working, in the new layout.
|
|
122
|
+
|
|
3
123
|
## 0.50.1
|
|
4
124
|
|
|
5
125
|
**Copy and Ask Buddy look and behave as a pair.** On every issue and question, both icons are the
|