Skip to content

feat: add admin.slackbot.skills.list method - #2837

Draft
zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-library-removefrom
zimeg-feat-slackbot-skills-admin-list
Draft

zimeg wants to merge 3 commits into
zimeg-feat-slackbot-skills-library-removefrom
zimeg-feat-slackbot-skills-admin-list

Conversation

@zimeg

@zimeg zimeg commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Summary

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

  • client.admin.slackbot.skills.list() takes optional category, cursor, limit, creator_ids, publish_status, auto_enabled_status, sort, and sort_dir, so it binds with bindApiCallWithOptionalArgument.
  • It needs a user token with admin.skills:read from an org-level install.
  • AdminSlackbotSkillsListResponse is generated with scripts/code_generator.rb from the json-logs/samples/api/admin.slackbot.skills.list.json sample recorded live in feat: add admin.slackbot.skills.list method java-slack-sdk#1691.
  • Adds type tests and a minor changeset.

This is part 7 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 👈 this pull request
  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. With an org admin's user token, call client.admin.slackbot.skills.list({ creator_ids: [userId] }) and confirm each skill has auto_enabled.
  2. Iterate client.paginate('admin.slackbot.skills.list', { limit: 1 }) and confirm it follows next_cursor across empty pages.

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: 290708e

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

Additional details and impacted files
@@                              Coverage Diff                              @@
##           zimeg-feat-slackbot-skills-library-remove    #2837      +/-   ##
=============================================================================
- Coverage                                      94.46%   94.34%   -0.13%     
=============================================================================
  Files                                             67       67              
  Lines                                          10729    10743      +14     
  Branches                                         680      680              
=============================================================================
  Hits                                           10135    10135              
- Misses                                           568      578      +10     
- Partials                                          26       30       +4     
Flag Coverage Δ
cli-hooks 94.21% <100.00%> (+<0.01%) ⬆️
cli-test 94.21% <100.00%> (+<0.01%) ⬆️
logger 94.21% <100.00%> (+<0.01%) ⬆️
oauth 94.21% <100.00%> (+<0.01%) ⬆️
socket-mode 94.21% <100.00%> (+<0.01%) ⬆️
web-api 94.21% <100.00%> (+<0.01%) ⬆️
webhook 94.21% <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
…eat-slackbot-skills-admin-list

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