Fix panel sidebars and reasoning streams for v1.5.70

Author: waydelyle

README.md

@@ -399,6 +399,17 @@ Operational docs: https://swarmclaw.ai/docs/observability
 
 ## Releases
 
+### v1.5.70 Highlights
+
+Fast-follow release for [#56](https://github.com/swarmclawai/swarmclaw/pull/56) by [@latentwill](https://github.com/latentwill). Thanks latentwill!
+
+Also includes fixes for [#57](https://github.com/swarmclawai/swarmclaw/issues/57) and [#58](https://github.com/swarmclawai/swarmclaw/issues/58) reported by [@zantak](https://github.com/zantak). Thanks zantak!
+
+- **Builtin provider saves work again.** Saving a builtin provider no longer sends the strict-schema rejected `type` field, and the provider update route is now covered by the runtime test script.
+- **Knowledge sources appear on direct visits.** Panel-backed routes such as Knowledge now auto-open their source/sidebar panel on desktop route changes, while mobile keeps the drawer closed by default.
+- **Reasoning content stays out of the reply body.** OpenAI-compatible `reasoning_content` and `reasoning` stream deltas now flow into the existing collapsed Thinking panel instead of being appended before the visible answer.
+- **macOS install guidance remains explicit.** Ad-hoc signed macOS desktop builds still document the quarantine workaround until Developer ID signing and notarization are available. Thanks [@yagudaev](https://github.com/yagudaev) for confirming the current workaround on Apple Silicon.
+
 ### v1.5.69 Highlights
 
 Fast-follow release for [#55](https://github.com/swarmclawai/swarmclaw/pull/55) by [@borislavnnikolov](https://github.com/borislavnnikolov). Thanks Borislav!

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@swarmclawai/swarmclaw",
-  "version": "1.5.69",
+  "version": "1.5.70",
   "description": "Build and run autonomous AI agents with OpenClaw, Hermes, multiple model providers, orchestration, delegation, memory, skills, schedules, and chat connectors.",
   "main": "electron-dist/main.js",
   "license": "MIT",
@@ -86,8 +86,8 @@
     "cli": "node ./bin/swarmclaw.js",
     "test:cli": "node --test src/cli/*.test.js bin/*.test.js scripts/postinstall.test.mjs scripts/run-next-build.test.mjs scripts/run-next-typegen.test.mjs",
     "test:setup": "tsx --test src/app/api/setup/check-provider/route.test.ts src/lib/server/provider-model-discovery.test.ts src/components/auth/setup-wizard/utils.test.ts src/components/auth/setup-wizard/types.test.ts src/hooks/setup-done-detection.test.ts src/lib/setup-defaults.test.ts src/lib/server/storage-auth.test.ts src/lib/server/storage-auth-docker.test.ts",
-    "test:openclaw": "tsx --test src/lib/openclaw/openclaw-agent-id.test.ts src/lib/openclaw/openclaw-endpoint.test.ts src/lib/server/agents/agent-runtime-config.test.ts src/lib/server/build-llm.test.ts src/lib/server/connectors/connector-routing.test.ts src/lib/server/connectors/openclaw.test.ts src/lib/server/connectors/swarmdock.test.ts src/lib/server/gateway/protocol.test.ts src/lib/server/llm-response-cache.test.ts src/lib/server/mcp-conformance.test.ts src/lib/server/openclaw/agent-resolver.test.ts src/lib/server/openclaw/deploy.test.ts src/lib/server/openclaw/skills-normalize.test.ts src/lib/server/session-tools/openclaw-nodes.test.ts src/lib/server/session-tools/swarmdock.test.ts src/lib/server/tasks/task-quality-gate.test.ts src/lib/server/tasks/task-validation.test.ts src/lib/server/tool-capability-policy.test.ts src/lib/providers/openclaw-exports.test.ts src/app/api/openclaw/dashboard-url/route.test.ts",
-    "test:runtime": "tsx --test src/lib/server/mcp-gateway-runtime.test.ts src/lib/server/mcp-connection-pool.test.ts src/lib/server/knowledge-sources.test.ts src/lib/server/chat-execution/chat-execution-grounding.test.ts src/lib/server/chat-execution/chat-turn-preparation.test.ts src/lib/server/chat-execution/iteration-timers.test.ts src/lib/server/chat-execution/post-stream-finalization.test.ts src/lib/server/chats/clear-undo-snapshots.test.ts src/lib/server/connectors/email.test.ts src/lib/server/protocols/protocol-service.test.ts src/lib/server/runtime/run-ledger.test.ts src/lib/server/observability/otel-config.test.ts src/lib/server/safe-parse-body.test.ts src/app/api/approvals/route.test.ts src/app/api/agents/agents-route.test.ts src/app/api/tasks/tasks-route.test.ts src/app/api/chats/chat-route.test.ts src/app/api/chats/clear-route.test.ts src/app/api/chats/compact-route.test.ts src/app/api/chats/context-status-route.test.ts src/app/api/connectors/connector-doctor-route.test.ts src/app/api/healthz/route.test.ts src/app/api/logs/route.test.ts src/app/api/providers/[id]/route.test.ts src/app/api/tts/route.test.ts",
+    "test:openclaw": "tsx --test src/lib/openclaw/openclaw-agent-id.test.ts src/lib/openclaw/openclaw-endpoint.test.ts src/lib/server/agents/agent-runtime-config.test.ts src/lib/server/build-llm.test.ts src/lib/server/connectors/connector-routing.test.ts src/lib/server/connectors/openclaw.test.ts src/lib/server/connectors/swarmdock.test.ts src/lib/server/gateway/protocol.test.ts src/lib/server/llm-response-cache.test.ts src/lib/server/mcp-conformance.test.ts src/lib/server/openclaw/agent-resolver.test.ts src/lib/server/openclaw/deploy.test.ts src/lib/server/openclaw/skills-normalize.test.ts src/lib/server/session-tools/openclaw-nodes.test.ts src/lib/server/session-tools/swarmdock.test.ts src/lib/server/tasks/task-quality-gate.test.ts src/lib/server/tasks/task-validation.test.ts src/lib/server/tool-capability-policy.test.ts src/lib/providers/openai.test.ts src/lib/providers/openclaw-exports.test.ts src/app/api/openclaw/dashboard-url/route.test.ts",
+    "test:runtime": "tsx --test src/lib/server/mcp-gateway-runtime.test.ts src/lib/server/mcp-connection-pool.test.ts src/lib/server/knowledge-sources.test.ts src/lib/server/chat-execution/chat-execution-grounding.test.ts src/lib/server/chat-execution/chat-turn-preparation.test.ts src/lib/server/chat-execution/iteration-timers.test.ts src/lib/server/chat-execution/post-stream-finalization.test.ts src/lib/server/chats/clear-undo-snapshots.test.ts src/lib/server/connectors/email.test.ts src/lib/server/protocols/protocol-service.test.ts src/lib/server/runtime/run-ledger.test.ts src/lib/server/observability/otel-config.test.ts src/lib/server/safe-parse-body.test.ts src/lib/app/view-constants.test.ts src/app/api/approvals/route.test.ts src/app/api/agents/agents-route.test.ts src/app/api/tasks/tasks-route.test.ts src/app/api/chats/chat-route.test.ts src/app/api/chats/clear-route.test.ts src/app/api/chats/compact-route.test.ts src/app/api/chats/context-status-route.test.ts src/app/api/connectors/connector-doctor-route.test.ts src/app/api/healthz/route.test.ts src/app/api/logs/route.test.ts src/app/api/providers/[id]/route.test.ts src/app/api/tts/route.test.ts",
     "test:builder": "tsx --test src/features/protocols/builder/utils/nodes-to-template.test.ts src/features/protocols/builder/utils/template-to-nodes.test.ts src/features/protocols/builder/validators/dag-validator.test.ts",
     "test:e2e": "tsx .workbench/browser-e2e/run.ts",
     "test:mcp:conformance": "node --import tsx ./scripts/mcp-conformance-check.ts",

package.json

@@ -1,6 +1,6 @@
 'use client'
 
-import { useEffect, useState, useCallback } from 'react'
+import { useEffect, useRef, useState, useCallback } from 'react'
 import { useRouter, usePathname } from 'next/navigation'
 import { initAudioContext } from '@/lib/tts'
 import { clearStoredAccessKey } from '@/lib/app/api-client'
@@ -13,6 +13,7 @@ import { useSwipe } from '@/hooks/use-swipe'
 import { useWs } from '@/hooks/use-ws'
 import { api } from '@/lib/app/api-client'
 import { pathToView, useNavigate } from '@/lib/app/navigation'
+import { shouldAutoOpenPanelSidebar } from '@/lib/app/view-constants'
 
 import { FullScreenLoader } from '@/components/ui/full-screen-loader'
 import { SidebarRail } from '@/components/layout/sidebar-rail'
@@ -45,6 +46,7 @@ export function DashboardShell({ children }: { children: React.ReactNode }) {
 
   const [bootTimedOut, setBootTimedOut] = useState(false)
   const [profileSheetOpen, setProfileSheetOpen] = useState(false)
+  const lastAutoOpenedPanelPathRef = useRef<string | null>(null)
   const sidebarOpen = useAppStore((s) => s.sidebarOpen)
   const setSidebarOpen = useAppStore((s) => s.setSidebarOpen)
   const appSettings = useAppStore((s) => s.appSettings)
@@ -165,6 +167,18 @@ export function DashboardShell({ children }: { children: React.ReactNode }) {
     }
   }, [pathname, isViewEnabled, router, isAuthPage])
 
+  useEffect(() => {
+    if (isAuthPage) return
+    const currentView = pathToView(pathname)
+    if (!shouldAutoOpenPanelSidebar(currentView, isDesktop)) {
+      lastAutoOpenedPanelPathRef.current = null
+      return
+    }
+    if (lastAutoOpenedPanelPathRef.current === pathname) return
+    lastAutoOpenedPanelPathRef.current = pathname
+    setSidebarOpen(true)
+  }, [isAuthPage, isDesktop, pathname, setSidebarOpen])
+
   // Extension sidebar items
   const refreshExtensionState = useCallback(() => {
     void loadExtensions()

src/components/layout/dashboard-shell.tsx

@@ -9,7 +9,7 @@ import { DaemonIndicator } from '@/components/layout/daemon-indicator'
 import { NotificationCenter } from '@/components/shared/notification-center'
 import { NavItem, RailTooltip } from '@/components/layout/nav-item'
 import { useWs } from '@/hooks/use-ws'
-import { FULL_WIDTH_VIEWS } from '@/lib/app/view-constants'
+import { FULL_WIDTH_VIEWS, isPanelSidebarView } from '@/lib/app/view-constants'
 import { pathToView, useNavigate } from '@/lib/app/navigation'
 import { safeStorageGet, safeStorageSet } from '@/lib/app/safe-storage'
 import type { AppView } from '@/types'
@@ -80,9 +80,9 @@ export function SidebarRail({
       setSidebarOpen(false)
       return
     }
-    if (FULL_WIDTH_VIEWS.has(view)) {
-      setSidebarOpen(false)
-    } else if (activeView === view && sidebarOpen) {
+    if (isPanelSidebarView(view)) {
+      setSidebarOpen(!(activeView === view && sidebarOpen))
+    } else if (FULL_WIDTH_VIEWS.has(view)) {
       setSidebarOpen(false)
     } else {
       setSidebarOpen(true)

src/components/layout/sidebar-rail.tsx

@@ -0,0 +1,21 @@
+import assert from 'node:assert/strict'
+import { describe, it } from 'node:test'
+
+import { isPanelSidebarView, shouldAutoOpenPanelSidebar } from './view-constants'
+
+describe('panel sidebar route helpers', () => {
+  it('treats knowledge as a panel-backed view', () => {
+    assert.equal(isPanelSidebarView('knowledge'), true)
+  })
+
+  it('auto-opens panel-backed views only on desktop', () => {
+    assert.equal(shouldAutoOpenPanelSidebar('knowledge', true), true)
+    assert.equal(shouldAutoOpenPanelSidebar('knowledge', false), false)
+  })
+
+  it('does not auto-open full-width views without panel layouts', () => {
+    assert.equal(shouldAutoOpenPanelSidebar('home', true), false)
+    assert.equal(shouldAutoOpenPanelSidebar('settings', true), false)
+    assert.equal(shouldAutoOpenPanelSidebar(null, true), false)
+  })
+})

src/lib/app/view-constants.test.ts

@@ -245,3 +245,28 @@ export const FULL_WIDTH_VIEWS = new Set<AppView>([
   'connectors', 'webhooks', 'mcp_servers', 'knowledge', 'extensions',
   'usage', 'runs', 'autonomy', 'logs', 'settings', 'activity', 'projects', 'swarmfeed', 'marketplace', 'missions',
 ])
+
+export const PANEL_SIDEBAR_VIEWS = new Set<AppView>([
+  'agents',
+  'connectors',
+  'extensions',
+  'knowledge',
+  'logs',
+  'mcp_servers',
+  'memory',
+  'providers',
+  'runs',
+  'schedules',
+  'secrets',
+  'skills',
+  'tasks',
+  'webhooks',
+])
+
+export function isPanelSidebarView(view: AppView | null | undefined): boolean {
+  return Boolean(view && PANEL_SIDEBAR_VIEWS.has(view))
+}
+
+export function shouldAutoOpenPanelSidebar(view: AppView | null | undefined, isDesktop: boolean): boolean {
+  return isDesktop && isPanelSidebarView(view)
+}

src/lib/app/view-constants.ts

@@ -0,0 +1,54 @@
+import assert from 'node:assert/strict'
+import test from 'node:test'
+
+import { streamOpenAiChat } from './openai'
+
+function sseChunk(data: unknown) {
+  return `data: ${JSON.stringify(data)}\n\n`
+}
+
+function parseSseEvents(frames: string[]) {
+  return frames
+    .flatMap((frame) => frame.trim().split('\n\n'))
+    .filter(Boolean)
+    .map((frame) => JSON.parse(frame.replace(/^data: /, '')) as { t: string; text?: string })
+}
+
+test('OpenAI-compatible reasoning deltas stream as thinking instead of visible text', async () => {
+  const originalFetch = globalThis.fetch
+  const encoded = new TextEncoder()
+  const frames = [
+    sseChunk({ choices: [{ delta: { reasoning_content: 'internal reasoning ' } }] }),
+    sseChunk({ choices: [{ delta: { content: 'visible answer' } }] }),
+    'data: [DONE]\n\n',
+  ]
+  const writes: string[] = []
+
+  globalThis.fetch = async () => new Response(new ReadableStream({
+    start(controller) {
+      for (const frame of frames) controller.enqueue(encoded.encode(frame))
+      controller.close()
+    },
+  }), {
+    status: 200,
+    headers: { 'content-type': 'text/event-stream' },
+  })
+
+  try {
+    const result = await streamOpenAiChat({
+      session: { id: 'session-1', provider: 'openai', model: 'test-model' },
+      message: 'hello',
+      write: (data) => writes.push(data),
+      active: new Map(),
+      loadHistory: () => [],
+    } as Parameters<typeof streamOpenAiChat>[0])
+
+    assert.equal(result, 'visible answer')
+    assert.deepEqual(parseSseEvents(writes), [
+      { t: 'thinking', text: 'internal reasoning ' },
+      { t: 'd', text: 'visible answer' },
+    ])
+  } finally {
+    globalThis.fetch = originalFetch
+  }
+})

src/lib/providers/openai.test.ts

@@ -166,13 +166,17 @@ export function streamOpenAiChat({ session, message, imagePath, imageUrl, apiKey
               try {
                 const parsed = JSON.parse(data)
                 const choice = parsed.choices?.[0]?.delta
-                const delta = choice?.content
-                  // Thinking/reasoning models (kimi-k2, etc.) put output in reasoning fields
-                  || choice?.reasoning_content
-                  || choice?.reasoning
-                if (delta) {
-                  fullResponse += delta
-                  writeSSE(write, 'd', delta)
+                const contentDelta = typeof choice?.content === 'string' ? choice.content : ''
+                const reasoningDelta =
+                  typeof choice?.reasoning_content === 'string' ? choice.reasoning_content
+                    : typeof choice?.reasoning === 'string' ? choice.reasoning
+                      : ''
+                if (reasoningDelta) {
+                  writeSSE(write, 'thinking', reasoningDelta)
+                }
+                if (contentDelta) {
+                  fullResponse += contentDelta
+                  writeSSE(write, 'd', contentDelta)
                 }
                 // Extract usage from the final chunk (stream_options: include_usage)
                 if (usageEnabled && parsed.usage && onUsage) {