fix(drafts): include channel id when updating drafts

Author: dedene

README.md

@@ -319,6 +319,7 @@ frontcli drafts get dra_xxx
 
 # Update draft (optimistic locking with version)
 frontcli drafts update dra_xxx --body "Updated draft" --draft-version draft-ver-123
+frontcli drafts update dra_xxx --channel cha_xxx --body-file ./reply.html --draft-version draft-ver-123
 
 # Delete draft
 frontcli drafts delete dra_xxx --draft-version draft-ver-123

internal/cmd/drafts.go

@@ -257,6 +257,7 @@ func (c *DraftGetCmd) Run(flags *RootFlags) error {
 
 type DraftUpdateCmd struct {
 	ID           string `arg:"" help:"Draft ID"`
+	Channel      string `help:"Channel ID required by Front for some draft updates"`
 	Body         string `help:"New body"`
 	BodyFile     string `help:"Read body from file" type:"existingfile"`
 	Subject      string `help:"New subject"`
@@ -294,6 +295,10 @@ func (c *DraftUpdateCmd) Run(flags *RootFlags) error {
 		req["body"] = body
 	}
 
+	if c.Channel != "" {
+		req["channel_id"] = c.Channel
+	}
+
 	if c.Subject != "" {
 		req["subject"] = c.Subject
 	}

internal/cmd/drafts_test.go

@@ -683,6 +683,55 @@ func TestDraftUpdateSendsStringVersion(t *testing.T) {
 	}
 }
 
+func TestDraftUpdateSendsChannelIDWhenProvided(t *testing.T) {
+	t.Setenv("HOME", t.TempDir())
+
+	var gotBody map[string]any
+
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if err := json.NewDecoder(r.Body).Decode(&gotBody); err != nil {
+			t.Fatalf("decode request body: %v", err)
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		_, _ = io.WriteString(w, `{
+			"id":"msg_draft_123",
+			"type":"email",
+			"is_inbound":false,
+			"draft_mode":"shared",
+			"version":"draft-ver-456",
+			"created_at":1710000000,
+			"body":"updated body",
+			"text":"updated body",
+			"subject":"Updated draft"
+		}`)
+	}))
+	defer srv.Close()
+
+	old := newClientFromAuth
+	newClientFromAuth = func(_, _ string) (*api.Client, error) {
+		return api.NewClientWithBaseURL(oauth2.StaticTokenSource(&oauth2.Token{AccessToken: "token"}), srv.URL), nil
+	}
+	t.Cleanup(func() { newClientFromAuth = old })
+
+	cmd := DraftUpdateCmd{
+		ID:           "msg_draft_123",
+		Channel:      "cha_v4x",
+		DraftVersion: "draft-ver-123",
+		Body:         "updated body",
+	}
+	flags := &RootFlags{JSON: true, Account: "test@example.com"}
+
+	if err := cmd.Run(flags); err != nil {
+		t.Fatalf("Run: %v", err)
+	}
+
+	channelID, ok := gotBody["channel_id"].(string)
+	if !ok || channelID != "cha_v4x" {
+		t.Fatalf("expected channel_id in request body, got %#v", gotBody["channel_id"])
+	}
+}
+
 func TestDraftUpdateHumanOutputUsesStringVersion(t *testing.T) {
 	t.Setenv("HOME", t.TempDir())