fix: stricter typing for list schemas

Author: srtaalej

slack-api-client/src/main/java/com/slack/api/methods/impl/MethodsClientImpl.java

@@ -3209,42 +3209,6 @@ public SearchFilesResponse searchFiles(RequestConfigurator<SearchFilesRequest.Se
         return searchFiles(req.configure(SearchFilesRequest.builder()).build());
     }
 
-    @Override
-    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
-    public StarsAddResponse starsAdd(StarsAddRequest req) throws IOException, SlackApiException {
-        return postFormWithTokenAndParseResponse(toForm(req), Methods.STARS_ADD, getToken(req), StarsAddResponse.class);
-    }
-
-    @Override
-    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
-    public StarsAddResponse starsAdd(RequestConfigurator<StarsAddRequest.StarsAddRequestBuilder> req) throws IOException, SlackApiException {
-        return starsAdd(req.configure(StarsAddRequest.builder()).build());
-    }
-
-    @Override
-    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
-    public StarsListResponse starsList(StarsListRequest req) throws IOException, SlackApiException {
-        return postFormWithTokenAndParseResponse(toForm(req), Methods.STARS_LIST, getToken(req), StarsListResponse.class);
-    }
-
-    @Override
-    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
-    public StarsListResponse starsList(RequestConfigurator<StarsListRequest.StarsListRequestBuilder> req) throws IOException, SlackApiException {
-        return starsList(req.configure(StarsListRequest.builder()).build());
-    }
-
-    @Override
-    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
-    public StarsRemoveResponse starsRemove(StarsRemoveRequest req) throws IOException, SlackApiException {
-        return postFormWithTokenAndParseResponse(toForm(req), Methods.STARS_REMOVE, getToken(req), StarsRemoveResponse.class);
-    }
-
-    @Override
-    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
-    public StarsRemoveResponse starsRemove(RequestConfigurator<StarsRemoveRequest.StarsRemoveRequestBuilder> req) throws IOException, SlackApiException {
-        return starsRemove(req.configure(StarsRemoveRequest.builder()).build());
-    }
-
     @Override
     public SlackListsAccessDeleteResponse slackListsAccessDelete(SlackListsAccessDeleteRequest req) throws IOException, SlackApiException {
         return postFormWithTokenAndParseResponse(toForm(req), Methods.SLACKLISTS_ACCESS_DELETE, getToken(req), SlackListsAccessDeleteResponse.class);
@@ -3365,6 +3329,42 @@ public SlackListsUpdateResponse slackListsUpdate(RequestConfigurator<SlackListsU
         return slackListsUpdate(req.configure(SlackListsUpdateRequest.builder()).build());
     }
 
+    @Override
+    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
+    public StarsAddResponse starsAdd(StarsAddRequest req) throws IOException, SlackApiException {
+        return postFormWithTokenAndParseResponse(toForm(req), Methods.STARS_ADD, getToken(req), StarsAddResponse.class);
+    }
+
+    @Override
+    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
+    public StarsAddResponse starsAdd(RequestConfigurator<StarsAddRequest.StarsAddRequestBuilder> req) throws IOException, SlackApiException {
+        return starsAdd(req.configure(StarsAddRequest.builder()).build());
+    }
+
+    @Override
+    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
+    public StarsListResponse starsList(StarsListRequest req) throws IOException, SlackApiException {
+        return postFormWithTokenAndParseResponse(toForm(req), Methods.STARS_LIST, getToken(req), StarsListResponse.class);
+    }
+
+    @Override
+    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
+    public StarsListResponse starsList(RequestConfigurator<StarsListRequest.StarsListRequestBuilder> req) throws IOException, SlackApiException {
+        return starsList(req.configure(StarsListRequest.builder()).build());
+    }
+
+    @Override
+    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
+    public StarsRemoveResponse starsRemove(StarsRemoveRequest req) throws IOException, SlackApiException {
+        return postFormWithTokenAndParseResponse(toForm(req), Methods.STARS_REMOVE, getToken(req), StarsRemoveResponse.class);
+    }
+
+    @Override
+    @Deprecated // https://docs.slack.dev/changelog/2023-07-its-later-already-for-stars-and-reminders
+    public StarsRemoveResponse starsRemove(RequestConfigurator<StarsRemoveRequest.StarsRemoveRequestBuilder> req) throws IOException, SlackApiException {
+        return starsRemove(req.configure(StarsRemoveRequest.builder()).build());
+    }
+
     @Override
     public TeamAccessLogsResponse teamAccessLogs(TeamAccessLogsRequest req) throws IOException, SlackApiException {
         this.teamId.ifPresent(currentTeamId -> {

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsAccessDeleteRequest.java

@@ -1,6 +1,5 @@
 package com.slack.api.methods.request.slack_lists;
 
-import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import java.util.List;
 import lombok.Builder;
@@ -21,19 +20,16 @@ public class SlackListsAccessDeleteRequest implements SlackApiRequest {
     /**
      * Encoded ID of the List.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**
      * List of channels you wish to update access for. Can only be used if user_ids is not provided. (Optional)
      */
-    @SerializedName("channel_ids")
     private List<String> channelIds;
 
     /**
      * List of users you wish to update access for. Can only be used if channel_ids is not provided. (Optional)
      */
-    @SerializedName("user_ids")
     private List<String> userIds;
 }
 

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsAccessSetRequest.java

@@ -1,6 +1,5 @@
 package com.slack.api.methods.request.slack_lists;
 
-import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import java.util.List;
 import lombok.Builder;
@@ -21,25 +20,21 @@ public class SlackListsAccessSetRequest implements SlackApiRequest {
     /**
      * Encoded ID of the List.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**
      * Desired level of access.
      */
-    @SerializedName("access_level")
     private String accessLevel;
 
     /**
      * List of channels you wish to update access for. Can only be used if user_ids is not provided. (Optional)
      */
-    @SerializedName("channel_ids")
     private List<String> channelIds;
 
     /**
      * List of users you wish to update access for. Can only be used if channel_ids is not provided. (Optional)
      */
-    @SerializedName("user_ids")
     private List<String> userIds;
 }
 

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsCreateRequest.java

@@ -3,6 +3,7 @@
 import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import com.slack.api.model.block.RichTextBlock;
+import com.slack.api.model.list.ListColumn;
 
 import java.util.List;
 import java.util.Map;
@@ -30,13 +31,12 @@ public class SlackListsCreateRequest implements SlackApiRequest {
     /**
      * A rich text description of the List. (Optional)
      */
-    @SerializedName("description_blocks")
     private List<RichTextBlock> descriptionBlocks;
 
     /**
      * Column definition for the List. (Optional)
      */
-    private List<Map<String, Object>> schema;
+    private List<ListColumn> schema;
 
     /**
      * ID of the List to copy. (Optional)
@@ -53,6 +53,5 @@ public class SlackListsCreateRequest implements SlackApiRequest {
     /**
      * Boolean indicating whether the List should be used to track todo tasks. (Optional)
      */
-    @SerializedName("todo_mode")
     private Boolean todoMode;
 }

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsDownloadGetRequest.java

@@ -1,6 +1,5 @@
 package com.slack.api.methods.request.slack_lists;
 
-import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import lombok.Builder;
 import lombok.Data;
@@ -20,12 +19,10 @@ public class SlackListsDownloadGetRequest implements SlackApiRequest {
     /**
      * ID of the List to export.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**
      * The ID of the recently started job to export the List.
      */
-    @SerializedName("job_id")
     private String jobId;
 }

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsDownloadStartRequest.java

@@ -1,6 +1,5 @@
 package com.slack.api.methods.request.slack_lists;
 
-import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import lombok.Builder;
 import lombok.Data;
@@ -20,12 +19,10 @@ public class SlackListsDownloadStartRequest implements SlackApiRequest {
     /**
      * ID of the List to export.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**
      * Include archived rows. (Optional)
      */
-    @SerializedName("include_archived")
     private Boolean includeArchived;
 }

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsItemsCreateRequest.java

@@ -2,9 +2,9 @@
 
 import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
+import com.slack.api.model.list.ListRecord;
 
 import java.util.List;
-import java.util.Map;
 
 import lombok.Builder;
 import lombok.Data;
@@ -24,7 +24,6 @@ public class SlackListsItemsCreateRequest implements SlackApiRequest {
     /**
      * ID of the List containing the items.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**
@@ -42,6 +41,5 @@ public class SlackListsItemsCreateRequest implements SlackApiRequest {
     /**
      * Initial item data. (Optional)
      */
-    @SerializedName("initial_fields")
-    private List<Map<String, Object>> initialFields;
+    private List<ListRecord.Field> initialFields;
 }

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsItemsDeleteMultipleRequest.java

@@ -1,6 +1,5 @@
 package com.slack.api.methods.request.slack_lists;
 
-import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import java.util.List;
 import lombok.Builder;
@@ -21,7 +20,6 @@ public class SlackListsItemsDeleteMultipleRequest implements SlackApiRequest {
     /**
      * ID of the List containing the items.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsItemsDeleteRequest.java

@@ -1,6 +1,5 @@
 package com.slack.api.methods.request.slack_lists;
 
-import com.google.gson.annotations.SerializedName;
 import com.slack.api.methods.SlackApiRequest;
 import lombok.Builder;
 import lombok.Data;
@@ -20,7 +19,6 @@ public class SlackListsItemsDeleteRequest implements SlackApiRequest {
     /**
      * ID of the List containing the items.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**

slack-api-client/src/main/java/com/slack/api/methods/request/slack_lists/SlackListsItemsInfoRequest.java

@@ -20,7 +20,6 @@ public class SlackListsItemsInfoRequest implements SlackApiRequest {
     /**
      * ID of the List containing the items.
      */
-    @SerializedName("list_id")
     private String listId;
 
     /**
@@ -31,7 +30,7 @@ public class SlackListsItemsInfoRequest implements SlackApiRequest {
     /**
      * Set to true to include is_subscribed data for the returned List row. (Optional)
      */
-    @SerializedName("include_is_subscribed")
+    //@SerializedName("include_is_subscribed")
     @Builder.Default
     private Boolean includeIsSubscribed = false;
 }