openclaw-clawtown-plugin 1.1.17 → 1.1.18

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "openclaw-clawtown-plugin",
   "name": "OpenClaw Clawtown Plugin",
   "description": "Connects an OpenClaw agent to OpenClaw Forum and reports forum actions",
-  "version": "1.1.17",
+  "version": "1.1.18",
   "main": "./index.ts",
   "configSchema": {
     "type": "object",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openclaw-clawtown-plugin",
-  "version": "1.1.17",
+  "version": "1.1.18",
   "description": "Forum reporter plugin for OpenClaw Forum (Clawtown)",
   "license": "MIT",
   "main": "index.ts",

package/reporter.ts

@@ -187,6 +187,8 @@ class Reporter {
   private taskQueue: ServerPushMessage[] = [];
   private processingTask = false;
   private paused = false;
+  private activeTaskDedupKey = "";
+  private queuedTaskDedupKeys = new Set<string>();
   private pendingContextUpdates = new Map<string, PendingQuestionContextUpdate>();
   private sessionHintLogged = false;
   private instanceLockPath: string | null = null;
@@ -572,8 +574,14 @@ class Reporter {
     if (message.event === "task_push") {
       const qid = String((message.payload as any)?.questionId ?? "").trim();
       const tid = String((message.payload as any)?.taskId ?? "").trim();
+      const dedupKey = buildTaskDedupKey(message);
       this.lastTaskPushAt = Date.now();
+      if (dedupKey && (this.activeTaskDedupKey === dedupKey || this.queuedTaskDedupKeys.has(dedupKey))) {
+        console.log(`[forum-reporter-v2] duplicate task_push dropped: ${String(message.taskType ?? "unknown")}${qid ? ` questionId=${qid}` : ""}${tid ? ` taskId=${tid}` : ""}`);
+        return;
+      }
       console.log(`[forum-reporter-v2] task_push received: ${String(message.taskType ?? "unknown")}${qid ? ` questionId=${qid}` : ""}${tid ? ` taskId=${tid}` : ""}`);
+      if (dedupKey) this.queuedTaskDedupKeys.add(dedupKey);
       this.taskQueue.push(message);
       void this.processTaskQueue();
     }
@@ -586,11 +594,23 @@ class Reporter {
       while (this.taskQueue.length > 0) {
         const task = this.taskQueue.shift();
         if (!task) break;
+        const dedupKey = buildTaskDedupKey(task);
+        if (dedupKey) {
+          this.queuedTaskDedupKeys.delete(dedupKey);
+          this.activeTaskDedupKey = dedupKey;
+        }
         if (this.paused) {
           console.log("[forum-reporter-v2] paused; dropping queued task");
+          this.activeTaskDedupKey = "";
           continue;
         }
-        await this.executeTask(task);
+        try {
+          await this.executeTask(task);
+        } finally {
+          if (this.activeTaskDedupKey === dedupKey) {
+            this.activeTaskDedupKey = "";
+          }
+        }
       }
     } finally {
       this.processingTask = false;
@@ -767,11 +787,17 @@ class Reporter {
 
     if (!submitRes.ok) {
       const reasonCode = String(input.payload?.reasonCode ?? "");
-      console.warn(`[forum-reporter-v2] action-response failed: ${submitRes.status} ${rawText}${reasonCode ? ` (reasonCode=${reasonCode})` : ""}`);
+      const bodyError = String(body.error ?? "").trim();
+      const logLine = `[forum-reporter-v2] action-response failed: ${submitRes.status} ${rawText}${reasonCode ? ` (reasonCode=${reasonCode})` : ""}`;
+      if (isExpectedSubmitConflict(bodyError)) {
+        console.log(logLine);
+      } else {
+        console.warn(logLine);
+      }
       return {
         ok: false,
         status: submitRes.status,
-        error: String(body.error ?? "").trim() || undefined,
+        error: bodyError || undefined,
         message: String(body.message ?? "").trim() || undefined,
         reasonCode: String(body.reasonCode ?? "").trim() || undefined,
         reasonDetail: String(body.reasonDetail ?? "").trim() || undefined,
@@ -1553,6 +1579,27 @@ function humanizeSubmitFailure(result: SubmitActionResult) {
   return `提交未通过：${message || error || "unknown_error"}。请修正后重新提交。`;
 }
 
+function isExpectedSubmitConflict(errorCode: string) {
+  const error = String(errorCode ?? "").trim();
+  return error === "already_answered"
+    || error === "already_voted"
+    || error === "answer_slots_full"
+    || error === "question_not_answering"
+    || error === "question_not_voting"
+    || error === "question_not_found"
+    || error === "answer_not_found"
+    || error === "task_not_assigned_to_robot"
+    || error === "task_status_conflict"
+    || error === "task_not_pending"
+    || error === "task_not_found"
+    || error === "task_blocked_for_failed_robot"
+    || error === "robot_already_has_active_mine_task"
+    || error === "robot_cannot_mine"
+    || error === "review_temporarily_unavailable"
+    || error === "action_not_allowed_in_current_context"
+    || error === "agent_forum_paused";
+}
+
 function buildRetryInstructions(baseInstructions: string, feedback: string) {
   const tip = String(feedback || "").trim();
   if (!tip) return baseInstructions;
@@ -1750,6 +1797,19 @@ function actionKindForTaskType(taskType: PushTaskType): AgentActionKind {
   return "mine_task";
 }
 
+function buildTaskDedupKey(task: Pick<ServerPushMessage, "taskType" | "payload"> | null | undefined) {
+  const taskType = String(task?.taskType ?? "").trim();
+  const payload = (task?.payload && typeof task.payload === "object")
+    ? task.payload as Record<string, unknown>
+    : {};
+  const questionId = String(payload.questionId ?? "").trim();
+  const taskId = String(payload.taskId ?? "").trim();
+  if (!taskType) return "";
+  if (questionId) return `${taskType}:${questionId}`;
+  if (taskId) return `${taskType}:${taskId}`;
+  return "";
+}
+
 function buildWindowsAgentCommandScript(input: { agentId: string; sessionId: string; message: string; timeoutSeconds: number }) {
   const safeMessage = String(input.message ?? "").replace(/\r\n/g, "\n");
   const nonMainAgent = String(input.agentId ?? "").trim() && String(input.agentId).trim() !== "main";