clawport-ui 0.8.5 → 0.8.7

package/components/OnboardingWizard.tsx

@@ -164,15 +164,24 @@ export function OnboardingWizard({ forceOpen, onClose }: OnboardingWizardProps)
 
     // Check crons (validates gateway + openclaw binary)
     fetch('/api/crons')
-      .then(r => {
-        if (!r.ok) throw new Error(`HTTP ${r.status}`)
+      .then(async r => {
+        if (!r.ok) {
+          const body = await r.json().catch(() => null)
+          const serverMsg = body?.error
+          throw new Error(serverMsg || `HTTP ${r.status}`)
+        }
         return r.json()
       })
       .then(() => {
         setCronsStatus('ok')
       })
-      .catch(() => {
-        setCronsError('Could not reach OpenClaw gateway. Run: openclaw gateway run')
+      .catch((err: Error) => {
+        const msg = err.message || ''
+        if (msg.includes('Failed to fetch cron') || msg.includes('JSON') || msg.includes('Unexpected token')) {
+          setCronsError('Cron list failed -- OpenClaw CLI may be printing log output before JSON. Try: OPENCLAW_LOG_LEVEL=error clawport dev')
+        } else {
+          setCronsError('Could not reach OpenClaw gateway. Run: openclaw gateway run')
+        }
         setCronsStatus('error')
       })
   }

package/lib/cli-utils.test.ts

@@ -41,4 +41,20 @@ hooks: Unrecognized key: "allowedAgentIds"
   it('throws on empty string', () => {
     expect(() => extractJson('')).toThrow()
   })
+
+  it('skips bracketed log lines like [plugins] before real JSON', () => {
+    const raw = `[plugins] [debug] Database schema initialized
+[plugins] [debug] Loading context engine
+{"jobs":[{"name":"daily-report","schedule":"0 8 * * *"}]}`
+    const result = extractJson(raw) as Record<string, unknown>
+    expect(result.jobs).toEqual([{ name: 'daily-report', schedule: '0 8 * * *' }])
+  })
+
+  it('skips bracketed log lines before JSON array', () => {
+    const raw = `[plugins] [debug] Database schema initialized
+[plugins] [info] Ready
+[{"id":"pulse","name":"daily-pulse"}]`
+    const result = extractJson(raw)
+    expect(result).toEqual([{ id: 'pulse', name: 'daily-pulse' }])
+  })
 })

package/lib/cli-utils.ts

@@ -1,25 +1,38 @@
 /**
  * Extract a JSON value from CLI output that may contain non-JSON preamble.
  *
- * Some OpenClaw versions print validation warnings (e.g. "Unrecognized key")
- * to stdout before the JSON payload. This function finds the first `[` or `{`
- * and parses from there, so ClawPort doesn't break on noisy CLI output.
+ * Some OpenClaw versions print validation warnings or debug log lines
+ * (e.g. "[plugins] [debug] ...") to stdout before the JSON payload.
+ * This function finds the actual JSON structure by trying each `[` or `{`
+ * position until one parses successfully.
  */
 export function extractJson(raw: string): unknown {
   // Fast path: raw is already valid JSON
   const trimmed = raw.trim()
   if (trimmed.startsWith('[') || trimmed.startsWith('{')) {
-    return JSON.parse(trimmed)
+    try {
+      return JSON.parse(trimmed)
+    } catch {
+      // May start with [ but be a log line like "[plugins] ..." -- fall through
+    }
   }
 
-  // Find the first JSON structure in the output
-  const arrStart = raw.indexOf('[')
-  const objStart = raw.indexOf('{')
-  const starts = [arrStart, objStart].filter(i => i >= 0)
-  if (starts.length === 0) {
-    throw new SyntaxError('No JSON found in CLI output')
+  // Try each potential JSON start position
+  let pos = 0
+  while (pos < raw.length) {
+    const arrStart = raw.indexOf('[', pos)
+    const objStart = raw.indexOf('{', pos)
+    const candidates = [arrStart, objStart].filter(i => i >= 0)
+    if (candidates.length === 0) break
+
+    const start = Math.min(...candidates)
+    try {
+      return JSON.parse(raw.slice(start))
+    } catch {
+      // This wasn't the real JSON start -- advance past it
+      pos = start + 1
+    }
   }
 
-  const start = Math.min(...starts)
-  return JSON.parse(raw.slice(start))
+  throw new SyntaxError('No JSON found in CLI output')
 }

package/lib/crons.test.ts

@@ -254,18 +254,25 @@ describe('getCrons - error and lastError', () => {
   })
 })
 
-// --- Error propagation (current implementation throws) ---
+// --- Graceful degradation (returns empty array on failure) ---
 
-describe('getCrons - error propagation', () => {
-  it('throws when execSync throws (CLI not installed)', async () => {
+describe('getCrons - graceful degradation', () => {
+  it('returns empty array when execSync throws (CLI not installed)', async () => {
     mockExecSync.mockImplementation(() => { throw new Error('ENOENT') })
-    await expect(getCrons()).rejects.toThrow('Failed to fetch cron jobs')
-    await expect(getCrons()).rejects.toThrow('ENOENT')
+    const result = await getCrons()
+    expect(result).toEqual([])
   })
 
-  it('throws for invalid JSON output', async () => {
+  it('returns empty array for invalid JSON output', async () => {
     mockExecSync.mockReturnValue('not valid json {{')
-    await expect(getCrons()).rejects.toThrow('Failed to fetch cron jobs')
+    const result = await getCrons()
+    expect(result).toEqual([])
+  })
+
+  it('returns empty array when OPENCLAW_BIN is not set', async () => {
+    vi.unstubAllEnvs()
+    const result = await getCrons()
+    expect(result).toEqual([])
   })
 })
 

package/lib/crons.ts

@@ -32,11 +32,22 @@ export async function getCrons(): Promise<CronJob[]> {
   }
 
   try {
-    const openclawBin = requireEnv('OPENCLAW_BIN')
-    const raw = execSync(`${openclawBin} cron list --json`, {
-      encoding: 'utf-8',
-      timeout: 10000,
-    })
+    const openclawBin = process.env.OPENCLAW_BIN
+    if (!openclawBin) {
+      // No binary configured -- return empty list instead of crashing
+      return []
+    }
+
+    let raw: string
+    try {
+      raw = execSync(`${openclawBin} cron list --json`, {
+        encoding: 'utf-8',
+        timeout: 10000,
+      })
+    } catch {
+      // CLI failed (binary not found, no crons, gateway down) -- return empty
+      return []
+    }
 
     const parsed = extractJson(raw) as Record<string, unknown>
     const jobs: unknown[] = Array.isArray(parsed)
@@ -113,9 +124,8 @@ export async function getCrons(): Promise<CronJob[]> {
 
     _cronsCache = { result, ts: Date.now() }
     return result
-  } catch (err) {
-    throw new Error(
-      `Failed to fetch cron jobs: ${err instanceof Error ? err.message : String(err)}`
-    )
+  } catch {
+    // JSON extraction or parsing failed -- return empty rather than 500
+    return []
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clawport-ui",
-  "version": "0.8.5",
+  "version": "0.8.7",
   "description": "Open-source dashboard for managing, monitoring, and chatting with your OpenClaw AI agents.",
   "homepage": "https://clawport.dev",
   "repository": {