fix: bundle @slack SDK into plugin to avoid CJS/ESM interop issues

Bun's runtime cannot resolve named exports from CJS modules that use
Object.defineProperty getters (e.g. @slack/web-api's WebClient). Static
imports, namespace imports, createRequire, and dynamic import() all
fail because Bun strips getter-defined properties from the ESM namespace
object.

The only reliable solution is to avoid runtime CJS/ESM interop entirely
by bundling the @slack/* packages into the plugin with tsup. This
eliminates the dependency on Bun correctly handling CJS module namespaces.

Signed-off-by: xuezhaojun <xuezhaokeepgoing@gmail.com>

Author: xuezhaojun

opencode-slack-plugin/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@kubeopencode/opencode-slack-plugin",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "type": "module",
   "license": "MIT",
   "description": "OpenCode plugin that connects to Slack via Socket Mode — zero port exposure, runs inside your OpenCode process",
@@ -34,12 +34,11 @@
     "typecheck": "tsc --noEmit",
     "dev": "tsup src/index.ts --format esm --dts --watch"
   },
-  "dependencies": {
-    "@slack/socket-mode": "^2.0.5",
-    "@slack/web-api": "^7.13.0"
-  },
+  "dependencies": {},
   "devDependencies": {
     "@opencode-ai/plugin": "^1.2.15",
+    "@slack/socket-mode": "^2.0.5",
+    "@slack/web-api": "^7.13.0",
     "@types/node": "^22.0.0",
     "tsup": "^8.5.0",
     "typescript": "^5.9.0"

opencode-slack-plugin/package.json

@@ -1,16 +1,8 @@
 // Copyright Contributors to the KubeOpenCode project
 import type { PluginModule, PluginInput, Hooks } from "@opencode-ai/plugin"
+import { WebClient } from "@slack/web-api"
+import { SocketModeClient } from "@slack/socket-mode"
 import * as os from "os"
-// CJS/ESM interop: @slack/web-api and @slack/socket-mode are CJS modules that
-// use Object.defineProperty for their exports. Bun's bundler cannot resolve
-// named imports (produces "X is not a constructor"), namespace imports lose
-// getter-defined properties in the bundled namespace, and require() is blocked
-// by Bun for async CJS modules. Dynamic import() forces Bun to handle the
-// interop at runtime, preserving all exports including constructors.
-const slackWebApi = await import("@slack/web-api") as typeof import("@slack/web-api")
-const slackSocketMode = await import("@slack/socket-mode") as typeof import("@slack/socket-mode")
-const { WebClient } = slackWebApi
-const { SocketModeClient } = slackSocketMode
 
 // ---------------------------------------------------------------------------
 // Types
@@ -219,7 +211,6 @@ const slack = async (input: PluginInput): Promise<Hooks> => {
   const appToken = process.env.SLACK_APP_TOKEN
 
   if (!botToken || !appToken) {
-    // Not configured — silently skip.
     return {}
   }