@diff-review-system/drs 1.0.0 → 1.1.2

package/dist/lib/issue-parser.d.ts

@@ -0,0 +1,29 @@
+import type { ReviewIssue } from './comment-formatter.js';
+/**
+ * Parse review issues from agent response messages
+ *
+ * Agents should output JSON in the following format:
+ * ```json
+ * {
+ *   "issues": [
+ *     {
+ *       "category": "SECURITY" | "QUALITY" | "STYLE" | "PERFORMANCE",
+ *       "severity": "CRITICAL" | "HIGH" | "MEDIUM" | "LOW",
+ *       "title": "Issue title",
+ *       "file": "path/to/file.ts",
+ *       "line": 42,
+ *       "problem": "Description of the problem",
+ *       "solution": "Suggested fix",
+ *       "references": ["https://link1", "https://link2"],
+ *       "agent": "security"
+ *     }
+ *   ]
+ * }
+ * ```
+ */
+export declare function parseReviewIssues(content: string, agentName?: string): ReviewIssue[];
+/**
+ * Extract agent name from session context or message
+ */
+export declare function extractAgentName(message: string): string;
+//# sourceMappingURL=issue-parser.d.ts.map

package/dist/lib/issue-parser.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"issue-parser.d.ts","sourceRoot":"","sources":["../../src/lib/issue-parser.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAE1D;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,SAAS,GAAE,MAAkB,GAAG,WAAW,EAAE,CAuE/F;AAqDD;;GAEG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAOxD"}

package/dist/lib/issue-parser.js

@@ -0,0 +1,151 @@
+/**
+ * Parse review issues from agent response messages
+ *
+ * Agents should output JSON in the following format:
+ * ```json
+ * {
+ *   "issues": [
+ *     {
+ *       "category": "SECURITY" | "QUALITY" | "STYLE" | "PERFORMANCE",
+ *       "severity": "CRITICAL" | "HIGH" | "MEDIUM" | "LOW",
+ *       "title": "Issue title",
+ *       "file": "path/to/file.ts",
+ *       "line": 42,
+ *       "problem": "Description of the problem",
+ *       "solution": "Suggested fix",
+ *       "references": ["https://link1", "https://link2"],
+ *       "agent": "security"
+ *     }
+ *   ]
+ * }
+ * ```
+ */
+export function parseReviewIssues(content, agentName = 'unknown') {
+    const issues = [];
+    try {
+        // Try to find JSON blocks in the content
+        // Look for code blocks with ```json or raw JSON objects
+        const jsonBlockRegex = /```json\s*([\s\S]*?)\s*```/g;
+        let match;
+        // First try to find JSON code blocks
+        while ((match = jsonBlockRegex.exec(content)) !== null) {
+            try {
+                const parsed = JSON.parse(match[1]);
+                if (parsed.issues && Array.isArray(parsed.issues)) {
+                    for (const issue of parsed.issues) {
+                        if (isValidIssue(issue)) {
+                            issues.push({
+                                ...issue,
+                                agent: issue.agent || agentName,
+                            });
+                        }
+                    }
+                }
+            }
+            catch (e) {
+                // Continue to next match
+            }
+        }
+        // If no code blocks found, try to find raw JSON objects
+        if (issues.length === 0) {
+            // Try to parse the entire content as JSON
+            try {
+                const parsed = JSON.parse(content);
+                if (parsed.issues && Array.isArray(parsed.issues)) {
+                    for (const issue of parsed.issues) {
+                        if (isValidIssue(issue)) {
+                            issues.push({
+                                ...issue,
+                                agent: issue.agent || agentName,
+                            });
+                        }
+                    }
+                }
+            }
+            catch (e) {
+                // Not valid JSON, try to find JSON objects with better bracket matching
+                const jsonObjects = extractJsonObjects(content);
+                for (const jsonStr of jsonObjects) {
+                    try {
+                        const parsed = JSON.parse(jsonStr);
+                        if (parsed.issues && Array.isArray(parsed.issues)) {
+                            for (const issue of parsed.issues) {
+                                if (isValidIssue(issue)) {
+                                    issues.push({
+                                        ...issue,
+                                        agent: issue.agent || agentName,
+                                    });
+                                }
+                            }
+                        }
+                    }
+                    catch (e) {
+                        // Continue to next object
+                    }
+                }
+            }
+        }
+    }
+    catch (error) {
+        console.warn('Failed to parse review issues from content:', error);
+    }
+    return issues;
+}
+/**
+ * Extract JSON objects from text by matching brackets
+ */
+function extractJsonObjects(text) {
+    const objects = [];
+    let depth = 0;
+    let start = -1;
+    for (let i = 0; i < text.length; i++) {
+        if (text[i] === '{') {
+            if (depth === 0) {
+                start = i;
+            }
+            depth++;
+        }
+        else if (text[i] === '}') {
+            depth--;
+            if (depth === 0 && start !== -1) {
+                const obj = text.substring(start, i + 1);
+                // Only consider objects that contain "issues"
+                if (obj.includes('"issues"')) {
+                    objects.push(obj);
+                }
+                start = -1;
+            }
+        }
+    }
+    return objects;
+}
+/**
+ * Validate that an object has the required ReviewIssue fields
+ */
+function isValidIssue(obj) {
+    return (obj &&
+        typeof obj === 'object' &&
+        typeof obj.category === 'string' &&
+        ['SECURITY', 'QUALITY', 'STYLE', 'PERFORMANCE'].includes(obj.category) &&
+        typeof obj.severity === 'string' &&
+        ['CRITICAL', 'HIGH', 'MEDIUM', 'LOW'].includes(obj.severity) &&
+        typeof obj.title === 'string' &&
+        typeof obj.file === 'string' &&
+        typeof obj.problem === 'string' &&
+        typeof obj.solution === 'string' &&
+        (obj.line === undefined || typeof obj.line === 'number') &&
+        (obj.references === undefined || Array.isArray(obj.references)) &&
+        (obj.agent === undefined || typeof obj.agent === 'string'));
+}
+/**
+ * Extract agent name from session context or message
+ */
+export function extractAgentName(message) {
+    // Try to extract from common patterns like "Agent: security" or "Reviewer: quality"
+    const agentMatch = message.match(/(?:agent|reviewer):\s*(\w+)/i);
+    if (agentMatch) {
+        return agentMatch[1];
+    }
+    return 'unknown';
+}
+//# sourceMappingURL=issue-parser.js.map

package/dist/lib/issue-parser.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"issue-parser.js","sourceRoot":"","sources":["../../src/lib/issue-parser.ts"],"names":[],"mappings":"AAEA;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAAe,EAAE,YAAoB,SAAS;IAC9E,MAAM,MAAM,GAAkB,EAAE,CAAC;IAEjC,IAAI,CAAC;QACH,yCAAyC;QACzC,wDAAwD;QACxD,MAAM,cAAc,GAAG,6BAA6B,CAAC;QAErD,IAAI,KAAK,CAAC;QAEV,qCAAqC;QACrC,OAAO,CAAC,KAAK,GAAG,cAAc,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YACvD,IAAI,CAAC;gBACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;gBACpC,IAAI,MAAM,CAAC,MAAM,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC;oBAClD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC;wBAClC,IAAI,YAAY,CAAC,KAAK,CAAC,EAAE,CAAC;4BACxB,MAAM,CAAC,IAAI,CAAC;gCACV,GAAG,KAAK;gCACR,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,SAAS;6BAChC,CAAC,CAAC;wBACL,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAAC,OAAO,CAAC,EAAE,CAAC;gBACX,yBAAyB;YAC3B,CAAC;QACH,CAAC;QAED,wDAAwD;QACxD,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACxB,0CAA0C;YAC1C,IAAI,CAAC;gBACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;gBACnC,IAAI,MAAM,CAAC,MAAM,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC;oBAClD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC;wBAClC,IAAI,YAAY,CAAC,KAAK,CAAC,EAAE,CAAC;4BACxB,MAAM,CAAC,IAAI,CAAC;gCACV,GAAG,KAAK;gCACR,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,SAAS;6BAChC,CAAC,CAAC;wBACL,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;YAAC,OAAO,CAAC,EAAE,CAAC;gBACX,wEAAwE;gBACxE,MAAM,WAAW,GAAG,kBAAkB,CAAC,OAAO,CAAC,CAAC;gBAChD,KAAK,MAAM,OAAO,IAAI,WAAW,EAAE,CAAC;oBAClC,IAAI,CAAC;wBACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;wBACnC,IAAI,MAAM,CAAC,MAAM,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC;4BAClD,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC;gCAClC,IAAI,YAAY,CAAC,KAAK,CAAC,EAAE,CAAC;oCACxB,MAAM,CAAC,IAAI,CAAC;wCACV,GAAG,KAAK;wCACR,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,SAAS;qCAChC,CAAC,CAAC;gCACL,CAAC;4BACH,CAAC;wBACH,CAAC;oBACH,CAAC;oBAAC,OAAO,CAAC,EAAE,CAAC;wBACX,0BAA0B;oBAC5B,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO,CAAC,IAAI,CAAC,6CAA6C,EAAE,KAAK,CAAC,CAAC;IACrE,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED;;GAEG;AACH,SAAS,kBAAkB,CAAC,IAAY;IACtC,MAAM,OAAO,GAAa,EAAE,CAAC;IAC7B,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,KAAK,GAAG,CAAC,CAAC,CAAC;IAEf,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACrC,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;YACpB,IAAI,KAAK,KAAK,CAAC,EAAE,CAAC;gBAChB,KAAK,GAAG,CAAC,CAAC;YACZ,CAAC;YACD,KAAK,EAAE,CAAC;QACV,CAAC;aAAM,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;YAC3B,KAAK,EAAE,CAAC;YACR,IAAI,KAAK,KAAK,CAAC,IAAI,KAAK,KAAK,CAAC,CAAC,EAAE,CAAC;gBAChC,MAAM,GAAG,GAAG,IAAI,CAAC,SAAS,CAAC,KAAK,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;gBACzC,8CAA8C;gBAC9C,IAAI,GAAG,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;oBAC7B,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;gBACpB,CAAC;gBACD,KAAK,GAAG,CAAC,CAAC,CAAC;YACb,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAED;;GAEG;AACH,SAAS,YAAY,CAAC,GAAQ;IAC5B,OAAO,CACL,GAAG;QACH,OAAO,GAAG,KAAK,QAAQ;QACvB,OAAO,GAAG,CAAC,QAAQ,KAAK,QAAQ;QAChC,CAAC,UAAU,EAAE,SAAS,EAAE,OAAO,EAAE,aAAa,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,QAAQ,CAAC;QACtE,OAAO,GAAG,CAAC,QAAQ,KAAK,QAAQ;QAChC,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,KAAK,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,QAAQ,CAAC;QAC5D,OAAO,GAAG,CAAC,KAAK,KAAK,QAAQ;QAC7B,OAAO,GAAG,CAAC,IAAI,KAAK,QAAQ;QAC5B,OAAO,GAAG,CAAC,OAAO,KAAK,QAAQ;QAC/B,OAAO,GAAG,CAAC,QAAQ,KAAK,QAAQ;QAChC,CAAC,GAAG,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,GAAG,CAAC,IAAI,KAAK,QAAQ,CAAC;QACxD,CAAC,GAAG,CAAC,UAAU,KAAK,SAAS,IAAI,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;QAC/D,CAAC,GAAG,CAAC,KAAK,KAAK,SAAS,IAAI,OAAO,GAAG,CAAC,KAAK,KAAK,QAAQ,CAAC,CAC3D,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,gBAAgB,CAAC,OAAe;IAC9C,oFAAoF;IACpF,MAAM,UAAU,GAAG,OAAO,CAAC,KAAK,CAAC,8BAA8B,CAAC,CAAC;IACjE,IAAI,UAAU,EAAE,CAAC;QACf,OAAO,UAAU,CAAC,CAAC,CAAC,CAAC;IACvB,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC"}

package/dist/lib/issue-parser.test.d.ts

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

package/dist/lib/issue-parser.test.d.ts.map

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

package/dist/lib/issue-parser.test.js

@@ -0,0 +1,281 @@
+import { describe, it, expect } from 'vitest';
+import { parseReviewIssues } from './issue-parser.js';
+describe('parseReviewIssues', () => {
+    it('should parse valid JSON code block with issues', () => {
+        const content = `Here are my findings:
+
+\`\`\`json
+{
+  "issues": [
+    {
+      "category": "SECURITY",
+      "severity": "CRITICAL",
+      "title": "SQL Injection vulnerability",
+      "file": "src/api/users.ts",
+      "line": 42,
+      "problem": "Query uses string concatenation",
+      "solution": "Use parameterized queries",
+      "references": ["https://owasp.org/sql-injection"],
+      "agent": "security"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content, 'security');
+        expect(issues).toHaveLength(1);
+        expect(issues[0]).toEqual({
+            category: 'SECURITY',
+            severity: 'CRITICAL',
+            title: 'SQL Injection vulnerability',
+            file: 'src/api/users.ts',
+            line: 42,
+            problem: 'Query uses string concatenation',
+            solution: 'Use parameterized queries',
+            references: ['https://owasp.org/sql-injection'],
+            agent: 'security',
+        });
+    });
+    it('should parse multiple issues in a single JSON block', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "QUALITY",
+      "severity": "HIGH",
+      "title": "High complexity function",
+      "file": "src/utils/helper.ts",
+      "line": 10,
+      "problem": "Function has cyclomatic complexity of 15",
+      "solution": "Break into smaller functions",
+      "agent": "quality"
+    },
+    {
+      "category": "QUALITY",
+      "severity": "MEDIUM",
+      "title": "Code duplication",
+      "file": "src/utils/helper.ts",
+      "line": 50,
+      "problem": "Duplicated validation logic",
+      "solution": "Extract to shared validator function"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content, 'quality');
+        expect(issues).toHaveLength(2);
+        expect(issues[0].severity).toBe('HIGH');
+        expect(issues[1].severity).toBe('MEDIUM');
+        expect(issues[1].agent).toBe('quality'); // Should use default agent name
+    });
+    it('should parse raw JSON without code block markers', () => {
+        const content = `{"issues":[{"category":"PERFORMANCE","severity":"LOW","title":"Inefficient loop","file":"src/app.ts","line":5,"problem":"Using nested loops","solution":"Use hash map"}]}`;
+        const issues = parseReviewIssues(content, 'performance');
+        expect(issues).toHaveLength(1);
+        expect(issues[0].category).toBe('PERFORMANCE');
+        expect(issues[0].file).toBe('src/app.ts');
+    });
+    it('should handle issues without line numbers', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "STYLE",
+      "severity": "LOW",
+      "title": "Missing file header",
+      "file": "src/main.ts",
+      "problem": "No copyright header",
+      "solution": "Add standard file header",
+      "agent": "style"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content, 'style');
+        expect(issues).toHaveLength(1);
+        expect(issues[0].line).toBeUndefined();
+    });
+    it('should handle issues without references', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "QUALITY",
+      "severity": "MEDIUM",
+      "title": "Long function",
+      "file": "src/utils.ts",
+      "line": 100,
+      "problem": "Function exceeds 50 lines",
+      "solution": "Refactor into smaller functions",
+      "agent": "quality"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content, 'quality');
+        expect(issues).toHaveLength(1);
+        expect(issues[0].references).toBeUndefined();
+    });
+    it('should return empty array for invalid JSON', () => {
+        const content = 'This is not valid JSON';
+        const issues = parseReviewIssues(content);
+        expect(issues).toEqual([]);
+    });
+    it('should return empty array for JSON without issues array', () => {
+        const content = `\`\`\`json
+{
+  "message": "No issues found"
+}
+\`\`\``;
+        const issues = parseReviewIssues(content);
+        expect(issues).toEqual([]);
+    });
+    it('should skip invalid issues that are missing required fields', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "SECURITY",
+      "severity": "HIGH",
+      "title": "Missing problem field",
+      "file": "src/test.ts",
+      "solution": "Fix it"
+    },
+    {
+      "category": "QUALITY",
+      "severity": "MEDIUM",
+      "title": "Valid issue",
+      "file": "src/valid.ts",
+      "line": 10,
+      "problem": "This is valid",
+      "solution": "This has all required fields"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content);
+        // Should only parse the valid issue
+        expect(issues).toHaveLength(1);
+        expect(issues[0].title).toBe('Valid issue');
+    });
+    it('should reject issues with invalid category', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "INVALID_CATEGORY",
+      "severity": "HIGH",
+      "title": "Test",
+      "file": "test.ts",
+      "problem": "Problem",
+      "solution": "Solution"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content);
+        expect(issues).toEqual([]);
+    });
+    it('should reject issues with invalid severity', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "SECURITY",
+      "severity": "SUPER_CRITICAL",
+      "title": "Test",
+      "file": "test.ts",
+      "problem": "Problem",
+      "solution": "Solution"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content);
+        expect(issues).toEqual([]);
+    });
+    it('should handle multiple JSON blocks in same content', () => {
+        const content = `First agent findings:
+
+\`\`\`json
+{
+  "issues": [
+    {
+      "category": "SECURITY",
+      "severity": "HIGH",
+      "title": "Issue 1",
+      "file": "src/a.ts",
+      "line": 1,
+      "problem": "Problem 1",
+      "solution": "Solution 1",
+      "agent": "security"
+    }
+  ]
+}
+\`\`\`
+
+Second agent findings:
+
+\`\`\`json
+{
+  "issues": [
+    {
+      "category": "QUALITY",
+      "severity": "MEDIUM",
+      "title": "Issue 2",
+      "file": "src/b.ts",
+      "line": 2,
+      "problem": "Problem 2",
+      "solution": "Solution 2",
+      "agent": "quality"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content);
+        expect(issues).toHaveLength(2);
+        expect(issues[0].agent).toBe('security');
+        expect(issues[1].agent).toBe('quality');
+    });
+    it('should use provided agent name when issue does not have agent field', () => {
+        const content = `\`\`\`json
+{
+  "issues": [
+    {
+      "category": "PERFORMANCE",
+      "severity": "LOW",
+      "title": "Slow operation",
+      "file": "src/perf.ts",
+      "line": 20,
+      "problem": "Inefficient algorithm",
+      "solution": "Use better algorithm"
+    }
+  ]
+}
+\`\`\``;
+        const issues = parseReviewIssues(content, 'performance-agent');
+        expect(issues).toHaveLength(1);
+        expect(issues[0].agent).toBe('performance-agent');
+    });
+    it('should handle whitespace in JSON code blocks', () => {
+        const content = `\`\`\`json
+
+{
+  "issues": [
+    {
+      "category": "STYLE",
+      "severity": "LOW",
+      "title": "Formatting",
+      "file": "src/style.ts",
+      "line": 1,
+      "problem": "Inconsistent formatting",
+      "solution": "Run prettier"
+    }
+  ]
+}
+
+\`\`\``;
+        const issues = parseReviewIssues(content);
+        expect(issues).toHaveLength(1);
+    });
+});
+//# sourceMappingURL=issue-parser.test.js.map

package/dist/lib/issue-parser.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"issue-parser.test.js","sourceRoot":"","sources":["../../src/lib/issue-parser.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AAEtD,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;;;;OAkBb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;QAEtD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;YACxB,QAAQ,EAAE,UAAU;YACpB,QAAQ,EAAE,UAAU;YACpB,KAAK,EAAE,6BAA6B;YACpC,IAAI,EAAE,kBAAkB;YACxB,IAAI,EAAE,EAAE;YACR,OAAO,EAAE,iCAAiC;YAC1C,QAAQ,EAAE,2BAA2B;YACrC,UAAU,EAAE,CAAC,iCAAiC,CAAC;YAC/C,KAAK,EAAE,UAAU;SAClB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC7D,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;;;;;;;;;;OAwBb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;QAErD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACxC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,gCAAgC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,OAAO,GAAG,2KAA2K,CAAC;QAE5L,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAEzD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;QAC/C,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,OAAO,GAAG;;;;;;;;;;;;;;OAcb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;QAEnD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,aAAa,EAAE,CAAC;IACzC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;OAeb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;QAErD,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,aAAa,EAAE,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,OAAO,GAAG,wBAAwB,CAAC;QAEzC,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;QACjE,MAAM,OAAO,GAAG;;;;OAIb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;;;;;;;OAqBb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,oCAAoC;QACpC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,OAAO,GAAG;;;;;;;;;;;;;OAab,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QACpD,MAAM,OAAO,GAAG;;;;;;;;;;;;;OAab,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC7B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC5D,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;OAoCb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACzC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,MAAM,OAAO,GAAG;;;;;;;;;;;;;;OAcb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAC;QAE/D,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC/B,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;QACtD,MAAM,OAAO,GAAG;;;;;;;;;;;;;;;;OAgBb,CAAC;QAEJ,MAAM,MAAM,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE1C,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IACjC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/lib/platform-client.d.ts

@@ -0,0 +1,130 @@
+/**
+ * Platform abstraction layer for GitHub and GitLab
+ *
+ * This module provides common interfaces for interacting with different
+ * code review platforms (GitHub, GitLab) in a unified way.
+ */
+/**
+ * Represents a file change in a pull/merge request
+ */
+export interface FileChange {
+    /** Path to the file */
+    filename: string;
+    /** Type of change */
+    status: 'added' | 'removed' | 'modified' | 'renamed' | 'copied' | 'changed' | 'unchanged';
+    /** Number of lines added */
+    additions: number;
+    /** Number of lines deleted */
+    deletions: number;
+    /** Unified diff patch */
+    patch?: string;
+    /** Previous filename (for renamed files) */
+    previousFilename?: string;
+}
+/**
+ * Represents a pull/merge request
+ */
+export interface PullRequest {
+    /** PR/MR number or ID */
+    number: number;
+    /** Title */
+    title: string;
+    /** Description/body */
+    description?: string;
+    /** Author username or name */
+    author: string;
+    /** Source branch */
+    sourceBranch: string;
+    /** Target branch */
+    targetBranch: string;
+    /** Head commit SHA */
+    headSha: string;
+    /** Additional platform-specific data */
+    platformData?: any;
+}
+/**
+ * Represents a comment on a PR/MR
+ */
+export interface Comment {
+    /** Comment ID */
+    id: number | string;
+    /** Comment body/content */
+    body: string;
+}
+/**
+ * Position data for inline comments
+ */
+export interface InlineCommentPosition {
+    /** File path */
+    path: string;
+    /** Line number in the new version */
+    line: number;
+    /** Commit SHA (GitHub) or diff refs (GitLab) */
+    commitSha?: string;
+    baseSha?: string;
+    headSha?: string;
+    startSha?: string;
+}
+/**
+ * Common interface for platform clients (GitHub, GitLab, etc.)
+ */
+export interface PlatformClient {
+    /**
+     * Get pull/merge request details
+     */
+    getPullRequest(projectId: string, prNumber: number): Promise<PullRequest>;
+    /**
+     * Get list of changed files in a PR/MR
+     */
+    getChangedFiles(projectId: string, prNumber: number): Promise<FileChange[]>;
+    /**
+     * Get existing comments on the PR/MR
+     */
+    getComments(projectId: string, prNumber: number): Promise<Comment[]>;
+    /**
+     * Get existing inline/review comments
+     */
+    getInlineComments(projectId: string, prNumber: number): Promise<Comment[]>;
+    /**
+     * Create a general comment on the PR/MR
+     */
+    createComment(projectId: string, prNumber: number, body: string): Promise<void>;
+    /**
+     * Update an existing comment
+     */
+    updateComment(projectId: string, prNumber: number, commentId: number | string, body: string): Promise<void>;
+    /**
+     * Create an inline comment at a specific line
+     */
+    createInlineComment(projectId: string, prNumber: number, body: string, position: InlineCommentPosition): Promise<void>;
+    /**
+     * Create multiple inline comments in bulk (if supported)
+     * Falls back to individual comments if not supported
+     */
+    createBulkInlineComments(projectId: string, prNumber: number, comments: Array<{
+        body: string;
+        position: InlineCommentPosition;
+    }>): Promise<void>;
+    /**
+     * Add labels to the PR/MR
+     */
+    addLabels(projectId: string, prNumber: number, labels: string[]): Promise<void>;
+    /**
+     * Check if PR/MR has a specific label
+     */
+    hasLabel(projectId: string, prNumber: number, label: string): Promise<boolean>;
+}
+/**
+ * Validator for checking if a line can be commented on
+ */
+export interface LineValidator {
+    /**
+     * Check if a line number is valid for commenting
+     */
+    isValidLine(file: string, line: number): boolean;
+}
+/**
+ * Factory function type for creating platform clients
+ */
+export type PlatformClientFactory = () => PlatformClient;
+//# sourceMappingURL=platform-client.d.ts.map

package/dist/lib/platform-client.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"platform-client.d.ts","sourceRoot":"","sources":["../../src/lib/platform-client.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;GAEG;AACH,MAAM,WAAW,UAAU;IACzB,uBAAuB;IACvB,QAAQ,EAAE,MAAM,CAAC;IACjB,qBAAqB;IACrB,MAAM,EAAE,OAAO,GAAG,SAAS,GAAG,UAAU,GAAG,SAAS,GAAG,QAAQ,GAAG,SAAS,GAAG,WAAW,CAAC;IAC1F,4BAA4B;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,8BAA8B;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,yBAAyB;IACzB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,4CAA4C;IAC5C,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B;AAED;;GAEG;AACH,MAAM,WAAW,WAAW;IAC1B,yBAAyB;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY;IACZ,KAAK,EAAE,MAAM,CAAC;IACd,uBAAuB;IACvB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,8BAA8B;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,oBAAoB;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,oBAAoB;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,sBAAsB;IACtB,OAAO,EAAE,MAAM,CAAC;IAChB,wCAAwC;IACxC,YAAY,CAAC,EAAE,GAAG,CAAC;CACpB;AAED;;GAEG;AACH,MAAM,WAAW,OAAO;IACtB,iBAAiB;IACjB,EAAE,EAAE,MAAM,GAAG,MAAM,CAAC;IACpB,2BAA2B;IAC3B,IAAI,EAAE,MAAM,CAAC;CACd;AAED;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC,gBAAgB;IAChB,IAAI,EAAE,MAAM,CAAC;IACb,qCAAqC;IACrC,IAAI,EAAE,MAAM,CAAC;IACb,gDAAgD;IAChD,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED;;GAEG;AACH,MAAM,WAAW,cAAc;IAC7B;;OAEG;IACH,cAAc,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC,CAAC;IAE1E;;OAEG;IACH,eAAe,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,UAAU,EAAE,CAAC,CAAC;IAE5E;;OAEG;IACH,WAAW,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,EAAE,CAAC,CAAC;IAErE;;OAEG;IACH,iBAAiB,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,EAAE,CAAC,CAAC;IAE3E;;OAEG;IACH,aAAa,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAEhF;;OAEG;IACH,aAAa,CACX,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,GAAG,MAAM,EAC1B,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,IAAI,CAAC,CAAC;IAEjB;;OAEG;IACH,mBAAmB,CACjB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,EACZ,QAAQ,EAAE,qBAAqB,GAC9B,OAAO,CAAC,IAAI,CAAC,CAAC;IAEjB;;;OAGG;IACH,wBAAwB,CACtB,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,qBAAqB,CAAA;KAAE,CAAC,GACjE,OAAO,CAAC,IAAI,CAAC,CAAC;IAEjB;;OAEG;IACH,SAAS,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAEhF;;OAEG;IACH,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;CAChF;AAED;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B;;OAEG;IACH,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC;CAClD;AAED;;GAEG;AACH,MAAM,MAAM,qBAAqB,GAAG,MAAM,cAAc,CAAC"}

package/dist/lib/platform-client.js

@@ -0,0 +1,8 @@
+/**
+ * Platform abstraction layer for GitHub and GitLab
+ *
+ * This module provides common interfaces for interacting with different
+ * code review platforms (GitHub, GitLab) in a unified way.
+ */
+export {};
+//# sourceMappingURL=platform-client.js.map

package/dist/lib/platform-client.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"platform-client.js","sourceRoot":"","sources":["../../src/lib/platform-client.ts"],"names":[],"mappings":"AAAA;;;;;GAKG"}

package/dist/lib/position-validator.d.ts

@@ -0,0 +1,36 @@
+/**
+ * Position validation utilities for platform-specific inline comment requirements
+ */
+import type { InlineCommentPosition } from './platform-client.js';
+/**
+ * Result of position validation
+ */
+export interface PositionValidationResult {
+    isValid: boolean;
+    error?: string;
+}
+/**
+ * Base interface for position validators
+ */
+export interface PositionValidator {
+    validate(position: InlineCommentPosition): PositionValidationResult;
+}
+/**
+ * GitHub position validator
+ * Requires: commitSha
+ */
+export declare class GitHubPositionValidator implements PositionValidator {
+    validate(position: InlineCommentPosition): PositionValidationResult;
+}
+/**
+ * GitLab position validator
+ * Requires: baseSha, headSha, startSha
+ */
+export declare class GitLabPositionValidator implements PositionValidator {
+    validate(position: InlineCommentPosition): PositionValidationResult;
+}
+/**
+ * Helper to validate and throw if invalid
+ */
+export declare function validatePositionOrThrow(position: InlineCommentPosition, validator: PositionValidator): void;
+//# sourceMappingURL=position-validator.d.ts.map