@dhf-openclaw/grix 0.4.13 → 0.4.15

package/dist/index.js

@@ -3570,6 +3570,87 @@ async function deliverAibotPayload(params) {
   return { sent, firstMessageId };
 }
 
+// src/group-semantics.ts
+function normalizeEventType(value) {
+  return String(value ?? "").trim().toLowerCase();
+}
+function normalizeMentionUserIds(value) {
+  if (!Array.isArray(value)) {
+    return [];
+  }
+  const deduped = /* @__PURE__ */ new Set();
+  for (const entry of value) {
+    const normalized = String(entry ?? "").trim();
+    if (normalized) {
+      deduped.add(normalized);
+    }
+  }
+  return [...deduped];
+}
+function resolveGrixInboundSemantics(event) {
+  const eventType = normalizeEventType(event.event_type);
+  const isGroup = Number(event.session_type ?? 0) === 2 || eventType.startsWith("group_");
+  const mentionUserIds = normalizeMentionUserIds(event.mention_user_ids);
+  const hasAnyMention = mentionUserIds.length > 0;
+  const wasMentioned = isGroup && eventType === "group_mention";
+  const mentionsOther = isGroup && hasAnyMention && !wasMentioned;
+  const mustReply = wasMentioned;
+  const allowSilent = isGroup && !mustReply;
+  const allowActions = !isGroup || mustReply;
+  return {
+    isGroup,
+    eventType,
+    wasMentioned,
+    hasAnyMention,
+    mentionsOther,
+    mentionUserIds,
+    mustReply,
+    allowSilent,
+    allowActions
+  };
+}
+function buildGrixGroupSystemPrompt(semantics) {
+  if (!semantics.isGroup) {
+    return void 0;
+  }
+  if (semantics.wasMentioned) {
+    return [
+      "This group turn explicitly targeted you.",
+      "You must reply or take the requested action when appropriate.",
+      "Do not return NO_REPLY for this turn."
+    ].join(" ");
+  }
+  if (semantics.mentionsOther) {
+    return [
+      "This group turn explicitly targeted someone else, not you.",
+      "You may reply only if you add clear value.",
+      "Otherwise return NO_REPLY.",
+      "Do not take action unless the task is clearly yours."
+    ].join(" ");
+  }
+  return [
+    "This group turn is visible context, not an explicit mention for you.",
+    "Reply only if it clearly helps the conversation.",
+    "Otherwise return NO_REPLY.",
+    "Do not take action unless the task is clearly yours."
+  ].join(" ");
+}
+function resolveGrixMentionFallbackText() {
+  return "I'm here.";
+}
+function resolveGrixDispatchResolution(params) {
+  if (params.visibleOutputSent || params.eventResultReported) {
+    return {
+      shouldCompleteSilently: false,
+      shouldSendMentionFallback: false
+    };
+  }
+  return {
+    shouldCompleteSilently: params.semantics.allowSilent,
+    shouldSendMentionFallback: params.semantics.mustReply
+  };
+}
+
 // src/monitor.ts
 var activeMonitorClients = /* @__PURE__ */ new Map();
 function registerActiveMonitor(accountId, client) {
@@ -3847,8 +3928,10 @@ async function processEvent(params) {
   const quotedMessageId = normalizeNumericMessageId(event.quoted_message_id);
   const bodyForAgent = buildBodyWithQuotedReplyId(rawBody, quotedMessageId);
   const senderId = toStringId2(event.sender_id);
-  const isGroup = Number(event.session_type ?? 0) === 2 || String(event.event_type ?? "").startsWith("group_");
+  const semantics = resolveGrixInboundSemantics(event);
+  const isGroup = semantics.isGroup;
   const chatType = isGroup ? "group" : "direct";
+  const groupSystemPrompt = buildGrixGroupSystemPrompt(semantics);
   const createdAt = toTimestampMs(event.created_at);
   const baseLogContext = buildEventLogContext({
     eventId,
@@ -3884,7 +3967,7 @@ async function processEvent(params) {
     return;
   }
   runtime2.log(
-    `[grix:${account.accountId}] inbound event ${baseLogContext} chatType=${chatType} bodyLen=${rawBody.length} quotedMessageId=${quotedMessageId || "-"}`
+    `[grix:${account.accountId}] inbound event ${baseLogContext} chatType=${chatType} eventType=${semantics.eventType || "-"} wasMentioned=${semantics.wasMentioned ? "true" : "false"} mentionsOther=${semantics.mentionsOther ? "true" : "false"} bodyLen=${rawBody.length} quotedMessageId=${quotedMessageId || "-"}`
   );
   let inboundEventAccepted = false;
   const commandOutcome = await handleExecApprovalCommand({
@@ -3994,6 +4077,7 @@ async function processEvent(params) {
       SessionKey: route.sessionKey,
       AccountId: route.accountId,
       ChatType: chatType,
+      GroupSystemPrompt: groupSystemPrompt,
       ConversationLabel: fromLabel,
       SenderName: senderId || void 0,
       SenderId: senderId || void 0,
@@ -4004,6 +4088,7 @@ async function processEvent(params) {
       // This field carries the inbound quoted message id from end user (event.quoted_message_id).
       // It is not the outbound reply anchor used when plugin sends replies back to Aibot.
       ReplyToMessageSid: quotedMessageId,
+      WasMentioned: isGroup ? semantics.wasMentioned : void 0,
       OriginatingChannel: "grix",
       OriginatingTo: to
     });
@@ -4357,6 +4442,49 @@ async function processEvent(params) {
             markVisibleOutputSent();
           }
         }
+        const dispatchResolution = resolveGrixDispatchResolution({
+          semantics,
+          visibleOutputSent,
+          eventResultReported
+        });
+        if (dispatchResolution.shouldCompleteSilently) {
+          runtime2.log(
+            `[grix:${account.accountId}] group dispatch completed silently ${baseLogContext} attempt=${attemptLabel} wasMentioned=${semantics.wasMentioned ? "true" : "false"}`
+          );
+          reportEventResult("responded");
+        }
+        if (dispatchResolution.shouldSendMentionFallback) {
+          outboundCounter++;
+          const stableClientMsgId = `reply_${messageSid}_${outboundCounter}`;
+          runtime2.log(
+            `[grix:${account.accountId}] explicit mention fallback reply ${buildEventLogContext({
+              eventId,
+              sessionId,
+              messageSid,
+              clientMsgId: stableClientMsgId,
+              outboundCounter
+            })}`
+          );
+          const didSendFallback = await deliverAibotMessage({
+            payload: {
+              text: resolveGrixMentionFallbackText()
+            },
+            client,
+            account,
+            sessionId,
+            abortSignal: runAbortController.signal,
+            eventId,
+            quotedMessageId: outboundQuotedMessageId,
+            runtime: runtime2,
+            statusSink,
+            stableClientMsgId,
+            tableMode
+          });
+          attemptHasOutbound = attemptHasOutbound || didSendFallback;
+          if (didSendFallback) {
+            markVisibleOutputSent();
+          }
+        }
         break;
       }
       if (!visibleOutputSent && !eventResultReported) {
@@ -4579,6 +4707,26 @@ function applySetupAccountConfig(params) {
   };
 }
 
+// src/group-adapter.ts
+function resolveGrixGroupRequireMention() {
+  return false;
+}
+function resolveGrixGroupIntroHint() {
+  return [
+    "All Grix group messages are visible to you.",
+    "If WasMentioned is true, you are the primary addressee and should respond.",
+    "If WasMentioned is false, reply only when you add clear value; otherwise use NO_REPLY."
+  ].join(" ");
+}
+
+// src/group-tool-policy.ts
+var GROUP_DEFAULT_DENY = ["message"];
+function resolveGrixGroupToolPolicy() {
+  return {
+    deny: [...GROUP_DEFAULT_DENY]
+  };
+}
+
 // src/channel.ts
 var meta = {
   id: "grix",
@@ -4795,6 +4943,11 @@ var aibotPlugin = {
       hasRepliedRef
     })
   },
+  groups: {
+    resolveRequireMention: () => resolveGrixGroupRequireMention(),
+    resolveGroupIntroHint: () => resolveGrixGroupIntroHint(),
+    resolveToolPolicy: () => resolveGrixGroupToolPolicy()
+  },
   agentPrompt: {
     messageToolHints: () => [
       "- Grix `action=unsend` is a silent cleanup action: unsend the target `messageId`, unsend the recall command message when applicable, then end with `NO_REPLY` and do not send any confirmation text. Omit `sessionId`/`to` only when targeting the current Grix chat."

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dhf-openclaw/grix",
-  "version": "0.4.13",
+  "version": "0.4.15",
   "description": "Unified Grix OpenClaw plugin with channel transport, typed admin tools, and operator CLI",
   "type": "module",
   "main": "./dist/index.js",

package/skills/grix-group/SKILL.md

@@ -77,10 +77,13 @@ Required input:
 
 Guardrails:
 
-1. This action is only for the current Agent leaving its own group membership.
-2. Never translate a request to remove other members into `leave`; use `remove_members` for that.
-3. Do not send `memberId`, `memberIds`, or `memberTypes` with this action.
-4. This action does not require scope and should not be described as a scope-grant workflow.
+1. Only the current Agent owner can request this action.
+2. If the requester is not the current Agent owner, reject the request directly and do not call `grix_group`.
+3. This action is only for the current Agent leaving its own group membership.
+4. Never translate a request to remove other members into `leave`; use `remove_members` for that.
+5. Do not send `memberId`, `memberIds`, or `memberTypes` with this action.
+6. This action does not require scope and should not be described as a scope-grant workflow.
+7. Execute `leave` silently: do not send any pre-leave or farewell message to the group before leaving.
 
 ### remove_members
 
@@ -154,6 +157,7 @@ Required input:
 2. Include key identifiers (`session_id`, member count, mute state) when successful.
 3. Include exact remediation when failed.
 4. Never hide scope or auth errors behind generic wording.
+5. For `leave`, report result to the requester only; do not post extra messages into the group session.
 
 ## References