@shenhh/popo 0.1.19 → 0.1.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shenhh/popo",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "type": "module",
   "description": "OpenClaw POPO channel plugin",
   "license": "MIT",

package/skills/popo-msg/SKILL.md

@@ -237,16 +237,6 @@ Common errors:
 
 Recall (retract) a sent message within the allowed time window.
 
-```json
-{
-  "action": "recall",
-  "msgId": "794076f2-68b1-4d0e-94eb-c15946580b56",
-  "target": "user@corp.netease.com"
-}
-```
-
-Or using POPO-specific parameters:
-
 ```json
 {
   "action": "recall",
@@ -258,11 +248,8 @@ Or using POPO-specific parameters:
 
 **Parameters:**
 - `msgId` (required): Message ID to recall
-- `target` (required*): Target user email or group ID (OpenClaw standard)
-- `sessionId` (required*): Session ID (user email for P2P, group ID for group) - alias for target
-- `sessionType` (optional): `1` for P2P, `3` for group (auto-detected from target if not provided)
-
-\* Either `target` or `sessionId` must be provided.
+- `sessionId` (required): Session ID (user email for P2P, group ID for group)
+- `sessionType` (required): `1` for P2P, `3` for group
 
 **Note:** Messages can only be recalled within a limited time (typically 2 minutes).
 
@@ -270,20 +257,6 @@ Or using POPO-specific parameters:
 
 Query read/unread status for group messages.
 
-```json
-{
-  "action": "read-ack",
-  "msgId": "1000016836-0000000004",
-  "target": "group:1234567",
-  "type": 1,
-  "st": 1672221224000,
-  "page": 1,
-  "size": 30
-}
-```
-
-Or using POPO-specific parameters:
-
 ```json
 {
   "action": "read-ack",
@@ -298,15 +271,12 @@ Or using POPO-specific parameters:
 
 **Parameters:**
 - `msgId` (required): Message ID
-- `target` (optional): Group ID for context (e.g., `group:1234567` or `1234567`)
-- `sessionType` (optional): Must be `3` (group), default is 3
-- `type` (optional): `1` for read list, `2` for unread list, default 1
-- `st` (optional): Query end timestamp (milliseconds), default is current time
+- `sessionType` (required): Must be `3` (group)
+- `type` (required): `1` for read list, `2` for unread list
+- `st` (required): Query end timestamp (milliseconds)
 - `page` (optional): Page number, default 1
 - `size` (optional): Page size, default 30
 
-**Note:** Read-ack only works for group messages, not P2P.
-
 **Response:**
 ```json
 {

package/src/channel.ts

@@ -60,6 +60,7 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
   agentPrompt: {
     messageToolHints: () => [
       "- POPO targeting: omit `target` to reply to the current conversation (auto-inferred). Explicit targets: `user:email@example.com` or `group:groupId`.",
+      "- POPO actions: send, card, unsend (target=to), addParticipant (target=tid), channel-info/channel-edit/channel-delete (channelId=tid), channel-create (no target), member-info (tid param). Custom actions without target: read-ack (msgId), download-file (fileId), update-team-mgmt, view-scope, modify-view-scope, publish-robot, update-instruction-options, configure-card-callback.",
     ],
   },
   groups: {
@@ -171,18 +172,22 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
       const enabled = cfg.channels?.popo?.enabled !== false;
       if (!enabled) return [];
       // Return all supported actions
+      // Use built-in action names to work with OpenClaw's target mode system:
+      // - unsend (recall), channel-create (create-team), addParticipant (invite-team)
+      // - channel-delete (drop-team), member-info (team-members), channel-info (team-info)
+      // - channel-edit (update-team-info)
       return [
         "send",
         "card",
-        "recall",
+        "unsend",
+        "channel-create",
+        "addParticipant",
+        "channel-delete",
+        "member-info",
+        "channel-info",
+        "channel-edit",
         "read-ack",
         "download-file",
-        "create-team",
-        "invite-team",
-        "drop-team",
-        "team-members",
-        "team-info",
-        "update-team-info",
         "update-team-mgmt",
         "view-scope",
         "modify-view-scope",
@@ -310,17 +315,22 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         };
       }
 
-      // Handle recall action
-      if (action === "recall") {
-        const msgId = typeof params.msgId === "string" ? params.msgId : typeof params.messageId === "string" ? params.messageId : "";
-        // Support both target (OpenClaw standard) and sessionId (POPO-specific)
-        const target = typeof params.target === "string" ? params.target : "";
-        const sessionId = typeof params.sessionId === "string" ? params.sessionId : target;
-        const sessionType = typeof params.sessionType === "number" ? params.sessionType : (target.includes("@") ? 1 : 3);
+      // Handle unsend action (recall message) - uses built-in "unsend" which has proper target mode
+      // Accepts: target (to), messageId, sessionId, sessionType
+      if (action === "unsend") {
+        const msgId = typeof params.messageId === "string" ? params.messageId : typeof params.msgId === "string" ? params.msgId : "";
+        // For unsend, sessionId can come from "to" (target) or explicit sessionId
+        const sessionId =
+          typeof params.sessionId === "string"
+            ? params.sessionId
+            : typeof params.to === "string"
+              ? params.to.trim()
+              : "";
+        const sessionType = typeof params.sessionType === "number" ? params.sessionType : 1;
         if (!msgId || !sessionId) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Recall requires msgId and target (or sessionId)." }],
+            content: [{ type: "text", text: "Unsend requires messageId and target (or sessionId)." }],
           };
         }
         const result = await recallMessagePopo({ cfg, msgId, sessionId, sessionType: sessionType as 1 | 3 });
@@ -341,8 +351,6 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
       // Handle read-ack action
       if (action === "read-ack") {
         const msgId = typeof params.msgId === "string" ? params.msgId : "";
-        // Support target (OpenClaw standard) for group identification
-        const target = typeof params.target === "string" ? params.target : "";
         const type = typeof params.type === "number" ? params.type : 1;
         const st = typeof params.st === "number" ? params.st : Date.now();
         const page = typeof params.page === "number" ? params.page : 1;
@@ -353,13 +361,6 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
             content: [{ type: "text", text: "Read-ack requires msgId." }],
           };
         }
-        // If target is provided, validate it's a group (read-ack only works for groups)
-        if (target && target.includes("@")) {
-          return {
-            isError: true,
-            content: [{ type: "text", text: "Read-ack only works for group messages, not P2P." }],
-          };
-        }
         const result = await getMessageReadAckPopo({
           cfg,
           msgId,
@@ -413,8 +414,9 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
 
       // ==================== Team Management Actions ====================
 
-      // Handle create-team action
-      if (action === "create-team") {
+      // Handle channel-create action (create team)
+      // Built-in action with targetMode: none
+      if (action === "channel-create") {
         const uid = typeof params.uid === "string" ? params.uid : "";
         const name = typeof params.name === "string" ? params.name : undefined;
         const uidList = Array.isArray(params.uidList) ? params.uidList : [];
@@ -422,7 +424,7 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         if (!uid || uidList.length === 0) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Create-team requires uid (owner) and uidList (members)." }],
+            content: [{ type: "text", text: "channel-create requires uid (owner) and uidList (members)." }],
           };
         }
         const result = await createTeam({ cfg, uid, name, uidList, photoUrl });
@@ -443,16 +445,22 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         };
       }
 
-      // Handle invite-team action
-      if (action === "invite-team") {
-        const tid = typeof params.tid === "string" ? params.tid : "";
-        const inviteList = Array.isArray(params.inviteList) ? params.inviteList : [];
+      // Handle addParticipant action (invite to team)
+      // Built-in action with targetMode: to (tid comes from target/to)
+      if (action === "addParticipant") {
+        const tid =
+          typeof params.tid === "string"
+            ? params.tid
+            : typeof params.to === "string"
+              ? params.to.trim()
+              : "";
+        const inviteList = Array.isArray(params.inviteList) ? params.inviteList : Array.isArray(params.participant) ? params.participant : [];
         const type = typeof params.type === "string" ? (params.type as "1" | "2") : "1";
         const text = typeof params.text === "string" ? params.text : undefined;
         if (!tid || inviteList.length === 0) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Invite-team requires tid and inviteList." }],
+            content: [{ type: "text", text: "addParticipant requires target (tid) and inviteList/participant." }],
           };
         }
         const result = await inviteToTeam({ cfg, tid, inviteList, type, text });
@@ -472,13 +480,19 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         };
       }
 
-      // Handle drop-team action
-      if (action === "drop-team") {
-        const tid = typeof params.tid === "string" ? params.tid : "";
+      // Handle channel-delete action (drop/disband team)
+      // Built-in action with targetMode: channelId
+      if (action === "channel-delete") {
+        const tid =
+          typeof params.tid === "string"
+            ? params.tid
+            : typeof params.channelId === "string"
+              ? params.channelId.trim()
+              : "";
         if (!tid) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Drop-team requires tid (team ID)." }],
+            content: [{ type: "text", text: "channel-delete requires channelId (team ID)." }],
           };
         }
         const result = await dropTeam({ cfg, tid });
@@ -496,15 +510,16 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         };
       }
 
-      // Handle team-members action
-      if (action === "team-members") {
+      // Handle member-info action (get team members)
+      // Built-in action with targetMode: none
+      if (action === "member-info") {
         const tid = typeof params.tid === "string" ? params.tid : "";
         const page = typeof params.page === "number" ? params.page : 1;
         const limit = typeof params.limit === "number" ? params.limit : 20;
         if (!tid) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Team-members requires tid (team ID)." }],
+            content: [{ type: "text", text: "member-info requires tid (team ID)." }],
           };
         }
         const result = await getTeamMembers({ cfg, tid, page, limit });
@@ -526,14 +541,20 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         };
       }
 
-      // Handle team-info action
-      if (action === "team-info") {
-        const tid = typeof params.tid === "string" ? params.tid : "";
+      // Handle channel-info action (get team info)
+      // Built-in action with targetMode: channelId
+      if (action === "channel-info") {
+        const tid =
+          typeof params.tid === "string"
+            ? params.tid
+            : typeof params.channelId === "string"
+              ? params.channelId.trim()
+              : "";
         const scopes = Array.isArray(params.scopes) ? params.scopes : undefined;
         if (!tid) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Team-info requires tid (team ID)." }],
+            content: [{ type: "text", text: "channel-info requires channelId (team ID)." }],
           };
         }
         const result = await getTeamInfo({ cfg, tid, scopes });
@@ -552,16 +573,22 @@ export const popoPlugin: ChannelPlugin<ResolvedPopoAccount> = {
         };
       }
 
-      // Handle update-team-info action
-      if (action === "update-team-info") {
-        const tid = typeof params.tid === "string" ? params.tid : "";
+      // Handle channel-edit action (update team info)
+      // Built-in action with targetMode: channelId
+      if (action === "channel-edit") {
+        const tid =
+          typeof params.tid === "string"
+            ? params.tid
+            : typeof params.channelId === "string"
+              ? params.channelId.trim()
+              : "";
         const name = typeof params.name === "string" ? params.name : "";
         const board = typeof params.board === "string" ? params.board : "";
         const type = typeof params.type === "number" ? params.type : undefined;
         if (!tid || !name || !board) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Update-team-info requires tid, name, and board." }],
+            content: [{ type: "text", text: "channel-edit requires channelId (tid), name, and board." }],
           };
         }
         const result = await updateTeamInfo({ cfg, tid, name, board, type });