feat: add webapi response changes as of 2026-08-07 - #1629
Open
slackapi[bot] wants to merge 10 commits into
Open
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1629 +/- ##
============================================
- Coverage 73.34% 73.33% -0.01%
Complexity 4528 4528
============================================
Files 478 478
Lines 14314 14314
Branches 1491 1491
============================================
- Hits 10498 10497 -1
- Misses 2924 2926 +2
+ Partials 892 891 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
ActionsResponse.Actions: List<String> workObjectRecord SchemasResponse: WorkObjectRecord class (id, appId, objectId, objectType) Actions: WorkObjectRecord inner class with enterprise_search_email_attachment_downloaded Additive, no breaking changes; verified against the live audit API response. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
rtmConnect(token) requests full user info, adding a users.info call between rtm.connect and the websocket. That second HTTPS request may or may not reuse the pooled proxy tunnel, so the proxy occasionally sees a third CONNECT and the callCount assertion flakes between 2 and 3. The test does not use the connected user info, so request it without the extra call to keep the count deterministic at 2. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
zimeg
reviewed
Aug 8, 2026
Comment on lines
+201
to
+203
| // fullUserInfoRequired=false: skip the extra users.info call so the proxy sees exactly | ||
| // two connections (rtm.connect + websocket), otherwise a third tunnel can be opened | ||
| try (RTMClient rtm = slack.rtmConnect(rtmBotToken, false)) { // slack-msgs.com |
Member
There was a problem hiding this comment.
💡 note: Added to reduce flakes of scheduled tests! We now hope to focus on new properties with the test runner.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the web API client and model based on the production E2E test results.
Category
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.