@diff-review-system/drs 1.0.0 → 2.0.0

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"}

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

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

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

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