flowmind 1.4.0 → 1.4.2

package/bin/flowmind.js

@@ -1351,6 +1351,7 @@ program
   .description('Launch enhanced TUI with split panels, skill browser, and dragon display')
   .action(async () => {
     let stdinWrapper = null;
+    let stdinForwarder = null;
     try {
       // Register .jsx extension for CJS
       require('module')._extensions['.jsx'] = require('module')._extensions['.js'];
@@ -1378,11 +1379,18 @@ program
         }
         return stdinWrapper;
       };
-      // Forward real stdin data to the wrapper
+      // Forward real stdin data to the wrapper (store reference for cleanup)
       if (realStdin.readable) {
-        realStdin.on('data', (chunk) => {
-          if (!stdinWrapper.destroyed) stdinWrapper.write(chunk);
-        });
+        stdinForwarder = (chunk) => {
+          if (!stdinWrapper.destroyed) {
+            try {
+              stdinWrapper.write(chunk);
+            } catch (e) {
+              // Ignore write-after-destroy errors
+            }
+          }
+        };
+        realStdin.on('data', stdinForwarder);
       }
 
       const { unmount, waitUntilExit } = render(
@@ -1397,6 +1405,16 @@ program
         console.log(chalk.yellow('Try running: npm install ink@3 react ink-text-input ink-spinner'));
       }
     } finally {
+      // Clean up stdin listener to prevent leak
+      if (stdinForwarder) {
+        process.stdin.removeListener('data', stdinForwarder);
+      }
+      // Restore stdin to normal mode
+      try {
+        if (process.stdin.isTTY && process.stdin.setRawMode) {
+          process.stdin.setRawMode(false);
+        }
+      } catch (e) { /* ignore */ }
       if (stdinWrapper && !stdinWrapper.destroyed) {
         stdinWrapper.destroy();
       }
@@ -1409,6 +1427,7 @@ program
   .description('Launch real-time monitoring dashboard for MCP activity and events')
   .action(async () => {
     let stdinWrapper = null;
+    let stdinForwarder = null;
     try {
       // Register .jsx extension for CJS
       require('module')._extensions['.jsx'] = require('module')._extensions['.js'];
@@ -1437,9 +1456,14 @@ program
         return stdinWrapper;
       };
       if (realStdin.readable) {
-        realStdin.on('data', (chunk) => {
-          if (!stdinWrapper.destroyed) stdinWrapper.write(chunk);
-        });
+        stdinForwarder = (chunk) => {
+          if (!stdinWrapper.destroyed) {
+            try {
+              stdinWrapper.write(chunk);
+            } catch (e) { /* ignore write-after-destroy */ }
+          }
+        };
+        realStdin.on('data', stdinForwarder);
       }
 
       const { unmount, waitUntilExit } = render(
@@ -1454,6 +1478,14 @@ program
         console.log(chalk.yellow('Try running: npm install ink@3 react'));
       }
     } finally {
+      if (stdinForwarder) {
+        process.stdin.removeListener('data', stdinForwarder);
+      }
+      try {
+        if (process.stdin.isTTY && process.stdin.setRawMode) {
+          process.stdin.setRawMode(false);
+        }
+      } catch (e) { /* ignore */ }
       if (stdinWrapper && !stdinWrapper.destroyed) {
         stdinWrapper.destroy();
       }

package/core/honor-engine.js

@@ -50,7 +50,7 @@ class HonorEngine {
       }
       this.initialized = true;
     } catch (error) {
-      // Non-blocking: create default data in memory
+      console.warn('HonorEngine init failed, using defaults:', error.message);
       this.data = this.createDefaultData();
       this.initialized = true;
     }
@@ -92,7 +92,7 @@ class HonorEngine {
         }
       }
     } catch (error) {
-      // Non-blocking
+      console.warn('HonorEngine seedKnownSkills failed:', error.message);
     }
   }
 
@@ -148,7 +148,7 @@ class HonorEngine {
         timestamp: this.data.lastUpdated
       });
     } catch (error) {
-      // Non-blocking
+      console.warn('HonorEngine award failed:', error.message);
     }
   }
 
@@ -170,7 +170,7 @@ class HonorEngine {
         await this.save();
       }
     } catch (error) {
-      // Non-blocking
+      console.warn('HonorEngine addKnownSkill failed:', error.message);
     }
   }
 
@@ -247,7 +247,7 @@ class HonorEngine {
       await fs.ensureDir(path.dirname(this.honorPath));
       await fs.writeJson(this.honorPath, this.data, { spaces: 2 });
     } catch (error) {
-      // Non-blocking
+      console.warn('HonorEngine save failed:', error.message);
     }
   }
 }

package/core/index.js

@@ -33,6 +33,13 @@ class FlowMind {
     if (this.initialized) return this;
 
     await this.config.load();
+
+    // Validate configuration
+    const validation = this.config.validate();
+    if (!validation.valid) {
+      console.warn('Configuration warnings:', validation.errors.join(', '));
+    }
+
     await this.components.init();
     await this.components.initAll();
     await this.honor.init();

package/core/learning-engine.js

@@ -7,6 +7,7 @@ const fs = require('fs-extra');
 const path = require('path');
 const { v4: uuidv4 } = require('uuid');
 const eventBus = require('./event-bus');
+const { expandPath } = require('./utils');
 
 /**
  * Per-key write queue to prevent concurrent read-modify-write races
@@ -580,10 +581,7 @@ class LearningEngine {
    * Helper methods
    */
   expandPath(filePath) {
-    if (filePath.startsWith('~')) {
-      return path.join(process.env.HOME || process.env.USERPROFILE, filePath.slice(1));
-    }
-    return filePath;
+    return expandPath(filePath);
   }
 
   extractCondition(input) {

package/core/scene-matcher.js

@@ -5,6 +5,7 @@
 
 const fs = require('fs-extra');
 const path = require('path');
+const { expandPath } = require('./utils');
 
 class SceneMatcher {
   constructor(config, learning) {
@@ -316,10 +317,7 @@ class SceneMatcher {
    * Helper to expand path
    */
   expandPath(filePath) {
-    if (filePath.startsWith('~')) {
-      return path.join(process.env.HOME || process.env.USERPROFILE, filePath.slice(1));
-    }
-    return filePath;
+    return expandPath(filePath);
   }
 }
 

package/core/skill-loader.js

@@ -327,6 +327,13 @@ class SkillLoader {
     const skill = this.skills.get(name);
     if (!skill) return null;
 
+    // Clear require cache to actually reload the module
+    const indexPath = require('path').join(skill.path, 'index.js');
+    const resolvedPath = require.resolve(indexPath);
+    if (require.cache[resolvedPath]) {
+      delete require.cache[resolvedPath];
+    }
+
     return await this.loadSkill(name, skill.path);
   }
 

package/core/utils.js

@@ -0,0 +1,18 @@
+/**
+ * Shared utility functions
+ */
+
+const path = require('path');
+const os = require('os');
+
+/**
+ * Expand ~ in file paths to home directory
+ */
+function expandPath(filePath) {
+  if (filePath.startsWith('~')) {
+    return path.join(process.env.HOME || process.env.USERPROFILE || os.homedir(), filePath.slice(1));
+  }
+  return filePath;
+}
+
+module.exports = { expandPath };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flowmind",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "description": "The AI Agent That Learns How You Work - Stop repeating yourself, FlowMind learns your workflows and applies them automatically.",
   "main": "core/index.js",
   "bin": {

package/tui/app.jsx

@@ -8,6 +8,7 @@ const StatusBar = require('./components/StatusBar.jsx');
 function App({ flowmind }) {
   const [results, setResults] = React.useState([]);
   const [isProcessing, setIsProcessing] = React.useState(false);
+  const [focusPanel, setFocusPanel] = React.useState('chat'); // 'chat' | 'sidebar'
   const mountedRef = React.useRef(true);
   const { exit } = useApp();
 
@@ -15,11 +16,16 @@ function App({ flowmind }) {
     return () => { mountedRef.current = false; };
   }, []);
 
+  // Ctrl+C always exits; Tab switches focus between panels
   useInput((input, key) => {
     if (key.ctrl && input === 'c') exit();
+    if (key.tab) {
+      setFocusPanel(prev => prev === 'chat' ? 'sidebar' : 'chat');
+    }
   });
 
   const handleCommand = React.useCallback(async (input, addResponse) => {
+    if (!mountedRef.current) return;
     setIsProcessing(true);
     try {
       const result = await flowmind.process(input);
@@ -41,6 +47,7 @@ function App({ flowmind }) {
   }, [flowmind]);
 
   const handleSkillSelect = React.useCallback((skill) => {
+    if (!mountedRef.current) return;
     try {
       setResults(prev => [...prev, {
         type: 'result',
@@ -55,9 +62,9 @@ function App({ flowmind }) {
   return (
     React.createElement(Box, { flexDirection: 'column', width: '100%', height: '100%' },
       React.createElement(Box, { flexDirection: 'row', flexGrow: 1 },
-        React.createElement(Sidebar, { flowmind: flowmind, width: 30, onSkillSelect: handleSkillSelect }),
+        React.createElement(Sidebar, { flowmind: flowmind, width: 30, onSkillSelect: handleSkillSelect, focused: focusPanel === 'sidebar' }),
         React.createElement(Box, { flexDirection: 'column', width: '70%', flexGrow: 1 },
-          React.createElement(ChatPanel, { onSubmit: handleCommand, isProcessing: isProcessing, onExit: exit }),
+          React.createElement(ChatPanel, { onSubmit: handleCommand, isProcessing: isProcessing, onExit: exit, focused: focusPanel === 'chat' }),
           React.createElement(ResultPanel, { results: results })
         )
       ),

package/tui/components/ChatPanel.jsx

@@ -1,20 +1,53 @@
 const React = require('react');
-const { Box, Text } = require('ink');
+const { Box, Text, useInput } = require('ink');
 const TextInput = require('ink-text-input').default || require('ink-text-input');
 const Spinner = require('ink-spinner').default || require('ink-spinner');
 
-function ChatPanel({ onSubmit, isProcessing, onExit }) {
+function ChatPanel({ onSubmit, isProcessing, onExit, focused }) {
   const [input, setInput] = React.useState('');
   const [history, setHistory] = React.useState([]);
+  const [cmdHistory, setCmdHistory] = React.useState([]);
+  const [historyIndex, setHistoryIndex] = React.useState(-1);
+  const [savedInput, setSavedInput] = React.useState('');
   const mountedRef = React.useRef(true);
 
   React.useEffect(() => {
     return () => { mountedRef.current = false; };
   }, []);
 
+  // Handle Up/Down arrow for command history (only when focused)
+  useInput((ch, key) => {
+    if (!focused || isProcessing) return;
+
+    if (key.upArrow && cmdHistory.length > 0) {
+      const newIndex = historyIndex === -1
+        ? cmdHistory.length - 1
+        : Math.max(0, historyIndex - 1);
+      if (historyIndex === -1) setSavedInput(input);
+      setHistoryIndex(newIndex);
+      setInput(cmdHistory[newIndex]);
+    } else if (key.downArrow) {
+      if (historyIndex === -1) return;
+      const newIndex = historyIndex + 1;
+      if (newIndex >= cmdHistory.length) {
+        setHistoryIndex(-1);
+        setInput(savedInput);
+      } else {
+        setHistoryIndex(newIndex);
+        setInput(cmdHistory[newIndex]);
+      }
+    }
+  });
+
   const handleSubmit = (value) => {
     if (!value.trim()) return;
     setHistory(prev => [...prev, { role: 'user', text: value }]);
+    // Add to command history (deduplicate consecutive)
+    if (cmdHistory.length === 0 || cmdHistory[cmdHistory.length - 1] !== value) {
+      setCmdHistory(prev => [...prev, value]);
+    }
+    setHistoryIndex(-1);
+    setSavedInput('');
     setInput('');
     if (value.toLowerCase() === 'exit' || value.toLowerCase() === 'quit') {
       if (onExit) onExit();

package/tui/components/Sidebar.jsx

@@ -2,7 +2,7 @@ const React = require('react');
 const { Box, Text, useInput } = require('ink');
 const DragonTotem = require('./DragonTotem.jsx');
 
-function Sidebar({ flowmind, width, onSkillSelect }) {
+function Sidebar({ flowmind, width, onSkillSelect, focused }) {
   const [selectedIndex, setSelectedIndex] = React.useState(0);
   const [skills, setSkills] = React.useState([]);
   const [honorData, setHonorData] = React.useState({ points: 0, level: 0, stats: {} });
@@ -24,6 +24,7 @@ function Sidebar({ flowmind, width, onSkillSelect }) {
   }, [flowmind]);
 
   useInput((input, key) => {
+    if (!focused) return; // Ignore input when sidebar is not focused
     if (key.upArrow) setSelectedIndex(prev => Math.max(0, prev - 1));
     else if (key.downArrow) setSelectedIndex(prev => Math.min(skills.length - 1, prev + 1));
     else if (key.return && skills[selectedIndex] && onSkillSelect) onSkillSelect(skills[selectedIndex]);