highperfocused/multica
1
0
Fork 0
mirror of https://github.com/multica-ai/multica.git synced 2026-07-05 21:39:54 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix(cli): address review feedback on --content-stdin flag

Browse Source 
- Make --content and --content-stdin mutually exclusive with explicit error
- Use TrimSuffix instead of TrimRight to only strip the trailing newline
- Return "stdin content is empty" instead of misleading "required" error

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Zohar Babin
2026-04-13 19:14:55 -04:00
parent 77dbcaefad
commit d37595b85e
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
server/cmd/multica/cmd_issue.go
View File

@@ -639,13 +639,23 @@ func runIssueCommentList(cmd *cobra.Command, args []string) error {
func runIssueCommentAdd(cmd *cobra.Command, args []string) error {
content, _ := cmd.Flags().GetString("content")
if useStdin, _ := cmd.Flags().GetBool("content-stdin"); useStdin {
useStdin, _ := cmd.Flags().GetBool("content-stdin")
if content != "" && useStdin {
return fmt.Errorf("--content and --content-stdin are mutually exclusive")
}
if useStdin {
data, err := io.ReadAll(os.Stdin)
if err != nil {
return fmt.Errorf("read stdin: %w", err)
}
content = strings.TrimRight(string(data), "\n")
content = strings.TrimSuffix(string(data), "\n")
if content == "" {
return fmt.Errorf("stdin content is empty")
}
}
if content == "" {
return fmt.Errorf("--content or --content-stdin is required")
}