Skip to content

Cherry-pick 8746362f5e: move Slack channel code to extensions/slack/src/#2041

Merged
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/8746362f5e-slack-extensions-move
Mar 29, 2026
Merged

Cherry-pick 8746362f5e: move Slack channel code to extensions/slack/src/#2041
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/8746362f5e-slack-extensions-move

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick from upstream

Upstream commit: 8746362f5e
Author: openclaw team
Tier: AUTO-PICK
Closes: #2028

refactor(slack): move Slack channel code to extensions/slack/src/ (openclaw#45621)

What this does

Moves the Slack adapter implementation from src/slack/ to extensions/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

  • 50 conflicted files (all in src/slack/) — resolved by accepting upstream shims
  • Rebranded all OpenClawConfigRemoteClawConfig, openclaw:remoteclaw: action ID prefixes, slash command defaults, test fixtures
  • Replaced 7 diverged files with fork's main versions (adjusted import paths): allow-list.ts, slash.ts, sent-thread-cache.test.ts, interactive-replies.test.ts, dispatch.ts, provider.ts, token.ts
  • Deleted slash-skill-commands.runtime.ts (gutted feature)
  • Moved fork-only manifest.ts/manifest.test.ts to extensions/slack/src/ with shims
  • Kept OPENCLAW_SLACK_PAIRING_ALLOWFROM_CACHE_TTL_MS env var (migration compat)

Exit criteria verification

  • pnpm check passes (typecheck + lint + all custom lints)
  • No openclaw references in moved files (except env var migration compat and upstream test fixture URLs)

…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)
@alexey-pelykh alexey-pelykh merged commit f8e6f80 into main Mar 29, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/8746362f5e-slack-extensions-move branch March 29, 2026 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants