Skip to content

feat(web-api): add code_channels app manifest feature - #2776

Open
zimeg wants to merge 3 commits into
mainfrom
clack/manifest-code-channels
Open

zimeg wants to merge 3 commits into
mainfrom
clack/manifest-code-channels

Conversation

@zimeg

@zimeg zimeg commented Oct 14, 2026

Copy link
Copy Markdown
Member

Summary

This pull request adds the code_channels app manifest feature to the hand-maintained manifest types in @slack/web-api (used by the apps.manifest.* methods), mirroring slackapi/manifest-schema and the Slack CLI (slackapi/slack-cli#641).

  • Adds features.code_channels with the optional enabled and slash_command_url fields. Descriptions are taken from the manifest schema.
  • Adds the code_channels:manage scope to the bot scopes, which the agents.conversations.* methods require.
  • Extends the apps.test-d.ts tsd coverage with a happy path for the new feature and scope, and a sad path for a non-boolean enabled.

Reference: Enabling code channels.

Requirements

🤖 Generated with Claude Code

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@zimeg zimeg added semver:minor enhancement M-T: A feature request for new functionality pkg:web-api applies to `@slack/web-api` labels Oct 14, 2026
@changeset-bot

changeset-bot Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 116086d

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

@zimeg zimeg self-assigned this Oct 14, 2026
@codecov

codecov Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.33%. Comparing base (8730b5e) to head (116086d).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2776      +/-   ##
==========================================
- Coverage   89.34%   89.33%   -0.01%     
==========================================
  Files          65       65              
  Lines       10548    10548              
  Branches      480      480              
==========================================
- Hits         9424     9423       -1     
- Misses       1092     1093       +1     
  Partials       32       32              
Flag Coverage Δ
cli-hooks 89.28% <ø> (ø)
cli-test 89.28% <ø> (ø)
logger 89.28% <ø> (ø)
oauth 89.28% <ø> (ø)
socket-mode 89.28% <ø> (ø)
web-api 89.28% <ø> (ø)
webhook 89.28% <ø> (ø)

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 zimeg left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

📝 Comments of nuance before requesting review.

Comment thread packages/web-api/src/types/request/manifest.ts Outdated
Comment thread packages/web-api/src/types/request/manifest.ts Outdated
…ence

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@zimeg zimeg added this to the web-api@next milestone Oct 20, 2026
@zimeg
zimeg marked this pull request as ready for review October 20, 2026 23:35
@zimeg
zimeg requested a review from a team as a code owner October 20, 2026 23:36

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