prreviewbuddy 0.25.8 → 0.27.2
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 +164 -0
- package/dist/{relative_time-D760FOL9.js → delete_review-B3xciJ03.js} +5198 -4577
- package/dist/main.js +914 -308
- package/dist/server.js +647 -104
- package/package.json +1 -1
- package/static/chunk-B80S6zbH.js +4 -4
- package/static/chunk-CIsM3PMS.js +1 -0
- package/static/{chunk-B9H2wUlQ.js → chunk-gBDI1-Cy.js} +1 -1
- package/static/reviews.js +1 -1
- package/static/settings.js +1 -1
- package/static/shell.css +55 -1
- package/static/workspace.js +42 -42
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,169 @@
|
|
|
1
1
|
# Changelog
|
|
2
2
|
|
|
3
|
+
## 0.27.2
|
|
4
|
+
|
|
5
|
+
**Choose the model a review runs under.** `prreviewbuddy review --model <model>` asks the agent for
|
|
6
|
+
a particular model instead of whichever one it normally uses. It works with any agent:
|
|
7
|
+
`--agent codex --model gpt-5.3-codex`, or `--model opus` for Claude Code, with `--lens kiss` as well
|
|
8
|
+
as with the standard review, and with several pull requests at once. The name is passed straight
|
|
9
|
+
through, so anything your agent
|
|
10
|
+
accepts works here, including one released after this version of PR Review Buddy.
|
|
11
|
+
`prreviewbuddy --help` says where to find the names each agent takes, and a name the agent does not
|
|
12
|
+
know is answered by the agent itself, in its own words, along with where to look for one that
|
|
13
|
+
works.
|
|
14
|
+
|
|
15
|
+
**A review keeps the model it was pinned to.** Every agent call a review makes asks for the model
|
|
16
|
+
that review was pinned to: updating it, re-analysing it, running `--lens kiss` on it, asking the
|
|
17
|
+
assistant a question about it, and drawing a diagram of it. Only starting a new analysis changes
|
|
18
|
+
that, with `prreviewbuddy review --reanalyse --model <model>`. A review that never named a model
|
|
19
|
+
stays that way, and will not silently acquire one.
|
|
20
|
+
|
|
21
|
+
**Reviews now say which model made them, and only when they can.** Analysis details record what was
|
|
22
|
+
asked for and, where the agent reports it, what actually answered. Those are shown as separate
|
|
23
|
+
facts, because asking for a model is not proof of having got one: Codex never states which model it
|
|
24
|
+
used, so a Codex review says so rather than repeating the request back as though it were confirmed.
|
|
25
|
+
A review that pinned nothing still reads as it always did, naming just the agent. Those two facts
|
|
26
|
+
describe the run that last worked on the review, so bringing a review up to date re-states them
|
|
27
|
+
from the run that did the updating rather than carrying the earlier analysis's answers forward.
|
|
28
|
+
What the review asked for is not a fact about one run and survives, which is what keeps a pinned
|
|
29
|
+
review pinned.
|
|
30
|
+
|
|
31
|
+
## 0.26.0
|
|
32
|
+
|
|
33
|
+
**Review several pull requests with one command.** `prreviewbuddy review --pr <url> <url> ...`
|
|
34
|
+
starts a review of each one. Every pull request gets an ordinary review of its own, with its own
|
|
35
|
+
link, exactly as if you had named it on its own. Two run at a time by default; `--concurrency <n>`
|
|
36
|
+
changes that. The link to all your reviews is printed as soon as the batch starts, so you can watch
|
|
37
|
+
the reviews appear there while they are being made, and one line per pull request shows the step
|
|
38
|
+
each has reached. At the end every pull request is listed with its own link. A pull request that
|
|
39
|
+
fails does not stop the others, and the command exits with 1 if any of them failed.
|
|
40
|
+
|
|
41
|
+
**Ctrl-C stops a review rather than failing it.** Pressing Ctrl-C during a batch stops the reviews
|
|
42
|
+
that are running and starts no more. A stopped review now says **Review stopped** on its page, with
|
|
43
|
+
a **Carry on** button, and **Analysis stopped** in the list of reviews. A review that genuinely went
|
|
44
|
+
wrong now says **Analysis failed** there instead. Running the same command again carries on from
|
|
45
|
+
where each review got to.
|
|
46
|
+
|
|
47
|
+
**Two pull requests from the same branch name are no longer mixed up.** Pull requests from two forks
|
|
48
|
+
that share a head branch name (such as `patch-1`) used to be handed each other's review. A review now
|
|
49
|
+
records its pull request number and is only handed back for that number. One consequence: a pull
|
|
50
|
+
request reviewed before this version, without its conversation attached, has no recorded number and
|
|
51
|
+
gets a fresh review the next time you ask for it.
|
|
52
|
+
|
|
53
|
+
## 0.25.15
|
|
54
|
+
|
|
55
|
+
**Buddy AI answers one question at a time.** While Buddy AI was answering, a suggested question
|
|
56
|
+
or **Explain in simple terms** still started a second one. The conversation briefly showed two
|
|
57
|
+
questions, and then the page reloaded.
|
|
58
|
+
|
|
59
|
+
Those controls are now greyed out while a question is being answered, and nothing new starts until
|
|
60
|
+
it finishes or you press **Stop**. If you stop a question while something is already in the ask
|
|
61
|
+
field, that draft is kept. Diagrams are unchanged and can still be drawn while a question is being
|
|
62
|
+
answered.
|
|
63
|
+
|
|
64
|
+
After you press **Stop**, you can ask again right away. Before, a question asked in the moment
|
|
65
|
+
after Stop could be refused as a second one while the stopped question was still winding down, and
|
|
66
|
+
the page reloaded. The same applies to stopping a diagram and drawing another.
|
|
67
|
+
|
|
68
|
+
## 0.25.14
|
|
69
|
+
|
|
70
|
+
**A question keeps being answered after you leave the review.** If you asked Buddy AI something
|
|
71
|
+
and then went to the reviews list or opened another review, the answer was thrown away. Coming
|
|
72
|
+
back showed an empty assistant, and your question had gone too. Drawing a diagram had the same
|
|
73
|
+
problem.
|
|
74
|
+
|
|
75
|
+
A question or a diagram now keeps going until it finishes, whichever page you are on. If you come
|
|
76
|
+
back while it is still working, the review shows your question with its progress, and the answer
|
|
77
|
+
appears when it is ready. **Stop** is the only thing that ends one early, and it works from any
|
|
78
|
+
page that has the review open. Anything you did on the review in the meantime, such as marking it
|
|
79
|
+
reviewed, is kept when the answer arrives.
|
|
80
|
+
|
|
81
|
+
## 0.25.13
|
|
82
|
+
|
|
83
|
+
**Two updates of the same review no longer run over each other.** Pressing **Update review** on the
|
|
84
|
+
page while `prreviewbuddy review --update` was working on the same review started a second update
|
|
85
|
+
alongside the first. Whichever finished last replaced the other's result without a word, after
|
|
86
|
+
both had spent minutes of agent time.
|
|
87
|
+
|
|
88
|
+
A second update is now refused while one is running, from wherever it was asked for. On the page,
|
|
89
|
+
the review reloads onto the running update's steps. In the terminal, the command says the review
|
|
90
|
+
is already being updated, gives the link where you can follow it, and exits without changing
|
|
91
|
+
anything. An analysis, a KISS review or a question to the assistant does not block an update.
|
|
92
|
+
|
|
93
|
+
## 0.25.12
|
|
94
|
+
|
|
95
|
+
**An update shows what it is doing, in the terminal and on the review.** `prreviewbuddy review
|
|
96
|
+
--update` printed nothing until the reassessment was over, usually minutes after you typed it, and
|
|
97
|
+
the review page showed nothing either. Only the reviews list noticed, and it called the whole update
|
|
98
|
+
"Preparing isolated checkout".
|
|
99
|
+
|
|
100
|
+
The command now prints the link straight away and then draws the same step list a new review does:
|
|
101
|
+
fetching the latest commits, re-reading the pull request conversation, preparing a checkout of the
|
|
102
|
+
new commits, and reassessing the review, with the agent's own notes under the step that is running.
|
|
103
|
+
Open the link while it works and those steps tick over above the review, which stays readable
|
|
104
|
+
underneath until the update lands. The page refreshes onto the result when it finishes, and the
|
|
105
|
+
reviews list names each step in the same words.
|
|
106
|
+
|
|
107
|
+
An update that finds no new commits stops after the fetch and says so, without ticking steps it
|
|
108
|
+
never ran. **Update review** on the page now shows the same live steps, where it used to show
|
|
109
|
+
captions guessed from a timer.
|
|
110
|
+
|
|
111
|
+
`--reanalyse` kept the terminal silent in the same way, for a whole analysis, and now draws its
|
|
112
|
+
progress too.
|
|
113
|
+
|
|
114
|
+
## 0.25.11
|
|
115
|
+
|
|
116
|
+
**A review with nothing committed to read now says why, straight away.** Running
|
|
117
|
+
`prreviewbuddy review` on `main` with a dozen edited files said "No changes found to review",
|
|
118
|
+
which looked like a bug to anyone who could see the edits. It now says there are no *committed*
|
|
119
|
+
changes, counts the uncommitted ones (new files included), and explains that PR Review Buddy
|
|
120
|
+
reviews a fixed snapshot of your code, so work that is not committed yet is not included. Then
|
|
121
|
+
it tells you what to do: commit on a branch, or just commit if you are already on one.
|
|
122
|
+
|
|
123
|
+
The refusal also comes at once. It used to arrive after an isolated checkout had been prepared,
|
|
124
|
+
reported as a paused review that would refuse again every time you tried to carry it on. Nothing
|
|
125
|
+
is created now. A branch that has since been merged still opens the review it already has.
|
|
126
|
+
|
|
127
|
+
## 0.25.10
|
|
128
|
+
|
|
129
|
+
**The reviews list now shows what is happening, and keeps showing it.** A review being analysed
|
|
130
|
+
right now, one whose analysis failed, and one that was never analysed all looked the same on the
|
|
131
|
+
list: every card said "No standard review yet". Start a review, go back to the list, and you
|
|
132
|
+
watched nothing happen for three minutes.
|
|
133
|
+
|
|
134
|
+
A card now says which step an analysis has reached, in the same words the terminal prints, and says
|
|
135
|
+
when one has stopped. It updates on its own, so a review that finishes while you are looking at the
|
|
136
|
+
list fills in its summary, risk and counts without you touching anything. A review started in
|
|
137
|
+
another terminal appears by itself too.
|
|
138
|
+
|
|
139
|
+
Where a review already has results, they stay put while a new analysis runs over them: a KISS run
|
|
140
|
+
no longer hides the issues you were reading.
|
|
141
|
+
|
|
142
|
+
The list checks in about once a second while something is being made and once every fifteen when
|
|
143
|
+
nothing is, and it waits rather than refreshing under you if you have rows ticked or are partway
|
|
144
|
+
through typing a search.
|
|
145
|
+
|
|
146
|
+
## 0.25.9
|
|
147
|
+
|
|
148
|
+
**Marking a review reviewed shows up straight away on the reviews list.** It used to keep saying
|
|
149
|
+
what it had said before, and catch up some minutes later on its own. Nothing was actually stale:
|
|
150
|
+
the mark is written to disk before the button finishes repainting, and the list is rebuilt from
|
|
151
|
+
those files on every request. The browser simply stopped asking. No page this server sends had
|
|
152
|
+
ever carried a caching rule, so a browser was free to reuse the copy it already had, and pressing
|
|
153
|
+
Back does not reload a page at all: it restores a frozen one from memory, JavaScript variables and
|
|
154
|
+
all, without a single request.
|
|
155
|
+
|
|
156
|
+
Both doors are shut now, and shut in one place rather than per page. Every response describing a
|
|
157
|
+
review, HTML and JSON alike, says it must not be stored, and that is a property of the two helpers
|
|
158
|
+
that write responses rather than something each route remembers. Every page now reloads itself
|
|
159
|
+
when the browser restores it from history, because a restored page may be stale in any of its
|
|
160
|
+
parts and not only the one that prompted this.
|
|
161
|
+
|
|
162
|
+
Assets went the other way, which is the arrangement they should always have had. The bundles whose
|
|
163
|
+
filenames carry a content hash, which are most of what a page downloads, may now be kept
|
|
164
|
+
indefinitely: a new build asks for different names. The handful named plainly still check in every
|
|
165
|
+
time, because those names survive an upgrade.
|
|
166
|
+
|
|
3
167
|
## 0.25.8
|
|
4
168
|
|
|
5
169
|
**A review that fails says so, instead of reporting itself as ready.** An analysis that failed
|