Skip to content

feat(issue-comment): accept multiple ISSUE_IDs - #208

Merged
ruby-automation merged 2 commits into
mainfrom
EXT-25-issue-comment-multiple-ids
Sep 3, 2026
Merged

feat(issue-comment): accept multiple ISSUE_IDs#208
ruby-automation merged 2 commits into
mainfrom
EXT-25-issue-comment-multiple-ids

Conversation

@ruby-automation

Copy link
Copy Markdown
Contributor

Summary

  • lc issue comment now accepts one or more ISSUE_ID arguments instead of exactly one, matching the pattern used by issue take, issue status, issue update, and issue move
  • When multiple IDs are given, the same comment body is posted to each issue concurrently (capped at 20 in flight via Task.async_stream)
  • Body resolution (-m, --body-file, or interactive prompt) happens once before the fan-out; the interactive prompt uses the first issue's context

Test plan

  • Existing single-ID tests pass unchanged (no call-site changes required — CRY-1 is now captured via unknown instead of a named positional arg, invisibly to tests)
  • New test: multiple ISSUE_IDs each receive the comment — verifies two commentCreate mutations fire for two IDs
  • New test: no ISSUE_IDs is a smells_bad error — verifies exit 22 with "No issue IDs provided!" when no IDs given
  • mix credo passes with no issues
  • mix format --check-formatted passes
  • Full test suite: 378/380 pass (2 pre-existing failures in LinearCli.GitTest, unrelated)

Closes EXT-25

🤖 Generated with Claude Code

ruby-automation and others added 2 commits September 3, 2026 12:03
Switch `lc issue comment` from a single required positional arg to
`allow_unknown_args: true`, matching the pattern used by `issue take`,
`issue status`, `issue update`, and `issue move`.

When multiple IDs are given the same comment body is posted to each
issue concurrently via Task.async_stream (capped at 20 in flight,
same as other bulk-mutation commands). Body resolution (prompt,
-m text, or --body-file) happens once before the fan-out, using the
first issue's context for the interactive prompt.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ruby-automation
ruby-automation merged commit 1ae4fba into main Sep 3, 2026
3 checks passed
@ruby-automation
ruby-automation deleted the EXT-25-issue-comment-multiple-ids branch September 3, 2026 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants