Skip to content

feat(issue-update): add --body-file to lc issue update - #227

Merged
ruby-automation merged 2 commits into
mainfrom
EXT-36-body-file-issue-update
Sep 5, 2026
Merged

feat(issue-update): add --body-file to lc issue update#227
ruby-automation merged 2 commits into
mainfrom
EXT-36-body-file-issue-update

Conversation

@ruby-automation

Copy link
Copy Markdown
Contributor

Summary

  • Adds --body-file PATH option to lc issue update, sourcing the issue description from a file (- for stdin) instead of requiring it as a --description shell argument.
  • Mutually exclusive with --description; errors with exit 22 if both are given.
  • --comment and --reason are unaffected.
  • Wires the existing validate_body_file_exclusion/3 and resolve_body_from_file/2 helpers into issue_update/1, consistent with issue create and issue comment.

Closes https://linear.app/the-rubyists/issue/EXT-36

Test plan

  • --body-file <path> reads description from file and posts it via issueUpdate mutation
  • --body-file - reads description from stdin
  • --description and --body-file together exits 22 with "give --description or --body-file, not both"
  • An unreadable --body-file path surfaces an error without making a GraphQL call
  • All 409 existing tests still pass
  • mix credo --strict reports no issues

ruby-automation and others added 2 commits September 4, 2026 23:19
Read the description from a file (- for stdin) instead of passing
it as a --description shell argument. Mutually exclusive with
--description; --comment and --reason are unaffected.

Wires the existing validate_body_file_exclusion/3 and
resolve_body_from_file/2 helpers into issue_update/1, matching how
issue create and issue comment already use them.

Adds CLI tests: file read, stdin read, mutual-exclusion error, and
unreadable-file error.

Closes EXT-36
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant