@pilatos/bitbucket-cli 1.16.0 → 1.17.0

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) {
@@ -18462,6 +18495,54 @@ var chalk = createChalk();
 var chalkStderr = createChalk({ level: stderrColor ? stderrColor.level : 0 });
 var source_default = chalk;
 
+// src/services/locale.ts
+var DEFAULT_LOCALE = "en-US";
+function normalizePosixLocale(value) {
+  const trimmed = value.trim();
+  if (trimmed.length === 0) {
+    return;
+  }
+  const base = trimmed.split(/[.@]/, 1)[0];
+  if (base.length === 0) {
+    return;
+  }
+  const upper = base.toUpperCase();
+  if (upper === "C" || upper === "POSIX") {
+    return DEFAULT_LOCALE;
+  }
+  return base.replace(/_/g, "-");
+}
+function detectSystemLocale(env2 = process.env) {
+  const candidates = [env2.LC_TIME, env2.LC_ALL, env2.LANG];
+  for (const candidate of candidates) {
+    if (typeof candidate !== "string") {
+      continue;
+    }
+    const normalized = normalizePosixLocale(candidate);
+    if (normalized) {
+      return normalized;
+    }
+  }
+  return DEFAULT_LOCALE;
+}
+function resolveLocale(options) {
+  const env2 = options.env ?? process.env;
+  if (typeof options.explicit === "string") {
+    const trimmed = options.explicit.trim();
+    if (trimmed.length > 0) {
+      return trimmed;
+    }
+  }
+  const fromEnvVar = env2.BB_LOCALE;
+  if (typeof fromEnvVar === "string") {
+    const trimmed = fromEnvVar.trim();
+    if (trimmed.length > 0) {
+      return trimmed;
+    }
+  }
+  return detectSystemLocale(env2);
+}
+
 // src/services/output.project.ts
 function projectFields(item, fields) {
   if (item === null || typeof item !== "object") {
@@ -18488,7 +18569,143 @@ function deepGet(source, path) {
   return current === undefined ? null : current;
 }
 
+// src/services/spinner.ts
+var FRAMES = ["\u280B", "\u2819", "\u2839", "\u2838", "\u283C", "\u2834", "\u2826", "\u2827", "\u2807", "\u280F"];
+var FRAME_INTERVAL_MS = 80;
+var ESC = "\x1B";
+var CLEAR_LINE = `\r${ESC}[2K`;
+var HIDE_CURSOR = `${ESC}[?25l`;
+var SHOW_CURSOR = `${ESC}[?25h`;
+var COLOR_CYAN = `${ESC}[36m`;
+var COLOR_GREEN = `${ESC}[32m`;
+var COLOR_RED = `${ESC}[31m`;
+var COLOR_RESET = `${ESC}[0m`;
+
+class Spinner {
+  text;
+  enabled;
+  noColor;
+  stream;
+  onStop;
+  timer = null;
+  frameIndex = 0;
+  active = false;
+  exitHandler = null;
+  constructor(text, options) {
+    this.text = text;
+    this.enabled = options.enabled;
+    this.noColor = options.noColor;
+    this.stream = options.stream;
+    this.onStop = options.onStop;
+  }
+  start() {
+    if (!this.enabled || this.active) {
+      return this;
+    }
+    this.active = true;
+    this.stream.write(HIDE_CURSOR);
+    this.render();
+    this.timer = setInterval(() => {
+      this.frameIndex = (this.frameIndex + 1) % FRAMES.length;
+      this.render();
+    }, FRAME_INTERVAL_MS);
+    this.timer.unref?.();
+    this.exitHandler = () => {
+      if (this.active) {
+        this.active = false;
+        if (this.timer) {
+          clearInterval(this.timer);
+          this.timer = null;
+        }
+        this.stream.write(`${CLEAR_LINE}${SHOW_CURSOR}`);
+      }
+    };
+    process.once("SIGINT", this.exitHandler);
+    process.once("SIGTERM", this.exitHandler);
+    process.once("exit", this.exitHandler);
+    return this;
+  }
+  stop() {
+    if (!this.active) {
+      return this.detach();
+    }
+    this.active = false;
+    if (this.timer) {
+      clearInterval(this.timer);
+      this.timer = null;
+    }
+    this.stream.write(`${CLEAR_LINE}${SHOW_CURSOR}`);
+    return this.detach();
+  }
+  succeed(message) {
+    this.stop();
+    if (this.enabled && message) {
+      const symbol = this.colorize("\u2713", COLOR_GREEN);
+      this.stream.write(`${symbol} ${message}
+`);
+    }
+    return this;
+  }
+  fail(message) {
+    this.stop();
+    if (this.enabled && message) {
+      const symbol = this.colorize("\u2717", COLOR_RED);
+      this.stream.write(`${symbol} ${message}
+`);
+    }
+    return this;
+  }
+  setText(text) {
+    this.text = text;
+    if (this.active) {
+      this.render();
+    }
+    return this;
+  }
+  render() {
+    if (!this.active)
+      return;
+    const frame = FRAMES[this.frameIndex] ?? FRAMES[0];
+    const symbol = this.colorize(frame, COLOR_CYAN);
+    this.stream.write(`${CLEAR_LINE}${symbol} ${this.text}`);
+  }
+  colorize(text, color) {
+    return this.noColor ? text : `${color}${text}${COLOR_RESET}`;
+  }
+  detach() {
+    if (this.exitHandler) {
+      process.removeListener("SIGINT", this.exitHandler);
+      process.removeListener("SIGTERM", this.exitHandler);
+      process.removeListener("exit", this.exitHandler);
+      this.exitHandler = null;
+    }
+    if (this.onStop) {
+      const cb = this.onStop;
+      this.onStop = undefined;
+      cb();
+    }
+    return this;
+  }
+}
+function createNoopSpinner() {
+  const noop = {
+    start: () => noop,
+    stop: () => noop,
+    succeed: () => noop,
+    fail: () => noop,
+    setText: () => noop
+  };
+  return noop;
+}
+
 // src/services/output.service.ts
+var DATE_FORMAT_OPTIONS = {
+  year: "numeric",
+  month: "short",
+  day: "numeric",
+  hour: "2-digit",
+  minute: "2-digit"
+};
 var CONTROL_CHARS = /(\x1b\[[0-9;?]*m)|\x1b\[[0-9;?]*[A-Za-ln-z]|\x1b\][^\x07\x1b]*(?:\x07|\x1b\\)|[\x00-\x08\x0B\x0C\x0E-\x1F\x7F\x9B\x9D]/g;
 function stripControl(value) {
   return value.replace(CONTROL_CHARS, (_match, sgr) => sgr ?? "");
@@ -18507,14 +18724,42 @@ var WRAPPER_ARRAY_KEYS = [
 
 class OutputService {
   noColor;
+  noUnicode;
+  locale;
   jsonFormatOptions = {};
+  activeSpinner = null;
   constructor(options) {
     this.noColor = options?.noColor ?? false;
+    this.noUnicode = options?.noUnicode ?? false;
+    this.locale = options?.locale ?? DEFAULT_LOCALE;
   }
   setJsonFormatOptions(options) {
     this.jsonFormatOptions = { ...options };
   }
+  isJsonMode() {
+    return this.jsonFormatOptions.json === true;
+  }
+  spinner(text) {
+    const enabled = !this.isJsonMode() && !!process.stdout.isTTY && true;
+    if (!enabled) {
+      return createNoopSpinner();
+    }
+    this.activeSpinner?.stop();
+    const spinner = new Spinner(text, {
+      enabled: true,
+      noColor: this.noColor,
+      stream: process.stdout,
+      onStop: () => {
+        if (this.activeSpinner === spinner) {
+          this.activeSpinner = null;
+        }
+      }
+    });
+    this.activeSpinner = spinner;
+    return spinner;
+  }
   async json(data) {
+    this.stopActiveSpinner();
     const { fields, jq } = this.jsonFormatOptions;
     let result = data;
     if (fields && fields.length > 0) {
@@ -18532,9 +18777,11 @@ class OutputService {
     console.log(JSON.stringify(result, null, 2));
   }
   jsonError(data) {
+    this.stopActiveSpinner();
     console.error(JSON.stringify(data));
   }
   table(headers, rows) {
+    this.stopActiveSpinner();
     if (rows.length === 0) {
       return;
     }
@@ -18553,33 +18800,63 @@ class OutputService {
     }
   }
   success(message) {
-    const symbol = this.format("\u2713", source_default.green);
+    this.stopActiveSpinner();
+    const symbol = this.format(this.symbol("\u2713", "OK"), source_default.green);
     console.log(`${symbol} ${stripControl(message)}`);
   }
   error(message) {
-    const symbol = this.format("\u2717", source_default.red);
+    this.stopActiveSpinner();
+    const symbol = this.format(this.symbol("\u2717", "ERR"), source_default.red);
     console.error(`${symbol} ${stripControl(message)}`);
   }
   warning(message) {
-    const symbol = this.format("\u26A0", source_default.yellow);
+    this.stopActiveSpinner();
+    const symbol = this.format(this.symbol("\u26A0", "!!"), source_default.yellow);
     console.warn(`${symbol} ${stripControl(message)}`);
   }
   info(message) {
-    const symbol = this.format("\u2139", source_default.blue);
+    this.stopActiveSpinner();
+    const symbol = this.format(this.symbol("\u2139", "i"), source_default.blue);
     console.log(`${symbol} ${stripControl(message)}`);
   }
+  symbol(unicode, ascii) {
+    return this.noUnicode ? ascii : unicode;
+  }
   text(message) {
+    this.stopActiveSpinner();
     console.log(stripControl(message));
   }
+  separator(width = 60) {
+    this.stopActiveSpinner();
+    if (width <= 0) {
+      console.log("");
+      return;
+    }
+    console.log(this.format(this.symbol("\u2500", "-").repeat(width), source_default.gray));
+  }
+  stopActiveSpinner() {
+    if (this.activeSpinner) {
+      const spinner = this.activeSpinner;
+      this.activeSpinner = null;
+      spinner.stop();
+    }
+  }
+  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", {
-      year: "numeric",
-      month: "short",
-      day: "numeric",
-      hour: "2-digit",
-      minute: "2-digit"
-    });
+    try {
+      return d.toLocaleDateString(this.locale, DATE_FORMAT_OPTIONS);
+    } catch {
+      return d.toLocaleDateString(DEFAULT_LOCALE, DATE_FORMAT_OPTIONS);
+    }
   }
   format(text, formatter) {
     if (this.noColor) {
@@ -18640,7 +18917,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 +19099,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;
     }
   }
@@ -22594,7 +22885,7 @@ function redactRequestUrl(requestUrl, baseUrl) {
     return queryIdx === -1 ? raw : `${raw.slice(0, queryIdx)}?[redacted]`;
   }
 }
-function createApiClient(credentialStore, oauthService) {
+function createApiClient(credentialStore, output, oauthService) {
   const instance = axios_default.create({
     baseURL: BASE_URL,
     headers: {
@@ -22658,7 +22949,9 @@ function createApiClient(credentialStore, oauthService) {
           const delay = getRetryDelay(error, config.__retryCount);
           const status = error.response.status;
           const label = status === 429 ? "Rate limited" : `Server error (${status})`;
-          console.error(`${label}, retrying in ${(delay / 1000).toFixed(1)}s (attempt ${config.__retryCount}/${MAX_RETRIES})...`);
+          if (!output.isJsonMode()) {
+            output.warning(`${label}, retrying in ${(delay / 1000).toFixed(1)}s (attempt ${config.__retryCount}/${MAX_RETRIES})...`);
+          }
           await sleep(delay);
           return instance(config);
         }
@@ -22946,7 +23239,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}
 `);
@@ -27184,6 +27482,7 @@ class BaseCommand {
   }
   async run(options, context) {
     this.output.setJsonFormatOptions({
+      json: !!context.globalOptions.json,
       fields: context.globalOptions.jsonFields,
       jq: context.globalOptions.jq
     });
@@ -27267,6 +27566,33 @@ 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;
+  }
+  truncateText(text, maxLength, opts = {}) {
+    if (opts.noTruncate) {
+      return text;
+    }
+    return this.output.truncate(text, maxLength);
+  }
+  requireConfirmation(confirmed, warning) {
+    if (confirmed)
+      return;
+    throw new BBError({
+      code: 5001 /* VALIDATION_REQUIRED */,
+      message: `${warning}
+Use --yes to confirm.`
+    });
+  }
   parseEnumOption(value, name, allowed) {
     if (!allowed.includes(value)) {
       throw new BBError({
@@ -27555,7 +27881,12 @@ class CloneCommand extends BaseCommand {
   async execute(options, context) {
     const { repository, directory } = options;
     const repoUrl = await this.resolveRepositoryUrl(repository);
-    await this.gitService.clone(repoUrl, directory);
+    const spinner = this.output.spinner(`Cloning ${repository}...`).start();
+    try {
+      await this.gitService.clone(repoUrl, directory);
+    } finally {
+      spinner.stop();
+    }
     const targetDir = directory || this.extractRepoName(repository);
     if (context.globalOptions.json) {
       await this.output.json({
@@ -27968,13 +28299,13 @@ class ListReposCommand extends BaseCommand {
       return;
     }
     if (repos.length === 0) {
-      this.output.text("No repositories found");
+      this.output.info("No repositories found");
       return;
     }
     const rows = repos.map((repo) => [
       repo.full_name ?? "",
       repo.is_private ? "private" : "public",
-      (repo.description || "").substring(0, 50)
+      this.truncateText(repo.description ?? "", 50, context.globalOptions)
     ]);
     this.output.table(["REPOSITORY", "VISIBILITY", "DESCRIPTION"], rows);
   }
@@ -28059,13 +28390,7 @@ class DeleteRepoCommand extends BaseCommand {
       contextOptions.repo = repository;
     }
     const repoContext = await this.contextService.requireRepoContext(contextOptions);
-    if (!yes) {
-      throw new BBError({
-        code: 5001 /* VALIDATION_REQUIRED */,
-        message: `This will permanently delete ${repoContext.workspace}/${repoContext.repoSlug}.
-` + "Use --yes to confirm deletion."
-      });
-    }
+    this.requireConfirmation(yes, `This will permanently delete ${repoContext.workspace}/${repoContext.repoSlug}.`);
     await this.repositoriesApi.repositoriesWorkspaceRepoSlugDelete({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug
@@ -28094,10 +28419,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 +28462,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,17 +28496,8 @@ class RemoveDefaultReviewerCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
-    const repoContext = await this.contextService.requireRepoContext({
-      ...context.globalOptions,
-      ...options
-    });
-    if (!options.yes) {
-      throw new BBError({
-        code: 5001 /* VALIDATION_REQUIRED */,
-        message: `This will remove ${options.username} from the default reviewers of ` + `${repoContext.workspace}/${repoContext.repoSlug}.
-` + "Use --yes to confirm."
-      });
-    }
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    this.requireConfirmation(options.yes, `This will remove ${options.username} from the default reviewers of ` + `${repoContext.workspace}/${repoContext.repoSlug}.`);
     const userResponse = await this.usersApi.usersSelectedUserGet({
       selectedUser: options.username
     });
@@ -28232,10 +28542,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();
@@ -28266,12 +28573,18 @@ class CreatePRCommand extends BaseCommand {
     if (reviewers.length > 0) {
       request.reviewers = reviewers.map((r) => ({ type: "user", uuid: r.uuid }));
     }
-    const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPost({
-      workspace: repoContext.workspace,
-      repoSlug: repoContext.repoSlug,
-      pullrequest: request
-    });
-    const pr = response.data;
+    const spinner = this.output.spinner("Creating pull request...").start();
+    let pr;
+    try {
+      const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPost({
+        workspace: repoContext.workspace,
+        repoSlug: repoContext.repoSlug,
+        pullrequest: request
+      });
+      pr = response.data;
+    } finally {
+      spinner.stop();
+    }
     const links = pr.links;
     if (context.globalOptions.json) {
       await this.output.json(pr);
@@ -28360,10 +28673,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;
@@ -28398,28 +28708,23 @@ class ListPRsCommand extends BaseCommand {
       return;
     }
     if (values.length === 0) {
-      this.output.text(`No ${state.toLowerCase()} pull requests found`);
+      this.output.info(`No ${state.toLowerCase()} pull requests found`);
       return;
     }
+    const arrow = this.output.symbol("\u2192", "->");
     const rows = values.map((pr) => {
       const title = pr.draft ? `[DRAFT] ${pr.title}` : pr.title;
       const source = pr.source;
       const destination = pr.destination;
       return [
         `#${pr.id}`,
-        this.truncate(title ?? "", 50),
+        this.truncateText(title ?? "", 50, context.globalOptions),
         pr.author?.display_name ?? "Unknown",
-        `${source?.branch?.name ?? "unknown"} \u2192 ${destination?.branch?.name ?? "unknown"}`
+        `${source?.branch?.name ?? "unknown"} ${arrow} ${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 +28748,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,
@@ -28470,7 +28772,7 @@ class ViewPRCommand extends BaseCommand {
     const draftLabel = pr.draft ? this.output.yellow(" [DRAFT]") : "";
     this.output.text("");
     this.output.text(`${this.output.bold(`#${pr.id}`)} ${pr.title}${draftLabel} ${stateColor(`[${pr.state}]`)}`);
-    this.output.text(this.output.gray("\u2500".repeat(60)));
+    this.output.separator();
   }
   renderDescription(pr) {
     if (pr.description) {
@@ -28483,7 +28785,7 @@ class ViewPRCommand extends BaseCommand {
     const destination = pr.destination;
     const sourceBranch = this.output.cyan(source?.branch?.name ?? "unknown");
     const destBranch = this.output.cyan(destination?.branch?.name ?? "unknown");
-    const arrow = this.output.gray(" \u2192 ");
+    const arrow = this.output.gray(this.output.symbol(" \u2192 ", " -> "));
     this.output.text(`${this.output.dim("Branch:")}   ${sourceBranch}${arrow}${destBranch}`);
     if (source?.commit?.hash || destination?.commit?.hash) {
       const sourceHash = source?.commit?.hash ? this.output.gray(source.commit.hash.slice(0, 7)) : this.output.gray("unknown");
@@ -28505,7 +28807,7 @@ class ViewPRCommand extends BaseCommand {
     if (mergeCommit?.hash) {
       this.output.text(`${this.output.dim("Merge:")}    ${this.output.magenta(mergeCommit.hash.slice(0, 7))}`);
     }
-    const closeBranchIndicator = pr.close_source_branch ? this.output.green("\u2713") : this.output.gray("\u2717");
+    const closeBranchIndicator = pr.close_source_branch ? this.output.green(this.output.symbol("\u2713", "yes")) : this.output.gray(this.output.symbol("\u2717", "no"));
     this.output.text(`${this.output.dim("Close Src:")} ${closeBranchIndicator} ${this.output.gray("(close source branch on merge)")}`);
     this.output.text(`${this.output.dim("Activity:")} ${pr.comment_count ?? 0} comments \xB7 ${pr.task_count ?? 0} tasks`);
   }
@@ -28514,7 +28816,7 @@ class ViewPRCommand extends BaseCommand {
     const reviewers = participants.filter((p) => p.role === "REVIEWER");
     if (reviewers.length === 0) {
       this.output.text("");
-      this.output.text(this.output.gray("No reviewers assigned"));
+      this.output.info("No reviewers assigned");
       return;
     }
     this.output.text("");
@@ -28529,17 +28831,26 @@ class ViewPRCommand extends BaseCommand {
   }
   getReviewerStatus(reviewer) {
     if (reviewer.approved) {
-      return { icon: this.output.green("\u2713"), label: "approved" };
+      return {
+        icon: this.output.green(this.output.symbol("\u2713", "[OK]")),
+        label: "approved"
+      };
     }
     if (reviewer.state === "changes_requested") {
-      return { icon: this.output.red("\u2717"), label: "changes requested" };
+      return {
+        icon: this.output.red(this.output.symbol("\u2717", "[X]")),
+        label: "changes requested"
+      };
     }
-    return { icon: this.output.yellow("\u25CB"), label: "pending" };
+    return {
+      icon: this.output.yellow(this.output.symbol("\u25CB", "[ ]")),
+      label: "pending"
+    };
   }
   renderFooter(pr) {
     const links = pr.links;
     this.output.text("");
-    this.output.text(this.output.gray("\u2500".repeat(60)));
+    this.output.separator();
     this.output.text(`${this.output.dim("URL:")} ${this.output.underline(this.output.blue(links?.html?.href ?? ""))}`);
     this.output.text("");
   }
@@ -28572,13 +28883,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 +28961,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.truncateText(pr.description, 100, context.globalOptions);
       this.output.text(`  ${this.output.dim("Description:")} ${truncatedDesc}`);
     }
     this.output.text(`  ${this.output.dim("URL:")} ${links?.html?.href}`);
@@ -28674,11 +28982,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"
     };
@@ -28691,13 +28996,19 @@ class MergePRCommand extends BaseCommand {
     if (options.strategy) {
       request.merge_strategy = this.parseEnumOption(options.strategy, "strategy", VALID_STRATEGIES);
     }
-    const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdMergePost({
-      workspace: repoContext.workspace,
-      repoSlug: repoContext.repoSlug,
-      pullRequestId: prId,
-      pullrequestMergeParameters: request
-    });
-    const pr = response.data;
+    const spinner = this.output.spinner(`Merging pull request #${prId}...`).start();
+    let pr;
+    try {
+      const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdMergePost({
+        workspace: repoContext.workspace,
+        repoSlug: repoContext.repoSlug,
+        pullRequestId: prId,
+        pullrequestMergeParameters: request
+      });
+      pr = response.data;
+    } finally {
+      spinner.stop();
+    }
     if (context.globalOptions.json) {
       await this.output.json({
         success: true,
@@ -28722,11 +29033,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 +29063,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 +29095,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 +29134,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 +29198,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 +29418,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({
@@ -29182,7 +29468,7 @@ class ActivityPRCommand extends BaseCommand {
         activityType.toUpperCase(),
         this.getActorName(activity),
         this.formatActivityDate(activity),
-        this.buildActivityDetails(activity, activityType)
+        this.buildActivityDetails(activity, activityType, context.globalOptions)
       ];
     });
     this.output.table(["TYPE", "ACTOR", "DATE", "DETAILS"], rows);
@@ -29237,19 +29523,19 @@ class ActivityPRCommand extends BaseCommand {
     }
     return this.output.formatDate(date);
   }
-  buildActivityDetails(activity, type) {
+  buildActivityDetails(activity, type, globalOptions) {
     switch (type) {
       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.truncateText(content, 80, globalOptions);
         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.truncateText(reason, 80, globalOptions) : "changes requested";
       }
       case "merge":
         return this.formatCommitDetail(activity.merge?.commit?.hash, "merged");
@@ -29262,7 +29548,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.truncateText(activity.update.title, 60, globalOptions)}`;
         }
         if (activity.update?.description) {
           return "description updated";
@@ -29280,12 +29566,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 +29599,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 +29656,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 +29674,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 +29691,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]" : this.truncateText(content, 60, context.globalOptions),
         this.output.formatDate(comment.created_on ?? "")
       ];
     });
@@ -29451,12 +29711,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,19 +29751,10 @@ 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");
-    if (!options.yes) {
-      throw new BBError({
-        code: 5001 /* VALIDATION_REQUIRED */,
-        message: `This will permanently delete comment #${commentId} on PR #${prId}.
-` + "Use --yes to confirm deletion."
-      });
-    }
+    const repoContext = await this.contextService.requireRepoContextFor(options, context);
+    const prId = this.parsePositiveInt(options.prId, "pr-id");
+    const commentId = this.parsePositiveInt(options.commentId, "comment-id");
+    this.requireConfirmation(options.yes, `This will permanently delete comment #${commentId} on PR #${prId}.`);
     await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdCommentsCommentIdDelete({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
@@ -29539,11 +29787,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 +29829,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 +29864,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 +29875,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 +29903,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,
@@ -29690,7 +29928,7 @@ class ChecksPRCommand extends BaseCommand {
       return;
     }
     this.renderHeader(prId, statuses.length);
-    this.renderStatuses(statuses, summary);
+    this.renderStatuses(statuses, summary, context.globalOptions);
   }
   formatStatusForJson(status) {
     return {
@@ -29709,9 +29947,9 @@ class ChecksPRCommand extends BaseCommand {
     this.output.text("");
     const title = this.output.bold("Pull Request #" + prId);
     this.output.text(`${title} - ${count} check${count === 1 ? "" : "s"}`);
-    this.output.text(this.output.gray("-".repeat(60)));
+    this.output.separator();
   }
-  renderStatuses(statuses, summary) {
+  renderStatuses(statuses, summary, globalOptions) {
     const rows = statuses.map((status) => {
       const stateIcon = this.getStateIcon(status.state);
       const stateLabel = this.getStateLabel(status.state);
@@ -29720,7 +29958,7 @@ class ChecksPRCommand extends BaseCommand {
       return [
         `${stateIcon} ${stateLabel}`,
         this.output.bold(name),
-        this.truncate(description, 40),
+        this.truncateText(description, 40, globalOptions),
         status.updated_on ? this.output.formatDate(status.updated_on) : "-"
       ];
     });
@@ -29770,12 +30008,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
@@ -29816,7 +30048,7 @@ class ListSnippetsCommand extends BaseCommand {
       return;
     }
     if (snippets.length === 0) {
-      this.output.text("No snippets found");
+      this.output.info("No snippets found");
       return;
     }
     const rows = snippets.map((snippet) => [
@@ -29877,9 +30109,10 @@ class ViewSnippetCommand extends BaseCommand {
         return;
       }
       this.renderSnippet(snippet, fileNames);
+      const fileSep = this.output.symbol("\u2500\u2500", "--");
       for (const name of fileNames) {
         this.output.text("");
-        this.output.text(this.output.bold(`\u2500\u2500 ${name} \u2500\u2500`));
+        this.output.text(this.output.bold(`${fileSep} ${name} ${fileSep}`));
         this.output.text(contents[name] ?? "");
       }
       return;
@@ -29901,7 +30134,7 @@ class ViewSnippetCommand extends BaseCommand {
     const visibility = snippet.is_private ? "private" : "public";
     this.output.text("");
     this.output.text(`${this.output.bold(String(snippet.id ?? ""))}  ${snippet.title ?? "Untitled"}  ${this.output.gray(`[${visibility}]`)}`);
-    this.output.text(this.output.gray("\u2500".repeat(60)));
+    this.output.separator();
     const creator = getUserDisplayName(snippet.creator);
     if (creator) {
       this.output.text(`Creator:    ${creator}`);
@@ -30062,13 +30295,7 @@ class DeleteSnippetCommand extends BaseCommand {
   }
   async execute(options, context) {
     const workspace = await this.contextService.requireWorkspace(options.workspace ?? context.globalOptions.workspace);
-    if (!options.yes) {
-      throw new BBError({
-        code: 5001 /* VALIDATION_REQUIRED */,
-        message: `This will permanently delete snippet ${options.id}.
-` + "Use --yes to confirm deletion."
-      });
-    }
+    this.requireConfirmation(options.yes, `This will permanently delete snippet ${options.id}.`);
     await this.snippetsApi.snippetsWorkspaceEncodedIdDelete({
       workspace,
       encodedId: options.id
@@ -30188,7 +30415,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.truncateText(content, 60, context.globalOptions)
       ];
     });
     this.output.table(["ID", "AUTHOR", "DATE", "CONTENT"], rows);
@@ -30247,7 +30474,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,14 +30512,8 @@ 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");
-    if (!options.yes) {
-      throw new BBError({
-        code: 5001 /* VALIDATION_REQUIRED */,
-        message: `This will permanently delete comment #${commentId} on snippet ${options.snippetId}.
-` + "Use --yes to confirm deletion."
-      });
-    }
+    const commentId = this.parsePositiveInt(options.commentId, "comment-id");
+    this.requireConfirmation(options.yes, `This will permanently delete comment #${commentId} on snippet ${options.snippetId}.`);
     await this.snippetsApi.snippetsWorkspaceEncodedIdCommentsCommentIdDelete({
       workspace,
       encodedId: options.snippetId,
@@ -30436,7 +30657,7 @@ class ListConfigCommand extends BaseCommand {
       String(value)
     ]);
     if (rows.length === 0) {
-      this.output.text("No configuration set");
+      this.output.info("No configuration set");
     } else {
       this.output.table(["KEY", "VALUE"], rows);
     }
@@ -30535,10 +30756,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 +30820,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 +30904,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;
@@ -30711,7 +30918,8 @@ function registerApiClient(container, token, ctor) {
   container.register(token, () => {
     const credentialStore = container.resolve(ServiceTokens.CredentialStore);
     const oauthService = container.resolve(ServiceTokens.OAuthService);
-    const axiosInstance = createApiClient(credentialStore, oauthService);
+    const outputService = container.resolve(ServiceTokens.OutputService);
+    const axiosInstance = createApiClient(credentialStore, outputService, oauthService);
     return new ctor(undefined, undefined, axiosInstance);
   });
 }
@@ -30726,7 +30934,11 @@ function bootstrap(options = {}) {
   container.register(ServiceTokens.ConfigService, () => new ConfigService);
   container.register(ServiceTokens.CredentialStore, () => container.resolve(ServiceTokens.ConfigService));
   container.register(ServiceTokens.GitService, () => new GitService);
-  container.register(ServiceTokens.OutputService, () => new OutputService({ noColor: options.noColor }));
+  container.register(ServiceTokens.OutputService, () => new OutputService({
+    noColor: options.noColor,
+    noUnicode: options.noUnicode,
+    locale: options.locale
+  }));
   registerCommand(container, ServiceTokens.OAuthService, OAuthService, [
     ServiceTokens.ConfigService,
     ServiceTokens.CredentialStore
@@ -30742,7 +30954,8 @@ function bootstrap(options = {}) {
   container.register(ServiceTokens.SnippetsAxios, () => {
     const credentialStore = container.resolve(ServiceTokens.CredentialStore);
     const oauthService = container.resolve(ServiceTokens.OAuthService);
-    return createApiClient(credentialStore, oauthService);
+    const outputService = container.resolve(ServiceTokens.OutputService);
+    return createApiClient(credentialStore, outputService, oauthService);
   });
   container.register(ServiceTokens.SnippetsApi, () => {
     const axiosInstance = container.resolve(ServiceTokens.SnippetsAxios);
@@ -31087,8 +31300,11 @@ var ROOT_COMPLETIONS = [
   "--version",
   "--json",
   "--no-color",
+  "--no-unicode",
+  "--no-truncate",
   "--workspace",
-  "--repo"
+  "--repo",
+  "--locale"
 ];
 var SUBCOMMAND_COMPLETIONS = new Map([
   ["auth", ["login", "logout", "status", "token"]],
@@ -31155,6 +31371,25 @@ if (process.argv.includes("--get-yargs-completions") || process.env.COMP_LINE) {
     process.exit(0);
   }
 }
+function extractLocaleArg(argv) {
+  for (let i = 0;i < argv.length; i++) {
+    const arg = argv[i];
+    if (arg === undefined) {
+      continue;
+    }
+    if (arg === "--locale") {
+      const next = argv[i + 1];
+      if (typeof next === "string" && !next.startsWith("-")) {
+        return next;
+      }
+      return;
+    }
+    if (arg.startsWith("--locale=")) {
+      return arg.slice("--locale=".length);
+    }
+  }
+  return;
+}
 function resolveNoColorSetting(argv, env2) {
   const hasColorArg = argv.includes("--color");
   const hasNoColorArg = argv.includes("--no-color");
@@ -31171,9 +31406,20 @@ function resolveNoColorSetting(argv, env2) {
   }
   return hasNoColorEnv;
 }
+function resolveNoUnicodeSetting(argv, env2) {
+  if (argv.includes("--no-unicode")) {
+    return true;
+  }
+  return env2.BB_NO_UNICODE !== undefined && env2.BB_NO_UNICODE !== "";
+}
 var noColor = resolveNoColorSetting(process.argv, process.env);
+var noUnicode = resolveNoUnicodeSetting(process.argv, process.env);
 var buildHelpText = createHelpTextBuilder(noColor);
-var container = bootstrap({ noColor });
+var locale = resolveLocale({
+  explicit: extractLocaleArg(process.argv),
+  env: process.env
+});
+var container = bootstrap({ noColor, noUnicode, locale });
 function createContext(program2) {
   const opts = program2.opts();
   const jsonOpt = opts.json;
@@ -31204,6 +31450,8 @@ function createContext(program2) {
       jsonFields,
       jq: jqOpt,
       noColor: opts.color === false,
+      noUnicode: opts.unicode === false || noUnicode,
+      noTruncate: opts.truncate === false,
       workspace: opts.workspace,
       repo: opts.repo
     },
@@ -31233,13 +31481,15 @@ function withGlobalOptions(options, context) {
   };
 }
 var cli = new Command;
-cli.name("bb").description("A command-line interface for Bitbucket Cloud").version(pkg2.version).option("--json [fields]", "Output as JSON; optionally project to a comma-separated field list (e.g. number,title,author.display_name)").option("--jq <expression>", `Filter the JSON output through a jq expression \u2014 runs in-process via embedded jq, requires --json (e.g. '.pullRequests[] | select(.state == "OPEN") | .title')`).option("--no-color", "Disable color output").option("-w, --workspace <workspace>", "Specify workspace").option("-r, --repo <repo>", "Specify repository").addHelpText("after", buildHelpText({
+cli.name("bb").description("A command-line interface for Bitbucket Cloud").version(pkg2.version).option("--json [fields]", "Output as JSON; optionally project to a comma-separated field list (e.g. number,title,author.display_name)").option("--jq <expression>", `Filter the JSON output through a jq expression \u2014 runs in-process via embedded jq, requires --json (e.g. '.pullRequests[] | select(.state == "OPEN") | .title')`).option("--no-color", "Disable color output").option("--no-unicode", "Use ASCII fallbacks for symbols (separators, arrows, status icons) \u2014 also enabled by BB_NO_UNICODE").option("--no-truncate", "Show full values in table output without truncation").option("--locale <locale>", "BCP-47 locale tag for date/time formatting (e.g. de-DE, ja-JP). Falls back to BB_LOCALE, then LC_TIME/LC_ALL/LANG, then en-US.").option("-w, --workspace <workspace>", "Specify workspace").option("-r, --repo <repo>", "Specify repository").addHelpText("after", buildHelpText({
   envVars: {
     BB_USERNAME: "Bitbucket username (fallback for auth login)",
     BB_API_TOKEN: "Bitbucket API token (fallback for auth login)",
     NO_COLOR: "Disable color output when set",
     FORCE_COLOR: "Force color output when set (and not '0')",
-    DEBUG: "Enable HTTP debug logging when 'true'"
+    BB_NO_UNICODE: "Use ASCII fallbacks for symbols when set (any non-empty value)",
+    DEBUG: "Enable HTTP debug logging when 'true'",
+    BB_LOCALE: "BCP-47 locale tag for date/time formatting; --locale takes precedence"
   },
   seeAlso: [
     {
@@ -31263,11 +31513,11 @@ cli.name("bb").description("A command-line interface for Bitbucket Cloud").versi
     const result = await versionService.checkForUpdate();
     if (result?.updateAvailable) {
       output.text("");
-      output.text("\u2500".repeat(50));
-      output.text(`\u26A0 A new version is available: ${result.latestVersion} (you have ${result.currentVersion})`);
-      output.text(`  Run '${versionService.getInstallCommand()}' to update`);
-      output.text(`  Or disable with 'bb config set skipVersionCheck true'`);
-      output.text("\u2500".repeat(50));
+      output.separator(50);
+      output.warning(`A new version is available: ${result.latestVersion} (you have ${result.currentVersion})
+` + `  Run '${versionService.getInstallCommand()}' to update
+` + `  Or disable with 'bb config set skipVersionCheck true'`);
+      output.separator(50);
     }
   } catch {}
   try {
@@ -31574,7 +31824,7 @@ prCmd.command("diff [id]").description("View pull request diff").option("--color
   await runCommand(ServiceTokens.DiffPRCommand, withGlobalOptions({ id, ...options }, context), cli, context);
 });
 var prCommentsCmd = new Command("comments").description("Manage pull request comments");
-prCommentsCmd.command("list <id>").description("List comments on a pull request").option("--limit <number>", "Maximum number of comments (default: 25)").option("--no-truncate", "Show full comment content without truncation").addHelpText("after", buildHelpText({
+prCommentsCmd.command("list <id>").description("List comments on a pull request").option("--limit <number>", "Maximum number of comments (default: 25)").addHelpText("after", buildHelpText({
   examples: [
     "bb pr comments list 42",
     "bb pr comments list 42 --no-truncate",
@@ -31841,3 +32091,6 @@ if (typeof Bun === "undefined") {
   process.exit(1);
 }
 cli.parse(process.argv);
+
+//# debugId=1F677A9695E119AD64756E2164756E21
+//# sourceMappingURL=index.js.map