Skip to content

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

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

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

  • adminSlackbotSkillsList takes category, cursor, limit, creatorIds, publishStatus, autoEnabledStatus, sort, and sortDir.
  • creator_ids is sent comma-joined, like other ID arrays in this SDK. The response's skills carry auto_enabled.permission_type and usergroup_ids.
  • SlackbotSkillsTest parses a response captured from the live API with unknown properties failing the test, so every returned field is typed.

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 #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 👈 this pull request
  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
  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.list 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>
zimeg and others added 2 commits November 11, 2026 23:09
Parses a live admin.slackbot.skills.list response with unknown properties failing the test.

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 93.33333% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (zimeg-feat-slackbot-skills-library-remove@cdfdc08). 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 90.90% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@                             Coverage Diff                              @@
##             zimeg-feat-slackbot-skills-library-remove    #1691   +/-   ##
============================================================================
  Coverage                                             ?   73.86%           
  Complexity                                           ?     4715           
============================================================================
  Files                                                ?      487           
  Lines                                                ?    14749           
  Branches                                             ?     1531           
============================================================================
  Hits                                                 ?    10895           
  Misses                                               ?     2944           
  Partials                                             ?      910           
Flag Coverage Δ
jdk-14 73.86% <93.33%> (?)

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.

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