@geekbeer/minion 3.26.0 → 3.27.1

package/core/lib/dag-step-poller.js

@@ -125,6 +125,7 @@ async function executeSkillNode(node) {
     assigned_role,
     input_data,
     revision_feedback,
+    review_history,
   } = node
 
   console.log(
@@ -187,6 +188,9 @@ async function executeSkillNode(node) {
     if (revision_feedback) {
       runPayload.revision_feedback = revision_feedback
     }
+    if (review_history && review_history.length > 0) {
+      runPayload.review_history = review_history
+    }
 
     const runUrl = `http://localhost:${config.AGENT_PORT || 8080}/api/skills/run`
     const runResp = await fetch(runUrl, {

package/core/routes/skills.js

@@ -342,6 +342,7 @@ async function skillRoutes(fastify, opts) {
       workflow_name,
       role,
       revision_feedback,
+      review_history,
       dag_node_execution_id,
     } = request.body || {}
 
@@ -391,6 +392,7 @@ async function skillRoutes(fastify, opts) {
     const runOptions = {}
     if (role) runOptions.role = role
     if (revision_feedback) runOptions.revisionFeedback = revision_feedback
+    if (review_history && review_history.length > 0) runOptions.reviewHistory = review_history
 
     // Run asynchronously — respond immediately
     const executionPromise = (async () => {

package/linux/routes/files.js

@@ -16,6 +16,28 @@ const fsSync = require('fs')
 const path = require('path')
 const { spawn } = require('child_process')
 
+/** Map file extensions to MIME types for preview */
+const PREVIEW_MIME_TYPES = {
+  '.md': 'text/markdown; charset=utf-8',
+  '.txt': 'text/plain; charset=utf-8',
+  '.log': 'text/plain; charset=utf-8',
+  '.csv': 'text/csv; charset=utf-8',
+  '.json': 'application/json; charset=utf-8',
+  '.xml': 'application/xml; charset=utf-8',
+  '.yaml': 'text/yaml; charset=utf-8',
+  '.yml': 'text/yaml; charset=utf-8',
+  '.toml': 'text/plain; charset=utf-8',
+  '.pdf': 'application/pdf',
+  '.png': 'image/png',
+  '.jpg': 'image/jpeg',
+  '.jpeg': 'image/jpeg',
+  '.gif': 'image/gif',
+  '.webp': 'image/webp',
+  '.svg': 'image/svg+xml',
+  '.ico': 'image/x-icon',
+  '.bmp': 'image/bmp',
+}
+
 const { verifyToken } = require('../../core/lib/auth')
 const { config } = require('../../core/config')
 
@@ -176,9 +198,13 @@ async function fileRoutes(fastify) {
       }
 
       const filename = path.basename(resolved)
+      const isPreview = request.query.preview === 'true'
+      const ext = path.extname(filename).toLowerCase()
+      const mimeType = isPreview && PREVIEW_MIME_TYPES[ext]
+
       reply
-        .type('application/octet-stream')
-        .header('Content-Disposition', `attachment; filename="${encodeURIComponent(filename)}"`)
+        .type(mimeType || 'application/octet-stream')
+        .header('Content-Disposition', `${isPreview && mimeType ? 'inline' : 'attachment'}; filename="${encodeURIComponent(filename)}"`)
         .header('Content-Length', stat.size)
 
       return reply.send(fsSync.createReadStream(resolved))

package/linux/workflow-runner.js

@@ -77,10 +77,20 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
     ? `You are acting as the ${options.role} role in this session. Read ~/.minion/roles/${options.role}.md for your role guidelines before proceeding.\n\n`
     : ''
 
-  // Inject revision feedback if this is a re-execution after reviewer requested changes
-  const revisionContext = options.revisionFeedback
-    ? `## Revision Feedback\nThe reviewer requested changes to your previous output. Address the following feedback:\n${options.revisionFeedback}\n\n`
-    : ''
+  // Inject revision feedback and review history if this is a re-execution after reviewer requested changes
+  let revisionContext = ''
+  if (options.reviewHistory && options.reviewHistory.length > 0) {
+    const historyLines = options.reviewHistory.map((entry, i) => {
+      const label = entry.status === 'revision_requested' ? 'Revision Requested'
+        : entry.status === 'approved' ? 'Approved'
+        : entry.status === 'rejected' ? 'Rejected'
+        : entry.status
+      return `${i + 1}. **${label}** (${entry.reviewed_at})${entry.comment ? `\n   ${entry.comment}` : ''}`
+    }).join('\n')
+    revisionContext = `## Review History\nThis task has been reviewed ${options.reviewHistory.length} time(s). Address all feedback:\n${historyLines}\n\n`
+  } else if (options.revisionFeedback) {
+    revisionContext = `## Revision Feedback\nThe reviewer requested changes to your previous output. Address the following feedback:\n${options.revisionFeedback}\n\n`
+  }
 
   const prompt = `${rolePrefix}${revisionContext}Run the following skills in order: ${skillCommands}.`
 

package/package.json

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

package/win/routes/files.js

@@ -13,6 +13,28 @@ const zlib = require('zlib')
 const { verifyToken } = require('../../core/lib/auth')
 const { config } = require('../../core/config')
 
+/** Map file extensions to MIME types for preview */
+const PREVIEW_MIME_TYPES = {
+  '.md': 'text/markdown; charset=utf-8',
+  '.txt': 'text/plain; charset=utf-8',
+  '.log': 'text/plain; charset=utf-8',
+  '.csv': 'text/csv; charset=utf-8',
+  '.json': 'application/json; charset=utf-8',
+  '.xml': 'application/xml; charset=utf-8',
+  '.yaml': 'text/yaml; charset=utf-8',
+  '.yml': 'text/yaml; charset=utf-8',
+  '.toml': 'text/plain; charset=utf-8',
+  '.pdf': 'application/pdf',
+  '.png': 'image/png',
+  '.jpg': 'image/jpeg',
+  '.jpeg': 'image/jpeg',
+  '.gif': 'image/gif',
+  '.webp': 'image/webp',
+  '.svg': 'image/svg+xml',
+  '.ico': 'image/x-icon',
+  '.bmp': 'image/bmp',
+}
+
 const FILES_DIR = path.join(config.HOME_DIR, 'files')
 const MAX_UPLOAD_SIZE = 50 * 1024 * 1024
 
@@ -173,9 +195,13 @@ async function fileRoutes(fastify) {
       }
 
       const filename = path.basename(resolved)
+      const isPreview = request.query.preview === 'true'
+      const ext = path.extname(filename).toLowerCase()
+      const mimeType = isPreview && PREVIEW_MIME_TYPES[ext]
+
       reply
-        .type('application/octet-stream')
-        .header('Content-Disposition', `attachment; filename="${encodeURIComponent(filename)}"`)
+        .type(mimeType || 'application/octet-stream')
+        .header('Content-Disposition', `${isPreview && mimeType ? 'inline' : 'attachment'}; filename="${encodeURIComponent(filename)}"`)
         .header('Content-Length', stat.size)
 
       return reply.send(fsSync.createReadStream(resolved))

package/win/workflow-runner.js

@@ -85,9 +85,19 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
     ? `You are acting as the ${options.role} role in this session. Read ~/.minion/roles/${options.role}.md for your role guidelines before proceeding.\n\n`
     : ''
 
-  const revisionContext = options.revisionFeedback
-    ? `## Revision Feedback\nThe reviewer requested changes to your previous output. Address the following feedback:\n${options.revisionFeedback}\n\n`
-    : ''
+  let revisionContext = ''
+  if (options.reviewHistory && options.reviewHistory.length > 0) {
+    const historyLines = options.reviewHistory.map((entry, i) => {
+      const label = entry.status === 'revision_requested' ? 'Revision Requested'
+        : entry.status === 'approved' ? 'Approved'
+        : entry.status === 'rejected' ? 'Rejected'
+        : entry.status
+      return `${i + 1}. **${label}** (${entry.reviewed_at})${entry.comment ? `\n   ${entry.comment}` : ''}`
+    }).join('\n')
+    revisionContext = `## Review History\nThis task has been reviewed ${options.reviewHistory.length} time(s). Address all feedback:\n${historyLines}\n\n`
+  } else if (options.revisionFeedback) {
+    revisionContext = `## Revision Feedback\nThe reviewer requested changes to your previous output. Address the following feedback:\n${options.revisionFeedback}\n\n`
+  }
 
   const prompt = `${rolePrefix}${revisionContext}Run the following skills in order: ${skillCommands}.`