duclaw-cli 1.8.38 → 1.8.40

package/dist/bundle.js

@@ -30242,7 +30242,7 @@ function printHelp() {
 `);
 }
 function printVersion() {
-  console.log(`duclaw-cli v${true ? "1.8.38" : "unknown"}`);
+  console.log(`duclaw-cli v${true ? "1.8.40" : "unknown"}`);
 }
 function getDuclawTemplate() {
   return {
@@ -42310,6 +42310,19 @@ var failCeoFollowup = (id, error) => {
           AND status IN ('pending', 'processing', 'failed')
     `).run(error.slice(0, 1e3), now, id);
 };
+var recoverStaleProcessingCeoFollowups = (staleBefore) => {
+  const db3 = createSqliteDB();
+  const now = Date.now();
+  const result = db3.prepare(`
+        UPDATE ceo_followups
+        SET status = 'failed',
+            last_error = 'stale_processing_recovered',
+            updated_at = ?
+        WHERE status = 'processing'
+          AND updated_at < ?
+    `).run(now, staleBefore);
+  return result.changes;
+};
 var completePendingCeoFollowupsForUser = (originUserId) => {
   const db3 = createSqliteDB();
   const now = Date.now();
@@ -42669,6 +42682,18 @@ var departmentCreate = {
 
 // src/tools/tools/department/DepartmentCommunicate.ts
 var CEO_MAILBOX_ID = `manager`;
+var MANAGER_ALIAS_PATTERN = /(^|::)(manager|main[-_\s]?manager|ceo)$/i;
+var looksLikePseudoManagerMailbox = (mailboxId) => {
+  if (mailboxId === CEO_MAILBOX_ID) return true;
+  if (!mailboxId.includes(`::`)) return false;
+  return MANAGER_ALIAS_PATTERN.test(mailboxId.trim());
+};
+var buildManagerRoutingRejection = (activeContext) => [
+  `[departmentCommunicate] \u62D2\u7EDD\u53D1\u9001\u5230\u4F2A manager mailbox\u3002`,
+  `Main Manager/CEO \u4E0D\u662F\u90E8\u95E8\u6210\u5458\uFF0C\u4E0D\u80FD\u7528 department_communicate \u53D1\u9001\u5230 manager\u3001Department::Manager\u3001CEO::Manager \u7B49\u5730\u5740\u3002`,
+  activeContext ? `\u5982\u679C\u8981\u5411\u4E0A\u6E38 CEO \u6C47\u62A5\uFF0C\u8BF7\u8C03\u7528 reply_mailbox(message_id="${activeContext.id}", content="...") \u6B63\u5F0F\u56DE\u590D\u539F\u90AE\u4EF6\uFF1B\u82E5\u53EA\u662F\u540C\u6B65\u9636\u6BB5\u8FDB\u5C55\uFF0C\u8BF7\u8C03\u7528 mailbox_followup(message_id="${activeContext.id}", content="...", kind="progress")\u3002` : `\u5982\u679C\u8981\u5411\u4E0A\u6E38 CEO \u6C47\u62A5\uFF0C\u8BF7\u56DE\u5230\u539F manager \u90AE\u4EF6\uFF0C\u4F7F\u7528 reply_mailbox(message_id="\u539F\u90AE\u4EF6ID", content="...") \u6216 mailbox_followup(message_id="\u539F\u90AE\u4EF6ID", content="...", kind="progress")\u3002`
+].join(`
+`);
 var getActiveMailboxContext = (actorMailboxId, messageId) => {
   if (!messageId) return null;
   const db3 = createSqliteDB();
@@ -42695,6 +42720,7 @@ var DESCRIPTION21 = `
 - CEO \u9ED8\u8BA4\u901A\u8FC7 department_list \u53EA\u770B\u5230\u90E8\u95E8\u548C Department Head\uFF0C\u4E0D\u4E3B\u52A8\u66B4\u9732 Executor\u3002
 - Department Head \u9ED8\u8BA4\u7BA1\u7406\u672C\u90E8\u95E8\u6210\u5458\u3002
 - \u5982\u679C\u667A\u80FD\u4F53\u5DF2\u7ECF\u901A\u8FC7\u6B63\u5E38\u6C9F\u901A\u77E5\u9053\u4E86\u5176\u4ED6\u6210\u5458\u7684 mailbox \u5730\u5740\uFF0C\u5DE5\u5177\u4E0D\u4F1A\u786C\u6027\u963B\u6B62\u901A\u4FE1\uFF1B\u8FD9\u7C7B\u975E\u9ED8\u8BA4\u8DEF\u5F84\u4F1A\u5199\u5165\u5BA1\u8BA1\u4FE1\u606F\u3002
+- Main Manager/CEO \u7684 mailbox id \u56FA\u5B9A\u662F manager\u3002Department agent \u4E0D\u5E94\u4F7F\u7528 department_communicate \u7ED9 manager \u6216 Department::Manager \u53D1\u4FE1\uFF1B\u8981\u56DE\u590D\u4E0A\u6E38\u5FC5\u987B\u4F7F\u7528\u539F\u90AE\u4EF6\u7684 reply_mailbox \u6216 mailbox_followup\u3002
 
 \u53C2\u6570\u8BF4\u660E\uFF1A
 - department_name + member_name: \u9ED8\u8BA4\u5BFB\u5740\u65B9\u5F0F\u3002
@@ -42739,18 +42765,6 @@ var departmentCommunicate = {
     const targetMailboxId = input.target_mailbox_id;
     const content = input.content;
     const reason = input.reason;
-    let toMailboxId = targetMailboxId;
-    let targetMember = targetMailboxId ? getDepartmentMemberByMailboxId(targetMailboxId) : null;
-    if (!toMailboxId) {
-      if (!departmentName || !memberName) {
-        return `[departmentCommunicate] department_name/member_name \u6216 target_mailbox_id \u81F3\u5C11\u9700\u8981\u63D0\u4F9B\u4E00\u79CD`;
-      }
-      targetMember = getDepartmentMemberByName(departmentName, memberName);
-      if (!targetMember) {
-        return `[departmentCommunicate] \u4E0D\u5B58\u5728 ${departmentName} \u90E8\u95E8\u7684\u6210\u5458: ${memberName}`;
-      }
-      toMailboxId = getMailBoxId(departmentName, memberName);
-    }
     let fromMailboxId;
     const departmentAgentId = userRequest.departmentAgentId;
     const actorMember = departmentAgentId ? getDepartmentMemberByMailboxId(
@@ -42765,6 +42779,30 @@ var departmentCommunicate = {
     } else {
       fromMailboxId = CEO_MAILBOX_ID;
     }
+    const activeContext = departmentAgentId ? getActiveMailboxContext(fromMailboxId, userRequest.requestId) : null;
+    let toMailboxId = targetMailboxId;
+    let targetMember = targetMailboxId ? getDepartmentMemberByMailboxId(targetMailboxId) : null;
+    if (departmentAgentId && targetMailboxId && looksLikePseudoManagerMailbox(targetMailboxId)) {
+      return buildManagerRoutingRejection(activeContext);
+    }
+    if (departmentAgentId && targetMailboxId && !targetMember) {
+      return [
+        `[departmentCommunicate] \u62D2\u7EDD\u53D1\u9001\u5230\u4E0D\u5B58\u5728\u7684 mailbox: ${targetMailboxId}`,
+        `Department agent \u4F7F\u7528 target_mailbox_id \u65F6\uFF0C\u76EE\u6807\u5FC5\u987B\u662F\u5DF2\u5B58\u5728\u7684\u90E8\u95E8\u6210\u5458 mailbox\u3002`,
+        `\u5982\u679C\u4F60\u60F3\u56DE\u590D\u4E0A\u6E38 CEO/Main Manager\uFF0C\u8BF7\u4E0D\u8981\u4F2A\u9020 manager \u5730\u5740\uFF1B\u8BF7\u4F7F\u7528\u539F\u90AE\u4EF6\u7684 reply_mailbox \u6216 mailbox_followup\u3002`
+      ].join(`
+`);
+    }
+    if (!toMailboxId) {
+      if (!departmentName || !memberName) {
+        return `[departmentCommunicate] department_name/member_name \u6216 target_mailbox_id \u81F3\u5C11\u9700\u8981\u63D0\u4F9B\u4E00\u79CD`;
+      }
+      targetMember = getDepartmentMemberByName(departmentName, memberName);
+      if (!targetMember) {
+        return `[departmentCommunicate] \u4E0D\u5B58\u5728 ${departmentName} \u90E8\u95E8\u7684\u6210\u5458: ${memberName}`;
+      }
+      toMailboxId = getMailBoxId(departmentName, memberName);
+    }
     const isCrossDepartment = Boolean(actorMember && targetMember && actorMember.departmentId !== targetMember.departmentId);
     const isKnownAddressPath = Boolean(targetMailboxId);
     const options = {
@@ -42781,7 +42819,6 @@ var departmentCommunicate = {
       options.originUserId = userRequest.userId;
       options.originPlatform = userRequest.platform;
     } else {
-      const activeContext = getActiveMailboxContext(fromMailboxId, userRequest.requestId);
       if (activeContext) {
         options.originUserId = activeContext.originUserId;
         options.originPlatform = activeContext.originPlatform;
@@ -47042,6 +47079,8 @@ var replyMailbox = {
 
 // src/cron/mailbox.ts
 var db2 = createSqliteDB();
+var DEFAULT_MAILBOX_PROCESSING_STALE_MS = 15 * 6e4;
+var PSEUDO_MANAGER_MAILBOX_PATTERN = /(^|::)(manager|main[-_\s]?manager|ceo)$/i;
 var selectStmt = db2.prepare(`select
         id,
         to_mailbox_id as toMailboxId,
@@ -47057,6 +47096,79 @@ var selectStmt = db2.prepare(`select
         work_item_role as workItemRole,
         upstream_message_id as upstreamMessageId
     from mailbox where status = ?`);
+var getMailboxLivenessSnapshot = () => {
+  const mailboxRows = db2.prepare(`
+        SELECT status, COUNT(*) as count
+        FROM mailbox
+        WHERE status IN ('pending', 'processing')
+        GROUP BY status
+    `).all();
+  const followupRows = db2.prepare(`
+        SELECT status, COUNT(*) as count
+        FROM ceo_followups
+        WHERE status IN ('pending', 'processing', 'failed')
+        GROUP BY status
+    `).all();
+  const mailboxCount = (status) => mailboxRows.find((row) => row.status === status)?.count ?? 0;
+  const followupCount = (status) => followupRows.find((row) => row.status === status)?.count ?? 0;
+  return {
+    pendingMailbox: mailboxCount("pending"),
+    processingMailbox: mailboxCount("processing"),
+    pendingCeoFollowups: followupCount("pending"),
+    processingCeoFollowups: followupCount("processing"),
+    failedCeoFollowups: followupCount("failed")
+  };
+};
+var hasUnresolvedInternalWork = (snapshot) => snapshot.pendingMailbox > 0 || snapshot.processingMailbox > 0 || snapshot.pendingCeoFollowups > 0 || snapshot.processingCeoFollowups > 0 || snapshot.failedCeoFollowups > 0;
+var looksLikePseudoManagerMailbox2 = (mailboxId) => mailboxId !== "manager" && mailboxId.includes("::") && PSEUDO_MANAGER_MAILBOX_PATTERN.test(mailboxId.trim());
+var reportMailboxLiveness = async (snapshot) => {
+  if (!hasUnresolvedInternalWork(snapshot)) return;
+  await reportRuntimeActivity({
+    kind: "mailbox",
+    toolName: "mailbox_poller",
+    status: "active",
+    metadata: {
+      ...snapshot,
+      inFlightDepartmentMailboxes: Array.from(inFlightDepartmentMailboxes),
+      inFlightCeoFollowups: Array.from(inFlightCeoReplyMessages)
+    }
+  });
+};
+var recoverStaleProcessingMailboxMessages = () => {
+  const staleMs = Number(process.env.MAILBOX_PROCESSING_STALE_MS ?? DEFAULT_MAILBOX_PROCESSING_STALE_MS);
+  if (staleMs <= 0) return 0;
+  const staleBefore = Date.now() - staleMs;
+  const rows = db2.prepare(`
+        SELECT
+            id,
+            to_mailbox_id as toMailboxId
+        FROM mailbox
+        WHERE status = 'processing'
+          AND updated_at < ?
+    `).all(staleBefore);
+  let recovered = 0;
+  for (const row of rows) {
+    if (row.toMailboxId === "manager") continue;
+    if (hasRunningAgent(row.toMailboxId) || inFlightDepartmentMailboxes.has(row.toMailboxId)) continue;
+    if (updateMailboxMessageStatus(row.id, "pending", {
+      fromStatus: "processing",
+      actorMailboxId: row.toMailboxId,
+      reason: "stale_processing_recovered"
+    })) {
+      recovered += 1;
+    }
+  }
+  return recovered;
+};
+var recoverStaleProcessingWork = () => {
+  const staleMs = Number(process.env.MAILBOX_PROCESSING_STALE_MS ?? DEFAULT_MAILBOX_PROCESSING_STALE_MS);
+  if (staleMs <= 0) return;
+  const mailboxRecovered = recoverStaleProcessingMailboxMessages();
+  const followupsRecovered = recoverStaleProcessingCeoFollowups(Date.now() - staleMs);
+  if (mailboxRecovered > 0 || followupsRecovered > 0) {
+    console.warn(`[mailbox] \u5DF2\u6062\u590D stale processing: mailbox=${mailboxRecovered}, ceo_followups=${followupsRecovered}`);
+  }
+};
 var markMailboxStatus = (msgId, status) => {
   const fromStatusMap = {
     processing: ["pending"],
@@ -47163,6 +47275,11 @@ var wakeDepartmentAgent = async (mailboxId, msgIds) => {
     const member = getDepartmentMemberByName(departmentName, memberName);
     if (!member) {
       console.warn(`[mailbox] \u76EE\u6807\u6210\u5458\u4E0D\u5B58\u5728\uFF08\u53EF\u80FD\u5DF2\u88AB\u5220\u9664\uFF09: ${mailboxId}`);
+      if (looksLikePseudoManagerMailbox2(mailboxId)) {
+        for (const id of msgIds) markMailboxStatus(id, "cancelled");
+        console.warn(`[mailbox] \u5DF2\u53D6\u6D88\u4F2A manager mailbox \u6295\u9012 ${mailboxId}\uFF0C\u4E0D\u518D\u751F\u6210 CEO \u7CFB\u7EDF\u566A\u58F0`);
+        return;
+      }
       const notifyContent = `[\u7CFB\u7EDF\u901A\u77E5] \u90E8\u95E8\u6210\u5458 ${mailboxId} \u4E0D\u5B58\u5728\uFF08\u53EF\u80FD\u6240\u5C5E\u90E8\u95E8\u5DF2\u89E3\u6563\u6216\u6210\u5458\u5DF2\u88AB\u5220\u9664\uFF09\uFF0C\u4EE5\u4E0B\u6D88\u606F\u65E0\u6CD5\u6295\u9012\u5DF2\u88AB\u53D6\u6D88\u3002\u8BF7\u6839\u636E\u60C5\u51B5\u51B3\u5B9A\u662F\u5426\u9700\u8981\u91CD\u65B0\u7EC4\u5EFA\u90E8\u95E8\u6216\u8C03\u6574\u8BA1\u5212\u3002`;
       const originStmt = db2.prepare(
         `SELECT origin_user_id as originUserId, origin_platform as originPlatform 
@@ -47303,6 +47420,8 @@ var pollMailbox = async () => {
   if (polling) return;
   polling = true;
   try {
+    recoverStaleProcessingWork();
+    await reportMailboxLiveness(getMailboxLivenessSnapshot());
     const mailBoxMsgs = selectStmt.all("pending");
     const pendingFollowups = listPendingCeoFollowups();
     if (mailBoxMsgs.length === 0 && pendingFollowups.length === 0) return;
@@ -52432,7 +52551,7 @@ var systemRoutes = new Hono2();
 var startTime = Date.now();
 systemRoutes.get("/system/info", (c) => {
   return c.json({
-    version: true ? "1.8.38" : "unknown",
+    version: true ? "1.8.40" : "unknown",
     uptime: Math.floor((Date.now() - startTime) / 1e3),
     env: process.env.NODE_ENV || "development",
     nodeVersion: process.version