@yeaft/webchat-agent 1.0.38 → 1.0.40

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "1.0.38",
+  "version": "1.0.40",
   "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/conversation/persist.js

@@ -206,6 +206,7 @@ function serializeMessage(msg) {
   // routing can filter/replay by thread without rescanning JSON blobs.
   // Defaults to 'main' for legacy messages (see migrate-messages-threadid.js).
   fm.push(`threadId: ${msg.threadId || 'main'}`);
+  if (msg.turnId) fm.push(`turnId: ${msg.turnId}`);
   // task-313: when a thread is merged into another, the messages keep
   // their original thread id in `sourceThreadId` so the UI can still
   // render a small "#source" pill next to each bubble.
@@ -216,6 +217,7 @@ function serializeMessage(msg) {
   // shows an empty pane.
   if (msg.sessionId) fm.push(`sessionId: ${msg.sessionId}`);
   if (msg.chatId) fm.push(`chatId: ${msg.chatId}`);
+  if (msg.clientMessageId) fm.push(`clientMessageId: ${msg.clientMessageId}`);
   // Session attribution: when a VP authors an assistant turn (either
   // its own reply or a route_forward injection from another VP), stamp
   // the speaker so the UI can render the message on the correct VP track.
@@ -333,9 +335,11 @@ export function parseMessage(raw) {
       case 'isError': msg.isError = value === 'true'; break;
       case 'tokens_est': msg.tokens_est = parseInt(value, 10); break;
       case 'threadId': msg.threadId = value; break;
+      case 'turnId': msg.turnId = value; break;
       case 'sourceThreadId': msg.sourceThreadId = value; break;
       case 'sessionId': msg.sessionId = value; break;
       case 'chatId': msg.chatId = value; break;
+      case 'clientMessageId': msg.clientMessageId = value; break;
       case 'speakerVpId': msg.speakerVpId = value; break;
       case 'attachmentsB64':
         try {

package/yeaft/engine.js

@@ -95,6 +95,36 @@ const RETRY_DEFAULTS = Object.freeze({
   jitterRatio: 0.25,
 });
 
+const EXPLICIT_SKILL_COMMAND_RE = /^\/skill:([^\s]+)(\s+|$)/;
+
+function parseExplicitSkillCommand(prompt) {
+  if (typeof prompt !== 'string') {
+    return { skillName: null, cleanedPrompt: prompt };
+  }
+  const match = prompt.match(EXPLICIT_SKILL_COMMAND_RE);
+  if (!match) {
+    return { skillName: null, cleanedPrompt: prompt };
+  }
+  return {
+    skillName: match[1],
+    cleanedPrompt: prompt.slice(match[0].length),
+  };
+}
+
+function stripLeadingSkillCommandFromPromptParts(promptParts) {
+  if (!Array.isArray(promptParts) || promptParts.length === 0) return promptParts;
+  let stripped = false;
+  return promptParts.map(part => {
+    if (stripped || !part || part.type !== 'text' || typeof part.text !== 'string') {
+      return part;
+    }
+    const parsed = parseExplicitSkillCommand(part.text);
+    if (!parsed.skillName) return part;
+    stripped = true;
+    return { ...part, text: parsed.cleanedPrompt };
+  });
+}
+
 function resolveRetryPolicy(config) {
   const raw = config?.llmRetry || {};
   const num = (v, d) => (Number.isFinite(v) && v >= 0 ? v : d);
@@ -932,13 +962,21 @@ export class Engine {
    * @param {string} [args.sessionAnnouncement]
    * @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, projectDoc, taskCtx, activeTasks } = {}) {
-    // Get relevant skill content if SkillManager is wired
+  #buildSystemPrompt({ prompt, memoryInjection, vpPersona, activeScope, sessionAnnouncement, projectDoc, taskCtx, activeTasks, explicitSkillName } = {}) {
+    // Get relevant skill content if SkillManager is wired. A leading
+    // /skill:<name> is explicit, not relevance matching: load that skill by
+    // name or inject a visible prompt warning when the command is unknown.
     let skillContent = '';
-    if (this.#skillManager && prompt) {
-      skillContent = this.#skillManager.getRelevantPromptContent(prompt);
+    if (this.#skillManager) {
+      if (explicitSkillName) {
+        skillContent = this.#skillManager.getPromptContent(explicitSkillName)
+          || `## Skill command error\n\nRequested skill "${explicitSkillName}" was not found. Continue without that skill and tell the user it is unavailable.`;
+      } else if (prompt) {
+        skillContent = this.#skillManager.getRelevantPromptContent(prompt);
+      }
     }
 
     // Get tool names from the appropriate source
@@ -1559,7 +1597,7 @@ export class Engine {
    *   string-prompt shape (no regression for existing callers).
    * @yields {EngineEvent}
    */
-  async *query({ prompt, promptParts = null, messages = [], signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, sessionId, sessionMembers, sessionTopics = null, vpPlan, sessionAnnouncement, workDir, userAlreadyPersisted = false, getCurrentTodos = null, setCurrentTodos = null, threadId = MAIN_THREAD_ID, drainPendingUserMessages = null, collabToolPolicy = null } = {}) {
+  async *query({ prompt, promptParts = null, messages = [], signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, sessionId, sessionMembers, sessionTopics = null, vpPlan, sessionAnnouncement, workDir, userAlreadyPersisted = false, getCurrentTodos = null, setCurrentTodos = null, threadId = MAIN_THREAD_ID, vpTurnId = null, drainPendingUserMessages = null, collabToolPolicy = null } = {}) {
     if (!prompt || typeof prompt !== 'string' || !prompt.trim()) {
       yield {
         type: 'error',
@@ -1583,7 +1621,11 @@ export class Engine {
     // the merge, so an invalid caller value (e.g. 'ULTRA') does not shadow a
     // valid prompt prefix.
     const parsed = parseEffortPrefix(prompt);
-    const effectivePrompt = parsed.cleanedPrompt;
+    const parsedSkill = parseExplicitSkillCommand(parsed.cleanedPrompt);
+    const effectivePrompt = parsedSkill.cleanedPrompt;
+    const effectivePromptParts = parsedSkill.skillName
+      ? stripLeadingSkillCommandFromPromptParts(promptParts)
+      : promptParts;
     const configuredEffort = normalizeEffort(this.#config?.modelEffort);
     const effectiveUserEffort = normalizeEffort(userEffort) || parsed.effort || configuredEffort || null;
     const effectiveCollabToolPolicy = collabToolPolicy === COLLAB_TOOL_POLICY.SINGLE_VP || collabToolPolicy === COLLAB_TOOL_POLICY.MULTI_VP
@@ -1623,7 +1665,7 @@ export class Engine {
 
     try {
       this.#currentThreadId = threadId || MAIN_THREAD_ID;
-      yield* this.#runQuery({ prompt: effectivePrompt, promptParts, messages, signal: runSignal, userEffort: effectiveUserEffort, scenario, vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, sessionId, sessionMembers, sessionTopics, vpPlan, sessionAnnouncement, workDir, userAlreadyPersisted, getCurrentTodos, setCurrentTodos, threadId: this.#currentThreadId, drainPendingUserMessages, collabToolPolicy: effectiveCollabToolPolicy });
+      yield* this.#runQuery({ prompt: effectivePrompt, promptParts: effectivePromptParts, messages, signal: runSignal, userEffort: effectiveUserEffort, scenario, vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, sessionId, sessionMembers, sessionTopics, vpPlan, sessionAnnouncement, workDir, userAlreadyPersisted, getCurrentTodos, setCurrentTodos, threadId: this.#currentThreadId, drainPendingUserMessages, collabToolPolicy: effectiveCollabToolPolicy, explicitSkillName: parsedSkill.skillName });
     } finally {
       if (signal) {
         try { signal.removeEventListener('abort', onExternalAbort); } catch { /* ignore */ }
@@ -1667,7 +1709,7 @@ export class Engine {
    * in a try/finally without indenting the whole loop.
    * @private
    */
-  async *#runQuery({ prompt, promptParts = null, messages, signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, sessionId, sessionMembers, sessionTopics = null, vpPlan, sessionAnnouncement, workDir, userAlreadyPersisted = false, getCurrentTodos = null, setCurrentTodos = null, threadId = MAIN_THREAD_ID, drainPendingUserMessages = null, collabToolPolicy = null }) {
+  async *#runQuery({ prompt, promptParts = null, messages, signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, sessionId, sessionMembers, sessionTopics = null, vpPlan, sessionAnnouncement, workDir, userAlreadyPersisted = false, getCurrentTodos = null, setCurrentTodos = null, threadId = MAIN_THREAD_ID, drainPendingUserMessages = null, collabToolPolicy = null, explicitSkillName = null }) {
 
     const effectiveCollabToolPolicy = collabToolPolicy === COLLAB_TOOL_POLICY.SINGLE_VP || collabToolPolicy === COLLAB_TOOL_POLICY.MULTI_VP
       ? collabToolPolicy
@@ -1783,6 +1825,7 @@ export class Engine {
       sessionAnnouncement,
       projectDoc,
       activeTasks,
+      explicitSkillName,
     });
 
     // ─── HARD INVARIANT: Compact ≠ Dream (read DESIGN-COMPACT-VS-DREAM.md) ─
@@ -2790,6 +2833,7 @@ export class Engine {
             // history replay can re-stamp them on reload.
             sessionId,
             threadId,
+            turnId: vpTurnId || queryTurnId,
             vpId: this.#vpId,
             // Multi-VP fan-out (history-dedup): skip the user-row append
             // in stop-hooks when the orchestrator already wrote it once

package/yeaft/stop-hooks.js

@@ -65,6 +65,7 @@ export async function runStopHooks(context) {
     // history replay can route messages back into the originating group.
     sessionId,
     threadId,
+    turnId,
     // Group VP attribution: persist the engine-bound VP id on assistant/tool
     // rows so history replay can route replies back to the same visible VP.
     vpId,
@@ -167,6 +168,9 @@ export async function runStopHooks(context) {
         // Bug 6: stamp sessionId / threadId so replay can re-route by group.
         if (sessionId) record.sessionId = sessionId;
         if (threadId) record.threadId = threadId;
+        if (turnId && (msg.role === 'assistant' || msg.role === 'tool')) {
+          record.turnId = turnId;
+        }
         if (vpId && (msg.role === 'assistant' || msg.role === 'tool')) {
           record.speakerVpId = vpId;
         }

package/yeaft/web-bridge.js

@@ -73,6 +73,8 @@ import { buildMcpFlattenedTools } from './tools/mcp-tools.js';
 import { getAgentRegistry, agentBelongsToScope } from './tools/agent.js';
 import { isPromptableAgentStatus } from './sub-agent/status.js';
 
+const SKILL_COMMAND_PREFIX = 'skill:';
+
 /** @type {import('./session.js').Session | null} */
 let session = null;
 
@@ -710,8 +712,9 @@ function projectPersistedToHistoryEntry(m) {
   const entry = { role: m.role, content: m.role === 'tool' ? m.content : __testNormalizePersistedVisibleContent(m.content) };
   if (m.id) entry.id = m.id;
   entry.threadId = m.threadId || m.turnId || 'main';
-  entry.turnId = m.turnId || entry.threadId;
+  if (m.turnId) entry.turnId = m.turnId;
   if (m.sessionId) entry.sessionId = m.sessionId;
+  if (m.clientMessageId) entry.clientMessageId = m.clientMessageId;
   if (m.speakerVpId) entry.speakerVpId = m.speakerVpId;
   if (m.toolCallId) entry.toolCallId = m.toolCallId;
   if (Array.isArray(m.toolCalls) && m.toolCalls.length > 0) {
@@ -811,8 +814,9 @@ function projectVisibleHistoryChunkMessages(messages = []) {
       content: m.content,
       ts: m.ts || null,
       sessionId: m.sessionId || null,
+      ...(m.clientMessageId ? { clientMessageId: m.clientMessageId } : {}),
       threadId: m.threadId || m.turnId || 'main',
-      turnId: m.turnId || m.threadId || 'main',
+      ...(m.turnId ? { turnId: m.turnId } : {}),
       ...(Array.isArray(m.attachments) && m.attachments.length > 0 ? { attachments: hydrateHistoryAttachmentPreviews(m.attachments) } : {}),
       ...(m.speakerVpId ? { speakerVpId: m.speakerVpId } : {}),
       ...(Number.isFinite(m.toolSummaryCount) && m.toolSummaryCount > 0
@@ -1390,6 +1394,7 @@ async function routeEnvelopeToVpThread(sessionId, vpId, envelope) {
       attachments: Array.isArray(envelope?.msg?.meta?.attachments) ? envelope.msg.meta.attachments : [],
       internal: isInternalAppend,
       ts: envelope?.msg?.ts || null,
+      clientMessageId: envelope?.msg?.meta?.clientMessageId || null,
     });
     thread.updatedAt = Date.now();
     try {
@@ -1487,6 +1492,7 @@ function ensureDriverRunning(sessionId, vpId, threadId = 'main') {
             attachments: Array.isArray(meta.attachments) ? meta.attachments : [],
             internal: isInternal,
             ts: envelope?.msg?.ts || null,
+            clientMessageId: meta.clientMessageId || null,
           });
         }
       } catch { /* never crash WS pipeline */ }
@@ -1690,6 +1696,37 @@ function sendSessionOutputFrame(data, { sessionId, chatId, vpId, turnId, threadI
   });
 }
 
+export function buildSkillSlashCommands(skillManager) {
+  if (!skillManager || typeof skillManager.list !== 'function') return { commands: [], descriptions: {} };
+  const commands = [];
+  const descriptions = {};
+  for (const skill of skillManager.list()) {
+    if (!skill?.name || typeof skill.name !== 'string') continue;
+    const commandName = `${SKILL_COMMAND_PREFIX}${skill.name}`;
+    commands.push(commandName);
+    descriptions[commandName] = skill.description || skill.trigger || 'Load Yeaft skill';
+  }
+  commands.sort((a, b) => a.localeCompare(b));
+  return { commands, descriptions };
+}
+
+function broadcastSkillSlashCommands(sessionLike) {
+  const { commands, descriptions } = buildSkillSlashCommands(sessionLike?.skillManager);
+  const slashCommands = [...new Set([...(ctx.slashCommands || []), ...commands])];
+  const slashCommandDescriptions = {
+    ...(ctx.slashCommandDescriptions || {}),
+    ...descriptions,
+  };
+  ctx.slashCommands = slashCommands;
+  ctx.slashCommandDescriptions = slashCommandDescriptions;
+  sendToServer({
+    type: 'slash_commands_update',
+    conversationId: '__preload__',
+    slashCommands,
+    slashCommandDescriptions,
+  });
+}
+
 /** Send a Yeaft Session metadata event over the legacy-compatible envelope. */
 function sendSessionEvent(event, { sessionId, chatId, vpId, turnId, threadId } = {}) {
   sendToServer({
@@ -3119,6 +3156,7 @@ export async function handleYeaftSessionSend(msg) {
         mentions,
         // Persisted form (no base64) — safe for jsonl-log.
         attachments: persistedAttachments,
+        clientMessageId: typeof msg.id === 'string' && msg.id ? msg.id : null,
       },
       // Live form — adapters need the base64 image blocks; runVpTurn
       // reads `_promptParts` off the envelope rather than going
@@ -3375,6 +3413,7 @@ async function ensureSessionLoaded() {
 
   yeaftConversationId = `yeaft-${Date.now()}`;
   hydrateYeaftStatusFromSession(session, { reason: 'session_ready', emitEvent: true });
+  broadcastSkillSlashCommands(session);
 
   // Per-group history is hydrated lazily on first `getOrCreateSessionHistory`
   // — there's no global "all conversations" tape any more.
@@ -3657,6 +3696,7 @@ async function runVpTurn({ prompt, promptParts = null, sessionId, vpId, threadId
         // would render the user's prompt N times.
         userAlreadyPersisted: true,
         threadId,
+        vpTurnId: turnId,
         drainPendingUserMessages: () => {
           if (!thread || !Array.isArray(thread.pendingQueries) || thread.pendingQueries.length === 0) return [];
           return thread.pendingQueries.splice(0);
@@ -3911,11 +3951,11 @@ function appendTurnToSessionHistory(sessionId, threadId, vpId, prompts, assistan
  * refresh replay can render chips without leaking image source data into
  * the message body.
  *
- * @param {{ msgId:string, text:string, sessionId:string, role?:string, speakerVpId?:string|null, attachments?:Array<object>, internal?:boolean, ts?:string|null }} args
+ * @param {{ msgId:string, text:string, sessionId:string, role?:string, speakerVpId?:string|null, attachments?:Array<object>, internal?:boolean, ts?:string|null, clientMessageId?:string|null }} args
  * @returns {boolean} true if this call wrote the row, false if a prior
  *   call already wrote it (dedup hit).
  */
-function persistInboundMessageOnceByMsgId({ msgId, text, sessionId, threadId = 'main', role, speakerVpId, attachments, internal = false, ts = null }) {
+function persistInboundMessageOnceByMsgId({ msgId, text, sessionId, threadId = 'main', role, speakerVpId, attachments, internal = false, ts = null, clientMessageId = null }) {
   if (!session?.conversationStore) return false;
   // No msgId means no dedup key — caller is responsible for guarding.
   // Both call sites already do (`if (envMsgId && text)` and
@@ -3959,6 +3999,9 @@ function persistInboundMessageOnceByMsgId({ msgId, text, sessionId, threadId = '
       threadId: threadId || 'main',
     };
     if (sessionId) record.sessionId = sessionId;
+    if (persistRole === 'user' && clientMessageId && typeof clientMessageId === 'string') {
+      record.clientMessageId = clientMessageId;
+    }
     // Stamp speakerVpId so the UI's loadHistory replay can route the row
     // to the correct VP block. Only meaningful when role='assistant'; for
     // a real user message we leave it unset (the UI's user track is