@bragduck/cli 2.37.0 → 2.38.1

package/dist/bin/bragduck.js

@@ -4275,6 +4275,13 @@ init_errors();
 init_logger();
 init_storage_service();
 init_constants();
+function adfToText(node) {
+  if (!node) return "";
+  if (typeof node === "string") return node;
+  if (node.text) return node.text;
+  if (node.content) return node.content.map(adfToText).join("");
+  return "";
+}
 var JiraService = class {
   MAX_DESCRIPTION_LENGTH = 5e3;
   /**
@@ -4469,9 +4476,10 @@ var JiraService = class {
    * Determine the type of contribution the current user made to an issue
    * Returns: 'created' | 'reported' | 'assigned-resolved' | 'edited'
    */
-  async determineJiraContributionType(issue, userEmail) {
-    const isCreator = issue.fields.creator.emailAddress === userEmail;
-    const isReporter = issue.fields.reporter?.emailAddress === userEmail;
+  async determineJiraContributionType(issue, userIdentifier) {
+    const matchesUser = (person) => !!person && (person.accountId === userIdentifier || person.emailAddress === userIdentifier);
+    const isCreator = matchesUser(issue.fields.creator);
+    const isReporter = matchesUser(issue.fields.reporter);
     if (isCreator) {
       return {
         type: "created",
@@ -4484,9 +4492,9 @@ var JiraService = class {
         details: `Reported ${issue.fields.issuetype.name}: ${issue.fields.status.name}`
       };
     }
-    const isAssigned = issue.fields.assignee?.emailAddress === userEmail;
+    const isAssigned = matchesUser(issue.fields.assignee);
     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) => matchesUser(history.author)) || [];
     const hasEdits = userEdits.length > 0;
     if (isAssigned && isResolved) {
       return {
@@ -4558,7 +4566,7 @@ var JiraService = class {
         );
         break;
       }
-      const endpoint = `/rest/api/3/search/jql?jql=${encodeURIComponent(jql)}&startAt=${startAt}&maxResults=${maxResults}&fields=${fields.join(",")}`;
+      const endpoint = `/rest/api/3/search?jql=${encodeURIComponent(jql)}&startAt=${startAt}&maxResults=${maxResults}&fields=${fields.join(",")}`;
       try {
         const response = await this.request(endpoint);
         if (response.issues.length === 0) {
@@ -4663,7 +4671,8 @@ var JiraService = class {
 
 ${contribution.details}`;
       if (contribution.type === "created" && issue.fields.description) {
-        const truncatedDesc = issue.fields.description.substring(0, this.MAX_DESCRIPTION_LENGTH);
+        const plainDesc = adfToText(issue.fields.description);
+        const truncatedDesc = plainDesc.substring(0, this.MAX_DESCRIPTION_LENGTH);
         message = `${contributionPrefix}${issue.fields.summary}
 
 ${truncatedDesc}
@@ -4673,7 +4682,8 @@ ${contribution.details}`;
     } else {
       message = issue.fields.summary;
       if (issue.fields.description) {
-        const truncatedDesc = issue.fields.description.substring(0, this.MAX_DESCRIPTION_LENGTH);
+        const plainDesc = adfToText(issue.fields.description);
+        const truncatedDesc = plainDesc.substring(0, this.MAX_DESCRIPTION_LENGTH);
         message = `${issue.fields.summary}
 
 ${truncatedDesc}`;
@@ -4921,16 +4931,13 @@ var ConfluenceService = class {
   }
   /**
    * Build CQL query from options
-   * Returns empty string if no filters need CQL (will use simple endpoint instead)
+   * Always returns a CQL string with at least a user filter so we never return all pages.
    */
   buildCQL(options) {
-    if (!options.days && !options.author && !options.cql) {
-      return "";
-    }
     const queries = [];
     queries.push("type = page");
     if (options.days) {
-      queries.push(`lastModified > startOfDay("-${options.days}d")`);
+      queries.push(`lastmodified > startOfDay("-${options.days}d")`);
     }
     if (options.author) {
       const escapedAuthor = options.author.replace(/"/g, '\\"');
@@ -4954,17 +4961,16 @@ var ConfluenceService = class {
    * Determine the type of contribution the current user made to a page
    * Returns: 'created' | 'edited' | 'commented'
    */
-  async determineContributionType(page, userEmail) {
-    if (page.history?.createdBy?.email === userEmail) {
+  async determineContributionType(page, userIdentifier) {
+    const matchesUser = (by) => !!by && (by.accountId === userIdentifier || by.email === userIdentifier);
+    if (matchesUser(page.history?.createdBy)) {
       return {
         type: "created",
         details: `Created page with ${page.version?.number || 1} version${(page.version?.number || 1) > 1 ? "s" : ""}`
       };
     }
-    const hasEdits = page.version?.by?.email === userEmail && (page.version?.number || 0) > 1;
-    const userComments = page.children?.comment?.results?.filter(
-      (comment) => comment.version?.by?.email === userEmail
-    ) || [];
+    const hasEdits = matchesUser(page.version?.by) && (page.version?.number || 0) > 1;
+    const userComments = page.children?.comment?.results?.filter((comment) => matchesUser(comment.version?.by)) || [];
     const hasComments = userComments.length > 0;
     if (hasEdits && hasComments) {
       return {
@@ -5008,9 +5014,7 @@ var ConfluenceService = class {
    */
   async getPages(options = {}) {
     const cql = this.buildCQL(options);
-    if (cql) {
-      logger.debug(`Using CQL query: ${cql}`);
-    }
+    logger.debug(`Using CQL query: ${cql}`);
     const allPages = [];
     const seenPageIds = /* @__PURE__ */ new Set();
     let start = 0;
@@ -5025,29 +5029,14 @@ var ConfluenceService = class {
         );
         break;
       }
-      let endpoint;
-      let params;
-      if (cql) {
-        params = {
-          cql,
-          start: start.toString(),
-          limit: limit.toString(),
-          expand: "version,history,children.comment,body.storage"
-        };
-        const queryString = Object.entries(params).map(([key, value]) => `${encodeURIComponent(key)}=${encodeURIComponent(value)}`).join("&");
-        endpoint = `/wiki/rest/api/content/search?${queryString}`;
-      } else {
-        logger.debug("Using simple content list (no CQL filters)");
-        params = {
-          type: "page",
-          status: "current",
-          start: start.toString(),
-          limit: limit.toString(),
-          expand: "version,history,children.comment,body.storage"
-        };
-        const queryString = Object.entries(params).map(([key, value]) => `${encodeURIComponent(key)}=${encodeURIComponent(value)}`).join("&");
-        endpoint = `/wiki/rest/api/content?${queryString}`;
-      }
+      const params = {
+        cql,
+        start: start.toString(),
+        limit: limit.toString(),
+        expand: "version,history,children.comment,body.storage"
+      };
+      const queryString = Object.entries(params).map(([key, value]) => `${encodeURIComponent(key)}=${encodeURIComponent(value)}`).join("&");
+      const endpoint = `/wiki/rest/api/content/search?${queryString}`;
       try {
         const response = await this.request(endpoint);
         if (response.results.length === 0) {
@@ -5892,6 +5881,10 @@ function formatSuccessMessage(count, brags) {
       const dateFormatted = formatDate(brag.date);
       bragsList += `
   ${colors.dim("\u2022")} ${colors.white(brag.title)} ${theme.secondary(`(${dateFormatted} \xB7 ${sourceLabel})`)}`;
+      if (brag.url) {
+        bragsList += `
+    ${colors.info(brag.url)}`;
+      }
     }
   }
   const hint = theme.secondary("\n\nRun ") + theme.command("bragduck list") + theme.secondary(" to see all your brags");
@@ -6977,7 +6970,7 @@ async function syncSingleService(sourceType, options, TOTAL_STEPS, sharedDays, s
         typeKey: refined.suggested_type_key,
         impactDescription: refined.impact_description,
         impact_description: refined.impact_description,
-        attachments: originalCommit?.url ? [originalCommit.url] : [],
+        attachments: [],
         orgId: selectedOrgId || void 0,
         // External fields for non-git sources (Jira, Confluence, etc.)
         externalId: originalCommit?.externalId,
@@ -7016,7 +7009,8 @@ async function syncSingleService(sourceType, options, TOTAL_STEPS, sharedDays, s
   const createdBrags = createResponse.brags.map((brag, index) => ({
     title: brag.title,
     date: selectedCommits[index]?.date || (/* @__PURE__ */ new Date()).toISOString(),
-    source: sourceType
+    source: sourceType,
+    url: brag.url
   }));
   return { created: createResponse.created, skipped: duplicates.length, createdBrags };
 }