Skip to content

feat: add slackbot.skills.delete method - #2841

Draft
zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-admin-unpublishfrom
zimeg-feat-slackbot-skills-delete
Draft

zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-admin-unpublishfrom
zimeg-feat-slackbot-skills-delete

Conversation

@zimeg

@zimeg zimeg commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Summary

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

  • client.slackbot.skills.delete({ skill_id }) deletes a skill and its canvas.
  • SlackbotSkillsDeleteResponse is generated with scripts/code_generator.rb from the json-logs/samples/api/slackbot.skills.delete.json sample recorded live in feat: add slackbot.skills.delete method java-slack-sdk#1695.
  • Adds type tests and a minor changeset.

This is part 11 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
  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 👈 this pull request

Testing

  1. Call client.slackbot.skills.delete({ skill_id }) on a skill you created, then confirm slackbot.skills.info returns skill_not_found.

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

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.35%. Comparing base (1acbc19) to head (e13a7d7).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                              Coverage Diff                               @@
##           zimeg-feat-slackbot-skills-admin-unpublish    #2841      +/-   ##
==============================================================================
- Coverage                                       94.42%   94.35%   -0.07%     
==============================================================================
  Files                                              67       67              
  Lines                                           10773    10780       +7     
  Branches                                          677      677              
==============================================================================
  Hits                                            10172    10172              
- Misses                                            574      580       +6     
- Partials                                           27       28       +1     
Flag Coverage Δ
cli-hooks 94.23% <100.00%> (+<0.01%) ⬆️
cli-test 94.23% <100.00%> (+<0.01%) ⬆️
logger 94.23% <100.00%> (+<0.01%) ⬆️
oauth 94.23% <100.00%> (+<0.01%) ⬆️
socket-mode 94.23% <100.00%> (+<0.01%) ⬆️
web-api 94.23% <100.00%> (+<0.01%) ⬆️
webhook 94.23% <100.00%> (+<0.01%) ⬆️

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
…feat-slackbot-skills-delete

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