fizzy-cli 0.3.0 → 0.5.0

package/cmd/comment_show_test.go

@@ -0,0 +1,104 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"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 TestCommentShowCommandSuccess(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.MethodGet {
+			t.Errorf("expected GET, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		response := api.Comment{
+			ID:        "comment-456",
+			CreatedAt: "2025-01-01T00:00:00Z",
+			UpdatedAt: "2025-01-01T00:00:00Z",
+			Creator:   api.User{ID: "user-1", Name: "John Doe"},
+			Card:      api.CardReference{ID: "card-123", Title: "Test Card"},
+		}
+		response.Body.PlainText = "This is a test comment"
+		response.Body.HTML = "<p>This is a test comment</p>"
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleShowComment(cmd, "123", "comment-456"); err != nil {
+		t.Fatalf("handleShowComment failed: %v", err)
+	}
+}
+
+func TestCommentShowCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleShowComment(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 TestCommentShowCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleShowComment(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 TestCommentShowCommandAPIError(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 := commentShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleShowComment(cmd, "123", "comment-456")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "fetching comment: unexpected status code 404: Comment not found" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/tag.go

@@ -0,0 +1,15 @@
+package cmd
+
+import (
+	"github.com/spf13/cobra"
+)
+
+var tagCmd = &cobra.Command{
+	Use:   "tag",
+	Short: "Manage tags",
+	Long:  `Manage tags in Fizzy`,
+}
+
+func init() {
+	rootCmd.AddCommand(tagCmd)
+}

package/cmd/tag_list.go

@@ -0,0 +1,47 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var tagListCmd = &cobra.Command{
+	Use:   "list",
+	Short: "List all tags",
+	Long:  `Retrieve and display all tags in the account`,
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleListTags(cmd); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleListTags(cmd *cobra.Command) error {
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	tags, err := a.Client.GetTags(context.Background())
+	if err != nil {
+		return fmt.Errorf("fetching tags: %w", err)
+	}
+
+	if len(tags) == 0 {
+		fmt.Println("No tags found")
+		return nil
+	}
+
+	for _, tag := range tags {
+		fmt.Printf("%s\n", tag.Title)
+	}
+
+	return nil
+}
+
+func init() {
+	tagCmd.AddCommand(tagListCmd)
+}

package/cmd/tag_list_test.go

@@ -0,0 +1,109 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"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 TestTagListCommand(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/tags" {
+			t.Errorf("expected /tags, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodGet {
+			t.Errorf("expected GET, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		response := []api.Tag{
+			{
+				ID:        "tag-123",
+				Title:     "bug",
+				CreatedAt: "2025-01-01T00:00:00Z",
+				URL:       "http://fizzy.localhost:3006/897362094/cards?tag_ids[]=tag-123",
+			},
+			{
+				ID:        "tag-456",
+				Title:     "feature",
+				CreatedAt: "2025-01-02T00:00:00Z",
+				URL:       "http://fizzy.localhost:3006/897362094/cards?tag_ids[]=tag-456",
+			},
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := tagListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListTags(cmd); err != nil {
+		t.Fatalf("handleListTags failed: %v", err)
+	}
+}
+
+func TestTagListCommandNoTags(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		w.Write([]byte("[]"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := tagListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListTags(cmd); err != nil {
+		t.Fatalf("handleListTags with no tags failed: %v", err)
+	}
+}
+
+func TestTagListCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusInternalServerError)
+		w.Write([]byte("Internal Server Error"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := tagListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListTags(cmd)
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+}
+
+func TestTagListCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := tagListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListTags(cmd)
+	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)
+	}
+}

package/docs/API.md

@@ -574,6 +574,7 @@ __Response:__
   "description_html": "<div class=\"action-text-content\"><p>Hello, World!</p></div>",
   "image_url": null,
   "tags": ["programming"],
+  "closed": false,
   "golden": false,
   "last_active_at": "2025-12-05T19:38:48.553Z",
   "created_at": "2025-12-05T19:38:48.540Z",
@@ -594,6 +595,15 @@ __Response:__
       "url": "http://fizzy.localhost:3006/897362094/users/03f5v9zjw7pz8717a4no1h8a7"
     }
   },
+  "column": {
+    "id": "03f5v9zkft4hj9qq0lsn9ohcn",
+    "name": "In Progress",
+    "color": {
+      "name": "Lime",
+      "value": "var(--color-card-4)"
+    },
+    "created_at": "2025-12-05T19:36:35.534Z"
+  },
   "creator": {
     "id": "03f5v9zjw7pz8717a4no1h8a7",
     "name": "David Heinemeier Hansson",
@@ -619,6 +629,8 @@ __Response:__
 }
 ```
 
+> **Note:** The `closed` field indicates whether the card is in the "Done" state. The `column` field is only present when the card has been triaged into a column; cards in "Maybe?", "Not Now" or "Done" will not have this field.
+
 ### `POST /:account_slug/boards/:board_id/cards`
 
 Creates a new card in a board.
@@ -683,6 +695,14 @@ __Response:__
 
 Returns `204 No Content` on success.
 
+### `DELETE /:account_slug/cards/:card_number/image`
+
+Removes the header image from a card.
+
+__Response:__
+
+Returns `204 No Content` on success.
+
 ### `POST /:account_slug/cards/:card_number/closure`
 
 Closes a card.
@@ -767,6 +787,22 @@ __Response:__
 
 Returns `204 No Content` on success.
 
+### `POST /:account_slug/cards/:card_number/goldness`
+
+Marks a card as golden.
+
+__Response:__
+
+Returns `204 No Content` on success.
+
+### `DELETE /:account_slug/cards/:card_number/goldness`
+
+Removes golden status from a card.
+
+__Response:__
+
+Returns `204 No Content` on success.
+
 ## Comments
 
 Comments are attached to cards and support rich text.

package/internal/api/client.go

@@ -84,7 +84,10 @@ func (c *Client) decodeResponse(req *http.Request, v any, expectedStatus ...int)
 	defer res.Body.Close()
 
 	if res.StatusCode != expectedCode {
-		body, _ := io.ReadAll(res.Body)
+		body, err := io.ReadAll(res.Body)
+		if err != nil {
+			return 0, fmt.Errorf("unexpected status code %d (failed to read error response: %w)", res.StatusCode, err)
+		}
 		return 0, fmt.Errorf("unexpected status code %d: %s", res.StatusCode, string(body))
 	}
 
@@ -307,6 +310,22 @@ func (c *Client) PostCardsClosure(ctx context.Context, cardNumber int) (bool, er
 	return true, nil
 }
 
+func (c *Client) PostCardNotNow(ctx context.Context, cardNumber int) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/not_now", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodPost, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create post not now request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
 func (c *Client) PostCardTriage(ctx context.Context, cardNumber int, columnID string) (bool, error) {
 	endpointURL := fmt.Sprintf("%s/cards/%d/triage", c.AccountBaseURL, cardNumber)
 
@@ -325,6 +344,86 @@ func (c *Client) PostCardTriage(ctx context.Context, cardNumber int, columnID st
 	return true, nil
 }
 
+func (c *Client) DeleteCardTriage(ctx context.Context, cardNumber int) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/triage", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodDelete, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create delete triage request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
+func (c *Client) PostCardWatch(ctx context.Context, cardNumber int) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/watch", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodPost, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create post watch request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
+func (c *Client) DeleteCardWatch(ctx context.Context, cardNumber int) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/watch", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodDelete, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create delete watch request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
+func (c *Client) PostCardGoldenness(ctx context.Context, cardNumber int) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/goldness", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodPost, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create post goldness request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
+func (c *Client) DeleteCardGoldenness(ctx context.Context, cardNumber int) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/goldness", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodDelete, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create delete goldness request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
 func (c *Client) DeleteCardsClosure(ctx context.Context, cardNumber int) (bool, error) {
 	endpointURL := fmt.Sprintf("%s/cards/%d/closure", c.AccountBaseURL, cardNumber)
 
@@ -359,6 +458,24 @@ func (c *Client) PostCardAssignments(ctx context.Context, cardNumber int, userID
 	return true, nil
 }
 
+func (c *Client) PostCardTagging(ctx context.Context, cardNumber int, tagTitle string) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/taggings", c.AccountBaseURL, cardNumber)
+
+	body := map[string]string{"tag_title": tagTitle}
+
+	req, err := c.newRequest(ctx, http.MethodPost, endpointURL, body)
+	if err != nil {
+		return false, fmt.Errorf("failed to create tagging request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}
+
 func (c *Client) GetMyIdentity(ctx context.Context) (*GetMyIdentityResponse, error) {
 	endpointURL := c.BaseURL + "/my/identity"
 
@@ -458,6 +575,78 @@ func (c *Client) PostBulkNotificationsReading(ctx context.Context) (bool, error)
 	return true, nil
 }
 
+func (c *Client) GetTags(ctx context.Context) ([]Tag, error) {
+	endpointURL := c.AccountBaseURL + "/tags"
+
+	req, err := c.newRequest(ctx, http.MethodGet, endpointURL, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create get tags request: %w", err)
+	}
+
+	var response []Tag
+	_, err = c.decodeResponse(req, &response)
+	if err != nil {
+		return nil, err
+	}
+
+	return response, nil
+}
+
+func (c *Client) GetCardComments(ctx context.Context, cardNumber int) ([]Comment, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/comments", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodGet, endpointURL, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create get card comments request: %w", err)
+	}
+
+	var response []Comment
+	_, err = c.decodeResponse(req, &response)
+	if err != nil {
+		return nil, err
+	}
+
+	return response, nil
+}
+
+func (c *Client) GetCardComment(ctx context.Context, cardNumber int, commentID string) (*Comment, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/comments/%s", c.AccountBaseURL, cardNumber, commentID)
+
+	req, err := c.newRequest(ctx, http.MethodGet, endpointURL, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create get card comment request: %w", err)
+	}
+
+	var response Comment
+	_, err = c.decodeResponse(req, &response)
+	if err != nil {
+		return nil, err
+	}
+
+	return &response, nil
+}
+
+func (c *Client) PostCardComment(ctx context.Context, cardNumber int, body string) (*Comment, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/comments", c.AccountBaseURL, cardNumber)
+
+	payload := map[string]map[string]string{
+		"comment": {"body": body},
+	}
+
+	req, err := c.newRequest(ctx, http.MethodPost, endpointURL, payload)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create post card comment request: %w", err)
+	}
+
+	var response Comment
+	_, err = c.decodeResponse(req, &response, http.StatusCreated)
+	if err != nil {
+		return nil, err
+	}
+
+	return &response, nil
+}
+
 type Board struct {
 	ID        string `json:"id"`
 	Name      string `json:"name"`
@@ -584,6 +773,27 @@ type CardReference struct {
 	URL    string `json:"url"`
 }
 
+type Tag struct {
+	ID        string `json:"id"`
+	Title     string `json:"title"`
+	CreatedAt string `json:"created_at"`
+	URL       string `json:"url"`
+}
+
+type Comment struct {
+	ID        string `json:"id"`
+	CreatedAt string `json:"created_at"`
+	UpdatedAt string `json:"updated_at"`
+	Body      struct {
+		PlainText string `json:"plain_text"`
+		HTML      string `json:"html"`
+	} `json:"body"`
+	Creator      User          `json:"creator"`
+	Card         CardReference `json:"card"`
+	ReactionsURL string        `json:"reactions_url"`
+	URL          string        `json:"url"`
+}
+
 type Color string
 
 // Color constants using centralized definitions

package/internal/ui/comment_list.go

@@ -0,0 +1,25 @@
+package ui
+
+import (
+	"fmt"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+)
+
+func DisplayComments(comments []api.Comment) error {
+	for _, comment := range comments {
+		fmt.Printf("%s - %s (%s)\n", comment.Creator.Name, comment.Body.PlainText, DisplayID(comment.ID))
+	}
+	return nil
+}
+
+func DisplayComment(comment *api.Comment) error {
+	fmt.Printf("Author: %s\n", comment.Creator.Name)
+	fmt.Printf("Created: %s\n", comment.CreatedAt)
+	if comment.UpdatedAt != comment.CreatedAt {
+		fmt.Printf("Updated: %s\n", comment.UpdatedAt)
+	}
+	fmt.Printf("Card: %s\n", comment.Card.Title)
+	fmt.Printf("\n%s\n", comment.Body.PlainText)
+	return nil
+}

package/package.json

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