@shenhh/popo 0.1.17 → 0.1.19

package/package.json

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

package/skills/popo-msg/SKILL.md

@@ -237,6 +237,16 @@ 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",
@@ -248,8 +258,11 @@ Recall (retract) a sent message within the allowed time window.
 
 **Parameters:**
 - `msgId` (required): Message ID to recall
-- `sessionId` (required): Session ID (user email for P2P, group ID for group)
-- `sessionType` (required): `1` for P2P, `3` for group
+- `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.
 
 **Note:** Messages can only be recalled within a limited time (typically 2 minutes).
 
@@ -257,6 +270,20 @@ Recall (retract) a sent message within the allowed time window.
 
 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",
@@ -271,12 +298,15 @@ Query read/unread status for group messages.
 
 **Parameters:**
 - `msgId` (required): Message ID
-- `sessionType` (required): Must be `3` (group)
-- `type` (required): `1` for read list, `2` for unread list
-- `st` (required): Query end timestamp (milliseconds)
+- `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
 - `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

@@ -313,12 +313,14 @@ 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 : "";
-        const sessionId = typeof params.sessionId === "string" ? params.sessionId : "";
-        const sessionType = typeof params.sessionType === "number" ? params.sessionType : 1;
+        // 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);
         if (!msgId || !sessionId) {
           return {
             isError: true,
-            content: [{ type: "text", text: "Recall requires msgId and sessionId." }],
+            content: [{ type: "text", text: "Recall requires msgId and target (or sessionId)." }],
           };
         }
         const result = await recallMessagePopo({ cfg, msgId, sessionId, sessionType: sessionType as 1 | 3 });
@@ -339,6 +341,8 @@ 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;
@@ -349,6 +353,13 @@ 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,