Skip to content

fix(comment): move markdown conversion to command layer, add --raw flag#997

Closed
hackerh3 wants to merge 27 commits into
ankitpokhrel:mainfrom
hackerh3:hhaecker/fix-comment-markup
Closed

fix(comment): move markdown conversion to command layer, add --raw flag#997
hackerh3 wants to merge 27 commits into
ankitpokhrel:mainfrom
hackerh3:hhaecker/fix-comment-markup

Conversation

@hackerh3
Copy link
Copy Markdown

Summary

  • Moves md.ToJiraMD() conversion from the Jira client layer (pkg/jira/issue.go) up to the command layer (add/edit commands), fixing the bug where comments created via API were double-converted
  • Adds --raw flag to jira issue comment add and jira issue comment edit to skip markdown-to-Jira conversion entirely (useful for pre-formatted or programmatic input)
  • Adds omitempty to the Properties field in CommentBody so empty properties are not sent to the API
  • Only sets properties when internal=true, preventing spurious empty objects in API requests

Testing

  • Added unit tests for both add and edit commands verifying --raw flag behavior
  • Existing pkg/jira/issue_test.go updated to reflect moved conversion logic

Closes #13

hackerh3 and others added 27 commits March 26, 2026 15:02
Adds --jira-template and --template-var flags to create issues from
Deviniti Issue Templates. Templates are fetched via the REST API,
variables are substituted, and fields (summary, description, labels)
are applied to the new issue.

Also includes minor cleanup: interface{} → any, Fprintf for error
formatting, maps.Copy for custom fields, and .gitignore updates.
feat(jira): add move-project command for Jira Server
feat: add move-project command and comment edit subcommand
ci: add semantic-release with conventionalcommits and goreleaser pipeline
fix(ci): use goreleaser append mode for binary uploads
## [1.8.1](1.8.0...1.8.1) (2026-05-12)

### Bug Fixes

* **ci:** flatten archive layout and build all platforms ([5cf0eca](5cf0eca))
- Move md.ToJiraMD() from client to add/edit commands
- Add --raw flag to skip markdown conversion
- Add omitempty to comment Properties field
- Only set properties when internal=true

Closes #13
@hackerh3 hackerh3 closed this May 13, 2026
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