feat: add webapi response changes as of 2026-05-19 by slackapi[bot] · Pull Request #1592 · slackapi/java-slack-sdk

slackapi · 2026-05-19T05:07:46Z

This PR updates the web API client and model based on the production E2E test results.

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

codecov · 2026-05-19T05:17:19Z

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.33%. Comparing base (8f4d633) to head (22dab56).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...java/com/slack/api/methods/RequestFormBuilder.java	0.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #1592      +/-   ##
============================================
- Coverage     73.39%   73.33%   -0.07%     
+ Complexity     4532     4527       -5     
============================================
  Files           478      478              
  Lines         14312    14314       +2     
  Branches       1490     1491       +1     
============================================
- Hits          10505    10497       -8     
- Misses         2919     2925       +6     
- Partials        888      892       +4

Flag	Coverage Δ
jdk-14	`73.33% <0.00%> (-0.07%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

The workspace admin token no longer has access to the target workspace (T06M2FAFCF3). Switch helper/setup calls in admin API tests to use the org admin token with explicit team_id, which is required for org-level tokens calling workspace-scoped methods. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

This reverts commit c68228b.

This reverts commit 0dafb70.

- Add ~100 missing audit action constants (LegalHold, SlackFirstCrm, MCP, AI actions) - Fix cant_kick_self by excluding userToken's own user from kick target selection Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

…lation failure Duplicate field in WorkspaceOrOrg class prevented Lombok annotation processing, breaking all Builder class generation in slack-api-client. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

… api

- Add thread_ts, permissions (Object), privateChannelAccess fields - Add audit actions: slack_ai_mcp_resource_read, pref_allow_native_giphy_gif_picker - Fix dnd_Test: limit to 50 active users - Fix emoji_Test: create emoji before listing - Fix functions_Test: add teamId parameter - Fix users_Test: filter deleted users from lookupByEmail, use teamId for Grid - Update SCIM2 json-logs from test runs Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

…it actions

…in tests

zimeg

📣 Writings as this becomes more stable! Another few are planned to follow.

zimeg · 2026-06-30T08:03:38Z

+    private List<Permission> permissions;
+
+    @Data
+    @Builder
+    @NoArgsConstructor
+    @AllArgsConstructor
+    public static class Permission {
+        private String visibility;
+        private String permissionType;
+        private List<String> userIds;
+        private List<String> teamIds;
+        private List<String> orgIds;
+    }


📚 note: Added the "permissions" argument to admin.functions.permissions.set for testing sake.

🔗 https://docs.slack.dev/reference/methods/admin.functions.permissions.set/

zimeg · 2026-06-30T08:06:20Z

            final List<String> appIds = Arrays.asList(System.getenv(Constants.SLACK_SDK_TEST_GRID_APP_IDS).split(","));
            AdminFunctionsListResponse response = methodsAsync.adminFunctionsList(r -> r
                    .appIds(appIds)
+                    .teamId(gridTeamId)


🪬 note: A teamId is required with the orgAdminUserToken used I find.

zimeg · 2026-07-20T20:13:24Z

            }
            // this should be rejected
            assertThat(callCount.get(), is(0));
+            slack.close();


👁️‍🗨️ note: This guards against leaking connections that caused unexpected counts following!

zimeg · 2026-07-31T05:54:33Z

+            ConversationsListResponse channelsList = slack.methods(teamAdminUserToken).conversationsList(r -> r
                    .excludeArchived(true)
                    .types(Arrays.asList(ConversationType.PUBLIC_CHANNEL))
                    .limit(2)
-            ).getChannels().stream().map(c -> c.getId()).collect(Collectors.toList());
+            );
+            assertThat(channelsList.getError(), is(nullValue()));


🔬 note: Added an error check here to troubleshoot issues with the initial request which shouldn't change test behavior!

zimeg · 2026-07-31T05:55:52Z

    @Test
    public void emoji() throws Exception {
        if (orgAdminUserToken != null) {
-            CompletableFuture<AdminEmojiListResponse> list = methodsAsync.adminEmojiList(r -> r.limit(100));


🎯 note: For testing on sandboxes without much customization we prefer to add a new emoji first.

zimeg · 2026-07-31T05:57:03Z

    public void getFile_public_channel() throws Exception {
        if (orgAdminUserToken != null) {
            AdminAnalyticsGetFileResponse response = methodsAsync.adminAnalyticsGetFile(r -> r
-                    .date("2025-01-20")


⌛ note: Here we use a more recent date that exists on a sandbox but this might require ongoing upkeep.

zimeg · 2026-07-31T07:25:01Z

+        private Object permissions;
        private String sharedTo; // channel_workspaces_updated
-        private String reason;
+        private Object reason;


reason — keep as Object. On the live audit response this field appears as both a string and an array of strings: a plain string on some entries, and an array on anomaly / session-reset entries (which this org can generate — anomaly events are org-wide). Retyping to String throws Expected a string but was BEGIN_ARRAY on any anomaly entry. Could be tightened later with a custom Gson TypeAdapter that normalizes both shapes to List<String>, but not worth it here. Note: the "reason": {} in the audit sample JSON is Quicktype output regenerated after the type became Object, so it reflects the model, not the wire — not evidence.

zimeg · 2026-07-31T07:25:24Z

        private String botId;
        private List<String> channels;
-        private List<Permission> permissions;
+        private Object permissions;


permissions — Object is defensible but not yet confirmed; the live audit JSON is the source of truth here. The orphaned Permission/Resource/Grant/WildCard classes (~L485) model a function-trigger permission-grant tree, but this was almost certainly widened to Object because some entry arrived with a shape that broke List<Permission>. Revert test: change to List<Permission> and run getLogs_all_actions — if it passes, the typed list is safe and those classes are no longer dead; if it throws (Expected BEGIN_ARRAY but was ...), Object was correct — revert and add an inline comment capturing the offending shape (the rationale the original change lacked).

zimeg · 2026-07-31T07:25:25Z

        private String stepFunctionType; // "Custom", "Builtin"
        private String stepFunctionAppId; // "A05QFAJ8LBA"
-        private String externalAuthConfiguration;
+        private java.util.List<Object> externalAuthConfiguration;


externalAuthConfiguration (List<Object>) — yellow flag; do not assume List is safe. On the live wire this field was observed as a plain string earlier this year (auto-generated from an E2E run) and later as an array, so it has been seen as BOTH. Either (1) the shape changed string→array and the string form is dead → a typed List<ExternalAuthConfiguration> (fields inputName, providerName, credentialSource) is right; or (2) it is still polymorphic → both List<...> and List<Object> throw on the string case and bare Object (or a custom adapter) is correct. getLogs_all_actions decides. Breaking-change note: the last released type is String, so this getter signature changes regardless when this lands — prefer shipping the most correct final type now rather than a second break later. No SDK consumers of the getter today.

zimeg

💾 Another comment on the properties being changed noted above. Will mark this LGTM but appreciate thoughts before merging!

zimeg · 2026-07-31T07:43:07Z

-        private List<Permission> permissions;
+        private Object permissions;
        private String sharedTo; // channel_workspaces_updated
-        private String reason;


👁️‍🗨️ note: I'm somewhat confident at changing this but find it matches the response logs adjacent:

https://github.com/slackapi/java-slack-sdk/pull/1592/changes#diff-568bead80fd2156c0258bba829bc7abb34441d59aa43241d603c242792b500abR238

feat: add webapi response changes as of 2026-05-19

24d0bdc

slackapi Bot requested a review from a team as a code owner May 19, 2026 05:07

slackapi Bot added project:slack-api-model project:slack-api-model semver:patch labels May 19, 2026

feat: add language property to Message model

dbbc096

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

zimeg and others added 3 commits May 18, 2026 23:16

Revert "fix: use org admin token with team_id for Grid admin tests"

0dafb70

This reverts commit c68228b.

Reapply "fix: use org admin token with team_id for Grid admin tests"

122c17e

This reverts commit 0dafb70.

zimeg marked this pull request as draft May 19, 2026 06:32

zimeg and others added 5 commits May 19, 2026 00:21

fix: remove duplicate pref_allow_slack_ai_changed field causing compi…

4d97010

…lation failure Duplicate field in WorkspaceOrOrg class prevented Lombok annotation processing, breaking all Builder class generation in slack-api-client. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

feat: add webapi response changes as of 2026-05-19

485af8f

feat: add webapi response changes as of 2026-05-22

49460fc

feat: add webapi response changes as of 2026-05-23

2b40739

WilliamBergamin approved these changes May 25, 2026

View reviewed changes

slackapi Bot and others added 14 commits May 28, 2026 00:41

feat: add webapi response changes as of 2026-05-28

087418e

chore: add latest logs and configuration

6735291

test: skip deprecated rtm test

cf33fcf

feat: add webapi response changes as of 2026-06-02

16af091

chore: ignore generated example compilations

af7fe5a

Merge branch 'api' of https://github.com/slackapi/java-slack-sdk into…

1ac131b

… api

test: expect unauthenticated requests from scim2 dummy tokens

108d84a

chore: json logs 2026-06-02

d2b40f4

feat: private channel access permissions

058313e

chore: audit log responses wip

794f818

fix: add salesforce_tenant_guid, affected_users to LogsResponse.Details

c69d0a0

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

fix: add provider_key to LogsResponse.Details

01134db

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

fix: add policy_name to LogsResponse.Details

4cb34e8

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

slackapi Bot and others added 23 commits July 21, 2026 00:47

feat: add webapi response changes as of 2026-07-21

56ab313

feat: add webapi response changes as of 2026-07-21

d35ea27

feat: add mcp list user channels action and audit details serverUrl

bd5d88b

feat: add webapi response changes as of 2026-07-21

c3413cb

feat: add webapi response changes as of 2026-07-22

6c23650

feat: add webapi response changes as of 2026-07-24

97032e6

feat: add webapi response changes as of 2026-07-27

2453d16

feat: add webapi response changes as of 2026-07-29

52f077d

feat: add app_mcp_server_auth_strategy_set audit action

fd9fb66

Merge remote-tracking branch 'origin/main' into api

9e12abe

feat: add newly detected ip range, allowlist, and mcp list record aud…

7a9c2dc

…it actions

test: assert getError is null before dereferencing getChannels in adm…

f3c9a9d

…in tests

feat: add webapi response changes as of 2026-07-29

73ca92e

feat: add webapi response changes as of 2026-07-29

6800281

feat: add webapi response changes as of 2026-07-29

97e60ee

feat: add webapi response changes as of 2026-07-29

3bb78ef

feat: add webapi response changes as of 2026-07-29

9390a67

feat: add webapi response changes as of 2026-07-30

8c91bd0

feat: add webapi response changes as of 2026-07-30

1ee56e8

feat: add webapi response changes as of 2026-07-31

6a269ac

feat: add mcp_slack_delete_list_record_tool_called audit action

fc59e19

Merge remote-tracking branch 'origin/main' into api

326bd6c

feat: add webapi response changes as of 2026-07-31

f918df7

zimeg reviewed Jul 31, 2026

View reviewed changes

zimeg marked this pull request as ready for review July 31, 2026 07:44

zimeg approved these changes Jul 31, 2026

View reviewed changes

feat: add webapi response changes as of 2026-08-01

22dab56

Uh oh!

Conversation

slackapi Bot commented May 19, 2026

Category

Requirements

Uh oh!

codecov Bot commented May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

codecov Bot commented May 19, 2026 •

edited

Loading