feat(chat): add react shorthand command for reactions create (#435)

- add 'chat messages react <message> <emoji>' as a direct shorthand for reaction creation
- keep 'reaction' as the alias for the 'reactions' command group
- add regression coverage for the shorthand path

Co-authored-by: Fernando Pina dos Santos <fernando.dossantos@trilogy.com>

Author: steipete

CHANGELOG.md

@@ -6,7 +6,7 @@
 - Admin: add Workspace Admin Directory commands for users and groups, including user list/get/create/suspend and group membership list/add/remove. (#403) — thanks @dl-alexandre.
 - Auth: add `--access-token` / `GOG_ACCESS_TOKEN` for direct access-token auth in headless or CI flows, bypassing stored refresh tokens. (#419) — thanks @mmkal.
 - Auth: add Application Default Credentials mode via `GOG_AUTH_MODE=adc` for Workload Identity, Cloud Run, and local `gcloud` ADC flows without stored OAuth refresh tokens. (#357) — thanks @tengis617.
-- Chat: add `chat messages reactions create|list|delete` to manage emoji reactions on messages; `react` and `reaction` are aliases for the reactions command group. (#426) — thanks @fernandopps.
+- Chat: add `chat messages reactions create|list|delete` to manage emoji reactions on messages; `chat messages react <message> <emoji>` as a shorthand for creating reactions; `reaction` is an alias for `reactions`. (#426) — thanks @fernandopps.
 - Sheets: add named range management (`sheets named-ranges`) and let range-based Sheets commands accept named range names where GridRange-backed operations are needed. (#278) — thanks @TheCrazyLex.
 - Sheets: add `add-tab`, `rename-tab`, and `delete-tab` commands for managing spreadsheet tabs, with delete dry-run/confirmation guardrails. (#309) — thanks @JulienMalige.
 - Docs: add `--tab-id` to editing commands so write/update/insert/delete/find-replace can target a specific Google Docs tab. (#330) — thanks @ignacioreyna.

README.md

@@ -1131,7 +1131,7 @@ gog chat messages list spaces/<spaceId> --thread <threadId>
 gog chat messages list spaces/<spaceId> --unread
 gog chat messages send spaces/<spaceId> --text "Build complete!" --thread spaces/<spaceId>/threads/<threadId>
 gog chat messages reactions list spaces/<spaceId>/messages/<messageId>
-gog chat messages react spaces/<spaceId>/messages/<messageId> "👍"
+gog chat messages react spaces/<spaceId>/messages/<messageId> "👍"  # shorthand for reactions create
 gog chat messages reactions delete spaces/<spaceId>/messages/<messageId>/reactions/<reactionId>
 
 # Threads

internal/cmd/chat_messages.go

@@ -15,7 +15,8 @@ import (
 type ChatMessagesCmd struct {
 	List      ChatMessagesListCmd      `cmd:"" name:"list" aliases:"ls" help:"List messages"`
 	Send      ChatMessagesSendCmd      `cmd:"" name:"send" aliases:"create,post" help:"Send a message"`
-	Reactions ChatMessagesReactionsCmd `cmd:"" name:"reactions" aliases:"react,reaction" help:"Manage emoji reactions on a message"`
+	React     ChatMessagesReactCmd     `cmd:"" name:"react" help:"Add an emoji reaction to a message"`
+	Reactions ChatMessagesReactionsCmd `cmd:"" name:"reactions" aliases:"reaction" help:"Manage emoji reactions on a message"`
 }
 
 type ChatMessagesListCmd struct {

internal/cmd/chat_reactions.go

@@ -12,6 +12,17 @@ import (
 	"github.com/steipete/gogcli/internal/ui"
 )
 
+type ChatMessagesReactCmd struct {
+	Message string `arg:"" name:"message" help:"Message resource (spaces/.../messages/...) or bare message ID"`
+	Emoji   string `arg:"" name:"emoji" help:"Emoji unicode character (e.g. 👍)"`
+	Space   string `name:"space" help:"Space name (required when message is a bare ID)"`
+}
+
+func (c *ChatMessagesReactCmd) Run(ctx context.Context, flags *RootFlags) error {
+	cmd := &ChatMessagesReactionsCreateCmd{Message: c.Message, Emoji: c.Emoji, Space: c.Space}
+	return cmd.Run(ctx, flags)
+}
+
 type ChatMessagesReactionsCmd struct {
 	Create ChatMessagesReactionsCreateCmd `cmd:"" name:"create" aliases:"add" help:"Add an emoji reaction to a message"`
 	List   ChatMessagesReactionsListCmd   `cmd:"" name:"list" aliases:"ls" help:"List reactions on a message"`

internal/cmd/chat_reactions_test.go

@@ -165,6 +165,60 @@ func TestExecute_ChatMessagesReactionsCreate_BareIDWithSpace(t *testing.T) {
 	}
 }
 
+func TestExecute_ChatMessagesReact_Shorthand(t *testing.T) {
+	origNew := newChatService
+	t.Cleanup(func() { newChatService = origNew })
+
+	var gotPath string
+	var gotEmoji string
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if !(r.Method == http.MethodPost && strings.Contains(r.URL.Path, "/reactions")) {
+			http.NotFound(w, r)
+			return
+		}
+		gotPath = r.URL.Path
+		var body map[string]any
+		_ = json.NewDecoder(r.Body).Decode(&body)
+		if emoji, ok := body["emoji"].(map[string]any); ok {
+			gotEmoji, _ = emoji["unicode"].(string)
+		}
+		w.Header().Set("Content-Type", "application/json")
+		_ = json.NewEncoder(w).Encode(map[string]any{
+			"name":  "spaces/AAA/messages/msg1/reactions/r1",
+			"emoji": map[string]any{"unicode": gotEmoji},
+		})
+	}))
+	defer srv.Close()
+
+	svc, err := chat.NewService(context.Background(),
+		option.WithoutAuthentication(),
+		option.WithHTTPClient(srv.Client()),
+		option.WithEndpoint(srv.URL+"/"),
+	)
+	if err != nil {
+		t.Fatalf("NewService: %v", err)
+	}
+	newChatService = func(context.Context, string) (*chat.Service, error) { return svc, nil }
+
+	out := captureStdout(t, func() {
+		_ = captureStderr(t, func() {
+			if err := Execute([]string{"--account", "a@b.com", "chat", "messages", "react", "spaces/AAA/messages/msg1", "📦"}); err != nil {
+				t.Fatalf("Execute: %v", err)
+			}
+		})
+	})
+
+	if !strings.Contains(gotPath, "spaces/AAA/messages/msg1/reactions") {
+		t.Fatalf("unexpected request path: %q", gotPath)
+	}
+	if gotEmoji != "📦" {
+		t.Fatalf("unexpected emoji sent: %q", gotEmoji)
+	}
+	if !strings.Contains(out, "spaces/AAA/messages/msg1/reactions/r1") {
+		t.Fatalf("unexpected output: %q", out)
+	}
+}
+
 func TestExecute_ChatMessagesReactionsCreate_ConsumerBlocked(t *testing.T) {
 	origNew := newChatService
 	t.Cleanup(func() { newChatService = origNew })