Skip to content

feat: add --details flag and thread nesting to bkt pr comments #108

@ivulub

Description

@ivulub

Description

bkt pr comments currently shows only ID, author, and truncated text. This PR adds:

  • File path and line number for inline comments
  • Resolved status for comment threads
  • Task vs Comment distinction (severity BLOCKER = task)
  • Task completion status
  • Nested reply threads (replies are silently dropped)

Proposed solution

  • Add --details flag for verbose multi-line output per comment
  • Distinguish "Comment" vs "Task" in the header
  • Use "Resolved: yes" for comment threads, "Complete: yes/no" for tasks (matching Bitbucket's own terminology)
  • Flatten nested comment replies with depth-based indentation
  • Show indented replies in default (non-details) mode too

Depends on #107 (activities endpoint fix).

Note: overlap between pr comments and pr task

bkt pr task list returns 404 on some DC instances — the /tasks endpoint doesn't exist there. Tasks are only surfaced as comments with severity: "BLOCKER" via the /activities endpoint.

This creates some overlap between pr comments --details and pr task list, since both can show tasks. However, pr comments --details serves a different use case: seeing all open items (comments and tasks) in one view, which is what a reviewee typically needs when working through a PR.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions