@swarmclawai/swarmclaw 0.8.4 → 0.8.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@swarmclawai/swarmclaw",
-  "version": "0.8.4",
+  "version": "0.8.5",
   "description": "Self-hosted AI agent orchestration dashboard — manage LLM providers, orchestrate agent swarms, schedule tasks, and bridge agents to chat platforms.",
   "license": "MIT",
   "publishConfig": {

package/src/lib/server/heartbeat-blocked-suppression.test.ts

@@ -0,0 +1,151 @@
+import assert from 'node:assert/strict'
+import { describe, it } from 'node:test'
+import {
+  stripBlockedItems,
+  isHeartbeatContentEffectivelyEmpty,
+  buildAgentHeartbeatPrompt,
+} from './heartbeat-service'
+
+describe('heartbeat blocked-item suppression', () => {
+  describe('stripBlockedItems', () => {
+    it('removes checklist items marked (blocked, no update)', () => {
+      const input = [
+        '# Heartbeat Tasks',
+        '## Active',
+        '- [ ] Pull SWGOH roster data (blocked, no update)',
+        '- [ ] Send daily summary',
+        '## Completed',
+        '- [x] Do laundry',
+      ].join('\n')
+
+      const result = stripBlockedItems(input)
+
+      assert.ok(!result.includes('SWGOH'), 'blocked item should be stripped')
+      assert.ok(result.includes('Send daily summary'), 'non-blocked item should remain')
+      assert.ok(result.includes('Do laundry'), 'completed item should remain')
+      assert.ok(result.includes('# Heartbeat Tasks'), 'headers should remain')
+    })
+
+    it('removes items with various blocked markers', () => {
+      const input = [
+        '- [ ] Task A (blocked: awaiting input)',
+        '- [ ] Task B (Blocked, pending user decision)',
+        '- [ ] Task C (BLOCKED)',
+        '- [ ] Task D - normal task',
+        '* [ ] Task E (blocked, no update)',
+      ].join('\n')
+
+      const result = stripBlockedItems(input)
+
+      assert.ok(!result.includes('Task A'), 'blocked: variant stripped')
+      assert.ok(!result.includes('Task B'), 'Blocked, variant stripped')
+      assert.ok(!result.includes('Task C'), 'BLOCKED variant stripped')
+      assert.ok(result.includes('Task D'), 'non-blocked task preserved')
+      assert.ok(!result.includes('Task E'), 'asterisk list item stripped')
+    })
+
+    it('preserves non-list lines that mention "blocked"', () => {
+      const input = [
+        '## Notes',
+        'Some items are blocked until user responds.',
+        '- [ ] Actual blocked task (blocked, no update)',
+      ].join('\n')
+
+      const result = stripBlockedItems(input)
+
+      assert.ok(result.includes('Some items are blocked'), 'prose mentioning blocked should stay')
+      assert.ok(!result.includes('Actual blocked task'), 'blocked list item should be stripped')
+    })
+
+    it('returns empty string for empty input', () => {
+      assert.equal(stripBlockedItems(''), '')
+      assert.equal(stripBlockedItems(null as unknown as string), '')
+    })
+
+    it('returns content unchanged when no blocked items', () => {
+      const input = '- [ ] Task A\n- [ ] Task B\n'
+      assert.equal(stripBlockedItems(input), input)
+    })
+  })
+
+  describe('blocked items + effectively empty', () => {
+    it('treats content with only blocked items as effectively empty', () => {
+      const input = [
+        '# Heartbeat Tasks',
+        '## Active',
+        '- [ ] Pull SWGOH data (blocked, no update)',
+        '## Completed',
+      ].join('\n')
+
+      const stripped = stripBlockedItems(input)
+      // After stripping, only headers remain — effectively empty
+      assert.equal(isHeartbeatContentEffectivelyEmpty(stripped), true)
+    })
+
+    it('treats content with blocked + active items as not empty', () => {
+      const input = [
+        '# Heartbeat Tasks',
+        '## Active',
+        '- [ ] Pull SWGOH data (blocked, no update)',
+        '- [ ] Send daily summary',
+      ].join('\n')
+
+      const stripped = stripBlockedItems(input)
+      assert.equal(isHeartbeatContentEffectivelyEmpty(stripped), false)
+    })
+  })
+
+  describe('buildAgentHeartbeatPrompt integration', () => {
+    it('does not include blocked items in the prompt sent to the LLM', () => {
+      const session = {
+        id: 'test-session',
+        cwd: '/tmp',
+        messages: [],
+      }
+      const agent = {
+        id: 'test-agent',
+        name: 'Test',
+        description: 'Test agent',
+      }
+
+      const heartbeatFileContent = [
+        '# Heartbeat Tasks',
+        '## Active',
+        '- [ ] Pull SWGOH roster data (blocked, no update)',
+        '- [ ] Check weather forecast',
+        '## Completed',
+        '- [x] Laundry done',
+      ].join('\n')
+
+      const prompt = buildAgentHeartbeatPrompt(session, agent, 'default prompt', heartbeatFileContent)
+
+      assert.ok(!prompt.includes('SWGOH'), 'blocked SWGOH task should not appear in prompt')
+      assert.ok(prompt.includes('Check weather forecast'), 'non-blocked task should appear')
+      assert.ok(prompt.includes('Laundry done'), 'completed task should appear')
+    })
+
+    it('produces no HEARTBEAT.md section when all active items are blocked', () => {
+      const session = {
+        id: 'test-session',
+        cwd: '/tmp',
+        messages: [],
+      }
+      const agent = {
+        id: 'test-agent',
+        name: 'Test',
+        description: 'Test agent',
+      }
+
+      const heartbeatFileContent = [
+        '# Heartbeat Tasks',
+        '## Active',
+        '- [ ] Task A (blocked, no update)',
+        '- [ ] Task B (blocked: awaiting user)',
+      ].join('\n')
+
+      const prompt = buildAgentHeartbeatPrompt(session, agent, 'default prompt', heartbeatFileContent)
+
+      assert.ok(!prompt.includes('HEARTBEAT.md contents:'), 'should not include HEARTBEAT.md section when all items are blocked')
+    })
+  })
+})

package/src/lib/server/heartbeat-service.ts

@@ -194,6 +194,31 @@ export function buildIdentityContext(session: Record<string, unknown> | undefine
   return `## Your Identity\n${lines.join('\n')}`
 }
 
+// ── Blocked-item suppression ────────────────────────────────────────────
+// Ported from OpenClaw's duplicate-suppression pattern: instead of letting
+// the LLM see blocked tasks every tick (and parrot "still blocked"), we
+// strip those lines before they ever reach the prompt.  A line is
+// considered blocked if it contains "(blocked" anywhere (case-insensitive),
+// which covers "(blocked, no update)", "(blocked: awaiting …)", etc.
+const BLOCKED_MARKER_RE = /\(blocked\b/i
+
+/**
+ * Remove blocked checklist items from HEARTBEAT.md content so the LLM
+ * doesn't keep surfacing them.  Headers and non-list lines pass through
+ * unchanged.
+ */
+export function stripBlockedItems(content: string): string {
+  if (!content) return ''
+  const lines = content.split('\n')
+  const filtered = lines.filter((line) => {
+    const trimmed = line.trim()
+    // Only filter checklist / list items that are explicitly marked blocked
+    if (/^[-*+]\s/.test(trimmed) && BLOCKED_MARKER_RE.test(trimmed)) return false
+    return true
+  })
+  return filtered.join('\n')
+}
+
 /** Detect HEARTBEAT.md files that contain only skeleton structure (headers, empty list items) but no real content. */
 export function isHeartbeatContentEffectivelyEmpty(content: string | undefined | null): boolean {
   if (!content || typeof content !== 'string') return true
@@ -236,8 +261,9 @@ export function buildAgentHeartbeatPrompt(session: any, agent: any, fallbackProm
     })
     .join('\n')
 
-  // Don't inject effectively-empty HEARTBEAT.md content
-  const effectiveFileContent = isHeartbeatContentEffectivelyEmpty(heartbeatFileContent) ? '' : heartbeatFileContent
+  // Strip blocked items, then check if anything meaningful remains
+  const strippedContent = stripBlockedItems(heartbeatFileContent)
+  const effectiveFileContent = isHeartbeatContentEffectivelyEmpty(strippedContent) ? '' : strippedContent
 
   return [
     'AGENT_HEARTBEAT_TICK',