fix save issue

Author: petertzy

frontend/package-lock.json

@@ -11,6 +11,7 @@
   "dependencies": {
     "@monaco-editor/react": "^4.6.0",
     "@tauri-apps/api": "^2.10.1",
+    "@tauri-apps/plugin-dialog": "^2.4.2",
     "monaco-editor": "^0.48.0",
     "next": "^14.2.0",
     "react": "^18.3.0",

frontend/package.json

@@ -24,3 +24,4 @@ log = "0.4"
 tauri = { version = "2.10.3", features = [] }
 tauri-plugin-log = "2"
 tauri-plugin-shell = "2"
+tauri-plugin-dialog = "2"

frontend/src-tauri/Cargo.lock

@@ -7,6 +7,7 @@
   ],
   "permissions": [
     "core:default",
-    "shell:default"
+    "shell:default",
+    "dialog:default"
   ]
 }

frontend/src-tauri/Cargo.toml

@@ -18,6 +18,7 @@ pub fn run() {
     let port_state = Arc::new(Mutex::new(None::<u16>));
 
     tauri::Builder::default()
+        .plugin(tauri_plugin_dialog::init())
         .plugin(tauri_plugin_shell::init())
         .manage(BackendPort(port_state))
         .invoke_handler(tauri::generate_handler![get_backend_port])

frontend/src-tauri/capabilities/default.json

@@ -23,6 +23,14 @@ export default function HomePage() {
   const [showAIPanel, setShowAIPanel] = useState(false);
   const monacoRef = useRef<MonacoEditor.IStandaloneCodeEditor | null>(null);
 
+  const handleSaveFile = useCallback(async () => {
+    try {
+      await editor.saveFile();
+    } catch (err) {
+      alert(`Save failed: ${err instanceof Error ? err.message : String(err)}`);
+    }
+  }, [editor]);
+
   // Load recent files on mount and initialise preview
   useEffect(() => {
     editor.loadRecentFiles();
@@ -37,12 +45,12 @@ export default function HomePage() {
     const onKeyDown = (e: KeyboardEvent) => {
       if ((e.ctrlKey || e.metaKey) && e.key === "s") {
         e.preventDefault();
-        editor.saveFile();
+        void handleSaveFile();
       }
     };
     window.addEventListener("keydown", onKeyDown);
     return () => window.removeEventListener("keydown", onKeyDown);
-  }, [editor]);
+  }, [handleSaveFile]);
 
   // "Open file" — uses a hidden file-input because Tauri/browser can't call
   // the native dialog directly without the Tauri API.  In Tauri mode this
@@ -119,7 +127,7 @@ export default function HomePage() {
       <div className="relative">
         <Toolbar
           onOpenFile={handleOpenFile}
-          onSaveFile={() => editor.saveFile()}
+          onSaveFile={() => { void handleSaveFile(); }}
           onExport={handleExport}
           onToggleDark={() => editor.setDarkMode((d) => !d)}
           onToggleAIPanel={() => setShowAIPanel((v) => !v)}

frontend/src-tauri/src/lib.rs

@@ -117,9 +117,80 @@ export function useEditor() {
   const saveFile = useCallback(
     async (filePath?: string) => {
       const path = filePath ?? activeTab.filePath;
-      if (!path) return;
-      await Files.write(path, activeTab.content);
-      updateTab(activeTabId, { dirty: false, filePath: path, label: path.split(/[/\\]/).pop() ?? path });
+      if (path) {
+        await Files.write(path, activeTab.content);
+        updateTab(activeTabId, { dirty: false, filePath: path, label: path.split(/[/\\]/).pop() ?? path });
+        return;
+      }
+
+      // Save As flow: ask user for a destination when the tab has no path yet.
+      const rawLabel = activeTab.label.trim() || "Untitled";
+      const suggestedName = /\.[A-Za-z0-9]+$/.test(rawLabel) ? rawLabel : `${rawLabel}.md`;
+
+      // In Tauri desktop mode, use native save dialog and persist via backend API.
+      const isTauri = typeof window !== "undefined" && "__TAURI__" in window;
+      if (isTauri) {
+        try {
+          const { save } = await import("@tauri-apps/plugin-dialog");
+          const target = await save({
+            defaultPath: suggestedName,
+            filters: [{ name: "Markdown", extensions: ["md", "markdown", "txt"] }],
+          });
+
+          if (!target) return;
+
+          const resolvedPath = Array.isArray(target) ? target[0] : target;
+          await Files.write(resolvedPath, activeTab.content);
+          updateTab(activeTabId, {
+            dirty: false,
+            filePath: resolvedPath,
+            label: resolvedPath.split(/[/\\]/).pop() ?? resolvedPath,
+          });
+          Files.addRecent(resolvedPath)
+            .then(({ entries }) => setRecentFiles(entries))
+            .catch(console.error);
+        } catch {
+          // No dialog capability in runtime: silently no-op.
+        }
+        return;
+      }
+
+      // In browser mode, use the native file save picker when available.
+      const maybePicker = (window as Window & {
+        showSaveFilePicker?: (options?: {
+          suggestedName?: string;
+          types?: Array<{ description?: string; accept: Record<string, string[]> }>;
+        }) => Promise<{
+          createWritable: () => Promise<{
+            write: (data: string) => Promise<void>;
+            close: () => Promise<void>;
+          }>;
+          name?: string;
+        }>;
+      }).showSaveFilePicker;
+
+      if (!maybePicker) return;
+
+      try {
+        const handle = await maybePicker({
+          suggestedName,
+          types: [
+            {
+              description: "Markdown files",
+              accept: {
+                "text/markdown": [".md", ".markdown"],
+                "text/plain": [".txt"],
+              },
+            },
+          ],
+        });
+        const writable = await handle.createWritable();
+        await writable.write(activeTab.content);
+        await writable.close();
+        updateTab(activeTabId, { dirty: false, label: handle.name || suggestedName });
+      } catch {
+        // No picker capability (or picker cancelled): silently no-op.
+      }
     },
     [activeTab, activeTabId, updateTab]
   );