prreviewbuddy 0.53.1 → 0.55.6

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 CHANGED
@@ -1,5 +1,140 @@
1
1
  # Changelog
2
2
 
3
+ ## 0.55.6
4
+
5
+ **File paths are shown whole.** A deep directory above a file name was cut short with an ellipsis,
6
+ and the full path appeared only in a browser tooltip on hover. The directory now wraps onto a
7
+ second line where it needs one, so the whole path is always on screen, and the tooltips are gone.
8
+ This covers file locations on issues and questions, related files, the Guide's file rows, the Files
9
+ tab and KISS locations, in the workspace and the extension. The extension's file list, which had no
10
+ tooltip, no longer hides the end of a long directory either. (Extension 0.25.4.)
11
+
12
+ ## 0.55.5
13
+
14
+ **Why this change? says what is known, plainly.** Each state now has a label anyone can read and a
15
+ colour that says how sure it is: **Author's reason** (green), the author's own words in the PR
16
+ description; **Possible reason** (gold), "The PR description doesn't give a clear reason. Based on
17
+ the change itself, this appears to…"; and **Reason unclear** (grey), "It isn't clear why this
18
+ change is needed.", with **Ask the author** and the question to ask. Colour is only a thin rule
19
+ beside the reason and the label's own colour; the text stays dark, and the label says it all
20
+ without colour. Once that question is marked answered the section reads **Answer available**: "This
21
+ question has been marked answered. Run the review again after adding the answer to the PR
22
+ description."
23
+
24
+ **A refused reason is never shown as nothing.** When the analysis says the author gave a reason but
25
+ the words it quotes can't be confirmed in the PR description, the section used to show nothing at
26
+ all. It now shows Reason unclear, asking the analysis's own question when it gave one, or "What
27
+ problem does this change solve, or what goal does it serve?". That question has the same id on
28
+ every analysis, so marking it answered survives a rerun, even when the analysis happens to ask it
29
+ in the same words. The refused wording is still never shown, and the evidence the refused reason
30
+ came with is not presented as a reason.
31
+
32
+ **What the harness corrected moves into Analysis details.** The amber box above the review, which
33
+ showed on every tab, is gone; its notes are now listed under Analysis details on the Overview. The
34
+ note for a refused reason says only what the check found, since a quote also fails when it is two
35
+ words or is the template's text. The refused quote (cut at 500 characters) and its evidence are
36
+ kept in the review's audit record, or the extension's dev trace.
37
+
38
+ The prompt asks for the author's words exactly as written, backticks and punctuation included, and
39
+ treats a reason found only in commits, review comments or a linked issue as inferred. Under
40
+ Possible reason, the kind of each piece of evidence ("Test", "Code") is now the size of the text
41
+ beside it. (Extension 0.25.3.)
42
+
43
+ ## 0.55.4
44
+
45
+ **Empty pages use the full width.** On the PR context page, the note saying no pull request
46
+ discussion was brought in, and on Diagrams, the card offering to draw a change map or an
47
+ implementation diagram, were held to a narrow column. They now fill the page's width like the
48
+ rest of the workspace.
49
+
50
+ ## 0.55.3
51
+
52
+ **Template comments stay hidden.** HTML comments in a PR description, such as a template's
53
+ `<!-- Describe your change -->`, showed as literal text in Author's context, in both the full view
54
+ and the preview. They are now hidden, as on GitHub. The same goes for comments in review comments
55
+ on the PR context page. A comment inside a code block or written as inline code is still shown,
56
+ since there it is code, and a code fence inside a comment no longer turns the rest of the
57
+ description into code. The preview never shows part of a comment. Model output is unchanged:
58
+ nothing it writes is hidden.
59
+
60
+ ## 0.55.2
61
+
62
+ **A lighter Overview.** Author's context, Why this change? and What's changing no longer read as
63
+ one block:
64
+
65
+ - **Author's context** opens on a short preview of the first thing the author wrote, past any
66
+ template heading such as "Summary": a paragraph's first two sentences (or about 400 characters,
67
+ never cut mid-word), a list's first three items, or "Code block included" for a long code block.
68
+ **View full PR description** shows the whole description, as written. A description that fits
69
+ the preview is shown whole, with no button.
70
+ - **Why this change?** shows the label and the motive. The author's quote behind a stated motive
71
+ is under **Author's words**, one click away, since the motive usually says the same thing.
72
+ - **What's changing** is asked for in about 80 to 120 words, and says what the change does without
73
+ repeating why it exists or judging it. Analyses made before this version keep their summary.
74
+ - More space between the three sections, and more again before the risk verdict and the
75
+ suggested review path.
76
+
77
+ ## 0.55.1
78
+
79
+ **Load more on the Notifications page.** The list shows the newest fifteen, and Load more reveals
80
+ fifteen more at a time, the same way each section of the Reviews page does. Mark all read still
81
+ covers every notification, including the ones not yet shown.
82
+
83
+ ## 0.55.0
84
+
85
+ Published with 0.54.0, which did not go to npm on its own: read both entries below.
86
+
87
+ **Why this change?** The Overview now says why the change exists, between Author's context and
88
+ What's changing, and how it knows:
89
+
90
+ - **Stated by the author**, with a short quote from the description. The quote is checked
91
+ against the description the analysis read, and **From Author's context** takes you to it.
92
+ - **Likely motivation**, when nothing states it but the change gives evidence: a test, an issue,
93
+ a commit message. The evidence is listed, with links to its files.
94
+ - **Motivation unclear**, when knowing why would change how the change should be read. The
95
+ analysis asks the author one question, filed first on the Questions page under **Intent**, with
96
+ the same Mark answered control as every other question.
97
+
98
+ Nothing is shown when the reason would not change the review, or for analyses made before this
99
+ version. A claim that fails its check is never shown in the model's words: an unverified quote or
100
+ an inference without evidence becomes "unclear" with a neutral sentence when the analysis asked a
101
+ question, and is left out when it did not. The workspace says so under **What the harness
102
+ corrected**.
103
+
104
+ - The workspace's prompt now reads the pull request's description. A local branch's commit
105
+ subjects are labelled as commits rather than as a description.
106
+ - Updating an analysis with new commits keeps its motivation, since commits show what changed
107
+ rather than why. The extension 0.25.0 shows the same Overview.
108
+ - Once the Intent question is marked answered, the Overview says **Motivation was unclear when
109
+ analysed** and offers **Run review again**, since marking a question answered records no answer
110
+ for an update to use. A new analysis reads the description and conversation as they now stand.
111
+ The workspace page can now start that analysis itself, as `review --reanalyse` does.
112
+ - Evidence follows one rule in both places. A changed file opens in the diff, and a real unchanged
113
+ file opens in the source view. A path the reviewed commit does not hold is removed and listed
114
+ under **What the harness corrected**. A path that cannot be checked, which is any path outside
115
+ the pull request in the extension, is shown as an unverified path rather than a link.
116
+ - A quote counts as the author's only if it is in the text they wrote. A partly filled template's
117
+ comments, unticked checklist items and bare headings no longer count.
118
+ - Tabbing into a closed Author's context preview opens it, so a focused link or Copy button is
119
+ never hidden under the fade.
120
+
121
+ ## 0.54.0
122
+
123
+ **The Overview starts with the change, then gives the verdict.** The risk card used to come first,
124
+ and its takeaway read like the pull request's description. The Overview now opens with
125
+ **Author's context**, the pull request's description exactly as the author wrote it. Under it is
126
+ **What's changing**, the analysis's summary of what the code does. The risk card, the counts and
127
+ the suggested review path follow, as before.
128
+
129
+ - Author's context appears only when the pull request has a description. A description that is
130
+ only an unfilled template counts as none, and so does a review of a local branch or commit.
131
+ - Each analysis keeps the description it was made against. If the author edits it later, older
132
+ analyses still show what they read, and the next analysis reads the new text.
133
+ - Long descriptions open as a preview with **Show full description**. All of it, checklists and
134
+ testing notes included, is one click away, and **Show less** folds it again.
135
+ - The workspace now fetches the description in the same `gh pr view` call it already made. The
136
+ extension 0.24.0 shows the same Overview.
137
+
3
138
  ## 0.53.1
4
139
 
5
140
  **The running KISS review has its normal layout back.** While a KISS review ran, its progress