git-reviewer 2.8.0 → 2.8.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/README.md CHANGED
@@ -3,7 +3,7 @@
3
3
  [![CI](https://github.com/dheerajjha/reviewer/actions/workflows/ci.yml/badge.svg)](https://github.com/dheerajjha/reviewer/actions/workflows/ci.yml)
4
4
  [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
5
5
  [![Node](https://img.shields.io/badge/node-%3E%3D18-brightgreen.svg)](package.json)
6
- [![Tests](https://img.shields.io/badge/tests-284-brightgreen.svg)](test/)
6
+ [![Tests](https://img.shields.io/badge/tests-285-brightgreen.svg)](test/)
7
7
  [![Dependencies](https://img.shields.io/badge/dependencies-3-brightgreen.svg)](package.json)
8
8
 
9
9
  When an agent writes the code, reading it becomes the bottleneck — and the tool
@@ -294,7 +294,7 @@ rather than a public issue.
294
294
 
295
295
  ```bash
296
296
  npm install # 3 dependencies, no build step, ~6MB
297
- npm test # 284 tests
297
+ npm test # 285 tests
298
298
  npm run test:watch
299
299
  npm run test:coverage
300
300
  ```
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "git-reviewer",
3
- "version": "2.8.0",
3
+ "version": "2.8.1",
4
4
  "description": "Review your coding agent's changes like a pull request — locally, with inline comments and threaded follow-ups, then hand the marked-up review back to the agent",
5
5
  "author": "Dheeraj Jha",
6
6
  "license": "MIT",
package/public/app.js CHANGED
@@ -222,6 +222,17 @@ async function loadRepo() {
222
222
  }
223
223
 
224
224
  // Display list of changed files
225
+ function fileStatusClass(status) {
226
+ switch (status) {
227
+ case 'A': return 'file-status-added';
228
+ case 'D': return 'file-status-deleted';
229
+ case 'R': return 'file-status-renamed';
230
+ case 'B': return 'file-status-binary';
231
+ case 'M':
232
+ default: return 'file-status-modified';
233
+ }
234
+ }
235
+
225
236
  function displayFiles(files) {
226
237
  const sidebar = document.getElementById('sidebar');
227
238
  const filesList = document.getElementById('filesList');
@@ -244,7 +255,7 @@ function displayFiles(files) {
244
255
  const displayText = path
245
256
  ? `${escapeHtml(path)}/<span class="filename">${escapeHtml(filename)}</span>`
246
257
  : `<span class="filename">${escapeHtml(filename)}</span>`;
247
- const statusClass = status === 'A' ? 'file-status-added' : 'file-status-modified';
258
+ const statusClass = fileStatusClass(status);
248
259
  const statusBadge = `<span class="file-status ${statusClass}">${status}</span>`;
249
260
 
250
261
  // The row used to carry the file name itself, inside a string literal
package/public/style.css CHANGED
@@ -221,6 +221,21 @@ button:disabled {
221
221
  color: white;
222
222
  }
223
223
 
224
+ .file-status-deleted {
225
+ background: #d73a49;
226
+ color: white;
227
+ }
228
+
229
+ .file-status-renamed {
230
+ background: #6f42c1;
231
+ color: white;
232
+ }
233
+
234
+ .file-status-binary {
235
+ background: #6a737d;
236
+ color: white;
237
+ }
238
+
224
239
  .file-item:hover {
225
240
  background: #f3f4f6;
226
241
  border-left-color: #0366d6;