Skip to content

feat(methods): add admin.apps.mcp.servers.permissions.set method example - #126

Merged
zimeg merged 138 commits into
mainfrom
zimeg-feat-admin-apps-mcp-servers-examples-permissions-set
Nov 13, 2026
Merged

zimeg merged 138 commits into
mainfrom
zimeg-feat-admin-apps-mcp-servers-examples-permissions-set

Conversation

@zimeg

@zimeg zimeg commented Nov 13, 2026

Copy link
Copy Markdown
Member

Summary

This pull request adds a method example for admin.apps.mcp.servers.permissions.set in methods/src/main/java/admin/apps/mcp/servers/permissions/AdminAppsMcpServersPermissionsSet.java, and lists it in methods/README.md.

  • The arguments follow the order of the reference page.
  • Adds admin.apps:write to admin/apps/mcp/servers/permissions/manifest.json.

This is part 3 of 3 in a stack that adds the admin.apps.mcp.servers method examples one at a time. A Slackbot skill with MCP servers takes its server_id from admin.apps.mcp.servers.permissions.list. Each pull request builds on the one before it and merges on its own, bottom to top. The stack sits on top of #123, the last layer of the Slackbot skills examples.

  1. feat(methods): add admin.apps.mcp.servers.list method example #124 admin.apps.mcp.servers.list
  2. feat(methods): add admin.apps.mcp.servers.permissions.list method example #125 admin.apps.mcp.servers.permissions.list
  3. feat(methods): add admin.apps.mcp.servers.permissions.set method example #126 admin.apps.mcp.servers.permissions.set 👈 this pull request

Dependency: stays a draft

This depends on slackapi/java-slack-sdk#1698, which adds adminAppsMcpServersPermissionsSet to slack-api-client. mvn compile fails against the pinned 1.54.0 until that ships in a release, as expected. This PR stays a draft until then, and the slack-api-client pin gets bumped to that release in this stack.

Testing

With slackapi/java-slack-sdk#1698 installed locally (./scripts/install_local.sh) and the pin pointed at that snapshot:

  1. cd methods and export SLACK_TOKEN=... with the org-installed app's user token (admin.apps:read, admin.apps:write).
  2. mvn compile exec:java -Dexec.mainClass=admin.apps.mcp.servers.permissions.AdminAppsMcpServersPermissionsSet and check the response. Replace placeholder IDs like A0123ABCD with real ones from your organization to see ok.

Each example in this stack so far ran in order against an Enterprise Grid developer sandbox on that layer's SDK branch, with the placeholder IDs swapped for real ones, and returned ok. The permissions.set run was followed by setting the server back to its original permission.

🤖 Generated with Claude Code

zimeg and others added 26 commits November 11, 2026 22:42
Adds an example that calls slackbot.skills.create with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls slackbot.skills.info with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls slackbot.skills.update with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls slackbot.skills.list with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls slackbot.skills.library.add with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls slackbot.skills.library.remove with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls admin.slackbot.skills.list with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls admin.slackbot.skills.publish with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls admin.slackbot.skills.setAutoEnabled with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls admin.slackbot.skills.unpublish with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Adds an example that calls slackbot.skills.delete with the arguments from its
reference page, and lists it in the methods README.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…reate example

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…pdate example

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…imeg-feat-slackbot-skills-examples-library-remove
…o zimeg-feat-slackbot-skills-examples-admin-list
…meg-feat-slackbot-skills-examples-admin-publish
… zimeg-feat-slackbot-skills-examples-admin-set-auto-enabled
…led' into zimeg-feat-slackbot-skills-examples-admin-unpublish
…to zimeg-feat-slackbot-skills-examples-delete
Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…mple

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@zimeg zimeg added enhancement New feature or request java Pull requests that update java code labels Nov 13, 2026
@zimeg zimeg self-assigned this Nov 13, 2026
Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
zimeg and others added 9 commits November 13, 2026 11:25
…imeg-feat-admin-apps-mcp-servers-examples-permissions-list
…list' into zimeg-feat-admin-apps-mcp-servers-examples-permissions-set
…list' into zimeg-feat-admin-apps-mcp-servers-examples-permissions-set
…ssions.list

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…ssions.set

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…s.list example

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…imeg-feat-admin-apps-mcp-servers-examples-permissions-list
…list' into zimeg-feat-admin-apps-mcp-servers-examples-permissions-set

@mwbrooks mwbrooks left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Thanks @zimeg! Matching the docs example values keeps this sample and the reference page in step.

@zimeg
zimeg changed the base branch from zimeg-feat-admin-apps-mcp-servers-examples-permissions-list to main November 13, 2026 22:33
zimeg and others added 2 commits November 13, 2026 14:33
The refresh from main dropped this layer's scope addition.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@zimeg

zimeg commented Nov 13, 2026

Copy link
Copy Markdown
Member Author

🎉 Thanks for the review @mwbrooks, landing admin.apps.mcp.servers.permissions.set now! The last commit restores this layer's admin.apps:write scope, which a refresh had dropped. The tree matches the head you approved.

@zimeg
zimeg merged commit 96a43da into main Nov 13, 2026
4 checks passed
@zimeg
zimeg deleted the zimeg-feat-admin-apps-mcp-servers-examples-permissions-set branch November 13, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request java Pull requests that update java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants