Skip to content

feat: add slackbot.skills.library.remove method - #2836

Draft
zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-library-addfrom
zimeg-feat-slackbot-skills-library-remove
Draft

zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-library-addfrom
zimeg-feat-slackbot-skills-library-remove

Conversation

@zimeg

@zimeg zimeg commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Summary

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

  • client.slackbot.skills.library.remove({ skill_id }) turns a skill off in the caller's library.
  • SlackbotSkillsLibraryRemoveResponse is generated with scripts/code_generator.rb from the json-logs/samples/api/slackbot.skills.library.remove.json sample recorded live in feat: add slackbot.skills.library.remove method java-slack-sdk#1690.
  • Adds type tests and a minor changeset.

This is part 6 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 👈 this pull request
  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
  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. Call client.slackbot.skills.library.remove({ skill_id }) and confirm ok.

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: b9dd87e

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.46%. Comparing base (dcf9224) to head (b9dd87e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                            Coverage Diff                             @@
##           zimeg-feat-slackbot-skills-library-add    #2836      +/-   ##
==========================================================================
+ Coverage                                   94.15%   94.46%   +0.30%     
==========================================================================
  Files                                          67       67              
  Lines                                       10719    10729      +10     
  Branches                                      680      680              
==========================================================================
+ Hits                                        10093    10135      +42     
+ Misses                                        594      568      -26     
+ Partials                                       32       26       -6     
Flag Coverage Δ
cli-hooks 94.20% <100.00%> (+0.08%) ⬆️
cli-test 94.20% <100.00%> (+0.08%) ⬆️
logger 94.20% <100.00%> (+0.08%) ⬆️
oauth 94.20% <100.00%> (+0.08%) ⬆️
socket-mode 94.20% <100.00%> (+0.08%) ⬆️
web-api 94.20% <100.00%> (+0.08%) ⬆️
webhook 94.20% <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-library-remove

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…he 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