@pilatos/bitbucket-cli 1.16.0 → 1.16.1

package/dist/index.js

@@ -17633,7 +17633,16 @@ class ConfigService {
       await this.verifyPermissions(this.configDir, CONFIG_DIR_MODE, "directory");
       await this.verifyPermissions(this.configFile, CONFIG_FILE_MODE, "file");
       const data = await fs.readFile(this.configFile, "utf-8");
-      this.configCache = JSON.parse(data);
+      try {
+        this.configCache = JSON.parse(data);
+      } catch (parseError) {
+        throw new BBError({
+          code: 4001 /* CONFIG_READ_FAILED */,
+          message: `Config file is not valid JSON: ${this.configFile}. Fix the file by hand or remove it and run \`bb auth login\` again.`,
+          cause: parseError instanceof Error ? parseError : undefined,
+          context: { configFile: this.configFile }
+        });
+      }
       return this.configCache;
     } catch (error) {
       if (error.code === "ENOENT") {
@@ -17771,10 +17780,14 @@ class ConfigService {
   }
 }
 // src/services/git.service.ts
+var DEFAULT_GIT_TIMEOUT_MS = 60000;
+
 class GitService {
   cwd;
-  constructor(cwd) {
+  timeoutMs;
+  constructor(cwd, options = {}) {
     this.cwd = cwd ?? process.cwd();
+    this.timeoutMs = options.timeoutMs ?? DEFAULT_GIT_TIMEOUT_MS;
   }
   async exec(args, cwd) {
     const proc = Bun.spawn(["git", ...args], {
@@ -17782,14 +17795,28 @@ class GitService {
       stdout: "pipe",
       stderr: "pipe"
     });
-    const stdout = await new Response(proc.stdout).text();
-    const stderr = await new Response(proc.stderr).text();
-    const exitCode = await proc.exited;
-    return {
-      stdout: stdout.trim(),
-      stderr: stderr.trim(),
-      exitCode
-    };
+    let timedOut = false;
+    const timer = setTimeout(() => {
+      timedOut = true;
+      try {
+        proc.kill();
+      } catch {}
+    }, this.timeoutMs);
+    try {
+      const stdout = await new Response(proc.stdout).text();
+      const stderr = await new Response(proc.stderr).text();
+      const exitCode = await proc.exited;
+      if (timedOut) {
+        throw new GitError(`git ${args.join(" ")} timed out after ${this.timeoutMs}ms`, `git ${args.join(" ")}`, exitCode);
+      }
+      return {
+        stdout: stdout.trim(),
+        stderr: stderr.trim(),
+        exitCode
+      };
+    } finally {
+      clearTimeout(timer);
+    }
   }
   async execOrError(args, cwd) {
     const result = await this.exec(args, cwd);
@@ -17840,7 +17867,7 @@ class GitService {
     return result.stdout;
   }
   withCwd(cwd) {
-    return new GitService(cwd);
+    return new GitService(cwd, { timeoutMs: this.timeoutMs });
   }
 }
 // src/services/context.service.ts
@@ -17940,6 +17967,12 @@ class ContextService {
     }
     return result.context;
   }
+  async requireRepoContextFor(options, context) {
+    return this.requireRepoContext({
+      ...context.globalOptions,
+      ...options
+    });
+  }
   buildRepoNotFoundMessage(reason, remoteUrl) {
     const fallback = "Use --workspace and --repo options, or run this command from within a Bitbucket repository.";
     switch (reason) {
@@ -18571,6 +18604,15 @@ class OutputService {
   text(message) {
     console.log(stripControl(message));
   }
+  truncate(text, maxLength, suffix = "...") {
+    if (maxLength <= 0 || text.length <= maxLength) {
+      return text;
+    }
+    if (suffix.length >= maxLength) {
+      return text.slice(0, maxLength);
+    }
+    return text.slice(0, maxLength - suffix.length) + suffix;
+  }
   formatDate(date) {
     const d = typeof date === "string" ? new Date(date) : date;
     return d.toLocaleDateString("en-US", {
@@ -18640,7 +18682,17 @@ function projectByFieldsRespectingWrapper(data, fields) {
   return projectFields(data, fields);
 }
 async function runJq(data, expression) {
-  const jq = await Promise.resolve().then(() => __toESM(require_dist(), 1));
+  let jq;
+  try {
+    jq = await Promise.resolve().then(() => __toESM(require_dist(), 1));
+  } catch (error) {
+    throw new BBError({
+      code: 8001 /* JQ_FAILED */,
+      message: "Failed to load the embedded jq runtime (jq-wasm). Reinstall the CLI or report this issue.",
+      cause: error instanceof Error ? error : undefined,
+      context: { expression }
+    });
+  }
   let result;
   try {
     result = await jq.raw(data, expression);
@@ -18812,7 +18864,11 @@ class VersionService {
         latestVersion,
         updateAvailable
       };
-    } catch {
+    } catch (error) {
+      if (process.env.DEBUG === "true") {
+        const message = error instanceof Error ? error.message : String(error);
+        console.error(`[version-check] skipped: ${message}`);
+      }
       return null;
     }
   }
@@ -22946,7 +23002,12 @@ class OAuthService {
         try {
           const open2 = (await Promise.resolve().then(() => (init_open(), exports_open))).default;
           await open2(authUrl);
-        } catch {}
+        } catch (err) {
+          if (process.env.DEBUG === "true") {
+            const message = err instanceof Error ? err.message : String(err);
+            console.error(`[oauth] could not open browser: ${message}`);
+          }
+        }
         console.error(`If the browser doesn't open, visit:
 ${authUrl}
 `);
@@ -27267,6 +27328,18 @@ class BaseCommand {
     }
     return parsed;
   }
+  parsePositiveInt(value, name) {
+    const trimmed = value.trim();
+    const parsed = Number.parseInt(trimmed, 10);
+    if (!Number.isFinite(parsed) || parsed < 1 || String(parsed) !== trimmed) {
+      throw new BBError({
+        code: 5002 /* VALIDATION_INVALID */,
+        message: this.appendHelpHint(`--${name} must be a positive integer.`),
+        context: { [name]: value }
+      });
+    }
+    return parsed;
+  }
   parseEnumOption(value, name, allowed) {
     if (!allowed.includes(value)) {
       throw new BBError({
@@ -28094,10 +28167,7 @@ class ListDefaultReviewersCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     const mode = options.repoOnly ? "direct" : "effective";
     const reviewers = await this.defaultReviewerService.list(repoContext, mode);
     if (context.globalOptions.json) {
@@ -28140,10 +28210,7 @@ class AddDefaultReviewerCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     const userResponse = await this.usersApi.usersSelectedUserGet({
       selectedUser: options.username
     });
@@ -28177,10 +28244,7 @@ class RemoveDefaultReviewerCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     if (!options.yes) {
       throw new BBError({
         code: 5001 /* VALIDATION_REQUIRED */,
@@ -28232,10 +28296,7 @@ class CreatePRCommand extends BaseCommand {
         message: this.appendHelpHint("Pull request title is required. Use --title option.")
       });
     }
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     let sourceBranch = options.source;
     if (!sourceBranch) {
       sourceBranch = await this.gitService.getCurrentBranch();
@@ -28360,10 +28421,7 @@ class ListPRsCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     const state = options.state ? this.parseEnumOption(options.state, "state", PR_STATES) : "OPEN";
     const limit = parseLimit(options.limit);
     const reviewerQuery = options.mine ? await this.buildMineFilter() : undefined;
@@ -28407,19 +28465,13 @@ class ListPRsCommand extends BaseCommand {
       const destination = pr.destination;
       return [
         `#${pr.id}`,
-        this.truncate(title ?? "", 50),
+        this.output.truncate(title ?? "", 50),
         pr.author?.display_name ?? "Unknown",
         `${source?.branch?.name ?? "unknown"} \u2192 ${destination?.branch?.name ?? "unknown"}`
       ];
     });
     this.output.table(["ID", "TITLE", "AUTHOR", "BRANCHES"], rows);
   }
-  truncate(text, maxLength) {
-    if (text.length <= maxLength) {
-      return text;
-    }
-    return text.substring(0, maxLength - 3) + "...";
-  }
   async buildMineFilter() {
     const response = await this.usersApi.userGet();
     const userUuid = response.data.uuid;
@@ -28443,11 +28495,8 @@ class ViewPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdGet({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -28572,13 +28621,10 @@ class EditPRCommand extends BaseCommand {
     this.gitService = gitService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     let prId;
     if (options.id) {
-      prId = Number.parseInt(options.id, 10);
+      prId = this.parsePositiveInt(options.id, "id");
     } else {
       const currentBranch = await this.gitService.getCurrentBranch();
       const matches = await collectPages({
@@ -28653,7 +28699,7 @@ class EditPRCommand extends BaseCommand {
     this.output.success(`Updated pull request #${pr.id}`);
     this.output.text(`  ${this.output.dim("Title:")} ${pr.title}`);
     if (pr.description) {
-      const truncatedDesc = pr.description.length > 100 ? pr.description.substring(0, 100) + "..." : pr.description;
+      const truncatedDesc = this.output.truncate(pr.description, 100);
       this.output.text(`  ${this.output.dim("Description:")} ${truncatedDesc}`);
     }
     this.output.text(`  ${this.output.dim("URL:")} ${links?.html?.href}`);
@@ -28674,11 +28720,8 @@ class MergePRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const request = {
       type: "pullrequest_merge_parameters"
     };
@@ -28722,11 +28765,8 @@ class ApprovePRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = Number.parseInt(options.id, 10);
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdApprovePost({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -28755,11 +28795,8 @@ class DeclinePRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = Number.parseInt(options.id, 10);
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdDeclinePost({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -28790,11 +28827,8 @@ class ReadyPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = Number.parseInt(options.id, 10);
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdPut({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -28832,11 +28866,8 @@ class CheckoutPRCommand extends BaseCommand {
     this.gitService = gitService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = Number.parseInt(options.id, 10);
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const prResponse = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdGet({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -28899,20 +28930,10 @@ class DiffPRCommand extends BaseCommand {
     this.gitService = gitService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     let prId;
     if (options.id) {
-      prId = Number.parseInt(options.id, 10);
-      if (Number.isNaN(prId)) {
-        throw new BBError({
-          code: 5002 /* VALIDATION_INVALID */,
-          message: "Invalid PR ID",
-          context: { id: options.id }
-        });
-      }
+      prId = this.parsePositiveInt(options.id, "id");
     } else {
       const currentBranch = await this.gitService.getCurrentBranch();
       const matches = await collectPages({
@@ -29129,11 +29150,8 @@ class ActivityPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const filterTypes = this.parseTypeFilter(options.type);
     const limit = parseLimit(options.limit);
     const activities = await collectPages({
@@ -29242,14 +29260,14 @@ class ActivityPRCommand extends BaseCommand {
       case "comment": {
         const content = getRawContent(activity.comment?.content) ?? "";
         const id = activity.comment?.id ? `#${activity.comment.id}` : "";
-        const snippet = this.truncate(content, 80);
+        const snippet = this.output.truncate(content, 80);
         return [id, snippet].filter(Boolean).join(" ");
       }
       case "approval":
         return "approved";
       case "changes_requested": {
         const reason = activity.changes_requested?.reason;
-        return reason ? this.truncate(reason, 80) : "changes requested";
+        return reason ? this.output.truncate(reason, 80) : "changes requested";
       }
       case "merge":
         return this.formatCommitDetail(activity.merge?.commit?.hash, "merged");
@@ -29262,7 +29280,7 @@ class ActivityPRCommand extends BaseCommand {
           return `state: ${activity.update.state}`;
         }
         if (activity.update?.title) {
-          return `title: ${this.truncate(activity.update.title, 60)}`;
+          return `title: ${this.output.truncate(activity.update.title, 60)}`;
         }
         if (activity.update?.description) {
           return "description updated";
@@ -29280,12 +29298,6 @@ class ActivityPRCommand extends BaseCommand {
     const shortHash = hash.slice(0, 7);
     return label ? `${label} ${shortHash}` : shortHash;
   }
-  truncate(text, maxLength) {
-    if (text.length <= maxLength) {
-      return text;
-    }
-    return text.substring(0, maxLength - 3) + "...";
-  }
 }
 
 // src/commands/pr/comment.command.ts
@@ -29319,33 +29331,14 @@ class CommentPRCommand extends BaseCommand {
         }
       });
     }
-    if (options.lineTo) {
-      const parsed = Number.parseInt(options.lineTo, 10);
-      if (Number.isNaN(parsed) || parsed < 1) {
-        throw new BBError({
-          code: 5002 /* VALIDATION_INVALID */,
-          message: "--line-to must be a positive integer"
-        });
-      }
-    }
-    if (options.lineFrom) {
-      const parsed = Number.parseInt(options.lineFrom, 10);
-      if (Number.isNaN(parsed) || parsed < 1) {
-        throw new BBError({
-          code: 5002 /* VALIDATION_INVALID */,
-          message: "--line-from must be a positive integer"
-        });
-      }
-    }
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = Number.parseInt(options.id, 10);
+    const lineTo = options.lineTo ? this.parsePositiveInt(options.lineTo, "line-to") : undefined;
+    const lineFrom = options.lineFrom ? this.parsePositiveInt(options.lineFrom, "line-from") : undefined;
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const inline = options.file ? {
       path: options.file,
-      ...options.lineTo ? { to: Number.parseInt(options.lineTo, 10) } : {},
-      ...options.lineFrom ? { from: Number.parseInt(options.lineFrom, 10) } : {}
+      ...lineTo !== undefined ? { to: lineTo } : {},
+      ...lineFrom !== undefined ? { from: lineFrom } : {}
     } : undefined;
     const body = {
       content: {
@@ -29395,11 +29388,8 @@ class ListCommentsPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const limit = parseLimit(options.limit);
     const values = await collectPages({
       limit,
@@ -29416,6 +29406,8 @@ class ListCommentsPRCommand extends BaseCommand {
     });
     if (context.globalOptions.json) {
       await this.output.json({
+        workspace: repoContext.workspace,
+        repoSlug: repoContext.repoSlug,
         pullRequestId: prId,
         count: values.length,
         comments: values
@@ -29431,7 +29423,7 @@ class ListCommentsPRCommand extends BaseCommand {
       return [
         comment.id?.toString() ?? "",
         getUserDisplayName(comment.user) ?? "Unknown",
-        comment.deleted ? "[deleted]" : options.truncate === false ? content : content.slice(0, 60) + (content.length > 60 ? "..." : ""),
+        comment.deleted ? "[deleted]" : options.truncate === false ? content : this.output.truncate(content, 60),
         this.output.formatDate(comment.created_on ?? "")
       ];
     });
@@ -29451,12 +29443,9 @@ class EditCommentPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.prId, "pr-id");
-    const commentId = this.parseIntOption(options.commentId, "comment-id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.prId, "pr-id");
+    const commentId = this.parsePositiveInt(options.commentId, "comment-id");
     const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdCommentsCommentIdPut({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -29494,12 +29483,9 @@ class DeleteCommentPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.prId, "pr-id");
-    const commentId = this.parseIntOption(options.commentId, "comment-id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.prId, "pr-id");
+    const commentId = this.parsePositiveInt(options.commentId, "comment-id");
     if (!options.yes) {
       throw new BBError({
         code: 5001 /* VALIDATION_REQUIRED */,
@@ -29539,11 +29525,8 @@ class AddReviewerPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const userResponse = await this.usersApi.usersSelectedUserGet({
       selectedUser: options.username
     });
@@ -29584,11 +29567,8 @@ class RemoveReviewerPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const userResponse = await this.usersApi.usersSelectedUserGet({
       selectedUser: options.username
     });
@@ -29622,11 +29602,8 @@ class ListReviewersPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdGet({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -29636,6 +29613,8 @@ class ListReviewersPRCommand extends BaseCommand {
     const reviewers = Array.from(pr.reviewers ?? []);
     if (context.globalOptions.json) {
       await this.output.json({
+        workspace: repoContext.workspace,
+        repoSlug: repoContext.repoSlug,
         pullRequestId: prId,
         count: reviewers.length,
         reviewers
@@ -29662,11 +29641,8 @@ class ChecksPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    const prId = this.parseIntOption(options.id, "id");
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.id, "id");
     const response = await this.commitStatusesApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdStatusesGet({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -29720,7 +29696,7 @@ class ChecksPRCommand extends BaseCommand {
       return [
         `${stateIcon} ${stateLabel}`,
         this.output.bold(name),
-        this.truncate(description, 40),
+        this.output.truncate(description, 40),
         status.updated_on ? this.output.formatDate(status.updated_on) : "-"
       ];
     });
@@ -29770,12 +29746,6 @@ class ChecksPRCommand extends BaseCommand {
       return acc;
     }, { successful: 0, failed: 0, pending: 0 });
   }
-  truncate(text, maxLength) {
-    if (text.length <= maxLength) {
-      return text;
-    }
-    return text.substring(0, maxLength - 3) + "...";
-  }
 }
 
 // src/commands/snippet/list.command.ts
@@ -30188,7 +30158,7 @@ class ListSnippetCommentsCommand extends BaseCommand {
         String(comment.id ?? ""),
         getUserDisplayName(comment.user) ?? "Unknown",
         this.output.formatDate(comment.created_on ?? ""),
-        content.slice(0, 60) + (content.length > 60 ? "..." : "")
+        this.output.truncate(content, 60)
       ];
     });
     this.output.table(["ID", "AUTHOR", "DATE", "CONTENT"], rows);
@@ -30247,7 +30217,7 @@ class EditSnippetCommentCommand extends BaseCommand {
   }
   async execute(options, context) {
     const workspace = await this.contextService.requireWorkspace(options.workspace ?? context.globalOptions.workspace);
-    const commentId = this.parseIntOption(options.commentId, "comment-id");
+    const commentId = this.parsePositiveInt(options.commentId, "comment-id");
     const body = {
       type: "snippet_comment",
       content: {
@@ -30285,7 +30255,7 @@ class DeleteSnippetCommentCommand extends BaseCommand {
   }
   async execute(options, context) {
     const workspace = await this.contextService.requireWorkspace(options.workspace ?? context.globalOptions.workspace);
-    const commentId = this.parseIntOption(options.commentId, "comment-id");
+    const commentId = this.parsePositiveInt(options.commentId, "comment-id");
     if (!options.yes) {
       throw new BBError({
         code: 5001 /* VALIDATION_REQUIRED */,
@@ -30535,10 +30505,7 @@ class BrowseCommand extends BaseCommand {
     this.urlBuilder = urlBuilder;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
     this.validateFlagCombination(options);
     const url2 = await this.resolveUrl(options, repoContext);
     const useJson = Boolean(context.globalOptions.json);
@@ -30602,7 +30569,7 @@ class BrowseCommand extends BaseCommand {
     const target = options.target?.trim();
     if (target && target.length > 0) {
       if (PR_NUMBER_PATTERN.test(target)) {
-        return this.urlBuilder.pullRequest(ctx, Number.parseInt(target, 10));
+        return this.urlBuilder.pullRequest(ctx, this.parsePositiveInt(target, "target"));
       }
       if (SHA_PATTERN.test(target)) {
         return this.urlBuilder.commit(ctx, target);
@@ -30686,17 +30653,6 @@ class BrowseCommand extends BaseCommand {
       return "HEAD";
     }
   }
-  parsePositiveInt(value, name) {
-    const parsed = Number.parseInt(value, 10);
-    if (!Number.isFinite(parsed) || parsed <= 0 || String(parsed) !== value.trim()) {
-      throw new BBError({
-        code: 5002 /* VALIDATION_INVALID */,
-        message: this.appendHelpHint(`--${name} must be a positive integer.`),
-        context: { [name]: value }
-      });
-    }
-    return parsed;
-  }
   async openInBrowser(url2) {
     this.output.info(`Opening ${url2} in your browser...`);
     const open2 = (await Promise.resolve().then(() => (init_open(), exports_open))).default;
@@ -31841,3 +31797,6 @@ if (typeof Bun === "undefined") {
   process.exit(1);
 }
 cli.parse(process.argv);
+
+//# debugId=A7729A51C3738B3664756E2164756E21
+//# sourceMappingURL=index.js.map