@pilatos/bitbucket-cli 1.7.1 → 1.8.1

package/dist/index.js

@@ -44211,7 +44211,7 @@ var ServiceTokens = {
 };
 
 // src/services/config.service.ts
-import { join } from "path";
+import { join, win32 } from "path";
 import { homedir } from "os";
 
 // src/types/errors.ts
@@ -44282,9 +44282,24 @@ class ConfigService {
   configDir;
   configFile;
   configCache = null;
-  constructor(configDir) {
-    this.configDir = configDir ?? join(homedir(), ".config", "bb");
-    this.configFile = join(this.configDir, "config.json");
+  constructor(configDir, options = {}) {
+    const platform = options.platform ?? process.platform;
+    const joinPath = platform === "win32" ? win32.join : join;
+    this.configDir = configDir ?? this.resolveDefaultConfigDir({ ...options, platform });
+    this.configFile = joinPath(this.configDir, "config.json");
+  }
+  resolveDefaultConfigDir(options) {
+    const platform = options.platform ?? process.platform;
+    if (platform === "win32") {
+      const appDataDir = options.appData ?? process.env.APPDATA;
+      if (appDataDir) {
+        return win32.join(appDataDir, "bb");
+      }
+      const homeDir2 = options.homeDir ?? homedir();
+      return win32.join(homeDir2, "AppData", "Roaming", "bb");
+    }
+    const homeDir = options.homeDir ?? homedir();
+    return join(homeDir, ".config", "bb");
   }
   async ensureConfigDir() {
     try {
@@ -44354,6 +44369,11 @@ class ConfigService {
       apiToken: credentials.apiToken
     });
   }
+  async clearCredentials() {
+    const config = await this.getConfig();
+    const { username: _username, apiToken: _apiToken, ...rest } = config;
+    await this.setConfig(rest);
+  }
   async clearConfig() {
     this.configCache = null;
     await this.setConfig({});
@@ -45018,6 +45038,9 @@ class OutputService {
   json(data) {
     console.log(JSON.stringify(data, null, 2));
   }
+  jsonError(data) {
+    console.error(JSON.stringify(data));
+  }
   table(headers, rows) {
     if (rows.length === 0) {
       return;
@@ -45103,6 +45126,109 @@ class OutputService {
     return this.format(text, source_default.underline);
   }
 }
+// src/types/config.ts
+var SETTABLE_CONFIG_KEYS = [
+  "defaultWorkspace",
+  "skipVersionCheck",
+  "versionCheckInterval"
+];
+var READABLE_CONFIG_KEYS = [
+  "username",
+  "defaultWorkspace",
+  "skipVersionCheck",
+  "versionCheckInterval"
+];
+function isSettableConfigKey(key) {
+  return SETTABLE_CONFIG_KEYS.includes(key);
+}
+function isReadableConfigKey(key) {
+  return READABLE_CONFIG_KEYS.includes(key);
+}
+function parseBooleanLiteral(value) {
+  const normalized = value.trim().toLowerCase();
+  if (normalized === "true") {
+    return true;
+  }
+  if (normalized === "false") {
+    return false;
+  }
+  return;
+}
+function parsePositiveIntegerLiteral(value) {
+  const normalized = value.trim();
+  if (!/^[1-9]\d*$/.test(normalized)) {
+    return;
+  }
+  const parsed = Number.parseInt(normalized, 10);
+  if (!Number.isSafeInteger(parsed)) {
+    return;
+  }
+  return parsed;
+}
+function parseSettableConfigValue(key, value) {
+  switch (key) {
+    case "defaultWorkspace":
+      return value;
+    case "skipVersionCheck": {
+      const parsed = parseBooleanLiteral(value);
+      if (parsed === undefined) {
+        throw new BBError({
+          code: 5002 /* VALIDATION_INVALID */,
+          message: "Invalid value for 'skipVersionCheck'. Expected 'true' or 'false'.",
+          context: { key, value }
+        });
+      }
+      return parsed;
+    }
+    case "versionCheckInterval": {
+      const parsed = parsePositiveIntegerLiteral(value);
+      if (parsed === undefined) {
+        throw new BBError({
+          code: 5002 /* VALIDATION_INVALID */,
+          message: "Invalid value for 'versionCheckInterval'. Expected a positive integer (1 or greater).",
+          context: { key, value }
+        });
+      }
+      return parsed;
+    }
+  }
+}
+function coerceSkipVersionCheckValue(value) {
+  if (typeof value === "boolean") {
+    return value;
+  }
+  if (typeof value === "string") {
+    return parseBooleanLiteral(value);
+  }
+  return;
+}
+function coerceVersionCheckIntervalValue(value) {
+  if (typeof value === "number") {
+    if (!Number.isSafeInteger(value) || value < 1) {
+      return;
+    }
+    return value;
+  }
+  if (typeof value === "string") {
+    return parsePositiveIntegerLiteral(value);
+  }
+  return;
+}
+function normalizeReadableConfigValue(key, value) {
+  if (value === undefined || value === null) {
+    return;
+  }
+  switch (key) {
+    case "skipVersionCheck":
+      return coerceSkipVersionCheckValue(value);
+    case "versionCheckInterval":
+      return coerceVersionCheckIntervalValue(value);
+    case "username":
+    case "defaultWorkspace":
+      return typeof value === "string" ? value : undefined;
+  }
+}
+
 // src/services/version.service.ts
 var NPM_REGISTRY_URL = "https://registry.npmjs.org/@pilatos/bitbucket-cli";
 var PACKAGE_NAME = "@pilatos/bitbucket-cli";
@@ -45115,7 +45241,8 @@ class VersionService {
     this.currentVersion = currentVersion;
   }
   async checkForUpdate() {
-    const skipCheck = await this.configService.getValue("skipVersionCheck");
+    const skipCheckRaw = await this.configService.getValue("skipVersionCheck");
+    const skipCheck = coerceSkipVersionCheckValue(skipCheckRaw);
     if (skipCheck === true) {
       return null;
     }
@@ -45148,7 +45275,7 @@ class VersionService {
     const now = new Date;
     const timeSinceLastCheck = now.getTime() - lastCheckDate.getTime();
     const intervalDays = await this.configService.getValue("versionCheckInterval");
-    const days = typeof intervalDays === "number" ? intervalDays : 1;
+    const days = coerceVersionCheckIntervalValue(intervalDays) ?? 1;
     const intervalMs = days * 24 * 60 * 60 * 1000;
     return timeSinceLastCheck >= intervalMs;
   }
@@ -51518,7 +51645,9 @@ class BaseCommand {
     }
   }
   handleError(error, context) {
-    if (error instanceof Error) {
+    if (context.globalOptions.json) {
+      this.output.jsonError(this.normalizeErrorForJson(error));
+    } else if (error instanceof Error) {
       this.output.error(error.message);
     } else {
       this.output.error(String(error));
@@ -51527,6 +51656,23 @@ class BaseCommand {
       process.exitCode = 1;
     }
   }
+  normalizeErrorForJson(error) {
+    if (error instanceof BBError) {
+      return error.toJSON();
+    }
+    if (error instanceof Error) {
+      return {
+        name: error.name,
+        code: 9999 /* UNKNOWN */,
+        message: error.message
+      };
+    }
+    return {
+      name: "Error",
+      code: 9999 /* UNKNOWN */,
+      message: String(error)
+    };
+  }
   requireOption(value, name, message) {
     if (value === undefined || value === null || value === "") {
       throw new BBError({
@@ -51553,10 +51699,16 @@ class LoginCommand extends BaseCommand {
     const username = options.username || process.env.BB_USERNAME;
     const apiToken = options.password || process.env.BB_API_TOKEN;
     if (!username) {
-      throw new Error("Username is required. Use --username option or set BB_USERNAME environment variable.");
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: "Username is required. Use --username option or set BB_USERNAME environment variable."
+      });
     }
     if (!apiToken) {
-      throw new Error("API token is required. Use --password option or set BB_API_TOKEN environment variable.");
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: "API token is required. Use --password option or set BB_API_TOKEN environment variable."
+      });
     }
     await this.configService.setCredentials({ username, apiToken });
     try {
@@ -51575,8 +51727,11 @@ class LoginCommand extends BaseCommand {
       }
       this.output.success(`Logged in as ${user.display_name} (${user.username})`);
     } catch (error) {
-      await this.configService.clearConfig();
-      throw new Error(`Authentication failed: ${error instanceof Error ? error.message : String(error)}`);
+      await this.configService.clearCredentials();
+      throw new BBError({
+        code: 1002 /* AUTH_INVALID */,
+        message: `Authentication failed: ${error instanceof Error ? error.message : String(error)}`
+      });
     }
   }
 }
@@ -51591,7 +51746,7 @@ class LogoutCommand extends BaseCommand {
     this.configService = configService;
   }
   async execute(_options, context) {
-    await this.configService.clearConfig();
+    await this.configService.clearCredentials();
     if (context.globalOptions.json) {
       this.output.json({ authenticated: false, success: true });
       return;
@@ -51645,7 +51800,11 @@ class StatusCommand extends BaseCommand {
         this.output.text(`  Default workspace: ${this.output.highlight(config.defaultWorkspace)}`);
       }
     } catch (error) {
-      throw new Error(`Authentication is invalid or expired. Run ${this.output.highlight("bb auth login")} to re-authenticate.`);
+      throw new BBError({
+        code: 1002 /* AUTH_INVALID */,
+        message: `Authentication is invalid or expired. Run ${this.output.highlight("bb auth login")} to re-authenticate.`,
+        cause: error instanceof Error ? error : undefined
+      });
     }
   }
 }
@@ -51662,7 +51821,10 @@ class TokenCommand extends BaseCommand {
   async execute(_options, context) {
     const credentials = await this.configService.getCredentials();
     if (!credentials.username || !credentials.apiToken) {
-      throw new Error("Not authenticated. Run 'bb auth login' first.");
+      throw new BBError({
+        code: 1001 /* AUTH_REQUIRED */,
+        message: "Not authenticated. Run 'bb auth login' first."
+      });
     }
     const token = Buffer.from(`${credentials.username}:${credentials.apiToken}`).toString("base64");
     if (context.globalOptions.json) {
@@ -51710,7 +51872,10 @@ class CloneCommand extends BaseCommand {
     if (parts.length === 1) {
       const config = await this.configService.getConfig();
       if (!config.defaultWorkspace) {
-        throw new Error("No workspace specified. Use workspace/repo format or set a default workspace.");
+        throw new BBError({
+          code: 6002 /* CONTEXT_WORKSPACE_NOT_FOUND */,
+          message: "No workspace specified. Use workspace/repo format or set a default workspace."
+        });
       }
       workspace = config.defaultWorkspace;
       repoSlug = parts[0];
@@ -51718,7 +51883,11 @@ class CloneCommand extends BaseCommand {
       workspace = parts[0];
       repoSlug = parts[1];
     } else {
-      throw new Error("Invalid repository format. Use workspace/repo or a full URL.");
+      throw new BBError({
+        code: 5002 /* VALIDATION_INVALID */,
+        message: "Invalid repository format. Use workspace/repo or a full URL.",
+        context: { repository }
+      });
     }
     return `git@bitbucket.org:${workspace}/${repoSlug}.git`;
   }
@@ -51726,7 +51895,11 @@ class CloneCommand extends BaseCommand {
     const parts = repository.split("/");
     const lastPart = parts.at(-1);
     if (!lastPart) {
-      throw new Error("Invalid repository format.");
+      throw new BBError({
+        code: 5002 /* VALIDATION_INVALID */,
+        message: "Invalid repository format.",
+        context: { repository }
+      });
     }
     return lastPart.replace(".git", "");
   }
@@ -51782,12 +51955,61 @@ class CreateRepoCommand extends BaseCommand {
     }
     const config = await this.configService.getConfig();
     if (!config.defaultWorkspace) {
-      throw new Error("No workspace specified. Use --workspace option or set a default workspace.");
+      throw new BBError({
+        code: 6002 /* CONTEXT_WORKSPACE_NOT_FOUND */,
+        message: "No workspace specified. Use --workspace option or set a default workspace."
+      });
     }
     return config.defaultWorkspace;
   }
 }
 
+// src/services/pagination.ts
+var DEFAULT_LIMIT = 25;
+var MAX_PAGE_LENGTH = 50;
+function parseLimit(limit, fallback = DEFAULT_LIMIT) {
+  if (!limit) {
+    return fallback;
+  }
+  const parsed = Number.parseInt(limit, 10);
+  if (Number.isNaN(parsed) || parsed <= 0) {
+    return fallback;
+  }
+  return parsed;
+}
+async function collectPages(options) {
+  const { fetchPage, shouldInclude } = options;
+  const limit = Math.max(0, options.limit);
+  if (limit === 0) {
+    return [];
+  }
+  const requestedPageSize = options.pageSize ?? limit;
+  const pagelen = Math.max(1, Math.min(requestedPageSize, MAX_PAGE_LENGTH));
+  const items = [];
+  let page = 1;
+  while (items.length < limit) {
+    const data = await fetchPage(page, pagelen);
+    const pageValues = data.values ? Array.from(data.values) : [];
+    if (pageValues.length === 0) {
+      break;
+    }
+    for (const value of pageValues) {
+      if (shouldInclude && !shouldInclude(value)) {
+        continue;
+      }
+      items.push(value);
+      if (items.length >= limit) {
+        return items;
+      }
+    }
+    if (!data.next) {
+      break;
+    }
+    page += 1;
+  }
+  return items;
+}
+
 // src/commands/repo/list.command.ts
 class ListReposCommand extends BaseCommand {
   repositoriesApi;
@@ -51801,11 +52023,18 @@ class ListReposCommand extends BaseCommand {
   }
   async execute(options, context) {
     const workspace = await this.resolveWorkspace(options.workspace ?? context.globalOptions.workspace);
-    const limit = Number.parseInt(options.limit || "25", 10);
-    const response = await this.repositoriesApi.repositoriesWorkspaceGet({
-      workspace
+    const limit = parseLimit(options.limit);
+    const repos = await collectPages({
+      limit,
+      fetchPage: async (page, pagelen) => {
+        const response = await this.repositoriesApi.repositoriesWorkspaceGet({
+          workspace
+        }, {
+          params: { page, pagelen }
+        });
+        return response.data;
+      }
     });
-    const repos = Array.from(response.data.values ?? []).slice(0, limit);
     if (context.globalOptions.json) {
       this.output.json({
         workspace,
@@ -51831,7 +52060,10 @@ class ListReposCommand extends BaseCommand {
     }
     const config = await this.configService.getConfig();
     if (!config.defaultWorkspace) {
-      throw new Error("No workspace specified. Use --workspace option or set a default workspace.");
+      throw new BBError({
+        code: 6002 /* CONTEXT_WORKSPACE_NOT_FOUND */,
+        message: "No workspace specified. Use --workspace option or set a default workspace."
+      });
     }
     return config.defaultWorkspace;
   }
@@ -51916,8 +52148,11 @@ class DeleteRepoCommand extends BaseCommand {
     }
     const repoContext = await this.contextService.requireRepoContext(contextOptions);
     if (!yes) {
-      throw new Error(`This will permanently delete ${repoContext.workspace}/${repoContext.repoSlug}.
-` + "Use --yes to confirm deletion.");
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: `This will permanently delete ${repoContext.workspace}/${repoContext.repoSlug}.
+` + "Use --yes to confirm deletion."
+      });
     }
     await this.repositoriesApi.repositoriesWorkspaceRepoSlugDelete({
       workspace: repoContext.workspace,
@@ -51950,7 +52185,10 @@ class CreatePRCommand extends BaseCommand {
   }
   async execute(options, context) {
     if (!options.title) {
-      throw new Error("Pull request title is required. Use --title option.");
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: "Pull request title is required. Use --title option."
+      });
     }
     const repoContext = await this.contextService.requireRepoContext({
       ...context.globalOptions,
@@ -52014,13 +52252,20 @@ class ListPRsCommand extends BaseCommand {
       ...options
     });
     const state = options.state || "OPEN";
-    const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsGet({
-      workspace: repoContext.workspace,
-      repoSlug: repoContext.repoSlug,
-      state
+    const limit = parseLimit(options.limit);
+    const values = await collectPages({
+      limit,
+      fetchPage: async (page, pagelen) => {
+        const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsGet({
+          workspace: repoContext.workspace,
+          repoSlug: repoContext.repoSlug,
+          state
+        }, {
+          params: { page, pagelen }
+        });
+        return response.data;
+      }
     });
-    const data = response.data;
-    const values = data.values ? Array.from(data.values) : [];
     if (context.globalOptions.json) {
       this.output.json({
         workspace: repoContext.workspace,
@@ -52203,18 +52448,31 @@ class EditPRCommand extends BaseCommand {
       prId = Number.parseInt(options.id, 10);
     } else {
       const currentBranch = await this.gitService.getCurrentBranch();
-      const prsResponse = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsGet({
-        workspace: repoContext.workspace,
-        repoSlug: repoContext.repoSlug,
-        state: "OPEN"
-      });
-      const values = prsResponse.data.values ? Array.from(prsResponse.data.values) : [];
-      const matchingPR = values.find((pr2) => {
-        const source = pr2.source;
-        return source?.branch?.name === currentBranch;
+      const matches = await collectPages({
+        limit: 1,
+        pageSize: MAX_PAGE_LENGTH,
+        fetchPage: async (page, pagelen) => {
+          const response2 = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsGet({
+            workspace: repoContext.workspace,
+            repoSlug: repoContext.repoSlug,
+            state: "OPEN"
+          }, {
+            params: { page, pagelen }
+          });
+          return response2.data;
+        },
+        shouldInclude: (pullRequest) => {
+          const source = pullRequest.source;
+          return source?.branch?.name === currentBranch;
+        }
       });
+      const matchingPR = matches[0];
       if (!matchingPR) {
-        throw new Error(`No open pull request found for current branch '${currentBranch}'. Specify a PR ID explicitly.`);
+        throw new BBError({
+          code: 2002 /* API_NOT_FOUND */,
+          message: `No open pull request found for current branch '${currentBranch}'. Specify a PR ID explicitly.`,
+          context: { branch: currentBranch }
+        });
       }
       prId = matchingPR.id;
     }
@@ -52223,11 +52481,19 @@ class EditPRCommand extends BaseCommand {
       try {
         body = fs.readFileSync(options.bodyFile, "utf-8");
       } catch (err) {
-        throw new Error(`Failed to read file '${options.bodyFile}': ${err instanceof Error ? err.message : "Unknown error"}`);
+        throw new BBError({
+          code: 9999 /* UNKNOWN */,
+          message: `Failed to read file '${options.bodyFile}': ${err instanceof Error ? err.message : "Unknown error"}`,
+          cause: err instanceof Error ? err : undefined,
+          context: { bodyFile: options.bodyFile }
+        });
       }
     }
     if (!options.title && !body) {
-      throw new Error("At least one of --title or --body (or --body-file) is required.");
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: "At least one of --title or --body (or --body-file) is required."
+      });
     }
     const request = {
       type: "pullrequest"
@@ -52444,7 +52710,11 @@ class CheckoutPRCommand extends BaseCommand {
     const branchName = pr.source?.branch?.name;
     const localBranchName = `pr-${prId}`;
     if (!branchName) {
-      throw new Error("Pull request source branch not found");
+      throw new BBError({
+        code: 2002 /* API_NOT_FOUND */,
+        message: "Pull request source branch not found",
+        context: { pullRequestId: prId }
+      });
     }
     await this.gitService.fetch();
     let checkedOutBranch;
@@ -52505,18 +52775,39 @@ class DiffPRCommand extends BaseCommand {
     if (options.id) {
       prId = Number.parseInt(options.id, 10);
       if (Number.isNaN(prId)) {
-        throw new TypeError("Invalid PR ID");
+        throw new BBError({
+          code: 5002 /* VALIDATION_INVALID */,
+          message: "Invalid PR ID",
+          context: { id: options.id }
+        });
       }
     } else {
       const currentBranch = await this.gitService.getCurrentBranch();
-      const prsResponse = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsGet({
-        workspace: repoContext.workspace,
-        repoSlug: repoContext.repoSlug,
-        state: "OPEN"
+      const matches = await collectPages({
+        limit: 1,
+        pageSize: MAX_PAGE_LENGTH,
+        fetchPage: async (page, pagelen) => {
+          const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsGet({
+            workspace: repoContext.workspace,
+            repoSlug: repoContext.repoSlug,
+            state: "OPEN"
+          }, {
+            params: { page, pagelen }
+          });
+          return response.data;
+        },
+        shouldInclude: (pullRequest) => {
+          const source = pullRequest.source;
+          return source?.branch?.name === currentBranch;
+        }
       });
-      const pr = Array.from(prsResponse.data.values ?? []).find((p) => p.source?.branch?.name === currentBranch);
+      const pr = matches[0];
       if (!pr) {
-        throw new Error(`No open pull request found for branch "${currentBranch}"`);
+        throw new BBError({
+          code: 2002 /* API_NOT_FOUND */,
+          message: `No open pull request found for branch "${currentBranch}"`,
+          context: { branch: currentBranch }
+        });
       }
       prId = pr.id;
     }
@@ -52591,11 +52882,13 @@ class DiffPRCommand extends BaseCommand {
       repoSlug,
       pullRequestId: prId
     });
-    const diffUrl = prResponse.data.links?.diff?.href;
-    if (!diffUrl) {
-      throw new Error("Could not get diff URL");
+    const links = prResponse.data.links;
+    const htmlUrl = links?.html?.href;
+    if (htmlUrl) {
+      const normalizedHtmlUrl = htmlUrl.replace(/\/$/, "");
+      return `${normalizedHtmlUrl}/diff`;
     }
-    return diffUrl.replace(/api\.bitbucket\.org\/2\.0\/repositories\/(.*?)\/pullrequests\/(\d+)\/diff/, "bitbucket.org/$1/pull-requests/$2/diff");
+    return `https://bitbucket.org/${workspace}/${repoSlug}/pull-requests/${prId}/diff`;
   }
   async showStat(workspace, repoSlug, prId, useJson) {
     const diffstatResponse = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdDiffstatGet({
@@ -52718,15 +53011,28 @@ class ActivityPRCommand extends BaseCommand {
       ...options
     });
     const prId = Number.parseInt(options.id, 10);
-    const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdActivityGet({
-      workspace: repoContext.workspace,
-      repoSlug: repoContext.repoSlug,
-      pullRequestId: prId
-    });
-    const data = response.data;
-    const values = data?.values ? Array.from(data.values) : [];
     const filterTypes = this.parseTypeFilter(options.type);
-    const activities = filterTypes.length > 0 ? values.filter((activity) => filterTypes.includes(this.getActivityType(activity))) : values;
+    const limit = parseLimit(options.limit);
+    const activities = await collectPages({
+      limit,
+      fetchPage: async (page, pagelen) => {
+        const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdActivityGet({
+          workspace: repoContext.workspace,
+          repoSlug: repoContext.repoSlug,
+          pullRequestId: prId
+        }, {
+          params: { page, pagelen }
+        });
+        return response.data;
+      },
+      shouldInclude: (activity) => {
+        if (filterTypes.length === 0) {
+          return true;
+        }
+        return filterTypes.includes(this.getActivityType(activity));
+      }
+    });
+    const typedActivities = activities;
     if (context.globalOptions.json) {
       this.output.json({
         workspace: repoContext.workspace,
@@ -52735,12 +53041,12 @@ class ActivityPRCommand extends BaseCommand {
         filters: {
           types: filterTypes
         },
-        count: activities.length,
-        activities
+        count: typedActivities.length,
+        activities: typedActivities
       });
       return;
     }
-    if (activities.length === 0) {
+    if (typedActivities.length === 0) {
       if (filterTypes.length > 0) {
         this.output.info("No activity entries matched the requested filter");
       } else {
@@ -52748,7 +53054,7 @@ class ActivityPRCommand extends BaseCommand {
       }
       return;
     }
-    const rows = activities.map((activity) => {
+    const rows = typedActivities.map((activity) => {
       const activityType = this.getActivityType(activity);
       return [
         activityType.toUpperCase(),
@@ -52866,31 +53172,79 @@ class CommentPRCommand extends BaseCommand {
     this.contextService = contextService;
   }
   async execute(options, context) {
+    if ((options.lineTo || options.lineFrom) && !options.file) {
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: "--file is required when using --line-to or --line-from"
+      });
+    }
+    if (options.file && !options.lineTo && !options.lineFrom) {
+      throw new BBError({
+        code: 5001 /* VALIDATION_REQUIRED */,
+        message: "At least one of --line-to or --line-from is required when using --file"
+      });
+    }
+    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 inline = options.file ? {
+      path: options.file,
+      ...options.lineTo ? { to: Number.parseInt(options.lineTo, 10) } : {},
+      ...options.lineFrom ? { from: Number.parseInt(options.lineFrom, 10) } : {}
+    } : undefined;
+    const body = {
+      content: {
+        raw: options.message
+      },
+      ...inline ? { inline } : {}
+    };
     const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdCommentsPost({
       workspace: repoContext.workspace,
       repoSlug: repoContext.repoSlug,
       pullRequestId: prId,
-      body: {
-        type: "pullrequest_comment",
-        content: {
-          raw: options.message
-        }
-      }
+      body
     });
     if (context.globalOptions.json) {
-      this.output.json({
+      const jsonOutput = {
         success: true,
         pullRequestId: prId,
         comment: response.data
-      });
+      };
+      if (inline) {
+        jsonOutput.inline = inline;
+      }
+      this.output.json(jsonOutput);
       return;
     }
-    this.output.success(`Added comment to pull request #${prId}`);
+    if (inline) {
+      if (inline.to) {
+        this.output.success(`Added inline comment on ${inline.path}:${inline.to} to pull request #${prId}`);
+      } else {
+        this.output.success(`Added inline comment on ${inline.path} (old line ${inline.from}) to pull request #${prId}`);
+      }
+    } else {
+      this.output.success(`Added comment to pull request #${prId}`);
+    }
   }
 }
 
@@ -52911,13 +53265,20 @@ class ListCommentsPRCommand extends BaseCommand {
       ...options
     });
     const prId = Number.parseInt(options.id, 10);
-    const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdCommentsGet({
-      workspace: repoContext.workspace,
-      repoSlug: repoContext.repoSlug,
-      pullRequestId: prId
+    const limit = parseLimit(options.limit);
+    const values = await collectPages({
+      limit,
+      fetchPage: async (page, pagelen) => {
+        const response = await this.pullrequestsApi.repositoriesWorkspaceRepoSlugPullrequestsPullRequestIdCommentsGet({
+          workspace: repoContext.workspace,
+          repoSlug: repoContext.repoSlug,
+          pullRequestId: prId
+        }, {
+          params: { page, pagelen }
+        });
+        return response.data;
+      }
     });
-    const data = response.data;
-    const values = data.values ? Array.from(data.values) : [];
     if (context.globalOptions.json) {
       this.output.json({
         pullRequestId: prId,
@@ -53306,25 +53667,6 @@ class ChecksPRCommand extends BaseCommand {
   }
 }
 
-// src/types/config.ts
-var SETTABLE_CONFIG_KEYS = [
-  "defaultWorkspace",
-  "skipVersionCheck",
-  "versionCheckInterval"
-];
-var READABLE_CONFIG_KEYS = [
-  "username",
-  "defaultWorkspace",
-  "skipVersionCheck",
-  "versionCheckInterval"
-];
-function isSettableConfigKey(key) {
-  return SETTABLE_CONFIG_KEYS.includes(key);
-}
-function isReadableConfigKey(key) {
-  return READABLE_CONFIG_KEYS.includes(key);
-}
-
 // src/commands/config/get.command.ts
 class GetConfigCommand extends BaseCommand {
   configService;
@@ -53338,12 +53680,21 @@ class GetConfigCommand extends BaseCommand {
   async execute(options, context) {
     const { key } = options;
     if (GetConfigCommand.HIDDEN_KEYS.includes(key)) {
-      throw new Error(`Cannot display '${key}' - use 'bb auth token' to get authentication credentials`);
+      throw new BBError({
+        code: 4003 /* CONFIG_INVALID_KEY */,
+        message: `Cannot display '${key}' - use 'bb auth token' to get authentication credentials`,
+        context: { key }
+      });
     }
     if (!isReadableConfigKey(key)) {
-      throw new Error(`Unknown config key '${key}'. Valid keys: username, defaultWorkspace`);
+      throw new BBError({
+        code: 4003 /* CONFIG_INVALID_KEY */,
+        message: `Unknown config key '${key}'. Valid keys: ${READABLE_CONFIG_KEYS.join(", ")}`,
+        context: { key }
+      });
     }
-    const value = await this.configService.getValue(key);
+    const rawValue = await this.configService.getValue(key);
+    const value = normalizeReadableConfigValue(key, rawValue);
     if (context.globalOptions.json) {
       this.output.json({
         key,
@@ -53351,7 +53702,7 @@ class GetConfigCommand extends BaseCommand {
       });
       return;
     }
-    this.output.text(String(value || ""));
+    this.output.text(String(value ?? ""));
   }
 }
 
@@ -53368,21 +53719,30 @@ class SetConfigCommand extends BaseCommand {
   async execute(options, context) {
     const { key, value } = options;
     if (SetConfigCommand.PROTECTED_KEYS.includes(key)) {
-      throw new Error(`Cannot set '${key}' directly. Use 'bb auth login' to configure authentication.`);
+      throw new BBError({
+        code: 4003 /* CONFIG_INVALID_KEY */,
+        message: `Cannot set '${key}' directly. Use 'bb auth login' to configure authentication.`,
+        context: { key }
+      });
     }
     if (!isSettableConfigKey(key)) {
-      throw new Error(`Unknown config key '${key}'. Valid keys: defaultWorkspace`);
+      throw new BBError({
+        code: 4003 /* CONFIG_INVALID_KEY */,
+        message: `Unknown config key '${key}'. Valid keys: ${SETTABLE_CONFIG_KEYS.join(", ")}`,
+        context: { key }
+      });
     }
-    await this.configService.setValue(key, value);
+    const parsedValue = parseSettableConfigValue(key, value);
+    await this.configService.setValue(key, parsedValue);
     if (context.globalOptions.json) {
       this.output.json({
         success: true,
         key,
-        value
+        value: parsedValue
       });
       return;
     }
-    this.output.success(`Set ${key} = ${value}`);
+    this.output.success(`Set ${key} = ${parsedValue}`);
   }
 }
 
@@ -53397,21 +53757,37 @@ class ListConfigCommand extends BaseCommand {
   }
   async execute(_options, context) {
     const config = await this.configService.getConfig();
-    const displayConfig = {
-      username: config.username || "",
-      defaultWorkspace: config.defaultWorkspace || "",
-      apiToken: config.apiToken ? "********" : ""
-    };
+    const displayConfig = {};
+    if (config.username) {
+      displayConfig.username = config.username;
+    }
+    if (config.defaultWorkspace) {
+      displayConfig.defaultWorkspace = config.defaultWorkspace;
+    }
+    if (config.apiToken) {
+      displayConfig.apiToken = "********";
+    }
+    const skipVersionCheck = coerceSkipVersionCheckValue(config.skipVersionCheck);
+    if (skipVersionCheck !== undefined) {
+      displayConfig.skipVersionCheck = skipVersionCheck;
+    }
+    const versionCheckInterval = coerceVersionCheckIntervalValue(config.versionCheckInterval);
+    if (versionCheckInterval !== undefined) {
+      displayConfig.versionCheckInterval = versionCheckInterval;
+    }
     if (context.globalOptions.json) {
       this.output.json({
         configPath: this.configService.getConfigPath(),
-        config: Object.fromEntries(Object.entries(displayConfig).filter(([, value]) => value !== ""))
+        config: displayConfig
       });
       return;
     }
     this.output.text(this.output.dim(`Config file: ${this.configService.getConfigPath()}`));
     this.output.text("");
-    const rows = Object.entries(displayConfig).filter(([, value]) => value !== "").map(([key, value]) => [key, value]);
+    const rows = Object.entries(displayConfig).map(([key, value]) => [
+      key,
+      String(value)
+    ]);
     if (rows.length === 0) {
       this.output.text("No configuration set");
       return;
@@ -53448,7 +53824,11 @@ class InstallCompletionCommand extends BaseCommand {
       this.output.success("Shell completions installed successfully!");
       this.output.text("Restart your shell or source your profile to enable completions.");
     } catch (error) {
-      throw new Error(`Failed to install completions: ${error}`);
+      throw new BBError({
+        code: 9999 /* UNKNOWN */,
+        message: `Failed to install completions: ${error}`,
+        cause: error instanceof Error ? error : undefined
+      });
     }
   }
 }
@@ -53479,7 +53859,11 @@ class UninstallCompletionCommand extends BaseCommand {
       }
       this.output.success("Shell completions uninstalled successfully!");
     } catch (error) {
-      throw new Error(`Failed to uninstall completions: ${error}`);
+      throw new BBError({
+        code: 9999 /* UNKNOWN */,
+        message: `Failed to uninstall completions: ${error}`,
+        cause: error instanceof Error ? error : undefined
+      });
     }
   }
 }
@@ -53898,7 +54282,7 @@ prCmd.command("checkout <id>").description("Checkout a pull request locally").ac
   const context = createContext(cli);
   await runCommand(ServiceTokens.CheckoutPRCommand, withGlobalOptions({ id, ...options }, context), cli, context);
 });
-prCmd.command("diff [id]").description("View pull request diff").option("--color <when>", "Colorize output (auto, always, never)", "auto").option("--name-only", "Show only names of changed files").option("--stat", "Show diffstat").option("-w, --web", "Open diff in web browser").action(async (id, options) => {
+prCmd.command("diff [id]").description("View pull request diff").option("--color <when>", "Colorize output (auto, always, never)", "auto").option("--name-only", "Show only names of changed files").option("--stat", "Show diffstat").option("--web", "Open diff in web browser").action(async (id, options) => {
   const context = createContext(cli);
   await runCommand(ServiceTokens.DiffPRCommand, withGlobalOptions({ id, ...options }, context), cli, context);
 });
@@ -53907,9 +54291,9 @@ prCommentsCmd.command("list <id>").description("List comments on a pull request"
   const context = createContext(cli);
   await runCommand(ServiceTokens.ListCommentsPRCommand, withGlobalOptions({ id, ...options }, context), cli, context);
 });
-prCommentsCmd.command("add <id> <message>").description("Add a comment to a pull request").action(async (id, message, options) => {
+prCommentsCmd.command("add <id> <message>").description("Add a comment to a pull request").option("--file <path>", "File path in the diff for inline comment").option("--line-to <number>", "Line number in the new file version").option("--line-from <number>", "Line number in the old file version").action(async (id, message, options) => {
   const context = createContext(cli);
-  await runCommand(ServiceTokens.CommentPRCommand, withGlobalOptions({ id, message }, context), cli, context);
+  await runCommand(ServiceTokens.CommentPRCommand, withGlobalOptions({ id, message, ...options }, context), cli, context);
 });
 prCommentsCmd.command("edit <pr-id> <comment-id> <message>").description("Edit a comment on a pull request").action(async (prId, commentId, message, options) => {
   const context = createContext(cli);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pilatos/bitbucket-cli",
-  "version": "1.7.1",
+  "version": "1.8.1",
   "description": "A command-line interface for Bitbucket Cloud",
   "author": "",
   "license": "MIT",