mobbdev 1.4.30 → 1.4.31

package/dist/index.mjs

@@ -8814,6 +8814,35 @@ function getBitbucketSdk(params) {
         return { accountId: null, email: null };
       }
     },
+    async listRecentCommits(params2) {
+      const sinceDate = new Date(params2.since);
+      const commits = [];
+      const MAX_PAGES = 100;
+      let page = 1;
+      let done = false;
+      while (!done && page <= MAX_PAGES) {
+        const res = await bitbucketClient.repositories.listCommits({
+          repo_slug: params2.repo_slug,
+          workspace: params2.workspace,
+          page: String(page),
+          pagelen: 100
+        });
+        const data = res.data;
+        const values = data?.values ?? [];
+        if (values.length === 0) break;
+        for (const commit of values) {
+          const date = commit?.["date"];
+          if (date && new Date(date) < sinceDate) {
+            done = true;
+            continue;
+          }
+          commits.push(commit);
+        }
+        if (!data?.next) break;
+        page++;
+      }
+      return commits;
+    },
     async getRepoCommitAuthors(params2) {
       try {
         const res = await bitbucketClient.repositories.listCommits({
@@ -9157,8 +9186,33 @@ var BitbucketSCMLib = class extends SCMLib {
   async getPullRequestMetrics(_prNumber) {
     throw new Error("getPullRequestMetrics not implemented for Bitbucket");
   }
-  async getRecentCommits(_since) {
-    throw new Error("getRecentCommits not implemented for Bitbucket");
+  async getRecentCommits(since) {
+    this._validateAccessTokenAndUrl();
+    const { workspace, repo_slug } = parseBitbucketOrganizationAndRepo(this.url);
+    const bitbucketSdk = createBitbucketSdk(this.accessToken);
+    const rawCommits = await bitbucketSdk.listRecentCommits({
+      workspace,
+      repo_slug,
+      since
+    });
+    return {
+      data: rawCommits.map((commit) => {
+        const c = commit;
+        const author = c["author"];
+        const match = author?.raw?.match(/^(.+?)\s*<([^>]+)>/);
+        return {
+          sha: c["hash"] ?? "",
+          commit: {
+            committer: c["date"] ? { date: c["date"] } : void 0,
+            author: match ? { name: match[1]?.trim(), email: match[2] } : void 0,
+            message: c["message"] ?? ""
+          },
+          // Bitbucket resolves the commit to an account when it can; account_id
+          // matches repo_contributor.external_id for Bitbucket rows.
+          authorExternalId: author?.user?.account_id ?? null
+        };
+      })
+    };
   }
   async getRateLimitStatus() {
     return null;
@@ -9423,7 +9477,7 @@ function getOctoKit(options) {
         octokit.log.warn(
           `Request quota exhausted for request ${options2.method} ${options2.url}`
         );
-        if (retryCount === 0) {
+        if (retryCount < 3) {
           octokit.log.info(`Retrying after ${retryAfter} seconds!`);
           return true;
         }
@@ -9433,7 +9487,7 @@ function getOctoKit(options) {
         octokit.log.warn(
           `SecondaryRateLimit detected for request ${options2.method} ${options2.url}`
         );
-        if (retryCount === 0) {
+        if (retryCount < 3) {
           octokit.log.info(`Retrying after ${retryAfter} seconds!`);
           return true;
         }
@@ -10198,10 +10252,16 @@ function getGithubSdk(params = {}) {
       });
       return members;
     },
-    async getUserProfile(params2) {
+    async getUserProfile(getProfileParams) {
+      const cache = params.userProfileCache;
+      const cached = cache?.get(getProfileParams.username);
+      if (cached) {
+        return cached;
+      }
       const { data } = await octokit.rest.users.getByUsername({
-        username: params2.username
+        username: getProfileParams.username
       });
+      cache?.set(getProfileParams.username, data);
       return data;
     },
     async getLatestRepoCommitByAuthor(params2) {
@@ -10294,12 +10354,16 @@ function determinePrStatus(state, isDraft) {
 }
 var GithubSCMLib = class extends SCMLib {
   // we don't always need a url, what's important is that we have an access token
-  constructor(url, accessToken, scmOrg) {
+  constructor(url, accessToken, scmOrg, options) {
     super(url, accessToken, scmOrg);
     __publicField(this, "githubSdk");
     this.githubSdk = getGithubSdk({
       auth: accessToken,
-      url
+      url,
+      // Honor GitHub's rate-limit/Retry-After backoff for bulk callers
+      // (e.g. the contributor sync) instead of failing fast.
+      isEnableRetries: options?.enableThrottling,
+      userProfileCache: options?.userProfileCache
     });
   }
   async createSubmitRequest(params) {
@@ -10429,7 +10493,13 @@ var GithubSCMLib = class extends SCMLib {
           author: c.commit.author ? { email: c.commit.author.email, name: c.commit.author.name } : void 0,
           message: c.commit.message
         },
-        parents: c.parents?.map((p) => ({ sha: p.sha }))
+        parents: c.parents?.map((p) => ({ sha: p.sha })),
+        // Top-level `author` is GitHub's resolution of the commit to an account
+        // (null when GitHub can't tie the commit email to a user). Its numeric
+        // id matches repo_contributor.external_id (stored as String(id)), and
+        // its type marks GitHub App / bot accounts authoritatively.
+        authorExternalId: c.author?.id != null ? String(c.author.id) : null,
+        authorIsBot: c.author?.type === "Bot"
       }))
     };
   }
@@ -10498,7 +10568,11 @@ var GithubSCMLib = class extends SCMLib {
                     author: c.login
                   }
                 );
-                const commitEmail = commit?.commit?.author?.email;
+                const commitAuthor = commit?.commit?.author;
+                const commitEmail = commitAuthor?.email;
+                if (commitAuthor?.name && displayName === c.login) {
+                  displayName = commitAuthor.name;
+                }
                 if (commitEmail) {
                   if (isRealEmail(commitEmail)) {
                     profileEmail = commitEmail;
@@ -10577,6 +10651,11 @@ var GithubSCMLib = class extends SCMLib {
               username: c.login
             });
           }
+          contextLogger.info("[GitHub] Contributor resolved", {
+            username: c.login,
+            profileName: displayName,
+            displayNameIsNull: displayName === null
+          });
           return {
             externalId: String(c.id),
             username: c.login ?? null,
@@ -12355,12 +12434,20 @@ var StubSCMLib = class extends SCMLib {
 };
 
 // src/features/analysis/scm/scmFactory.ts
-async function createScmLib({ url, accessToken, scmType, scmOrg }, { propagateExceptions = false, skipValidation = false } = {}) {
+async function createScmLib({ url, accessToken, scmType, scmOrg }, {
+  propagateExceptions = false,
+  skipValidation = false,
+  enableThrottling = false,
+  githubUserProfileCache
+} = {}) {
   const trimmedUrl = url ? url.trim().replace(/\/$/, "").replace(/.git$/i, "") : void 0;
   try {
     switch (scmType) {
       case "GITHUB" /* GITHUB */: {
-        const scm = new GithubSCMLib(trimmedUrl, accessToken, scmOrg);
+        const scm = new GithubSCMLib(trimmedUrl, accessToken, scmOrg, {
+          enableThrottling,
+          userProfileCache: githubUserProfileCache
+        });
         if (!skipValidation) await scm.validateParams();
         return scm;
       }
@@ -19977,7 +20064,7 @@ function createLogger(config2) {
 
 // src/features/claude_code/hook_logger.ts
 var DD_RUM_TOKEN = true ? "pubf59c0182545bfb4c299175119f1abf9b" : "";
-var CLI_VERSION = true ? "1.4.30" : "unknown";
+var CLI_VERSION = true ? "1.4.31" : "unknown";
 var NAMESPACE = "mobbdev-claude-code-hook-logs";
 var claudeCodeVersion;
 function buildDdTags() {

package/package.json

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