@geekbeer/minion 2.51.1 → 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/package.json

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