@yeaft/webchat-agent 1.0.556 → 1.0.558

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "1.0.556",
+  "version": "1.0.558",
   "description": "Remote worker agent for Yeaft Web Code Agent — connects the native Yeaft engine, CLI providers, and workbench tools",
   "main": "index.js",
   "type": "module",

package/yeaft/tools/create-work-item.js

@@ -21,7 +21,7 @@ Use this when work must continue beyond the current turn, needs role handoffs, r
     properties: {
       title: {
         type: 'string',
-        description: { en: 'Short work item title', zh: '简短的工作项标题' },
+        description: { en: 'Optional explicit title; otherwise the Coordinator generates one', zh: '可选的显式标题；省略时由 Coordinator 生成' },
       },
       goal: {
         type: 'string',
@@ -45,7 +45,7 @@ Use this when work must continue beyond the current turn, needs role handoffs, r
         description: { en: 'Start Coordinator execution immediately (default true)', zh: '是否立即启动 Coordinator 执行（默认 true）' },
       },
     },
-    required: ['title', 'goal'],
+    required: ['goal'],
   },
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
@@ -57,14 +57,14 @@ Use this when work must continue beyond the current turn, needs role handoffs, r
     if (!sessionId) throw new Error('CreateWorkItem requires an active Session');
     const title = typeof input?.title === 'string' ? input.title.trim() : '';
     const goal = typeof input?.goal === 'string' ? input.goal.trim() : '';
-    if (!title || !goal) throw new Error('title and goal are required');
+    if (!goal) throw new Error('goal is required');
 
     // Dynamic import avoids tools/index -> create-work-item -> bridge -> runner
     // -> tools/index becoming a static initialization cycle.
     const { createWorkItemFromProducer, snapshotCurrentSessionContext } = await import('../work-center/bridge.js');
     const sessionContext = await snapshotCurrentSessionContext(sessionId);
     const detail = await createWorkItemFromProducer({
-      title,
+      ...(title ? { title } : {}),
       goal,
       acceptanceCriteria: cleanCriteria(input.acceptanceCriteria),
       workItemType: typeof input.workItemType === 'string' ? input.workItemType.trim() : 'auto',

package/yeaft/web-bridge.js

@@ -4290,12 +4290,14 @@ function handleEngineEvent(event, hctx) {
     case 'tool_end': {
       const images = Array.isArray(event.displayImages) ? event.displayImages : [];
       const displayOutput = typeof event.output === 'string' ? event.output : JSON.stringify(event.output ?? '');
-      for (const image of images) {
+      for (const [sourceImageIndex, image] of images.entries()) {
         const persistedImage = imageMetadataForPersistence(image);
         try {
           if (!ctx.assetOutbox) throw new Error('asset outbox is unavailable');
           const deliveryId = ctx.assetOutbox.enqueue({
             conversationId: yeaftConversationId,
+            sourceToolCallId: event.id,
+            sourceImageIndex,
             metadata: persistedImage,
             sessionId: hctx.sessionId,
             vpId: hctx.vpId,

package/yeaft/work-center/coordinator.js

@@ -230,6 +230,7 @@ Return exactly one JSON object and no surrounding prose:
   "decision": {
     "kind": "answer|create_actions|guide_actions|request_human|complete",
     "reason": "short audit reason",
+    "title": null,
     "question": null,
     "workItemType": null,
     "contractPatch": null,
@@ -242,8 +243,9 @@ Return exactly one JSON object and no surrounding prose:
 }
 
 Rules:
+- When workItem.titleSource is coordinator_pending, include a concise title (prefer 6–12 words or a short Chinese phrase; at most 200 characters) in decision.title. This display label summarizes the original goal; it is not a contractPatch and must not rewrite or shorten the goal. Otherwise leave decision.title null. A missing or oversized display title is normalized by the runtime and must not change the substantive decision.
 - answer: explain state only. Never use it for an automatic advance trigger.
-- Never mutate title, goal, acceptanceCriteria, or deliveryTarget during automatic advance/recovery. contractPatch is allowed only for explicit user-originated refinement, never to make existing evidence pass. For an older WorkItem with no acceptance criteria, request_human to establish its completion condition before commissioning new work.
+- Never mutate goal, acceptanceCriteria, or deliveryTarget during automatic advance/recovery. decision.title is the only automatic title-generation path and is allowed only while titleSource is coordinator_pending; contractPatch (including a user-specified title) is allowed only for explicit user-originated refinement, never to make existing evidence pass. For an older WorkItem with no acceptance criteria, request_human to establish its completion condition before commissioning new work.
 - create_actions: create 1..8 currently runnable Actions. Every Action needs type, objective, approach, expectedOutcome, capability, candidateVpIds, assignmentReason, sourceActionIds, workspaceMode, and optional maxAttempts/separateFromActionTypes. sourceActionIds are context/audit references, never scheduling dependencies. Do not include dependsOnActionIds, dependsOnStageIds, stages, or a graph.
 - A missing skill/capability label is not a missing execution capability. Prefer an existing VP with a task-specific brief. Missing tools, credentials, or authorization require request_human; never expand roles as a workaround. create_vp is only appropriate when creating a persistent role is itself an explicit user deliverable.
 - closeActions may accompany create_actions. Each entry is {"actionId":"failed or waiting durable Action id","reason":"why it is no longer required"}. Close only work made obsolete by replacement evidence or a clarified contract. Closed Actions remain audit history, are never acceptance evidence, and do not block completion.
@@ -290,6 +292,13 @@ function cleanText(value, limit, name) {
   return text;
 }
 
+function coordinatorDisplayTitle(value, detail) {
+  if (detail.titleSource !== 'coordinator_pending') return null;
+  const proposed = typeof value === 'string' ? value.trim() : '';
+  const fallback = String(detail.goal || detail.title || 'Work Item').trim().replace(/\s+/g, ' ');
+  return (proposed || fallback || 'Work Item').slice(0, 200);
+}
+
 function requiresDeliveryBoundaryDecision(detail, actions) {
   const requested = Array.isArray(actions) ? actions : [];
   return !detail?.deliveryTarget && requested.some(action => (
@@ -460,8 +469,9 @@ export function normalizeCoordinatorResponse(value, detail, options = {}) {
   const kind = allowedKinds.includes(source.kind) ? source.kind : '';
   if (!kind) throw new Error('Work Center Coordinator decision kind is invalid');
   const reason = cleanText(source.reason, 2_000, 'decision reason');
+  const generatedTitle = coordinatorDisplayTitle(source.title, detail);
   if (kind === 'answer') {
-    return { reply, decision: { kind, reason, contractPatch: null, guidance: [], actions: [] } };
+    return { reply, decision: { kind, reason, title: generatedTitle, contractPatch: null, guidance: [], actions: [] } };
   }
   if (kind === 'guide_actions') {
     const guidance = normalizeGuidance(source.guidance, detail);
@@ -478,6 +488,7 @@ export function normalizeCoordinatorResponse(value, detail, options = {}) {
       decision: {
         kind,
         reason,
+        title: generatedTitle,
         contractPatch: null,
         guidance,
         actions: [],
@@ -491,6 +502,7 @@ export function normalizeCoordinatorResponse(value, detail, options = {}) {
       decision: {
         kind,
         reason,
+        title: generatedTitle,
         question: cleanText(source.question, COORDINATOR_MAX_REPLY_CHARS, 'human question'),
         contractPatch: dynamic && options.automatic !== true ? contractPatch : null,
         guidance: [],
@@ -504,6 +516,7 @@ export function normalizeCoordinatorResponse(value, detail, options = {}) {
       decision: {
         kind,
         reason,
+        title: generatedTitle,
         closeActions: normalizeDynamicActionClosures(source.closeActions, detail.actions || []),
         completion: source.completion,
         contractPatch: null,
@@ -520,6 +533,7 @@ export function normalizeCoordinatorResponse(value, detail, options = {}) {
     const decision = {
       kind,
       reason,
+      title: generatedTitle,
       workItemType: source.workItemType,
       contractPatch,
       closeActions: source.closeActions,
@@ -551,6 +565,7 @@ export function normalizeCoordinatorResponse(value, detail, options = {}) {
     decision: {
       kind,
       reason,
+      title: generatedTitle,
       contractPatch,
       guidance: [],
       actions: normalizeCoordinatorActionReferences(source.actions, detail),
@@ -616,6 +631,7 @@ export function coordinatorSnapshot(detail) {
     ledgerRevision: detail.ledgerRevision,
     status: truncateUtf8(detail.status, 64),
     title: truncateUtf8(detail.title, 1 * 1024),
+    titleSource: detail.titleSource || 'explicit',
     goal: truncateUtf8(detail.goal, 4 * 1024),
     deliveryTarget: detail.deliveryTarget || null,
     acceptanceCriteria,

package/yeaft/work-center/durable-model.js

@@ -1,6 +1,6 @@
 import { createHash, randomUUID } from 'node:crypto';
 
-export const WORK_CENTER_SCHEMA_VERSION = 39;
+export const WORK_CENTER_SCHEMA_VERSION = 40;
 
 const MIGRATIONS = [
   ['23-conversation-stream', migrateConversationStream],

package/yeaft/work-center/projection.js

@@ -793,18 +793,39 @@ function enforceWorkItemBrowserDtoBudget(value, options = {}) {
   if (jsonByteLength(dto) <= MAX_WORK_ITEM_BROWSER_DTO_BYTES) return dto;
 
   const originalCount = actions.length;
-  const retained = keep ? [stripActionBody(keep, true)] : [];
+  // Detail evidence and attempt links need the Action identity AND its readable
+  // name. Drop bulky context/dependencies before dropping their navigation
+  // targets. If even these stubs cannot fit, the minimal fallback below omits
+  // evidence with the Actions instead of leaving known sources unresolvable.
+  const retained = Array.isArray(workItem.runReferences) && Array.isArray(workItem.actions)
+    ? actions.map(action => ({
+        id: action.id,
+        sequence: action.sequence,
+        generation: action.generation,
+        type: action.type,
+        stageId: action.stageId,
+        status: action.status,
+        progressRevision: action.progressRevision,
+        messageCount: action.messageCount,
+        createdAt: action.createdAt,
+        updatedAt: action.updatedAt,
+        executionDurationMs: action.executionDurationMs,
+        executionStartedAt: action.executionStartedAt,
+        brief: { objective: truncateUtf8(action.brief?.objective || action.contentSummary || '', 512) },
+      }))
+    : keep ? [stripActionBody(keep, true)] : [];
   if (Array.isArray(workItem.actions)) workItem.actions = retained;
   else workItem.actionStats = retained;
   workItem.omittedActionCount = originalCount - retained.length;
   if (jsonByteLength(dto) <= MAX_WORK_ITEM_BROWSER_DTO_BYTES) return dto;
 
-  if (retained[0]) {
+  if (retained[0] && !Array.isArray(workItem.runReferences)) {
     delete retained[0].brief;
     delete retained[0].failure;
   }
   workItem.title = truncateUtf8(workItem.title, 4 * 1024);
   workItem.goal = truncateUtf8(workItem.goal, 4 * 1024);
+  if (workItem.requirement !== undefined) workItem.requirement = truncateUtf8(workItem.requirement, 4 * 1024);
   workItem.waitingReason = truncateUtf8(workItem.waitingReason, 4 * 1024);
   workItem.actionSummary = truncateUtf8(workItem.actionSummary, 4 * 1024);
   if (Array.isArray(workItem.acceptanceCriteria)) workItem.acceptanceCriteria = [];
@@ -817,6 +838,7 @@ function enforceWorkItemBrowserDtoBudget(value, options = {}) {
     revision: count(workItem.revision),
     title: truncateUtf8(workItem.title, 4 * 1024),
     goal: truncateUtf8(workItem.goal, 4 * 1024),
+    ...(workItem.requirement !== undefined ? { requirement: truncateUtf8(workItem.requirement, 4 * 1024) } : {}),
     status: truncateUtf8(workItem.status, 256),
     currentActionId: truncateUtf8(workItem.currentActionId, 4 * 1024) || null,
     executionStats: workItem.executionStats,
@@ -1054,6 +1076,7 @@ export function projectWorkItemDetail(detail, options = {}) {
           .map(risk => truncateUtf8(risk, MAX_ACTION_MESSAGE_CHARS)).slice(0, 24) : [],
     } : null,
     title: detail.title,
+    requirement: detail.requirement ?? detail.goal,
     goal: detail.goal,
     acceptanceCriteria: Array.isArray(detail.acceptanceCriteria) ? detail.acceptanceCriteria : [],
     goalProgress: projectGoalProgress(detail.goalProgress),
@@ -1127,6 +1150,25 @@ export function projectWorkItemDetail(detail, options = {}) {
         }))
       : [],
   };
+  // Only identities needed by visible evidence links, never Run bodies or traces.
+  // Historical evidence may belong to an older generation of the same Action.
+  const evidenceRunIds = new Set([
+    ...(projected.goalProgress?.evidenceRunIds || []),
+    ...(projected.goalProgress?.criteria || []).flatMap(check => check.evidenceRunIds || []),
+    ...(projected.goalProgress?.delivery?.evidenceRunIds || []),
+    ...(projected.finalResult?.responses || []).map(response => response.runId),
+    ...projected.outputs.map(output => output.runId),
+  ].filter(Boolean));
+  const actionIds = new Set(projected.actions.map(action => action.id));
+  // The evidence collections above already have bounded browser projections.
+  // Keep every retained reference resolvable; an independent count cap would
+  // starve later criteria and delivery evidence. The whole DTO budget still applies.
+  projected.runReferences = [...evidenceRunIds].flatMap(id => {
+    const run = runById.get(id);
+    if (!run || !actionIds.has(run.actionId)
+        || (run.workItemId && run.workItemId !== detail.id)) return [];
+    return [{ id: truncateUtf8(id, 256), actionId: truncateUtf8(run.actionId, 256) }];
+  });
   return enforceWorkItemBrowserDtoBudget(projected, { keepActionId: liveActionId });
 }
 

package/yeaft/work-center/service.js

@@ -248,11 +248,16 @@ export class WorkCenterService {
           });
           const shouldStart = payload.start === undefined ? settings.startImmediately : payload.start !== false;
           const goal = requiredString(payload.goal, 'goal');
+          const explicitTitle = payload.titleSource !== 'coordinator_pending' && typeof payload.title === 'string'
+            ? payload.title.trim() : '';
           const requestedCriteria = Array.isArray(payload.acceptanceCriteria)
             ? payload.acceptanceCriteria.map(value => String(value).trim()).filter(Boolean) : [];
           this.controller.create({
             id: workItemId,
-            title: requiredString(payload.title, 'title'),
+            // Goal remains the original requirement. Until coordination succeeds,
+            // title is only a backwards-compatible display fallback.
+            title: explicitTitle || goal.replace(/\s+/g, ' ').slice(0, 80),
+            titleSource: explicitTitle ? 'explicit' : 'coordinator_pending',
             goal,
             // With no separate criteria, the user's goal itself is the minimum
             // contract. Do not force a follow-up or invent broader requirements.

package/yeaft/work-center/store.js

@@ -113,6 +113,8 @@ function mapWorkItem(row) {
     finalResult: parseJson(row.final_result, null),
     deliveryTarget: row.delivery_target || null,
     title: row.title,
+    titleSource: row.title_source || 'explicit',
+    requirement: row.requirement ?? row.goal,
     goal: row.goal,
     acceptanceCriteria: parseJson(row.acceptance_criteria, []),
     workflowTemplate: row.workflow_template,
@@ -857,6 +859,8 @@ export class WorkItemStore {
         final_result TEXT,
         delivery_target TEXT,
         title TEXT NOT NULL,
+        title_source TEXT NOT NULL DEFAULT 'explicit',
+        requirement TEXT,
         goal TEXT NOT NULL,
         acceptance_criteria TEXT NOT NULL,
         workflow_template TEXT NOT NULL,
@@ -1139,6 +1143,15 @@ export class WorkItemStore {
     if (!hasColumn(this.db, 'work_items', 'delivery_target')) {
       this.db.exec('ALTER TABLE work_items ADD COLUMN delivery_target TEXT');
     }
+    if (!hasColumn(this.db, 'work_items', 'title_source')) {
+      // Old titles were explicit under the previous creation contract.
+      this.db.exec("ALTER TABLE work_items ADD COLUMN title_source TEXT NOT NULL DEFAULT 'explicit'");
+    }
+    if (!hasColumn(this.db, 'work_items', 'requirement')) {
+      // Old items cannot reconstruct an earlier goal; retain their current contract.
+      this.db.exec('ALTER TABLE work_items ADD COLUMN requirement TEXT');
+      this.db.exec('UPDATE work_items SET requirement = goal');
+    }
     if (!hasColumn(this.db, 'actions', 'source_action_ids')) {
       this.db.exec("ALTER TABLE actions ADD COLUMN source_action_ids TEXT NOT NULL DEFAULT '[]'");
     }
@@ -2387,15 +2400,17 @@ export class WorkItemStore {
       const workspaceKey = canonicalWorkspaceKey(input.workDir);
       this.db.prepare(`INSERT INTO work_items
         (id, revision, execution_schema_version, ledger_revision, coordination_mode, final_result, delivery_target,
-         title, goal, acceptance_criteria, workflow_template, workflow_snapshot, status,
+         title, title_source, requirement, goal, acceptance_criteria, workflow_template, workflow_snapshot, status,
          current_action_id, current_run_id, work_dir, workspace_key, reuse_memory, origin, linked_session_ids,
          session_context, attachments, created_at, updated_at)
-        VALUES (?, 1, ?, 0, ?, NULL, ?, ?, ?, ?, ?, ?, ?, NULL, NULL, ?, ?, ?, ?, ?, ?, ?, ?, ?)`).run(
+        VALUES (?, 1, ?, 0, ?, NULL, ?, ?, ?, ?, ?, ?, ?, ?, ?, NULL, NULL, ?, ?, ?, ?, ?, ?, ?, ?, ?)`).run(
         id,
         Number.isInteger(input.executionSchemaVersion) ? input.executionSchemaVersion : 2,
         input.coordinationMode || 'legacy',
         input.deliveryTarget || null,
         input.title,
+        input.titleSource === 'coordinator_pending' ? 'coordinator_pending' : 'explicit',
+        input.goal,
         input.goal,
         stringify(input.acceptanceCriteria || []),
         input.workflowTemplate || 'software-change',
@@ -3759,6 +3774,19 @@ export class WorkItemStore {
     turnId, result, expected, workItem, messages, assistantIndex, activeActions, now,
   }) {
     const decision = result?.decision || {};
+    const generatedTitle = workItem.titleSource === 'coordinator_pending'
+      ? (typeof decision.title === 'string' && decision.title.trim()
+        ? decision.title.trim().slice(0, 200)
+        : String(workItem.goal || workItem.title || 'Work Item').trim().replace(/\s+/g, ' ').slice(0, 200))
+      : null;
+    if (generatedTitle) {
+      const changedTitle = this.db.prepare(`UPDATE work_items SET title = ?, title_source = 'coordinator',
+        updated_at = ? WHERE id = ? AND title_source = 'coordinator_pending'`).run(
+        generatedTitle, now, workItem.id,
+      );
+      if (Number(changedTitle.changes) !== 1) throw new Error('Coordinator title generation lost its fence');
+      workItem = this.getWorkItem(workItem.id);
+    }
     const decisionPatch = normalizeContractPatch(decision.contractPatch);
     const mutationPatch = normalizeContractPatch(result?.mutation?.contractPatch);
     if (JSON.stringify(decisionPatch) !== JSON.stringify(mutationPatch) && mutationPatch) {
@@ -3783,17 +3811,23 @@ export class WorkItemStore {
       throw new Error('WorkItem delivery target must be confirmed before creating mutating or delivery Actions');
     }
     const refined = { ...workItem, ...(contractPatch || {}) };
+    const hasExplicitTitle = Object.hasOwn(contractPatch || {}, 'title');
     const contractChanged = ['title', 'goal', 'deliveryTarget', 'acceptanceCriteria']
       .some(key => JSON.stringify(refined[key]) !== JSON.stringify(workItem[key]));
     if (contractChanged) {
       this.#invalidateExecution(workItem, 'superseded', 'superseded', 'User refined the WorkItem contract', now);
       this.db.prepare(`UPDATE work_items SET title = ?, goal = ?, acceptance_criteria = ?,
-        delivery_target = ?, revision = revision + 1, updated_at = ? WHERE id = ? AND revision = ?`).run(
+        delivery_target = ?, title_source = CASE WHEN ? THEN 'explicit' ELSE title_source END,
+        revision = revision + 1, updated_at = ? WHERE id = ? AND revision = ?`).run(
         refined.title, refined.goal, stringify(refined.acceptanceCriteria), refined.deliveryTarget,
-        now, workItem.id, workItem.revision,
+        hasExplicitTitle ? 1 : 0, now, workItem.id, workItem.revision,
       );
       workItem = this.getWorkItem(workItem.id);
       activeActions = [];
+    } else if (hasExplicitTitle && workItem.titleSource !== 'explicit') {
+      this.db.prepare(`UPDATE work_items SET title_source = 'explicit', updated_at = ?
+        WHERE id = ? AND revision = ?`).run(now, workItem.id, workItem.revision);
+      workItem = this.getWorkItem(workItem.id);
     }
 
     if (decision.kind === 'create_actions') {