@unblocklabs/slack-subagent-card 1.0.1 → 1.0.3

package/index.ts

@@ -156,8 +156,11 @@ export default definePluginEntry({
 
     const web = createSlackWebClient(token);
 
+    log.info(`slack-subagent-card: registering hooks (registrationMode=${pluginApi.registrationMode}, typeof api.on=${typeof pluginApi.on})`);
+
     pluginApi.on("subagent_spawned", async (event, ctx) => {
       try {
+        log.info(`slack-subagent-card: subagent_spawned fired — runId=${(event as SubagentSpawnedEvent).runId ?? ctx.runId} requesterSessionKey=${ctx.requesterSessionKey} threadRequested=${(event as SubagentSpawnedEvent).threadRequested} requester=${JSON.stringify((event as SubagentSpawnedEvent).requester)}`);
         await handleSpawned(pluginApi, web, shared, event as SubagentSpawnedEvent, ctx);
       } catch (error) {
         log.warn(`slack-subagent-card: subagent_spawned failed: ${stringifyError(error)}`);
@@ -166,6 +169,7 @@ export default definePluginEntry({
 
     pluginApi.on("subagent_ended", async (event, ctx) => {
       try {
+        log.info(`slack-subagent-card: subagent_ended fired — runId=${(event as SubagentEndedEvent).runId ?? ctx.runId} outcome=${(event as SubagentEndedEvent).outcome}`);
         await handleEnded(pluginApi, web, shared, event as SubagentEndedEvent, ctx);
       } catch (error) {
         log.warn(`slack-subagent-card: subagent_ended failed: ${stringifyError(error)}`);
@@ -174,6 +178,7 @@ export default definePluginEntry({
 
     pluginApi.on("subagent_delivery_target", async (event, ctx) => {
       try {
+        log.info(`slack-subagent-card: subagent_delivery_target fired — runId=${(event as SubagentDeliveryTargetEvent).childRunId ?? ctx.runId}, updating card early`);
         await handleDeliveryTarget(pluginApi, web, shared, event as SubagentDeliveryTargetEvent, ctx);
       } catch (error) {
         log.warn(`slack-subagent-card: subagent_delivery_target failed: ${stringifyError(error)}`);
@@ -244,7 +249,7 @@ async function handleSpawned(
 
   cleanupStaleRuns(shared, api.logger);
 
-  const target = resolveSlackThreadTarget(requesterSessionKey, event.requester);
+  const target = await resolveSlackThreadTarget(requesterSessionKey, event.requester, web, api.logger);
   if (!target) {
     api.logger.debug?.(
       `slack-subagent-card: no Slack thread target for runId=${runId} requesterSessionKey=${requesterSessionKey}`,
@@ -263,10 +268,10 @@ async function handleSpawned(
       web.chat.postMessage({
         channel: target.channelId,
         thread_ts: target.threadTs,
-        text: `${CARD_TEXT_PREFIX}${cardTitle}: Running`,
+        text: `${CARD_TEXT_PREFIX}${cardTitle}: SubAgent Running`,
         blocks: buildBlocks({
           label: cardTitle,
-          statusText: "⏳ Running",
+          statusText: "⏳ SubAgent Running",
           taskId: runId,
         }) as any,
       }),
@@ -574,21 +579,36 @@ function normalizeOutcome(value: unknown): Outcome {
   return value === "error" || value === "timeout" || value === "killed" ? value : "ok";
 }
 
-function resolveSlackThreadTarget(
+async function resolveSlackThreadTarget(
   requesterSessionKey: string,
-  requester?: SlackRequester,
-): SlackThreadTarget | null {
+  requester: SlackRequester | undefined,
+  web: WebClient,
+  log: Logger,
+): Promise<SlackThreadTarget | null> {
   const fromSession = parseSlackThreadSessionKey(requesterSessionKey);
-  if (fromSession) return fromSession;
+  if (fromSession) {
+    // For DM sessions the captured ID is a user ID (U...), not a DM channel (D...).
+    // Slack's chat.postMessage needs the D-prefixed channel ID for thread replies.
+    if (/^U/i.test(fromSession.channelId)) {
+      const resolved = await resolveUserToDmChannel(web, fromSession.channelId, log);
+      if (!resolved) return null;
+      return { channelId: resolved, threadTs: fromSession.threadTs };
+    }
+    return fromSession;
+  }
 
   const rawTarget = asNonEmptyString(requester?.to);
   const threadTs = asNonEmptyString(requester?.threadId);
   if (!rawTarget || !threadTs) return null;
 
-  return {
-    channelId: normalizeSlackChannelId(stripSlackTargetPrefix(rawTarget)),
-    threadTs,
-  };
+  let channelId = normalizeSlackChannelId(stripSlackTargetPrefix(rawTarget));
+  if (/^U/i.test(channelId)) {
+    const resolved = await resolveUserToDmChannel(web, channelId, log);
+    if (!resolved) return null;
+    channelId = resolved;
+  }
+
+  return { channelId, threadTs };
 }
 
 function parseSlackThreadSessionKey(sessionKey: string): SlackThreadTarget | null {
@@ -734,7 +754,38 @@ function escapeMrkdwn(value: string): string {
 }
 
 function normalizeSlackChannelId(channelId: string): string {
-  return /^[cgd]/i.test(channelId) ? channelId.toUpperCase() : channelId;
+  return /^[cgdu]/i.test(channelId) ? channelId.toUpperCase() : channelId;
+}
+
+/** Cache of userId → DM channelId to avoid repeated conversations.open calls */
+const dmChannelCache = new Map<string, string>();
+
+async function resolveUserToDmChannel(
+  web: WebClient,
+  userId: string,
+  log: Logger,
+): Promise<string | undefined> {
+  const normalized = userId.toUpperCase();
+  const cached = dmChannelCache.get(normalized);
+  if (cached) return cached;
+
+  try {
+    const result = await withSlackRetry(
+      () => web.conversations.open({ users: normalized, return_im: true }),
+      log,
+    );
+    const channelId = asNonEmptyString((result as any)?.channel?.id);
+    if (channelId) {
+      dmChannelCache.set(normalized, channelId);
+      log.info(`slack-subagent-card: resolved DM channel for ${normalized} → ${channelId}`);
+      return channelId;
+    }
+    log.warn(`slack-subagent-card: conversations.open returned no channel for ${normalized}`);
+    return undefined;
+  } catch (error) {
+    log.warn(`slack-subagent-card: failed to resolve DM channel for ${normalized}: ${stringifyError(error)}`);
+    return undefined;
+  }
 }
 
 function cleanupStaleRuns(shared: SharedState, log: Logger): void {

package/openclaw.plugin.json

@@ -1,14 +1,13 @@
 {
   "id": "slack-subagent-card",
   "name": "Slack Subagent Card",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Posts and updates a Slack Block Kit status card for thread-bound sub-agents, nudges quiet parent threads after completion, wakes parent sessions, and writes completion markers.",
-  "entry": "index.ts",
   "configSchema": {
     "type": "object",
     "properties": {
       "nudgeDelaySec": {
-        "type": "number",
+        "type": "integer",
         "minimum": 10,
         "maximum": 300,
         "default": 30,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unblocklabs/slack-subagent-card",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "OpenClaw plugin: Slack Block Kit status cards for sub-agents, nudge timer, parent session wake, and completion markers.",
   "type": "module",
   "openclaw": {
@@ -20,7 +20,7 @@
   "author": "Unblock Labs",
   "repository": {
     "type": "git",
-    "url": "https://github.com/unblocklabs-ai/openclaw-slack-subagent-card"
+    "url": "https://github.com/bill492/slack-subagent-card"
   },
   "engines": {
     "node": ">=22"