@sentry/warden 0.0.0 → 0.1.0

package/dist/output/renderer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"renderer.js","sourceRoot":"","sources":["../../src/output/renderer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,wBAAwB,EAAE,MAAM,mBAAmB,CAAC;AAG7E,OAAO,EAAE,kBAAkB,EAAE,eAAe,EAAE,SAAS,EAAE,MAAM,6BAA6B,CAAC;AAC7F,OAAO,EAAE,mBAAmB,EAAE,cAAc,EAAE,MAAM,YAAY,CAAC;AACjE,OAAO,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AAE/C,MAAM,cAAc,GAA6B;IAC/C,QAAQ,EAAE,kBAAkB;IAC5B,IAAI,EAAE,WAAW;IACjB,MAAM,EAAE,iBAAiB;IACzB,GAAG,EAAE,qBAAqB;IAC1B,IAAI,EAAE,sBAAsB;CAC7B,CAAC;AAEF,MAAM,UAAU,iBAAiB,CAAC,MAAmB,EAAE,UAAyB,EAAE;IAChF,MAAM,EAAE,kBAAkB,GAAG,IAAI,EAAE,WAAW,EAAE,WAAW,GAAG,IAAI,EAAE,SAAS,EAAE,WAAW,EAAE,aAAa,EAAE,GAAG,OAAO,CAAC;IAEtH,oEAAoE;IACpE,MAAM,gBAAgB,GAAG,wBAAwB,CAAC,MAAM,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC;IAC9E,MAAM,QAAQ,GAAG,WAAW,CAAC,CAAC,CAAC,gBAAgB,CAAC,KAAK,CAAC,CAAC,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC,gBAAgB,CAAC;IACzF,MAAM,cAAc,GAAG,CAAC,GAAG,QAAQ,CAAC,CAAC,IAAI,CACvC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,cAAc,CAAC,CAAC,CAAC,QAAQ,CAAC,CAClE,CAAC;IAEF,gDAAgD;IAChD,MAAM,KAAK,GAAG,aAAa,IAAI,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC;IACtD,MAAM,WAAW,GAAG,KAAK,GAAG,cAAc,CAAC,MAAM,CAAC;IAElD,MAAM,MAAM,GAAG,YAAY,CAAC,cAAc,EAAE,MAAM,EAAE,kBAAkB,CAAC,CAAC;IACxE,MAAM,cAAc,GAAG,oBAAoB,CAAC,MAAM,EAAE,cAAc,EAAE,WAAW,EAAE,WAAW,EAAE,WAAW,CAAC,CAAC;IAE3G,OAAO,EAAE,MAAM,EAAE,cAAc,EAAE,CAAC;AACpC,CAAC;AAED,SAAS,YAAY,CACnB,QAAmB,EACnB,MAAmB,EACnB,kBAA2B;IAE3B,MAAM,oBAAoB,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;IAEhE,IAAI,oBAAoB,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACtC,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,MAAM,QAAQ,GAAoB,oBAAoB,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE;QACrE,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC;QAClC,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,uDAAuD,CAAC,CAAC;QAC3E,CAAC;QACD,MAAM,cAAc,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC,KAAK,OAAO,CAAC,UAAU,cAAc,CAAC,CAAC,CAAC,EAAE,CAAC;QACvF,IAAI,IAAI,GAAG,KAAK,cAAc,CAAC,OAAO,CAAC,QAAQ,CAAC,IAAI,UAAU,CAAC,OAAO,CAAC,KAAK,CAAC,KAAK,cAAc,OAAO,UAAU,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAC;QAEzI,IAAI,kBAAkB,IAAI,OAAO,CAAC,YAAY,EAAE,CAAC;YAC/C,IAAI,IAAI,OAAO,gBAAgB,CAAC,OAAO,CAAC,YAAY,CAAC,WAAW,EAAE,OAAO,CAAC,YAAY,CAAC,IAAI,CAAC,EAAE,CAAC;QACjG,CAAC;QAED,2BAA2B;QAC3B,IAAI,IAAI,yBAAyB,MAAM,CAAC,KAAK,QAAQ,CAAC;QAEtD,2BAA2B;QAC3B,MAAM,WAAW,GAAG,mBAAmB,CAAC,OAAO,CAAC,KAAK,EAAE,OAAO,CAAC,WAAW,CAAC,CAAC;QAC5E,MAAM,IAAI,GAAG,QAAQ,CAAC,OAAO,IAAI,QAAQ,CAAC,SAAS,CAAC;QACpD,IAAI,IAAI,KAAK,cAAc,CAAC,QAAQ,CAAC,IAAI,EAAE,IAAI,EAAE,WAAW,CAAC,EAAE,CAAC;QAEhE,MAAM,WAAW,GAAG,QAAQ,CAAC,OAAO,IAAI,QAAQ,CAAC,SAAS,KAAK,QAAQ,CAAC,OAAO,CAAC;QAEhF,OAAO;YACL,IAAI;YACJ,IAAI,EAAE,QAAQ,CAAC,IAAI;YACnB,IAAI,EAAE,QAAQ,CAAC,OAAO,IAAI,QAAQ,CAAC,SAAS;YAC5C,IAAI,EAAE,OAAgB;YACtB,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS;YACxD,UAAU,EAAE,WAAW,CAAC,CAAC,CAAE,OAAiB,CAAC,CAAC,CAAC,SAAS;SACzD,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,MAAM,mBAAmB,GAAG,QAAQ,CAAC,IAAI,CACvC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,UAAU,IAAI,CAAC,CAAC,QAAQ,KAAK,MAAM,CAC1D,CAAC;IACF,MAAM,KAAK,GAA0B,mBAAmB,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,CAAC,SAAS,CAAC;IAEzF,OAAO;QACL,KAAK;QACL,IAAI,EAAE,EAAE;QACR,QAAQ;KACT,CAAC;AACJ,CAAC;AAED,SAAS,gBAAgB,CAAC,WAAmB,EAAE,IAAY;IACzD,MAAM,eAAe,GAAG,IAAI;SACzB,KAAK,CAAC,IAAI,CAAC;SACX,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC;SACjE,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;IAEhC,IAAI,eAAe,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACjC,OAAO,sBAAsB,UAAU,CAAC,WAAW,CAAC,EAAE,CAAC;IACzD,CAAC;IAED,OAAO,sBAAsB,UAAU,CAAC,WAAW,CAAC,yBAAyB,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC;AACpH,CAAC;AAED,SAAS,wBAAwB,CAAC,WAAmB,EAAE,WAAmB;IACxE,OAAO,SAAS,WAAW,eAAe,SAAS,CAAC,WAAW,EAAE,SAAS,CAAC,eAAe,WAAW,GAAG,CAAC;AAC3G,CAAC;AAED,SAAS,oBAAoB,CAC3B,MAAmB,EACnB,QAAmB,EACnB,WAAoB,EACpB,WAAoB,EACpB,WAAoB;IAEpB,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,CAAC,IAAI,CAAC,MAAM,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;IACjC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC;IACvC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC1B,KAAK,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;QACrC,uDAAuD;QACvD,IAAI,WAAW,IAAI,WAAW,GAAG,CAAC,IAAI,WAAW,EAAE,CAAC;YAClD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YACf,KAAK,CAAC,IAAI,CAAC,wBAAwB,CAAC,WAAW,EAAE,WAAW,CAAC,CAAC,CAAC;QACjE,CAAC;QACD,mDAAmD;QACnD,MAAM,SAAS,GAAG,kBAAkB,CAAC,MAAM,CAAC,UAAU,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;QACtE,IAAI,SAAS,EAAE,CAAC;YACd,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,KAAK,EAAE,QAAQ,SAAS,QAAQ,CAAC,CAAC;QACnD,CAAC;QACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1B,CAAC;IAED,MAAM,MAAM,GAAG,eAAe,CAAC,QAAQ,CAAC,CAAC;IACzC,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IAC1B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CACR;;EAEF,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC;SACrB,MAAM,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,CAAC;SAChC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,cAAc,CAAC,CAAa,CAAC,GAAG,cAAc,CAAC,CAAa,CAAC,CAAC;SACjF,GAAG,CAAC,CAAC,CAAC,QAAQ,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,KAAK,cAAc,CAAC,QAAoB,CAAC,IAAI,QAAQ,MAAM,KAAK,IAAI,CAAC;SAChG,IAAI,CAAC,IAAI,CAAC,EAAE,CACZ,CAAC;IACF,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;IAC3B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,IAAI,WAAW,EAAE,CAAC;QAChB,MAAM,MAAM,GAAG,mBAAmB,CAAC,QAAQ,CAAC,CAAC;QAC7C,KAAK,MAAM,CAAC,IAAI,EAAE,YAAY,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC;YAC1D,KAAK,CAAC,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,CAAC;YAC/B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YACf,KAAK,MAAM,OAAO,IAAI,YAAY,EAAE,CAAC;gBACnC,KAAK,CAAC,IAAI,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAC,CAAC;YACzC,CAAC;YACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjB,CAAC;QAED,MAAM,UAAU,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC;QACvD,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC1B,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;YAC3B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YACf,KAAK,MAAM,OAAO,IAAI,UAAU,EAAE,CAAC;gBACjC,KAAK,CAAC,IAAI,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAC,CAAC;YACzC,CAAC;QACH,CAAC;IACH,CAAC;SAAM,CAAC;QACN,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,KAAK,CAAC,IAAI,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAC,CAAC;QACzC,CAAC;IACH,CAAC;IAED,uDAAuD;IACvD,IAAI,WAAW,IAAI,WAAW,GAAG,CAAC,IAAI,WAAW,EAAE,CAAC;QAClD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,wBAAwB,CAAC,WAAW,EAAE,WAAW,CAAC,CAAC,CAAC;IACjE,CAAC;IAED,mBAAmB;IACnB,MAAM,SAAS,GAAG,kBAAkB,CAAC,MAAM,CAAC,UAAU,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;IACtE,IAAI,SAAS,EAAE,CAAC;QACd,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,KAAK,EAAE,QAAQ,SAAS,QAAQ,CAAC,CAAC;IACnD,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED,SAAS,eAAe,CAAC,GAA4C;IACnE,IAAI,GAAG,CAAC,OAAO,EAAE,CAAC;QAChB,OAAO,IAAI,GAAG,CAAC,SAAS,IAAI,GAAG,CAAC,OAAO,EAAE,CAAC;IAC5C,CAAC;IACD,OAAO,IAAI,GAAG,CAAC,SAAS,EAAE,CAAC;AAC7B,CAAC;AAED,SAAS,iBAAiB,CAAC,OAAgB;IACzC,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC,CAAC,CAAC,KAAK,eAAe,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;IACnF,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC,KAAK,OAAO,CAAC,UAAU,cAAc,CAAC,CAAC,CAAC,EAAE,CAAC;IACnF,OAAO,KAAK,cAAc,CAAC,OAAO,CAAC,QAAQ,CAAC,MAAM,UAAU,CAAC,OAAO,CAAC,KAAK,CAAC,KAAK,QAAQ,GAAG,UAAU,KAAK,UAAU,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE,CAAC;AAC9I,CAAC;AAED,SAAS,mBAAmB,CAAC,QAAmB;IAC9C,MAAM,MAAM,GAA8B,EAAE,CAAC;IAC7C,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,IAAI,OAAO,CAAC,QAAQ,EAAE,CAAC;YACrB,MAAM,IAAI,GAAG,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAC;YACnC,MAAM,CAAC,IAAI,CAAC,KAAK,EAAE,CAAC;YACpB,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAC7B,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}

package/dist/output/renderer.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=renderer.test.d.ts.map

package/dist/output/renderer.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"renderer.test.d.ts","sourceRoot":"","sources":["../../src/output/renderer.test.ts"],"names":[],"mappings":""}

package/dist/output/renderer.test.js

@@ -0,0 +1,645 @@
+import { describe, it, expect } from 'vitest';
+import { renderSkillReport } from './renderer.js';
+import { parseMarker } from './dedup.js';
+describe('renderSkillReport', () => {
+    const baseReport = {
+        skill: 'security-review',
+        summary: 'Found 2 potential issues',
+        findings: [],
+    };
+    it('renders empty findings report', () => {
+        const result = renderSkillReport(baseReport);
+        expect(result.review).toBeUndefined();
+        expect(result.summaryComment).toContain('security-review');
+        expect(result.summaryComment).toContain('No findings to report');
+    });
+    it('renders findings with inline comments', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'sql-injection-1',
+                    severity: 'critical',
+                    title: 'SQL Injection',
+                    description: 'User input passed directly to query',
+                    location: {
+                        path: 'src/db.ts',
+                        startLine: 42,
+                        endLine: 45,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        expect(result.review).toBeDefined();
+        const review = result.review;
+        expect(review.event).toBe('REQUEST_CHANGES');
+        expect(review.comments).toHaveLength(1);
+        expect(review.comments[0].path).toBe('src/db.ts');
+        expect(review.comments[0].line).toBe(45);
+        expect(review.comments[0].body).toContain('SQL Injection');
+    });
+    it('includes skill attribution footnote in comments', () => {
+        const report = {
+            ...baseReport,
+            skill: 'code-review',
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'medium',
+                    title: 'Issue',
+                    description: 'Details',
+                    location: {
+                        path: 'src/file.ts',
+                        startLine: 10,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        expect(result.review).toBeDefined();
+        expect(result.review.comments[0].body).toContain('<sub>warden: code-review</sub>');
+    });
+    it('includes deduplication marker in comments', () => {
+        const report = {
+            ...baseReport,
+            skill: 'security-review',
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'high',
+                    title: 'SQL Injection',
+                    description: 'User input passed to query',
+                    location: {
+                        path: 'src/db.ts',
+                        startLine: 42,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        expect(result.review).toBeDefined();
+        const body = result.review.comments[0].body;
+        // Verify marker is present and parseable
+        const marker = parseMarker(body);
+        expect(marker).not.toBeNull();
+        expect(marker.path).toBe('src/db.ts');
+        expect(marker.line).toBe(42);
+        expect(marker.contentHash).toMatch(/^[a-f0-9]{8}$/);
+    });
+    it('sets start_line for multi-line findings', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'multi-line-1',
+                    severity: 'medium',
+                    title: 'Multi-line issue',
+                    description: 'Spans multiple lines',
+                    location: {
+                        path: 'src/code.ts',
+                        startLine: 10,
+                        endLine: 15,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        const comment = result.review.comments[0];
+        expect(comment.line).toBe(15);
+        expect(comment.start_line).toBe(10);
+        expect(comment.start_side).toBe('RIGHT');
+    });
+    it('does not set start_line for single-line findings', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'single-line-1',
+                    severity: 'medium',
+                    title: 'Single-line issue',
+                    description: 'On one line',
+                    location: {
+                        path: 'src/code.ts',
+                        startLine: 25,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        const comment = result.review.comments[0];
+        expect(comment.line).toBe(25);
+        expect(comment.start_line).toBeUndefined();
+        expect(comment.start_side).toBeUndefined();
+    });
+    it('does not set start_line when startLine equals endLine', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'same-line-1',
+                    severity: 'medium',
+                    title: 'Same line issue',
+                    description: 'Start and end are same',
+                    location: {
+                        path: 'src/code.ts',
+                        startLine: 30,
+                        endLine: 30,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        const comment = result.review.comments[0];
+        expect(comment.line).toBe(30);
+        expect(comment.start_line).toBeUndefined();
+        expect(comment.start_side).toBeUndefined();
+    });
+    it('renders suggested fixes as GitHub suggestions', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'fix-1',
+                    severity: 'medium',
+                    title: 'Use parameterized query',
+                    description: 'Replace string concatenation with parameters',
+                    location: {
+                        path: 'src/db.ts',
+                        startLine: 10,
+                    },
+                    suggestedFix: {
+                        description: 'Use prepared statement',
+                        diff: `--- a/src/db.ts
++++ b/src/db.ts
+@@ -10,1 +10,1 @@
+-const query = "SELECT * FROM users WHERE id = " + id;
++const query = "SELECT * FROM users WHERE id = ?";`,
+                    },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        const review = result.review;
+        expect(review.comments[0].body).toContain('```suggestion');
+        expect(review.comments[0].body).toContain('const query = "SELECT * FROM users WHERE id = ?";');
+    });
+    it('groups findings by file in summary', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'medium',
+                    title: 'Issue A',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 10 },
+                },
+                {
+                    id: 'f2',
+                    severity: 'low',
+                    title: 'Issue B',
+                    description: 'Details',
+                    location: { path: 'src/b.ts', startLine: 20 },
+                },
+                {
+                    id: 'f3',
+                    severity: 'info',
+                    title: 'Issue C',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 30 },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        expect(result.summaryComment).toContain('`src/a.ts`');
+        expect(result.summaryComment).toContain('`src/b.ts`');
+    });
+    it('sorts findings by severity', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'low',
+                    title: 'Low Issue',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 10 },
+                },
+                {
+                    id: 'f2',
+                    severity: 'critical',
+                    title: 'Critical Issue',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 20 },
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        const review = result.review;
+        expect(review.comments[0].body).toContain('Critical Issue');
+    });
+    it('requests changes for critical/high severity', () => {
+        const criticalReport = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'critical',
+                    title: 'Critical',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 1 },
+                },
+            ],
+        };
+        const highReport = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'high',
+                    title: 'High',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 1 },
+                },
+            ],
+        };
+        const mediumReport = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'medium',
+                    title: 'Medium',
+                    description: 'Details',
+                    location: { path: 'src/a.ts', startLine: 1 },
+                },
+            ],
+        };
+        const criticalResult = renderSkillReport(criticalReport);
+        const highResult = renderSkillReport(highReport);
+        const mediumResult = renderSkillReport(mediumReport);
+        expect(criticalResult.review.event).toBe('REQUEST_CHANGES');
+        expect(highResult.review.event).toBe('REQUEST_CHANGES');
+        expect(mediumResult.review.event).toBe('COMMENT');
+    });
+    it('respects maxFindings option', () => {
+        const report = {
+            ...baseReport,
+            findings: Array.from({ length: 10 }, (_, i) => ({
+                id: `f${i}`,
+                severity: 'info',
+                title: `Finding ${i}`,
+                description: 'Details',
+                location: { path: 'src/a.ts', startLine: i + 1 },
+            })),
+        };
+        const result = renderSkillReport(report, { maxFindings: 3 });
+        expect(result.review.comments).toHaveLength(3);
+    });
+    it('handles findings without location', () => {
+        const report = {
+            ...baseReport,
+            findings: [
+                {
+                    id: 'f1',
+                    severity: 'medium',
+                    title: 'General Issue',
+                    description: 'Applies to whole project',
+                },
+            ],
+        };
+        const result = renderSkillReport(report);
+        expect(result.review).toBeUndefined();
+        expect(result.summaryComment).toContain('General Issue');
+        expect(result.summaryComment).toContain('General');
+    });
+    describe('commentOn filtering', () => {
+        it('filters findings by commentOn threshold', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'critical',
+                        title: 'Critical Issue',
+                        description: 'Critical details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                    {
+                        id: 'f2',
+                        severity: 'high',
+                        title: 'High Issue',
+                        description: 'High details',
+                        location: { path: 'src/a.ts', startLine: 20 },
+                    },
+                    {
+                        id: 'f3',
+                        severity: 'medium',
+                        title: 'Medium Issue',
+                        description: 'Medium details',
+                        location: { path: 'src/a.ts', startLine: 30 },
+                    },
+                    {
+                        id: 'f4',
+                        severity: 'low',
+                        title: 'Low Issue',
+                        description: 'Low details',
+                        location: { path: 'src/a.ts', startLine: 40 },
+                    },
+                ],
+            };
+            // commentOn='high' should only include critical and high
+            const result = renderSkillReport(report, { commentOn: 'high' });
+            expect(result.review).toBeDefined();
+            expect(result.review.comments).toHaveLength(2);
+            expect(result.review.comments.map((c) => c.body)).toEqual([
+                expect.stringContaining('Critical Issue'),
+                expect.stringContaining('High Issue'),
+            ]);
+        });
+        it('shows all findings when commentOn is not specified', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'critical',
+                        title: 'Critical Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                    {
+                        id: 'f2',
+                        severity: 'info',
+                        title: 'Info Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 20 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report);
+            expect(result.review.comments).toHaveLength(2);
+        });
+        it('returns empty review when all findings are filtered out', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'low',
+                        title: 'Low Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                    {
+                        id: 'f2',
+                        severity: 'info',
+                        title: 'Info Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 20 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report, { commentOn: 'high' });
+            expect(result.review).toBeUndefined();
+            expect(result.summaryComment).toContain('No findings to report');
+        });
+        it('applies commentOn filter before maxFindings limit', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'critical',
+                        title: 'Critical Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                    {
+                        id: 'f2',
+                        severity: 'low',
+                        title: 'Low Issue 1',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 20 },
+                    },
+                    {
+                        id: 'f3',
+                        severity: 'low',
+                        title: 'Low Issue 2',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 30 },
+                    },
+                ],
+            };
+            // With commentOn='high' and maxFindings=2, should only show critical (1 finding)
+            // because low findings are filtered out first
+            const result = renderSkillReport(report, { commentOn: 'high', maxFindings: 2 });
+            expect(result.review.comments).toHaveLength(1);
+            expect(result.review.comments[0].body).toContain('Critical Issue');
+        });
+    });
+    describe('stats footer', () => {
+        it('includes stats footer in summary comment when stats are available', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'medium',
+                        title: 'Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                ],
+                durationMs: 15800,
+                usage: {
+                    inputTokens: 3000,
+                    outputTokens: 680,
+                    costUSD: 0.0048,
+                },
+            };
+            const result = renderSkillReport(report);
+            expect(result.summaryComment).toContain('⏱ 15.8s');
+            expect(result.summaryComment).toContain('3.0k in / 680 out');
+            expect(result.summaryComment).toContain('$0.0048');
+            expect(result.summaryComment).toContain('<sub>');
+        });
+        it('review body is empty (stats only in summary comment)', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'medium',
+                        title: 'Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                ],
+                durationMs: 12300,
+                usage: {
+                    inputTokens: 2500,
+                    outputTokens: 450,
+                    costUSD: 0.0032,
+                },
+            };
+            const result = renderSkillReport(report);
+            expect(result.review).toBeDefined();
+            expect(result.review.body).toBe('');
+        });
+        it('includes stats footer in empty findings report', () => {
+            const report = {
+                ...baseReport,
+                findings: [],
+                durationMs: 8200,
+                usage: {
+                    inputTokens: 1800,
+                    outputTokens: 320,
+                    costUSD: 0.0021,
+                },
+            };
+            const result = renderSkillReport(report);
+            expect(result.summaryComment).toContain('No findings to report');
+            expect(result.summaryComment).toContain('⏱ 8.2s');
+            expect(result.summaryComment).toContain('1.8k in / 320 out');
+            expect(result.summaryComment).toContain('$0.0021');
+        });
+        it('omits stats footer when no stats available', () => {
+            const report = {
+                ...baseReport,
+                findings: [],
+            };
+            const result = renderSkillReport(report);
+            expect(result.summaryComment).not.toContain('⏱');
+            expect(result.summaryComment).not.toContain('<sub>');
+        });
+    });
+    describe('check run link', () => {
+        it('shows link to full report when findings are filtered out', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'high',
+                        title: 'High Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                    {
+                        id: 'f2',
+                        severity: 'low',
+                        title: 'Low Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 20 },
+                    },
+                    {
+                        id: 'f3',
+                        severity: 'info',
+                        title: 'Info Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 30 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report, {
+                commentOn: 'high',
+                checkRunUrl: 'https://github.com/owner/repo/runs/123',
+                totalFindings: 3,
+            });
+            expect(result.summaryComment).toContain('View 2 additional findings in Checks');
+            expect(result.summaryComment).toContain('https://github.com/owner/repo/runs/123');
+        });
+        it('shows singular "finding" when only one is hidden', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'high',
+                        title: 'High Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                    {
+                        id: 'f2',
+                        severity: 'low',
+                        title: 'Low Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 20 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report, {
+                commentOn: 'high',
+                checkRunUrl: 'https://github.com/owner/repo/runs/123',
+                totalFindings: 2,
+            });
+            expect(result.summaryComment).toContain('View 1 additional finding in Checks');
+        });
+        it('shows link when all findings filtered out', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'low',
+                        title: 'Low Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report, {
+                commentOn: 'high',
+                checkRunUrl: 'https://github.com/owner/repo/runs/123',
+                totalFindings: 1,
+            });
+            expect(result.summaryComment).toContain('No findings to report');
+            expect(result.summaryComment).toContain('View 1 additional finding in Checks');
+        });
+        it('does not show link when no checkRunUrl provided', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'high',
+                        title: 'High Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report, {
+                commentOn: 'high',
+                totalFindings: 3,
+            });
+            expect(result.summaryComment).not.toContain('View');
+            expect(result.summaryComment).not.toContain('additional finding');
+        });
+        it('does not show link when no findings are hidden', () => {
+            const report = {
+                ...baseReport,
+                findings: [
+                    {
+                        id: 'f1',
+                        severity: 'high',
+                        title: 'High Issue',
+                        description: 'Details',
+                        location: { path: 'src/a.ts', startLine: 10 },
+                    },
+                ],
+            };
+            const result = renderSkillReport(report, {
+                commentOn: 'high',
+                checkRunUrl: 'https://github.com/owner/repo/runs/123',
+                totalFindings: 1,
+            });
+            expect(result.summaryComment).not.toContain('View');
+            expect(result.summaryComment).not.toContain('additional finding');
+        });
+    });
+});
+//# sourceMappingURL=renderer.test.js.map