@diff-review-system/drs 1.0.0 → 1.1.2

package/dist/lib/code-quality-report.test.d.ts

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

package/dist/lib/code-quality-report.test.d.ts.map

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

package/dist/lib/code-quality-report.test.js

@@ -0,0 +1,327 @@
+import { describe, it, expect } from 'vitest';
+import { convertToCodeQualityIssue, generateCodeQualityReport, formatCodeQualityReport, } from './code-quality-report.js';
+describe('code-quality-report', () => {
+    describe('convertToCodeQualityIssue', () => {
+        it('should convert a DRS issue to GitLab code quality format', () => {
+            const drsIssue = {
+                category: 'SECURITY',
+                severity: 'CRITICAL',
+                title: 'SQL Injection',
+                file: 'src/api/users.ts',
+                line: 42,
+                problem: 'Query uses string concatenation.',
+                solution: 'Use parameterized queries instead.',
+                agent: 'security',
+            };
+            const result = convertToCodeQualityIssue(drsIssue);
+            expect(result).toMatchObject({
+                description: 'Query uses string concatenation. Use parameterized queries instead.',
+                check_name: 'drs-security',
+                severity: 'blocker',
+                location: {
+                    path: 'src/api/users.ts',
+                    lines: {
+                        begin: 42,
+                    },
+                },
+            });
+            expect(result.fingerprint).toBeDefined();
+            expect(result.fingerprint.length).toBeGreaterThan(0);
+        });
+        it('should map CRITICAL severity to blocker', () => {
+            const issue = {
+                category: 'SECURITY',
+                severity: 'CRITICAL',
+                title: 'Test',
+                file: 'test.ts',
+                line: 1,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.severity).toBe('blocker');
+        });
+        it('should map HIGH severity to critical', () => {
+            const issue = {
+                category: 'QUALITY',
+                severity: 'HIGH',
+                title: 'Test',
+                file: 'test.ts',
+                line: 1,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.severity).toBe('critical');
+        });
+        it('should map MEDIUM severity to major', () => {
+            const issue = {
+                category: 'QUALITY',
+                severity: 'MEDIUM',
+                title: 'Test',
+                file: 'test.ts',
+                line: 1,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.severity).toBe('major');
+        });
+        it('should map LOW severity to minor', () => {
+            const issue = {
+                category: 'STYLE',
+                severity: 'LOW',
+                title: 'Test',
+                file: 'test.ts',
+                line: 1,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.severity).toBe('minor');
+        });
+        it('should generate check_name from category', () => {
+            const issue = {
+                category: 'QUALITY',
+                severity: 'MEDIUM',
+                title: 'Test',
+                file: 'test.ts',
+                line: 1,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.check_name).toBe('drs-quality');
+        });
+        it('should handle issues with empty solution', () => {
+            const issue = {
+                category: 'SECURITY',
+                severity: 'HIGH',
+                title: 'Test',
+                file: 'test.ts',
+                line: 1,
+                problem: 'This is a problem',
+                solution: '',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.description).toBe('This is a problem');
+        });
+        it('should default to line 1 if no line number provided', () => {
+            const issue = {
+                category: 'STYLE',
+                severity: 'LOW',
+                title: 'Missing header',
+                file: 'src/main.ts',
+                problem: 'No copyright header',
+                solution: 'Add standard file header',
+                agent: 'test',
+            };
+            const result = convertToCodeQualityIssue(issue);
+            expect(result.location.lines.begin).toBe(1);
+        });
+        it('should create consistent fingerprints for identical issues', () => {
+            const issue1 = {
+                category: 'SECURITY',
+                severity: 'CRITICAL',
+                title: 'Test',
+                file: 'test.ts',
+                line: 42,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const issue2 = {
+                category: 'SECURITY',
+                severity: 'CRITICAL',
+                title: 'Test',
+                file: 'test.ts',
+                line: 42,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result1 = convertToCodeQualityIssue(issue1);
+            const result2 = convertToCodeQualityIssue(issue2);
+            expect(result1.fingerprint).toBe(result2.fingerprint);
+        });
+        it('should create different fingerprints for different issues', () => {
+            const issue1 = {
+                category: 'SECURITY',
+                severity: 'CRITICAL',
+                title: 'Test',
+                file: 'test.ts',
+                line: 42,
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const issue2 = {
+                category: 'SECURITY',
+                severity: 'CRITICAL',
+                title: 'Test',
+                file: 'test.ts',
+                line: 43, // Different line
+                problem: 'Problem',
+                solution: 'Solution',
+                agent: 'test',
+            };
+            const result1 = convertToCodeQualityIssue(issue1);
+            const result2 = convertToCodeQualityIssue(issue2);
+            expect(result1.fingerprint).not.toBe(result2.fingerprint);
+        });
+    });
+    describe('generateCodeQualityReport', () => {
+        it('should convert multiple DRS issues to code quality format', () => {
+            const issues = [
+                {
+                    category: 'SECURITY',
+                    severity: 'CRITICAL',
+                    title: 'SQL Injection',
+                    file: 'src/api/users.ts',
+                    line: 42,
+                    problem: 'Query uses string concatenation',
+                    solution: 'Use parameterized queries',
+                    agent: 'security',
+                },
+                {
+                    category: 'QUALITY',
+                    severity: 'HIGH',
+                    title: 'High complexity',
+                    file: 'src/utils/helper.ts',
+                    line: 10,
+                    problem: 'Function has cyclomatic complexity of 15',
+                    solution: 'Break into smaller functions',
+                    agent: 'quality',
+                },
+            ];
+            const report = generateCodeQualityReport(issues);
+            expect(report).toHaveLength(2);
+            expect(report[0].severity).toBe('blocker');
+            expect(report[0].check_name).toBe('drs-security');
+            expect(report[1].severity).toBe('critical');
+            expect(report[1].check_name).toBe('drs-quality');
+        });
+        it('should handle empty issues array', () => {
+            const report = generateCodeQualityReport([]);
+            expect(report).toEqual([]);
+        });
+        it('should preserve all issues in order', () => {
+            const issues = [
+                {
+                    category: 'SECURITY',
+                    severity: 'CRITICAL',
+                    title: 'First',
+                    file: 'a.ts',
+                    line: 1,
+                    problem: 'Problem 1',
+                    solution: 'Solution 1',
+                    agent: 'test',
+                },
+                {
+                    category: 'QUALITY',
+                    severity: 'MEDIUM',
+                    title: 'Second',
+                    file: 'b.ts',
+                    line: 2,
+                    problem: 'Problem 2',
+                    solution: 'Solution 2',
+                    agent: 'test',
+                },
+                {
+                    category: 'STYLE',
+                    severity: 'LOW',
+                    title: 'Third',
+                    file: 'c.ts',
+                    line: 3,
+                    problem: 'Problem 3',
+                    solution: 'Solution 3',
+                    agent: 'test',
+                },
+            ];
+            const report = generateCodeQualityReport(issues);
+            expect(report).toHaveLength(3);
+            expect(report[0].location.path).toBe('a.ts');
+            expect(report[1].location.path).toBe('b.ts');
+            expect(report[2].location.path).toBe('c.ts');
+        });
+    });
+    describe('formatCodeQualityReport', () => {
+        it('should format report as valid JSON', () => {
+            const report = [
+                {
+                    description: 'Query uses string concatenation',
+                    check_name: 'drs-security',
+                    fingerprint: 'abc123',
+                    severity: 'blocker',
+                    location: {
+                        path: 'src/api/users.ts',
+                        lines: {
+                            begin: 42,
+                        },
+                    },
+                },
+            ];
+            const json = formatCodeQualityReport(report);
+            expect(() => JSON.parse(json)).not.toThrow();
+            const parsed = JSON.parse(json);
+            expect(parsed).toHaveLength(1);
+            expect(parsed[0].description).toBe('Query uses string concatenation');
+        });
+        it('should format with pretty printing', () => {
+            const report = [
+                {
+                    description: 'Test issue',
+                    check_name: 'drs-test',
+                    fingerprint: 'test123',
+                    severity: 'minor',
+                    location: {
+                        path: 'test.ts',
+                        lines: {
+                            begin: 1,
+                        },
+                    },
+                },
+            ];
+            const json = formatCodeQualityReport(report);
+            // Should have indentation (pretty printed)
+            expect(json).toContain('\n');
+            expect(json).toContain('  ');
+        });
+        it('should handle empty report', () => {
+            const json = formatCodeQualityReport([]);
+            expect(json).toBe('[]');
+        });
+        it('should produce valid GitLab code quality format', () => {
+            const report = [
+                {
+                    description: "'unused' is assigned a value but never used.",
+                    check_name: 'no-unused-vars',
+                    fingerprint: '7815696ecbf1c96e6894b779456d330e',
+                    severity: 'minor',
+                    location: {
+                        path: 'lib/index.js',
+                        lines: {
+                            begin: 42,
+                        },
+                    },
+                },
+            ];
+            const json = formatCodeQualityReport(report);
+            const parsed = JSON.parse(json);
+            // Verify all required fields are present
+            expect(parsed[0]).toHaveProperty('description');
+            expect(parsed[0]).toHaveProperty('fingerprint');
+            expect(parsed[0]).toHaveProperty('location');
+            expect(parsed[0].location).toHaveProperty('path');
+            expect(parsed[0].location).toHaveProperty('lines');
+            expect(parsed[0].location.lines).toHaveProperty('begin');
+        });
+    });
+});
+//# sourceMappingURL=code-quality-report.test.js.map

package/dist/lib/code-quality-report.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"code-quality-report.test.js","sourceRoot":"","sources":["../../src/lib/code-quality-report.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EACL,yBAAyB,EACzB,yBAAyB,EACzB,uBAAuB,GAExB,MAAM,0BAA0B,CAAC;AAGlC,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACzC,EAAE,CAAC,0DAA0D,EAAE,GAAG,EAAE;YAClE,MAAM,QAAQ,GAAgB;gBAC5B,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,eAAe;gBACtB,IAAI,EAAE,kBAAkB;gBACxB,IAAI,EAAE,EAAE;gBACR,OAAO,EAAE,kCAAkC;gBAC3C,QAAQ,EAAE,oCAAoC;gBAC9C,KAAK,EAAE,UAAU;aAClB,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,QAAQ,CAAC,CAAC;YAEnD,MAAM,CAAC,MAAM,CAAC,CAAC,aAAa,CAAC;gBAC3B,WAAW,EAAE,qEAAqE;gBAClF,UAAU,EAAE,cAAc;gBAC1B,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE;oBACR,IAAI,EAAE,kBAAkB;oBACxB,KAAK,EAAE;wBACL,KAAK,EAAE,EAAE;qBACV;iBACF;aACF,CAAC,CAAC;YACH,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,WAAW,EAAE,CAAC;YACzC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,CAAC;gBACP,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QAC1C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;YAC9C,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,MAAM;gBAChB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,CAAC;gBACP,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC3C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,QAAQ;gBAClB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,CAAC;gBACP,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;YAC1C,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,OAAO;gBACjB,QAAQ,EAAE,KAAK;gBACf,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,CAAC;gBACP,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,QAAQ;gBAClB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,CAAC;gBACP,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,MAAM;gBAChB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,CAAC;gBACP,OAAO,EAAE,mBAAmB;gBAC5B,QAAQ,EAAE,EAAE;gBACZ,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;YAC7D,MAAM,KAAK,GAAgB;gBACzB,QAAQ,EAAE,OAAO;gBACjB,QAAQ,EAAE,KAAK;gBACf,KAAK,EAAE,gBAAgB;gBACvB,IAAI,EAAE,aAAa;gBACnB,OAAO,EAAE,qBAAqB;gBAC9B,QAAQ,EAAE,0BAA0B;gBACpC,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,KAAK,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC9C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;YACpE,MAAM,MAAM,GAAgB;gBAC1B,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,EAAE;gBACR,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAgB;gBAC1B,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,EAAE;gBACR,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,OAAO,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;YAClD,MAAM,OAAO,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;YAElD,MAAM,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;YACnE,MAAM,MAAM,GAAgB;gBAC1B,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,EAAE;gBACR,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,MAAM,GAAgB;gBAC1B,QAAQ,EAAE,UAAU;gBACpB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,SAAS;gBACf,IAAI,EAAE,EAAE,EAAE,iBAAiB;gBAC3B,OAAO,EAAE,SAAS;gBAClB,QAAQ,EAAE,UAAU;gBACpB,KAAK,EAAE,MAAM;aACd,CAAC;YAEF,MAAM,OAAO,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;YAClD,MAAM,OAAO,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;YAElD,MAAM,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;QAC5D,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACzC,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;YACnE,MAAM,MAAM,GAAkB;gBAC5B;oBACE,QAAQ,EAAE,UAAU;oBACpB,QAAQ,EAAE,UAAU;oBACpB,KAAK,EAAE,eAAe;oBACtB,IAAI,EAAE,kBAAkB;oBACxB,IAAI,EAAE,EAAE;oBACR,OAAO,EAAE,iCAAiC;oBAC1C,QAAQ,EAAE,2BAA2B;oBACrC,KAAK,EAAE,UAAU;iBAClB;gBACD;oBACE,QAAQ,EAAE,SAAS;oBACnB,QAAQ,EAAE,MAAM;oBAChB,KAAK,EAAE,iBAAiB;oBACxB,IAAI,EAAE,qBAAqB;oBAC3B,IAAI,EAAE,EAAE;oBACR,OAAO,EAAE,0CAA0C;oBACnD,QAAQ,EAAE,8BAA8B;oBACxC,KAAK,EAAE,SAAS;iBACjB;aACF,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;YAEjD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YAC3C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;YAClD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAC5C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kCAAkC,EAAE,GAAG,EAAE;YAC1C,MAAM,MAAM,GAAG,yBAAyB,CAAC,EAAE,CAAC,CAAC;YAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC7B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;YAC7C,MAAM,MAAM,GAAkB;gBAC5B;oBACE,QAAQ,EAAE,UAAU;oBACpB,QAAQ,EAAE,UAAU;oBACpB,KAAK,EAAE,OAAO;oBACd,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,CAAC;oBACP,OAAO,EAAE,WAAW;oBACpB,QAAQ,EAAE,YAAY;oBACtB,KAAK,EAAE,MAAM;iBACd;gBACD;oBACE,QAAQ,EAAE,SAAS;oBACnB,QAAQ,EAAE,QAAQ;oBAClB,KAAK,EAAE,QAAQ;oBACf,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,CAAC;oBACP,OAAO,EAAE,WAAW;oBACpB,QAAQ,EAAE,YAAY;oBACtB,KAAK,EAAE,MAAM;iBACd;gBACD;oBACE,QAAQ,EAAE,OAAO;oBACjB,QAAQ,EAAE,KAAK;oBACf,KAAK,EAAE,OAAO;oBACd,IAAI,EAAE,MAAM;oBACZ,IAAI,EAAE,CAAC;oBACP,OAAO,EAAE,WAAW;oBACpB,QAAQ,EAAE,YAAY;oBACtB,KAAK,EAAE,MAAM;iBACd;aACF,CAAC;YAEF,MAAM,MAAM,GAAG,yBAAyB,CAAC,MAAM,CAAC,CAAC;YAEjD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;YAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,yBAAyB,EAAE,GAAG,EAAE;QACvC,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,MAAM,MAAM,GAAuB;gBACjC;oBACE,WAAW,EAAE,iCAAiC;oBAC9C,UAAU,EAAE,cAAc;oBAC1B,WAAW,EAAE,QAAQ;oBACrB,QAAQ,EAAE,SAAS;oBACnB,QAAQ,EAAE;wBACR,IAAI,EAAE,kBAAkB;wBACxB,KAAK,EAAE;4BACL,KAAK,EAAE,EAAE;yBACV;qBACF;iBACF;aACF,CAAC;YAEF,MAAM,IAAI,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAC;YAE7C,MAAM,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,EAAE,CAAC;YAC7C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;YAChC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,iCAAiC,CAAC,CAAC;QACxE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;YAC5C,MAAM,MAAM,GAAuB;gBACjC;oBACE,WAAW,EAAE,YAAY;oBACzB,UAAU,EAAE,UAAU;oBACtB,WAAW,EAAE,SAAS;oBACtB,QAAQ,EAAE,OAAO;oBACjB,QAAQ,EAAE;wBACR,IAAI,EAAE,SAAS;wBACf,KAAK,EAAE;4BACL,KAAK,EAAE,CAAC;yBACT;qBACF;iBACF;aACF,CAAC;YAEF,MAAM,IAAI,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAC;YAE7C,2CAA2C;YAC3C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;YAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAC/B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4BAA4B,EAAE,GAAG,EAAE;YACpC,MAAM,IAAI,GAAG,uBAAuB,CAAC,EAAE,CAAC,CAAC;YACzC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;YACzD,MAAM,MAAM,GAAuB;gBACjC;oBACE,WAAW,EAAE,8CAA8C;oBAC3D,UAAU,EAAE,gBAAgB;oBAC5B,WAAW,EAAE,kCAAkC;oBAC/C,QAAQ,EAAE,OAAO;oBACjB,QAAQ,EAAE;wBACR,IAAI,EAAE,cAAc;wBACpB,KAAK,EAAE;4BACL,KAAK,EAAE,EAAE;yBACV;qBACF;iBACF;aACF,CAAC;YAEF,MAAM,IAAI,GAAG,uBAAuB,CAAC,MAAM,CAAC,CAAC;YAC7C,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;YAEhC,yCAAyC;YACzC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC;YAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC;YAClD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC;YACnD,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC;QAC3D,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/{gitlab → lib}/comment-formatter.d.ts

@@ -19,12 +19,14 @@ export interface ReviewSummary {
 }
 /**
  * Format a single review issue as a GitLab comment
+ * @param fingerprint Optional fingerprint to embed in comment for deduplication
  */
-export declare function formatIssueComment(issue: ReviewIssue): string;
+export declare function formatIssueComment(issue: ReviewIssue, fingerprint?: string): string;
 /**
  * Format a review summary as a GitLab comment
+ * @param commentId Optional comment ID to embed for update identification
  */
-export declare function formatSummaryComment(summary: ReviewSummary, issues: ReviewIssue[]): string;
+export declare function formatSummaryComment(summary: ReviewSummary, issues: ReviewIssue[], commentId?: string): string;
 /**
  * Format issue for terminal output with colors
  */

package/dist/lib/comment-formatter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"comment-formatter.d.ts","sourceRoot":"","sources":["../../src/lib/comment-formatter.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,aAAa,GAAG,UAAU,GAAG,MAAM,GAAG,QAAQ,GAAG,KAAK,CAAC;AACnE,MAAM,MAAM,aAAa,GAAG,UAAU,GAAG,SAAS,GAAG,OAAO,GAAG,aAAa,CAAC;AAE7E,MAAM,WAAW,WAAW;IAC1B,QAAQ,EAAE,aAAa,CAAC;IACxB,QAAQ,EAAE,aAAa,CAAC;IACxB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,MAAM,EAAE,CAAC;IACtB,KAAK,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,aAAa;IAC5B,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;IAC1C,UAAU,EAAE,MAAM,CAAC,aAAa,EAAE,MAAM,CAAC,CAAC;CAC3C;AAgBD;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,WAAW,EAAE,WAAW,CAAC,EAAE,MAAM,GAAG,MAAM,CA0BnF;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAClC,OAAO,EAAE,aAAa,EACtB,MAAM,EAAE,WAAW,EAAE,EACrB,SAAS,CAAC,EAAE,MAAM,GACjB,MAAM,CA8ER;AAED;;GAEG;AACH,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,WAAW,GAAG,MAAM,CAY9D;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,WAAW,EAAE,GAAG,aAAa,CAwB5F"}

package/dist/{gitlab → lib}/comment-formatter.js

@@ -12,14 +12,20 @@ const CATEGORY_EMOJI = {
 };
 /**
  * Format a single review issue as a GitLab comment
+ * @param fingerprint Optional fingerprint to embed in comment for deduplication
  */
-export function formatIssueComment(issue) {
+export function formatIssueComment(issue, fingerprint) {
     const emoji = CATEGORY_EMOJI[issue.category];
     const severityEmoji = SEVERITY_EMOJI[issue.severity];
-    let comment = `## ${emoji} ${issue.category} - ${issue.title}\n\n`;
+    // Add hidden fingerprint for deduplication
+    let comment = '';
+    if (fingerprint) {
+        comment += `<!-- issue-fp: ${fingerprint} -->\n`;
+    }
+    comment += `## ${emoji} ${issue.category} - ${issue.title}\n\n`;
     comment += `**File**: \`${issue.file}${issue.line ? `:${issue.line}` : ''}\`\n`;
     comment += `**Severity**: ${severityEmoji} ${issue.severity}\n`;
-    comment += `**Reviewer**: ${issue.agent}\n\n`;
+    comment += `**Analysis by**: ${issue.agent}\n\n`;
     comment += `### Problem\n${issue.problem}\n\n`;
     comment += `### Solution\n${issue.solution}\n`;
     if (issue.references && issue.references.length > 0) {
@@ -32,9 +38,15 @@ export function formatIssueComment(issue) {
 }
 /**
  * Format a review summary as a GitLab comment
+ * @param commentId Optional comment ID to embed for update identification
  */
-export function formatSummaryComment(summary, issues) {
-    let comment = `# 🤖 AI Code Review Summary\n\n`;
+export function formatSummaryComment(summary, issues, commentId) {
+    // Add hidden identifier for update-or-create logic
+    let comment = '';
+    if (commentId) {
+        comment += `<!-- drs-comment-id: ${commentId} -->\n`;
+    }
+    comment += `# 📋 Code Review Analysis\n\n`;
     comment += `## 📊 Statistics\n\n`;
     comment += `- **Files Reviewed**: ${summary.filesReviewed}\n`;
     comment += `- **Total Issues**: ${summary.issuesFound}\n\n`;
@@ -49,30 +61,51 @@ export function formatSummaryComment(summary, issues) {
         comment += `- ${CATEGORY_EMOJI.QUALITY} **Quality**: ${summary.byCategory.QUALITY}\n`;
         comment += `- ${CATEGORY_EMOJI.STYLE} **Style**: ${summary.byCategory.STYLE}\n`;
         comment += `- ${CATEGORY_EMOJI.PERFORMANCE} **Performance**: ${summary.byCategory.PERFORMANCE}\n\n`;
-        // List critical and high issues
-        const criticalIssues = issues.filter(i => i.severity === 'CRITICAL');
-        const highIssues = issues.filter(i => i.severity === 'HIGH');
+        // Group issues by severity
+        const criticalIssues = issues.filter((i) => i.severity === 'CRITICAL');
+        const highIssues = issues.filter((i) => i.severity === 'HIGH');
+        const mediumIssues = issues.filter((i) => i.severity === 'MEDIUM');
+        const lowIssues = issues.filter((i) => i.severity === 'LOW');
+        // Helper function to format issue details
+        const formatIssueDetails = (issue) => {
+            let details = `### ${CATEGORY_EMOJI[issue.category]} ${issue.title}\n\n`;
+            details += `**File**: \`${issue.file}${issue.line ? `:${issue.line}` : ''}\` | **Category**: ${issue.category}\n\n`;
+            details += `**Problem**: ${issue.problem}\n\n`;
+            details += `**Solution**: ${issue.solution}\n`;
+            if (issue.references && issue.references.length > 0) {
+                details += `\n**References**: ${issue.references.join(', ')}\n`;
+            }
+            return details + '\n';
+        };
         if (criticalIssues.length > 0) {
             comment += `## 🔴 Critical Issues\n\n`;
             for (const issue of criticalIssues) {
-                comment += `- **${issue.title}** in \`${issue.file}${issue.line ? `:${issue.line}` : ''}\`\n`;
+                comment += formatIssueDetails(issue);
             }
-            comment += `\n`;
         }
         if (highIssues.length > 0) {
             comment += `## 🟡 High Priority Issues\n\n`;
             for (const issue of highIssues) {
-                comment += `- **${issue.title}** in \`${issue.file}${issue.line ? `:${issue.line}` : ''}\`\n`;
+                comment += formatIssueDetails(issue);
+            }
+        }
+        if (mediumIssues.length > 0) {
+            comment += `## 🟠 Medium Priority Issues\n\n`;
+            for (const issue of mediumIssues) {
+                comment += formatIssueDetails(issue);
+            }
+        }
+        if (lowIssues.length > 0) {
+            comment += `## ⚪ Low Priority Issues\n\n`;
+            for (const issue of lowIssues) {
+                comment += formatIssueDetails(issue);
             }
-            comment += `\n`;
         }
-        comment += `\n---\n`;
-        comment += `*Detailed findings have been posted as individual discussion threads on the affected lines.*\n`;
     }
     else {
         comment += `✅ **No issues found!** The code looks good.\n`;
     }
-    comment += `\n*Powered by DRS - GitLab Review Bot*\n`;
+    comment += `\n---\n\n*Analyzed by **DRS** | Diff Review System*\n`;
     return comment;
 }
 /**

package/dist/lib/comment-formatter.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"comment-formatter.js","sourceRoot":"","sources":["../../src/lib/comment-formatter.ts"],"names":[],"mappings":"AAsBA,MAAM,cAAc,GAAkC;IACpD,QAAQ,EAAE,IAAI;IACd,IAAI,EAAE,IAAI;IACV,MAAM,EAAE,IAAI;IACZ,GAAG,EAAE,GAAG;CACT,CAAC;AAEF,MAAM,cAAc,GAAkC;IACpD,QAAQ,EAAE,IAAI;IACd,OAAO,EAAE,IAAI;IACb,KAAK,EAAE,GAAG;IACV,WAAW,EAAE,GAAG;CACjB,CAAC;AAEF;;;GAGG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAkB,EAAE,WAAoB;IACzE,MAAM,KAAK,GAAG,cAAc,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IAC7C,MAAM,aAAa,GAAG,cAAc,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IAErD,2CAA2C;IAC3C,IAAI,OAAO,GAAG,EAAE,CAAC;IACjB,IAAI,WAAW,EAAE,CAAC;QAChB,OAAO,IAAI,kBAAkB,WAAW,QAAQ,CAAC;IACnD,CAAC;IAED,OAAO,IAAI,MAAM,KAAK,IAAI,KAAK,CAAC,QAAQ,MAAM,KAAK,CAAC,KAAK,MAAM,CAAC;IAChE,OAAO,IAAI,eAAe,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,MAAM,CAAC;IAChF,OAAO,IAAI,iBAAiB,aAAa,IAAI,KAAK,CAAC,QAAQ,IAAI,CAAC;IAChE,OAAO,IAAI,oBAAoB,KAAK,CAAC,KAAK,MAAM,CAAC;IAEjD,OAAO,IAAI,gBAAgB,KAAK,CAAC,OAAO,MAAM,CAAC;IAC/C,OAAO,IAAI,iBAAiB,KAAK,CAAC,QAAQ,IAAI,CAAC;IAE/C,IAAI,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACpD,OAAO,IAAI,oBAAoB,CAAC;QAChC,KAAK,MAAM,GAAG,IAAI,KAAK,CAAC,UAAU,EAAE,CAAC;YACnC,OAAO,IAAI,KAAK,GAAG,IAAI,CAAC;QAC1B,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,oBAAoB,CAClC,OAAsB,EACtB,MAAqB,EACrB,SAAkB;IAElB,mDAAmD;IACnD,IAAI,OAAO,GAAG,EAAE,CAAC;IACjB,IAAI,SAAS,EAAE,CAAC;QACd,OAAO,IAAI,wBAAwB,SAAS,QAAQ,CAAC;IACvD,CAAC;IAED,OAAO,IAAI,+BAA+B,CAAC;IAE3C,OAAO,IAAI,sBAAsB,CAAC;IAClC,OAAO,IAAI,yBAAyB,OAAO,CAAC,aAAa,IAAI,CAAC;IAC9D,OAAO,IAAI,uBAAuB,OAAO,CAAC,WAAW,MAAM,CAAC;IAE5D,IAAI,OAAO,CAAC,WAAW,GAAG,CAAC,EAAE,CAAC;QAC5B,OAAO,IAAI,mBAAmB,CAAC;QAC/B,OAAO,IAAI,KAAK,cAAc,CAAC,QAAQ,kBAAkB,OAAO,CAAC,UAAU,CAAC,QAAQ,IAAI,CAAC;QACzF,OAAO,IAAI,KAAK,cAAc,CAAC,IAAI,cAAc,OAAO,CAAC,UAAU,CAAC,IAAI,IAAI,CAAC;QAC7E,OAAO,IAAI,KAAK,cAAc,CAAC,MAAM,gBAAgB,OAAO,CAAC,UAAU,CAAC,MAAM,IAAI,CAAC;QACnF,OAAO,IAAI,KAAK,cAAc,CAAC,GAAG,aAAa,OAAO,CAAC,UAAU,CAAC,GAAG,MAAM,CAAC;QAE5E,OAAO,IAAI,mBAAmB,CAAC;QAC/B,OAAO,IAAI,KAAK,cAAc,CAAC,QAAQ,kBAAkB,OAAO,CAAC,UAAU,CAAC,QAAQ,IAAI,CAAC;QACzF,OAAO,IAAI,KAAK,cAAc,CAAC,OAAO,iBAAiB,OAAO,CAAC,UAAU,CAAC,OAAO,IAAI,CAAC;QACtF,OAAO,IAAI,KAAK,cAAc,CAAC,KAAK,eAAe,OAAO,CAAC,UAAU,CAAC,KAAK,IAAI,CAAC;QAChF,OAAO,IAAI,KAAK,cAAc,CAAC,WAAW,qBAAqB,OAAO,CAAC,UAAU,CAAC,WAAW,MAAM,CAAC;QAEpG,2BAA2B;QAC3B,MAAM,cAAc,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,UAAU,CAAC,CAAC;QACvE,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC;QAC/D,MAAM,YAAY,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC;QACnE,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,KAAK,CAAC,CAAC;QAE7D,0CAA0C;QAC1C,MAAM,kBAAkB,GAAG,CAAC,KAAkB,EAAE,EAAE;YAChD,IAAI,OAAO,GAAG,OAAO,cAAc,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,KAAK,CAAC,KAAK,MAAM,CAAC;YACzE,OAAO,IAAI,eAAe,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,sBAAsB,KAAK,CAAC,QAAQ,MAAM,CAAC;YACpH,OAAO,IAAI,gBAAgB,KAAK,CAAC,OAAO,MAAM,CAAC;YAC/C,OAAO,IAAI,iBAAiB,KAAK,CAAC,QAAQ,IAAI,CAAC;YAC/C,IAAI,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBACpD,OAAO,IAAI,qBAAqB,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC;YAClE,CAAC;YACD,OAAO,OAAO,GAAG,IAAI,CAAC;QACxB,CAAC,CAAC;QAEF,IAAI,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC9B,OAAO,IAAI,2BAA2B,CAAC;YACvC,KAAK,MAAM,KAAK,IAAI,cAAc,EAAE,CAAC;gBACnC,OAAO,IAAI,kBAAkB,CAAC,KAAK,CAAC,CAAC;YACvC,CAAC;QACH,CAAC;QAED,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC1B,OAAO,IAAI,gCAAgC,CAAC;YAC5C,KAAK,MAAM,KAAK,IAAI,UAAU,EAAE,CAAC;gBAC/B,OAAO,IAAI,kBAAkB,CAAC,KAAK,CAAC,CAAC;YACvC,CAAC;QACH,CAAC;QAED,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC5B,OAAO,IAAI,kCAAkC,CAAC;YAC9C,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;gBACjC,OAAO,IAAI,kBAAkB,CAAC,KAAK,CAAC,CAAC;YACvC,CAAC;QACH,CAAC;QAED,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACzB,OAAO,IAAI,8BAA8B,CAAC;YAC1C,KAAK,MAAM,KAAK,IAAI,SAAS,EAAE,CAAC;gBAC9B,OAAO,IAAI,kBAAkB,CAAC,KAAK,CAAC,CAAC;YACvC,CAAC;QACH,CAAC;IACH,CAAC;SAAM,CAAC;QACN,OAAO,IAAI,+CAA+C,CAAC;IAC7D,CAAC;IAED,OAAO,IAAI,uDAAuD,CAAC;IAEnE,OAAO,OAAO,CAAC;AACjB,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,mBAAmB,CAAC,KAAkB;IACpD,MAAM,KAAK,GAAG,cAAc,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IAC7C,MAAM,aAAa,GAAG,cAAc,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IAErD,IAAI,MAAM,GAAG,2CAA2C,CAAC;IACzD,MAAM,IAAI,GAAG,aAAa,IAAI,KAAK,CAAC,QAAQ,KAAK,KAAK,IAAI,KAAK,CAAC,QAAQ,MAAM,KAAK,CAAC,KAAK,IAAI,CAAC;IAC9F,MAAM,IAAI,yCAAyC,CAAC;IACpD,MAAM,IAAI,MAAM,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,MAAM,CAAC;IACtE,MAAM,IAAI,GAAG,KAAK,CAAC,OAAO,MAAM,CAAC;IACjC,MAAM,IAAI,UAAU,KAAK,CAAC,QAAQ,IAAI,CAAC;IAEvC,OAAO,MAAM,CAAC;AAChB,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,gBAAgB,CAAC,aAAqB,EAAE,MAAqB;IAC3E,MAAM,OAAO,GAAkB;QAC7B,aAAa;QACb,WAAW,EAAE,MAAM,CAAC,MAAM;QAC1B,UAAU,EAAE;YACV,QAAQ,EAAE,CAAC;YACX,IAAI,EAAE,CAAC;YACP,MAAM,EAAE,CAAC;YACT,GAAG,EAAE,CAAC;SACP;QACD,UAAU,EAAE;YACV,QAAQ,EAAE,CAAC;YACX,OAAO,EAAE,CAAC;YACV,KAAK,EAAE,CAAC;YACR,WAAW,EAAE,CAAC;SACf;KACF,CAAC;IAEF,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,OAAO,CAAC,UAAU,CAAC,KAAK,CAAC,QAAQ,CAAC,EAAE,CAAC;QACrC,OAAO,CAAC,UAAU,CAAC,KAAK,CAAC,QAAQ,CAAC,EAAE,CAAC;IACvC,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/dist/lib/comment-manager.d.ts

@@ -0,0 +1,61 @@
+import type { ReviewIssue } from './comment-formatter.js';
+/**
+ * Platform-agnostic comment management for deduplication and updates
+ */
+export declare const BOT_COMMENT_ID = "drs-review-summary";
+/**
+ * Create a unique fingerprint for an issue to detect duplicates
+ */
+export declare function createIssueFingerprint(issue: ReviewIssue): string;
+/**
+ * Extract bot comment ID from comment body
+ */
+export declare function extractCommentId(body: string): string | null;
+/**
+ * Extract issue fingerprints from comment body
+ */
+export declare function extractIssueFingerprints(body: string): Set<string>;
+/**
+ * Filter issues to only include CRITICAL and HIGH severity
+ * (for inline comments - reduces noise and API calls)
+ */
+export declare function filterCriticalAndHigh(issues: ReviewIssue[]): ReviewIssue[];
+/**
+ * Filter out duplicate issues based on existing fingerprints
+ */
+export declare function filterDuplicateIssues(issues: ReviewIssue[], existingFingerprints: Set<string>): ReviewIssue[];
+/**
+ * Represents a comment from any platform
+ */
+export interface PlatformComment {
+    id: number | string;
+    body: string;
+}
+/**
+ * Find existing summary comment using bot marker
+ */
+export declare function findExistingSummaryComment(comments: PlatformComment[]): PlatformComment | null;
+/**
+ * Collect all existing issue fingerprints from comments
+ */
+export declare function collectExistingFingerprints(comments: PlatformComment[]): Set<string>;
+/**
+ * Result of preparing issues for posting
+ */
+export interface PreparedIssues {
+    /** Issues to post as inline comments (CRITICAL/HIGH, new, with valid lines) */
+    inlineIssues: ReviewIssue[];
+    /** Number of issues that were deduplicated */
+    deduplicatedCount: number;
+    /** Number of medium/low severity issues (not posted inline) */
+    nonInlineCount: number;
+}
+/**
+ * Prepare issues for posting, applying all filters:
+ * 1. Filter to CRITICAL/HIGH severity only
+ * 2. Filter out duplicates based on existing comments
+ * 3. Filter to only issues with line numbers
+ * 4. Optionally filter to valid line numbers (platform-specific)
+ */
+export declare function prepareIssuesForPosting(allIssues: ReviewIssue[], existingComments: PlatformComment[], validLinesChecker?: (issue: ReviewIssue) => boolean): PreparedIssues;
+//# sourceMappingURL=comment-manager.d.ts.map

package/dist/lib/comment-manager.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"comment-manager.d.ts","sourceRoot":"","sources":["../../src/lib/comment-manager.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAE1D;;GAEG;AAGH,eAAO,MAAM,cAAc,uBAAuB,CAAC;AAEnD;;GAEG;AACH,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,WAAW,GAAG,MAAM,CAEjE;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAG5D;AAED;;GAEG;AACH,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,CAQlE;AAED;;;GAGG;AACH,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,WAAW,EAAE,GAAG,WAAW,EAAE,CAE1E;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,WAAW,EAAE,EACrB,oBAAoB,EAAE,GAAG,CAAC,MAAM,CAAC,GAChC,WAAW,EAAE,CAKf;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,EAAE,EAAE,MAAM,GAAG,MAAM,CAAC;IACpB,IAAI,EAAE,MAAM,CAAC;CACd;AAED;;GAEG;AACH,wBAAgB,0BAA0B,CAAC,QAAQ,EAAE,eAAe,EAAE,GAAG,eAAe,GAAG,IAAI,CAE9F;AAED;;GAEG;AACH,wBAAgB,2BAA2B,CAAC,QAAQ,EAAE,eAAe,EAAE,GAAG,GAAG,CAAC,MAAM,CAAC,CAOpF;AAED;;GAEG;AACH,MAAM,WAAW,cAAc;IAC7B,+EAA+E;IAC/E,YAAY,EAAE,WAAW,EAAE,CAAC;IAC5B,8CAA8C;IAC9C,iBAAiB,EAAE,MAAM,CAAC;IAC1B,+DAA+D;IAC/D,cAAc,EAAE,MAAM,CAAC;CACxB;AAED;;;;;;GAMG;AACH,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,WAAW,EAAE,EACxB,gBAAgB,EAAE,eAAe,EAAE,EACnC,iBAAiB,CAAC,EAAE,CAAC,KAAK,EAAE,WAAW,KAAK,OAAO,GAClD,cAAc,CAuBhB"}

package/dist/lib/comment-manager.js

@@ -0,0 +1,91 @@
+/**
+ * Platform-agnostic comment management for deduplication and updates
+ */
+// Bot identifier for tracking our comments
+export const BOT_COMMENT_ID = 'drs-review-summary';
+/**
+ * Create a unique fingerprint for an issue to detect duplicates
+ */
+export function createIssueFingerprint(issue) {
+    return `${issue.file}:${issue.line || 'general'}:${issue.category}:${issue.title}`;
+}
+/**
+ * Extract bot comment ID from comment body
+ */
+export function extractCommentId(body) {
+    const match = body.match(/<!-- drs-comment-id: (.*?) -->/);
+    return match ? match[1] : null;
+}
+/**
+ * Extract issue fingerprints from comment body
+ */
+export function extractIssueFingerprints(body) {
+    const fingerprints = new Set();
+    const regex = /<!-- issue-fp: (.*?) -->/g;
+    let match;
+    while ((match = regex.exec(body)) !== null) {
+        fingerprints.add(match[1]);
+    }
+    return fingerprints;
+}
+/**
+ * Filter issues to only include CRITICAL and HIGH severity
+ * (for inline comments - reduces noise and API calls)
+ */
+export function filterCriticalAndHigh(issues) {
+    return issues.filter((i) => i.severity === 'CRITICAL' || i.severity === 'HIGH');
+}
+/**
+ * Filter out duplicate issues based on existing fingerprints
+ */
+export function filterDuplicateIssues(issues, existingFingerprints) {
+    return issues.filter((issue) => {
+        const fingerprint = createIssueFingerprint(issue);
+        return !existingFingerprints.has(fingerprint);
+    });
+}
+/**
+ * Find existing summary comment using bot marker
+ */
+export function findExistingSummaryComment(comments) {
+    return comments.find((c) => extractCommentId(c.body) === BOT_COMMENT_ID) || null;
+}
+/**
+ * Collect all existing issue fingerprints from comments
+ */
+export function collectExistingFingerprints(comments) {
+    const allFingerprints = new Set();
+    for (const comment of comments) {
+        const fingerprints = extractIssueFingerprints(comment.body);
+        fingerprints.forEach((fp) => allFingerprints.add(fp));
+    }
+    return allFingerprints;
+}
+/**
+ * Prepare issues for posting, applying all filters:
+ * 1. Filter to CRITICAL/HIGH severity only
+ * 2. Filter out duplicates based on existing comments
+ * 3. Filter to only issues with line numbers
+ * 4. Optionally filter to valid line numbers (platform-specific)
+ */
+export function prepareIssuesForPosting(allIssues, existingComments, validLinesChecker) {
+    // Step 1: Filter to CRITICAL/HIGH only
+    const criticalAndHigh = filterCriticalAndHigh(allIssues);
+    const nonInlineCount = allIssues.length - criticalAndHigh.length;
+    // Step 2: Filter out duplicates
+    const existingFingerprints = collectExistingFingerprints(existingComments);
+    const newIssues = filterDuplicateIssues(criticalAndHigh, existingFingerprints);
+    const deduplicatedCount = criticalAndHigh.length - newIssues.length;
+    // Step 3: Filter to only issues with line numbers
+    let inlineIssues = newIssues.filter((issue) => issue.line !== undefined && issue.line !== null);
+    // Step 4: Optionally filter based on valid lines (platform-specific)
+    if (validLinesChecker) {
+        inlineIssues = inlineIssues.filter(validLinesChecker);
+    }
+    return {
+        inlineIssues,
+        deduplicatedCount,
+        nonInlineCount,
+    };
+}
+//# sourceMappingURL=comment-manager.js.map

package/dist/lib/comment-manager.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"comment-manager.js","sourceRoot":"","sources":["../../src/lib/comment-manager.ts"],"names":[],"mappings":"AAEA;;GAEG;AAEH,2CAA2C;AAC3C,MAAM,CAAC,MAAM,cAAc,GAAG,oBAAoB,CAAC;AAEnD;;GAEG;AACH,MAAM,UAAU,sBAAsB,CAAC,KAAkB;IACvD,OAAO,GAAG,KAAK,CAAC,IAAI,IAAI,KAAK,CAAC,IAAI,IAAI,SAAS,IAAI,KAAK,CAAC,QAAQ,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;AACrF,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,gBAAgB,CAAC,IAAY;IAC3C,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,gCAAgC,CAAC,CAAC;IAC3D,OAAO,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AACjC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,wBAAwB,CAAC,IAAY;IACnD,MAAM,YAAY,GAAG,IAAI,GAAG,EAAU,CAAC;IACvC,MAAM,KAAK,GAAG,2BAA2B,CAAC;IAC1C,IAAI,KAAK,CAAC;IACV,OAAO,CAAC,KAAK,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAC3C,YAAY,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;IAC7B,CAAC;IACD,OAAO,YAAY,CAAC;AACtB,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,qBAAqB,CAAC,MAAqB;IACzD,OAAO,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,UAAU,IAAI,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC;AAClF,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,qBAAqB,CACnC,MAAqB,EACrB,oBAAiC;IAEjC,OAAO,MAAM,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE;QAC7B,MAAM,WAAW,GAAG,sBAAsB,CAAC,KAAK,CAAC,CAAC;QAClD,OAAO,CAAC,oBAAoB,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;AACL,CAAC;AAUD;;GAEG;AACH,MAAM,UAAU,0BAA0B,CAAC,QAA2B;IACpE,OAAO,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,gBAAgB,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,cAAc,CAAC,IAAI,IAAI,CAAC;AACnF,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,2BAA2B,CAAC,QAA2B;IACrE,MAAM,eAAe,GAAG,IAAI,GAAG,EAAU,CAAC;IAC1C,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,MAAM,YAAY,GAAG,wBAAwB,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5D,YAAY,CAAC,OAAO,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,eAAe,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;IACxD,CAAC;IACD,OAAO,eAAe,CAAC;AACzB,CAAC;AAcD;;;;;;GAMG;AACH,MAAM,UAAU,uBAAuB,CACrC,SAAwB,EACxB,gBAAmC,EACnC,iBAAmD;IAEnD,uCAAuC;IACvC,MAAM,eAAe,GAAG,qBAAqB,CAAC,SAAS,CAAC,CAAC;IACzD,MAAM,cAAc,GAAG,SAAS,CAAC,MAAM,GAAG,eAAe,CAAC,MAAM,CAAC;IAEjE,gCAAgC;IAChC,MAAM,oBAAoB,GAAG,2BAA2B,CAAC,gBAAgB,CAAC,CAAC;IAC3E,MAAM,SAAS,GAAG,qBAAqB,CAAC,eAAe,EAAE,oBAAoB,CAAC,CAAC;IAC/E,MAAM,iBAAiB,GAAG,eAAe,CAAC,MAAM,GAAG,SAAS,CAAC,MAAM,CAAC;IAEpE,kDAAkD;IAClD,IAAI,YAAY,GAAG,SAAS,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,KAAK,SAAS,IAAI,KAAK,CAAC,IAAI,KAAK,IAAI,CAAC,CAAC;IAEhG,qEAAqE;IACrE,IAAI,iBAAiB,EAAE,CAAC;QACtB,YAAY,GAAG,YAAY,CAAC,MAAM,CAAC,iBAAiB,CAAC,CAAC;IACxD,CAAC;IAED,OAAO;QACL,YAAY;QACZ,iBAAiB;QACjB,cAAc;KACf,CAAC;AACJ,CAAC"}