@bragduck/cli 2.28.0 → 2.28.1

package/dist/bin/bragduck.js

@@ -3653,33 +3653,6 @@ var JiraService = class {
       return null;
     }
   }
-  /**
-   * Check if a user object matches the given identifier (accountId, email, or username)
-   */
-  isMatchingUser(candidate, userIdentifier) {
-    if (!candidate) return false;
-    return candidate.email === userIdentifier || candidate.emailAddress === userIdentifier || candidate.accountId === userIdentifier || candidate.username === userIdentifier || candidate.name === userIdentifier;
-  }
-  /**
-   * Filter issues to only those where the user made contributions within the date range.
-   * Excludes issues where the user's only involvement is a static role (creator/assignee)
-   * with a date outside the scan period.
-   */
-  filterIssuesByUserContribution(issues, userIdentifier, sinceDate) {
-    const results = [];
-    for (const issue of issues) {
-      const userChanges = (issue.changelog?.histories || []).filter(
-        (h) => this.isMatchingUser(h.author, userIdentifier) && new Date(h.created) >= sinceDate
-      );
-      const isCreatorInRange = this.isMatchingUser(issue.fields.creator, userIdentifier) && new Date(issue.fields.created) >= sinceDate;
-      if (userChanges.length > 0 || isCreatorInRange) {
-        results.push({ issue, userChanges });
-      } else {
-        logger.debug(`Excluding issue ${issue.key} - no user contributions in date range`);
-      }
-    }
-    return results;
-  }
   /**
    * Build JQL query from options
    */
@@ -3737,7 +3710,7 @@ var JiraService = class {
     }
     const isAssigned = issue.fields.assignee?.emailAddress === userEmail;
     const isResolved = issue.fields.resolutiondate !== null && issue.fields.resolutiondate !== void 0;
-    const userEdits = issue.changelog?.histories?.filter((history) => history.author?.emailAddress === userEmail) || [];
+    const userEdits = issue.changelog?.histories?.filter((history) => history.author.emailAddress === userEmail) || [];
     const hasEdits = userEdits.length > 0;
     if (isAssigned && isResolved) {
       return {
@@ -3772,77 +3745,6 @@ var JiraService = class {
     };
     return Math.ceil(baseComplexity * multipliers[contributionType]);
   }
-  /**
-   * Summarize the user's specific changes from changelog entries into human-readable lines.
-   * This enriches the brag message so the AI refinement can generate a more specific brag.
-   */
-  summarizeUserChanges(userChanges) {
-    const MAX_LINES = 10;
-    const allItems = [];
-    for (const entry of userChanges) {
-      for (const item of entry.items) {
-        allItems.push(item);
-      }
-    }
-    if (allItems.length === 0) return "";
-    const latestByField = /* @__PURE__ */ new Map();
-    for (const item of allItems) {
-      latestByField.set(item.field, { fromString: item.fromString, toString: item.toString });
-    }
-    const lines = [];
-    for (const [field, change] of latestByField) {
-      if (lines.length >= MAX_LINES) break;
-      const from = change.fromString || "";
-      const to = change.toString || "";
-      switch (field.toLowerCase()) {
-        case "status":
-          lines.push(from ? `Moved status from '${from}' to '${to}'` : `Set status to '${to}'`);
-          break;
-        case "resolution":
-          lines.push(to ? `Resolved as '${to}'` : "Reopened issue");
-          break;
-        case "assignee":
-          lines.push(to ? `Assigned to ${to}` : "Unassigned");
-          break;
-        case "priority":
-          lines.push(
-            from ? `Changed priority from '${from}' to '${to}'` : `Set priority to '${to}'`
-          );
-          break;
-        case "summary":
-          lines.push("Updated issue title");
-          break;
-        case "description":
-          lines.push("Updated description");
-          break;
-        case "comment":
-          lines.push("Added comment");
-          break;
-        case "labels":
-          lines.push(to ? `Updated labels: ${to}` : "Removed labels");
-          break;
-        case "fix version":
-        case "fixversions":
-          lines.push(to ? `Set fix version: ${to}` : "Removed fix version");
-          break;
-        case "sprint":
-          lines.push(to ? `Moved to sprint: ${to}` : "Removed from sprint");
-          break;
-        case "story points":
-        case "story point estimate":
-          lines.push(`Set story points to ${to}`);
-          break;
-        default:
-          if (to) {
-            lines.push(`Updated ${field}`);
-          }
-          break;
-      }
-    }
-    if (lines.length === 0) return "";
-    return `User changes:
-${lines.map((l) => `- ${l}`).join("\n")}`;
-  }
   /**
    * Fetch issues with optional filtering
    */
@@ -3880,7 +3782,7 @@ ${lines.map((l) => `- ${l}`).join("\n")}`;
         );
         break;
       }
-      const endpoint = `/rest/api/3/search/jql?jql=${encodeURIComponent(jql)}&startAt=${startAt}&maxResults=${maxResults}&fields=${fields.join(",")}&expand=changelog`;
+      const endpoint = `/rest/api/3/search/jql?jql=${encodeURIComponent(jql)}&startAt=${startAt}&maxResults=${maxResults}&fields=${fields.join(",")}`;
       try {
         const response = await this.request(endpoint);
         if (response.issues.length === 0) {
@@ -3906,7 +3808,14 @@ ${lines.map((l) => `- ${l}`).join("\n")}`;
           break;
         }
         if (options.limit && allIssues.length >= options.limit) {
-          break;
+          const email2 = await this.getCurrentUser();
+          const limitedIssues = allIssues.slice(0, options.limit);
+          const commits2 = [];
+          for (const issue of limitedIssues) {
+            const commit = await this.transformIssueToCommit(issue, void 0, email2 || void 0);
+            commits2.push(commit);
+          }
+          return commits2;
         }
         startAt += maxResults;
       } catch (error) {
@@ -3930,23 +3839,9 @@ ${lines.map((l) => `- ${l}`).join("\n")}`;
         throw error;
       }
     }
-    const issuesToProcess = options.limit ? allIssues.slice(0, options.limit) : allIssues;
     const email = await this.getCurrentUser();
-    const sinceDate = options.days ? new Date(Date.now() - options.days * 24 * 60 * 60 * 1e3) : void 0;
-    if (sinceDate && email) {
-      const filtered = this.filterIssuesByUserContribution(issuesToProcess, email, sinceDate);
-      logger.debug(
-        `Date-scoped filtering: ${issuesToProcess.length} issues -> ${filtered.length} with user contributions in range`
-      );
-      const commits2 = [];
-      for (const { issue, userChanges } of filtered) {
-        const commit = await this.transformIssueToCommit(issue, void 0, email, userChanges);
-        commits2.push(commit);
-      }
-      return commits2;
-    }
     const commits = [];
-    for (const issue of issuesToProcess) {
+    for (const issue of allIssues) {
       const commit = await this.transformIssueToCommit(issue, void 0, email || void 0);
       commits.push(commit);
     }
@@ -3968,7 +3863,7 @@ ${lines.map((l) => `- ${l}`).join("\n")}`;
   /**
    * Transform Jira issue to GitCommit format with contribution-specific data
    */
-  async transformIssueToCommit(issue, instanceUrl, userEmail, userChanges) {
+  async transformIssueToCommit(issue, instanceUrl, userEmail) {
     let contribution = null;
     if (userEmail) {
       contribution = await this.determineJiraContributionType(issue, userEmail);
@@ -4005,21 +3900,11 @@ ${contribution.details}`;
 ${truncatedDesc}`;
       }
     }
-    if (userChanges && userChanges.length > 0) {
-      const changeSummary = this.summarizeUserChanges(userChanges);
-      if (changeSummary) {
-        message += `
-
-${changeSummary}`;
-      }
-    }
     let date;
     if (contribution?.type === "created" || contribution?.type === "reported") {
       date = issue.fields.created;
     } else if (contribution?.type === "assigned-resolved" && issue.fields.resolutiondate) {
       date = issue.fields.resolutiondate;
-    } else if (userChanges && userChanges.length > 0) {
-      date = userChanges[userChanges.length - 1].created;
     } else {
       date = issue.fields.updated;
     }
@@ -4209,60 +4094,6 @@ var ConfluenceService = class {
       return null;
     }
   }
-  /**
-   * Check if a user object matches the given identifier (accountId, email, or username)
-   */
-  isMatchingUser(candidate, userIdentifier) {
-    if (!candidate) return false;
-    return candidate.email === userIdentifier || candidate.emailAddress === userIdentifier || candidate.accountId === userIdentifier || candidate.username === userIdentifier || candidate.name === userIdentifier;
-  }
-  /**
-   * Fetch full version history for a page
-   */
-  async getPageVersionHistory(pageId) {
-    const allVersions = [];
-    let start = 0;
-    const limit = 50;
-    while (true) {
-      const response = await this.request(
-        `/wiki/rest/api/content/${pageId}/version?start=${start}&limit=${limit}`
-      );
-      allVersions.push(...response.results);
-      if (response.size < limit) break;
-      start += limit;
-    }
-    return allVersions;
-  }
-  /**
-   * Filter pages to only those where the user made contributions within the date range.
-   * Fetches version history per page and checks if the user has versions in range.
-   */
-  async filterPagesByUserContribution(pages, userIdentifier, sinceDate) {
-    const results = [];
-    for (let i = 0; i < pages.length; i++) {
-      const page = pages[i];
-      if (i > 0) {
-        await new Promise((resolve) => globalThis.setTimeout(resolve, 100));
-      }
-      try {
-        const versions = await this.getPageVersionHistory(page.id);
-        const userVersions = versions.filter(
-          (v) => this.isMatchingUser(v.by, userIdentifier) && new Date(v.when) >= sinceDate
-        );
-        const userCommentsInRange = page.children?.comment?.results?.filter(
-          (comment) => this.isMatchingUser(comment.version?.by || {}, userIdentifier) && new Date(comment.version?.when || 0) >= sinceDate
-        ) || [];
-        if (userVersions.length > 0 || userCommentsInRange.length > 0) {
-          results.push({ page, userVersions });
-        } else {
-          logger.debug(`Excluding page "${page.title}" - no user contributions in date range`);
-        }
-      } catch (error) {
-        logger.debug(`Skipping version history for page ${page.id}: ${error}`);
-      }
-    }
-    return results;
-  }
   /**
    * Build CQL query from options
    * Returns empty string if no filters need CQL (will use simple endpoint instead)
@@ -4298,31 +4129,28 @@ var ConfluenceService = class {
    * Determine the type of contribution the current user made to a page
    * Returns: 'created' | 'edited' | 'commented'
    */
-  async determineContributionType(page, userEmail, userVersions) {
-    const createdByUser = page.history?.createdBy ? this.isMatchingUser(page.history.createdBy, userEmail) : false;
-    if (createdByUser) {
+  async determineContributionType(page, userEmail) {
+    if (page.history?.createdBy?.email === userEmail) {
       return {
         type: "created",
         details: `Created page with ${page.version?.number || 1} version${(page.version?.number || 1) > 1 ? "s" : ""}`
       };
     }
-    const hasEdits = userVersions ? userVersions.some((v) => !v.minorEdit || v.number > 1) : page.version?.by ? this.isMatchingUser(page.version.by, userEmail) && (page.version?.number || 0) > 1 : false;
+    const hasEdits = page.version?.by?.email === userEmail && (page.version?.number || 0) > 1;
     const userComments = page.children?.comment?.results?.filter(
-      (comment) => comment.version?.by ? this.isMatchingUser(comment.version.by, userEmail) : false
+      (comment) => comment.version?.by?.email === userEmail
     ) || [];
     const hasComments = userComments.length > 0;
     if (hasEdits && hasComments) {
-      const editCount = userVersions?.length || 1;
       return {
         type: "edited",
-        details: `Edited page (${editCount} edit${editCount > 1 ? "s" : ""}) and added ${userComments.length} comment${userComments.length > 1 ? "s" : ""}`
+        details: `Edited page (v${page.version?.number || 1}) and added ${userComments.length} comment${userComments.length > 1 ? "s" : ""}`
       };
     }
     if (hasEdits) {
-      const editCount = userVersions?.length || 1;
       return {
         type: "edited",
-        details: `Edited page (${editCount} edit${editCount > 1 ? "s" : ""})`
+        details: `Edited page to version ${page.version?.number || 1}`
       };
     }
     if (hasComments) {
@@ -4350,31 +4178,6 @@ var ConfluenceService = class {
     };
     return Math.ceil(baseSize * multipliers[contributionType]);
   }
-  /**
-   * Summarize the user's version edits into human-readable lines.
-   * This enriches the brag message so the AI refinement can generate a more specific brag.
-   */
-  summarizeUserVersions(userVersions) {
-    const MAX_ENTRIES = 5;
-    const versionsWithMessages = userVersions.filter((v) => v.message && v.message.trim());
-    if (versionsWithMessages.length > 0) {
-      const lines = versionsWithMessages.slice(0, MAX_ENTRIES).map((v) => {
-        const suffix = v.minorEdit ? " (minor edit)" : "";
-        return `- v${v.number}: ${v.message.trim()}${suffix}`;
-      });
-      return `Edit notes:
-${lines.join("\n")}`;
-    }
-    if (userVersions.length > 0) {
-      const major = userVersions.filter((v) => !v.minorEdit).length;
-      const minor = userVersions.filter((v) => v.minorEdit).length;
-      const parts = [];
-      if (major > 0) parts.push(`${major} major`);
-      if (minor > 0) parts.push(`${minor} minor`);
-      return `Made ${userVersions.length} edit${userVersions.length > 1 ? "s" : ""} to this page (${parts.join(", ")})`;
-    }
-    return "";
-  }
   /**
    * Fetch pages with optional filtering
    */
@@ -4445,7 +4248,14 @@ ${lines.join("\n")}`;
           break;
         }
         if (options.limit && allPages.length >= options.limit) {
-          break;
+          const email2 = await this.getCurrentUser();
+          const limitedPages = allPages.slice(0, options.limit);
+          const commits2 = [];
+          for (const page of limitedPages) {
+            const commit = await this.transformPageToCommit(page, void 0, email2 || void 0);
+            commits2.push(commit);
+          }
+          return commits2;
         }
         start += limit;
       } catch (error) {
@@ -4469,23 +4279,9 @@ ${lines.join("\n")}`;
         throw error;
       }
     }
-    const pagesToProcess = options.limit ? allPages.slice(0, options.limit) : allPages;
     const email = await this.getCurrentUser();
-    const sinceDate = options.days ? new Date(Date.now() - options.days * 24 * 60 * 60 * 1e3) : void 0;
-    if (sinceDate && email) {
-      const filtered = await this.filterPagesByUserContribution(pagesToProcess, email, sinceDate);
-      logger.debug(
-        `Date-scoped filtering: ${pagesToProcess.length} pages -> ${filtered.length} with user contributions in range`
-      );
-      const commits2 = [];
-      for (const { page, userVersions } of filtered) {
-        const commit = await this.transformPageToCommit(page, void 0, email, userVersions);
-        commits2.push(commit);
-      }
-      return commits2;
-    }
     const commits = [];
-    for (const page of pagesToProcess) {
+    for (const page of allPages) {
       const commit = await this.transformPageToCommit(page, void 0, email || void 0);
       commits.push(commit);
     }
@@ -4507,10 +4303,10 @@ ${lines.join("\n")}`;
   /**
    * Transform Confluence page to GitCommit format with contribution-specific data
    */
-  async transformPageToCommit(page, instanceUrl, userEmail, userVersions) {
+  async transformPageToCommit(page, instanceUrl, userEmail) {
     let contribution = null;
     if (userEmail) {
-      contribution = await this.determineContributionType(page, userEmail, userVersions);
+      contribution = await this.determineContributionType(page, userEmail);
     }
     let message;
     let contributionPrefix = "";
@@ -4531,14 +4327,6 @@ ${contribution.details}
 
 [Confluence Page v${page.version?.number || 1}]`;
     }
-    if (userVersions && userVersions.length > 0) {
-      const versionSummary = this.summarizeUserVersions(userVersions);
-      if (versionSummary) {
-        message += `
-
-${versionSummary}`;
-      }
-    }
     let baseUrl = "https://confluence.atlassian.net";
     if (instanceUrl) {
       baseUrl = instanceUrl.startsWith("http") ? instanceUrl : `https://${instanceUrl}`;
@@ -4556,14 +4344,7 @@ ${versionSummary}`;
     const impactScore = contribution ? this.calculateContributionImpact(contribution.type, baseSize) : baseSize;
     const author = page.history?.createdBy?.displayName || page.version?.by?.displayName || "Unknown Author";
     const authorEmail = page.history?.createdBy?.email || page.version?.by?.email || "unknown@example.com";
-    let date;
-    if (contribution?.type === "created") {
-      date = page.history?.createdDate || page.version?.when;
-    } else if (userVersions && userVersions.length > 0) {
-      date = userVersions[userVersions.length - 1].when;
-    } else {
-      date = page.version?.when || (/* @__PURE__ */ new Date()).toISOString();
-    }
+    const date = contribution?.type === "created" ? page.history?.createdDate || page.version?.when : page.version?.when || (/* @__PURE__ */ new Date()).toISOString();
     return {
       sha: page.id,
       message,