@yeaft/webchat-agent 0.1.727 → 0.1.730

package/connection/message-router.js

@@ -36,7 +36,7 @@ import { sendToServer, flushMessageBuffer } from './buffer.js';
 import { handleRestartAgent, handleUpgradeAgent } from './upgrade.js';
 import { loadMcpServers, updateMcpConfig } from '../mcp.js';
 import { getLlmConfig, updateLlmConfig, getUnifySettings, updateUnifySettings, getSearchSettings, updateSearchSettings, fetchTavilyUsage } from '../unify/config-api.js';
-import { handleUnifyGroupChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyAbortTurn, handleUnifyVpSubscribe, handleUnifyVpCreate, handleUnifyVpUpdate, handleUnifyVpDelete, handleUnifyVpRead, handleUnifyFeatureMessage, handleUnifyFetchSummaryHistory, handleUnifyFeatureCrud, handleUnifyListGroups, handleUnifyCreateGroup, handleUnifyRenameGroup, handleUnifyUpdateGroup, handleUnifyArchiveGroup, handleUnifyDeleteGroup, handleUnifyAddMember, handleUnifyRemoveMember, handleUnifySetDefaultVp, handleUnifyDreamTrigger, broadcastLanguageChange } from '../unify/web-bridge.js';
+import { handleUnifyGroupChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyAbortTurn, handleUnifyVpSubscribe, handleUnifyVpCreate, handleUnifyVpUpdate, handleUnifyVpDelete, handleUnifyVpRead, handleUnifyFetchSummaryHistory, handleUnifyFeatureCrud, handleUnifyListGroups, handleUnifyCreateGroup, handleUnifyRenameGroup, handleUnifyUpdateGroup, handleUnifyArchiveGroup, handleUnifyDeleteGroup, handleUnifyAddMember, handleUnifyRemoveMember, handleUnifySetDefaultVp, handleUnifyDreamTrigger, broadcastLanguageChange } from '../unify/web-bridge.js';
 
 export async function handleMessage(msg) {
   switch (msg.type) {
@@ -456,13 +456,6 @@ export async function handleMessage(msg) {
       handleUnifyVpRead(msg);
       break;
 
-    // task-334h (R6 §Δ28 / §Δ31.6): feature-scoped direct message echo.
-    // Replaces the withdrawn R3 `unify_task_private_chat`. Agent validates,
-    // stamps msgId + ts, and broadcasts the `feature_message` mirror back.
-    case 'unify_feature_message':
-      handleUnifyFeatureMessage(msg);
-      break;
-
     // R6 G1a — feature summary history + feature affiliation CRUD.
     case 'unify_fetch_summary_history':
       await handleUnifyFetchSummaryHistory(msg);

package/package.json

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

package/unify/web-bridge.js

@@ -29,7 +29,6 @@ import { handleVpSubscribe } from './vp/vp-bridge.js';
 import { createVp, updateVp, deleteVp, readVp, VpCrudError } from './vp/vp-crud.js';
 import { scanVpLibrary } from './vp/vp-store.js';
 import { createRouter } from './routing/router.js';
-import { handleUnifyFeatureMessage as _handleUnifyFeatureMessage } from './feature-message.js';
 import {
   GroupCrudError,
   createGroupFromSpec,
@@ -637,10 +636,6 @@ export function handleUnifyVpDelete(msg) {
   }
 }
 
-export function handleUnifyFeatureMessage(msg) {
-  _handleUnifyFeatureMessage(msg, sendUnifyEvent);
-}
-
 export function handleUnifyVpRead(msg) {
   const requestId = msg && msg.requestId;
   const vpId = msg && msg.vpId;

package/unify/feature-message.js

@@ -1,144 +0,0 @@
-/**
- * feature-message.js — R6 §Δ28 / §Δ31.6 feature-scoped direct messaging.
- *
- * Replaces the withdrawn R3 `unify_feature_private_chat` event with a simple
- * echo-able `feature_message` pair:
- *
- *   inbound  (web → agent):  `unify_feature_message`
- *     { type, groupId, featureId, vpId, text, mentions?, replyTo?, requestId? }
- *   outbound (agent → web):  `feature_message`
- *     { type, groupId, featureId, vpId, msgId, text, mentions, replyTo,
- *       ts, requestId? }
- *
- * This module owns only the *wire adapter* — the payload is validated,
- * stamped with msgId + ts, and broadcast back so the sender's UI and any
- * other connected views converge on the same record. Persistence + feature
- * ACL enforcement are deliberately deferred to task-334l (per PM dispatch:
- * "user_memory_* 实际 ingestion 归 334l"); the parallel feature-private
- * storage hook follows the same phasing.
- *
- * Invariants:
- *   • Never throws on the WS hot path — bad payloads reply with a
- *     `feature_message_rejected` event carrying a stable `code` string for
- *     UI i18n (mirrors the vp_crud_result contract from 334-ui-g).
- *   • The outbound event field order and keys are considered wire-frozen
- *     per R6 §Δ31.6 table; additive fields only in future slices.
- */
-
-import { nextMsgId, isValidVpId } from './groups/ids.js';
-
-/** Known `reject` codes — kept stable so 334-ui-* can key i18n on them. */
-export const FEATURE_MESSAGE_REJECT_CODES = Object.freeze({
-  MISSING_GROUP_ID: 'missing_group_id',
-  MISSING_FEATURE_ID: 'missing_feature_id',
-  MISSING_VP_ID: 'missing_vp_id',
-  INVALID_VP_ID: 'invalid_vp_id',
-  EMPTY_TEXT: 'empty_text',
-  TEXT_TOO_LONG: 'text_too_long',
-});
-
-/** Soft body cap — matches the shard-entry cap used elsewhere in R6 (§Δ26.3). */
-export const MAX_TEXT_LENGTH = 16_384;
-
-/**
- * Pure validator. Returns `{ ok: true, payload }` or `{ ok: false, code }`.
- * No IO, no clock reads — safe to unit-test in isolation.
- *
- * @param {any} msg — raw WS message from the web client
- */
-export function validateFeatureMessage(msg) {
-  if (!msg || typeof msg !== 'object') {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.MISSING_GROUP_ID };
-  }
-  const { groupId, featureId, vpId, text } = msg;
-  if (!groupId || typeof groupId !== 'string') {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.MISSING_GROUP_ID };
-  }
-  if (!featureId || typeof featureId !== 'string') {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.MISSING_FEATURE_ID };
-  }
-  if (!vpId || typeof vpId !== 'string') {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.MISSING_VP_ID };
-  }
-  // Allow the reserved `user` sentinel as a speaker here — features can have
-  // human-user messages alongside VP messages. Any other vpId must pass
-  // the full shape check (rejects `all`, `system`, pure digits, etc.).
-  if (vpId !== 'user' && !isValidVpId(vpId)) {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.INVALID_VP_ID };
-  }
-  if (typeof text !== 'string' || text.length === 0) {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.EMPTY_TEXT };
-  }
-  if (text.length > MAX_TEXT_LENGTH) {
-    return { ok: false, code: FEATURE_MESSAGE_REJECT_CODES.TEXT_TOO_LONG };
-  }
-
-  const mentions = Array.isArray(msg.mentions)
-    ? msg.mentions.filter(m => typeof m === 'string' && m.length > 0).slice(0, 32)
-    : [];
-  const replyTo = typeof msg.replyTo === 'string' && msg.replyTo.length > 0
-    ? msg.replyTo
-    : null;
-
-  return {
-    ok: true,
-    payload: { groupId, featureId, vpId, text, mentions, replyTo },
-  };
-}
-
-/**
- * Build the outbound `feature_message` event from a validated payload.
- * Exposed separately so tests can snapshot the wire shape without
- * needing a live send fn.
- *
- * @param {{groupId:string,featureId:string,vpId:string,text:string,mentions:string[],replyTo:?string}} payload
- * @param {{now?:()=>number, msgId?:()=>string, requestId?:string}} [opts]
- */
-export function buildFeatureMessageEvent(payload, opts = {}) {
-  const now = typeof opts.now === 'function' ? opts.now : Date.now;
-  const mkId = typeof opts.msgId === 'function' ? opts.msgId : nextMsgId;
-  const evt = {
-    type: 'feature_message',
-    groupId: payload.groupId,
-    featureId: payload.featureId,
-    vpId: payload.vpId,
-    msgId: mkId(),
-    text: payload.text,
-    mentions: payload.mentions,
-    replyTo: payload.replyTo,
-    ts: now(),
-  };
-  if (opts.requestId) evt.requestId = opts.requestId;
-  return evt;
-}
-
-/**
- * Build the outbound `feature_message_rejected` event.
- * @param {string} code — one of FEATURE_MESSAGE_REJECT_CODES
- * @param {any} msg — original inbound msg (for requestId echo)
- */
-export function buildFeatureMessageRejected(code, msg) {
-  const evt = { type: 'feature_message_rejected', code };
-  if (msg && typeof msg.requestId === 'string') evt.requestId = msg.requestId;
-  if (msg && typeof msg.groupId === 'string') evt.groupId = msg.groupId;
-  if (msg && typeof msg.featureId === 'string') evt.featureId = msg.featureId;
-  return evt;
-}
-
-/**
- * WS handler entry point. Validates, echoes, never throws.
- *
- * @param {any} msg
- * @param {(event:object)=>void} sendUnifyEvent
- * @param {{now?:()=>number, msgId?:()=>string}} [opts] — test seams
- */
-export function handleUnifyFeatureMessage(msg, sendUnifyEvent, opts = {}) {
-  const result = validateFeatureMessage(msg);
-  if (!result.ok) {
-    try { sendUnifyEvent(buildFeatureMessageRejected(result.code, msg)); } catch { /* best-effort */ }
-    return;
-  }
-  const requestId = msg && typeof msg.requestId === 'string' ? msg.requestId : undefined;
-  const evt = buildFeatureMessageEvent(result.payload, { ...opts, requestId });
-  try { sendUnifyEvent(evt); } catch { /* never crash WS pipeline */ }
-}