mobbdev 1.2.51 → 1.2.52

package/dist/args/commands/upload_ai_blame.mjs

@@ -4557,6 +4557,9 @@ function subscribeStream(query, variables, handlers, wsClientOptions) {
   };
 }
 
+// src/features/analysis/scm/ado/ado.ts
+import pLimit from "p-limit";
+
 // src/features/analysis/scm/utils/index.ts
 import { z as z15 } from "zod";
 
@@ -6521,6 +6524,8 @@ var debug3 = Debug3("mobbdev:scm:ado");
 
 // src/features/analysis/scm/ado/AdoSCMLib.ts
 import { setTimeout as setTimeout2 } from "timers/promises";
+import pLimit2 from "p-limit";
+init_client_generates();
 
 // src/features/analysis/scm/scmSubmit/index.ts
 init_GitService();
@@ -6593,7 +6598,7 @@ import {
   Gitlab
 } from "@gitbeaker/rest";
 import Debug5 from "debug";
-import pLimit from "p-limit";
+import pLimit3 from "p-limit";
 import {
   Agent,
   fetch as undiciFetch,
@@ -7879,7 +7884,7 @@ async function getRepositoryUrl() {
     }
     const remoteUrl = await gitService.getRemoteUrl();
     const parsed = parseScmURL(remoteUrl);
-    return parsed?.scmType === "GitHub" /* GitHub */ || parsed?.scmType === "GitLab" /* GitLab */ ? remoteUrl : null;
+    return parsed?.scmType && parsed.scmType !== "Unknown" ? remoteUrl : null;
   } catch {
     return null;
   }

package/dist/index.mjs

@@ -4170,6 +4170,9 @@ import multimatch from "multimatch";
 import StreamZip from "node-stream-zip";
 import tmp from "tmp";
 
+// src/features/analysis/scm/ado/ado.ts
+import pLimit from "p-limit";
+
 // src/features/analysis/scm/errors.ts
 var InvalidAccessTokenError = class extends Error {
   constructor(m, scmType) {
@@ -7094,6 +7097,136 @@ async function getAdoSdk(params) {
       );
       return res.pullRequestId;
     },
+    async getAdoPullRequestMetadata({
+      repoUrl,
+      prNumber
+    }) {
+      const { repo, projectName } = parseAdoOwnerAndRepo(repoUrl);
+      const git = await api2.getGitApi();
+      const pr = await git.getPullRequest(repo, prNumber, projectName);
+      if (!pr) {
+        throw new Error(`Pull request #${prNumber} not found`);
+      }
+      return {
+        title: pr.title,
+        targetBranch: (pr.targetRefName || "").replace("refs/heads/", ""),
+        sourceBranch: (pr.sourceRefName || "").replace("refs/heads/", ""),
+        headCommitSha: pr.lastMergeSourceCommit?.commitId || pr.lastMergeCommit?.commitId || ""
+      };
+    },
+    async getAdoPrFiles({
+      repoUrl,
+      prNumber
+    }) {
+      const { repo, projectName } = parseAdoOwnerAndRepo(repoUrl);
+      const git = await api2.getGitApi();
+      const iterations = await git.getPullRequestIterations(
+        repo,
+        prNumber,
+        projectName
+      );
+      if (!iterations || iterations.length === 0) {
+        return [];
+      }
+      const lastIteration = iterations[iterations.length - 1];
+      if (!lastIteration?.id) {
+        return [];
+      }
+      const iterationId = lastIteration.id;
+      const changes = await git.getPullRequestIterationChanges(
+        repo,
+        prNumber,
+        iterationId,
+        projectName
+      );
+      if (!changes?.changeEntries) {
+        return [];
+      }
+      return changes.changeEntries.filter((entry) => {
+        const changeType = entry.changeType;
+        return changeType !== 16 && entry.item?.path;
+      }).map((entry) => {
+        const path27 = entry.item.path;
+        return path27.startsWith("/") ? path27.slice(1) : path27;
+      });
+    },
+    async searchAdoPullRequests({
+      repoUrl,
+      status,
+      skip,
+      top
+    }) {
+      const { repo, projectName } = parseAdoOwnerAndRepo(repoUrl);
+      const git = await api2.getGitApi();
+      const searchCriteria = {
+        // Cast to number to avoid TypeScript enum compatibility issues between
+        // our AdoPullRequestStatus and azure-devops-node-api's PullRequestStatus
+        status: status ?? 4 /* All */
+      };
+      const prs = await git.getPullRequests(
+        repo,
+        searchCriteria,
+        projectName,
+        void 0,
+        // maxCommentLength
+        skip,
+        top
+      );
+      return prs;
+    },
+    async getAdoPullRequestMetrics({
+      repoUrl,
+      prNumber
+    }) {
+      const { repo, projectName } = parseAdoOwnerAndRepo(repoUrl);
+      const git = await api2.getGitApi();
+      const pr = await git.getPullRequest(repo, prNumber, projectName);
+      if (!pr) {
+        throw new Error(`Pull request #${prNumber} not found`);
+      }
+      const threads = await git.getThreads(repo, prNumber, projectName);
+      const commentIds = (threads || []).filter((t) => t.id && t.comments && t.comments.length > 0).map((t) => String(t.id));
+      return {
+        pr,
+        commentIds,
+        linesAdded: 0
+      };
+    },
+    async getAdoRecentCommits({
+      repoUrl,
+      since
+    }) {
+      const { repo, projectName } = parseAdoOwnerAndRepo(repoUrl);
+      const git = await api2.getGitApi();
+      const repository = await git.getRepository(
+        decodeURI(repo),
+        projectName ? decodeURI(projectName) : void 0
+      );
+      const defaultBranch = repository.defaultBranch?.replace("refs/heads/", "");
+      const commits = await git.getCommits(
+        repo,
+        {
+          fromDate: since,
+          itemVersion: defaultBranch ? { version: defaultBranch } : void 0,
+          $top: 100
+        },
+        projectName
+      );
+      return commits;
+    },
+    async getAdoPrCommits({
+      repoUrl,
+      prNumber
+    }) {
+      const { repo, projectName } = parseAdoOwnerAndRepo(repoUrl);
+      const git = await api2.getGitApi();
+      const commits = await git.getPullRequestCommits(
+        repo,
+        prNumber,
+        projectName
+      );
+      return (commits || []).filter((c) => c.commitId).map((c) => c.commitId);
+    },
     async getAdoRepoDefaultBranch({
       repoUrl
     }) {
@@ -7232,28 +7365,38 @@ async function getAdoRepoList({
       });
       const gitOrg = await orgApi.getGitApi();
       const orgRepos = await gitOrg.getRepositories();
+      const repoLimit = pLimit(5);
       const repoInfoList = (await Promise.allSettled(
-        orgRepos.map(async (repo) => {
-          if (!repo.name || !repo.remoteUrl || !repo.defaultBranch) {
-            throw new InvalidRepoUrlError("bad repo");
-          }
-          const branch = await gitOrg.getBranch(
-            repo.name,
-            repo.defaultBranch.replace(/^refs\/heads\//, ""),
-            repo.project?.name
-          );
-          return {
-            repoName: repo.name,
-            repoUrl: repo.remoteUrl.replace(
-              /^[hH][tT][tT][pP][sS]:\/\/[^/]+@/,
-              "https://"
-            ),
-            repoOwner: org,
-            repoIsPublic: repo.project?.visibility === 2 /* Public */,
-            repoLanguages: [],
-            repoUpdatedAt: branch.commit?.committer?.date?.toDateString() || repo.project?.lastUpdateTime?.toDateString() || (/* @__PURE__ */ new Date()).toDateString()
-          };
-        })
+        orgRepos.map(
+          (repo) => repoLimit(async () => {
+            if (!repo.name || !repo.remoteUrl || !repo.defaultBranch) {
+              throw new InvalidRepoUrlError("bad repo");
+            }
+            const branch = await gitOrg.getBranch(
+              repo.name,
+              repo.defaultBranch.replace(/^refs\/heads\//, ""),
+              repo.project?.name
+            );
+            return {
+              repoName: repo.name,
+              repoUrl: repo.remoteUrl.replace(
+                /^[hH][tT][tT][pP][sS]:\/\/[^/]+@/,
+                "https://"
+              ),
+              repoOwner: org,
+              repoIsPublic: repo.project?.visibility === 2 /* Public */,
+              repoLanguages: [],
+              repoUpdatedAt: (
+                // Use the latest available timestamp.
+                // branch.commit.committer.date = last commit on default branch.
+                // project.lastUpdateTime = last project-level change (may be stale).
+                // ADO doesn't expose a per-repo "last pushed" date like GitHub does,
+                // so feature branch pushes won't be reflected until merged.
+                branch.commit?.committer?.date?.toISOString() || repo.project?.lastUpdateTime?.toISOString() || (/* @__PURE__ */ new Date()).toISOString()
+              )
+            };
+          })
+        )
       )).reduce((acc, res) => {
         if (res.status === "fulfilled") {
           acc.push(res.value);
@@ -7273,6 +7416,8 @@ async function getAdoRepoList({
 
 // src/features/analysis/scm/ado/AdoSCMLib.ts
 import { setTimeout as setTimeout2 } from "timers/promises";
+import pLimit2 from "p-limit";
+init_client_generates();
 
 // src/features/analysis/scm/scmSubmit/index.ts
 init_GitService();
@@ -7407,6 +7552,19 @@ var SCMLib = class {
   }
 };
 
+// src/features/analysis/scm/utils/cursorValidation.ts
+var MAX_CURSOR_VALUE = 1e5;
+function parseCursorSafe(cursor, defaultValue = 0, maxValue = MAX_CURSOR_VALUE) {
+  if (cursor === null || cursor === void 0 || cursor === "") {
+    return defaultValue;
+  }
+  const parsed = parseInt(cursor, 10);
+  if (isNaN(parsed) || parsed < 0 || parsed > maxValue) {
+    return defaultValue;
+  }
+  return parsed;
+}
+
 // src/features/analysis/scm/ado/AdoSCMLib.ts
 async function initAdoSdk(params) {
   const { url, accessToken, scmOrg } = params;
@@ -7599,28 +7757,190 @@ var AdoSCMLib = class extends SCMLib {
       markdownComment: comment
     });
   }
-  async getSubmitRequestMetadata(_submitRequestId) {
-    throw new Error("getSubmitRequestMetadata not implemented for ADO");
+  async getSubmitRequestMetadata(submitRequestId) {
+    this._validateAccessTokenAndUrl();
+    const adoSdk = await this.getAdoSdk();
+    return adoSdk.getAdoPullRequestMetadata({
+      repoUrl: this.url,
+      prNumber: Number(submitRequestId)
+    });
   }
-  async getPrFiles(_prNumber) {
-    throw new Error("getPrFiles not implemented for ADO");
+  async getPrFiles(prNumber) {
+    this._validateAccessTokenAndUrl();
+    const adoSdk = await this.getAdoSdk();
+    return adoSdk.getAdoPrFiles({
+      repoUrl: this.url,
+      prNumber
+    });
   }
-  async searchSubmitRequests(_params) {
-    throw new Error("searchSubmitRequests not implemented for ADO");
+  async searchSubmitRequests(params) {
+    this._validateAccessToken();
+    const adoSdk = await this.getAdoSdk();
+    const skip = parseCursorSafe(params.cursor, 0);
+    const top = params.limit || 10;
+    let status = 4 /* All */;
+    if (params.filters?.state === "open") {
+      status = 1 /* Active */;
+    } else if (params.filters?.state === "closed") {
+      status = 4 /* All */;
+    }
+    const prs = await adoSdk.searchAdoPullRequests({
+      repoUrl: params.repoUrl,
+      status,
+      skip,
+      top: top + 1
+      // Fetch one extra to determine hasMore
+    });
+    const apiHasMore = prs.length > top;
+    const prsPage = prs.slice(0, top);
+    const getAdoPrUpdatedDate = (pr) => pr.closedDate || pr.lastMergeSourceCommit?.committer?.date || pr.creationDate || /* @__PURE__ */ new Date();
+    let filtered = prsPage;
+    if (params.filters?.updatedAfter) {
+      const afterDate = params.filters.updatedAfter;
+      filtered = prsPage.filter((pr) => getAdoPrUpdatedDate(pr) > afterDate);
+    }
+    if (params.filters?.state === "closed") {
+      filtered = filtered.filter(
+        (pr) => pr.status === 3 /* Completed */ || pr.status === 2 /* Abandoned */
+      );
+    }
+    const results = filtered.map((pr) => {
+      let prStatus = "open";
+      if (pr.status === 3 /* Completed */) {
+        prStatus = "merged";
+      } else if (pr.status === 2 /* Abandoned */) {
+        prStatus = "closed";
+      }
+      return {
+        submitRequestId: String(pr.pullRequestId),
+        submitRequestNumber: pr.pullRequestId || 0,
+        title: pr.title || "",
+        status: prStatus,
+        sourceBranch: (pr.sourceRefName || "").replace("refs/heads/", ""),
+        targetBranch: (pr.targetRefName || "").replace("refs/heads/", ""),
+        authorName: pr.createdBy?.displayName,
+        authorEmail: pr.createdBy?.uniqueName,
+        createdAt: pr.creationDate || /* @__PURE__ */ new Date(),
+        updatedAt: getAdoPrUpdatedDate(pr),
+        description: pr.description,
+        tickets: [],
+        changedLines: { added: 0, removed: 0 }
+      };
+    });
+    return {
+      results,
+      nextCursor: apiHasMore ? String(skip + top) : void 0,
+      hasMore: apiHasMore
+    };
   }
-  async searchRepos(_params) {
-    throw new Error("searchRepos not implemented for ADO");
+  // TODO: Performance — this fetches ALL repositories on every call, then paginates
+  // in-memory. For orgs with thousands of repos this can be slow and wasteful,
+  // especially when the UI pages through results (each page triggers a full re-fetch).
+  // Consider caching the fetched list with a short TTL, or using ADO's
+  // getRepositoriesPaged() API per-project for true server-side pagination.
+  async searchRepos(params) {
+    this._validateAccessToken();
+    if (this.url && new URL(this.url).origin !== scmCloudUrl.Ado) {
+      throw new Error(
+        `Oauth token is not supported for ADO on prem - ${this.url}`
+      );
+    }
+    const allRepos = await getAdoRepoList({
+      orgName: params.scmOrg,
+      accessToken: this.accessToken,
+      tokenOrg: this.scmOrg
+    });
+    allRepos.sort((a, b) => {
+      const dateA = a.repoUpdatedAt ? new Date(a.repoUpdatedAt).getTime() : 0;
+      const dateB = b.repoUpdatedAt ? new Date(b.repoUpdatedAt).getTime() : 0;
+      return dateB - dateA;
+    });
+    const page = parseCursorSafe(params.cursor, 0);
+    const limit = params.limit || 100;
+    const start = page;
+    const pageResults = allRepos.slice(start, start + limit);
+    const hasMore = start + limit < allRepos.length;
+    return {
+      results: pageResults,
+      nextCursor: hasMore ? String(start + limit) : void 0,
+      hasMore
+    };
   }
-  // TODO: Add comprehensive tests for getPullRequestMetrics (ADO)
-  // See clients/cli/src/features/analysis/scm/__tests__/github.test.ts:589-648 for reference
-  async getPullRequestMetrics(_prNumber) {
-    throw new Error("getPullRequestMetrics not implemented for ADO");
+  async getPrCommitsBatch(repoUrl, prNumbers) {
+    this._validateAccessToken();
+    const adoSdk = await this.getAdoSdk();
+    const result = /* @__PURE__ */ new Map();
+    const limit = pLimit2(5);
+    await Promise.all(
+      prNumbers.map(
+        (prNumber) => limit(async () => {
+          try {
+            const commits = await adoSdk.getAdoPrCommits({
+              repoUrl,
+              prNumber
+            });
+            result.set(prNumber, commits);
+          } catch (error) {
+            console.warn(
+              `[AdoSCMLib.getPrCommitsBatch] Failed to fetch commits for PR #${prNumber}:`,
+              error instanceof Error ? error.message : String(error)
+            );
+            result.set(prNumber, []);
+          }
+        })
+      )
+    );
+    return result;
   }
-  async getRecentCommits(_since) {
-    throw new Error("getRecentCommits not implemented for ADO");
+  async getPullRequestMetrics(prNumber) {
+    this._validateAccessTokenAndUrl();
+    const adoSdk = await this.getAdoSdk();
+    const { pr, commentIds, linesAdded } = await adoSdk.getAdoPullRequestMetrics({
+      repoUrl: this.url,
+      prNumber
+    });
+    let prStatus = "ACTIVE" /* Active */;
+    if (pr.status === 3) {
+      prStatus = "MERGED" /* Merged */;
+    } else if (pr.status === 2) {
+      prStatus = "CLOSED" /* Closed */;
+    } else if (pr.isDraft) {
+      prStatus = "DRAFT" /* Draft */;
+    }
+    return {
+      prId: String(prNumber),
+      repositoryUrl: this.url,
+      prCreatedAt: pr.creationDate || /* @__PURE__ */ new Date(),
+      prMergedAt: pr.closedDate && pr.status === 3 ? pr.closedDate : null,
+      linesAdded,
+      prStatus,
+      commentIds
+    };
+  }
+  async getRecentCommits(since) {
+    this._validateAccessTokenAndUrl();
+    const adoSdk = await this.getAdoSdk();
+    const commits = await adoSdk.getAdoRecentCommits({
+      repoUrl: this.url,
+      since
+    });
+    return {
+      data: commits.map((c) => ({
+        sha: c.commitId || "",
+        commit: {
+          committer: c.committer?.date ? { date: c.committer.date.toISOString() } : void 0,
+          author: c.author ? { email: c.author.email, name: c.author.name } : void 0,
+          message: c.comment
+        },
+        parents: c.parents?.map((sha) => ({ sha }))
+      }))
+    };
   }
   async getRateLimitStatus() {
-    return null;
+    return {
+      remaining: 1e4,
+      reset: new Date(Date.now() + 36e5)
+    };
   }
 };
 
@@ -8218,19 +8538,6 @@ init_env();
 import { z as z22 } from "zod";
 init_client_generates();
 
-// src/features/analysis/scm/utils/cursorValidation.ts
-var MAX_CURSOR_VALUE = 1e5;
-function parseCursorSafe(cursor, defaultValue = 0, maxValue = MAX_CURSOR_VALUE) {
-  if (cursor === null || cursor === void 0 || cursor === "") {
-    return defaultValue;
-  }
-  const parsed = parseInt(cursor, 10);
-  if (isNaN(parsed) || parsed < 0 || parsed > maxValue) {
-    return defaultValue;
-  }
-  return parsed;
-}
-
 // src/features/analysis/scm/github/github.ts
 import { RequestError } from "@octokit/request-error";
 
@@ -9597,7 +9904,7 @@ import {
   Gitlab
 } from "@gitbeaker/rest";
 import Debug3 from "debug";
-import pLimit from "p-limit";
+import pLimit3 from "p-limit";
 import {
   Agent,
   fetch as undiciFetch,
@@ -10026,7 +10333,7 @@ async function getGitlabMrCommitsBatch({
     url: repoUrl,
     gitlabAuthToken: accessToken
   });
-  const limit = pLimit(GITLAB_API_CONCURRENCY);
+  const limit = pLimit3(GITLAB_API_CONCURRENCY);
   const results = await Promise.all(
     mrNumbers.map(
       (mrNumber) => limit(async () => {
@@ -10065,7 +10372,7 @@ async function getGitlabMrDataBatch({
     url: repoUrl,
     gitlabAuthToken: accessToken
   });
-  const limit = pLimit(GITLAB_API_CONCURRENCY);
+  const limit = pLimit3(GITLAB_API_CONCURRENCY);
   const results = await Promise.all(
     mrNumbers.map(
       (mrNumber) => limit(async () => {
@@ -13060,7 +13367,7 @@ async function getRepositoryUrl() {
     }
     const remoteUrl = await gitService.getRemoteUrl();
     const parsed = parseScmURL(remoteUrl);
-    return parsed?.scmType === "GitHub" /* GitHub */ || parsed?.scmType === "GitLab" /* GitLab */ ? remoteUrl : null;
+    return parsed?.scmType && parsed.scmType !== "Unknown" ? remoteUrl : null;
   } catch {
     return null;
   }

package/package.json

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