@yeaft/webchat-agent 1.0.514 → 1.0.516

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "1.0.514",
+  "version": "1.0.516",
   "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/engine.js

@@ -1250,8 +1250,8 @@ export class Engine {
   }
 
   /**
-   * Build the system prompt with the AMS-rendered Memory block, the
-   * Active Scope block, and skill content. The legacy multi-path
+   * Build the system prompt with the AMS-rendered Memory block, routing
+   * metadata, and skill content. The legacy multi-path
    * Memory injection (FTS-formatted + AMS snapshot + Layer-A summaries +
    * userProfile + coreMemory) was retired in DESIGN-PROMPT v1; callers
    * now thread a single `memoryInjection` string composed upstream from
@@ -1260,8 +1260,7 @@ export class Engine {
    * Routes through `buildWorkerPrompt`, which:
    *   - Lays in the persona-as-identity block (or Yeaft identity fallback)
    *   - Adds the Memory section (passed in as `memoryInjection`)
-   *   - Adds the structured Active Scope block (`activeScope`)
-   *   - Forwards optional `taskCtx` for the legacy task-context sub-block
+   *   - Adds multi-VP routing metadata from `activeScope` when applicable
    *
    * @param {object} args
    * @param {string} args.prompt — user prompt (for skill relevance matching)
@@ -1273,11 +1272,10 @@ export class Engine {
    * @param {string} [args.projectLabel] — current Project name and id for prompt attribution
    * @param {string} [args.workCenterInstructions] — frozen Agent-level Work Center policy
    * @param {string} [args.projectDoc] — resolved CLAUDE.md / AGENTS.md text (already truncated)
-   * @param {object} [args.taskCtx] — legacy task-context sub-block (optional)
    * @param {string} [args.explicitSkillName] — leading /skill:<name> command, if present
    * @returns {string}
    */
-  #buildSystemPrompt({ prompt, memoryInjection, vpPersona, activeScope, sessionAnnouncement, projectInstruction, projectLabel, workCenterInstructions, projectDoc, taskCtx, activeTasks, collabToolPolicy = null, activeToolNames = null, promptNotices = [], explicitSkillName, resolvedSkillContent = null } = {}) {
+  #buildSystemPrompt({ prompt, memoryInjection, vpPersona, activeScope, sessionAnnouncement, projectInstruction, projectLabel, workCenterInstructions, projectDoc, promptNotices = [], explicitSkillName, resolvedSkillContent = null } = {}) {
     // #runQuery resolves Skill selection at each provider-request boundary so
     // a live Plugin policy change cannot leave stale content in the next prompt.
     // Keep the local fallback for internal callers that do not need selection
@@ -1292,21 +1290,8 @@ export class Engine {
       }
     }
 
-    // Prompt guidance must describe the same canonical capability
-    // intersection that reaches provider schemas and execution.
-    const registeredToolNames = this.#toolRegistry
-      ? this.#toolRegistry.getToolNames({
-          plugins: this.#config?.plugins,
-          collabToolPolicy,
-        })
-      : Array.from(this.#tools.keys());
-    const toolNames = activeToolNames instanceof Set
-      ? registeredToolNames.filter(name => activeToolNames.has(name))
-      : registeredToolNames;
-
     return buildWorkerPrompt({
       language: this.#config.language || 'en',
-      toolNames,
       memoryInjection,
       skillContent,
       vpPersona,
@@ -1317,8 +1302,6 @@ export class Engine {
       workCenterInstructions,
       projectDoc,
       runtimePlatform: getRuntimePlatformInfo(),
-      taskCtx,
-      activeTasks,
       promptNotices,
       // Worker-shape harness is descriptive metadata for human inspection;
       // production prompts skip it to save tokens. Re-enable via env when
@@ -1453,19 +1436,14 @@ export class Engine {
       // VP-aware tool). Undefined when running in non-group / no-VP flows.
       router: vpCtx?.router,
       senderVpId: vpCtx?.senderVpId,
-      // Active VP persona — surfaced so tools like `StartPlan` can read
-      // the optional `planInstruction` override without re-reading
-      // role.md. Mirrors the symmetry already present in
-      // `parentEngineDeps.parentVpPersona` below — sub-agents inherit it
-      // through the parent deps; tools at this level read it directly.
+      // Active VP persona for VP-aware tools and inherited sub-agent context.
       // Null in non-VP / test contexts.
       vpPersona: vpCtx?.vpPersona || null,
       inboundEnvelope: vpCtx?.inboundEnvelope,
       taskId: vpCtx?.taskId,
       taskMembers: vpCtx?.taskMembers,
-      // TodoWrite per-VP cache hooks. Threaded from web-bridge so each
-      // VP keeps its own todo list (see todo-write.js, web-bridge.js).
-      // Null in non-VP / test contexts — tools tolerate missing slots.
+      // Legacy todo cache hooks retained for embedding callers and replay.
+      // Native checklist tools are no longer registered.
       getCurrentTodos: vpCtx?.getCurrentTodos || null,
       setCurrentTodos: vpCtx?.setCurrentTodos || null,
       askUser: typeof vpCtx?.askUser === 'function'
@@ -2306,18 +2284,13 @@ export class Engine {
         }))
       : [];
 
-    // ─── Active Scope (DESIGN-PROMPT §3 ④) ──────────────────────
-    // Structured per-turn scope summary: session + vp + members + envelope routing
-    // info. Long-form scope content lives in AMS — this block carries
-    // only IDs + tiny labels. (Feature scope retired 2026-05-13.)
-    const activeSessionTopics = topicScopesForResident
-      .map(scope => scope.replace(/^sessions\/[^/]+\/topic\//, ''));
+    // ─── Multi-VP routing scope ──────────────────────────────────
+    // Only identity and peer membership affect the routing contract rendered in
+    // the system prompt. Session bookkeeping and inferred topics remain in their
+    // owning runtime/memory paths instead of being mirrored into prompt metadata.
     const activeScope = {
-      sessionId: sessionId || '',
-      sessionMember: ownVpIdForAms || '',
+      vpId: ownVpIdForAms || '',
       sessionMembers: Array.isArray(sessionMembers) ? sessionMembers : [],
-      sessionTopics: activeSessionTopics,
-      envelope: inboundEnvelope || null,
     };
 
     let projectDocSource = this.#getProjectDocBlock(workDir);
@@ -2332,11 +2305,6 @@ export class Engine {
       && typeof this.#taskManager.listActiveTasks === 'function'
       ? this.#taskManager.listActiveTasks(runtimeSessionId)
       : [];
-    let activeTasks = this.#taskManager
-      ? this.#taskManager.renderActiveTasksForPrompt(runtimeSessionId, {
-          language: this.#config.language || 'en',
-        })
-      : '';
     const registeredToolNames = this.#toolRegistry
       ? this.#toolRegistry.getToolNames({
           collabToolPolicy: effectiveCollabToolPolicy,
@@ -2407,9 +2375,6 @@ export class Engine {
       projectLabel,
       workCenterInstructions,
       projectDoc: projectDocContext.text,
-      activeTasks,
-      collabToolPolicy: effectiveCollabToolPolicy,
-      activeToolNames,
       promptNotices,
       explicitSkillName,
       resolvedSkillContent,
@@ -2603,12 +2568,6 @@ export class Engine {
     // control to other VPs cleanly. Reset to null at the top of every
     // outer-loop iteration so the flag never carries across turns.
     let endTurnRequested = null;
-    // StartPlan is a control tool. If the model emits only the checklist after
-    // it, there is no new workspace fact to interpret: persist the plan and
-    // close the turn instead of spending another provider request on a
-    // TodoWrite-only control round. A later user turn can continue the first
-    // pending step; a batch that includes a real work tool always continues.
-    let planBootstrapPending = false;
 
     // LLM retry bookkeeping (rate-limit / 5xx / transient network errors).
     // Counts CONSECUTIVE retryable failures on the same turn — reset to 0
@@ -2825,11 +2784,6 @@ export class Engine {
         && typeof this.#taskManager.listActiveTasks === 'function'
         ? this.#taskManager.listActiveTasks(runtimeSessionId)
         : [];
-      activeTasks = this.#taskManager
-        ? this.#taskManager.renderActiveTasksForPrompt(runtimeSessionId, {
-            language: this.#config.language || 'en',
-          })
-        : '';
       activeToolNames = resolveCurrentActiveToolNames();
       for (const name of [...discoveredToolNames]) {
         if (this.#toolRegistry?.has(name)) activeToolNames?.add(name);
@@ -4686,9 +4640,6 @@ export class Engine {
           reusedCallId: reusedReadOnlyCallId,
         });
 
-        if (!skipped && !duplicateCallSuppressed && !reusedReadOnlyResult && tc.name === 'StartPlan') {
-          planBootstrapPending = true;
-        }
         if (!skipped && !duplicateCallSuppressed && !reusedReadOnlyResult
             && !readOnlyToolReuseDisabled && cacheableTool) {
           readOnlyToolResults.set(duplicateKey, {
@@ -4783,36 +4734,6 @@ export class Engine {
         break;
       }
 
-      // A plan bootstrap that produced only a TodoWrite has no executable work
-      // to feed back to the provider. Close it here. This is intentionally
-      // narrow: StartPlan + TodoWrite is a valid planning result, but a batch
-      // containing any other tool must continue so the model can inspect its
-      // result before deciding what to do next.
-      const onlyPlanControls = planBootstrapPending
-        && toolCalls.length > 0
-        && toolCalls.every(call => call.name === 'StartPlan' || call.name === 'TodoWrite')
-        && toolCalls.some(call => call.name === 'TodoWrite')
-        && !toolBatchBarrier
-        && !endTurnRequested
-        && !abortedDuringTools
-        && !signal?.aborted;
-      if (onlyPlanControls) {
-        const hasPendingStep = toolCalls
-          .filter(call => call.name === 'TodoWrite')
-          .some(call => Array.isArray(call.input?.todos)
-            && call.input.todos.some(todo => todo?.status === 'pending' || todo?.status === 'in_progress'));
-        yield {
-          type: 'turn_end',
-          turnNumber,
-          stopReason: 'plan_recorded',
-          detail: { nextStep: hasPendingStep ? 'pending_work_tools' : 'none', toolCount: toolCalls.length },
-          threadId,
-          terminal: true,
-        };
-        break;
-      }
-      planBootstrapPending = false;
-
       // A batch barrier deliberately returns control to the provider. Any
       // handoff requested by an earlier call belongs to the invalidated plan
       // and must not leak into a later provider-generated batch.