@bragduck/cli 2.2.0 → 2.3.2

package/dist/bin/bragduck.js

@@ -1166,11 +1166,19 @@ var init_api_service = __esm({
        * Refine brags using AI
        */
       async refineBrags(request) {
-        logger.debug(`Refining ${request.brags.length} brags`);
+        const MAX_TEXT_LENGTH = 1e4;
+        const trimmedRequest = {
+          ...request,
+          brags: request.brags.map((brag) => ({
+            ...brag,
+            text: brag.text && brag.text.length > MAX_TEXT_LENGTH ? brag.text.substring(0, MAX_TEXT_LENGTH) : brag.text
+          }))
+        };
+        logger.debug(`Refining ${trimmedRequest.brags.length} brags`);
         try {
           const response = await this.makeRequest(API_ENDPOINTS.BRAGS.REFINE, {
             method: "POST",
-            body: request
+            body: trimmedRequest
           });
           logger.debug(`Successfully refined ${response.refined_brags.length} brags`);
           return response;
@@ -2161,12 +2169,12 @@ function formatRefinedCommitsTable(brags, selectedCommits) {
   });
   brags.forEach((brag, index) => {
     const isNewBragType = !("sha" in brag);
-    let displaySha;
+    let displaySha = `#${index + 1}`;
     if (isNewBragType && selectedCommits) {
       const commit = selectedCommits[index];
-      if (commit.sha.startsWith("pr-")) {
+      if (commit && commit.sha.startsWith("pr-")) {
         displaySha = commit.sha.replace("pr-", "#");
-      } else {
+      } else if (commit) {
         displaySha = commit.sha.substring(0, 7);
       }
     } else if (!isNewBragType) {
@@ -2312,12 +2320,14 @@ async function promptReviewBrags(refinedBrags, selectedCommits) {
     let reviewingBrag = true;
     while (reviewingBrag) {
       const isNewBragType = !("sha" in currentBrag);
-      let displaySha;
+      let displaySha = `#${i + 1}`;
       let prUrl;
       if (isNewBragType && selectedCommits) {
         const commit = selectedCommits[i];
-        displaySha = commit.sha.startsWith("pr-") ? commit.sha.replace("pr-", "#") : commit.sha.substring(0, 7);
-        prUrl = commit.url;
+        if (commit) {
+          displaySha = commit.sha.startsWith("pr-") ? commit.sha.replace("pr-", "#") : commit.sha.substring(0, 7);
+          prUrl = commit.url;
+        }
       } else if (!isNewBragType) {
         const commit = currentBrag;
         displaySha = commit.sha.startsWith("pr-") ? commit.sha.replace("pr-", "#") : commit.sha.substring(0, 7);
@@ -2579,23 +2589,25 @@ async function scanCommand(options = {}) {
     logger.log("");
     logger.log(formatCommitStats(commits));
     logger.log("");
-    const sortOption = await promptSortOption();
-    logger.log("");
     let sortedCommits = [...commits];
-    if (sortOption === "date") {
-      sortedCommits.sort((a, b) => new Date(b.date).getTime() - new Date(a.date).getTime());
-    } else if (sortOption === "size") {
-      sortedCommits.sort((a, b) => {
-        const sizeA = (a.diffStats?.insertions || 0) + (a.diffStats?.deletions || 0);
-        const sizeB = (b.diffStats?.insertions || 0) + (b.diffStats?.deletions || 0);
-        return sizeB - sizeA;
-      });
-    } else if (sortOption === "files") {
-      sortedCommits.sort((a, b) => {
-        const filesA = a.diffStats?.filesChanged || 0;
-        const filesB = b.diffStats?.filesChanged || 0;
-        return filesB - filesA;
-      });
+    if (commits.length > 1) {
+      const sortOption = await promptSortOption();
+      logger.log("");
+      if (sortOption === "date") {
+        sortedCommits.sort((a, b) => new Date(b.date).getTime() - new Date(a.date).getTime());
+      } else if (sortOption === "size") {
+        sortedCommits.sort((a, b) => {
+          const sizeA = (a.diffStats?.insertions || 0) + (a.diffStats?.deletions || 0);
+          const sizeB = (b.diffStats?.insertions || 0) + (b.diffStats?.deletions || 0);
+          return sizeB - sizeA;
+        });
+      } else if (sortOption === "files") {
+        sortedCommits.sort((a, b) => {
+          const filesA = a.diffStats?.filesChanged || 0;
+          const filesB = b.diffStats?.filesChanged || 0;
+          return filesB - filesA;
+        });
+      }
     }
     const selectedShas = await promptSelectCommits(sortedCommits);
     if (selectedShas.length === 0) {
@@ -2607,14 +2619,38 @@ async function scanCommand(options = {}) {
     const selectedCommits = sortedCommits.filter((c) => selectedShas.includes(c.sha));
     logger.log(formatSelectionSummary(selectedCommits.length, selectedCommits));
     logger.log("");
+    const existingBrags = await apiService.listBrags({ limit: 100 });
+    logger.debug(`Fetched ${existingBrags.brags.length} existing brags`);
+    const existingUrls = new Set(existingBrags.brags.flatMap((b) => b.attachments || []));
+    logger.debug(`Existing PR URLs in attachments: ${existingUrls.size}`);
+    const duplicates = selectedCommits.filter((c) => c.url && existingUrls.has(c.url));
+    const newCommits = selectedCommits.filter((c) => !c.url || !existingUrls.has(c.url));
+    logger.debug(`Duplicates: ${duplicates.length}, New: ${newCommits.length}`);
+    if (duplicates.length > 0) {
+      logger.log("");
+      logger.info(
+        colors.warning(
+          `${duplicates.length} PR${duplicates.length > 1 ? "s" : ""} already added to Bragduck - skipping`
+        )
+      );
+      logger.log("");
+    }
+    if (newCommits.length === 0) {
+      logger.log("");
+      logger.info(
+        theme.secondary("All selected PRs already exist in Bragduck. Nothing to refine.")
+      );
+      logger.log("");
+      return;
+    }
     const refineSpinner = createStepSpinner(
       3,
       TOTAL_STEPS,
-      `Refining ${theme.count(selectedCommits.length)} PR${selectedCommits.length > 1 ? "s" : ""} with AI`
+      `Refining ${theme.count(newCommits.length)} PR${newCommits.length > 1 ? "s" : ""} with AI`
     );
     refineSpinner.start();
     const refineRequest = {
-      brags: selectedCommits.map((c) => ({
+      brags: newCommits.map((c) => ({
         text: c.message,
         date: c.date,
         title: c.message.split("\n")[0]
@@ -2627,9 +2663,9 @@ async function scanCommand(options = {}) {
     logger.log("");
     logger.info("Preview of refined brags:");
     logger.log("");
-    logger.log(formatRefinedCommitsTable(refinedBrags, selectedCommits));
+    logger.log(formatRefinedCommitsTable(refinedBrags, newCommits));
     logger.log("");
-    const acceptedBrags = await promptReviewBrags(refinedBrags, selectedCommits);
+    const acceptedBrags = await promptReviewBrags(refinedBrags, newCommits);
     if (acceptedBrags.length === 0) {
       logger.log("");
       logger.info(theme.secondary("No brags selected for creation. Scan cancelled."));
@@ -2645,7 +2681,7 @@ async function scanCommand(options = {}) {
     createSpinner2.start();
     const createRequest = {
       brags: acceptedBrags.map((refined, index) => {
-        const originalCommit = selectedCommits[index];
+        const originalCommit = newCommits[index];
         return {
           commit_sha: originalCommit?.sha || `brag-${index}`,
           title: refined.refined_title,
@@ -2655,7 +2691,8 @@ async function scanCommand(options = {}) {
           date: refined.date,
           commit_url: originalCommit?.url || "",
           impact_score: refined.suggested_impactLevel,
-          impact_description: refined.impact_description
+          impact_description: refined.impact_description,
+          attachments: originalCommit?.url ? [originalCommit.url] : []
         };
       })
     };