fizzy-cli 0.5.0 → 0.6.1

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # Changelog
 
+## 0.6.1 - 2026-01-26
+
+### Fixes
+
+- Fixed API client to not parse empty responses from POST endpoints, this was
+  causing the create sub-command of comment, reaction and step to show an error
+  message, even though the resource was actually created on the API.
+
+## 0.6.0 - 2026-01-25
+
+### Features
+
+#### Reaction Management
+
+- `fizzy reaction list <card_number> <comment_id>` - List all reactions on a comment
+- `fizzy reaction add <card_number> <comment_id> <emoji>` - Add a reaction to a comment
+- `fizzy reaction remove <card_number> <comment_id> <reaction_id>` - Remove a reaction from a comment
+
+#### Step Management
+
+- `fizzy step create <card_number> --content <text> [--completed]` - Create a new step on a card
+- `fizzy step update <card_number> <step_id> [--content <text>] [--completed]` - Update a step
+- `fizzy step delete <card_number> <step_id>` - Delete a step from a card
+
+#### Comment Management
+
+- `fizzy comment delete <card_number> <comment_id>` - Delete a comment from a card
+- `fizzy comment update <card_number> <comment_id> --body <text>` - Update a comment on a card
+
+### Improvements
+
+- Refactored API client logic into separate files for better organization
+- Enhanced card update command to use `Flags().Changed()` for better flag handling
+- Comprehensive test coverage for all new commands (168 tests total)
+
 ## 0.5.0 - 2026-01-25
 
 ### Features

package/cmd/card_update.go

@@ -33,23 +33,33 @@ func handleUpdateCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("API client not available")
 	}
 
-	// Read flag values directly from command
-	title, _ := cmd.Flags().GetString("title")
-	description, _ := cmd.Flags().GetString("description")
-	status, _ := cmd.Flags().GetString("status")
-	tagIDs, _ := cmd.Flags().GetStringSlice("tag-id")
-	lastActiveAt, _ := cmd.Flags().GetString("last-active-at")
+	// Build payload only with flags that were explicitly set
+	var payload api.UpdateCardPayload
+	hasChanges := false
 
-	if title == "" && description == "" && status == "" && len(tagIDs) == 0 && lastActiveAt == "" {
-		return fmt.Errorf("must provide at least one flag to update (--title, --description, --status, --tag-id, or --last-active-at)")
+	if cmd.Flags().Changed("title") {
+		payload.Title, _ = cmd.Flags().GetString("title")
+		hasChanges = true
+	}
+	if cmd.Flags().Changed("description") {
+		payload.Description, _ = cmd.Flags().GetString("description")
+		hasChanges = true
+	}
+	if cmd.Flags().Changed("status") {
+		payload.Status, _ = cmd.Flags().GetString("status")
+		hasChanges = true
+	}
+	if cmd.Flags().Changed("tag-id") {
+		payload.TagIDS, _ = cmd.Flags().GetStringSlice("tag-id")
+		hasChanges = true
+	}
+	if cmd.Flags().Changed("last-active-at") {
+		payload.LastActiveAt, _ = cmd.Flags().GetString("last-active-at")
+		hasChanges = true
 	}
 
-	payload := api.UpdateCardPayload{
-		Title:        title,
-		Description:  description,
-		Status:       status,
-		TagIDS:       tagIDs,
-		LastActiveAt: lastActiveAt,
+	if !hasChanges {
+		return fmt.Errorf("must provide at least one flag to update (--title, --description, --status, --tag-id, or --last-active-at)")
 	}
 
 	card, err := a.Client.PutCard(context.Background(), cardNum, payload)

package/cmd/comment_create.go

@@ -34,12 +34,12 @@ func handleCreateComment(cmd *cobra.Command, cardNumber string) error {
 
 	body, _ := cmd.Flags().GetString("body")
 
-	comment, err := a.Client.PostCardComment(context.Background(), cardNum, body)
+	_, err = a.Client.PostCardComment(context.Background(), cardNum, body)
 	if err != nil {
 		return fmt.Errorf("creating comment: %w", err)
 	}
 
-	fmt.Printf("✓ Comment created successfully (id: %s)\n", comment.ID)
+	fmt.Printf("✓ Comment created successfully\n")
 	return nil
 }
 

package/cmd/comment_delete.go

@@ -0,0 +1,46 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var commentDeleteCmd = &cobra.Command{
+	Use:   "delete <card_number> <comment_id>",
+	Short: "Delete a comment",
+	Long:  `Delete a comment from a card`,
+	Args:  cobra.ExactArgs(2),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleDeleteComment(cmd, args[0], args[1]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleDeleteComment(cmd *cobra.Command, cardNumber, commentID string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	_, err = a.Client.DeleteCardComment(context.Background(), cardNum, commentID)
+	if err != nil {
+		return fmt.Errorf("deleting comment: %w", err)
+	}
+
+	fmt.Printf("✓ Comment deleted successfully\n")
+	return nil
+}
+
+func init() {
+	commentCmd.AddCommand(commentDeleteCmd)
+}

package/cmd/comment_delete_test.go

@@ -0,0 +1,92 @@
+package cmd
+
+import (
+	"context"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestCommentDeleteCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/comments/comment-456" {
+			t.Errorf("expected /cards/123/comments/comment-456, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodDelete {
+			t.Errorf("expected DELETE, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		w.WriteHeader(http.StatusNoContent)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleDeleteComment(cmd, "123", "comment-456"); err != nil {
+		t.Fatalf("handleDeleteComment failed: %v", err)
+	}
+}
+
+func TestCommentDeleteCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteComment(cmd, "not-a-number", "comment-456")
+	if err == nil {
+		t.Errorf("expected error for invalid card number")
+	}
+	if err.Error() != "invalid card number: strconv.Atoi: parsing \"not-a-number\": invalid syntax" {
+		t.Errorf("expected invalid card number error, got %v", err)
+	}
+}
+
+func TestCommentDeleteCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteComment(cmd, "123", "comment-456")
+	if err == nil {
+		t.Errorf("expected error when client not available")
+	}
+	if err.Error() != "API client not available" {
+		t.Errorf("expected 'client not available' error, got %v", err)
+	}
+}
+
+func TestCommentDeleteCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Comment not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteComment(cmd, "123", "comment-456")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "deleting comment: unexpected status code 404: Comment not found" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/comment_update.go

@@ -0,0 +1,51 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var commentUpdateCmd = &cobra.Command{
+	Use:   "update <card_number> <comment_id>",
+	Short: "Update an existing comment",
+	Long:  `Update the body of an existing comment on a card`,
+	Args:  cobra.ExactArgs(2),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleUpdateComment(cmd, args[0], args[1]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleUpdateComment(cmd *cobra.Command, cardNumber, commentID string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	body, _ := cmd.Flags().GetString("body")
+
+	comment, err := a.Client.PutCardComment(context.Background(), cardNum, commentID, body)
+	if err != nil {
+		return fmt.Errorf("updating comment: %w", err)
+	}
+
+	fmt.Printf("✓ Comment updated successfully (id: %s)\n", comment.ID)
+	return nil
+}
+
+func init() {
+	commentUpdateCmd.Flags().StringP("body", "b", "", "New comment body (required)")
+	commentUpdateCmd.MarkFlagRequired("body")
+
+	commentCmd.AddCommand(commentUpdateCmd)
+}

package/cmd/comment_update_test.go

@@ -0,0 +1,130 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"io"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestCommentUpdateCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/comments/comment-456" {
+			t.Errorf("expected /cards/123/comments/comment-456, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodPut {
+			t.Errorf("expected PUT, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		if r.Header.Get("Content-Type") != "application/json" {
+			t.Errorf("expected Content-Type: application/json, got %s", r.Header.Get("Content-Type"))
+		}
+
+		body, _ := io.ReadAll(r.Body)
+		var payload map[string]map[string]string
+		if err := json.Unmarshal(body, &payload); err != nil {
+			t.Fatalf("failed to unmarshal request body: %v", err)
+		}
+
+		commentPayload := payload["comment"]
+		if commentPayload["body"] != "Updated comment text" {
+			t.Errorf("expected body 'Updated comment text', got %s", commentPayload["body"])
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		response := api.Comment{
+			ID:        "comment-456",
+			CreatedAt: "2025-01-01T00:00:00Z",
+			UpdatedAt: "2025-01-02T00:00:00Z",
+			Creator:   api.User{ID: "user-1", Name: "John Doe"},
+		}
+		response.Body.PlainText = "Updated comment text"
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Updated comment text",
+	})
+
+	if err := handleUpdateComment(cmd, "123", "comment-456"); err != nil {
+		t.Fatalf("handleUpdateComment failed: %v", err)
+	}
+}
+
+func TestCommentUpdateCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Updated text",
+	})
+
+	err := handleUpdateComment(cmd, "not-a-number", "comment-456")
+	if err == nil {
+		t.Errorf("expected error for invalid card number")
+	}
+	if err.Error() != "invalid card number: strconv.Atoi: parsing \"not-a-number\": invalid syntax" {
+		t.Errorf("expected invalid card number error, got %v", err)
+	}
+}
+
+func TestCommentUpdateCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Updated text",
+	})
+
+	err := handleUpdateComment(cmd, "123", "comment-456")
+	if err == nil {
+		t.Errorf("expected error when client not available")
+	}
+	if err.Error() != "API client not available" {
+		t.Errorf("expected 'client not available' error, got %v", err)
+	}
+}
+
+func TestCommentUpdateCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Comment not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Updated text",
+	})
+
+	err := handleUpdateComment(cmd, "123", "comment-456")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "updating comment: unexpected status code 404: Comment not found" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/reaction.go

@@ -0,0 +1,13 @@
+package cmd
+
+import "github.com/spf13/cobra"
+
+var reactionCmd = &cobra.Command{
+	Use:   "reaction",
+	Short: "Manage comment reactions",
+	Long:  `Manage reactions on comments in Fizzy`,
+}
+
+func init() {
+	rootCmd.AddCommand(reactionCmd)
+}

package/cmd/reaction_create.go

@@ -0,0 +1,46 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var reactionCreateCmd = &cobra.Command{
+	Use:   "create <card_number> <comment_id> <emoji>",
+	Short: "Create a reaction on a comment",
+	Long:  `Create an emoji reaction on a comment`,
+	Args:  cobra.ExactArgs(3),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleCreateReaction(cmd, args[0], args[1], args[2]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleCreateReaction(cmd *cobra.Command, cardNumber, commentID, emoji string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	_, err = a.Client.PostCommentReaction(context.Background(), cardNum, commentID, emoji)
+	if err != nil {
+		return fmt.Errorf("creating reaction: %w", err)
+	}
+
+	fmt.Printf("✓ Reaction %s created successfully\n", emoji)
+	return nil
+}
+
+func init() {
+	reactionCmd.AddCommand(reactionCreateCmd)
+}

package/cmd/reaction_create_test.go

@@ -0,0 +1,113 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"io"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestReactionCreateCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/comments/comment-456/reactions" {
+			t.Errorf("expected /cards/123/comments/comment-456/reactions, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodPost {
+			t.Errorf("expected POST, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		body, _ := io.ReadAll(r.Body)
+		var payload map[string]map[string]string
+		if err := json.Unmarshal(body, &payload); err != nil {
+			t.Fatalf("failed to unmarshal request body: %v", err)
+		}
+
+		reactionPayload := payload["reaction"]
+		if reactionPayload["content"] != "👍" {
+			t.Errorf("expected content '👍', got %s", reactionPayload["content"])
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusCreated)
+		response := api.Reaction{
+			ID:      "reaction-789",
+			Content: "👍",
+			Reacter: api.User{ID: "user-1", Name: "John Doe"},
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := reactionCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleCreateReaction(cmd, "123", "comment-456", "👍"); err != nil {
+		t.Fatalf("handleCreateReaction failed: %v", err)
+	}
+}
+
+func TestReactionCreateCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := reactionCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleCreateReaction(cmd, "not-a-number", "comment-456", "👍")
+	if err == nil {
+		t.Errorf("expected error for invalid card number")
+	}
+	if err.Error() != "invalid card number: strconv.Atoi: parsing \"not-a-number\": invalid syntax" {
+		t.Errorf("expected invalid card number error, got %v", err)
+	}
+}
+
+func TestReactionCreateCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := reactionCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleCreateReaction(cmd, "123", "comment-456", "👍")
+	if err == nil {
+		t.Errorf("expected error when client not available")
+	}
+	if err.Error() != "API client not available" {
+		t.Errorf("expected 'client not available' error, got %v", err)
+	}
+}
+
+func TestReactionCreateCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Comment not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := reactionCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleCreateReaction(cmd, "123", "comment-456", "👍")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "creating reaction: unexpected status code 404: Comment not found" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/reaction_delete.go

@@ -0,0 +1,46 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var reactionDeleteCmd = &cobra.Command{
+	Use:   "delete <card_number> <comment_id> <reaction_id>",
+	Short: "Delete a reaction from a comment",
+	Long:  `Delete an emoji reaction from a comment`,
+	Args:  cobra.ExactArgs(3),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleDeleteReaction(cmd, args[0], args[1], args[2]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleDeleteReaction(cmd *cobra.Command, cardNumber, commentID, reactionID string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	_, err = a.Client.DeleteCommentReaction(context.Background(), cardNum, commentID, reactionID)
+	if err != nil {
+		return fmt.Errorf("deleting reaction: %w", err)
+	}
+
+	fmt.Printf("✓ Reaction deleted successfully\n")
+	return nil
+}
+
+func init() {
+	reactionCmd.AddCommand(reactionDeleteCmd)
+}