diff --git a/packages/ui/src/apps/MobileFilesSurface.tsx b/packages/ui/src/apps/MobileFilesSurface.tsx index 5ec4e314..0bf72858 100644 --- a/packages/ui/src/apps/MobileFilesSurface.tsx +++ b/packages/ui/src/apps/MobileFilesSurface.tsx @@ -28,7 +28,7 @@ import { copyTextToClipboard } from '@/lib/clipboard'; import { useI18n } from '@/lib/i18n'; import { ensurePierreThemeRegistered } from '@/lib/shiki/appThemeRegistry'; import { getDefaultTheme } from '@/lib/theme/themes'; -import { getImageMimeType, getLanguageFromExtension, isImageFile } from '@/lib/toolHelpers'; +import { getImageMimeType, getLanguageFromExtension, isBinaryFile, isImageFile, isPdfFile, isSvgFile, looksLikeBinaryText } from '@/lib/toolHelpers'; import type { FileListEntry, FileSearchResult } from '@/lib/api/types'; import { runtimeFetch } from '@/lib/runtime-fetch'; import { cn } from '@/lib/utils'; @@ -98,6 +98,7 @@ export const MobileFilesSurface: React.FC = ({ onClose const [imageSrc, setImageSrc] = React.useState(''); const [fileError, setFileError] = React.useState(null); const [isLoadingFile, setIsLoadingFile] = React.useState(false); + const [binaryBlocked, setBinaryBlocked] = React.useState(false); const directoryLoadRequestIdRef = React.useRef(0); React.useEffect(() => { @@ -174,8 +175,9 @@ export const MobileFilesSurface: React.FC = ({ onClose setFileContent(''); setImageSrc(''); setFileError(null); + setBinaryBlocked(false); - if (isImageFile(route.path) && !route.path.toLowerCase().endsWith('.svg')) { + if (isImageFile(route.path) && !isSvgFile(route.path)) { let cancelled = false; let objectUrl = ''; setIsLoadingFile(true); @@ -203,6 +205,14 @@ export const MobileFilesSurface: React.FC = ({ onClose }; } + // Never load PDF/office/archives/etc. as UTF-8 text — that path can corrupt originals + // if a future write path is added, and it shows gibberish in the viewer. + if (isBinaryFile(route.path) || isPdfFile(route.path)) { + setBinaryBlocked(true); + setIsLoadingFile(false); + return; + } + if (!files.readFile) { setFileError(t('mobile.files.error.readUnavailable')); setIsLoadingFile(false); @@ -214,6 +224,11 @@ export const MobileFilesSurface: React.FC = ({ onClose void files.readFile(route.path) .then((result) => { if (cancelled) return; + if (looksLikeBinaryText(result.content)) { + setBinaryBlocked(true); + setFileContent(''); + return; + } setFileContent(result.content.length > MAX_MOBILE_FILE_CHARS ? `${result.content.slice(0, MAX_MOBILE_FILE_CHARS)}\n\n${t('mobile.files.file.truncated')}` : result.content); @@ -263,6 +278,7 @@ export const MobileFilesSurface: React.FC = ({ onClose imageSrc={imageSrc} error={fileError} isLoading={isLoadingFile} + binaryBlocked={binaryBlocked} onBack={() => setRoute({ type: 'browser', directory: route.returnDirectory })} onCopyPath={() => void handleCopyPath(route.path)} onCopyContent={() => void handleCopyContent()} @@ -413,10 +429,11 @@ const MobileFileDetail: React.FC<{ imageSrc: string; error: string | null; isLoading: boolean; + binaryBlocked: boolean; onBack: () => void; onCopyPath: () => void; onCopyContent: () => void; -}> = ({ path, content, imageSrc, error, isLoading, onBack, onCopyPath, onCopyContent }) => { +}> = ({ path, content, imageSrc, error, isLoading, binaryBlocked, onBack, onCopyPath, onCopyContent }) => { const { t } = useI18n(); return ( @@ -433,7 +450,7 @@ const MobileFileDetail: React.FC<{

{getNameFromPath(path)}

- {!isImageFile(path) ? ( + {!isImageFile(path) && !binaryBlocked ? ( @@ -455,6 +472,11 @@ const MobileFileDetail: React.FC<{ {getNameFromPath(path)} + ) : binaryBlocked ? ( +
+
{t('filesView.editor.cannotPreviewBinary')}
+
{t('filesView.editor.binaryFileDescription')}
+
) : ( )} diff --git a/packages/ui/src/components/sections/openchamber/OpenChamberPage.tsx b/packages/ui/src/components/sections/openchamber/OpenChamberPage.tsx index ebae3dc5..73f870ca 100644 --- a/packages/ui/src/components/sections/openchamber/OpenChamberPage.tsx +++ b/packages/ui/src/components/sections/openchamber/OpenChamberPage.tsx @@ -146,6 +146,7 @@ const GeneralSectionContent: React.FC = () => { {!isVSCode && } = [ { id: 'left', labelKey: 'settings.openchamber.desktopNetwork.option.windowControlsLeft' }, @@ -324,6 +324,8 @@ export const OpenChamberVisualSettings: React.FC const setPromptNavigatorEnabled = useUIStore(state => state.setPromptNavigatorEnabled); const expandedEditorToolbar = useUIStore(state => state.expandedEditorToolbar); const setExpandedEditorToolbar = useUIStore(state => state.setExpandedEditorToolbar); + const autoSaveEnabled = useUIStore(state => state.autoSaveEnabled); + const setAutoSaveEnabled = useUIStore(state => state.setAutoSaveEnabled); const wideChatLayoutEnabled = useUIStore(state => state.wideChatLayoutEnabled); const setWideChatLayoutEnabled = useUIStore(state => state.setWideChatLayoutEnabled); const codeBlockLineWrap = useUIStore(state => state.codeBlockLineWrap); @@ -630,7 +632,7 @@ export const OpenChamberVisualSettings: React.FC ? hasLocalizationSettings : (shouldShow('theme') || showWindowControlsPositionSetting || showMobileLayoutSetting || shouldShow('pwaInstallName') || shouldShow('pwaOrientation') || shouldShow('timeFormat') || shouldShow('weekStart')); const hasLayoutSettings = shouldShow('fontSize') || shouldShow('terminalFontSize') || shouldShow('editorFontSize') || shouldShow('spacing') || (shouldShow('inputBarOffset') && isMobile); - const hasNavigationSettings = (shouldShow('terminalQuickKeys') && !isMobile) || ((shouldShow('terminalShell') || shouldShow('terminalLoginShell')) && !isVSCode) || shouldShow('fileEditorKeymap') || (shouldShow('expandedEditorToolbar') && !isVSCode); + const hasNavigationSettings = (shouldShow('terminalQuickKeys') && !isMobile) || ((shouldShow('terminalShell') || shouldShow('terminalLoginShell')) && !isVSCode) || shouldShow('fileEditorKeymap') || shouldShow('autoSaveEnabled') || (shouldShow('expandedEditorToolbar') && !isVSCode); const hasBehaviorSettings = shouldShow('mermaidRendering') || (shouldShow('sessionGoal') && !isVSCode) || shouldShow('userMessageRendering') @@ -1482,6 +1484,16 @@ export const OpenChamberVisualSettings: React.FC )}
+ {shouldShow('autoSaveEnabled') && ( + + )} {shouldShow('expandedEditorToolbar') && !isVSCode && ( { }; const MAX_VIEW_CHARS = 200_000; -const FILE_EDITOR_AUTO_SAVE_KEY = 'openchamber:files:auto-save-enabled'; type FileLineEnding = '\n' | '\r\n'; const detectFileLineEnding = (content: string): FileLineEnding => { @@ -332,18 +332,6 @@ const serializeEditorContent = (content: string, lineEnding: FileLineEnding): st return lineEnding === '\r\n' ? normalized.replace(/\n/g, '\r\n') : normalized; }; -const getInitialAutoSaveEnabled = (): boolean => { - if (typeof window === 'undefined') { - return true; - } - - try { - return window.localStorage.getItem(FILE_EDITOR_AUTO_SAVE_KEY) !== 'false'; - } catch { - return true; - } -}; - const getFileIcon = (filePath: string, extension?: string): React.ReactNode => { return ; }; @@ -924,7 +912,9 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { const loadingFilePathRef = React.useRef(null); const [autoSaveStatus, setAutoSaveStatus] = React.useState<'idle' | 'saved'>('idle'); const [diagramSaved, setDiagramSaved] = React.useState(false); - const [autoSaveEnabled, setAutoSaveEnabled] = React.useState(getInitialAutoSaveEnabled); + const [contentDetectedBinary, setContentDetectedBinary] = React.useState(false); + const autoSaveEnabled = useUIStore((state) => state.autoSaveEnabled); + const setAutoSaveEnabled = useUIStore((state) => state.setAutoSaveEnabled); const [confirmDiscardOpen, setConfirmDiscardOpen] = React.useState(false); const pendingSelectFileRef = React.useRef(null); @@ -1626,16 +1616,25 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { return false; } - if (!isDirty) { - return true; - } - - if (draftContent === '' && fileContent !== '' && loadedFilePath !== selectedFile.path) { - console.warn( - `[saveDraft] refusing to save empty draft for "${selectedFile.path}" (${fileContent.length} bytes were expected). ` + - 'The file may have been read during a concurrent write (O_TRUNC race). ' + - 'Try again after content finishes loading if the save was intentional.', - ); + const selectedIsBinary = isBinaryFile(selectedFile.path) || contentDetectedBinary; + if (!shouldAllowFileDraftSave({ + selectedFilePath: selectedFile.path, + loadedFilePath, + fileLoading, + isDirty, + draftContent, + fileContent, + isNonEditableBinary: selectedIsBinary, + })) { + if (selectedIsBinary) { + console.warn(`[saveDraft] refusing to save binary file "${selectedFile.path}".`); + } else if (draftContent === '' && fileContent !== '' && loadedFilePath !== selectedFile.path) { + console.warn( + `[saveDraft] refusing to save empty draft for "${selectedFile.path}" (${fileContent.length} bytes were expected). ` + + 'The file may have been read during a concurrent write (O_TRUNC race). ' + + 'Try again after content finishes loading if the save was intentional.', + ); + } return false; } @@ -1668,7 +1667,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { } finally { setIsSaving(false); } - }, [draftContent, fileContent, files, isDirty, loadedFileLineEnding, loadedFilePath, readFileStat, selectedFile, t]); + }, [contentDetectedBinary, draftContent, fileContent, fileLoading, files, isDirty, loadedFileLineEnding, loadedFilePath, readFileStat, selectedFile, t]); React.useEffect(() => { if (!isDirty) { @@ -1696,14 +1695,6 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { }; }, [isDirty, setMainTabGuard]); - React.useEffect(() => { - try { - window.localStorage.setItem(FILE_EDITOR_AUTO_SAVE_KEY, autoSaveEnabled ? 'true' : 'false'); - } catch { - // Ignore localStorage errors; the in-memory preference still applies. - } - }, [autoSaveEnabled]); - React.useEffect(() => { if (autoSaveEnabled) { return; @@ -1721,7 +1712,17 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { React.useEffect(() => { const canWrite = Boolean(selectedFile && files.writeFile); - if (!autoSaveEnabled || !isDirty || !canWrite || isSaving) { + const selectedIsBinary = Boolean(selectedFile?.path && (isBinaryFile(selectedFile.path) || contentDetectedBinary)); + if (!shouldScheduleFileAutosave({ + autoSaveEnabled, + isDirty, + canWrite, + isSaving, + fileLoading, + selectedFilePath: selectedFile?.path, + loadedFilePath, + isNonEditableBinary: selectedIsBinary, + })) { return; } @@ -1739,7 +1740,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { autoSaveTimerRef.current = null; } }; - }, [autoSaveEnabled, draftContent, isDirty, selectedFile, files.writeFile, isSaving, saveDraft]); + }, [autoSaveEnabled, contentDetectedBinary, draftContent, fileLoading, isDirty, loadedFilePath, selectedFile, files.writeFile, isSaving, saveDraft]); // Reset auto-save status when switching files React.useEffect(() => { @@ -1789,10 +1790,12 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { setFileError(null); setDesktopImageSrc(''); setLoadedFilePath(null); + setContentDetectedBinary(false); const selectedIsImage = isImageFile(node.path); - const isSvg = node.path.toLowerCase().endsWith('.svg'); + const isSvg = isSvgFile(node.path); const selectedIsPdf = isPdfFile(node.path); + const selectedIsBinary = isBinaryFile(node.path); if (isMobile) { setShowMobilePageContent(true); @@ -1823,6 +1826,16 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { return; } + // Other known binaries (docx/xlsx/zip/…) must never be opened as text — + // a later autosave would corrupt them. + if (selectedIsBinary) { + setFileContent(''); + setDraftContent(''); + setLoadedFilePath(node.path); + setFileLoading(false); + return; + } + setFileLoading(true); const outsideFileGrant = getOutsideFileGrant(node.path); @@ -1836,6 +1849,13 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { if (!isCurrentLoad()) { return; } + if (looksLikeBinaryText(content)) { + setContentDetectedBinary(true); + setFileContent(''); + setDraftContent(''); + setLoadedFilePath(node.path); + return; + } const editorContent = normalizeEditorLineEndings(content); setLoadedFileLineEnding(detectFileLineEnding(content)); setFileContent(editorContent); @@ -2302,8 +2322,13 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { } const isSelectedImage = Boolean(selectedFile?.path && isImageFile(selectedFile.path)); - const isSelectedSvg = Boolean(selectedFile?.path && selectedFile.path.toLowerCase().endsWith('.svg')); + const isSelectedSvg = Boolean(selectedFile?.path && isSvgFile(selectedFile.path)); const isSelectedPdf = Boolean(selectedFile?.path && isPdfFile(selectedFile.path)); + const isSelectedBinary = Boolean( + selectedFile?.path + && (isBinaryFile(selectedFile.path) || contentDetectedBinary) + ); + const isUnsupportedBinary = isSelectedBinary && !isSelectedImage && !isSelectedPdf; const pendingNavigationTargetPath = React.useMemo( () => normalizePath(pendingFileNavigation?.path ?? ''), [pendingFileNavigation?.path], @@ -2316,21 +2341,22 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { && !fileLoading && !fileError && !isSelectedImage - && !isSelectedPdf, + && !isSelectedPdf + && !isUnsupportedBinary, ); const displaySelectedPath = React.useMemo(() => { return getDisplayPath(root, selectedFilePath); }, [selectedFilePath, root]); - const canCopy = Boolean(selectedFile && (!isSelectedImage || isSelectedSvg) && !isSelectedPdf && fileContent.length > 0); + const canCopy = Boolean(selectedFile && (!isSelectedImage || isSelectedSvg) && !isSelectedPdf && !isUnsupportedBinary && fileContent.length > 0); const canCopyPath = Boolean(selectedFile && displaySelectedPath.length > 0); - const canEdit = Boolean(selectedFile && !selectedFileIsOutsideWorkspace && !isSelectedImage && !isSelectedPdf && files.writeFile && fileContent.length <= MAX_VIEW_CHARS); + const canEdit = Boolean(selectedFile && !selectedFileIsOutsideWorkspace && !isSelectedBinary && files.writeFile && fileContent.length <= MAX_VIEW_CHARS); const isMarkdown = Boolean(selectedFile?.path && isMarkdownFile(selectedFile.path)); const isJson = Boolean(selectedFile?.path && isJsonFile(selectedFile.path)); const isHtml = Boolean(selectedFile?.path && isHtmlFile(selectedFile.path)); const isDrawio = Boolean(selectedFile?.path && isDrawioFile(selectedFile.path)); - const isTextFile = Boolean(selectedFile && !isSelectedImage && !isSelectedPdf); + const isTextFile = Boolean(selectedFile && !isSelectedBinary); const canUseShikiFileView = isTextFile && !isMarkdown && !isDrawio && !(isHtml && htmlViewMode === 'preview'); const isEditingFile = (isMarkdown && mdViewMode === 'edit') || (isHtml && htmlViewMode === 'edit') @@ -2535,7 +2561,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { const handleDiagramChange = React.useCallback((xml: string) => { diagramXmlRef.current = xml; - if (!selectedFile?.path || drawioViewMode !== 'preview' || !files.writeFile) { + if (!autoSaveEnabled || !selectedFile?.path || drawioViewMode !== 'preview' || !files.writeFile) { return; } @@ -2554,7 +2580,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { toast.error(error instanceof Error ? error.message : t('filesView.toast.saveFailed')); }); }, AUTO_SAVE_DELAY); - }, [drawioViewMode, files.writeFile, saveDiagramXml, selectedFile?.path, t]); + }, [autoSaveEnabled, drawioViewMode, files.writeFile, saveDiagramXml, selectedFile?.path, t]); const diagramEditorXml = React.useMemo(() => { if (!isDrawio) { @@ -2674,7 +2700,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { return; } - if (fileError || isSelectedImage || isSelectedPdf) { + if (fileError || isSelectedImage || isSelectedPdf || isUnsupportedBinary) { setPendingFileNavigation(null); pendingNavigationCycleRef.current = { key: '', attempts: 0 }; return; @@ -2746,6 +2772,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { fileLoading, isSelectedImage, isSelectedPdf, + isUnsupportedBinary, loadedFilePath, handleSelectFile, pendingFileNavigation, @@ -2783,7 +2810,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { // JSON tree, images, PDFs) never mount a CodeMirror editor, so the request // must clear regardless — otherwise it lingers and replays on every // dependency change. - if (!fileError && !isSelectedImage && !isSelectedPdf && canEdit && textViewMode === 'edit') { + if (!fileError && !isSelectedImage && !isSelectedPdf && !isUnsupportedBinary && canEdit && textViewMode === 'edit') { editorViewRef.current?.focus(); } @@ -2794,6 +2821,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => { fileLoading, isSelectedImage, isSelectedPdf, + isUnsupportedBinary, loadedFilePath, pendingFileFocusPath, root, @@ -3189,7 +3217,7 @@ export const FilesView: React.FC = ({ mode = 'full' }) => {
) : isSelectedPdf ? ( renderPdfPreview(selectedFile) + ) : isUnsupportedBinary ? ( +
+
{t('filesView.editor.cannotPreviewBinary')}
+
{t('filesView.editor.binaryFileDescription')}
+ {files.downloadFile ? ( + + ) : null} +
) : selectedFile && isDrawio && drawioViewMode === 'preview' ? (
= ({ mode = 'full' }) => {
) : isSelectedPdf ? ( renderPdfPreview(selectedFile) + ) : isUnsupportedBinary ? ( +
+
{t('filesView.editor.cannotPreviewBinary')}
+
{t('filesView.editor.binaryFileDescription')}
+ {files.downloadFile ? ( + + ) : null} +
) : isMarkdown && getMdViewMode() === 'preview' ? (
{fileContent.length > 500 * 1024 && ( diff --git a/packages/ui/src/lib/api/types.ts b/packages/ui/src/lib/api/types.ts index e97a1ada..04e329bf 100644 --- a/packages/ui/src/lib/api/types.ts +++ b/packages/ui/src/lib/api/types.ts @@ -637,6 +637,7 @@ export interface SettingsPayload { nativeNotificationsEnabled?: boolean; notificationMode?: 'always' | 'hidden-only'; autoDeleteEnabled?: boolean; + autoSaveEnabled?: boolean; autoDeleteAfterDays?: number; sessionRetentionAction?: 'archive' | 'delete'; followUpBehavior?: 'steer' | 'queue'; diff --git a/packages/ui/src/lib/appearanceAutoSave.ts b/packages/ui/src/lib/appearanceAutoSave.ts index 10d57fd5..d0b314a2 100644 --- a/packages/ui/src/lib/appearanceAutoSave.ts +++ b/packages/ui/src/lib/appearanceAutoSave.ts @@ -31,6 +31,7 @@ type AppearanceSlice = { summaryLength: number; maxLastMessageLength: number; autoDeleteEnabled: boolean; + autoSaveEnabled: boolean; autoDeleteAfterDays: number; sessionRetentionAction: 'archive' | 'delete'; fontSize: number; @@ -78,6 +79,7 @@ export const startAppearanceAutoSave = (): void => { summaryLength: useUIStore.getState().summaryLength, maxLastMessageLength: useUIStore.getState().maxLastMessageLength, autoDeleteEnabled: useUIStore.getState().autoDeleteEnabled, + autoSaveEnabled: useUIStore.getState().autoSaveEnabled, autoDeleteAfterDays: useUIStore.getState().autoDeleteAfterDays, sessionRetentionAction: useUIStore.getState().sessionRetentionAction, fontSize: useUIStore.getState().fontSize, @@ -117,6 +119,7 @@ export const startAppearanceAutoSave = (): void => { summaryLength: state.summaryLength, maxLastMessageLength: state.maxLastMessageLength, autoDeleteEnabled: state.autoDeleteEnabled, + autoSaveEnabled: state.autoSaveEnabled, autoDeleteAfterDays: state.autoDeleteAfterDays, sessionRetentionAction: state.sessionRetentionAction, fontSize: state.fontSize, @@ -196,6 +199,9 @@ export const startAppearanceAutoSave = (): void => { if (current.autoDeleteEnabled !== previous.autoDeleteEnabled) { diff.autoDeleteEnabled = current.autoDeleteEnabled; } + if (current.autoSaveEnabled !== previous.autoSaveEnabled) { + diff.autoSaveEnabled = current.autoSaveEnabled; + } if (current.autoDeleteAfterDays !== previous.autoDeleteAfterDays) { diff.autoDeleteAfterDays = current.autoDeleteAfterDays; } diff --git a/packages/ui/src/lib/desktop.ts b/packages/ui/src/lib/desktop.ts index b0b895a7..f98a2ebe 100644 --- a/packages/ui/src/lib/desktop.ts +++ b/packages/ui/src/lib/desktop.ts @@ -105,6 +105,7 @@ export type DesktopSettings = { renamedGroups?: Record; // groupId -> custom label }>; // Per-provider custom model groups configuration autoDeleteEnabled?: boolean; + autoSaveEnabled?: boolean; autoDeleteAfterDays?: number; sessionRetentionAction?: 'archive' | 'delete'; tunnelProvider?: string; diff --git a/packages/ui/src/lib/fileEditorAutosave.test.ts b/packages/ui/src/lib/fileEditorAutosave.test.ts new file mode 100644 index 00000000..1b78c1da --- /dev/null +++ b/packages/ui/src/lib/fileEditorAutosave.test.ts @@ -0,0 +1,60 @@ +import { describe, expect, test } from 'bun:test'; + +import { shouldAllowFileDraftSave, shouldScheduleFileAutosave } from './fileEditorAutosave'; + +describe('shouldScheduleFileAutosave', () => { + const ready = { + autoSaveEnabled: true, + isDirty: true, + canWrite: true, + isSaving: false, + fileLoading: false, + selectedFilePath: '/repo/a.txt', + loadedFilePath: '/repo/a.txt', + isNonEditableBinary: false, + }; + + test('schedules when dirty text file is fully loaded', () => { + expect(shouldScheduleFileAutosave(ready)).toBe(true); + }); + + test('skips while loading or when loaded path mismatches selection', () => { + expect(shouldScheduleFileAutosave({ ...ready, fileLoading: true })).toBe(false); + expect(shouldScheduleFileAutosave({ ...ready, loadedFilePath: null })).toBe(false); + expect(shouldScheduleFileAutosave({ ...ready, loadedFilePath: '/repo/other.txt' })).toBe(false); + }); + + test('skips when autosave disabled or file is binary', () => { + expect(shouldScheduleFileAutosave({ ...ready, autoSaveEnabled: false })).toBe(false); + expect(shouldScheduleFileAutosave({ ...ready, isNonEditableBinary: true })).toBe(false); + }); + + test('skips when not dirty, cannot write, or already saving', () => { + expect(shouldScheduleFileAutosave({ ...ready, isDirty: false })).toBe(false); + expect(shouldScheduleFileAutosave({ ...ready, canWrite: false })).toBe(false); + expect(shouldScheduleFileAutosave({ ...ready, isSaving: true })).toBe(false); + }); +}); + +describe('shouldAllowFileDraftSave', () => { + const ready = { + selectedFilePath: '/repo/a.txt', + loadedFilePath: '/repo/a.txt', + fileLoading: false, + isDirty: true, + draftContent: 'edited', + fileContent: 'original', + isNonEditableBinary: false, + }; + + test('allows save for loaded dirty text', () => { + expect(shouldAllowFileDraftSave(ready)).toBe(true); + }); + + test('refuses incomplete load, binary, or clean draft', () => { + expect(shouldAllowFileDraftSave({ ...ready, fileLoading: true })).toBe(false); + expect(shouldAllowFileDraftSave({ ...ready, loadedFilePath: null })).toBe(false); + expect(shouldAllowFileDraftSave({ ...ready, isNonEditableBinary: true })).toBe(false); + expect(shouldAllowFileDraftSave({ ...ready, isDirty: false })).toBe(false); + }); +}); diff --git a/packages/ui/src/lib/fileEditorAutosave.ts b/packages/ui/src/lib/fileEditorAutosave.ts new file mode 100644 index 00000000..6081877b --- /dev/null +++ b/packages/ui/src/lib/fileEditorAutosave.ts @@ -0,0 +1,54 @@ +export type FileEditorAutosaveGate = { + autoSaveEnabled: boolean; + isDirty: boolean; + canWrite: boolean; + isSaving: boolean; + fileLoading: boolean; + selectedFilePath: string | null | undefined; + loadedFilePath: string | null; + /** True when the selected file must never be written as text (binary / non-editable). */ + isNonEditableBinary: boolean; +}; + +/** + * Whether the FilesView autosave effect should schedule a debounced save. + * Incomplete loads and binary files must never trigger a write. + */ +export function shouldScheduleFileAutosave(gate: FileEditorAutosaveGate): boolean { + if (!gate.autoSaveEnabled || !gate.isDirty || !gate.canWrite || gate.isSaving) { + return false; + } + if (gate.fileLoading || gate.isNonEditableBinary) { + return false; + } + if (!gate.selectedFilePath || gate.loadedFilePath !== gate.selectedFilePath) { + return false; + } + return true; +} + +export type FileEditorSaveDraftGate = { + selectedFilePath: string | null | undefined; + loadedFilePath: string | null; + fileLoading: boolean; + isDirty: boolean; + draftContent: string; + fileContent: string; + isNonEditableBinary: boolean; +}; + +/** + * Whether saveDraft may write. Refuses empty drafts against stale content and any binary target. + */ +export function shouldAllowFileDraftSave(gate: FileEditorSaveDraftGate): boolean { + if (!gate.selectedFilePath || !gate.isDirty) { + return false; + } + if (gate.fileLoading || gate.loadedFilePath !== gate.selectedFilePath || gate.isNonEditableBinary) { + return false; + } + if (gate.draftContent === '' && gate.fileContent !== '' && gate.loadedFilePath !== gate.selectedFilePath) { + return false; + } + return true; +} diff --git a/packages/ui/src/lib/i18n/messages/en.settings.ts b/packages/ui/src/lib/i18n/messages/en.settings.ts index 0f2e9be7..19106f41 100644 --- a/packages/ui/src/lib/i18n/messages/en.settings.ts +++ b/packages/ui/src/lib/i18n/messages/en.settings.ts @@ -1878,6 +1878,9 @@ export const settingsDict = { 'settings.openchamber.visual.field.promptNavigatorEnabled': 'Prompt Navigator', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'Always show editor toolbar', 'settings.openchamber.visual.field.expandedEditorToolbar': 'Always show editor toolbar (docked under the file tabs)', + 'settings.openchamber.visual.field.autoSaveEnabledAria': 'Auto-save files', + 'settings.openchamber.visual.field.autoSaveEnabled': 'Auto-save files', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': 'Automatically save file edits after you stop typing. Disable to require manual save.', 'settings.openchamber.visual.field.wideChatLayoutAria': 'Wide chat layout', 'settings.openchamber.visual.field.wideChatLayout': 'Wide Chat Layout', 'settings.openchamber.visual.field.codeBlockLineWrapAria': 'Wrap code block lines', diff --git a/packages/ui/src/lib/i18n/messages/en.ts b/packages/ui/src/lib/i18n/messages/en.ts index 23f5011c..b201235b 100644 --- a/packages/ui/src/lib/i18n/messages/en.ts +++ b/packages/ui/src/lib/i18n/messages/en.ts @@ -1247,6 +1247,8 @@ export const dict = { 'filesView.editor.showControlsAria': 'Show editor controls', 'filesView.editor.controlsTitle': 'Editor controls', 'filesView.editor.pickFileFromTree': 'Pick a file from the tree.', + 'filesView.editor.cannotPreviewBinary': 'Cannot preview binary file', + 'filesView.editor.binaryFileDescription': 'This file is binary and cannot be edited in OpenChamber. Download it to open with another app.', 'filesView.state.loading': 'Loading...', 'filesView.state.openingFileAtChange': 'Opening file at change...', 'filesView.tree.search.placeholder': 'Search files...', diff --git a/packages/ui/src/lib/i18n/messages/es.settings.ts b/packages/ui/src/lib/i18n/messages/es.settings.ts index 68538aab..66036395 100644 --- a/packages/ui/src/lib/i18n/messages/es.settings.ts +++ b/packages/ui/src/lib/i18n/messages/es.settings.ts @@ -1845,6 +1845,9 @@ export const settingsDict = { "settings.openchamber.visual.field.promptNavigatorEnabled": "Navegador de prompts", "settings.openchamber.visual.field.expandedEditorToolbarAria": "Mostrar siempre la barra de herramientas del editor", "settings.openchamber.visual.field.expandedEditorToolbar": "Mostrar siempre la barra de herramientas del editor (anclada bajo las pestañas)", + "settings.openchamber.visual.field.autoSaveEnabledAria": "Guardado automático de archivos", + "settings.openchamber.visual.field.autoSaveEnabled": "Guardado automático de archivos", + "settings.openchamber.visual.field.autoSaveEnabledInfo": "Guarda automáticamente las ediciones del archivo después de dejar de escribir. Desactívalo para exigir un guardado manual.", "settings.openchamber.visual.field.wideChatLayoutAria": "Diseño de chat ancho", "settings.openchamber.visual.field.wideChatLayout": "Diseño de chat ancho", "settings.openchamber.visual.field.showSplitAssistantMessageActionsAria": "Acciones en línea del asistente", diff --git a/packages/ui/src/lib/i18n/messages/es.ts b/packages/ui/src/lib/i18n/messages/es.ts index e5f46751..f5144b9d 100644 --- a/packages/ui/src/lib/i18n/messages/es.ts +++ b/packages/ui/src/lib/i18n/messages/es.ts @@ -1213,6 +1213,8 @@ export const dict: Record = { "filesView.editor.showControlsAria": "Mostrar controles del editor", "filesView.editor.controlsTitle": "Controles del editor", "filesView.editor.pickFileFromTree": "Selecciona un archivo del árbol.", + "filesView.editor.cannotPreviewBinary": "No se puede previsualizar el archivo binario", + "filesView.editor.binaryFileDescription": "Este archivo es binario y no se puede editar en OpenChamber. Descárgalo para abrirlo con otra aplicación.", "filesView.state.loading": "Cargando...", "filesView.state.openingFileAtChange": "Abriendo archivo en cambio...", "filesView.tree.search.placeholder": "Buscar archivos...", diff --git a/packages/ui/src/lib/i18n/messages/fr.settings.ts b/packages/ui/src/lib/i18n/messages/fr.settings.ts index e1fdd8c5..d181fcca 100644 --- a/packages/ui/src/lib/i18n/messages/fr.settings.ts +++ b/packages/ui/src/lib/i18n/messages/fr.settings.ts @@ -1750,6 +1750,9 @@ export const settingsDict = { 'settings.openchamber.visual.field.promptNavigatorEnabled': 'Navigateur de prompts', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'Toujours afficher la barre d’outils de l’éditeur', 'settings.openchamber.visual.field.expandedEditorToolbar': 'Toujours afficher la barre d’outils de l’éditeur (ancrée sous les onglets de fichiers)', + 'settings.openchamber.visual.field.autoSaveEnabledAria': 'Enregistrement automatique des fichiers', + 'settings.openchamber.visual.field.autoSaveEnabled': 'Enregistrement automatique des fichiers', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': 'Enregistre automatiquement les modifications après l’arrêt de la saisie. Désactivez pour exiger un enregistrement manuel.', 'settings.openchamber.visual.field.wideChatLayoutAria': 'Large disposition de discussion', 'settings.openchamber.visual.field.wideChatLayout': 'Disposition de discussion large', 'settings.openchamber.visual.field.showSplitAssistantMessageActionsAria': 'Actions intégrées de l\'assistant', diff --git a/packages/ui/src/lib/i18n/messages/fr.ts b/packages/ui/src/lib/i18n/messages/fr.ts index 60ba4fc0..159be205 100644 --- a/packages/ui/src/lib/i18n/messages/fr.ts +++ b/packages/ui/src/lib/i18n/messages/fr.ts @@ -1075,6 +1075,8 @@ export const dict = { 'filesView.editor.showControlsAria': 'Afficher les contrôles de l\'éditeur', 'filesView.editor.controlsTitle': 'Contrôles de l\'éditeur', 'filesView.editor.pickFileFromTree': 'Choisissez un fichier dans l\'arborescence.', + 'filesView.editor.cannotPreviewBinary': 'Impossible de prévisualiser le fichier binaire', + 'filesView.editor.binaryFileDescription': 'Ce fichier est binaire et ne peut pas être modifié dans OpenChamber. Téléchargez-le pour l’ouvrir avec une autre application.', 'filesView.state.loading': 'Chargement...', 'filesView.state.openingFileAtChange': 'Ouverture du fichier lors du changement...', 'filesView.tree.search.placeholder': 'Rechercher des fichiers...', diff --git a/packages/ui/src/lib/i18n/messages/ja.settings.ts b/packages/ui/src/lib/i18n/messages/ja.settings.ts index fd95fe94..059acb7b 100644 --- a/packages/ui/src/lib/i18n/messages/ja.settings.ts +++ b/packages/ui/src/lib/i18n/messages/ja.settings.ts @@ -1878,6 +1878,9 @@ export const settingsDict = { 'settings.openchamber.visual.field.promptNavigatorEnabled': 'プロンプトナビゲーター', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'エディターツールバーを常に表示', 'settings.openchamber.visual.field.expandedEditorToolbar': 'エディターツールバーを常に表示（ファイルタブの下にドッキング）', + 'settings.openchamber.visual.field.autoSaveEnabledAria': 'ファイルの自動保存', + 'settings.openchamber.visual.field.autoSaveEnabled': 'ファイルの自動保存', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': '入力を止めた後にファイルの編集内容を自動保存します。無効にすると手動保存が必要になります。', 'settings.openchamber.visual.field.wideChatLayoutAria': 'ワイドチャットレイアウト', 'settings.openchamber.visual.field.wideChatLayout': 'ワイドチャットレイアウト', 'settings.openchamber.visual.field.showSplitAssistantMessageActionsAria': 'インラインアシスタントアクション', diff --git a/packages/ui/src/lib/i18n/messages/ja.ts b/packages/ui/src/lib/i18n/messages/ja.ts index ef32a963..d31555e2 100644 --- a/packages/ui/src/lib/i18n/messages/ja.ts +++ b/packages/ui/src/lib/i18n/messages/ja.ts @@ -1243,6 +1243,8 @@ export const dict: Record = { 'filesView.editor.showControlsAria': 'エディターコントロールを表示', 'filesView.editor.controlsTitle': 'エディターコントロール', 'filesView.editor.pickFileFromTree': 'ツリーからファイルを選択してください。', + 'filesView.editor.cannotPreviewBinary': 'バイナリファイルはプレビューできません', + 'filesView.editor.binaryFileDescription': 'このファイルはバイナリのため、OpenChamberでは編集できません。別のアプリで開くにはダウンロードしてください。', 'filesView.state.loading': '読み込み中...', 'filesView.state.openingFileAtChange': '変更箇所のファイルを開いています...', 'filesView.tree.search.placeholder': 'ファイルを検索...', diff --git a/packages/ui/src/lib/i18n/messages/ko.settings.ts b/packages/ui/src/lib/i18n/messages/ko.settings.ts index d8de884a..2269af8a 100644 --- a/packages/ui/src/lib/i18n/messages/ko.settings.ts +++ b/packages/ui/src/lib/i18n/messages/ko.settings.ts @@ -1845,6 +1845,9 @@ export const settingsDict = { 'settings.openchamber.visual.field.promptNavigatorEnabled': '프롬프트 탐색기', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'Always show editor toolbar', 'settings.openchamber.visual.field.expandedEditorToolbar': 'Always show editor toolbar (docked under the file tabs)', + 'settings.openchamber.visual.field.autoSaveEnabledAria': '파일 자동 저장', + 'settings.openchamber.visual.field.autoSaveEnabled': '파일 자동 저장', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': '입력을 멈춘 후 파일 편집 내용을 자동으로 저장합니다. 끄면 수동으로 저장해야 합니다.', 'settings.openchamber.visual.field.wideChatLayoutAria': '넓은 채팅 레이아웃', 'settings.openchamber.visual.field.wideChatLayout': '넓은 채팅 레이아웃', 'settings.openchamber.visual.field.showSplitAssistantMessageActionsAria': '인라인 어시스턴트 작업', diff --git a/packages/ui/src/lib/i18n/messages/ko.ts b/packages/ui/src/lib/i18n/messages/ko.ts index 607b1dbf..a06fbcda 100644 --- a/packages/ui/src/lib/i18n/messages/ko.ts +++ b/packages/ui/src/lib/i18n/messages/ko.ts @@ -1250,6 +1250,8 @@ export const dict: Record = { 'filesView.editor.showControlsAria': '편집기 컨트롤 표시', 'filesView.editor.controlsTitle': '편집기 컨트롤', 'filesView.editor.pickFileFromTree': '트리에서 파일을 선택하세요.', + 'filesView.editor.cannotPreviewBinary': '바이너리 파일을 미리볼 수 없음', + 'filesView.editor.binaryFileDescription': '이 파일은 바이너리이므로 OpenChamber에서 편집할 수 없습니다. 다른 앱으로 열려면 다운로드하세요.', 'filesView.state.loading': '로드 중…', 'filesView.state.openingFileAtChange': '변경 위치에서 파일 여는 중…', 'filesView.tree.search.placeholder': '파일 검색…', diff --git a/packages/ui/src/lib/i18n/messages/pl.settings.ts b/packages/ui/src/lib/i18n/messages/pl.settings.ts index aeb894da..078f08f2 100644 --- a/packages/ui/src/lib/i18n/messages/pl.settings.ts +++ b/packages/ui/src/lib/i18n/messages/pl.settings.ts @@ -1085,6 +1085,9 @@ export const settingsDict = { 'settings.openchamber.visual.field.promptNavigatorEnabled': 'Nawigator promptów', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'Always show editor toolbar', 'settings.openchamber.visual.field.expandedEditorToolbar': 'Always show editor toolbar (docked under the file tabs)', + 'settings.openchamber.visual.field.autoSaveEnabledAria': 'Autozapis plików', + 'settings.openchamber.visual.field.autoSaveEnabled': 'Autozapis plików', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': 'Automatycznie zapisuje edycje pliku po zatrzymaniu pisania. Wyłącz, aby wymagać ręcznego zapisu.', 'settings.openchamber.visual.field.terminalFontSize': 'Rozmiar czcionki terminala', 'settings.openchamber.visual.field.terminalShell': 'Powłoka terminala', 'settings.openchamber.visual.field.terminalShellAria': 'Wybierz powłokę terminala', diff --git a/packages/ui/src/lib/i18n/messages/pl.ts b/packages/ui/src/lib/i18n/messages/pl.ts index ad56768c..ac1f8e11 100644 --- a/packages/ui/src/lib/i18n/messages/pl.ts +++ b/packages/ui/src/lib/i18n/messages/pl.ts @@ -1728,6 +1728,8 @@ export const dict: Record = { 'filesView.editor.openFilesAria': 'Otwarte pliki', 'filesView.editor.openInDesktopApp': 'Otwórz w aplikacji desktopowej', 'filesView.editor.pickFileFromTree': 'Wybierz plik z drzewa.', + 'filesView.editor.cannotPreviewBinary': 'Nie można podglądać pliku binarnego', + 'filesView.editor.binaryFileDescription': 'Ten plik jest binarny i nie można go edytować w OpenChamber. Pobierz go, aby otworzyć w innej aplikacji.', 'filesView.editor.refreshApps': 'Odśwież aplikacje', 'filesView.editor.saveAria': 'Zapisz ({shortcut})', 'filesView.editor.saveFile': 'Zapisz plik', diff --git a/packages/ui/src/lib/i18n/messages/pt-BR.settings.ts b/packages/ui/src/lib/i18n/messages/pt-BR.settings.ts index 45fb008d..b0102e86 100644 --- a/packages/ui/src/lib/i18n/messages/pt-BR.settings.ts +++ b/packages/ui/src/lib/i18n/messages/pt-BR.settings.ts @@ -1845,6 +1845,9 @@ export const settingsDict = { "settings.openchamber.visual.field.promptNavigatorEnabled": "Navegador de prompts", "settings.openchamber.visual.field.expandedEditorToolbarAria": "Mostrar sempre a barra de ferramentas do editor", "settings.openchamber.visual.field.expandedEditorToolbar": "Mostrar sempre a barra de ferramentas do editor (ancorada sob as abas)", + "settings.openchamber.visual.field.autoSaveEnabledAria": "Salvamento automático de arquivos", + "settings.openchamber.visual.field.autoSaveEnabled": "Salvamento automático de arquivos", + "settings.openchamber.visual.field.autoSaveEnabledInfo": "Salva automaticamente as edições do arquivo depois que você parar de digitar. Desative para exigir salvamento manual.", "settings.openchamber.visual.field.wideChatLayoutAria": "Layout de chat amplo", "settings.openchamber.visual.field.wideChatLayout": "Layout de chat amplo", "settings.openchamber.visual.field.showSplitAssistantMessageActionsAria": "Ações inline do assistente", diff --git a/packages/ui/src/lib/i18n/messages/pt-BR.ts b/packages/ui/src/lib/i18n/messages/pt-BR.ts index 4c1496dd..a7812c58 100644 --- a/packages/ui/src/lib/i18n/messages/pt-BR.ts +++ b/packages/ui/src/lib/i18n/messages/pt-BR.ts @@ -1213,6 +1213,8 @@ export const dict: Record = { "filesView.editor.showControlsAria": "Mostrar controles do editor", "filesView.editor.controlsTitle": "Controles do editor", "filesView.editor.pickFileFromTree": "Selecione um arquivo na árvore.", + "filesView.editor.cannotPreviewBinary": "Não é possível pré-visualizar o arquivo binário", + "filesView.editor.binaryFileDescription": "Este arquivo é binário e não pode ser editado no OpenChamber. Baixe-o para abrir em outro aplicativo.", "filesView.state.loading": "Carregando...", "filesView.state.openingFileAtChange": "Abrindo arquivo na alteração...", "filesView.tree.search.placeholder": "Pesquisar arquivos...", diff --git a/packages/ui/src/lib/i18n/messages/uk.settings.ts b/packages/ui/src/lib/i18n/messages/uk.settings.ts index e5e97146..bd2cc95a 100644 --- a/packages/ui/src/lib/i18n/messages/uk.settings.ts +++ b/packages/ui/src/lib/i18n/messages/uk.settings.ts @@ -1845,6 +1845,9 @@ export const settingsDict = { "settings.openchamber.visual.field.promptNavigatorEnabled": "Навігатор промптів", "settings.openchamber.visual.field.expandedEditorToolbarAria": "Завжди показувати панель інструментів редактора", "settings.openchamber.visual.field.expandedEditorToolbar": "Завжди показувати панель інструментів редактора (закріплена під вкладками)", + "settings.openchamber.visual.field.autoSaveEnabledAria": "Автозбереження файлів", + "settings.openchamber.visual.field.autoSaveEnabled": "Автозбереження файлів", + "settings.openchamber.visual.field.autoSaveEnabledInfo": "Автоматично зберігати зміни у файлі після того, як ви припините друкувати. Вимкніть, щоб зберігати лише вручну.", "settings.openchamber.visual.field.wideChatLayoutAria": "Широкий макет чату", "settings.openchamber.visual.field.wideChatLayout": "Широкий макет чату", "settings.openchamber.visual.field.showSplitAssistantMessageActionsAria": "Вбудовані дії асистента", diff --git a/packages/ui/src/lib/i18n/messages/uk.ts b/packages/ui/src/lib/i18n/messages/uk.ts index cc76d4d2..a0095e81 100644 --- a/packages/ui/src/lib/i18n/messages/uk.ts +++ b/packages/ui/src/lib/i18n/messages/uk.ts @@ -1213,6 +1213,8 @@ export const dict: Record = { "filesView.editor.showControlsAria": "Показати елементи керування редактора", "filesView.editor.controlsTitle": "Елементи керування редактора", "filesView.editor.pickFileFromTree": "Вибрати файл із дерева.", + "filesView.editor.cannotPreviewBinary": "Неможливо попередньо переглянути бінарний файл", + "filesView.editor.binaryFileDescription": "Цей файл є бінарним і його не можна редагувати в OpenChamber. Завантажте його, щоб відкрити в іншій програмі.", "filesView.state.loading": "Завантаження...", "filesView.state.openingFileAtChange": "Відкриття файлу на зміні...", "filesView.tree.search.placeholder": "Пошук файлів...", diff --git a/packages/ui/src/lib/i18n/messages/zh-CN.settings.ts b/packages/ui/src/lib/i18n/messages/zh-CN.settings.ts index 9ece1ed5..39068a15 100644 --- a/packages/ui/src/lib/i18n/messages/zh-CN.settings.ts +++ b/packages/ui/src/lib/i18n/messages/zh-CN.settings.ts @@ -1845,6 +1845,9 @@ export const settingsDict = { 'settings.openchamber.visual.field.promptNavigatorEnabled': '提示词导航', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'Always show editor toolbar', 'settings.openchamber.visual.field.expandedEditorToolbar': 'Always show editor toolbar (docked under the file tabs)', + 'settings.openchamber.visual.field.autoSaveEnabledAria': '自动保存文件', + 'settings.openchamber.visual.field.autoSaveEnabled': '自动保存文件', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': '停止输入后自动保存文件编辑内容。关闭后需手动保存。', 'settings.openchamber.visual.field.wideChatLayoutAria': '宽聊天布局', 'settings.openchamber.visual.field.wideChatLayout': '宽聊天布局', 'settings.openchamber.visual.field.showSplitAssistantMessageActionsAria': '内联助手操作', diff --git a/packages/ui/src/lib/i18n/messages/zh-CN.ts b/packages/ui/src/lib/i18n/messages/zh-CN.ts index d9c335c3..cdc65f88 100644 --- a/packages/ui/src/lib/i18n/messages/zh-CN.ts +++ b/packages/ui/src/lib/i18n/messages/zh-CN.ts @@ -1213,6 +1213,8 @@ export const dict: Record = { 'filesView.editor.showControlsAria': '显示编辑器控制项', 'filesView.editor.controlsTitle': '编辑器控制项', 'filesView.editor.pickFileFromTree': '请从文件树中选择一个文件。', + 'filesView.editor.cannotPreviewBinary': '无法预览二进制文件', + 'filesView.editor.binaryFileDescription': '此文件为二进制文件，无法在 OpenChamber 中编辑。请下载后使用其他应用打开。', 'filesView.state.loading': '加载中...', 'filesView.state.openingFileAtChange': '正在打开变更处的文件...', 'filesView.tree.search.placeholder': '搜索文件...', diff --git a/packages/ui/src/lib/i18n/messages/zh-TW.settings.ts b/packages/ui/src/lib/i18n/messages/zh-TW.settings.ts index 9d88b44e..bbb542eb 100644 --- a/packages/ui/src/lib/i18n/messages/zh-TW.settings.ts +++ b/packages/ui/src/lib/i18n/messages/zh-TW.settings.ts @@ -1751,6 +1751,9 @@ 'settings.openchamber.visual.field.promptNavigatorEnabled': '提示詞導覽', 'settings.openchamber.visual.field.expandedEditorToolbarAria': 'Always show editor toolbar', 'settings.openchamber.visual.field.expandedEditorToolbar': 'Always show editor toolbar (docked under the file tabs)', + 'settings.openchamber.visual.field.autoSaveEnabledAria': '自動儲存檔案', + 'settings.openchamber.visual.field.autoSaveEnabled': '自動儲存檔案', + 'settings.openchamber.visual.field.autoSaveEnabledInfo': '停止輸入後自動儲存檔案編輯內容。關閉後需手動儲存。', 'settings.openchamber.visual.field.wideChatLayoutAria': '寬聊天佈局', 'settings.openchamber.visual.field.wideChatLayout': '寬聊天佈局', 'settings.openchamber.visual.field.showSplitAssistantMessageActionsAria': '行內助理操作', diff --git a/packages/ui/src/lib/i18n/messages/zh-TW.ts b/packages/ui/src/lib/i18n/messages/zh-TW.ts index 79d88b7b..ba25880a 100644 --- a/packages/ui/src/lib/i18n/messages/zh-TW.ts +++ b/packages/ui/src/lib/i18n/messages/zh-TW.ts @@ -1224,6 +1224,8 @@ export const dict: Record = { 'filesView.editor.showControlsAria': '顯示編輯器控制項', 'filesView.editor.controlsTitle': '編輯器控制項', 'filesView.editor.pickFileFromTree': '請從檔案樹中選擇一個檔案。', + 'filesView.editor.cannotPreviewBinary': '無法預覽二進位檔案', + 'filesView.editor.binaryFileDescription': '此檔案為二進位檔案，無法在 OpenChamber 中編輯。請下載後使用其他應用程式開啟。', 'filesView.state.loading': '載入中...', 'filesView.state.openingFileAtChange': '正在開啟變更處的檔案...', 'filesView.tree.search.placeholder': '搜尋檔案...', diff --git a/packages/ui/src/lib/persistence.ts b/packages/ui/src/lib/persistence.ts index 7db5b302..4a3d846e 100644 --- a/packages/ui/src/lib/persistence.ts +++ b/packages/ui/src/lib/persistence.ts @@ -531,6 +531,7 @@ const materializeAuthoritativeUiSettings = (settings: DesktopSettings): DesktopS sessionGoalDefaultBudget: defaults.sessionGoalDefaultBudget, collapsibleThinkingBlocks: defaults.collapsibleThinkingBlocks, autoDeleteEnabled: defaults.autoDeleteEnabled, + autoSaveEnabled: defaults.autoSaveEnabled, autoDeleteAfterDays: defaults.autoDeleteAfterDays, sessionRetentionAction: defaults.sessionRetentionAction, followUpBehavior: DEFAULT_FOLLOW_UP_BEHAVIOR, @@ -636,6 +637,9 @@ const applyDesktopUiPreferences = (settings: DesktopSettings) => { if (typeof settings.autoDeleteEnabled === 'boolean' && settings.autoDeleteEnabled !== store.autoDeleteEnabled) { store.setAutoDeleteEnabled(settings.autoDeleteEnabled); } + if (typeof settings.autoSaveEnabled === 'boolean' && settings.autoSaveEnabled !== store.autoSaveEnabled) { + store.setAutoSaveEnabled(settings.autoSaveEnabled); + } if (typeof settings.autoDeleteAfterDays === 'number' && Number.isFinite(settings.autoDeleteAfterDays)) { const normalized = Math.max(1, Math.min(365, settings.autoDeleteAfterDays)); if (normalized !== store.autoDeleteAfterDays) { @@ -1084,6 +1088,9 @@ const sanitizeWebSettings = (payload: unknown): DesktopSettings | null => { if (typeof candidate.autoDeleteEnabled === 'boolean') { result.autoDeleteEnabled = candidate.autoDeleteEnabled; } + if (typeof candidate.autoSaveEnabled === 'boolean') { + result.autoSaveEnabled = candidate.autoSaveEnabled; + } if (typeof candidate.autoDeleteAfterDays === 'number' && Number.isFinite(candidate.autoDeleteAfterDays)) { result.autoDeleteAfterDays = candidate.autoDeleteAfterDays; } diff --git a/packages/ui/src/lib/settings/search.ts b/packages/ui/src/lib/settings/search.ts index 34a5f415..74dc95ca 100644 --- a/packages/ui/src/lib/settings/search.ts +++ b/packages/ui/src/lib/settings/search.ts @@ -147,6 +147,13 @@ const SETTINGS_SEARCH_ITEMS: readonly SettingsSearchItem[] = [ // Only the mobile composer applies this offset (ChatInput gates on isMobile). isAvailable: (ctx) => ctx.isMobile, }, + { + id: 'appearance.auto-save-enabled', + page: 'general', + titleKey: 'settings.openchamber.visual.field.autoSaveEnabled', + descriptionKey: 'settings.openchamber.visual.field.autoSaveEnabledInfo', + keywords: ['editor', 'autosave', 'auto-save', 'files', 'save'], + }, { id: 'appearance.expanded-editor-toolbar', page: 'general', diff --git a/packages/ui/src/lib/toolHelpers.binary.test.ts b/packages/ui/src/lib/toolHelpers.binary.test.ts new file mode 100644 index 00000000..03a294c8 --- /dev/null +++ b/packages/ui/src/lib/toolHelpers.binary.test.ts @@ -0,0 +1,44 @@ +import { describe, expect, test } from 'bun:test'; + +import { + getFileExtension, + isBinaryFile, + isImageFile, + isPdfFile, + isSvgFile, + looksLikeBinaryText, +} from './toolHelpers'; + +describe('binary file helpers', () => { + test('classifies common binary extensions', () => { + expect(isBinaryFile('/repo/docs/report.pdf')).toBe(true); + expect(isBinaryFile('/repo/sheet.XLSX')).toBe(true); + expect(isBinaryFile('archive.zip')).toBe(true); + expect(isBinaryFile('photo.png')).toBe(true); + expect(isBinaryFile('notes.docx')).toBe(true); + expect(isPdfFile('report.pdf')).toBe(true); + expect(isImageFile('photo.png')).toBe(true); + }); + + test('keeps text and SVG editable', () => { + expect(isBinaryFile('/repo/README.md')).toBe(false); + expect(isBinaryFile('/repo/src/main.ts')).toBe(false); + expect(isBinaryFile('/repo/icon.svg')).toBe(false); + expect(isSvgFile('/repo/icon.svg')).toBe(true); + expect(isBinaryFile('/repo/.env')).toBe(false); + }); + + test('getFileExtension ignores leading dots and path separators', () => { + expect(getFileExtension('/a/b/c.PDF')).toBe('pdf'); + expect(getFileExtension('.gitignore')).toBe(''); + expect(getFileExtension('Makefile')).toBe(''); + }); + + test('looksLikeBinaryText detects nulls, PDF, ZIP, and replacement-heavy content', () => { + expect(looksLikeBinaryText('hello\0world')).toBe(true); + expect(looksLikeBinaryText('%PDF-1.7\nstream\n...')).toBe(true); + expect(looksLikeBinaryText(`PK\u0003\u0004${'x'.repeat(20)}`)).toBe(true); + expect(looksLikeBinaryText(`${'\uFFFD'.repeat(40)}${'a'.repeat(40)}`)).toBe(true); + expect(looksLikeBinaryText('plain text file\nwith newlines\n')).toBe(false); + }); +}); diff --git a/packages/ui/src/lib/toolHelpers.ts b/packages/ui/src/lib/toolHelpers.ts index 603485cb..c4f2a7c5 100644 --- a/packages/ui/src/lib/toolHelpers.ts +++ b/packages/ui/src/lib/toolHelpers.ts @@ -705,6 +705,84 @@ export function isPdfFile(filePath: string): boolean { return ext === 'pdf'; } +export function isSvgFile(filePath: string): boolean { + return filePath.toLowerCase().endsWith('.svg'); +} + +/** Known non-text extensions that must not be opened or saved as UTF-8 text. */ +const BINARY_FILE_EXTENSIONS = new Set([ + // Documents / office + 'pdf', 'doc', 'docx', 'xls', 'xlsx', 'ppt', 'pptx', 'odt', 'ods', 'odp', + // Archives / packages + 'zip', 'rar', '7z', 'gz', 'tgz', 'tar', 'bz2', 'xz', 'jar', 'war', 'apk', 'dmg', 'iso', + 'deb', 'rpm', 'msi', + // Images (svg is text and is excluded via isSvgFile) + ...IMAGE_EXTENSIONS.filter((ext) => ext !== 'svg'), + // Audio / video + 'mp3', 'mp4', 'm4a', 'aac', 'flac', 'ogg', 'wav', 'wma', 'avi', 'mov', 'mkv', 'webm', 'wmv', + // Fonts + 'ttf', 'otf', 'woff', 'woff2', 'eot', + // Native / bytecode + 'exe', 'dll', 'so', 'dylib', 'bin', 'class', 'o', 'a', 'lib', 'wasm', 'node', + // Databases / locks / misc binary + 'sqlite', 'sqlite3', 'db', 'dat', 'parquet', 'feather', 'pickle', 'pyc', 'pyo', 'lockb', +]); + +export function getFileExtension(filePath: string): string { + const base = filePath.split(/[/\\]/).pop() ?? filePath; + const dot = base.lastIndexOf('.'); + if (dot <= 0 || dot === base.length - 1) { + return ''; + } + return base.slice(dot + 1).toLowerCase(); +} + +/** True for known binary extensions (including images/PDF). SVG is not binary. */ +export function isBinaryFile(filePath: string): boolean { + if (isSvgFile(filePath)) { + return false; + } + const ext = getFileExtension(filePath); + return BINARY_FILE_EXTENSIONS.has(ext); +} + +/** + * Heuristic for UTF-8 text that is actually binary (or was lossily decoded). + * Used as defense-in-depth when extension checks miss a binary file. + */ +export function looksLikeBinaryText(content: string): boolean { + if (!content) { + return false; + } + + const sample = content.length > 8192 ? content.slice(0, 8192) : content; + if (sample.includes('\0')) { + return true; + } + if (sample.startsWith('%PDF')) { + return true; + } + // ZIP-based formats (docx/xlsx/pptx/jar/apk…) and raw ZIP. + if (sample.startsWith('PK\u0003\u0004') || sample.startsWith('PK\u0005\u0006') || sample.startsWith('PK\u0007\u0008')) { + return true; + } + + let suspicious = 0; + for (let index = 0; index < sample.length; index += 1) { + const code = sample.charCodeAt(index); + if (code === 0xFFFD) { + suspicious += 1; + continue; + } + // C0 controls excluding common whitespace (TAB/LF/VT/FF/CR). + if (code < 9 || (code > 13 && code < 32) || code === 127) { + suspicious += 1; + } + } + + return sample.length > 0 && suspicious / sample.length > 0.1; +} + export function getImageMimeType(filePath: string): string { const ext = filePath.split('.').pop()?.toLowerCase(); const mimeMap: Record = { diff --git a/packages/ui/src/stores/useUIStore.ts b/packages/ui/src/stores/useUIStore.ts index 817ec27f..2e39e955 100644 --- a/packages/ui/src/stores/useUIStore.ts +++ b/packages/ui/src/stores/useUIStore.ts @@ -620,6 +620,8 @@ interface UIStore { activityRenderMode: ActivityRenderMode; showDeletionDialog: boolean; autoDeleteEnabled: boolean; + /** Global file-editor autosave. Default true for backward compatibility. */ + autoSaveEnabled: boolean; autoDeleteAfterDays: number; sessionRetentionAction: SessionRetentionAction; autoDeleteLastRunAt: number | null; @@ -782,6 +784,7 @@ interface UIStore { setActivityRenderMode: (value: ActivityRenderMode) => void; setShowDeletionDialog: (value: boolean) => void; setAutoDeleteEnabled: (value: boolean) => void; + setAutoSaveEnabled: (value: boolean) => void; setAutoDeleteAfterDays: (days: number) => void; setSessionRetentionAction: (value: SessionRetentionAction) => void; setAutoDeleteLastRunAt: (timestamp: number | null) => void; @@ -938,6 +941,7 @@ export const useUIStore = create()( activityRenderMode: 'summary', showDeletionDialog: true, autoDeleteEnabled: false, + autoSaveEnabled: true, autoDeleteAfterDays: 30, sessionRetentionAction: 'archive', autoDeleteLastRunAt: null, @@ -1678,6 +1682,10 @@ export const useUIStore = create()( set({ autoDeleteEnabled: value }); }, + setAutoSaveEnabled: (value) => { + set({ autoSaveEnabled: value }); + }, + setAutoDeleteAfterDays: (days) => { const clampedDays = Math.max(1, Math.min(365, days)); set({ autoDeleteAfterDays: clampedDays }); @@ -2247,13 +2255,32 @@ export const useUIStore = create()( { name: 'ui-store', storage: createDeferredSafeJSONStorage(), - version: 12, + version: 13, migrate: (persistedState, version) => { if (!persistedState || typeof persistedState !== 'object') { return persistedState; } const state = persistedState as Record; + // v12 -> v13: promote FilesView localStorage autosave toggle into the store. + if (version < 13) { + if (typeof state.autoSaveEnabled !== 'boolean') { + let legacyEnabled = true; + try { + if (typeof localStorage !== 'undefined') { + const legacy = localStorage.getItem('openchamber:files:auto-save-enabled'); + if (legacy !== null) { + legacyEnabled = legacy !== 'false'; + localStorage.removeItem('openchamber:files:auto-save-enabled'); + } + } + } catch { + legacyEnabled = true; + } + state.autoSaveEnabled = legacyEnabled; + } + } + // v11 -> v12: drop legacy window-controls "auto" (always meant right). if (version < 12) { if (state.desktopWindowControlsPosition === 'auto' || state.desktopWindowControlsPosition == null) { @@ -2353,6 +2380,10 @@ export const useUIStore = create()( state.fileEditorKeymap = normalizeFileEditorKeymap(state.fileEditorKeymap); + if (typeof state.autoSaveEnabled !== 'boolean') { + state.autoSaveEnabled = true; + } + state.contextRailOrder = Array.isArray(state.contextRailOrder) ? (state.contextRailOrder as unknown[]).filter((id): id is string => typeof id === 'string' && id.trim() !== '') : []; @@ -2389,6 +2420,7 @@ export const useUIStore = create()( activityRenderMode: state.activityRenderMode, showDeletionDialog: state.showDeletionDialog, autoDeleteEnabled: state.autoDeleteEnabled, + autoSaveEnabled: state.autoSaveEnabled, autoDeleteAfterDays: state.autoDeleteAfterDays, sessionRetentionAction: state.sessionRetentionAction, autoDeleteLastRunAt: state.autoDeleteLastRunAt,