fizzy-cli 0.6.0 → 0.6.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # 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

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/reaction_create.go

@@ -32,12 +32,12 @@ func handleCreateReaction(cmd *cobra.Command, cardNumber, commentID, emoji strin
 		return fmt.Errorf("API client not available")
 	}
 
-	reaction, err := a.Client.PostCommentReaction(context.Background(), cardNum, commentID, emoji)
+	_, 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 (id: %s)\n", reaction.Content, reaction.ID)
+	fmt.Printf("✓ Reaction %s created successfully\n", emoji)
 	return nil
 }
 

package/cmd/step_create.go

@@ -35,12 +35,12 @@ func handleCreateStep(cmd *cobra.Command, cardNumber string) error {
 	content, _ := cmd.Flags().GetString("content")
 	completed, _ := cmd.Flags().GetBool("completed")
 
-	step, err := a.Client.PostCardStep(context.Background(), cardNum, content, completed)
+	_, err = a.Client.PostCardStep(context.Background(), cardNum, content, completed)
 	if err != nil {
 		return fmt.Errorf("creating step: %w", err)
 	}
 
-	fmt.Printf("✓ Step created successfully (id: %s)\n", step.ID)
+	fmt.Printf("✓ Step created successfully\n")
 	return nil
 }
 

package/internal/api/comments.go

@@ -3,6 +3,7 @@ package api
 import (
 	"context"
 	"fmt"
+	"io"
 	"net/http"
 )
 
@@ -52,13 +53,21 @@ func (c *Client) PostCardComment(ctx context.Context, cardNumber int, body strin
 		return nil, fmt.Errorf("failed to create post card comment request: %w", err)
 	}
 
-	var response Comment
-	_, err = c.decodeResponse(req, &response, http.StatusCreated)
+	res, err := c.HTTPClient.Do(req)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("failed to make request: %w", err)
 	}
+	defer res.Body.Close()
 
-	return &response, nil
+	if res.StatusCode != http.StatusCreated {
+		body, _ := io.ReadAll(res.Body)
+		return nil, fmt.Errorf("unexpected status code %d: %s", res.StatusCode, string(body))
+	}
+
+	// API returns 201 Created with Location header, not a response body
+	// Return a minimal Comment object with empty fields
+	// The comment ID would be in the Location header but we'll use it to fetch full details
+	return &Comment{}, nil
 }
 
 func (c *Client) PutCardComment(ctx context.Context, cardNumber int, commentID string, body string) (*Comment, error) {

package/internal/api/reactions.go

@@ -3,6 +3,7 @@ package api
 import (
 	"context"
 	"fmt"
+	"io"
 	"net/http"
 )
 
@@ -35,13 +36,20 @@ func (c *Client) PostCommentReaction(ctx context.Context, cardNumber int, commen
 		return nil, fmt.Errorf("failed to create post comment reaction request: %w", err)
 	}
 
-	var response Reaction
-	_, err = c.decodeResponse(req, &response, http.StatusCreated)
+	res, err := c.HTTPClient.Do(req)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("failed to make request: %w", err)
+	}
+	defer res.Body.Close()
+
+	if res.StatusCode != http.StatusCreated {
+		body, _ := io.ReadAll(res.Body)
+		return nil, fmt.Errorf("unexpected status code %d: %s", res.StatusCode, string(body))
 	}
 
-	return &response, nil
+	// API returns 201 Created with Location header, not a response body
+	// Return a Reaction object with the content field set for reference
+	return &Reaction{Content: content}, nil
 }
 
 func (c *Client) DeleteCommentReaction(ctx context.Context, cardNumber int, commentID string, reactionID string) (bool, error) {

package/internal/api/steps.go

@@ -3,6 +3,7 @@ package api
 import (
 	"context"
 	"fmt"
+	"io"
 	"net/http"
 )
 
@@ -38,13 +39,20 @@ func (c *Client) PostCardStep(ctx context.Context, cardNumber int, content strin
 		return nil, fmt.Errorf("failed to create post card step request: %w", err)
 	}
 
-	var response Step
-	_, err = c.decodeResponse(req, &response, http.StatusCreated)
+	res, err := c.HTTPClient.Do(req)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("failed to make request: %w", err)
 	}
+	defer res.Body.Close()
 
-	return &response, nil
+	if res.StatusCode != http.StatusCreated {
+		body, _ := io.ReadAll(res.Body)
+		return nil, fmt.Errorf("unexpected status code %d: %s", res.StatusCode, string(body))
+	}
+
+	// API returns 201 Created with Location header, not a response body
+	// Return a Step object with the content field set for reference
+	return &Step{Content: content, Completed: completed}, nil
 }
 
 func (c *Client) PutCardStep(ctx context.Context, cardNumber int, stepID string, content *string, completed *bool) (*Step, error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fizzy-cli",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "CLI for https://fizzy.do",
   "main": "bin/fizzy",
   "type": "module",