@geekbeer/minion 1.0.3 → 1.0.5

package/api.js

@@ -37,21 +37,6 @@ async function request(endpoint, options = {}) {
   return data
 }
 
-/**
- * Send heartbeat with status update
- * @param {string} status - 'online' | 'offline' | 'busy'
- * @param {string|null} currentTask - Current task description
- */
-async function sendHeartbeat(status, currentTask = null) {
-  return request('/heartbeat', {
-    method: 'POST',
-    body: JSON.stringify({
-      status,
-      current_task: currentTask,
-    }),
-  })
-}
-
 /**
  * Send a log entry
  * @param {string} message - Log message
@@ -71,6 +56,5 @@ async function log(message, level = 'info', metadata = null) {
 
 module.exports = {
   request,
-  sendHeartbeat,
   log,
 }

package/config.js

@@ -7,11 +7,10 @@
  * - MINION_ID: The minion's UUID (assigned by HQ)
  *
  * If HQ_URL and API_TOKEN are not set, the agent runs in standalone mode
- * (no heartbeat, no HQ communication).
+ * (no HQ communication).
  *
  * Other optional environment variables:
  * - AGENT_PORT: Port for the local agent server (default: 3001)
- * - HEARTBEAT_INTERVAL: Interval between heartbeats in seconds (default: 30)
  */
 
 const config = {
@@ -22,9 +21,6 @@ const config = {
 
   // Server settings
   AGENT_PORT: parseInt(process.env.AGENT_PORT, 10) || 3001,
-
-  // Heartbeat interval in seconds
-  HEARTBEAT_INTERVAL: parseInt(process.env.HEARTBEAT_INTERVAL, 10) || 30,
 }
 
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@geekbeer/minion",
-  "version": "1.0.3",
-  "description": "AI Agent runtime for Minion - manages heartbeat, status, and skill deployment on VPS",
+  "version": "1.0.5",
+  "description": "AI Agent runtime for Minion - manages status and skill deployment on VPS",
   "main": "server.js",
   "bin": {
     "minion-cli": "./minion-cli.sh"
@@ -9,7 +9,6 @@
   "files": [
     "server.js",
     "config.js",
-    "heartbeat.js",
     "api.js",
     "minion-cli.sh",
     ".env.example"

package/server.js

@@ -12,9 +12,12 @@ const execAsync = promisify(exec)
 
 const fastify = require('fastify')({ logger: true })
 const { config, validate, isHqConfigured } = require('./config')
-const heartbeat = require('./heartbeat')
 const api = require('./api')
 
+// In-memory status (read by health check via GET /api/status)
+let currentStatus = 'online'
+let currentTask = null
+
 // Detect process manager (matching minion-cli.sh logic)
 function detectProcessManager() {
   try {
@@ -124,8 +127,8 @@ fastify.get('/api/health', async () => {
 // Get current status
 fastify.get('/api/status', async () => {
   return {
-    status: heartbeat.getStatus(),
-    current_task: heartbeat.getTask(),
+    status: currentStatus,
+    current_task: currentTask,
     uptime: process.uptime(),
     timestamp: new Date().toISOString(),
   }
@@ -135,23 +138,19 @@ fastify.get('/api/status', async () => {
 fastify.post('/api/status', async (request, reply) => {
   const { status, current_task } = request.body || {}
 
-  try {
-    if (status) {
-      heartbeat.setStatus(status)
-    }
-
-    if (current_task !== undefined) {
-      heartbeat.setTask(current_task)
+  if (status) {
+    if (!['online', 'offline', 'busy'].includes(status)) {
+      reply.code(400)
+      return { success: false, error: `Invalid status: ${status}` }
     }
+    currentStatus = status
+  }
 
-    // Immediately send heartbeat with new status
-    await heartbeat.sendHeartbeat()
-
-    return { success: true }
-  } catch (error) {
-    reply.code(400)
-    return { success: false, error: error.message }
+  if (current_task !== undefined) {
+    currentTask = current_task || null
   }
+
+  return { success: true }
 })
 
 // Send log entry
@@ -455,17 +454,7 @@ ${content}`
 async function shutdown(signal) {
   console.log(`[Server] Received ${signal}, shutting down...`)
 
-  // Send final offline heartbeat if HQ is configured
-  if (isHqConfigured()) {
-    heartbeat.setStatus('offline')
-    try {
-      await heartbeat.sendHeartbeat()
-      console.log('[Server] Status set to offline')
-    } catch (error) {
-      console.error(`[Server] Failed to update status: ${error.message}`)
-    }
-    heartbeat.stop()
-  }
+  currentStatus = 'offline'
 
   // Close server
   await fastify.close()
@@ -544,9 +533,6 @@ async function start() {
     if (isHqConfigured()) {
       console.log(`[Server] HQ URL: ${config.HQ_URL}`)
 
-      // Start heartbeat
-      heartbeat.start()
-
       // Sync VNC password from HQ (non-blocking, run in background)
       syncVncPassword().catch(err => {
         console.error('[VNC] Background sync failed:', err.message)

package/heartbeat.js

@@ -1,91 +0,0 @@
-/**
- * Heartbeat Manager for Minion Agent
- * Sends periodic heartbeats to the API
- */
-
-const { config } = require('./config')
-const api = require('./api')
-
-// Current state
-let currentStatus = 'online'
-let currentTask = null
-let intervalId = null
-
-/**
- * Start the heartbeat loop
- */
-function start() {
-  console.log(`[Heartbeat] Starting with interval: ${config.HEARTBEAT_INTERVAL}s`)
-
-  // Send initial heartbeat
-  sendHeartbeat()
-
-  // Schedule periodic heartbeats
-  intervalId = setInterval(sendHeartbeat, config.HEARTBEAT_INTERVAL * 1000)
-}
-
-/**
- * Stop the heartbeat loop
- */
-function stop() {
-  if (intervalId) {
-    clearInterval(intervalId)
-    intervalId = null
-    console.log('[Heartbeat] Stopped')
-  }
-}
-
-/**
- * Send a heartbeat to the API
- */
-async function sendHeartbeat() {
-  try {
-    await api.sendHeartbeat(currentStatus, currentTask)
-    console.log(`[Heartbeat] Sent: status=${currentStatus}, task=${currentTask || 'none'}`)
-  } catch (error) {
-    console.error(`[Heartbeat] Failed: ${error.message}`)
-  }
-}
-
-/**
- * Set the current status
- * @param {'online'|'offline'|'busy'} status
- */
-function setStatus(status) {
-  if (!['online', 'offline', 'busy'].includes(status)) {
-    throw new Error(`Invalid status: ${status}`)
-  }
-  currentStatus = status
-}
-
-/**
- * Set the current task
- * @param {string|null} task
- */
-function setTask(task) {
-  currentTask = task || null
-}
-
-/**
- * Get the current status
- */
-function getStatus() {
-  return currentStatus
-}
-
-/**
- * Get the current task
- */
-function getTask() {
-  return currentTask
-}
-
-module.exports = {
-  start,
-  stop,
-  sendHeartbeat,
-  setStatus,
-  setTask,
-  getStatus,
-  getTask,
-}