@yeaft/webchat-agent 0.1.847 → 0.1.849

package/package.json

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

package/unify/dream-v2/apply.js

@@ -228,7 +228,23 @@ export async function applyMergedTarget(merged, opts) {
   await writeMemory(scope, stamped, { root: opts.root });
   await writeSummary(scope, summaryMd || '', { root: opts.root, language: opts.language });
 
-  if (opts.onProgress) opts.onProgress({ phase: 'apply', target: merged.target, status: 'done', batches: batchesUsed });
+  if (opts.onProgress) {
+    // feat-dream-debug-detail: surface a truncated copy of what was
+    // actually written so the debug panel can show "what segments were
+    // generated" instead of just "done". The full bytes are on disk
+    // anyway — this preview is for at-a-glance debugging.
+    opts.onProgress({
+      phase: 'apply',
+      target: merged.target,
+      status: 'done',
+      batches: batchesUsed,
+      kind: merged.kind,
+      memoryMdPreview: truncateForDebug(stamped),
+      summaryMdPreview: truncateForDebug(summaryMd || ''),
+      memoryMdLength: (stamped || '').length,
+      summaryMdLength: (summaryMd || '').length,
+    });
+  }
   return { target: merged.target, kind: merged.kind, batches: batchesUsed };
 }
 
@@ -246,3 +262,23 @@ function scopeRelDir(scope) {
 }
 
 function oneLine(s) { return String(s || '').replace(/\s+/g, ' ').trim().slice(0, 200); }
+
+/**
+ * Per-field truncation cap for debug previews emitted on `apply/done`.
+ * Keep this small — the dream panel only needs a recognisable snippet.
+ * Total worst-case payload is `PREVIEW_MAX * 2 * targets_per_run` per
+ * dream pass; with N=50 targets that's ~200 KB. The full bytes are on
+ * disk under <root>/<scope>/{memory,summary}.md anyway — these previews
+ * are for at-a-glance debugging only.
+ */
+const PREVIEW_MAX = 2048;
+
+/**
+ * Truncate a markdown blob for inclusion in a debug-panel cell. Adds a
+ * "…(+N chars)" marker so the user knows it was cut.
+ */
+function truncateForDebug(s, max = PREVIEW_MAX) {
+  const str = String(s || '');
+  if (str.length <= max) return str;
+  return str.slice(0, max) + `…(+${str.length - max} chars)`;
+}

package/unify/dream-v2/session-wiring.js

@@ -7,6 +7,35 @@
  *
  * Memory v2 is the only path. The legacy `config.memoryV2` opt-out flag was
  * retired (task-710) — the wiring is unconditional.
+ *
+ * ─────────────────────────────────────────────────────────────────────
+ * Wire contract — DreamEvent (consumed by web/components/UnifyDebugPanel.js)
+ * ─────────────────────────────────────────────────────────────────────
+ * The events persisted to trace_events with these event_type values are
+ * load-bearing for the debug panel. Renaming a field on this side
+ * silently degrades that UI to a generic JSON dump.
+ *
+ *   dream_turn_open:   { type:'turn_open',  turnId, userPrompt, vpId, groupId, at }
+ *   dream_loop:        { type:'loop',       turnId, loopNumber, pass, model,
+ *                        systemPrompt: string,
+ *                        messages: [{ role:'user', content:string }],
+ *                        response: string,
+ *                        toolCalls: [], usage: { inputTokens, outputTokens, totalTokens },
+ *                        latencyMs, ttfbMs, stopReason, rawRequest, rawResponse }
+ *   dream_turn_close:  { type:'turn_close', turnId, totalMs, totalTokens, loopCount,
+ *                        metrics: { llmCallCount, inputTokens, outputTokens,
+ *                                   totalTokens, durationMs, passBreakdown:{[pass]:{
+ *                                     llmCallCount, inputTokens, outputTokens,
+ *                                     totalTokens, durationMs }} } }
+ *   dream_run:         { type:'dream_run',  turnId, phase:'result',
+ *                        status:'done'|'error', metrics, resultSummary:{ groups,
+ *                        targets, error, skipped, skippedReason } }
+ *   dream_progress:    runner-emitted phase events (`start`/`load-diff`/`triage`/
+ *                      `merge`/`apply`/`done`). The `apply/done` variant carries
+ *                      `kind, memoryMdPreview, summaryMdPreview, memoryMdLength,
+ *                      summaryMdLength` (see apply.js).
+ *
+ * `groupId` may be inherited via `stampDreamScope()` when a scope is active.
  */
 
 import { join } from 'path';

package/unify/groups/pre-flow.js

@@ -23,7 +23,7 @@
  */
 
 import { runPreflow as runFtsPreflow } from '../memory/preflow.js';
-import { isMember, resolveFallbackVp } from './roster.js';
+import { resolveFallbackVp, resolveMemberId } from './roster.js';
 
 /** Matches `@vp-id` where id is [A-Za-z0-9_-]+. Captures the id. */
 const MENTION_RE = /(^|\s)@([A-Za-z0-9_][A-Za-z0-9_-]*)/g;
@@ -115,15 +115,16 @@ export function selectRespondingVps(input) {
     const dispatched = [];
     const errors = [];
     for (const vpId of mentions) {
-      if (!isMember(meta, vpId)) {
+      const canonicalVpId = resolveMemberId(meta, vpId);
+      if (!canonicalVpId) {
         errors.push({ vpId, error: 'not_in_roster' });
         continue;
       }
-      if (taskMembers && !taskMembers.includes(vpId)) {
+      if (taskMembers && !taskMembers.includes(canonicalVpId)) {
         errors.push({ vpId, error: 'not_in_task_members' });
         continue;
       }
-      dispatched.push(vpId);
+      if (!dispatched.includes(canonicalVpId)) dispatched.push(canonicalVpId);
     }
     return { dispatched, fallback: null, errors, reason: 'mention' };
   }

package/unify/groups/roster.js

@@ -55,6 +55,33 @@ export function isMember(meta, vpId) {
   return meta.roster.includes(vpId);
 }
 
+/**
+ * Resolve a user/tool supplied VP target to its canonical roster id.
+ *
+ * The persisted roster stores canonical VP ids such as `linus`, while UI
+ * mentions and route_forward callers may still send the display-style
+ * `vp-linus` alias. Prefer exact roster ids first so a real `vp-foo` member
+ * is never collapsed to `foo`.
+ *
+ * @param {object} meta
+ * @param {string} target
+ * @returns {string|null}
+ */
+export function resolveMemberId(meta, target) {
+  if (!meta || !Array.isArray(meta.roster) || typeof target !== 'string') {
+    return null;
+  }
+  if (meta.roster.includes(target)) return target;
+
+  const prefix = 'vp-';
+  if (target.startsWith(prefix)) {
+    const unprefixed = target.slice(prefix.length);
+    if (meta.roster.includes(unprefixed)) return unprefixed;
+  }
+
+  return null;
+}
+
 /**
  * Resolve which VP should answer a message with no explicit @-mention.
  * Per architecture G2: defaultVpId if set, else roster[0], else null.

package/unify/routing/router.js

@@ -25,7 +25,7 @@
  *   (d) Tool schema uses defineTool (agent/unify/tools/types.js).
  */
 
-import { isMember } from '../groups/roster.js';
+import { resolveMemberId } from '../groups/roster.js';
 import { createLoopGuard, extendCausedBy } from './loop-guard.js';
 
 /**
@@ -86,19 +86,20 @@ export function createRouter(deps = {}) {
     if (typeof text !== 'string' || text.length === 0) {
       return { ok: false, error: 'text_required' };
     }
-    if (to === from) {
-      return { ok: false, error: 'self_forward_rejected' };
-    }
-
     const meta = coordinator.group.getMeta();
     if (!meta) return { ok: false, error: 'group_not_initialised' };
 
     // Roster membership — `all` is reserved broadcast sentinel handled by
-    // coordinator; anything else must be a real member so we fail fast with
-    // a VP-friendly error before hitting Coordinator.
-    if (to !== 'all' && !isMember(meta, to)) {
+    // coordinator; anything else must resolve to a real member so we fail fast
+    // with a VP-friendly error before hitting Coordinator. `vp-<id>` is a
+    // tolerated UI/tool alias for canonical roster ids such as `linus`.
+    const targetVpId = to === 'all' ? 'all' : resolveMemberId(meta, to);
+    if (targetVpId !== 'all' && !targetVpId) {
       return { ok: false, error: 'target_not_in_roster' };
     }
+    if (targetVpId === from) {
+      return { ok: false, error: 'self_forward_rejected' };
+    }
 
     // Build the causedBy chain BEFORE constructing the synthetic user-like
     // message. We don't know the new msgId yet (coordinator mints it on
@@ -110,7 +111,7 @@ export function createRouter(deps = {}) {
     // Loop guard: for broadcast, use 'all' as the target key so one VP
     // spamming @all still gets throttled even if each cycle hits different
     // member inboxes.
-    const guardKey = to === 'all' ? 'all' : to;
+    const guardKey = targetVpId;
     const verdict = guard.check({
       groupId: meta.id,
       targetVpId: guardKey,
@@ -131,9 +132,9 @@ export function createRouter(deps = {}) {
     // stamp + `synthetic` marker let Coordinator's `selectRespondingVps`
     // still treat this like a routed turn (target VPs need to respond) even
     // though role is now 'assistant'.
-    const injectText = to === 'all'
+    const injectText = targetVpId === 'all'
       ? `@all ${text}`
-      : `@${to} ${text}`;
+      : `@${targetVpId} ${text}`;
 
     const report = coordinator.ingest(
       {