From b5f2785602987c1710c85d5050dd2d29678ce0f1 Mon Sep 17 00:00:00 2001
From: musnows <ezplayingd@126.com>
Date: Tue, 23 Jun 2026 23:42:24 +0800
Subject: [PATCH] fix(settings): show full workspace path

---
 .../settings-section-general.test.ts          | 118 ++++++++++++++++++
 .../components/settings-section-general.tsx   |   8 +-
 2 files changed, 123 insertions(+), 3 deletions(-)
 create mode 100644 src/renderer/src/components/settings-section-general.test.ts

diff --git a/src/renderer/src/components/settings-section-general.test.ts b/src/renderer/src/components/settings-section-general.test.ts
new file mode 100644
index 00000000..db36c41f
--- /dev/null
+++ b/src/renderer/src/components/settings-section-general.test.ts
@@ -0,0 +1,118 @@
+import { createElement } from 'react'
+import { renderToStaticMarkup } from 'react-dom/server'
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
+import { GeneralSettingsSection } from './settings-section-general'
+
+const labels: Record<string, string> = {
+  sectionGeneral: 'General',
+  workspaceRoot: 'Default workspace',
+  workspaceRootDesc: 'Default workspace description',
+  workspaceRootPlaceholder: '~/.kun/default_workspace',
+  restoreWorkspaceDefault: 'Restore default',
+  browse: 'Browse'
+}
+
+function t(key: string, values?: Record<string, unknown>): string {
+  if (!values) return labels[key] ?? key
+  return labels[key] ?? key
+}
+
+function baseCtx(): Record<string, unknown> {
+  const noop = () => undefined
+  return {
+    t,
+    tCommon: t,
+    form: {
+      locale: 'zh',
+      theme: 'dark',
+      uiFontScale: 'medium',
+      workspaceRoot: '~/data/code/python/Kook-Voices',
+      cursorSpotlight: true,
+      cursorSpotlightColor: '#3b82f6',
+      appBehavior: {
+        openAtLogin: false,
+        startMinimized: false,
+        closeToTray: false,
+        closeAction: 'ask'
+      },
+      notifications: {
+        turnComplete: false
+      },
+      log: {
+        enabled: false,
+        retentionDays: 3
+      }
+    },
+    kun: {},
+    update: noop,
+    updateKun: noop,
+    showRuntimeToken: false,
+    setShowRuntimeToken: noop,
+    portError: '',
+    selectControlClass: 'select',
+    openOnboardingPreview: noop,
+    pickWorkspace: async () => undefined,
+    resetWorkspaceToDefault: noop,
+    workspacePickerError: '',
+    logPath: '',
+    logDirOpenError: '',
+    setLogDirOpenError: noop,
+    compactHomePath: (path: string) => path,
+    expandHomePath: (path: string) => path,
+    pickWriteWorkspace: async () => undefined,
+    resetWriteWorkspaceToDefault: noop,
+    writeWorkspacePickerError: '',
+    writeInlineBaseUrlInherited: false,
+    effectiveWriteInlineBaseUrl: '',
+    writeInlineModelInherited: false,
+    effectiveWriteInlineModel: '',
+    setWriteDebugModalOpen: noop,
+    loadWriteDebugEntries: async () => undefined,
+    scrollToAgentSection: noop,
+    agentsSectionRef: { current: null },
+    skillSectionRef: { current: null },
+    mcpSectionRef: { current: null },
+    permissionsSectionRef: { current: null },
+    selectedSkillRoot: null,
+    skillRootOptions: [],
+    skillRootId: '',
+    setSkillRootId: noop,
+    skillNotice: null,
+    openSkillRoot: async () => undefined,
+    openPlugins: noop,
+    mcpConfigPath: '',
+    mcpConfigExists: false,
+    mcpConfigText: '',
+    setMcpConfigText: noop,
+    mcpLoading: false,
+    mcpBusy: false,
+    mcpNotice: null,
+    saveMcpConfig: async () => undefined,
+    loadMcpConfig: async () => undefined,
+    openMcpConfigDir: async () => undefined,
+    pickClawWorkspace: async () => undefined,
+    resetClawWorkspaceToDefault: noop,
+    clawWorkspacePickerError: '',
+    splitSettingsList: (value: string) => value.split('\n').filter(Boolean),
+    listSettingsText: (values: string[]) => values.join('\n')
+  }
+}
+
+describe('GeneralSettingsSection workspace layout', () => {
+  beforeEach(() => {
+    vi.stubGlobal('window', { kunGui: {} })
+  })
+
+  afterEach(() => {
+    vi.unstubAllGlobals()
+  })
+
+  it('keeps the workspace path input full width above the action buttons', () => {
+    const html = renderToStaticMarkup(createElement(GeneralSettingsSection, { ctx: baseCtx() }))
+
+    expect(html).toContain('grid w-full min-w-0 gap-2 md:max-w-xl')
+    expect(html).toContain('w-full min-w-0 rounded-xl border border-ds-border')
+    expect(html).toContain('flex flex-wrap justify-end gap-2')
+    expect(html.indexOf('~/data/code/python/Kook-Voices')).toBeLessThan(html.indexOf('Restore default'))
+  })
+})
diff --git a/src/renderer/src/components/settings-section-general.tsx b/src/renderer/src/components/settings-section-general.tsx
index 759f5bbd..033c356b 100644
--- a/src/renderer/src/components/settings-section-general.tsx
+++ b/src/renderer/src/components/settings-section-general.tsx
@@ -309,14 +309,16 @@ export function GeneralSettingsSection({ ctx }: { ctx: Record<string, any> }): R
                   title={t('workspaceRoot')}
                   description={t('workspaceRootDesc')}
                   control={
-                    <div className="w-full min-w-[200px] md:max-w-xl">
-                      <div className="flex items-center gap-2">
+                    <div className="grid w-full min-w-0 gap-2 md:max-w-xl">
+                      <div className="min-w-0">
                         <input
-                          className="w-full rounded-xl border border-ds-border bg-ds-card px-3 py-2 text-[14px] text-ds-ink shadow-sm focus:border-accent/40 focus:outline-none focus:ring-1 focus:ring-accent/30"
+                          className="w-full min-w-0 rounded-xl border border-ds-border bg-ds-card px-3 py-2 text-[14px] text-ds-ink shadow-sm focus:border-accent/40 focus:outline-none focus:ring-1 focus:ring-accent/30"
                           value={compactHomePath(form.workspaceRoot)}
                           onChange={(e) => update({ workspaceRoot: expandHomePath(e.target.value) })}
                           placeholder={t('workspaceRootPlaceholder')}
                         />
+                      </div>
+                      <div className="flex flex-wrap justify-end gap-2">
                         <button
                           type="button"
                           onClick={resetWorkspaceToDefault}