mobbdev 1.2.21 → 1.2.24

package/dist/args/commands/upload_ai_blame.mjs

@@ -881,6 +881,7 @@ var init_client_generates = __esm({
     ... on ProcessAIBlameRequestedResult {
       status
       requestIds
+      commitBlameRequestIds
     }
   }
 }
@@ -2571,11 +2572,20 @@ var init_urlParser = __esm({
   }
 });
 
+// src/features/analysis/scm/shared/src/urlParser/repoSlug.ts
+var init_repoSlug = __esm({
+  "src/features/analysis/scm/shared/src/urlParser/repoSlug.ts"() {
+    "use strict";
+    init_urlParser();
+  }
+});
+
 // src/features/analysis/scm/shared/src/urlParser/index.ts
 var init_urlParser2 = __esm({
   "src/features/analysis/scm/shared/src/urlParser/index.ts"() {
     "use strict";
     init_types2();
+    init_repoSlug();
     init_urlParser();
   }
 });
@@ -4210,7 +4220,9 @@ function subscribeStream(query, variables, handlers, wsClientOptions) {
         finalizeError(error);
       },
       complete: () => {
-        return;
+        finalizeError(
+          new Error("Subscription completed by server before resolving")
+        );
       }
     }
   );
@@ -6146,9 +6158,6 @@ var GetReferenceResultZ = z14.object({
   type: z14.nativeEnum(ReferenceType)
 });
 
-// src/features/analysis/scm/utils/diffUtils.ts
-import parseDiff from "parse-diff";
-
 // src/features/analysis/scm/ado/constants.ts
 init_types2();
 var DEFUALT_ADO_ORIGIN = scmCloudUrl.Ado;
@@ -6249,8 +6258,6 @@ var REPORT_DEFAULT_FILE_NAME = "report.json";
 init_env();
 
 // src/features/analysis/scm/github/GithubSCMLib.ts
-init_env();
-import pLimit from "p-limit";
 import { z as z22 } from "zod";
 init_client_generates();
 
@@ -6274,7 +6281,7 @@ import {
   Gitlab
 } from "@gitbeaker/rest";
 import Debug4 from "debug";
-import pLimit2 from "p-limit";
+import pLimit from "p-limit";
 import {
   Agent,
   fetch as undiciFetch,

package/dist/index.mjs

@@ -881,6 +881,7 @@ var init_client_generates = __esm({
     ... on ProcessAIBlameRequestedResult {
       status
       requestIds
+      commitBlameRequestIds
     }
   }
 }
@@ -2620,11 +2621,20 @@ var init_urlParser = __esm({
   }
 });
 
+// src/features/analysis/scm/shared/src/urlParser/repoSlug.ts
+var init_repoSlug = __esm({
+  "src/features/analysis/scm/shared/src/urlParser/repoSlug.ts"() {
+    "use strict";
+    init_urlParser();
+  }
+});
+
 // src/features/analysis/scm/shared/src/urlParser/index.ts
 var init_urlParser2 = __esm({
   "src/features/analysis/scm/shared/src/urlParser/index.ts"() {
     "use strict";
     init_types2();
+    init_repoSlug();
     init_urlParser();
   }
 });
@@ -6109,34 +6119,6 @@ var GetReferenceResultZ = z13.object({
   type: z13.nativeEnum(ReferenceType)
 });
 
-// src/features/analysis/scm/utils/diffUtils.ts
-import parseDiff from "parse-diff";
-function parseAddedLinesByFile(diff) {
-  const result = /* @__PURE__ */ new Map();
-  const parsedDiff = parseDiff(diff);
-  for (const file of parsedDiff) {
-    if (!file.to || file.to === "/dev/null") {
-      continue;
-    }
-    const filePath = file.to;
-    const addedLines = [];
-    if (file.chunks) {
-      for (const chunk of file.chunks) {
-        for (const change of chunk.changes) {
-          if (change.type === "add") {
-            const addChange = change;
-            addedLines.push(addChange.ln);
-          }
-        }
-      }
-    }
-    if (addedLines.length > 0) {
-      result.set(filePath, addedLines);
-    }
-  }
-  return result;
-}
-
 // src/features/analysis/scm/utils/scm.ts
 var safeBody = (body, maxBodyLength) => {
   const truncationNotice = "\n\n... Message was cut here because it is too long";
@@ -7101,28 +7083,16 @@ var SCMLib = class {
    *
    * @param repoUrl - Repository URL
    * @param prNumbers - Array of PR numbers to fetch commits for
-   * @returns Map of PR number to array of commit SHAs
+   * @returns Map of PR number to array of commit SHAs in chronological order
+   *          (oldest first, HEAD commit last)
    */
   async getPrCommitsBatch(_repoUrl, _prNumbers) {
     throw new Error("getPrCommitsBatch not implemented for this SCM provider");
   }
-  /**
-   * Fetches additions and deletions counts for multiple PRs in batch.
-   * More efficient than fetching individual PR details.
-   *
-   * @param repoUrl - Repository URL
-   * @param prNumbers - Array of PR numbers to fetch metrics for
-   * @returns Map of PR number to additions/deletions count
-   */
-  async getPrAdditionsDeletionsBatch(_repoUrl, _prNumbers) {
-    throw new Error(
-      "getPrAdditionsDeletionsBatch not implemented for this SCM provider"
-    );
-  }
   /**
    * Batch fetch PR data (additions/deletions + comments) for multiple PRs.
    * Only implemented for GitHub (via GraphQL). Other providers should override if supported.
-   * This is more efficient than calling getPrAdditionsDeletionsBatch separately.
+   * This is more efficient than calling separate batch methods.
    *
    * @param _repoUrl - Repository URL
    * @param _prNumbers - Array of PR numbers to fetch data for
@@ -7364,11 +7334,8 @@ var AdoSCMLib = class extends SCMLib {
       markdownComment: comment
     });
   }
-  async getCommitDiff(_commitSha) {
-    throw new Error("getCommitDiff not implemented for ADO");
-  }
-  async getSubmitRequestDiff(_submitRequestId) {
-    throw new Error("getSubmitRequestDiff not implemented for ADO");
+  async getSubmitRequestMetadata(_submitRequestId) {
+    throw new Error("getSubmitRequestMetadata not implemented for ADO");
   }
   async searchSubmitRequests(_params) {
     throw new Error("searchSubmitRequests not implemented for ADO");
@@ -7946,11 +7913,8 @@ var BitbucketSCMLib = class extends SCMLib {
       markdownComment: comment
     });
   }
-  async getCommitDiff(_commitSha) {
-    throw new Error("getCommitDiff not implemented for Bitbucket");
-  }
-  async getSubmitRequestDiff(_submitRequestId) {
-    throw new Error("getSubmitRequestDiff not implemented for Bitbucket");
+  async getSubmitRequestMetadata(_submitRequestId) {
+    throw new Error("getSubmitRequestMetadata not implemented for Bitbucket");
   }
   async searchSubmitRequests(_params) {
     throw new Error("searchSubmitRequests not implemented for Bitbucket");
@@ -7980,8 +7944,6 @@ var REPORT_DEFAULT_FILE_NAME = "report.json";
 init_env();
 
 // src/features/analysis/scm/github/GithubSCMLib.ts
-init_env();
-import pLimit from "p-limit";
 import { z as z21 } from "zod";
 init_client_generates();
 
@@ -8054,18 +8016,6 @@ var GET_PR_METRICS_QUERY = `
         mergedAt
         additions
         deletions
-        commits(first: 100) {
-          totalCount
-          nodes {
-            commit {
-              oid
-              committedDate
-              author {
-                date
-              }
-            }
-          }
-        }
         comments(first: 100) {
           totalCount
           nodes {
@@ -8506,33 +8456,6 @@ function getGithubSdk(params = {}) {
         commit_sha: commitSha
       });
     },
-    async getCommitWithDiff({
-      commitSha,
-      owner,
-      repo
-    }) {
-      const [commitData, diffData] = await Promise.all([
-        // Get commit metadata
-        octokit.rest.repos.getCommit({
-          repo,
-          owner,
-          ref: commitSha
-        }),
-        // Get commit diff
-        octokit.request("GET /repos/{owner}/{repo}/commits/{ref}", {
-          owner,
-          repo,
-          ref: commitSha,
-          headers: {
-            Accept: "application/vnd.github.v3.diff"
-          }
-        })
-      ]);
-      return {
-        commit: commitData.data,
-        diff: diffData.data
-      };
-    },
     async getTagDate({
       tag,
       owner,
@@ -8716,14 +8639,6 @@ function getGithubSdk(params = {}) {
     async getUserInfo() {
       return octokit.request(GET_USER);
     },
-    async getPrCommits(params2) {
-      const data = await octokit.paginate(octokit.rest.pulls.listCommits, {
-        owner: params2.owner,
-        repo: params2.repo,
-        pull_number: params2.pull_number
-      });
-      return { data };
-    },
     async getUserRepos() {
       return octokit.rest.repos.listForAuthenticatedUser({
         visibility: "all",
@@ -8767,30 +8682,6 @@ function getGithubSdk(params = {}) {
       });
       return { data };
     },
-    /**
-     * Batch fetch additions/deletions for multiple PRs via GraphQL.
-     * Uses GITHUB_GRAPHQL_FRAGMENTS.PR_CHANGES for the field selection.
-     */
-    async getPrAdditionsDeletionsBatch(params2) {
-      return executeBatchGraphQL(octokit, params2.owner, params2.repo, {
-        items: params2.prNumbers,
-        aliasPrefix: "pr",
-        buildFragment: (prNumber, index) => `
-          pr${index}: pullRequest(number: ${prNumber}) {
-            ${GITHUB_GRAPHQL_FRAGMENTS.PR_CHANGES}
-          }`,
-        extractResult: (data) => {
-          const prData = data;
-          if (prData.additions !== void 0 && prData.deletions !== void 0) {
-            return {
-              additions: prData.additions,
-              deletions: prData.deletions
-            };
-          }
-          return void 0;
-        }
-      });
-    },
     /**
      * Batch fetch comments for multiple PRs via GraphQL.
      * Uses GITHUB_GRAPHQL_FRAGMENTS.PR_COMMENTS for the field selection.
@@ -8818,7 +8709,7 @@ function getGithubSdk(params = {}) {
     /**
      * Batch fetch PR data (additions/deletions + comments) for multiple PRs via GraphQL.
      * Combines PR_CHANGES and PR_COMMENTS fragments into a single API call for efficiency.
-     * This is more efficient than calling getPrAdditionsDeletionsBatch and getPrCommentsBatch separately.
+     * This is more efficient than calling separate batch methods.
      */
     async getPrDataBatch(params2) {
       return executeBatchGraphQL(octokit, params2.owner, params2.repo, {
@@ -9209,82 +9100,21 @@ var GithubSCMLib = class _GithubSCMLib extends SCMLib {
       comment_id: commentId
     });
   }
-  async getCommitDiff(commitSha, options) {
+  async getSubmitRequestMetadata(submitRequestId) {
     this._validateAccessTokenAndUrl();
     const { owner, repo } = parseGithubOwnerAndRepo(this.url);
-    const { commit, diff } = await this.githubSdk.getCommitWithDiff({
+    const prNumber = Number(submitRequestId);
+    const prRes = await this.githubSdk.getPr({
       owner,
       repo,
-      commitSha
+      pull_number: prNumber
     });
-    const commitTimestamp = commit.commit.committer?.date ? new Date(commit.commit.committer.date) : new Date(commit.commit.author?.date || Date.now());
-    let repositoryCreatedAt = options?.repositoryCreatedAt;
-    if (repositoryCreatedAt === void 0) {
-      try {
-        const repoData = await this.githubSdk.getRepository({ owner, repo });
-        repositoryCreatedAt = repoData.data.created_at ? new Date(repoData.data.created_at) : void 0;
-      } catch (error) {
-        console.error("Failed to fetch repository creation date", {
-          error,
-          owner,
-          repo
-        });
-        repositoryCreatedAt = void 0;
-      }
-    }
-    return {
-      diff,
-      commitTimestamp,
-      commitSha: commit.sha,
-      authorName: commit.commit.author?.name,
-      authorEmail: commit.commit.author?.email,
-      message: commit.commit.message,
-      repositoryCreatedAt
-    };
-  }
-  async getSubmitRequestDiff(submitRequestId) {
-    this._validateAccessTokenAndUrl();
-    const { owner, repo } = parseGithubOwnerAndRepo(this.url);
-    const prNumber = Number(submitRequestId);
-    const [prRes, commitsRes, repoData, prDiff] = await Promise.all([
-      this.githubSdk.getPr({ owner, repo, pull_number: prNumber }),
-      this.githubSdk.getPrCommits({ owner, repo, pull_number: prNumber }),
-      this.githubSdk.getRepository({ owner, repo }),
-      this.getPrDiff({ pull_number: prNumber })
-    ]);
     const pr = prRes.data;
-    const repositoryCreatedAt = repoData.data.created_at ? new Date(repoData.data.created_at) : void 0;
-    const limit = pLimit(GITHUB_API_CONCURRENCY);
-    const commits = await Promise.all(
-      commitsRes.data.map(
-        (commit) => limit(
-          () => this.getCommitDiff(commit.sha, {
-            repositoryCreatedAt
-          })
-        )
-      )
-    );
-    const addedLinesByFile = parseAddedLinesByFile(prDiff);
-    const diffLines = [];
-    for (const [file, lines] of addedLinesByFile) {
-      for (const line of lines) {
-        diffLines.push({ file, line });
-      }
-    }
     return {
-      diff: prDiff,
-      createdAt: new Date(pr.created_at),
-      updatedAt: new Date(pr.updated_at),
-      submitRequestId,
-      submitRequestNumber: prNumber,
-      sourceBranch: pr.head.ref,
-      targetBranch: pr.base.ref,
-      authorName: pr.user?.name || pr.user?.login,
-      authorEmail: pr.user?.email || void 0,
       title: pr.title,
-      description: pr.body || void 0,
-      commits,
-      diffLines
+      targetBranch: pr.base.ref,
+      sourceBranch: pr.head.ref,
+      headCommitSha: pr.head.sha
     };
   }
   /**
@@ -9342,7 +9172,7 @@ var GithubSCMLib = class _GithubSCMLib extends SCMLib {
   }
   /**
    * Fetches commits for multiple PRs in a single GraphQL request.
-   * Much more efficient than calling getSubmitRequestDiff for each PR.
+   * Much more efficient than fetching commits for each PR individually.
    *
    * @param repoUrl - Repository URL
    * @param prNumbers - Array of PR numbers to fetch commits for
@@ -9353,23 +9183,6 @@ var GithubSCMLib = class _GithubSCMLib extends SCMLib {
     const { owner, repo } = parseGithubOwnerAndRepo(repoUrl);
     return this.githubSdk.getPrCommitsBatch({ owner, repo, prNumbers });
   }
-  /**
-   * Fetches additions and deletions counts for multiple PRs in a single GraphQL request.
-   * Used to enrich search results with changed lines data.
-   *
-   * @param repoUrl - Repository URL
-   * @param prNumbers - Array of PR numbers to fetch metrics for
-   * @returns Map of PR number to additions/deletions count
-   */
-  async getPrAdditionsDeletionsBatch(repoUrl, prNumbers) {
-    this._validateAccessToken();
-    const { owner, repo } = parseGithubOwnerAndRepo(repoUrl);
-    return this.githubSdk.getPrAdditionsDeletionsBatch({
-      owner,
-      repo,
-      prNumbers
-    });
-  }
   /**
    * Batch fetch PR data (additions/deletions + comments) for multiple PRs.
    * Combines both metrics into a single GraphQL call for efficiency.
@@ -9469,19 +9282,6 @@ var GithubSCMLib = class _GithubSCMLib extends SCMLib {
       throw new Error(`Pull request #${prNumber} not found`);
     }
     const prStatus = determinePrStatus(pr.state, pr.isDraft);
-    const firstCommit = pr.commits.nodes[0];
-    const firstCommitDate = firstCommit ? new Date(
-      firstCommit.commit.author?.date || firstCommit.commit.committedDate || pr.createdAt
-    ) : null;
-    let commitShas = pr.commits.nodes.map((node) => node.commit.oid);
-    if (pr.commits.totalCount > 100) {
-      const commitsRes = await this.githubSdk.getPrCommits({
-        owner,
-        repo,
-        pull_number: prNumber
-      });
-      commitShas = commitsRes.data.map((c) => c.sha);
-    }
     let commentIds = pr.comments.nodes.map((node) => node.id);
     if (pr.comments.totalCount > 100) {
       const commentsRes = await this.githubSdk.getGeneralPrComments({
@@ -9496,10 +9296,7 @@ var GithubSCMLib = class _GithubSCMLib extends SCMLib {
       repositoryUrl: this.url,
       prCreatedAt: new Date(pr.createdAt),
       prMergedAt: pr.mergedAt ? new Date(pr.mergedAt) : null,
-      firstCommitDate,
       linesAdded: pr.additions,
-      commitsCount: pr.commits.totalCount,
-      commitShas,
       prStatus,
       commentIds
     };
@@ -9537,7 +9334,7 @@ import {
   Gitlab
 } from "@gitbeaker/rest";
 import Debug3 from "debug";
-import pLimit2 from "p-limit";
+import pLimit from "p-limit";
 import {
   Agent,
   fetch as undiciFetch,
@@ -9619,14 +9416,6 @@ function removeTrailingSlash2(str) {
   return str.trim().replace(/\/+$/, "");
 }
 var MAX_GITLAB_PR_BODY_LENGTH = 1048576;
-function buildUnifiedDiff(diffs) {
-  return diffs.filter((d) => d.diff).map((d) => {
-    const oldPath = d.old_path || d.new_path || "";
-    const newPath = d.new_path || d.old_path || "";
-    return `diff --git a/${oldPath} b/${newPath}
-${d.diff}`;
-  }).join("\n");
-}
 function getRandomGitlabCloudAnonToken() {
   if (!GITLAB_API_TOKEN || typeof GITLAB_API_TOKEN !== "string") {
     return void 0;
@@ -9928,7 +9717,7 @@ async function getGitlabMrCommitsBatch({
     url: repoUrl,
     gitlabAuthToken: accessToken
   });
-  const limit = pLimit2(GITLAB_API_CONCURRENCY);
+  const limit = pLimit(GITLAB_API_CONCURRENCY);
   const results = await Promise.all(
     mrNumbers.map(
       (mrNumber) => limit(async () => {
@@ -9937,7 +9726,7 @@ async function getGitlabMrCommitsBatch({
             projectPath,
             mrNumber
           );
-          return [mrNumber, commits.map((c) => c.id)];
+          return [mrNumber, commits.map((c) => c.id).reverse()];
         } catch (error) {
           contextLogger.warn(
             "[getGitlabMrCommitsBatch] Failed to fetch commits for MR",
@@ -9967,28 +9756,12 @@ async function getGitlabMrDataBatch({
     url: repoUrl,
     gitlabAuthToken: accessToken
   });
-  const limit = pLimit2(GITLAB_API_CONCURRENCY);
+  const limit = pLimit(GITLAB_API_CONCURRENCY);
   const results = await Promise.all(
     mrNumbers.map(
       (mrNumber) => limit(async () => {
         try {
-          const [diffs, notes] = await Promise.all([
-            api2.MergeRequests.allDiffs(projectPath, mrNumber),
-            api2.MergeRequestNotes.all(projectPath, mrNumber)
-          ]);
-          let additions = 0;
-          let deletions = 0;
-          for (const diff of diffs) {
-            if (diff.diff) {
-              for (const line of diff.diff.split("\n")) {
-                if (line.startsWith("+") && !line.startsWith("+++")) {
-                  additions++;
-                } else if (line.startsWith("-") && !line.startsWith("---")) {
-                  deletions++;
-                }
-              }
-            }
-          }
+          const notes = await api2.MergeRequestNotes.all(projectPath, mrNumber);
           const comments = notes.map((note) => ({
             author: note.author ? {
               login: note.author.username,
@@ -9998,7 +9771,7 @@ async function getGitlabMrDataBatch({
           }));
           return [
             mrNumber,
-            { changedLines: { additions, deletions }, comments }
+            { changedLines: { additions: 0, deletions: 0 }, comments }
           ];
         } catch (error) {
           contextLogger.warn(
@@ -10022,39 +9795,6 @@ async function getGitlabMrDataBatch({
   );
   return new Map(results);
 }
-async function getGitlabMergeRequestLinesAdded({
-  url,
-  prNumber,
-  accessToken
-}) {
-  try {
-    const { projectPath } = parseGitlabOwnerAndRepo(url);
-    const api2 = getGitBeaker({
-      url,
-      gitlabAuthToken: accessToken
-    });
-    const diffs = await api2.MergeRequests.allDiffs(projectPath, prNumber);
-    let linesAdded = 0;
-    for (const diff of diffs) {
-      if (diff.diff) {
-        const addedLines = diff.diff.split("\n").filter(
-          (line) => line.startsWith("+") && !line.startsWith("+++")
-        ).length;
-        linesAdded += addedLines;
-      }
-    }
-    return linesAdded;
-  } catch (error) {
-    contextLogger.warn(
-      "[getGitlabMergeRequestLinesAdded] Failed to fetch diffs for MR",
-      {
-        prNumber,
-        error
-      }
-    );
-    return 0;
-  }
-}
 async function getGitlabMergeRequestMetrics({
   url,
   prNumber,
@@ -10065,26 +9805,17 @@ async function getGitlabMergeRequestMetrics({
     url,
     gitlabAuthToken: accessToken
   });
-  const [mr, commits, linesAdded, notes] = await Promise.all([
+  const [mr, notes] = await Promise.all([
     api2.MergeRequests.show(projectPath, prNumber),
-    api2.MergeRequests.allCommits(projectPath, prNumber),
-    getGitlabMergeRequestLinesAdded({ url, prNumber, accessToken }),
     api2.MergeRequestNotes.all(projectPath, prNumber)
   ]);
-  const sortedCommits = [...commits].sort(
-    (a, b) => new Date(a.created_at).getTime() - new Date(b.created_at).getTime()
-  );
-  const firstCommitDate = sortedCommits[0]?.created_at ?? null;
   const commentIds = notes.filter((note) => !note.system).map((note) => String(note.id));
   return {
     state: mr.state,
     isDraft: mr.draft ?? false,
     createdAt: mr.created_at,
     mergedAt: mr.merged_at ?? null,
-    linesAdded,
-    commitsCount: commits.length,
-    commitShas: commits.map((c) => c.id),
-    firstCommitDate,
+    linesAdded: 0,
     commentIds
   };
 }
@@ -10220,29 +9951,6 @@ async function getGitlabRecentCommits({
   }
   return allCommits;
 }
-async function getGitlabCommitDiff({
-  repoUrl,
-  accessToken,
-  commitSha
-}) {
-  const { projectPath } = parseGitlabOwnerAndRepo(repoUrl);
-  const api2 = getGitBeaker({ url: repoUrl, gitlabAuthToken: accessToken });
-  const [commit, diffs] = await Promise.all([
-    api2.Commits.show(projectPath, commitSha),
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    api2.Commits.showDiff(projectPath, commitSha, { unidiff: true })
-  ]);
-  const diffString = buildUnifiedDiff(diffs);
-  const commitTimestamp = commit.committed_date ? new Date(commit.committed_date) : /* @__PURE__ */ new Date();
-  return {
-    diff: diffString,
-    commitTimestamp,
-    commitSha: commit.id,
-    authorName: commit.author_name,
-    authorEmail: commit.author_email,
-    message: commit.message
-  };
-}
 async function getGitlabRateLimitStatus({
   repoUrl,
   accessToken
@@ -10324,88 +10032,6 @@ async function brokerRequestHandler(endpoint, options) {
     };
   throw new Error(`gitbeaker: ${response.statusText}`);
 }
-async function getGitlabMergeRequestDiff({
-  repoUrl,
-  accessToken,
-  mrNumber
-}) {
-  debug4("[getGitlabMergeRequestDiff] Starting for MR #%d", mrNumber);
-  const { projectPath } = parseGitlabOwnerAndRepo(repoUrl);
-  const api2 = getGitBeaker({ url: repoUrl, gitlabAuthToken: accessToken });
-  debug4(
-    "[getGitlabMergeRequestDiff] Fetching MR details, diffs, and commits..."
-  );
-  const startMrFetch = Date.now();
-  const [mr, mrDiffs, mrCommitsRaw] = await Promise.all([
-    api2.MergeRequests.show(projectPath, mrNumber),
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    api2.MergeRequests.allDiffs(projectPath, mrNumber, { unidiff: true }),
-    api2.MergeRequests.allCommits(projectPath, mrNumber)
-  ]);
-  debug4(
-    "[getGitlabMergeRequestDiff] MR fetch took %dms. Diffs: %d, Commits: %d",
-    Date.now() - startMrFetch,
-    mrDiffs.length,
-    mrCommitsRaw.length
-  );
-  const diffString = buildUnifiedDiff(mrDiffs);
-  debug4(
-    "[getGitlabMergeRequestDiff] Fetching commit diffs for %d commits...",
-    mrCommitsRaw.length
-  );
-  const startCommitFetch = Date.now();
-  const commitDiffLimit = pLimit2(GITLAB_API_CONCURRENCY);
-  const commits = await Promise.all(
-    mrCommitsRaw.map(
-      (commit) => commitDiffLimit(async () => {
-        const commitDiff = await getGitlabCommitDiff({
-          repoUrl,
-          accessToken,
-          commitSha: commit.id
-        });
-        return {
-          diff: commitDiff.diff,
-          commitTimestamp: commitDiff.commitTimestamp,
-          commitSha: commitDiff.commitSha,
-          authorName: commitDiff.authorName,
-          authorEmail: commitDiff.authorEmail,
-          message: commitDiff.message
-        };
-      })
-    )
-  );
-  commits.sort(
-    (a, b) => new Date(a.commitTimestamp).getTime() - new Date(b.commitTimestamp).getTime()
-  );
-  debug4(
-    "[getGitlabMergeRequestDiff] Commit diffs fetch took %dms",
-    Date.now() - startCommitFetch
-  );
-  const addedLinesByFile = parseAddedLinesByFile(diffString);
-  const diffLines = [];
-  for (const [file, lines] of addedLinesByFile) {
-    for (const line of lines) {
-      diffLines.push({ file, line });
-    }
-  }
-  return {
-    diff: diffString,
-    createdAt: new Date(mr.created_at),
-    updatedAt: new Date(mr.updated_at),
-    submitRequestId: String(mrNumber),
-    submitRequestNumber: mrNumber,
-    sourceBranch: mr.source_branch,
-    targetBranch: mr.target_branch,
-    authorName: mr.author?.name || mr.author?.username,
-    authorEmail: void 0,
-    // GitLab MR API doesn't expose author email directly
-    title: mr.title,
-    description: mr.description || void 0,
-    commits,
-    headCommitSha: mr.sha,
-    diffLines
-  };
-}
 
 // src/features/analysis/scm/gitlab/GitlabSCMLib.ts
 init_client_generates();
@@ -10570,33 +10196,23 @@ var GitlabSCMLib = class extends SCMLib {
     this._validateAccessTokenAndUrl();
     return `${this.url}/-/commits/${branchName}`;
   }
-  async getCommitDiff(commitSha) {
-    this._validateAccessTokenAndUrl();
-    const result = await getGitlabCommitDiff({
-      repoUrl: this.url,
-      accessToken: this.accessToken,
-      commitSha
-    });
-    return {
-      diff: result.diff,
-      commitTimestamp: result.commitTimestamp,
-      commitSha: result.commitSha,
-      authorName: result.authorName,
-      authorEmail: result.authorEmail,
-      message: result.message
-    };
-  }
-  async getSubmitRequestDiff(submitRequestId) {
+  async getSubmitRequestMetadata(submitRequestId) {
     this._validateAccessTokenAndUrl();
     const mrNumber = parseInt(submitRequestId, 10);
     if (isNaN(mrNumber) || mrNumber <= 0) {
       throw new Error(`Invalid merge request ID: ${submitRequestId}`);
     }
-    return getGitlabMergeRequestDiff({
-      repoUrl: this.url,
-      accessToken: this.accessToken,
-      mrNumber
+    const mr = await getGitlabMergeRequest({
+      url: this.url,
+      prNumber: mrNumber,
+      accessToken: this.accessToken
     });
+    return {
+      title: mr.title,
+      targetBranch: mr.target_branch,
+      sourceBranch: mr.source_branch,
+      headCommitSha: mr.sha
+    };
   }
   async searchSubmitRequests(params) {
     this._validateAccessTokenAndUrl();
@@ -10704,10 +10320,7 @@ var GitlabSCMLib = class extends SCMLib {
       repositoryUrl: this.url,
       prCreatedAt: new Date(metrics.createdAt),
       prMergedAt: metrics.mergedAt ? new Date(metrics.mergedAt) : null,
-      firstCommitDate: metrics.firstCommitDate ? new Date(metrics.firstCommitDate) : null,
       linesAdded: metrics.linesAdded,
-      commitsCount: metrics.commitsCount,
-      commitShas: metrics.commitShas,
       prStatus,
       commentIds: metrics.commentIds
     };
@@ -10837,34 +10450,8 @@ var StubSCMLib = class extends SCMLib {
   async addCommentToSubmitRequest(_submitRequestId, _comment) {
     console.warn("addCommentToSubmitRequest() no-op");
   }
-  async getCommitDiff(_commitSha) {
-    console.warn("getCommitDiff() returning stub diff");
-    return {
-      diff: "",
-      commitTimestamp: /* @__PURE__ */ new Date(),
-      commitSha: _commitSha,
-      authorName: void 0,
-      authorEmail: void 0,
-      message: void 0
-    };
-  }
-  async getSubmitRequestDiff(_submitRequestId) {
-    console.warn("getSubmitRequestDiff() returning stub diff");
-    return {
-      diff: "",
-      createdAt: /* @__PURE__ */ new Date(),
-      updatedAt: /* @__PURE__ */ new Date(),
-      submitRequestId: _submitRequestId,
-      submitRequestNumber: parseInt(_submitRequestId) || 0,
-      sourceBranch: "",
-      targetBranch: "",
-      authorName: void 0,
-      authorEmail: void 0,
-      title: void 0,
-      description: void 0,
-      commits: [],
-      diffLines: []
-    };
+  async getSubmitRequestMetadata(_submitRequestId) {
+    throw new Error("getSubmitRequestMetadata() not implemented");
   }
   async getPullRequestMetrics(_prNumber) {
     console.warn("getPullRequestMetrics() returning empty object");
@@ -12001,7 +11588,9 @@ function subscribeStream(query, variables, handlers, wsClientOptions) {
         finalizeError(error);
       },
       complete: () => {
-        return;
+        finalizeError(
+          new Error("Subscription completed by server before resolving")
+        );
       }
     }
   );
@@ -12903,7 +12492,7 @@ import Debug10 from "debug";
 
 // src/features/analysis/add_fix_comments_for_pr/utils/utils.ts
 import Debug9 from "debug";
-import parseDiff2 from "parse-diff";
+import parseDiff from "parse-diff";
 import { z as z27 } from "zod";
 
 // src/features/analysis/utils/by_key.ts
@@ -13276,7 +12865,7 @@ Refresh the page in order to see the changes.`,
 }
 async function getRelevantVulenrabilitiesFromDiff(params) {
   const { gqlClient, diff, vulnerabilityReportId } = params;
-  const parsedDiff = parseDiff2(diff);
+  const parsedDiff = parseDiff(diff);
   const fileHunks = parsedDiff.map((file) => {
     const fileNumbers = file.chunks.flatMap((chunk) => chunk.changes).filter((change) => change.type === "add").map((_change) => {
       const change = _change;
@@ -21710,7 +21299,7 @@ import {
   writeFileSync
 } from "fs";
 import fs22 from "fs/promises";
-import parseDiff3 from "parse-diff";
+import parseDiff2 from "parse-diff";
 import path21 from "path";
 var PatchApplicationService = class {
   /**
@@ -22394,7 +21983,7 @@ var PatchApplicationService = class {
     fixId,
     scanContext
   }) {
-    const parsedPatch = parseDiff3(patch);
+    const parsedPatch = parseDiff2(patch);
     if (!parsedPatch || parsedPatch.length === 0) {
       throw new Error("Failed to parse patch - no changes found");
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mobbdev",
-  "version": "1.2.21",
+  "version": "1.2.24",
   "description": "Automated secure code remediation tool",
   "repository": "git+https://github.com/mobb-dev/bugsy.git",
   "main": "dist/index.mjs",