@geekbeer/minion 2.44.0 → 2.48.1

package/win/lib/process-manager.js

@@ -8,6 +8,7 @@
  */
 
 const { execSync } = require('child_process')
+const fs = require('fs')
 const path = require('path')
 const os = require('os')
 
@@ -18,7 +19,6 @@ const os = require('os')
  */
 function detectProcessManager() {
   // Check for user-process mode (PID file or start-agent.ps1 exists)
-  const fs = require('fs')
   const dataDir = path.join(os.homedir(), '.minion')
   const startScript = path.join(dataDir, 'start-agent.ps1')
   if (fs.existsSync(startScript)) {
@@ -40,6 +40,56 @@ function detectProcessManager() {
   return 'user-process'
 }
 
+/**
+ * Generate a temporary PowerShell script that:
+ * 1. Stops the running agent process (releasing file locks)
+ * 2. Runs npm install -g
+ * 3. Restarts the agent
+ *
+ * This is necessary because node-pty's conpty.node DLL is locked by the
+ * running process, causing EBUSY errors if npm tries to overwrite it in-place.
+ *
+ * @param {string} npmInstallCmd - The npm install command to run
+ * @param {string} stopCmd - Command/script block to stop the agent
+ * @param {string} startCmd - Command/script block to start the agent
+ * @returns {string} - Shell command that writes and launches the update script
+ */
+function buildUpdateScript(npmInstallCmd, stopCmd, startCmd) {
+  const dataDir = path.join(os.homedir(), '.minion')
+  const scriptPath = path.join(dataDir, 'update-agent.ps1')
+  const logPath = path.join(dataDir, 'update-agent.log')
+
+  // PowerShell script content: stop → install → start, with logging
+  const ps1 = [
+    `$ErrorActionPreference = 'Stop'`,
+    `$logFile = '${logPath.replace(/\\/g, '\\\\')}'`,
+    `function Log($msg) { "$(Get-Date -Format o) $msg" | Out-File -Append $logFile }`,
+    `Log 'Update started'`,
+    `try {`,
+    `  Log 'Stopping agent...'`,
+    `  ${stopCmd}`,
+    `  Start-Sleep -Seconds 3`,
+    `  Log 'Installing package...'`,
+    `  $out = & cmd /c "${npmInstallCmd} 2>&1"`,
+    `  Log "npm output: $out"`,
+    `  if ($LASTEXITCODE -ne 0) { throw "npm install failed (exit code $LASTEXITCODE)" }`,
+    `  Log 'Starting agent...'`,
+    `  ${startCmd}`,
+    `  Log 'Update completed successfully'`,
+    `} catch {`,
+    `  Log "Update failed: $_"`,
+    `  Log 'Attempting to restart agent anyway...'`,
+    `  ${startCmd}`,
+    `}`,
+  ].join('\n')
+
+  // Write the script to disk and launch it detached
+  try { fs.mkdirSync(dataDir, { recursive: true }) } catch { /* exists */ }
+  fs.writeFileSync(scriptPath, ps1, 'utf-8')
+
+  return `powershell -ExecutionPolicy Bypass -Command "Start-Process powershell -ArgumentList '-ExecutionPolicy Bypass -WindowStyle Hidden -File \\"${scriptPath}\\"' -WindowStyle Hidden"`
+}
+
 /**
  * Build allowed commands for the detected process manager.
  * @param {string} procMgr - Process manager type
@@ -53,19 +103,30 @@ function buildAllowedCommands(procMgr) {
     const pidFile = path.join(dataDir, 'minion-agent.pid')
     const startScript = path.join(dataDir, 'start-agent.ps1')
 
+    const stopBlock = `$pid = Get-Content '${pidFile}' -ErrorAction SilentlyContinue; if ($pid) { Get-CimInstance Win32_Process -Filter \\"ParentProcessId = $pid\\" -ErrorAction SilentlyContinue | ForEach-Object { Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue }; Stop-Process -Id $pid -Force -ErrorAction SilentlyContinue }`
+    const startBlock = `Start-Process powershell -ArgumentList '-ExecutionPolicy Bypass -WindowStyle Hidden -File \\"${startScript}\\"' -WindowStyle Hidden`
+
     commands['restart-agent'] = {
       description: 'Restart the minion agent process',
-      command: `powershell -Command "$pid = Get-Content '${pidFile}' -ErrorAction SilentlyContinue; if ($pid) { Get-CimInstance Win32_Process -Filter \\"ParentProcessId = $pid\\" -ErrorAction SilentlyContinue | ForEach-Object { Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue }; Stop-Process -Id $pid -Force -ErrorAction SilentlyContinue }; Start-Sleep -Seconds 2; Start-Process powershell -ArgumentList '-ExecutionPolicy Bypass -WindowStyle Hidden -File \\"${startScript}\\"' -WindowStyle Hidden"`,
+      command: `powershell -Command "${stopBlock}; Start-Sleep -Seconds 2; ${startBlock}"`,
       deferred: true,
     }
     commands['update-agent'] = {
       description: 'Update @geekbeer/minion to latest version and restart',
-      command: `npm install -g @geekbeer/minion@latest && powershell -Command "$pid = Get-Content '${pidFile}' -ErrorAction SilentlyContinue; if ($pid) { Get-CimInstance Win32_Process -Filter \\"ParentProcessId = $pid\\" -ErrorAction SilentlyContinue | ForEach-Object { Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue }; Stop-Process -Id $pid -Force -ErrorAction SilentlyContinue }; Start-Sleep -Seconds 2; Start-Process powershell -ArgumentList '-ExecutionPolicy Bypass -WindowStyle Hidden -File \\"${startScript}\\"' -WindowStyle Hidden"`,
+      command: buildUpdateScript(
+        'npm install -g @geekbeer/minion@latest',
+        stopBlock,
+        startBlock,
+      ),
       deferred: true,
     }
     commands['update-agent-dev'] = {
       description: 'Update @geekbeer/minion from Verdaccio (dev) and restart',
-      command: `npm install -g @geekbeer/minion@latest --registry http://verdaccio:4873 && powershell -Command "$pid = Get-Content '${pidFile}' -ErrorAction SilentlyContinue; if ($pid) { Get-CimInstance Win32_Process -Filter \\"ParentProcessId = $pid\\" -ErrorAction SilentlyContinue | ForEach-Object { Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue }; Stop-Process -Id $pid -Force -ErrorAction SilentlyContinue }; Start-Sleep -Seconds 2; Start-Process powershell -ArgumentList '-ExecutionPolicy Bypass -WindowStyle Hidden -File \\"${startScript}\\"' -WindowStyle Hidden"`,
+      command: buildUpdateScript(
+        'npm install -g @geekbeer/minion@latest --registry http://verdaccio:4873',
+        stopBlock,
+        startBlock,
+      ),
       deferred: true,
     }
     commands['status-services'] = {
@@ -80,12 +141,20 @@ function buildAllowedCommands(procMgr) {
     }
     commands['update-agent'] = {
       description: 'Update @geekbeer/minion to latest version and restart',
-      command: 'npm install -g @geekbeer/minion@latest && nssm restart minion-agent',
+      command: buildUpdateScript(
+        'npm install -g @geekbeer/minion@latest',
+        'nssm stop minion-agent',
+        'nssm start minion-agent',
+      ),
       deferred: true,
     }
     commands['update-agent-dev'] = {
       description: 'Update @geekbeer/minion from Verdaccio (dev) and restart',
-      command: 'npm install -g @geekbeer/minion@latest --registry http://verdaccio:4873 && nssm restart minion-agent',
+      command: buildUpdateScript(
+        'npm install -g @geekbeer/minion@latest --registry http://verdaccio:4873',
+        'nssm stop minion-agent',
+        'nssm start minion-agent',
+      ),
       deferred: true,
     }
     commands['restart-display'] = {
@@ -104,12 +173,20 @@ function buildAllowedCommands(procMgr) {
     }
     commands['update-agent'] = {
       description: 'Update @geekbeer/minion to latest version and restart',
-      command: 'npm install -g @geekbeer/minion@latest && net stop minion-agent & net start minion-agent',
+      command: buildUpdateScript(
+        'npm install -g @geekbeer/minion@latest',
+        'net stop minion-agent',
+        'net start minion-agent',
+      ),
       deferred: true,
     }
     commands['update-agent-dev'] = {
       description: 'Update @geekbeer/minion from Verdaccio (dev) and restart',
-      command: 'npm install -g @geekbeer/minion@latest --registry http://verdaccio:4873 && net stop minion-agent & net start minion-agent',
+      command: buildUpdateScript(
+        'npm install -g @geekbeer/minion@latest --registry http://verdaccio:4873',
+        'net stop minion-agent',
+        'net start minion-agent',
+      ),
       deferred: true,
     }
     commands['status-services'] = {

package/win/routes/chat.js

@@ -4,11 +4,12 @@
  * Same as linux/routes/chat.js but with Windows-compatible PATH and shell construction.
  *
  * Endpoints:
- *   POST   /api/chat          - Send message, get SSE stream
- *   GET    /api/chat/session   - Get active session (messages + session_id)
- *   POST   /api/chat/clear     - Clear session and start fresh
- *   POST   /api/chat/abort     - Kill the active LLM CLI process
- *   POST   /api/chat/reset     - Summarize conversation and start fresh session
+ *   POST   /api/chat              - Send message, get SSE stream
+ *   GET    /api/chat/session       - Get active session (messages + session_id)
+ *   POST   /api/chat/clear         - Clear session and start fresh
+ *   POST   /api/chat/abort         - Kill the active LLM CLI process
+ *   POST   /api/chat/reset         - Summarize conversation and start fresh session
+ *   POST   /api/chat/end-of-day    - Generate daily log + extract memories from conversation
  */
 
 const { spawn } = require('child_process')
@@ -17,7 +18,10 @@ const path = require('path')
 const { verifyToken } = require('../../core/lib/auth')
 const { config } = require('../../core/config')
 const chatStore = require('../../core/stores/chat-store')
+const memoryStore = require('../../core/stores/memory-store')
+const dailyLogStore = require('../../core/stores/daily-log-store')
 const { buildExtendedPath } = require('../../core/lib/platform')
+const { runEndOfDay } = require('../../core/lib/end-of-day')
 
 let activeChatChild = null
 
@@ -34,7 +38,7 @@ async function chatRoutes(fastify) {
       return { success: false, error: 'message is required' }
     }
 
-    const prompt = context ? buildContextPrefix(message, context) : message
+    const prompt = await buildContextPrefix(message, context, session_id)
     const currentSessionId = session_id || null
 
     if (currentSessionId) {
@@ -142,10 +146,45 @@ async function chatRoutes(fastify) {
     console.log(`[Chat] session reset with summary (${summary?.length || 0} chars)`)
     return { success: true, summary }
   })
+
+  // POST /api/chat/end-of-day - Generate daily log + extract memories
+  fastify.post('/api/chat/end-of-day', async (request, reply) => {
+    if (!verifyToken(request)) {
+      reply.code(401)
+      return { success: false, error: 'Unauthorized' }
+    }
+
+    const { clear_session = false } = request.body || {}
+
+    const result = await runEndOfDay({
+      runQuickLlmCall,
+      clearSession: clear_session,
+    })
+
+    return { success: true, ...result }
+  })
 }
 
-function buildContextPrefix(message, context) {
+async function buildContextPrefix(message, context, sessionId) {
   const parts = []
+
+  // Inject memory + daily logs on new sessions only (not on --resume)
+  if (!sessionId) {
+    try {
+      const memorySnippet = await memoryStore.getContextSnippet(2000)
+      const dailyLogSnippet = await dailyLogStore.getContextSnippet(3, 1500)
+
+      if (memorySnippet) {
+        parts.push('[ミニオンメモリ（長期記憶）]', memorySnippet, '')
+      }
+      if (dailyLogSnippet) {
+        parts.push('[最近のデイリーログ]', dailyLogSnippet, '')
+      }
+    } catch (err) {
+      console.error('[Chat] Failed to load memory/daily-log context:', err.message)
+    }
+  }
+
   if (context) {
     switch (context.type) {
       case 'skill':
@@ -218,7 +257,8 @@ function streamLlmResponse(res, prompt, sessionId) {
 
     const args = ['-p', '--verbose', '--model', 'sonnet', '--output-format', 'stream-json']
     if (sessionId) args.push('--resume', sessionId)
-    args.push(prompt)
+    // Prompt is passed via stdin (not as CLI argument) to avoid
+    // shell argument parsing issues with spaces/special characters.
 
     console.log(`[Chat] spawning: ${binary} ${sessionId ? `--resume ${sessionId}` : '(new session)'}`)
 
@@ -236,6 +276,8 @@ function streamLlmResponse(res, prompt, sessionId) {
     })
 
     activeChatChild = child
+    // Write prompt to stdin and close — claude -p reads from stdin when no positional arg
+    child.stdin.write(prompt)
     child.stdin.end()
 
     console.log(`[Chat] child PID: ${child.pid}`)
@@ -401,7 +443,7 @@ function runQuickLlmCall(prompt) {
     const binary = fs.existsSync(binaryPath) ? binaryPath : binaryName
     const extendedPath = buildExtendedPath(config.HOME_DIR)
 
-    const args = ['-p', '--model', 'haiku', '--max-turns', '1', '--output-format', 'json', prompt]
+    const args = ['-p', '--model', 'haiku', '--max-turns', '1', '--output-format', 'json']
 
     const child = spawn(binary, args, {
       cwd: config.HOME_DIR,
@@ -416,6 +458,7 @@ function runQuickLlmCall(prompt) {
       },
     })
 
+    child.stdin.write(prompt)
     child.stdin.end()
 
     let stdout = ''
@@ -441,4 +484,4 @@ function runQuickLlmCall(prompt) {
   })
 }
 
-module.exports = { chatRoutes }
+module.exports = { chatRoutes, runQuickLlmCall }

package/win/routes/config.js

@@ -13,7 +13,12 @@ const { clearLlmCache } = require('../../core/lib/llm-checker')
 const { config, updateConfig } = require('../../core/config')
 const { resolveEnvFilePath } = require('../../core/lib/platform')
 
-const ALLOWED_ENV_KEYS = ['LLM_COMMAND']
+const reflectionScheduler = require('../../core/lib/reflection-scheduler')
+
+const ALLOWED_ENV_KEYS = ['LLM_COMMAND', 'REFLECTION_TIME', 'TIMEZONE']
+
+/** Keys that trigger a reflection scheduler reschedule when changed */
+const REFLECTION_KEYS = ['REFLECTION_TIME', 'TIMEZONE']
 
 const BACKUP_PATHS = [
   '~/.claude',
@@ -217,6 +222,12 @@ function configRoutes(fastify, _opts, done) {
       updateConfig(key, value)
 
       clearLlmCache()
+
+      // Reschedule reflection if relevant key changed
+      if (REFLECTION_KEYS.includes(key)) {
+        reflectionScheduler.reschedule()
+      }
+
       return { success: true, restart_required: false }
     } catch (err) {
       console.error(`[Config] Failed to update ${key} in ${envPath}:`, err.message)

package/win/routine-runner.js

@@ -17,6 +17,7 @@ const executionStore = require('../core/stores/execution-store')
 const routineStore = require('../core/stores/routine-store')
 const logManager = require('../core/lib/log-manager')
 const { MARKER_DIR, buildExtendedPath } = require('../core/lib/platform')
+const variableStore = require('../core/stores/variable-store')
 const { activeSessions } = require('./workflow-runner')
 
 const activeJobs = new Map()
@@ -105,8 +106,11 @@ async function executeRoutineSession(routine, executionId, skillNames) {
     const escapedPrompt = prompt.replace(/'/g, "''")
     const llmCommand = config.LLM_COMMAND.replace(/\{prompt\}/g, escapedPrompt)
 
+    // Inject minion variables/secrets (routines don't receive HQ vars)
+    const injectedEnv = variableStore.buildEnv()
     const env = {
       ...process.env,
+      ...injectedEnv,
       HOME: homeDir,
       USERPROFILE: homeDir,
       PATH: extendedPath,

package/win/server.js

@@ -16,6 +16,7 @@ const fastify = require('fastify')({ logger: true })
 const { config, validate, isHqConfigured } = require('../core/config')
 const { sendHeartbeat } = require('../core/api')
 const { version } = require('../package.json')
+const { getCapabilities } = require('../core/lib/capability-checker')
 const workflowStore = require('../core/stores/workflow-store')
 const routineStore = require('../core/stores/routine-store')
 
@@ -30,12 +31,13 @@ const routineRunner = require('./routine-runner')
 // Pull-model daemons (from core/)
 const stepPoller = require('../core/lib/step-poller')
 const revisionWatcher = require('../core/lib/revision-watcher')
+const reflectionScheduler = require('../core/lib/reflection-scheduler')
 const { commandRoutes, getProcessManager, getAllowedCommands } = require('./routes/commands')
 const { terminalRoutes, cleanupSessions } = require('./routes/terminal')
 const { startTerminalServer, stopTerminalServer } = require('./terminal-server')
 const { fileRoutes } = require('./routes/files')
 const { directiveRoutes } = require('./routes/directives')
-const { chatRoutes } = require('./routes/chat')
+const { chatRoutes, runQuickLlmCall } = require('./routes/chat')
 const { configRoutes } = require('./routes/config')
 
 // Compatible route modules (reused from Linux)
@@ -45,6 +47,9 @@ const { skillRoutes } = require('../core/routes/skills')
 const { workflowRoutes } = require('../core/routes/workflows')
 const { routineRoutes } = require('../core/routes/routines')
 const { authRoutes } = require('../core/routes/auth')
+const { variableRoutes } = require('../core/routes/variables')
+const { memoryRoutes } = require('../core/routes/memory')
+const { dailyLogRoutes } = require('../core/routes/daily-logs')
 
 // Validate configuration
 validate()
@@ -69,7 +74,7 @@ async function shutdown(signal) {
   if (isHqConfigured()) {
     try {
       await Promise.race([
-        sendHeartbeat({ status: 'offline', version }),
+        sendHeartbeat({ status: 'offline', version, capabilities: getCapabilities() }),
         new Promise(resolve => setTimeout(resolve, 3000)),
       ])
     } catch {
@@ -79,6 +84,7 @@ async function shutdown(signal) {
 
   stepPoller.stop()
   revisionWatcher.stop()
+  reflectionScheduler.stop()
   workflowRunner.stopAll()
   routineRunner.stopAll()
   stopTerminalServer()
@@ -192,6 +198,9 @@ async function registerRoutes(app) {
   await app.register(workflowRoutes, { workflowRunner })
   await app.register(routineRoutes, { routineRunner })
   await app.register(authRoutes)
+  await app.register(variableRoutes)
+  await app.register(memoryRoutes)
+  await app.register(dailyLogRoutes)
 
   // Windows-specific routes
   await app.register(commandRoutes)
@@ -249,20 +258,23 @@ async function start() {
       console.error('[Server] Failed to load cached routines:', err.message)
     }
 
+    // Start reflection scheduler (self-reflection time)
+    reflectionScheduler.start(runQuickLlmCall)
+
     if (isHqConfigured()) {
       console.log(`[Server] HQ URL: ${config.HQ_URL}`)
 
       // Send initial online heartbeat
       const { getStatus } = require('../core/routes/health')
       const { currentTask } = getStatus()
-      sendHeartbeat({ status: 'online', current_task: currentTask, version }).catch(err => {
+      sendHeartbeat({ status: 'online', current_task: currentTask, version, capabilities: getCapabilities() }).catch(err => {
         console.error('[Heartbeat] Initial heartbeat failed:', err.message)
       })
 
       // Start periodic heartbeat
       heartbeatTimer = setInterval(() => {
         const { currentStatus, currentTask } = getStatus()
-        sendHeartbeat({ status: currentStatus, current_task: currentTask, version }).catch(err => {
+        sendHeartbeat({ status: currentStatus, current_task: currentTask, version, capabilities: getCapabilities() }).catch(err => {
           console.error('[Heartbeat] Periodic heartbeat failed:', err.message)
         })
       }, HEARTBEAT_INTERVAL_MS)

package/win/workflow-runner.js

@@ -24,6 +24,7 @@ const executionStore = require('../core/stores/execution-store')
 const workflowStore = require('../core/stores/workflow-store')
 const logManager = require('../core/lib/log-manager')
 const { buildExtendedPath } = require('../core/lib/platform')
+const variableStore = require('../core/stores/variable-store')
 
 // Active cron jobs keyed by workflow ID
 const activeJobs = new Map()
@@ -112,9 +113,11 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
     const escapedPrompt = prompt.replace(/'/g, "''")
     const llmCommand = config.LLM_COMMAND.replace(/\{prompt\}/g, escapedPrompt)
 
-    // Build environment
+    // Build environment: base + minion variables/secrets + extra vars from HQ
+    const injectedEnv = variableStore.buildEnv(options.extraEnv || {})
     const env = {
       ...process.env,
+      ...injectedEnv,
       HOME: homeDir,
       USERPROFILE: homeDir,
       PATH: extendedPath,