Skip to content

feat: add admin.slackbot.skills.publish method - #2838

Draft
zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-admin-listfrom
zimeg-feat-slackbot-skills-admin-publish
Draft

zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-admin-listfrom
zimeg-feat-slackbot-skills-admin-publish

Conversation

@zimeg

@zimeg zimeg commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Summary

This pull request adds the admin.slackbot.skills.publish method to @slack/web-api.

  • client.admin.slackbot.skills.publish({ skill_ids }) publishes skills to the org catalog. failed_for_some_skills responses include failed_skills.
  • AdminSlackbotSkillsPublishResponse is generated with scripts/code_generator.rb from the json-logs/samples/api/admin.slackbot.skills.publish.json sample recorded live in feat: add admin.slackbot.skills.publish method java-slack-sdk#1692.
  • Adds type tests and a minor changeset.

This is part 8 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 #2830.

  1. feat: add slackbot.skills.create method #2831 slackbot.skills.create
  2. feat: add slackbot.skills.info method #2832 slackbot.skills.info
  3. feat: add slackbot.skills.update method #2833 slackbot.skills.update
  4. feat: add slackbot.skills.list method #2834 slackbot.skills.list
  5. feat: add slackbot.skills.library.add method #2835 slackbot.skills.library.add
  6. feat: add slackbot.skills.library.remove method #2836 slackbot.skills.library.remove
  7. feat: add admin.slackbot.skills.list method #2837 admin.slackbot.skills.list
  8. feat: add admin.slackbot.skills.publish method #2838 admin.slackbot.skills.publish 👈 this pull request
  9. feat: add admin.slackbot.skills.setAutoEnabled method #2839 admin.slackbot.skills.setAutoEnabled
  10. feat: add admin.slackbot.skills.unpublish method #2840 admin.slackbot.skills.unpublish
  11. feat: add slackbot.skills.delete method #2841 slackbot.skills.delete

Testing

  1. With an admin.skills:write user token, call client.admin.slackbot.skills.publish({ skill_ids: [skillId] }) and confirm ok.
  2. Add an unknown ID and confirm failed_for_some_skills with that ID in failed_skills.

Each layer of this stack was checked against an Enterprise developer sandbox by running every method added so far, in order, with that layer's WebClient.

Requirements

🤖 Generated with Claude Code

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@zimeg zimeg added this to the web-api@next milestone Nov 12, 2026
@zimeg zimeg added semver:minor enhancement M-T: A feature request for new functionality area:typescript issues that specifically impact using the package from typescript projects pkg:web-api applies to `@slack/web-api` labels Nov 12, 2026
@zimeg zimeg self-assigned this Nov 12, 2026
@changeset-bot

changeset-bot Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d7aedc8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@slack/web-api Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov

codecov Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.28%. Comparing base (290708e) to head (d7aedc8).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                            Coverage Diff                            @@
##           zimeg-feat-slackbot-skills-admin-list    #2838      +/-   ##
=========================================================================
- Coverage                                  94.34%   94.28%   -0.06%     
=========================================================================
  Files                                         67       67              
  Lines                                      10743    10753      +10     
  Branches                                     680      680              
=========================================================================
+ Hits                                       10135    10139       +4     
- Misses                                       578      586       +8     
+ Partials                                      30       28       -2     
Flag Coverage Δ
cli-hooks 94.13% <100.00%> (-0.08%) ⬇️
cli-test 94.13% <100.00%> (-0.08%) ⬇️
logger 94.13% <100.00%> (-0.08%) ⬇️
oauth 94.13% <100.00%> (-0.08%) ⬇️
socket-mode 94.13% <100.00%> (-0.08%) ⬇️
web-api 94.13% <100.00%> (-0.08%) ⬇️
webhook 94.13% <100.00%> (-0.08%) ⬇️

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

zimeg and others added 2 commits November 11, 2026 23:00
…slackbot-skills-admin-publish

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…e recorded java-slack-sdk sample

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

area:typescript issues that specifically impact using the package from typescript projects enhancement M-T: A feature request for new functionality pkg:web-api applies to `@slack/web-api` semver:minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant