Cherry-pick 8746362f5e: move Slack channel code to extensions/slack/src/#2041
Merged
alexey-pelykh merged 1 commit intomainfrom Mar 29, 2026
Merged
Conversation
…enclaw#45621) Move all Slack channel implementation files from src/slack/ to extensions/slack/src/ and replace originals with shim re-exports. This follows the extension migration pattern for channel plugins. - Copy all .ts files to extensions/slack/src/ (preserving directory structure: monitor/, http/, monitor/events/, monitor/message-handler/) - Transform import paths: external src/ imports use relative paths back to src/, internal slack imports stay relative within extension - Replace all src/slack/ files with shim re-exports pointing to the extension copies - Update tsconfig.plugin-sdk.dts.json rootDir from "src" to "." so the DTS build can follow shim chains into extensions/ - Update write-plugin-sdk-entry-dts.ts re-export path accordingly - Preserve extensions/slack/index.ts, package.json, openclaw.plugin.json, src/channel.ts, src/runtime.ts, src/channel.test.ts (untouched) (cherry picked from commit 8746362)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry-pick from upstream
Upstream commit:
8746362f5eAuthor: openclaw team
Tier: AUTO-PICK
Closes: #2028
What this does
Moves the Slack adapter implementation from
src/slack/toextensions/slack/src/, leaving re-export shims at the original paths for backward compatibility. This aligns with the upstream extensions-move pattern already applied to other channel adapters.Conflict resolution
src/slack/) — resolved by accepting upstream shimsOpenClawConfig→RemoteClawConfig,openclaw:→remoteclaw:action ID prefixes, slash command defaults, test fixturesmainversions (adjusted import paths):allow-list.ts,slash.ts,sent-thread-cache.test.ts,interactive-replies.test.ts,dispatch.ts,provider.ts,token.tsslash-skill-commands.runtime.ts(gutted feature)manifest.ts/manifest.test.tstoextensions/slack/src/with shimsOPENCLAW_SLACK_PAIRING_ALLOWFROM_CACHE_TTL_MSenv var (migration compat)Exit criteria verification
pnpm checkpasses (typecheck + lint + all custom lints)openclawreferences in moved files (except env var migration compat and upstream test fixture URLs)