@2en/clawly-plugins 1.16.2 → 1.16.3

package/gateway/offline-push.test.ts

@@ -1,20 +1,42 @@
 import {afterEach, beforeEach, describe, expect, mock, test} from 'bun:test'
 import type {PluginApi} from '../index'
-import {PUSH_COOLDOWN_S, _resetCooldowns, registerOfflinePush} from './offline-push'
+import {
+  PUSH_COOLDOWN_S,
+  _resetCooldowns,
+  getLastAssistantPreview,
+  registerOfflinePush,
+} from './offline-push'
 
 // ── Mocks ────────────────────────────────────────────────────────
 
 let mockOnline = false
 let mockPushSent = true
+let lastPushOpts: {body: string; title?: string; data?: Record<string, unknown>} | null = null
+let mockIdentity: {name?: string; emoji?: string} | null = null
 
 mock.module('./presence', () => ({
   isClientOnline: async () => mockOnline,
 }))
 
 mock.module('./notification', () => ({
-  sendPushNotification: async () => mockPushSent,
+  sendPushNotification: async (opts: {
+    body: string
+    title?: string
+    data?: Record<string, unknown>
+  }) => {
+    lastPushOpts = opts
+    return mockPushSent
+  },
 }))
 
+mock.module('./offline-push', () => {
+  const original = require('./offline-push')
+  return {
+    ...original,
+    getAgentIdentity: async () => mockIdentity,
+  }
+})
+
 // ── Helpers ──────────────────────────────────────────────────────
 
 function createMockApi(): {
@@ -44,6 +66,8 @@ function createMockApi(): {
 beforeEach(() => {
   mockOnline = false
   mockPushSent = true
+  mockIdentity = null
+  lastPushOpts = null
   _resetCooldowns()
 })
 
@@ -146,4 +170,131 @@ describe('offline-push', () => {
   test('exports PUSH_COOLDOWN_S as 30', () => {
     expect(PUSH_COOLDOWN_S).toBe(30)
   })
+
+  // ── Rich title/body tests ────────────────────────────────────
+
+  test('title shows agent emoji + name when identity is available', async () => {
+    mockIdentity = {name: 'Luna', emoji: '🐱'}
+    const {api, handlers} = createMockApi()
+    registerOfflinePush(api)
+
+    await handlers.get('agent_end')!({sessionKey: 'sess-1'})
+
+    expect(lastPushOpts?.title).toBe('🐱 Luna')
+  })
+
+  test('title falls back to 🦞 Clawly when identity fetch fails', async () => {
+    mockIdentity = null
+    const {api, handlers} = createMockApi()
+    registerOfflinePush(api)
+
+    await handlers.get('agent_end')!({sessionKey: 'sess-1'})
+
+    expect(lastPushOpts?.title).toBe('🦞 Clawly')
+  })
+
+  test('title uses default emoji when identity has name but no emoji', async () => {
+    mockIdentity = {name: 'Atlas'}
+    const {api, handlers} = createMockApi()
+    registerOfflinePush(api)
+
+    await handlers.get('agent_end')!({sessionKey: 'sess-1'})
+
+    expect(lastPushOpts?.title).toBe('🦞 Atlas')
+  })
+
+  test('body shows assistant message preview when messages are present', async () => {
+    const {api, handlers} = createMockApi()
+    registerOfflinePush(api)
+
+    await handlers.get('agent_end')!({
+      sessionKey: 'sess-1',
+      messages: [
+        {role: 'user', content: 'Hello'},
+        {role: 'assistant', content: 'Hi there! How can I help you today?'},
+      ],
+    })
+
+    expect(lastPushOpts?.body).toBe('Hi there! How can I help you today?')
+  })
+
+  test('body falls back when no messages', async () => {
+    const {api, handlers} = createMockApi()
+    registerOfflinePush(api)
+
+    await handlers.get('agent_end')!({sessionKey: 'sess-1'})
+
+    expect(lastPushOpts?.body).toBe('Your response is ready')
+  })
+
+  test('body falls back when messages has no assistant role', async () => {
+    const {api, handlers} = createMockApi()
+    registerOfflinePush(api)
+
+    await handlers.get('agent_end')!({
+      sessionKey: 'sess-1',
+      messages: [{role: 'user', content: 'Hello'}],
+    })
+
+    expect(lastPushOpts?.body).toBe('Your response is ready')
+  })
+})
+
+// ── getLastAssistantPreview unit tests ──────────────────────────
+
+describe('getLastAssistantPreview', () => {
+  test('returns null for non-array input', () => {
+    expect(getLastAssistantPreview(undefined)).toBeNull()
+    expect(getLastAssistantPreview(null)).toBeNull()
+    expect(getLastAssistantPreview('string')).toBeNull()
+  })
+
+  test('extracts string content from last assistant message', () => {
+    const messages = [
+      {role: 'user', content: 'Hi'},
+      {role: 'assistant', content: 'First reply'},
+      {role: 'user', content: 'Another question'},
+      {role: 'assistant', content: 'Second reply'},
+    ]
+    expect(getLastAssistantPreview(messages)).toBe('Second reply')
+  })
+
+  test('handles array content with text parts', () => {
+    const messages = [
+      {
+        role: 'assistant',
+        content: [
+          {type: 'text', text: 'Hello '},
+          {type: 'text', text: 'world!'},
+        ],
+      },
+    ]
+    expect(getLastAssistantPreview(messages)).toBe('Hello world!')
+  })
+
+  test('collapses newlines into spaces', () => {
+    const messages = [{role: 'assistant', content: 'Line one\n\nLine two\nLine three'}]
+    expect(getLastAssistantPreview(messages)).toBe('Line one Line two Line three')
+  })
+
+  test('truncates long messages with ellipsis', () => {
+    const longText = 'a'.repeat(200)
+    const messages = [{role: 'assistant', content: longText}]
+    const result = getLastAssistantPreview(messages, 140)!
+    expect(result.length).toBe(141) // 140 chars + "…"
+    expect(result.endsWith('…')).toBe(true)
+  })
+
+  test('skips assistant messages with empty content', () => {
+    const messages = [
+      {role: 'assistant', content: 'Good reply'},
+      {role: 'assistant', content: ''},
+    ]
+    expect(getLastAssistantPreview(messages)).toBe('Good reply')
+  })
+
+  test('returns null when no assistant messages exist', () => {
+    const messages = [{role: 'user', content: 'Hello'}]
+    expect(getLastAssistantPreview(messages)).toBeNull()
+  })
 })

package/gateway/offline-push.ts

@@ -8,16 +8,80 @@
  * time for cron/external triggers) via a 30-second cooldown window.
  */
 
+import {$} from 'zx'
 import type {PluginApi} from '../index'
+import {stripCliLogs} from '../lib/stripCliLogs'
 import {sendPushNotification} from './notification'
 import {isClientOnline} from './presence'
 
+$.verbose = false
+
 /** Minimum seconds between consecutive push notifications per session. */
 export const PUSH_COOLDOWN_S = 30
 
 /** Per-session cooldown tracker. Key = sessionKey (or "__global__" for unknown). */
 const lastPushBySession = new Map<string, number>()
 
+/**
+ * Fetch agent display identity (name, emoji) via gateway RPC.
+ * Returns null on failure so the caller can fall back to defaults.
+ */
+export async function getAgentIdentity(
+  agentId: string | undefined,
+): Promise<{name?: string; emoji?: string} | null> {
+  if (!agentId) return null
+  try {
+    const rpcParams = JSON.stringify({agentId})
+    const result = await $`openclaw gateway call agent.identity.get --json --params ${rpcParams}`
+    const parsed = JSON.parse(stripCliLogs(result.stdout))
+    return {
+      name: typeof parsed.name === 'string' ? parsed.name : undefined,
+      emoji: typeof parsed.emoji === 'string' ? parsed.emoji : undefined,
+    }
+  } catch {
+    return null
+  }
+}
+
+/**
+ * Extract the last assistant message text and truncate to `maxLen` chars.
+ * Handles both string content and `{type:'text', text}[]` content formats.
+ */
+export function getLastAssistantPreview(messages: unknown, maxLen = 140): string | null {
+  if (!Array.isArray(messages)) return null
+
+  // Walk backwards to find the last assistant message
+  for (let i = messages.length - 1; i >= 0; i--) {
+    const msg = messages[i]
+    if (typeof msg !== 'object' || msg === null) continue
+    if ((msg as any).role !== 'assistant') continue
+
+    const content = (msg as any).content
+    let text: string | undefined
+
+    if (typeof content === 'string') {
+      text = content
+    } else if (Array.isArray(content)) {
+      // Concatenate all text parts
+      text = content
+        .filter((p: any) => typeof p === 'object' && p !== null && p.type === 'text')
+        .map((p: any) => p.text)
+        .join('')
+    }
+
+    if (!text) continue
+
+    // Collapse newlines → spaces, trim
+    const collapsed = text.replace(/\n+/g, ' ').trim()
+    if (!collapsed) continue
+
+    if (collapsed.length <= maxLen) return collapsed
+    return `${collapsed.slice(0, maxLen)}…`
+  }
+
+  return null
+}
+
 export function registerOfflinePush(api: PluginApi) {
   api.on('agent_end', async (event: Record<string, unknown>) => {
     try {
@@ -37,9 +101,16 @@ export function registerOfflinePush(api: PluginApi) {
         return
       }
 
+      const agentId = typeof event.agentId === 'string' ? event.agentId : undefined
+      const identity = await getAgentIdentity(agentId)
+      const title = `${identity?.emoji ?? '🦞'} ${identity?.name ?? 'Clawly'}`
+      const preview = getLastAssistantPreview(event.messages, 140)
+      const body = preview ?? 'Your response is ready'
+
       const sent = await sendPushNotification(
         {
-          body: 'Your response is ready',
+          title,
+          body,
           data: {
             type: 'agent_end',
             ...(sessionKey ? {sessionKey} : {}),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@2en/clawly-plugins",
-  "version": "1.16.2",
+  "version": "1.16.3",
   "module": "index.ts",
   "type": "module",
   "repository": {