fizzy-cli 0.7.0 → 0.8.0

package/cmd/user_show_test.go

@@ -0,0 +1,110 @@
+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 TestUserShowCommand(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/users/user-123" {
+			t.Errorf("expected /users/user-123, 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.User{
+			ID:        "user-123",
+			Name:      "John Doe",
+			Email:     "john@example.com",
+			Role:      "admin",
+			Active:    true,
+			CreatedAt: "2025-01-01T00:00:00Z",
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := userShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleShowUser(cmd, "user-123"); err != nil {
+		t.Fatalf("handleShowUser failed: %v", err)
+	}
+}
+
+func TestUserShowCommandNotFound(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("User not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := userShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleShowUser(cmd, "nonexistent-user")
+	if err == nil {
+		t.Errorf("expected error for user not found")
+	}
+	if err.Error() != "fetching user: unexpected status code 404: User not found" {
+		t.Errorf("expected user not found error, got %v", err)
+	}
+}
+
+func TestUserShowCommandAPIError(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 := userShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleShowUser(cmd, "user-123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "fetching user: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}
+
+func TestUserShowCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := userShowCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleShowUser(cmd, "user-123")
+	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/cmd/user_update.go

@@ -0,0 +1,71 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var userUpdateCmd = &cobra.Command{
+	Use:   "update <user_id>",
+	Short: "Update a user",
+	Long: `Update user settings such as name and avatar.
+
+Avatar must be provided as a URL (e.g., https://example.com/avatar.jpg).
+
+Example:
+  fizzy user update user-123 --name "John Doe"
+  fizzy user update user-123 --avatar https://example.com/avatar.jpg`,
+	Args: cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleUpdateUser(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleUpdateUser(cmd *cobra.Command, userID string) error {
+	if !cmd.Flags().Changed("name") && !cmd.Flags().Changed("avatar") {
+		return fmt.Errorf("at least one flag must be provided (--name or --avatar)")
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	payload := api.UpdateUserPayload{}
+
+	if cmd.Flags().Changed("name") {
+		name, err := cmd.Flags().GetString("name")
+		if err != nil {
+			return fmt.Errorf("invalid name flag: %w", err)
+		}
+		payload.Name = name
+	}
+	if cmd.Flags().Changed("avatar") {
+		avatar, err := cmd.Flags().GetString("avatar")
+		if err != nil {
+			return fmt.Errorf("invalid avatar flag: %w", err)
+		}
+		payload.Avatar = avatar
+	}
+
+	err := a.Client.PutUser(context.Background(), userID, payload)
+	if err != nil {
+		return fmt.Errorf("updating user: %w", err)
+	}
+
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ User '%s' updated successfully\n", userID)
+	return nil
+}
+
+func init() {
+	userUpdateCmd.Flags().StringP("name", "n", "", "User name")
+	userUpdateCmd.Flags().String("avatar", "", "Avatar URL (e.g., https://example.com/avatar.jpg)")
+
+	userCmd.AddCommand(userUpdateCmd)
+}

package/cmd/user_update_test.go

@@ -0,0 +1,177 @@
+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"
+	"github.com/spf13/cobra"
+)
+
+func TestUserUpdateCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/users/user-123" {
+			t.Errorf("expected /users/user-123, 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)
+		}
+
+		body, _ := io.ReadAll(r.Body)
+		var payload map[string]api.UpdateUserPayload
+		if err := json.Unmarshal(body, &payload); err != nil {
+			t.Fatalf("failed to unmarshal request body: %v", err)
+		}
+
+		userPayload := payload["user"]
+		if userPayload.Name != "Updated Name" {
+			t.Errorf("expected name 'Updated Name', got %s", userPayload.Name)
+		}
+
+		w.WriteHeader(http.StatusNoContent)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := userUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{"--name", "Updated Name"})
+
+	if err := handleUpdateUser(cmd, "user-123"); err != nil {
+		t.Fatalf("handleUpdateUser failed: %v", err)
+	}
+}
+
+func TestUserUpdateCommandWithAllFlags(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.Method != http.MethodPut {
+			t.Errorf("expected PUT, got %s", r.Method)
+		}
+
+		body, _ := io.ReadAll(r.Body)
+		var payload map[string]api.UpdateUserPayload
+		json.Unmarshal(body, &payload)
+
+		userPayload := payload["user"]
+		if userPayload.Name != "Updated Name" {
+			t.Errorf("expected name 'Updated Name', got %s", userPayload.Name)
+		}
+		if userPayload.Avatar != "https://example.com/avatar.png" {
+			t.Errorf("expected avatar 'https://example.com/avatar.png', got %s", userPayload.Avatar)
+		}
+
+		w.WriteHeader(http.StatusNoContent)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := userUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--name", "Updated Name",
+		"--avatar", "https://example.com/avatar.png",
+	})
+
+	if err := handleUpdateUser(cmd, "user-123"); err != nil {
+		t.Fatalf("handleUpdateUser failed: %v", err)
+	}
+}
+
+func TestUserUpdateCommandNoFlags(t *testing.T) {
+	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := &cobra.Command{
+		Use:  "update <user_id>",
+		Args: cobra.ExactArgs(1),
+	}
+	cmd.Flags().String("name", "", "User name")
+	cmd.Flags().String("avatar", "", "Avatar URL or file path")
+
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleUpdateUser(cmd, "user-123")
+	if err == nil {
+		t.Errorf("expected error when no flags provided")
+	}
+	if err.Error() != "at least one flag must be provided (--name or --avatar)" {
+		t.Errorf("expected flag requirement error, got %v", err)
+	}
+}
+
+func TestUserUpdateCommandNotFound(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("User not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := userUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{"--name", "Updated Name"})
+
+	err := handleUpdateUser(cmd, "nonexistent-user")
+	if err == nil {
+		t.Errorf("expected error for user not found")
+	}
+	if err.Error() != "updating user: unexpected status code 404: User not found" {
+		t.Errorf("expected user not found error, got %v", err)
+	}
+}
+
+func TestUserUpdateCommandAPIError(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 := userUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{"--name", "Updated Name"})
+
+	err := handleUpdateUser(cmd, "user-123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "updating user: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}
+
+func TestUserUpdateCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := userUpdateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{"--name", "Updated Name"})
+
+	err := handleUpdateUser(cmd, "user-123")
+	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

@@ -553,7 +553,8 @@ __Response:__
       "created_at": "2025-12-05T19:36:35.401Z",
       "url": "http://fizzy.localhost:3006/897362094/users/03f5v9zjw7pz8717a4no1h8a7"
     },
-    "comments_url": "http://fizzy.localhost:3006/897362094/cards/4/comments"
+    "comments_url": "http://fizzy.localhost:3006/897362094/cards/4/comments",
+    "reactions_url": "http://fizzy.localhost:3006/897362094/cards/4/reactions"
   },
 ]
 ```
@@ -614,6 +615,7 @@ __Response:__
     "url": "http://fizzy.localhost:3006/897362094/users/03f5v9zjw7pz8717a4no1h8a7"
   },
   "comments_url": "http://fizzy.localhost:3006/897362094/cards/4/comments",
+  "reactions_url": "http://fizzy.localhost:3006/897362094/cards/4/reactions",
   "steps": [
     {
       "id": "03f8huu0sog76g3s975963b5e",
@@ -928,7 +930,66 @@ __Response:__
 
 Returns `204 No Content` on success.
 
-## Reactions
+## Card Reactions (Boosts)
+
+Card reactions (also called "boosts") let users add short responses directly to cards. These are limited to 16 characters.
+
+### `GET /:account_slug/cards/:card_number/reactions`
+
+Returns a list of reactions on a card.
+
+__Response:__
+
+```json
+[
+  {
+    "id": "03f5v9zo9qlcwwpyc0ascnikz",
+    "content": "👍",
+    "reacter": {
+      "id": "03f5v9zjw7pz8717a4no1h8a7",
+      "name": "David Heinemeier Hansson",
+      "role": "owner",
+      "active": true,
+      "email_address": "david@example.com",
+      "created_at": "2025-12-05T19:36:35.401Z",
+      "url": "http://fizzy.localhost:3006/897362094/users/03f5v9zjw7pz8717a4no1h8a7"
+    },
+    "url": "http://fizzy.localhost:3006/897362094/cards/3/reactions/03f5v9zo9qlcwwpyc0ascnikz"
+  }
+]
+```
+
+### `POST /:account_slug/cards/:card_number/reactions`
+
+Adds a reaction (boost) to a card.
+
+| Parameter | Type | Required | Description |
+|-----------|------|----------|-------------|
+| `content` | string | Yes | The reaction text (max 16 characters) |
+
+__Request:__
+
+```json
+{
+  "reaction": {
+    "content": "Great 👍"
+  }
+}
+```
+
+__Response:__
+
+Returns `201 Created` on success.
+
+### `DELETE /:account_slug/cards/:card_number/reactions/:reaction_id`
+
+Removes your reaction from a card. Only the reaction creator can remove their own reactions.
+
+__Response:__
+
+Returns `204 No Content` on success.
+
+## Comment Reactions
 
 Reactions are short (16-character max) responses to comments.
 

package/internal/api/cards.go

@@ -14,14 +14,48 @@ func (c *Client) GetCards(ctx context.Context, filters CardFilters) ([]Card, err
 		return nil, fmt.Errorf("failed to create get cards request: %w", err)
 	}
 
-	if len(filters.BoardIDs) > 0 {
-		q := req.URL.Query()
-		for _, boardID := range filters.BoardIDs {
-			q.Add("board_ids[]", boardID)
-		}
-		req.URL.RawQuery = q.Encode()
+	q := req.URL.Query()
+
+	for _, boardID := range filters.BoardIDs {
+		q.Add("board_ids[]", boardID)
+	}
+	for _, tagID := range filters.TagIDs {
+		q.Add("tag_ids[]", tagID)
+	}
+	for _, assigneeID := range filters.AssigneeIDs {
+		q.Add("assignee_ids[]", assigneeID)
+	}
+	for _, creatorID := range filters.CreatorIDs {
+		q.Add("creator_ids[]", creatorID)
+	}
+	for _, closerID := range filters.CloserIDs {
+		q.Add("closer_ids[]", closerID)
+	}
+	for _, cardID := range filters.CardIDs {
+		q.Add("card_ids[]", cardID)
+	}
+	for _, term := range filters.Terms {
+		q.Add("terms[]", term)
 	}
 
+	if filters.IndexedBy != "" {
+		q.Set("indexed_by", filters.IndexedBy)
+	}
+	if filters.SortedBy != "" {
+		q.Set("sorted_by", filters.SortedBy)
+	}
+	if filters.AssignmentStatus != "" {
+		q.Set("assignment_status", filters.AssignmentStatus)
+	}
+	if filters.CreationStatus != "" {
+		q.Set("creation", filters.CreationStatus)
+	}
+	if filters.ClosureStatus != "" {
+		q.Set("closure", filters.ClosureStatus)
+	}
+
+	req.URL.RawQuery = q.Encode()
+
 	var response []Card
 	_, err = c.decodeResponse(req, &response)
 	if err != nil {

package/internal/api/reactions.go

@@ -67,3 +67,64 @@ func (c *Client) DeleteCommentReaction(ctx context.Context, cardNumber int, comm
 
 	return true, nil
 }
+
+func (c *Client) GetCardReactions(ctx context.Context, cardNumber int) ([]Reaction, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/reactions", c.AccountBaseURL, cardNumber)
+
+	req, err := c.newRequest(ctx, http.MethodGet, endpointURL, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create get card reactions request: %w", err)
+	}
+
+	var response []Reaction
+	_, err = c.decodeResponse(req, &response)
+	if err != nil {
+		return nil, err
+	}
+
+	return response, nil
+}
+
+func (c *Client) PostCardReaction(ctx context.Context, cardNumber int, content string) (*Reaction, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/reactions", c.AccountBaseURL, cardNumber)
+
+	payload := map[string]map[string]string{
+		"reaction": {"content": content},
+	}
+
+	req, err := c.newRequest(ctx, http.MethodPost, endpointURL, payload)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create post card reaction request: %w", err)
+	}
+
+	res, err := c.HTTPClient.Do(req)
+	if err != nil {
+		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))
+	}
+
+	// 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) DeleteCardReaction(ctx context.Context, cardNumber int, reactionID string) (bool, error) {
+	endpointURL := fmt.Sprintf("%s/cards/%d/reactions/%s", c.AccountBaseURL, cardNumber, reactionID)
+
+	req, err := c.newRequest(ctx, http.MethodDelete, endpointURL, nil)
+	if err != nil {
+		return false, fmt.Errorf("failed to create delete card reaction request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return false, err
+	}
+
+	return true, nil
+}

package/internal/api/types.go

@@ -121,6 +121,11 @@ type User struct {
 	URL       string `json:"url"`
 }
 
+type UpdateUserPayload struct {
+	Name   string `json:"name,omitempty"`
+	Avatar string `json:"avatar,omitempty"`
+}
+
 type Notification struct {
 	ID        string        `json:"id"`
 	Read      bool          `json:"read"`

package/internal/api/users.go

@@ -0,0 +1,75 @@
+package api
+
+import (
+	"context"
+	"fmt"
+	"net/http"
+)
+
+func (c *Client) GetUsers(ctx context.Context) ([]User, error) {
+	endpointURL := c.AccountBaseURL + "/users"
+
+	req, err := c.newRequest(ctx, http.MethodGet, endpointURL, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create request: %w", err)
+	}
+
+	var response []User
+	_, err = c.decodeResponse(req, &response)
+	if err != nil {
+		return nil, err
+	}
+
+	return response, nil
+}
+
+func (c *Client) GetUser(ctx context.Context, userID string) (*User, error) {
+	endpointURL := c.AccountBaseURL + "/users/" + userID
+
+	req, err := c.newRequest(ctx, http.MethodGet, endpointURL, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create request: %w", err)
+	}
+
+	var response User
+	_, err = c.decodeResponse(req, &response)
+	if err != nil {
+		return nil, err
+	}
+
+	return &response, nil
+}
+
+func (c *Client) PutUser(ctx context.Context, userID string, payload UpdateUserPayload) error {
+	endpointURL := c.AccountBaseURL + "/users/" + userID
+
+	body := map[string]UpdateUserPayload{"user": payload}
+
+	req, err := c.newRequest(ctx, http.MethodPut, endpointURL, body)
+	if err != nil {
+		return fmt.Errorf("failed to create update user request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return err
+	}
+
+	return nil
+}
+
+func (c *Client) DeleteUser(ctx context.Context, userID string) error {
+	endpointURL := c.AccountBaseURL + "/users/" + userID
+
+	req, err := c.newRequest(ctx, http.MethodDelete, endpointURL, nil)
+	if err != nil {
+		return fmt.Errorf("failed to create delete user request: %w", err)
+	}
+
+	_, err = c.decodeResponse(req, nil, http.StatusNoContent)
+	if err != nil {
+		return err
+	}
+
+	return nil
+}

package/internal/ui/user_list.go

@@ -0,0 +1,19 @@
+package ui
+
+import (
+	"fmt"
+	"io"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+)
+
+func DisplayUsers(w io.Writer, users []api.User) error {
+	for _, user := range users {
+		status := "active"
+		if !user.Active {
+			status = "inactive"
+		}
+		fmt.Fprintf(w, "%s (%s) - %s [%s]\n", user.Name, user.Email, DisplayID(user.ID), status)
+	}
+	return nil
+}

package/internal/ui/user_show.go

@@ -0,0 +1,23 @@
+package ui
+
+import (
+	"fmt"
+	"io"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+)
+
+func DisplayUser(w io.Writer, user *api.User) error {
+	status := "active"
+	if !user.Active {
+		status = "inactive"
+	}
+
+	fmt.Fprintf(w, "User: %s\n", user.Name)
+	fmt.Fprintf(w, "ID: %s\n", user.ID)
+	fmt.Fprintf(w, "Email: %s\n", user.Email)
+	fmt.Fprintf(w, "Role: %s\n", user.Role)
+	fmt.Fprintf(w, "Status: %s\n", status)
+	fmt.Fprintf(w, "Created At: %s\n", FormatTime(user.CreatedAt))
+	return nil
+}