@pilatos/bitbucket-cli 1.1.0 → 1.2.0

package/README.md

@@ -83,7 +83,7 @@ bb pr list
 |----------|----------|
 | **Authentication** | `login`, `logout`, `status`, `token` |
 | **Repositories** | `clone`, `create`, `list`, `view`, `delete` |
-| **Pull Requests** | `create`, `list`, `view`, `edit`, `merge`, `approve`, `decline`, `checkout`, `diff` |
+| **Pull Requests** | `create`, `list`, `view`, `edit`, `merge`, `approve`, `decline`, `checkout`, `diff`, `comment`, `comments` |
 | **Configuration** | `get`, `set`, `list` |
 | **Shell Completion** | `install`, `uninstall` |
 

package/dist/index.js

@@ -33264,6 +33264,10 @@ var ServiceTokens = {
   DeclinePRCommand: "DeclinePRCommand",
   CheckoutPRCommand: "CheckoutPRCommand",
   DiffPRCommand: "DiffPRCommand",
+  CommentPRCommand: "CommentPRCommand",
+  ListCommentsPRCommand: "ListCommentsPRCommand",
+  EditCommentPRCommand: "EditCommentPRCommand",
+  DeleteCommentPRCommand: "DeleteCommentPRCommand",
   GetConfigCommand: "GetConfigCommand",
   SetConfigCommand: "SetConfigCommand",
   ListConfigCommand: "ListConfigCommand",
@@ -34279,6 +34283,11 @@ class HttpClient {
         body: body ? JSON.stringify(body) : undefined,
         signal: controller.signal
       });
+      if (process.env.DEBUG === "true") {
+        console.debug(`[HTTP] ${method} ${url} - ${response.status}`);
+        console.debug(`[HTTP] Response Headers:`, Object.fromEntries(response.headers.entries()));
+        console.debug(`[HTTP] Response Body:`, await response.clone().text());
+      }
       clearTimeout(timeoutId);
       return acceptText ? this.handleTextResponse(response) : this.handleResponse(response);
     } catch (error) {
@@ -34394,6 +34403,16 @@ class UserRepository {
     return this.httpClient.get("/user");
   }
 }
+// src/constants.ts
+var API_PAGELEN_LIMITS = {
+  PULL_REQUESTS: 50,
+  REPOSITORIES: 100
+};
+var DEFAULT_PAGELEN = {
+  PULL_REQUESTS: 25,
+  REPOSITORIES: 25
+};
+
 // src/repositories/repo.repository.ts
 class RepoRepository {
   httpClient;
@@ -34404,7 +34423,8 @@ class RepoRepository {
     return this.httpClient.get(`/repositories/${encodeURIComponent(workspace)}/${encodeURIComponent(repoSlug)}`);
   }
   async list(workspace, limit = 25) {
-    return this.httpClient.get(`/repositories/${encodeURIComponent(workspace)}?pagelen=${limit}`);
+    const safeLimit = Math.min(limit, API_PAGELEN_LIMITS.REPOSITORIES);
+    return this.httpClient.get(`/repositories/${encodeURIComponent(workspace)}?pagelen=${safeLimit}`);
   }
   async create(workspace, request) {
     const repoSlug = request.name.toLowerCase().replace(/\s+/g, "-");
@@ -34428,7 +34448,8 @@ class PullRequestRepository {
     return this.httpClient.get(this.buildPath(workspace, repoSlug, `/${id}`));
   }
   async list(workspace, repoSlug, state = "OPEN", limit = 25) {
-    return this.httpClient.get(this.buildPath(workspace, repoSlug, `?state=${state}&pagelen=${limit}`));
+    const safeLimit = Math.min(limit, API_PAGELEN_LIMITS.PULL_REQUESTS);
+    return this.httpClient.get(this.buildPath(workspace, repoSlug, `?state=${state}&pagelen=${safeLimit}`));
   }
   async create(workspace, repoSlug, request) {
     return this.httpClient.post(this.buildPath(workspace, repoSlug), request);
@@ -34440,10 +34461,10 @@ class PullRequestRepository {
     return this.httpClient.post(this.buildPath(workspace, repoSlug, `/${id}/merge`), request);
   }
   async approve(workspace, repoSlug, id) {
-    return this.httpClient.post(this.buildPath(workspace, repoSlug, `/${id}/approve`));
+    return this.httpClient.post(this.buildPath(workspace, repoSlug, `/${id}/approve`), {});
   }
   async decline(workspace, repoSlug, id) {
-    return this.httpClient.post(this.buildPath(workspace, repoSlug, `/${id}/decline`));
+    return this.httpClient.post(this.buildPath(workspace, repoSlug, `/${id}/decline`), {});
   }
   async getDiff(workspace, repoSlug, id) {
     return this.httpClient.getText(this.buildPath(workspace, repoSlug, `/${id}/diff`));
@@ -34451,6 +34472,22 @@ class PullRequestRepository {
   async getDiffstat(workspace, repoSlug, id) {
     return this.httpClient.get(this.buildPath(workspace, repoSlug, `/${id}/diffstat`));
   }
+  async listComments(workspace, repoSlug, prId, limit = 25) {
+    const safeLimit = Math.min(limit, API_PAGELEN_LIMITS.PULL_REQUESTS);
+    return this.httpClient.get(`/repositories/${encodeURIComponent(workspace)}/${encodeURIComponent(repoSlug)}/pullrequests/${prId}/comments?pagelen=${safeLimit}`);
+  }
+  async getComment(workspace, repoSlug, prId, commentId) {
+    return this.httpClient.get(`/repositories/${encodeURIComponent(workspace)}/${encodeURIComponent(repoSlug)}/pullrequests/${prId}/comments/${commentId}`);
+  }
+  async createComment(workspace, repoSlug, prId, content) {
+    return this.httpClient.post(`/repositories/${encodeURIComponent(workspace)}/${encodeURIComponent(repoSlug)}/pullrequests/${prId}/comments`, { content: { raw: content } });
+  }
+  async updateComment(workspace, repoSlug, prId, commentId, content) {
+    return this.httpClient.put(`/repositories/${encodeURIComponent(workspace)}/${encodeURIComponent(repoSlug)}/pullrequests/${prId}/comments/${commentId}`, { content: { raw: content } });
+  }
+  async deleteComment(workspace, repoSlug, prId, commentId) {
+    return this.httpClient.delete(`/repositories/${encodeURIComponent(workspace)}/${encodeURIComponent(repoSlug)}/pullrequests/${prId}/comments/${commentId}`);
+  }
 }
 // src/core/base-command.ts
 class BaseCommand {
@@ -35109,7 +35146,7 @@ class EditPRCommand extends BaseCommand {
         return branchResult;
       }
       const currentBranch = branchResult.value;
-      const prsResult = await this.prRepository.list(workspace, repoSlug, "OPEN");
+      const prsResult = await this.prRepository.list(workspace, repoSlug, "OPEN", DEFAULT_PAGELEN.PULL_REQUESTS);
       if (!prsResult.success) {
         this.handleResult(prsResult, context);
         return prsResult;
@@ -35377,7 +35414,7 @@ class DiffPRCommand extends BaseCommand {
         return Result.err(error);
       }
       const currentBranch = currentBranchResult.value;
-      const prsResult = await this.prRepository.list(workspace, repoSlug, "OPEN", 100);
+      const prsResult = await this.prRepository.list(workspace, repoSlug, "OPEN", DEFAULT_PAGELEN.PULL_REQUESTS);
       if (!prsResult.success) {
         this.handleResult(prsResult, context);
         return prsResult;
@@ -35543,6 +35580,139 @@ class DiffPRCommand extends BaseCommand {
   }
 }
 
+// src/commands/pr/comment.command.ts
+class CommentPRCommand extends BaseCommand {
+  prRepository;
+  contextService;
+  name = "comment";
+  description = "Add a comment to a pull request";
+  constructor(prRepository, contextService, output) {
+    super(output);
+    this.prRepository = prRepository;
+    this.contextService = contextService;
+  }
+  async execute(options, context) {
+    const repoContextResult = await this.contextService.requireRepoContext({
+      ...context.globalOptions,
+      ...options
+    });
+    if (!repoContextResult.success) {
+      this.handleResult(repoContextResult, context);
+      return repoContextResult;
+    }
+    const { workspace, repoSlug } = repoContextResult.value;
+    const prId = parseInt(options.id, 10);
+    const result = await this.prRepository.createComment(workspace, repoSlug, prId, options.message);
+    this.handleResult(result, context, () => {
+      this.output.success(`Added comment to pull request #${prId}`);
+    });
+    return result;
+  }
+}
+
+// src/commands/pr/comments.list.command.ts
+class ListCommentsPRCommand extends BaseCommand {
+  prRepository;
+  contextService;
+  name = "comments";
+  description = "List comments on a pull request";
+  constructor(prRepository, contextService, output) {
+    super(output);
+    this.prRepository = prRepository;
+    this.contextService = contextService;
+  }
+  async execute(options, context) {
+    const repoContextResult = await this.contextService.requireRepoContext({
+      ...context.globalOptions,
+      ...options
+    });
+    if (!repoContextResult.success) {
+      this.handleResult(repoContextResult, context);
+      return repoContextResult;
+    }
+    const { workspace, repoSlug } = repoContextResult.value;
+    const prId = parseInt(options.id, 10);
+    const limit = options.limit ? parseInt(options.limit, 10) : 25;
+    const result = await this.prRepository.listComments(workspace, repoSlug, prId, limit);
+    this.handleResult(result, context, (data) => {
+      if (data.values.length === 0) {
+        this.output.info("No comments found on this pull request");
+        return;
+      }
+      const rows = data.values.map((comment) => [
+        comment.id.toString(),
+        comment.author?.username ?? comment.author?.display_name ?? "Unknown",
+        comment.content.raw.slice(0, 60) + (comment.content.raw.length > 60 ? "..." : ""),
+        this.output.formatDate(comment.created_on)
+      ]);
+      this.output.table(["ID", "Author", "Content", "Date"], rows);
+    });
+    return result;
+  }
+}
+
+// src/commands/pr/comments.edit.command.ts
+class EditCommentPRCommand extends BaseCommand {
+  prRepository;
+  contextService;
+  name = "edit";
+  description = "Edit a comment on a pull request";
+  constructor(prRepository, contextService, output) {
+    super(output);
+    this.prRepository = prRepository;
+    this.contextService = contextService;
+  }
+  async execute(options, context) {
+    const repoContextResult = await this.contextService.requireRepoContext({
+      ...context.globalOptions,
+      ...options
+    });
+    if (!repoContextResult.success) {
+      this.handleResult(repoContextResult, context);
+      return repoContextResult;
+    }
+    const { workspace, repoSlug } = repoContextResult.value;
+    const prId = parseInt(options.prId, 10);
+    const commentId = parseInt(options.commentId, 10);
+    const result = await this.prRepository.updateComment(workspace, repoSlug, prId, commentId, options.message);
+    this.handleResult(result, context, () => {
+      this.output.success(`Updated comment #${commentId}`);
+    });
+    return result;
+  }
+}
+
+// src/commands/pr/comments.delete.command.ts
+class DeleteCommentPRCommand extends BaseCommand {
+  prRepository;
+  contextService;
+  name = "delete";
+  description = "Delete a comment on a pull request";
+  constructor(prRepository, contextService, output) {
+    super(output);
+    this.prRepository = prRepository;
+    this.contextService = contextService;
+  }
+  async execute(options, context) {
+    const repoContextResult = await this.contextService.requireRepoContext({
+      ...context.globalOptions,
+      ...options
+    });
+    if (!repoContextResult.success) {
+      this.handleResult(repoContextResult, context);
+      return repoContextResult;
+    }
+    const { workspace, repoSlug } = repoContextResult.value;
+    const prId = parseInt(options.prId, 10);
+    const commentId = parseInt(options.commentId, 10);
+    const result = await this.prRepository.deleteComment(workspace, repoSlug, prId, commentId);
+    this.handleResult(result, context, () => {
+      this.output.success(`Deleted comment #${commentId} from PR #${prId}`);
+    });
+    return result;
+  }
+}
+
 // src/types/config.ts
 var SETTABLE_CONFIG_KEYS = ["defaultWorkspace"];
 var READABLE_CONFIG_KEYS = ["username", "defaultWorkspace"];
@@ -35859,6 +36029,30 @@ function bootstrap() {
     const output = container.resolve(ServiceTokens.OutputService);
     return new DiffPRCommand(prRepo, contextService, gitService, output);
   });
+  container.register(ServiceTokens.CommentPRCommand, () => {
+    const prRepo = container.resolve(ServiceTokens.PullRequestRepository);
+    const contextService = container.resolve(ServiceTokens.ContextService);
+    const output = container.resolve(ServiceTokens.OutputService);
+    return new CommentPRCommand(prRepo, contextService, output);
+  });
+  container.register(ServiceTokens.ListCommentsPRCommand, () => {
+    const prRepo = container.resolve(ServiceTokens.PullRequestRepository);
+    const contextService = container.resolve(ServiceTokens.ContextService);
+    const output = container.resolve(ServiceTokens.OutputService);
+    return new ListCommentsPRCommand(prRepo, contextService, output);
+  });
+  container.register(ServiceTokens.EditCommentPRCommand, () => {
+    const prRepo = container.resolve(ServiceTokens.PullRequestRepository);
+    const contextService = container.resolve(ServiceTokens.ContextService);
+    const output = container.resolve(ServiceTokens.OutputService);
+    return new EditCommentPRCommand(prRepo, contextService, output);
+  });
+  container.register(ServiceTokens.DeleteCommentPRCommand, () => {
+    const prRepo = container.resolve(ServiceTokens.PullRequestRepository);
+    const contextService = container.resolve(ServiceTokens.ContextService);
+    const output = container.resolve(ServiceTokens.OutputService);
+    return new DeleteCommentPRCommand(prRepo, contextService, output);
+  });
   container.register(ServiceTokens.GetConfigCommand, () => {
     const configService = container.resolve(ServiceTokens.ConfigService);
     const output = container.resolve(ServiceTokens.OutputService);
@@ -35909,7 +36103,7 @@ if (process.argv.includes("--get-yargs-completions") || process.env.COMP_LINE) {
     } else if (env2.prev === "repo") {
       completions.push("clone", "create", "list", "view", "delete");
     } else if (env2.prev === "pr") {
-      completions.push("create", "list", "view", "edit", "merge", "approve", "decline", "checkout", "diff");
+      completions.push("create", "list", "view", "edit", "merge", "approve", "decline", "checkout", "diff", "comments");
     } else if (env2.prev === "config") {
       completions.push("get", "set", "list");
     } else if (env2.prev === "completion") {
@@ -36083,7 +36277,41 @@ prCmd.command("diff [id]").description("View pull request diff").option("--color
     process.exit(1);
   }
 });
+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)").action(async (id, options) => {
+  const cmd = container.resolve(ServiceTokens.ListCommentsPRCommand);
+  const context = createContext(cli);
+  const result = await cmd.execute(withGlobalOptions({ id, ...options }, context), context);
+  if (!result.success) {
+    process.exit(1);
+  }
+});
+prCommentsCmd.command("add <id> <message>").description("Add a comment to a pull request").action(async (id, message, options) => {
+  const cmd = container.resolve(ServiceTokens.CommentPRCommand);
+  const context = createContext(cli);
+  const result = await cmd.execute(withGlobalOptions({ id, message }, context), context);
+  if (!result.success) {
+    process.exit(1);
+  }
+});
+prCommentsCmd.command("edit <pr-id> <comment-id> <message>").description("Edit a comment on a pull request").action(async (prId, commentId, message, options) => {
+  const cmd = container.resolve(ServiceTokens.EditCommentPRCommand);
+  const context = createContext(cli);
+  const result = await cmd.execute(withGlobalOptions({ prId, commentId, message }, context), context);
+  if (!result.success) {
+    process.exit(1);
+  }
+});
+prCommentsCmd.command("delete <pr-id> <comment-id>").description("Delete a comment on a pull request").action(async (prId, commentId, options) => {
+  const cmd = container.resolve(ServiceTokens.DeleteCommentPRCommand);
+  const context = createContext(cli);
+  const result = await cmd.execute(withGlobalOptions({ prId, commentId }, context), context);
+  if (!result.success) {
+    process.exit(1);
+  }
+});
 cli.addCommand(prCmd);
+prCmd.addCommand(prCommentsCmd);
 var configCmd = new Command("config").description("Manage configuration");
 configCmd.command("get <key>").description("Get a configuration value").action(async (key) => {
   const cmd = container.resolve(ServiceTokens.GetConfigCommand);

package/package.json

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