pr-review-agent 1.0.0 → 1.1.0

package/agents/pr-reviewer.md

@@ -105,10 +105,21 @@ gh api repos/{OWNER/REPO}/pulls/{PR_NUMBER}/files --paginate --jq '.[] | select(
 
 2. Analyze against REVIEW-PLAN.md checklist categories.
 
-3. For each finding, record:
-   - file path, category, severity (critical/warning/suggestion)
-   - title (short, descriptive), body (detailed explanation with code examples)
-   - line number (approximate)
+3. For each finding, record a JSON object with exactly these fields:
+   - `file` (string): Full file path relative to repo root
+   - `line` (number): Line number where the issue starts (0 if not applicable)
+   - `severity` (string): One of `"critical"`, `"warning"`, `"suggestion"`
+   - `category` (string): Category key matching REVIEW-PLAN.md (e.g. `"architecture"`, `"security"`, `"i18n"`)
+   - `title` (string): Short, descriptive title (under 80 chars)
+   - `body` (string): Detailed explanation of the issue and why it matters. Can include HTML (`<code>`, `<ul>`, `<pre>`, `<strong>`)
+   - `snippet` (string): A concise code snippet showing **what the code looks like now → what it should look like**. Use `→` to separate current from expected. For structural issues, show the problematic pattern. Keep it short (1-5 lines). Example:
+     ```
+     "snippet": "bg-red-900/20 text-red-400 → bg-error-bg text-error"
+     ```
+     ```
+     "snippet": "const repo = new XRepository(handler);\n→ useExecuteUseCase(xUseCase)"
+     ```
+     If no clear code fix exists, show only the problematic code.
 
 **Deduplication rules:**
 - If the same pattern violation appears in 5+ files, consolidate into one finding referencing all files
@@ -119,7 +130,7 @@ gh api repos/{OWNER/REPO}/pulls/{PR_NUMBER}/files --paginate --jq '.[] | select(
 ## Step 3: Generate Review Output
 
 ### 3a. Write findings JSON
-Write the findings array to `$PR_REVIEW_DIR/findings.json`
+Write the findings array to `$PR_REVIEW_DIR/findings.json`. Each finding MUST have all 7 fields: `file`, `line`, `severity`, `category`, `title`, `body`, `snippet`. Never omit `snippet` — every finding needs a code reference showing the issue or the fix.
 
 ### 3b. Update config.json
 Write/update `$PR_REVIEW_DIR/config.json` with PR metadata and category definitions.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-review-agent",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "AI-powered PR review agent for Claude Code, OpenCode, and other AI coding assistants. Analyzes pull requests against project-specific architectural patterns with an interactive preview UI.",
   "bin": {
     "pr-review-agent": "bin/install.js"
@@ -29,4 +29,4 @@
     "type": "git",
     "url": "https://github.com/TheRocketCodeMX/pr-review-agent"
   }
-}
+}

package/template/index.html

@@ -107,6 +107,7 @@
     .comment-body strong { color: var(--text-primary); }
     .comment-body ul { padding-left: 20px; margin: 6px 0; }
     .comment-body li { margin: 3px 0; }
+    .comment-snippet { font-family: var(--font-mono); font-size: 12px; background: var(--bg-primary); border: 1px solid var(--border); border-left: 3px solid var(--accent); border-radius: var(--radius); padding: 10px 14px; margin: 8px 0 0; overflow-x: auto; line-height: 1.6; color: var(--text-secondary); white-space: pre-wrap; word-break: break-word; }
     .line-ref { font-family: var(--font-mono); font-size: 11px; color: var(--text-muted); margin-left: auto; }
 
     /* Card hover actions */
@@ -360,6 +361,10 @@
           <textarea class="edit-textarea" id="comBody" placeholder="Detailed explanation (supports HTML: &lt;code&gt;, &lt;ul&gt;, &lt;pre&gt;, &lt;strong&gt;)" style="font-family:var(--font-mono);font-size:12px;min-height:120px" oninput="this.style.height='auto';this.style.height=this.scrollHeight+'px'"></textarea>
           <div class="edit-preview" id="comBodyPreview" style="display:none"></div>
         </div>
+        <div class="modal-field">
+          <label for="comSnippet">Snippet</label>
+          <textarea class="edit-textarea" id="comSnippet" placeholder="Code snippet: current → expected" style="font-family:var(--font-mono);font-size:12px;min-height:60px" oninput="this.style.height='auto';this.style.height=this.scrollHeight+'px'"></textarea>
+        </div>
       </div>
       <div class="modal-footer">
         <button class="btn" onclick="closeCommentModal()">Cancel</button>
@@ -653,6 +658,7 @@
       document.getElementById('comSeverity').value = 'warning';
       document.getElementById('comTitle').value = '';
       document.getElementById('comBody').value = '';
+      document.getElementById('comSnippet').value = '';
       document.getElementById('comBody').style.display = 'block';
       document.getElementById('comBodyPreview').style.display = 'none';
       // Reset tabs to Write
@@ -686,7 +692,8 @@
         severity: document.getElementById('comSeverity').value,
         category: document.getElementById('comCategory').value,
         title,
-        body: document.getElementById('comBody').value
+        body: document.getElementById('comBody').value,
+        snippet: document.getElementById('comSnippet').value
       });
       closeCommentModal();
       activeFilters = { severity: null, category: null, file: null };
@@ -722,6 +729,7 @@
       reviewData[idx].severity = form.querySelector('[data-field="severity"]').value;
       reviewData[idx].category = form.querySelector('[data-field="category"]').value;
       reviewData[idx].line = parseInt(form.querySelector('[data-field="line"]').value) || 0;
+      reviewData[idx].snippet = form.querySelector('[data-field="snippet"]').value;
       editingIdx = null;
       render();
       await persistToDisk();
@@ -859,6 +867,7 @@
           ${i.line ? `<span class="line-ref">L${i.line}</span>` : ''}
         </div>
         <div class="comment-body">${i.body}</div>
+        ${i.snippet ? `<pre class="comment-snippet">${i.snippet.replace(/</g,'&lt;').replace(/>/g,'&gt;')}</pre>` : ''}
       </div>`;
     }
 
@@ -899,6 +908,7 @@
                 <textarea class="edit-textarea" data-field="body" id="edit-body-${id}" placeholder="Detailed explanation (supports HTML: &lt;code&gt;, &lt;ul&gt;, &lt;pre&gt;, &lt;strong&gt;)" oninput="this.style.height='auto';this.style.height=this.scrollHeight+'px'">${bodyEscaped}</textarea>
                 <div class="edit-preview" id="edit-preview-${id}" style="display:none"></div>
               </div>
+              <div class="edit-field"><label>Snippet</label><textarea class="edit-textarea" data-field="snippet" placeholder="Code snippet: current → expected" style="font-family:var(--font-mono);font-size:12px;min-height:60px" oninput="this.style.height='auto';this.style.height=this.scrollHeight+'px'">${(i.snippet||'').replace(/</g,'&lt;').replace(/>/g,'&gt;')}</textarea></div>
             </div>
           </div>
           <div class="edit-form-footer">