Skip to content

feat: add admin.slackbot.skills.setAutoEnabled method - #1693

Draft
zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-admin-publishfrom
zimeg-feat-slackbot-skills-admin-set-auto-enabled
Draft

zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-admin-publishfrom
zimeg-feat-slackbot-skills-admin-set-auto-enabled

Conversation

@zimeg

@zimeg zimeg commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

This pull request adds the admin.slackbot.skills.setAutoEnabled method to the sync and async clients.

  • adminSlackbotSkillsSetAutoEnabled takes skillId and permissionType (required), and usergroupIds.
  • usergroup_ids is sent comma-joined. The response includes invalid_usergroup_ids for usergroup_not_found, which the docs name in the errors table but don't show in an example.
  • SlackbotSkillsTest parses a response captured from the live API with unknown properties failing the test, so every returned field is typed.

This is part 9 of 11 in a stack that adds the Slackbot skills methods one at a time, following a skill from creation to deletion. Each pull request builds on the one before it and merges on its own, bottom to top. It replaces #1683.

  1. feat: add slackbot.skills.create method #1685 slackbot.skills.create
  2. feat: add slackbot.skills.info method #1686 slackbot.skills.info
  3. feat: add slackbot.skills.update method #1687 slackbot.skills.update
  4. feat: add slackbot.skills.list method #1688 slackbot.skills.list
  5. feat: add slackbot.skills.library.add method #1689 slackbot.skills.library.add
  6. feat: add slackbot.skills.library.remove method #1690 slackbot.skills.library.remove
  7. feat: add admin.slackbot.skills.list method #1691 admin.slackbot.skills.list
  8. feat: add admin.slackbot.skills.publish method #1692 admin.slackbot.skills.publish
  9. feat: add admin.slackbot.skills.setAutoEnabled method #1693 admin.slackbot.skills.setAutoEnabled 👈 this pull request
  10. feat: add admin.slackbot.skills.unpublish method #1694 admin.slackbot.skills.unpublish
  11. feat: add slackbot.skills.delete method #1695 slackbot.skills.delete

Testing

Run the remote tests for this layer:

  • test_with_remote_apis.methods.slackbot_skills_Test with SLACK_SDK_TEST_BOT_TOKEN (canvases:write, skills:read, skills:write)
  • test_with_remote_apis.methods_admin_api.AdminApi_slackbot_skills_Test with SLACK_SDK_TEST_GRID_ORG_ADMIN_USER_TOKEN (admin.skills:read, admin.skills:write, skills:write)

Each layer of this stack was checked against an Enterprise Grid developer sandbox by running every method added so far, in order. Test skills are deleted at the end.

Category (place an x in each of the [ ])

  • bolt (Bolt for Java)
  • bolt-{sub modules} (Bolt for Java - optional modules)
  • slack-api-client (Slack API Clients)
  • slack-api-model (Slack API Data Models)
  • slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
  • slack-app-backend (The primitive layer of Bolt for Java)

Requirements

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

🤖 Generated with Claude Code

Adds the admin.slackbot.skills.setAutoEnabled Web API method to the sync and async clients, with its
request and response types, mock fixture, rate limit tier, and tests.

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

codecov Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (zimeg-feat-slackbot-skills-admin-publish@1751318). Learn more about missing BASE report.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...java/com/slack/api/methods/RequestFormBuilder.java 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@                             Coverage Diff                             @@
##             zimeg-feat-slackbot-skills-admin-publish    #1693   +/-   ##
===========================================================================
  Coverage                                            ?   73.88%           
  Complexity                                          ?     4725           
===========================================================================
  Files                                               ?      487           
  Lines                                               ?    14767           
  Branches                                            ?     1533           
===========================================================================
  Hits                                                ?    10911           
  Misses                                              ?     2944           
  Partials                                            ?      912           
Flag Coverage Δ
jdk-14 73.88% <90.00%> (?)

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.

zimeg and others added 2 commits November 11, 2026 23:09
…on test

Parses a live admin.slackbot.skills.setAutoEnabled response with unknown properties failing the test.

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

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality project:slack-api-client project:slack-api-client semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant