flowmind 1.5.2 → 1.5.4

package/skills/yuque-sync-design/index.js

@@ -13,7 +13,9 @@ module.exports = {
   },
 
   async execute(input, context) {
-    const knowledgeBase = context.componentRegistry?.getAdapter('knowledgeBase');
+    const knowledgeBase = context.componentRegistry?.getAdapter('knowledgeBase')
+      || context.componentRegistry?.getAdapterByProvider?.('knowledgeBase', 'yuque')
+      || null;
     const learnedBinding = context.resourceBinding?.componentType === 'knowledgeBase'
       ? context.resourceBinding
       : null;
@@ -29,17 +31,33 @@ module.exports = {
       };
     }
 
+    if (!knowledgeBase) {
+      return {
+        type: 'result',
+        skill: 'yuque-sync-design',
+        message: 'Yuque adapter is not available in the current registry.',
+        data: {
+          hint: 'Make sure the knowledgeBase component is configured and loaded',
+          binding: learnedBinding
+        },
+        input,
+        timestamp: new Date().toISOString()
+      };
+    }
+
     const params = parseYuqueParams(input);
 
     if (params.action === 'search') {
+      const execution = await knowledgeBase.search(params.keyword, 'doc');
       return {
         type: 'result',
         skill: 'yuque-sync-design',
         message: `Searching Yuque for: ${params.keyword || 'all'}`,
         data: {
-          mcpTool: 'search',
+          action: 'search',
           args: { q: params.keyword, type: 'doc' },
-          binding: learnedBinding
+          binding: learnedBinding,
+          execution: summarizeYuqueExecution('search', execution)
         },
         input,
         timestamp: new Date().toISOString()
@@ -47,14 +65,16 @@ module.exports = {
     }
 
     if (params.action === 'list') {
+      const execution = await knowledgeBase.getRepos();
       return {
         type: 'result',
         skill: 'yuque-sync-design',
         message: 'Listing Yuque repositories',
         data: {
-          mcpTool: 'get_user_repos',
+          action: 'list',
           args: {},
-          binding: learnedBinding
+          binding: learnedBinding,
+          execution: summarizeYuqueExecution('list', execution)
         },
         input,
         timestamp: new Date().toISOString()
@@ -62,7 +82,7 @@ module.exports = {
     }
 
     if (params.action === 'sync') {
-      return syncDesignDoc(params, input, learnedBinding);
+      return syncDesignDoc(params, input, learnedBinding, knowledgeBase);
     }
 
     return {
@@ -98,7 +118,7 @@ function parseYuqueParams(input) {
   return params;
 }
 
-async function syncDesignDoc(params, input, learnedBinding) {
+async function syncDesignDoc(params, input, learnedBinding, knowledgeBase) {
   const filePath = params.path || 'DESIGN.md';
 
   if (!(await fs.pathExists(filePath))) {
@@ -113,27 +133,126 @@ async function syncDesignDoc(params, input, learnedBinding) {
 
   const content = await fs.readFile(filePath, 'utf-8');
   const title = extractTitle(content) || path.basename(filePath, '.md');
+  const namespace = params.repo || learnedBinding?.connection?.namespace;
+  const execution = await knowledgeBase.createDoc(namespace, {
+    slug: title.toLowerCase().replace(/\s+/g, '-'),
+    title,
+    body: content,
+    format: 'markdown'
+  });
 
   return {
     type: 'result',
     skill: 'yuque-sync-design',
     message: `Ready to sync "${title}" to Yuque`,
     data: {
-      mcpTool: 'create_doc',
+      action: 'sync',
       args: {
-        namespace: params.repo || learnedBinding?.connection?.namespace,
+        namespace,
         slug: title.toLowerCase().replace(/\s+/g, '-'),
         title,
-        body: content,
         format: 'markdown'
       },
-      binding: learnedBinding
+      binding: learnedBinding,
+      execution: summarizeYuqueExecution('sync', execution)
     },
     input,
     timestamp: new Date().toISOString()
   };
 }
 
+function summarizeYuqueExecution(action, execution) {
+  const payload = unwrapPayload(execution);
+  const data = payload && typeof payload === 'object' ? payload : {};
+  const items = extractItems(data);
+
+  return {
+    action,
+    status: extractStatus(data) || 'ok',
+    total: extractTotal(data, items),
+    items: items.slice(0, 5).map(summarizeItem),
+    id: data.id || data.doc_id || data.slug || null
+  };
+}
+
+function unwrapPayload(payload) {
+  if (!payload || typeof payload !== 'object' || Array.isArray(payload)) {
+    return payload;
+  }
+
+  const textPayload = extractContentText(payload);
+  if (textPayload) {
+    return parseMaybeJson(textPayload);
+  }
+
+  if (payload.data && typeof payload.data === 'object' && !Array.isArray(payload.data)) {
+    return payload.data;
+  }
+
+  return payload;
+}
+
+function extractItems(data) {
+  const candidates = [data.list, data.items, data.repos, data.docs, data.result];
+  for (const candidate of candidates) {
+    if (Array.isArray(candidate)) {
+      return candidate;
+    }
+  }
+
+  return [];
+}
+
+function extractStatus(data) {
+  if (!data || typeof data !== 'object') {
+    return null;
+  }
+
+  if (typeof data.status === 'string') {
+    return data.status;
+  }
+
+  if (data.success === false) {
+    return 'error';
+  }
+
+  return null;
+}
+
+function extractTotal(data, items) {
+  return data.total || items.length || null;
+}
+
+function extractContentText(payload) {
+  const content = Array.isArray(payload.content) ? payload.content : [];
+  const textItem = content.find((item) => item && typeof item.text === 'string');
+  return textItem ? textItem.text : null;
+}
+
+function parseMaybeJson(value) {
+  if (typeof value !== 'string') {
+    return value;
+  }
+
+  try {
+    return JSON.parse(value);
+  } catch (error) {
+    return { text: value };
+  }
+}
+
+function summarizeItem(item) {
+  if (!item || typeof item !== 'object') {
+    return item;
+  }
+
+  return {
+    id: item.id || item.slug || item.namespace || null,
+    title: item.title || item.name || item.slug || null,
+    type: item.type || item.format || null
+  };
+}
+
 function extractTitle(content) {
   const match = content.match(/^#\s+(.+)$/m);
   return match ? match[1].trim() : null;

package/tui/app.jsx

@@ -6,11 +6,14 @@ const StatusBar = require('./components/StatusBar.jsx');
 const { formatResultText } = require('./format-result');
 
 function App({ flowmind, asciiMode = false }) {
+  const MAX_MESSAGES = 60;
   const [messages, setMessages] = React.useState([]);
   const [isProcessing, setIsProcessing] = React.useState(false);
   const [focusPanel, setFocusPanel] = React.useState('chat'); // 'chat' | 'sidebar'
   const mountedRef = React.useRef(true);
   const messageIdRef = React.useRef(0);
+  const commandIdRef = React.useRef(0);
+  const activeCommandRef = React.useRef(false);
   const { exit } = useApp();
 
   React.useEffect(() => {
@@ -19,7 +22,7 @@ function App({ flowmind, asciiMode = false }) {
 
   const appendMessage = React.useCallback((role, text, metadata = {}) => {
     const id = ++messageIdRef.current;
-    setMessages(prev => [...prev, { id, role, text, metadata }]);
+    setMessages(prev => [...prev, { id, role, text, metadata }].slice(-MAX_MESSAGES));
   }, []);
 
   // Ctrl+C always exits; Tab switches focus between panels
@@ -31,13 +34,15 @@ function App({ flowmind, asciiMode = false }) {
   });
 
   const handleCommand = React.useCallback(async (input) => {
-    if (!mountedRef.current) return;
+    if (!mountedRef.current || activeCommandRef.current) return;
+    activeCommandRef.current = true;
+    const commandId = ++commandIdRef.current;
     appendMessage('user', input);
     setIsProcessing(true);
 
     try {
       const result = await flowmind.process(input);
-      if (!mountedRef.current) return;
+      if (!mountedRef.current || commandId !== commandIdRef.current) return;
 
       appendMessage('flowmind', formatResultText(result), {
         type: result.type,
@@ -46,9 +51,14 @@ function App({ flowmind, asciiMode = false }) {
         scene: result.metadata?.sceneMatch?.scene?.name
       });
     } catch (e) {
-      if (mountedRef.current) appendMessage('flowmind', 'Error: ' + e.message, { type: 'error' });
+      if (mountedRef.current && commandId === commandIdRef.current) {
+        appendMessage('flowmind', 'Error: ' + e.message, { type: 'error' });
+      }
     } finally {
-      if (mountedRef.current) setIsProcessing(false);
+      if (mountedRef.current && commandId === commandIdRef.current) {
+        setIsProcessing(false);
+      }
+      activeCommandRef.current = false;
     }
   }, [appendMessage, flowmind]);
 

package/tui/components/ChatPanel.jsx

@@ -36,7 +36,7 @@ function ChatPanel({ messages, onSubmit, isProcessing, onExit, focused, asciiMod
 
   const handleSubmit = (value) => {
     const normalized = value.trim();
-    if (!normalized) return;
+    if (!normalized || isProcessing) return;
     // Add to command history (deduplicate consecutive)
     if (cmdHistory.length === 0 || cmdHistory[cmdHistory.length - 1] !== normalized) {
       setCmdHistory(prev => [...prev, normalized]);

package/tui/format-result.js

@@ -2,6 +2,12 @@ function isPlainObject(value) {
   return value && typeof value === 'object' && !Array.isArray(value);
 }
 
+const MAX_RENDER_LINES = 120;
+const MAX_LINE_LENGTH = 220;
+const MAX_RENDER_CHARS = 8000;
+const MAX_ARRAY_ITEMS = 20;
+const MAX_SCALAR_LENGTH = 160;
+
 function unwrapResultPayload(result) {
   let current = result;
 
@@ -58,11 +64,40 @@ function pruneDisplayValue(value) {
 }
 
 function formatScalar(value) {
-  if (typeof value === 'string') return value;
+  if (typeof value === 'string') {
+    if (value.length <= MAX_SCALAR_LENGTH) return value;
+    return `${value.slice(0, MAX_SCALAR_LENGTH - 1)}…`;
+  }
   if (typeof value === 'number' || typeof value === 'boolean') return String(value);
   return JSON.stringify(value);
 }
 
+function sanitizeForTerminal(text) {
+  return String(text)
+    .replace(/[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f]/g, ' ')
+    .replace(/\r\n/g, '\n');
+}
+
+function truncateRenderedText(text) {
+  const sanitized = sanitizeForTerminal(text);
+  const lines = sanitized.split('\n');
+  const clippedLines = lines.slice(0, MAX_RENDER_LINES).map((line) => {
+    if (line.length <= MAX_LINE_LENGTH) return line;
+    return `${line.slice(0, MAX_LINE_LENGTH - 1)}…`;
+  });
+
+  if (lines.length > MAX_RENDER_LINES) {
+    clippedLines.push(`… truncated ${lines.length - MAX_RENDER_LINES} line(s)`);
+  }
+
+  let output = clippedLines.join('\n');
+  if (output.length > MAX_RENDER_CHARS) {
+    output = `${output.slice(0, MAX_RENDER_CHARS - 1)}…`;
+  }
+
+  return output;
+}
+
 function renderValueLines(value, indent = '') {
   if (value === undefined) return [];
 
@@ -71,7 +106,11 @@ function renderValueLines(value, indent = '') {
 
     const allScalars = value.every((item) => !isPlainObject(item) && !Array.isArray(item));
     if (allScalars) {
-      return [indent + value.map(formatScalar).join(', ')];
+      const shown = value.slice(0, MAX_ARRAY_ITEMS).map(formatScalar);
+      if (value.length > MAX_ARRAY_ITEMS) {
+        shown.push(`… +${value.length - MAX_ARRAY_ITEMS} more`);
+      }
+      return [indent + shown.join(', ')];
     }
 
     const lines = [];
@@ -150,10 +189,10 @@ function formatResultText(result) {
   }
 
   if (detailLines.length > 0) {
-    return detailLines.join('\n');
+    return truncateRenderedText(detailLines.join('\n'));
   }
 
-  return JSON.stringify(payload, null, 2);
+  return truncateRenderedText(JSON.stringify(payload, null, 2));
 }
 
 module.exports = {