Address SDK bump review feedback

Author: robzolkos

internal/commands/board_test.go

@@ -404,9 +404,13 @@ func TestBoardUpdate(t *testing.T) {
 		if mock.PatchCalls[0].Path != "/boards/123" {
 			t.Errorf("expected path '/boards/123', got '%s'", mock.PatchCalls[0].Path)
 		}
-		if got := responseDataMap(t, result)["name"]; got != "Updated Name" {
+		data := responseDataMap(t, result)
+		if got := data["name"]; got != "Updated Name" {
 			t.Errorf("expected update response body name, got %#v", got)
 		}
+		if got := data["id"]; got != "123" {
+			t.Errorf("expected update response body id, got %#v", got)
+		}
 	})
 
 	t.Run("handles API error", func(t *testing.T) {

internal/commands/column_test.go

@@ -282,9 +282,13 @@ func TestColumnUpdate(t *testing.T) {
 		if mock.PatchCalls[0].Path != "/boards/123/columns/col-1" {
 			t.Errorf("expected path '/boards/123/columns/col-1', got '%s'", mock.PatchCalls[0].Path)
 		}
-		if got := responseDataMap(t, result)["name"]; got != "Updated Column" {
+		data := responseDataMap(t, result)
+		if got := data["name"]; got != "Updated Column" {
 			t.Errorf("expected update response body name, got %#v", got)
 		}
+		if got := data["id"]; got != "col-1" {
+			t.Errorf("expected update response body id, got %#v", got)
+		}
 	})
 
 	t.Run("requires board flag", func(t *testing.T) {

internal/commands/identity.go

@@ -53,18 +53,25 @@ var identityTimezoneUpdateCmd = &cobra.Command{
 			return newRequiredFlagError("timezone")
 		}
 
-		_, err := getSDKClient().Identity().UpdateMyTimezone(cmd.Context(), cfg.Account, &generated.UpdateMyTimezoneRequest{
+		resp, err := getSDKClient().Identity().UpdateMyTimezone(cmd.Context(), cfg.Account, &generated.UpdateMyTimezoneRequest{
 			TimezoneName: identityTimezoneUpdateTimezone,
 		})
 		if err != nil {
 			return convertSDKError(err)
 		}
 
+		data := any(map[string]any{"timezone_name": identityTimezoneUpdateTimezone})
+		if resp != nil && len(resp.Data) > 0 {
+			if normalized := normalizeAny(resp.Data); normalized != nil {
+				data = normalized
+			}
+		}
+
 		breadcrumbs := []Breadcrumb{
 			breadcrumb("show", "fizzy identity show", "View identity"),
 		}
 
-		printMutation(map[string]any{"timezone_name": identityTimezoneUpdateTimezone}, "Timezone updated", breadcrumbs)
+		printMutation(data, "Timezone updated", breadcrumbs)
 		return nil
 	},
 }

internal/commands/identity_test.go

@@ -10,7 +10,13 @@ import (
 func TestIdentityTimezoneUpdate(t *testing.T) {
 	t.Run("updates timezone", func(t *testing.T) {
 		mock := NewMockClient()
-		mock.PatchResponse = &client.APIResponse{StatusCode: 204, Data: nil}
+		mock.PatchResponse = &client.APIResponse{
+			StatusCode: 200,
+			Data: map[string]any{
+				"timezone_name": "America/New_York",
+				"updated_at":    "2026-06-03T21:15:00Z",
+			},
+		}
 
 		result := SetTestModeWithSDK(mock)
 		SetTestConfig("token", "account", "https://api.example.com")
@@ -42,6 +48,29 @@ func TestIdentityTimezoneUpdate(t *testing.T) {
 		if result.Response.Summary != "Timezone updated" {
 			t.Errorf("expected timezone summary, got %q", result.Response.Summary)
 		}
+		if got := responseDataMap(t, result)["updated_at"]; got != "2026-06-03T21:15:00Z" {
+			t.Errorf("expected timezone response body updated_at, got %#v", got)
+		}
+	})
+
+	t.Run("falls back to requested timezone for empty response", func(t *testing.T) {
+		mock := NewMockClient()
+		mock.PatchResponse = &client.APIResponse{StatusCode: 204, Data: nil}
+
+		result := SetTestModeWithSDK(mock)
+		SetTestConfig("token", "account", "https://api.example.com")
+		identityTimezoneUpdateTimezone = "America/New_York"
+		defer func() {
+			identityTimezoneUpdateTimezone = ""
+			resetTest()
+		}()
+
+		err := identityTimezoneUpdateCmd.RunE(identityTimezoneUpdateCmd, []string{})
+		assertExitCode(t, err, 0)
+
+		if got := responseDataMap(t, result)["timezone_name"]; got != "America/New_York" {
+			t.Errorf("expected fallback timezone_name, got %#v", got)
+		}
 	})
 
 	t.Run("requires timezone", func(t *testing.T) {

internal/commands/user_test.go

@@ -107,9 +107,13 @@ func TestUserUpdate(t *testing.T) {
 		if body["name"] != "New Name" {
 			t.Errorf("expected name 'New Name', got '%v'", body["name"])
 		}
-		if got := responseDataMap(t, result)["name"]; got != "New Name" {
+		data := responseDataMap(t, result)
+		if got := data["name"]; got != "New Name" {
 			t.Errorf("expected update response body name, got %#v", got)
 		}
+		if got := data["id"]; got != "user-1" {
+			t.Errorf("expected update response body id, got %#v", got)
+		}
 	})
 
 	t.Run("updates user avatar via multipart", func(t *testing.T) {