@geekbeer/minion 2.50.2 → 2.53.2

package/core/lib/capability-checker.js

@@ -11,6 +11,7 @@ const path = require('path')
 const { execSync } = require('child_process')
 const { config } = require('../config')
 const { IS_WINDOWS, buildExtendedPath } = require('./platform')
+const variableStore = require('../stores/variable-store')
 
 const CACHE_TTL_MS = 300000 // 5 minutes
 
@@ -79,8 +80,19 @@ function checkTool(name) {
 const TOOL_NAMES = ['git', 'node', 'npx', 'claude', 'docker', 'tmux']
 
 /**
- * Get all capabilities (MCP servers + CLI tools), cached for 5 minutes.
- * @returns {{ mcp_servers: { name: string, configured: boolean }[], cli_tools: { name: string, available: boolean, version?: string }[] }}
+ * Get all available environment variable keys (variables + secrets).
+ * Returns deduplicated key names without values (secrets stay hidden).
+ * @returns {string[]}
+ */
+function getEnvVarKeys() {
+  const varKeys = variableStore.listKeys('variables')
+  const secretKeys = variableStore.listKeys('secrets')
+  return [...new Set([...varKeys, ...secretKeys])]
+}
+
+/**
+ * Get all capabilities (MCP servers + CLI tools + env var keys), cached for 5 minutes.
+ * @returns {{ mcp_servers: { name: string, configured: boolean }[], cli_tools: { name: string, available: boolean, version?: string }[], env_var_keys: string[] }}
  */
 function getCapabilities() {
   const now = Date.now()
@@ -90,8 +102,9 @@ function getCapabilities() {
 
   const mcp_servers = getMcpServers()
   const cli_tools = TOOL_NAMES.map(name => checkTool(name))
+  const env_var_keys = getEnvVarKeys()
 
-  cachedResult = { mcp_servers, cli_tools }
+  cachedResult = { mcp_servers, cli_tools, env_var_keys }
   cachedAt = now
   return cachedResult
 }
@@ -102,4 +115,14 @@ function clearCapabilityCache() {
   cachedAt = 0
 }
 
-module.exports = { getCapabilities, clearCapabilityCache }
+/**
+ * Check arbitrary tool names on-demand (not cached).
+ * Used by readiness checks to verify skill-declared cli_tools.
+ * @param {string[]} names - Tool names to check
+ * @returns {{ name: string, available: boolean, version?: string }[]}
+ */
+function checkTools(names) {
+  return names.map(name => checkTool(name))
+}
+
+module.exports = { getCapabilities, clearCapabilityCache, checkTools }

package/core/routes/health.js

@@ -5,13 +5,14 @@
  * - GET  /api/health  - Health check
  * - GET  /api/status  - Get current status
  * - POST /api/status  - Update status
+ * - POST /api/capabilities/check-tools - Check arbitrary CLI tools on-demand
  */
 
 const { version } = require('../../package.json')
 const { config, isHqConfigured } = require('../config')
 const { sendHeartbeat } = require('../api')
 const { getLlmServices, isLlmCommandConfigured } = require('../lib/llm-checker')
-const { getCapabilities } = require('../lib/capability-checker')
+const { getCapabilities, checkTools } = require('../lib/capability-checker')
 
 function maskToken(token) {
   if (!token || token.length < 8) return token ? '***' : ''
@@ -105,6 +106,18 @@ async function healthRoutes(fastify) {
 
     return { success: true }
   })
+
+  // Check arbitrary CLI tools on-demand
+  fastify.post('/api/capabilities/check-tools', async (request, reply) => {
+    const { tools } = request.body || {}
+    if (!Array.isArray(tools) || tools.length === 0) {
+      reply.code(400)
+      return { error: 'tools must be a non-empty array of tool names' }
+    }
+    // Limit to 20 tools per request to avoid abuse
+    const limited = tools.slice(0, 20).filter(t => typeof t === 'string' && t.length > 0)
+    return { cli_tools: checkTools(limited) }
+  })
 }
 
 module.exports = {

package/core/routes/skills.js

@@ -58,17 +58,24 @@ async function writeSkillToLocal(name, { content, description, display_name, typ
   await fs.mkdir(skillDir, { recursive: true })
   await fs.mkdir(filesDir, { recursive: true })
 
-  // Build frontmatter with all available metadata
-  const frontmatterLines = [
-    `name: ${name}`,
-    display_name ? `display_name: ${display_name}` : null,
-    type ? `type: ${type}` : null,
-    `description: ${description || ''}`,
-  ].filter(Boolean).join('\n')
+  // If content already has frontmatter, write as-is; otherwise build frontmatter
+  const hasFrontmatter = content.startsWith('---\n')
+  let fileContent
+  if (hasFrontmatter) {
+    fileContent = content
+  } else {
+    const frontmatterLines = [
+      `name: ${name}`,
+      display_name ? `display_name: ${display_name}` : null,
+      type ? `type: ${type}` : null,
+      `description: ${description || ''}`,
+    ].filter(Boolean).join('\n')
+    fileContent = `---\n${frontmatterLines}\n---\n\n${content}`
+  }
 
   await fs.writeFile(
     path.join(skillDir, 'SKILL.md'),
-    `---\n${frontmatterLines}\n---\n\n${content}`,
+    fileContent,
     'utf-8'
   )
 
@@ -128,7 +135,7 @@ async function pushSkillToHQ(name) {
       name: metadata.name || name,
       display_name: metadata.display_name || metadata.name || name,
       description: metadata.description || '',
-      content: body,
+      content: rawContent,
       type: metadata.type || 'workflow',
       files,
     }),

package/core/routes/workflows.js

@@ -176,6 +176,7 @@ async function workflowRoutes(fastify, opts) {
           pipeline_skill_names: pipelineSkillNames,
           content: workflow.content || '',
           project_id: effectiveProjectId,
+          ...(workflow.pipeline_steps?.length > 0 && { pipeline_steps: workflow.pipeline_steps }),
         }),
       })
 
@@ -244,6 +245,10 @@ async function workflowRoutes(fastify, opts) {
       const updatedWorkflows = await workflowStore.upsertByName({
         name: workflow.name,
         pipeline_skill_names: workflow.pipeline_skill_names,
+        pipeline_steps: (workflow.pipeline || []).map(step => ({
+          assigned_role: step.assigned_role,
+          requires_review: step.requires_review || false,
+        })),
         content: workflow.content || '',
         project_id: workflow.project_id || null,
       })

package/core/stores/workflow-store.js

@@ -82,7 +82,7 @@ async function findByName(name) {
  * Upsert a workflow by name.
  * If exists: updates definition only (preserves schedule/local state).
  * If new: creates with inactive schedule.
- * @param {object} workflowData - { name, pipeline_skill_names, content, project_id }
+ * @param {object} workflowData - { name, pipeline_skill_names, pipeline_steps, content, project_id }
  * @returns {Promise<Array>} Updated workflows array
  */
 async function upsertByName(workflowData) {
@@ -99,11 +99,15 @@ async function upsertByName(workflowData) {
     if (workflowData.project_id !== undefined) {
       workflows[index].project_id = workflowData.project_id
     }
+    if (workflowData.pipeline_steps !== undefined) {
+      workflows[index].pipeline_steps = workflowData.pipeline_steps
+    }
   } else {
     workflows.push({
       id: crypto.randomUUID(),
       name: workflowData.name,
       pipeline_skill_names: workflowData.pipeline_skill_names,
+      pipeline_steps: workflowData.pipeline_steps || [],
       content: workflowData.content || '',
       project_id: workflowData.project_id || null,
       cron_expression: '',

package/linux/routes/chat.js

@@ -22,9 +22,8 @@ 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 { runEndOfDay } = require('../../core/lib/end-of-day')
+const { DATA_DIR } = require('../../core/lib/platform')
 
 /** @type {import('child_process').ChildProcess | null} */
 let activeChatChild = null
@@ -205,21 +204,24 @@ async function chatRoutes(fastify) {
 async function buildContextPrefix(message, context, sessionId) {
   const parts = []
 
-  // Inject memory + daily logs on new sessions only (not on --resume)
+  // Tell the LLM where to find memory and daily logs (content is NOT injected)
   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)
-    }
+    const memoryDir = require('path').join(DATA_DIR, 'memory')
+    const dailyLogDir = require('path').join(DATA_DIR, 'daily-logs')
+    parts.push(
+      '[長期記憶・デイリーログについて]',
+      'あなたには長期記憶（メモリ）とデイリーログがあります。内容は毎回読み込まれません。',
+      '必要なときだけ以下のファイルを読んで参照してください：',
+      `- メモリ索引: ${memoryDir}/MEMORY.md（概要一覧。詳細は個別の .md ファイルを読む）`,
+      `- デイリーログ: ${dailyLogDir}/YYYY-MM-DD.md（日付別の作業記録）`,
+      '',
+      '参照すべきタイミング:',
+      '- 過去の作業や決定事項を思い出す必要があるとき',
+      '- ユーザーの好みやフィードバックを確認したいとき',
+      '- 以前の作業の続きをするとき',
+      '- わからないことがあり、過去に記録した情報が役立ちそうなとき',
+      ''
+    )
   }
 
   if (context) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geekbeer/minion",
-  "version": "2.50.2",
+  "version": "2.53.2",
   "description": "AI Agent runtime for Minion - manages status and skill deployment on VPS",
   "main": "linux/server.js",
   "bin": {

package/win/routes/chat.js

@@ -18,9 +18,7 @@ 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 { buildExtendedPath, DATA_DIR } = require('../../core/lib/platform')
 const { runEndOfDay } = require('../../core/lib/end-of-day')
 
 let activeChatChild = null
@@ -168,21 +166,24 @@ async function chatRoutes(fastify) {
 async function buildContextPrefix(message, context, sessionId) {
   const parts = []
 
-  // Inject memory + daily logs on new sessions only (not on --resume)
+  // Tell the LLM where to find memory and daily logs (content is NOT injected)
   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)
-    }
+    const memoryDir = require('path').join(DATA_DIR, 'memory')
+    const dailyLogDir = require('path').join(DATA_DIR, 'daily-logs')
+    parts.push(
+      '[長期記憶・デイリーログについて]',
+      'あなたには長期記憶（メモリ）とデイリーログがあります。内容は毎回読み込まれません。',
+      '必要なときだけ以下のファイルを読んで参照してください：',
+      `- メモリ索引: ${memoryDir}/MEMORY.md（概要一覧。詳細は個別の .md ファイルを読む）`,
+      `- デイリーログ: ${dailyLogDir}/YYYY-MM-DD.md（日付別の作業記録）`,
+      '',
+      '参照すべきタイミング:',
+      '- 過去の作業や決定事項を思い出す必要があるとき',
+      '- ユーザーの好みやフィードバックを確認したいとき',
+      '- 以前の作業の続きをするとき',
+      '- わからないことがあり、過去に記録した情報が役立ちそうなとき',
+      ''
+    )
   }
 
   if (context) {