@geekbeer/minion 2.48.3 → 2.49.1

package/core/routes/skills.js

@@ -48,15 +48,15 @@ function parseFrontmatter(content) {
  * @param {string} opts.content - Skill body (markdown without frontmatter)
  * @param {string} [opts.description] - Skill description for frontmatter
  * @param {string} [opts.display_name] - Display name for frontmatter
- * @param {Array<{filename: string, content: string}>} [opts.references] - Reference files
- * @returns {Promise<{path: string, references_count: number}>}
+ * @param {Array<{path: string, content?: string}>} [opts.files] - Skill files from HQ storage
+ * @returns {Promise<{path: string, files_count: number}>}
  */
-async function writeSkillToLocal(name, { content, description, display_name, type, references = [] }) {
+async function writeSkillToLocal(name, { content, description, display_name, type, files = [] }) {
   const skillDir = path.join(config.HOME_DIR, '.claude', 'skills', name)
-  const referencesDir = path.join(skillDir, 'references')
+  const filesDir = path.join(skillDir, 'files')
 
   await fs.mkdir(skillDir, { recursive: true })
-  await fs.mkdir(referencesDir, { recursive: true })
+  await fs.mkdir(filesDir, { recursive: true })
 
   // Build frontmatter with all available metadata
   const frontmatterLines = [
@@ -72,15 +72,15 @@ async function writeSkillToLocal(name, { content, description, display_name, typ
     'utf-8'
   )
 
-  // Write reference files
-  for (const ref of references) {
-    if (ref.filename && ref.content) {
-      const safeFilename = path.basename(ref.filename)
-      await fs.writeFile(path.join(referencesDir, safeFilename), ref.content, 'utf-8')
+  // Write skill files
+  for (const file of files) {
+    if (file.path && file.content) {
+      const safeFilename = path.basename(file.path)
+      await fs.writeFile(path.join(filesDir, safeFilename), file.content, 'utf-8')
     }
   }
 
-  return { path: skillDir, references_count: references.length }
+  return { path: skillDir, files_count: files.length }
 }
 
 /**
@@ -99,17 +99,27 @@ async function pushSkillToHQ(name) {
   const rawContent = await fs.readFile(skillMdPath, 'utf-8')
   const { metadata, body } = parseFrontmatter(rawContent)
 
-  // Read references
-  const referencesDir = path.join(skillDir, 'references')
-  const references = []
+  // Read skill files
+  const filesDir = path.join(skillDir, 'files')
+  const files = []
   try {
-    const refEntries = await fs.readdir(referencesDir)
-    for (const filename of refEntries) {
-      const refContent = await fs.readFile(path.join(referencesDir, filename), 'utf-8')
-      references.push({ filename, content: refContent })
+    const fileEntries = await fs.readdir(filesDir)
+    for (const filename of fileEntries) {
+      const fileContent = await fs.readFile(path.join(filesDir, filename), 'utf-8')
+      files.push({ path: filename, content: fileContent })
     }
   } catch {
-    // No references directory
+    // No files directory — try legacy references/ for backward compatibility
+    try {
+      const legacyDir = path.join(skillDir, 'references')
+      const legacyEntries = await fs.readdir(legacyDir)
+      for (const filename of legacyEntries) {
+        const fileContent = await fs.readFile(path.join(legacyDir, filename), 'utf-8')
+        files.push({ path: filename, content: fileContent })
+      }
+    } catch {
+      // No files at all
+    }
   }
 
   return api.request('/skills', {
@@ -120,7 +130,7 @@ async function pushSkillToHQ(name) {
       description: metadata.description || '',
       content: body,
       type: metadata.type || 'workflow',
-      references,
+      files,
     }),
   })
 }
@@ -243,7 +253,7 @@ async function skillRoutes(fastify, opts) {
         description: skill.description,
         display_name: skill.display_name,
         type: skill.type,
-        references: skill.references || [],
+        files: skill.files || [],
       })
 
       console.log(`[Skills] Skill fetched and deployed: ${name}`)

package/linux/workflow-runner.js

@@ -277,6 +277,23 @@ async function runWorkflow(workflow, options = {}) {
     log_file: logFile,
   })
 
+  // Extract summary from execution log file (captured via tmux pipe-pane)
+  let summary = result.success
+    ? `All skills completed successfully: ${pipelineSkillNames.join(', ')}`
+    : `Workflow failed: ${result.error || 'unknown error'}`
+  try {
+    const logData = await logManager.readLog(executionId, { tail: 200 })
+    if (logData && logData.content && logData.content.trim().length > 0) {
+      const MAX_SUMMARY_LENGTH = 10000
+      const content = logData.content.trim()
+      summary = content.length > MAX_SUMMARY_LENGTH
+        ? content.slice(-MAX_SUMMARY_LENGTH)
+        : content
+    }
+  } catch (err) {
+    console.error(`[WorkflowRunner] Failed to read log for summary: ${err.message}`)
+  }
+
   // Report outcome via local API (same data the execution-report skill used to send)
   try {
     const resp = await fetch(`http://localhost:${config.AGENT_PORT || 8080}/api/executions/${executionId}/outcome`, {
@@ -284,9 +301,7 @@ async function runWorkflow(workflow, options = {}) {
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({
         outcome,
-        summary: result.success
-          ? `All skills completed successfully: ${pipelineSkillNames.join(', ')}`
-          : `Workflow failed: ${result.error || 'unknown error'}`,
+        summary,
       }),
     })
     if (!resp.ok) {

package/package.json

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

package/win/workflow-runner.js

@@ -183,12 +183,19 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
         // Close log stream
         try { logStream.end() } catch { /* ignore */ }
 
+        // Capture output for summary
+        const MAX_SUMMARY_LENGTH = 10000
+        const output = outputBuffer.trim()
+        const capturedOutput = output.length > MAX_SUMMARY_LENGTH
+          ? output.slice(-MAX_SUMMARY_LENGTH)
+          : output
+
         if (exitCode === 0) {
           console.log(`[WorkflowRunner] Workflow ${workflow.name} completed successfully`)
-          resolve({ success: true, sessionName })
+          resolve({ success: true, sessionName, output: capturedOutput })
         } else {
           console.error(`[WorkflowRunner] Workflow ${workflow.name} failed with exit code: ${exitCode}`)
-          resolve({ success: false, error: `Exit code: ${exitCode}`, sessionName })
+          resolve({ success: false, error: `Exit code: ${exitCode}`, sessionName, output: capturedOutput })
         }
       })
     })
@@ -261,6 +268,13 @@ async function runWorkflow(workflow, options = {}) {
     log_file: logFile,
   })
 
+  // Use captured output as summary, falling back to generic message
+  const summary = (result.output && result.output.length > 0)
+    ? result.output
+    : result.success
+      ? `All skills completed successfully: ${pipelineSkillNames.join(', ')}`
+      : `Workflow failed: ${result.error || 'unknown error'}`
+
   // Report outcome via local API
   try {
     const resp = await fetch(`http://localhost:${config.AGENT_PORT || 8080}/api/executions/${executionId}/outcome`, {
@@ -268,9 +282,7 @@ async function runWorkflow(workflow, options = {}) {
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({
         outcome,
-        summary: result.success
-          ? `All skills completed successfully: ${pipelineSkillNames.join(', ')}`
-          : `Workflow failed: ${result.error || 'unknown error'}`,
+        summary,
       }),
     })
     if (!resp.ok) {