Add missing Web API methods (Java SDK / changelog parity)#64
Open
StephenTangCook wants to merge 1 commit into
Open
Add missing Web API methods (Java SDK / changelog parity)#64StephenTangCook wants to merge 1 commit into
StephenTangCook wants to merge 1 commit into
Conversation
Ports 21 Web API methods that exist in the official Java SDK and Slack
changelog but were missing from this client:
- admin.analytics.getFile, admin.apps.config.{lookup,set}
(request types already existed in request.ts but were never wired)
- assistant.search.{context,info} (Real-time Search API for AI apps)
- functions.distributions.permissions.{set,list,add,remove},
functions.workflows.steps.list (custom-function distribution)
- workflows.featured.{add,set,remove,list}
- calls.{add,end,info,update}, calls.participants.{add,remove}
- apps.user.connection.update
Each method gets a request interface, a typed response (named types for
payload-bearing methods, generic SlackAPIResponse for ok-only ones), and
wiring in api-client.ts. Param shapes taken from the live Slack docs.
src_deno regenerated via scripts/generate-deno-source.sh (also picks up
pre-existing underline/format drift in src_deno).
Verified: tsc --noEmit, deno fmt/lint/check, and vitest (35 tests) all pass.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
Summary
Ports 21 Web API methods that exist in the official Java SDK and the Slack changelog but were missing here. Found by diffing this client's surface against the Java SDK
MethodsClientand the changelog (slack-edge needs nothing new from this client; Block Kit & events were already at parity).admin.analytics.getFile,admin.apps.config.lookup,admin.apps.config.setassistant.search.context,assistant.search.infofunctions.distributions.permissions.{set,list,add,remove},functions.workflows.steps.listworkflows.featured.{add,set,remove,list}calls.{add,end,info,update},calls.participants.{add,remove}apps.user.connection.updateThe three
admin.*methods already had request interfaces inrequest.tsthat were never wired intoapi-client.ts(dead types) — this finishes them.Approach
Each method gets a request interface, a typed response (named types in
custom-response/for payload-bearing methods, genericSlackAPIResponsefor ok-only ones), and a#bindApiCallinapi-client.ts. Parameter shapes were taken from the live Slack method docs, not guessed.src_denowas regenerated viascripts/generate-deno-source.sh(which also picked up pre-existingunderline/formatting drift in the Deno mirror).Method count: 259 → 280.
Verification
tsc --noEmit— cleandeno fmt/deno lint/deno check— cleanvitest— 35/35 passing🤖 Generated with Claude Code