@jphutchins/code-review 0.1.0-alpha.67 → 0.1.0-alpha.69
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/README.md +2 -2
- package/dist/index.js +154 -49
- package/dist/index.js.map +1 -1
- package/package.json +1 -1
- package/templates/comment.eta +2 -2
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@jphutchins/code-review",
|
|
3
|
-
"version": "0.1.0-alpha.
|
|
3
|
+
"version": "0.1.0-alpha.69",
|
|
4
4
|
"description": "Deterministic commenter for agentic PR review — gather, render, inline, post, adapt, extract, cost, validate, print-schema",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"license": "MIT",
|
package/templates/comment.eta
CHANGED
|
@@ -168,9 +168,9 @@ _The reviewer rebutted a maintainer's answer, and a maintainer answered again. E
|
|
|
168
168
|
**Findings:** <%~ parts.join(' · ') %>
|
|
169
169
|
<% } else if (!it.incomplete && (it.systemic.length > 0 || it.contested.length > 0)) { %>
|
|
170
170
|
<% const held = [] %><% if (it.contested.length > 0) { held.push(it.contested.length + ' contested item' + (it.contested.length === 1 ? '' : 's')) } %><% if (it.systemic.length > 0) { held.push(it.systemic.length + ' systemic problem' + (it.systemic.length === 1 ? '' : 's')) } %>
|
|
171
|
-
_No findings beyond the <%= held.join(' and ') %> above._
|
|
171
|
+
_No findings beyond the <%= held.join(' and ') %> above<%= it.unfinishedCaveat %>._
|
|
172
172
|
<% } else if (!it.incomplete && !it.answeredReRaiseNote) { %>
|
|
173
|
-
<% if (it.unverifiedNoLogs) { %>_No findings — but no failing-job logs were staged, so this pass could not read the CI failure. "No findings" is not evidence of none._<% } else { %>_No findings — clean review._<% } %>
|
|
173
|
+
<% if (it.unverifiedNoLogs) { %>_No findings — but no failing-job logs were staged, so this pass could not read the CI failure. "No findings" is not evidence of none._<% } else if (it.unfinishedCaveat) { %>_No findings in the checks this round completed<%= it.unfinishedCaveat %>._<% } else { %>_No findings — clean review._<% } %>
|
|
174
174
|
<% } %>
|
|
175
175
|
|
|
176
176
|
<% if (it.inlineDisposition && it.inlineDisposition.kind === 'posted') { %>
|