fizzy-cli 0.8.0 → 0.9.0

package/cmd/notification_list.go

@@ -1,69 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/api"
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var notificationListCmd = &cobra.Command{
-	Use:   "list",
-	Short: "List all notifications",
-	Long:  `Retrieve and display all notifications from Fizzy`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleListNotifications(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleListNotifications(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	notifications, err := a.Client.GetNotifications(context.Background())
-	if err != nil {
-		return fmt.Errorf("fetching notifications: %w", err)
-	}
-
-	read, _ := cmd.Flags().GetBool("read")
-	unread, _ := cmd.Flags().GetBool("unread")
-
-	filtered := filterNotifications(notifications, read, unread)
-
-	if len(filtered) == 0 {
-		fmt.Println("No notifications found")
-		return nil
-	}
-
-	return ui.DisplayNotifications(filtered)
-}
-
-func filterNotifications(notifications []api.Notification, read bool, unread bool) []api.Notification {
-	if !read && !unread {
-		return notifications
-	}
-
-	var filtered []api.Notification
-	for _, notification := range notifications {
-		if read && notification.Read {
-			filtered = append(filtered, notification)
-		} else if unread && !notification.Read {
-			filtered = append(filtered, notification)
-		}
-	}
-
-	return filtered
-}
-
-func init() {
-	notificationCmd.AddCommand(notificationListCmd)
-	notificationListCmd.Flags().BoolP("read", "r", false, "Show only read notifications")
-	notificationListCmd.Flags().BoolP("unread", "u", false, "Show only unread notifications")
-}

package/cmd/notification_list_test.go

@@ -1,288 +0,0 @@
-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/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestNotificationListCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/notifications" {
-			t.Errorf("expected /notifications, 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.Notification{
-			{
-				ID:        "notif-123",
-				Read:      false,
-				ReadAt:    "",
-				CreatedAt: "2025-01-01T00:00:00Z",
-				Title:     "Plain text mentions",
-				Body:      "Assigned to self",
-				Creator: api.User{
-					ID:        "user-123",
-					Name:      "David Heinemeier Hansson",
-					Email:     "david@example.com",
-					Role:      "owner",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.401Z",
-					URL:       "http://fizzy.localhost:3006/897362094/users/03f5v9zjw7pz8717a4no1h8a7",
-				},
-				Card: api.CardReference{
-					ID:     "card-123",
-					Title:  "Plain text mentions",
-					Status: "published",
-					URL:    "http://fizzy.localhost:3006/897362094/cards/3",
-				},
-				URL: "http://fizzy.localhost:3006/897362094/notifications/03f5va03bpuvkcjemcxl73ho2",
-			},
-			{
-				ID:        "notif-456",
-				Read:      true,
-				ReadAt:    "2025-01-02T00:00:00Z",
-				CreatedAt: "2025-01-01T12:00:00Z",
-				Title:     "Comment reply",
-				Body:      "Someone replied to your comment",
-				Creator: api.User{
-					ID:        "user-456",
-					Name:      "Jason Fried",
-					Email:     "jason@example.com",
-					Role:      "member",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.419Z",
-					URL:       "http://fizzy.localhost:3006/897362094/users/03f5v9zjysoy0fqs9yg0ei3hq",
-				},
-				Card: api.CardReference{
-					ID:     "card-456",
-					Title:  "Fix bug",
-					Status: "in_progress",
-					URL:    "http://fizzy.localhost:3006/897362094/cards/4",
-				},
-				URL: "http://fizzy.localhost:3006/897362094/notifications/notif-456",
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListNotifications(cmd); err != nil {
-		t.Fatalf("handleListNotifications failed: %v", err)
-	}
-}
-
-func TestNotificationListCommandNoNotifications(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.Header().Set("Content-Type", "application/json")
-		json.NewEncoder(w).Encode([]api.Notification{})
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListNotifications(cmd); err != nil {
-		t.Fatalf("handleListNotifications failed: %v", err)
-	}
-}
-
-func TestNotificationListCommandAPIError(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, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListNotifications(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching notifications: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestNotificationListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := notificationListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListNotifications(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)
-	}
-}
-
-func TestNotificationListCommandWithReadFilter(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.Header().Set("Content-Type", "application/json")
-		response := []api.Notification{
-			{
-				ID:        "notif-123",
-				Read:      false,
-				CreatedAt: "2025-01-01T00:00:00Z",
-				Title:     "Unread notification",
-				Body:      "This should be filtered out",
-				Creator: api.User{
-					ID:        "user-123",
-					Name:      "David Heinemeier Hansson",
-					Email:     "david@example.com",
-					Role:      "owner",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.401Z",
-				},
-				Card: api.CardReference{
-					ID:     "card-123",
-					Title:  "Test card",
-					Status: "published",
-				},
-			},
-			{
-				ID:        "notif-456",
-				Read:      true,
-				ReadAt:    "2025-01-02T00:00:00Z",
-				CreatedAt: "2025-01-01T12:00:00Z",
-				Title:     "Read notification",
-				Body:      "This should be shown",
-				Creator: api.User{
-					ID:        "user-456",
-					Name:      "Jason Fried",
-					Email:     "jason@example.com",
-					Role:      "member",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.419Z",
-				},
-				Card: api.CardReference{
-					ID:     "card-456",
-					Title:  "Another card",
-					Status: "in_progress",
-				},
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.Flags().Set("read", "true")
-
-	if err := handleListNotifications(cmd); err != nil {
-		t.Fatalf("handleListNotifications failed: %v", err)
-	}
-}
-
-func TestNotificationListCommandWithUnreadFilter(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.Header().Set("Content-Type", "application/json")
-		response := []api.Notification{
-			{
-				ID:        "notif-123",
-				Read:      false,
-				CreatedAt: "2025-01-01T00:00:00Z",
-				Title:     "Unread notification",
-				Body:      "This should be shown",
-				Creator: api.User{
-					ID:        "user-123",
-					Name:      "David Heinemeier Hansson",
-					Email:     "david@example.com",
-					Role:      "owner",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.401Z",
-				},
-				Card: api.CardReference{
-					ID:     "card-123",
-					Title:  "Test card",
-					Status: "published",
-				},
-			},
-			{
-				ID:        "notif-456",
-				Read:      true,
-				ReadAt:    "2025-01-02T00:00:00Z",
-				CreatedAt: "2025-01-01T12:00:00Z",
-				Title:     "Read notification",
-				Body:      "This should be filtered out",
-				Creator: api.User{
-					ID:        "user-456",
-					Name:      "Jason Fried",
-					Email:     "jason@example.com",
-					Role:      "member",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.419Z",
-				},
-				Card: api.CardReference{
-					ID:     "card-456",
-					Title:  "Another card",
-					Status: "in_progress",
-				},
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.Flags().Set("unread", "true")
-
-	if err := handleListNotifications(cmd); err != nil {
-		t.Fatalf("handleListNotifications failed: %v", err)
-	}
-}

package/cmd/notification_read.go

@@ -1,51 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strings"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var notificationReadCmd = &cobra.Command{
-	Use:   "read <notification_id>",
-	Short: "Mark notification as read and display it",
-	Long:  `Mark a notification as read and display its content`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleReadNotification(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleReadNotification(cmd *cobra.Command, notificationID string) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	if _, err := a.Client.PostNotificationReading(context.Background(), notificationID); err != nil {
-		if strings.Contains(err.Error(), "404") {
-			return fmt.Errorf("notification not found")
-		}
-		return fmt.Errorf("marking notification as read: %w", err)
-	}
-
-	notification, err := a.Client.GetNotification(context.Background(), notificationID)
-	if err != nil {
-		if strings.Contains(err.Error(), "404") {
-			return fmt.Errorf("notification not found")
-		}
-		return fmt.Errorf("fetching notification: %w", err)
-	}
-
-	return ui.DisplayNotification(notification)
-}
-
-func init() {
-	notificationCmd.AddCommand(notificationReadCmd)
-}

package/cmd/notification_read_all.go

@@ -1,38 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var notificationReadAllCmd = &cobra.Command{
-	Use:   "read-all",
-	Short: "Mark all unread notifications as read",
-	Long:  `Mark all unread notifications as read`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleReadAllNotifications(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleReadAllNotifications(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	if _, err := a.Client.PostBulkNotificationsReading(context.Background()); err != nil {
-		return fmt.Errorf("marking all notifications as read: %w", err)
-	}
-
-	fmt.Println("✓ All notifications marked as read successfully")
-	return nil
-}
-
-func init() {
-	notificationCmd.AddCommand(notificationReadAllCmd)
-}

package/cmd/notification_read_all_test.go

@@ -1,75 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestNotificationReadAllCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path == "/notifications/bulk_reading" && r.Method == http.MethodPost {
-			w.WriteHeader(http.StatusNoContent)
-			return
-		}
-
-		t.Errorf("unexpected request: %s %s", r.Method, r.URL.Path)
-		w.WriteHeader(http.StatusNotFound)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationReadAllCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleReadAllNotifications(cmd); err != nil {
-		t.Fatalf("handleReadAllNotifications failed: %v", err)
-	}
-}
-
-func TestNotificationReadAllCommandAPIError(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, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationReadAllCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReadAllNotifications(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-}
-
-func TestNotificationReadAllCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := notificationReadAllCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReadAllNotifications(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/cmd/notification_read_test.go

@@ -1,138 +0,0 @@
-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/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestNotificationReadCommand(t *testing.T) {
-	requestCount := 0
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		requestCount++
-
-		if r.URL.Path == "/notifications/notif-123/reading" && r.Method == http.MethodPost {
-			w.WriteHeader(http.StatusNoContent)
-			return
-		}
-
-		if r.URL.Path == "/notifications/notif-123" && r.Method == http.MethodGet {
-			w.Header().Set("Content-Type", "application/json")
-			response := api.Notification{
-				ID:        "notif-123",
-				Read:      true,
-				ReadAt:    "2025-01-01T00:00:00Z",
-				CreatedAt: "2025-01-01T00:00:00Z",
-				Title:     "Test Notification",
-				Body:      "This is a test notification",
-				Creator: api.User{
-					ID:        "user-123",
-					Name:      "David Heinemeier Hansson",
-					Email:     "david@example.com",
-					Role:      "owner",
-					Active:    true,
-					CreatedAt: "2025-12-05T19:36:35.401Z",
-				},
-				Card: api.CardReference{
-					ID:     "card-123",
-					Title:  "Test Card",
-					Status: "published",
-					URL:    "http://fizzy.localhost:3006/897362094/cards/1",
-				},
-				URL: "http://fizzy.localhost:3006/897362094/notifications/notif-123",
-			}
-			json.NewEncoder(w).Encode(response)
-			return
-		}
-
-		t.Errorf("unexpected request: %s %s", r.Method, r.URL.Path)
-		w.WriteHeader(http.StatusNotFound)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationReadCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleReadNotification(cmd, "notif-123"); err != nil {
-		t.Fatalf("handleReadNotification failed: %v", err)
-	}
-
-	if requestCount != 2 {
-		t.Errorf("expected 2 requests (POST and GET), got %d", requestCount)
-	}
-}
-
-func TestNotificationReadCommandNotFound(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNotFound)
-		w.Write([]byte("Notification not found"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationReadCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReadNotification(cmd, "notif-invalid")
-	if err == nil {
-		t.Errorf("expected error for invalid notification")
-	}
-	if err.Error() != "notification not found" {
-		t.Errorf("expected 'notification not found' error, got %v", err)
-	}
-}
-
-func TestNotificationReadCommandAPIError(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, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := notificationReadCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReadNotification(cmd, "notif-123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-}
-
-func TestNotificationReadCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := notificationReadCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReadNotification(cmd, "notif-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/notification_unread.go

@@ -1,44 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strings"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var notificationUnreadCmd = &cobra.Command{
-	Use:   "unread <notification_id>",
-	Short: "Mark notification as unread",
-	Long:  `Mark a notification as unread`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleUnreadNotification(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleUnreadNotification(cmd *cobra.Command, notificationID string) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	_, err := a.Client.DeleteNotificationReading(context.Background(), notificationID)
-	if err != nil {
-		if strings.Contains(err.Error(), "404") {
-			return fmt.Errorf("notification not found")
-		}
-		return fmt.Errorf("marking notification as unread: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Notification marked as unread successfully\n")
-	return nil
-}
-
-func init() {
-	notificationCmd.AddCommand(notificationUnreadCmd)
-}