@yeaft/webchat-agent 0.1.498 → 0.1.499

package/crew/routing.js

@@ -235,7 +235,7 @@ export function resolveRoleName(to, session, fromRole) {
  * @param {Array<{mimeType, data}>} [turnImages] - auto-attached images from the turn (max 3)
  */
 export async function executeRoute(session, fromRole, route, turnImages = []) {
-  const { to, summary, taskId, taskTitle } = route;
+  let { to, summary, taskId, taskTitle } = route;
 
   // Auto-resume: paused/stopped → running (route execution means work should continue)
   if (session.status === 'paused' || session.status === 'stopped') {
@@ -244,14 +244,51 @@ export async function executeRoute(session, fromRole, route, turnImages = []) {
     sendStatusUpdate(session);
   }
 
+  // ─── task-321: taskId fallback chain ─────────────────────────────
+  // When a ROUTE omits `task:` (shorthand, bare dispatch, human messages,
+  // PM forgetting the field), fall back to:
+  //   (a) the sender's currentTask.taskId
+  //   (b) the most recent non-system entry in session.messageHistory
+  // This keeps prev-* / designer / architect / shorthand messages from
+  // becoming taskId=null orphans that never appear on any feature card.
+  if (!taskId) {
+    const fromRoleState = session.roleStates?.get(fromRole);
+    if (fromRoleState?.currentTask?.taskId) {
+      taskId = fromRoleState.currentTask.taskId;
+      taskTitle = taskTitle || fromRoleState.currentTask.taskTitle || null;
+    } else if (Array.isArray(session.messageHistory) && session.messageHistory.length > 0) {
+      for (let i = session.messageHistory.length - 1; i >= 0; i--) {
+        const h = session.messageHistory[i];
+        if (h && h.from !== 'system' && h.taskId) {
+          taskId = h.taskId;
+          break;
+        }
+      }
+    }
+    // Mirror the fallback back into the route object so downstream
+    // consumers (dispatchToRole / sendCrewOutput) see the inferred id.
+    if (taskId) {
+      route.taskId = taskId;
+      if (taskTitle) route.taskTitle = taskTitle;
+    }
+  }
+
   // Task 文件自动管理（fire-and-forget）
   if (taskId && summary) {
     const fromRoleConfig = session.roles.get(fromRole);
-    if (fromRoleConfig?.isDecisionMaker && taskTitle && to !== 'human') {
-      ensureTaskFile(session, taskId, taskTitle, to, summary)
+    // task-321: Auto-create feature file even when a non-PM role is the
+    // first to mention the taskId. Any role carrying a taskId (PM, devs,
+    // reviewers, designer, architect) now triggers creation — not just PM
+    // with explicit taskTitle. appendTaskRecord itself also creates the
+    // file if missing, so this is a best-effort fast path.
+    const effectiveTitle = taskTitle
+      || session.features?.get(taskId)?.taskTitle
+      || null;
+    if (effectiveTitle && to !== 'human') {
+      ensureTaskFile(session, taskId, effectiveTitle, fromRoleConfig?.isDecisionMaker ? to : fromRole, summary)
         .catch(e => console.warn(`[Crew] Failed to create task file ${taskId}:`, e.message));
     }
-    appendTaskRecord(session, taskId, fromRole, summary)
+    appendTaskRecord(session, taskId, fromRole, summary, { taskTitle: effectiveTitle })
       .catch(e => console.warn(`[Crew] Failed to append task record ${taskId}:`, e.message));
 
     // 更新工作看板：推断状态
@@ -375,8 +412,13 @@ export async function dispatchToRole(session, roleName, content, fromSource, tas
   }
 
   // 设置 task
+  // task-321: keep currentTask sticky. A new taskId updates it; a dispatch
+  // without taskId preserves the previous currentTask so subsequent
+  // sendCrewOutput calls (which read roleState.currentTask.taskId) keep
+  // attaching to the right feature card — instead of falling back to null
+  // the moment the sender omits the `task:` field.
   if (taskId) {
-    roleState.currentTask = { taskId, taskTitle };
+    roleState.currentTask = { taskId, taskTitle: taskTitle || roleState.currentTask?.taskTitle || null };
   }
 
   // Task 上下文注入

package/crew/task-files.js

@@ -59,15 +59,41 @@ ${m.workRecord}
 
 /**
  * 追加工作记录到 task 文件
+ *
+ * task-321: auto-create the feature file if it doesn't exist yet. Previously
+ * a missing file silently dropped the record, which meant that whenever a
+ * non-PM role was first to mention a taskId, the file was never created and
+ * every subsequent record — including from PM — would also be dropped. We
+ * now recover a title from opts.taskTitle → session.features cache → taskId
+ * itself, and create the file on the fly.
+ *
+ * @param {object} session
+ * @param {string} taskId
+ * @param {string} roleName
+ * @param {string} summary
+ * @param {{ taskTitle?: string|null, assignee?: string|null }} [opts]
  */
-export async function appendTaskRecord(session, taskId, roleName, summary) {
+export async function appendTaskRecord(session, taskId, roleName, summary, opts = {}) {
   const filePath = join(session.sharedDir, 'context', 'features', `${taskId}.md`);
 
+  let exists = true;
   try {
     await fs.access(filePath);
   } catch {
-    // 文件不存在，跳过
-    return;
+    exists = false;
+  }
+
+  if (!exists) {
+    const recoveredTitle = opts.taskTitle
+      || session.features?.get(taskId)?.taskTitle
+      || taskId;
+    const assignee = opts.assignee || roleName;
+    try {
+      await ensureTaskFile(session, taskId, recoveredTitle, assignee, summary);
+    } catch (e) {
+      console.warn(`[Crew] Failed to auto-create task file ${taskId} on append:`, e.message);
+      return;
+    }
   }
 
   const role = session.roles.get(roleName);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "0.1.498",
+  "version": "0.1.499",
   "description": "Remote agent for Yeaft WebChat — connects worker machines to the central server",
   "main": "index.js",
   "type": "module",