@yeaft/webchat-agent 1.0.201 → 1.0.202

package/local-runtime/version.json

@@ -1 +1 @@
-{"version":"1.0.201"}
+{"version":"1.0.202"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "1.0.201",
+  "version": "1.0.202",
   "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/work-center/completion-contract.js

@@ -0,0 +1,53 @@
+function normalizeCriteria(value) {
+  if (!Array.isArray(value)) return null;
+  return value.map(item => String(item).trim()).filter(Boolean);
+}
+
+export function normalizeContractPatch(value) {
+  if (!value || typeof value !== 'object' || Array.isArray(value)) return null;
+  const patch = {};
+  if (typeof value.goal === 'string' && value.goal.trim()) patch.goal = value.goal.trim();
+  if (Object.hasOwn(value, 'acceptanceCriteria')) {
+    const criteria = normalizeCriteria(value.acceptanceCriteria);
+    if (!criteria) throw new Error('contractPatch.acceptanceCriteria must be an array');
+    patch.acceptanceCriteria = criteria;
+  }
+  return Object.keys(patch).length > 0 ? patch : null;
+}
+
+function normalizeAcceptanceChecks(value, criteria) {
+  if (!Array.isArray(value) || value.length !== criteria.length) return null;
+  const checks = value.map((raw, index) => {
+    if (!raw || typeof raw !== 'object' || Array.isArray(raw)) return null;
+    const criterion = typeof raw.criterion === 'string' ? raw.criterion.trim() : '';
+    const status = ['passed', 'deferred', 'not_applicable'].includes(raw.status) ? raw.status : '';
+    const evidence = typeof raw.evidence === 'string' ? raw.evidence.trim().slice(0, 1_000) : '';
+    if (criterion !== criteria[index] || !status || !evidence) return null;
+    return { criterion, status, evidence };
+  });
+  return checks.every(Boolean) ? checks : null;
+}
+
+export function validateCompletedResult(result, action, workItem) {
+  if (result.outcome !== 'completed') return;
+  if (result.evidence.length === 0) {
+    result.outcome = 'failed';
+    result.error = 'Completed Action requires at least one concrete evidence item';
+    return;
+  }
+  const criteria = result.contractPatch?.acceptanceCriteria
+    ?? (Array.isArray(workItem.acceptanceCriteria) ? workItem.acceptanceCriteria : []);
+  const checks = normalizeAcceptanceChecks(result.acceptanceChecks, criteria);
+  if (!checks) {
+    result.outcome = 'failed';
+    result.error = 'Completed Action requires one ordered acceptance check with evidence for every acceptance criterion';
+    return;
+  }
+  const mustVerify = action.type === 'test'
+    || action.type === 'deliver'
+    || (action.type === 'review' && result.reviewDecision === 'approved');
+  if (mustVerify && checks.some(check => check.status !== 'passed')) {
+    result.outcome = 'failed';
+    result.error = `${action.type} Action requires every acceptance check to pass`;
+  }
+}

package/yeaft/work-center/controller.js

@@ -9,60 +9,7 @@ import {
 import { renderSessionContextSnapshot } from './session-context.js';
 import { normalizeEvidence } from './evidence.js';
 import { applyAdditivePlanProposal } from './plan-mutation.js';
-
-function normalizeCriteria(value) {
-  if (!Array.isArray(value)) return null;
-  return value.map(item => String(item).trim()).filter(Boolean);
-}
-
-function normalizeContractPatch(value) {
-  if (!value || typeof value !== 'object' || Array.isArray(value)) return null;
-  const patch = {};
-  if (typeof value.goal === 'string' && value.goal.trim()) patch.goal = value.goal.trim();
-  if (Object.prototype.hasOwnProperty.call(value, 'acceptanceCriteria')) {
-    const criteria = normalizeCriteria(value.acceptanceCriteria);
-    if (!criteria) throw new Error('contractPatch.acceptanceCriteria must be an array');
-    patch.acceptanceCriteria = criteria;
-  }
-  return Object.keys(patch).length > 0 ? patch : null;
-}
-
-function normalizeAcceptanceChecks(value, criteria) {
-  if (!Array.isArray(value) || value.length !== criteria.length) return null;
-  const checks = value.map((raw, index) => {
-    if (!raw || typeof raw !== 'object' || Array.isArray(raw)) return null;
-    const criterion = typeof raw.criterion === 'string' ? raw.criterion.trim() : '';
-    const status = ['passed', 'deferred', 'not_applicable'].includes(raw.status) ? raw.status : '';
-    const evidence = typeof raw.evidence === 'string' ? raw.evidence.trim().slice(0, 1_000) : '';
-    if (criterion !== criteria[index] || !status || !evidence) return null;
-    return { criterion, status, evidence };
-  });
-  return checks.every(Boolean) ? checks : null;
-}
-
-function validateCompletedResult(result, action, workItem) {
-  if (result.outcome !== 'completed') return;
-  if (result.evidence.length === 0) {
-    result.outcome = 'failed';
-    result.error = 'Completed Action requires at least one concrete evidence item';
-    return;
-  }
-  const criteria = result.contractPatch?.acceptanceCriteria
-    ?? (Array.isArray(workItem.acceptanceCriteria) ? workItem.acceptanceCriteria : []);
-  const checks = normalizeAcceptanceChecks(result.acceptanceChecks, criteria);
-  if (!checks) {
-    result.outcome = 'failed';
-    result.error = 'Completed Action requires one ordered acceptance check with evidence for every acceptance criterion';
-    return;
-  }
-  const mustVerify = action.type === 'test'
-    || action.type === 'deliver'
-    || (action.type === 'review' && result.reviewDecision === 'approved');
-  if (mustVerify && checks.some(check => check.status !== 'passed')) {
-    result.outcome = 'failed';
-    result.error = `${action.type} Action requires every acceptance check to pass`;
-  }
-}
+import { normalizeContractPatch, validateCompletedResult } from './completion-contract.js';
 
 function normalizeTerminalResult(result, action) {
   if (!result || !RUN_OUTCOMES.includes(result.outcome)) {

package/yeaft/work-center/runner.js

@@ -31,7 +31,9 @@ import { loadMCPConfig } from '../config.js';
 import { MCPManager } from '../mcp.js';
 import { buildMcpFlattenedTools } from '../tools/mcp-tools.js';
 import { recallWorkspaceSessionContext } from './workspace-context.js';
-import { BUILT_IN_ACTION_TYPES } from './workflow.js';
+import { applyGeneratedPlan, BUILT_IN_ACTION_TYPES } from './workflow.js';
+import { normalizeContractPatch, validateCompletedResult } from './completion-contract.js';
+import { normalizeEvidence } from './evidence.js';
 import {
   MAINLINE_CONTEXT_HARD_LIMIT_BYTES,
   buildMainlineContextSnapshot,
@@ -300,14 +302,20 @@ export function planningVpCatalog(vps) {
   }));
 }
 
-export function createSubmitWorkItemPlanTool({ vps, collector, isRunActive }) {
+export function createSubmitWorkItemPlanTool({
+  vps,
+  workItem,
+  collector,
+  isRunActive,
+  reservedStageIds = [],
+}) {
   const vpCatalog = planningVpCatalog(vps);
   const vpIds = vpCatalog.map(vp => vp.id);
   const actionTypes = BUILT_IN_ACTION_TYPES.filter(type => type !== 'triage');
   const catalogDescription = `Action types: ${actionTypes.join(', ')}. Available VPs: ${vpCatalog.map(vp => `${vp.id} (${vp.role || vp.area || 'VP'}; ${vp.traits.join(', ') || 'no traits'})`).join('; ')}.`;
   return defineTool({
     name: 'SubmitWorkItemPlan',
-    description: `Submit the complete initial WorkItem contract and executable Action DAG. Every Action must describe this WorkItem's concrete objective, repository-aware approach, and verifiable expected outcome; never copy generic Action-type text. The reference workflow catalog does not replace this Action list. This tool records a Run-local proposal only; Work Center validates and persists it in the current Run finalization transaction. ${catalogDescription}`,
+    description: `Submit the complete initial WorkItem contract and executable Action DAG. Every Action must describe this WorkItem's concrete objective, repository-aware approach, and verifiable expected outcome; never copy generic Action-type text. The reference workflow catalog does not replace this Action list. If any Action uses isolated-write workspace mode, the plan must contain exactly one integrate Action in integrate workspace mode; that Action must depend directly on every isolated-write Action, and all later Actions must consume those writes through it. This tool validates the proposal immediately so you can correct an invalid graph in the same triage loop; Work Center persists only a valid proposal in the current Run finalization transaction. ${catalogDescription}`,
     parameters: {
       type: 'object',
       additionalProperties: false,
@@ -331,6 +339,28 @@ export function createSubmitWorkItemPlanTool({ vps, collector, isRunActive }) {
     async execute(input, ctx = {}) {
       if (!isRunActive()) throw new Error('Work Center Run is no longer active');
       if (collector.value) throw new Error('WorkItem plan was already submitted for this Run');
+      const contractPatch = normalizeContractPatch(input.contractPatch);
+      const proposedResult = {
+        outcome: 'completed',
+        evidence: normalizeEvidence(input.evidence),
+        contractPatch,
+        acceptanceChecks: input.acceptanceChecks,
+      };
+      validateCompletedResult(proposedResult, { type: 'triage' }, workItem);
+      if (proposedResult.outcome !== 'completed') throw new Error(proposedResult.error);
+      const effectiveWorkItem = contractPatch ? {
+        ...workItem,
+        goal: contractPatch.goal ?? workItem.goal,
+        acceptanceCriteria: contractPatch.acceptanceCriteria ?? workItem.acceptanceCriteria,
+      } : workItem;
+      applyGeneratedPlan(effectiveWorkItem, {
+        workItemType: input.workItemType,
+        actions: input.actions,
+      }, {
+        availableVpIds: vpIds,
+        reservedStageIds,
+      });
+      if (!isRunActive()) throw new Error('Work Center Run is no longer active');
       collector.value = structuredClone(input);
       ctx.requestEndTurn?.({ kind: 'work_item_plan_submitted' });
       return JSON.stringify({ submitted: true, actionCount: input.actions.length });
@@ -860,7 +890,13 @@ export class WorkItemRunner {
       && workItem?.workflowSnapshot?.planningMode === 'ai';
     const runTools = [];
     if (planToolEnabled) runTools.push(createSubmitWorkItemPlanTool({
-      vps: this.registry.listVps(), collector: planCollector, isRunActive,
+      vps: this.registry.listVps(),
+      workItem,
+      collector: planCollector,
+      isRunActive,
+      reservedStageIds: executionAction.stageId?.startsWith('replan-')
+        ? this.store.getWorkItemDetail(workItem.id).actions.map(item => item.stageId)
+        : [],
     }));
     if (!planToolEnabled && workItem?.workflowSnapshot?.executionMode === 'graph') {
       runTools.push(createProposeWorkItemActionsTool({

package/yeaft/work-center/workflow.js

@@ -373,7 +373,7 @@ export function resolvePlanningWorkflowSnapshot(settings, requestedWorkItemType
   const typeInstruction = requestedType
     ? `The user explicitly selected workItemType "${requestedType}". Keep that exact type.`
     : 'Infer one specific workItemType from the contract.';
-  const triageInstruction = `${normalized.actionInstructions.triage}\n\n${typeInstruction}\nReference workflow catalog:\n${catalog || '(none)'}\nUse the catalog only to understand established task categories and sequencing patterns. Always submit the smallest reliable graph of 1 to 8 task-specific Actions; never omit Actions or copy template brief text. Split independent work into separate Actions and declare dependsOnActionIds. Use workspaceMode read for analysis, isolated-write for independent Git changes, integrate for an integrate Action that combines isolated-write dependencies, and shared for serial side effects. Non-Git or dirty workspaces are serialized automatically. Every generated Action must state objective, approach, expectedOutcome, capability, dependencies, and workspaceMode. The objective, approach, and expectedOutcome must be specific to this WorkItem and that Action: describe the concrete work, the repository-aware execution method, and the verifiable result that will guide the executor. Generic Action-type boilerplate is invalid. Add only Actions required by this task. Do not copy a generic workflow.`;
+  const triageInstruction = `${normalized.actionInstructions.triage}\n\n${typeInstruction}\nReference workflow catalog:\n${catalog || '(none)'}\nUse the catalog only to understand established task categories and sequencing patterns. Always submit the smallest reliable graph of 1 to 8 task-specific Actions; never omit Actions or copy template brief text. Split independent work into separate Actions and declare dependsOnActionIds. Use workspaceMode read for analysis, isolated-write for independent Git changes, integrate for an integrate Action that combines isolated-write dependencies, and shared for serial side effects. If any Action uses isolated-write, add exactly one Action with type integrate and workspaceMode integrate; it must depend directly on every isolated-write Action, and all later Actions must depend on the integration result rather than an isolated-write Action. Non-Git or dirty workspaces are serialized automatically. Every generated Action must state objective, approach, expectedOutcome, capability, dependencies, and workspaceMode. The objective, approach, and expectedOutcome must be specific to this WorkItem and that Action: describe the concrete work, the repository-aware execution method, and the verifiable result that will guide the executor. Generic Action-type boilerplate is invalid. Add only Actions required by this task. Do not copy a generic workflow.`;
   return normalizeWorkflowDefinition({
     id: 'ai-planned',
     name: 'AI planned',