update

Author: petertzy

README.MD

@@ -14,9 +14,12 @@ Planning notes: [scripts/ProjectPlan.md](scripts/ProjectPlan.md)
 
 ### Development mode
 
-- Frontend: Next.js on `http://localhost:3000`
-- Backend: FastAPI on `http://127.0.0.1:8000`
-- Recommended start command: `./scripts/dev.sh`
+- Launches a real Tauri desktop window instead of a browser tab
+- Frontend still runs from the Next.js dev server on `http://localhost:3000`
+- Backend runs from the FastAPI dev server on `http://127.0.0.1:8000`
+- Recommended start command: `./scripts/dev-tauri.sh`
+- In this mode, the Tauri shell talks to the fixed dev backend on port `8000`
+- The packaged Python sidecar is not used in debug mode
 
 ### Desktop (packaged) mode
 
@@ -36,7 +39,7 @@ MarkdownReader/
 ├── backend/                    # FastAPI app and routers
 ├── frontend/                   # Next.js + Tauri desktop shell
 ├── markdown_reader/            # Existing Python business logic (preserved)
-├── scripts/dev.sh              # One-command dev startup
+├── scripts/dev-tauri.sh        # Recommended native dev startup
 ├── tests/                      # Unit/integration tests
 ├── README_OLD.MD               # Legacy tkinter-era documentation
 └── pyproject.toml
@@ -76,17 +79,17 @@ cp frontend/.env.local.example frontend/.env.local
 ### 3) Start services
 
 ```bash
-./scripts/dev.sh
+./scripts/dev-tauri.sh
 ```
 
 Open:
 
-- Frontend: http://localhost:3000
 - Backend docs: http://127.0.0.1:8000/docs
+- Tauri native app window: launched automatically by `./scripts/dev-tauri.sh`
 
 ### 4) Stop services
 
-Press `Ctrl+C` in the terminal running `./scripts/dev.sh`.
+Press `Ctrl+C` in the terminal running `./scripts/dev-tauri.sh`.
 
 ---
 

frontend/package.json

@@ -4,6 +4,7 @@
   "private": true,
   "scripts": {
     "dev": "next dev --port 3000",
+    "tauri:dev": "tauri dev",
     "build": "next build",
     "start": "next start --port 3000",
     "lint": "next lint"

frontend/src-tauri/src/lib.rs

@@ -24,14 +24,17 @@ pub fn run() {
         .invoke_handler(tauri::generate_handler![get_backend_port])
         .setup(|app| {
             if cfg!(debug_assertions) {
+                *app.state::<BackendPort>().0.lock().unwrap() = Some(8000);
                 app.handle().plugin(
                     tauri_plugin_log::Builder::default()
                         .level(log::LevelFilter::Info)
                         .build(),
-                )?
+                )?;
+
+                return Ok(());
             }
 
-            // Spawn sidecar and capture its stdout to learn which port it picked.
+            // In packaged mode, spawn sidecar and capture its stdout to learn which port it picked.
             let sidecar = app
                 .shell()
                 .sidecar("markdown-reader-backend")

frontend/src/hooks/useEditor.ts

@@ -73,12 +73,22 @@ export function useEditor() {
   // ── content change ─────────────────────────────────────────────────────────
   const handleContentChange = useCallback(
     (value: string | undefined) => {
-      const v = value ?? "";
-      if (v === activeTab.content) return;
-      updateTab(activeTabId, { content: v, dirty: true });
-      refreshPreview(v);
+      if (typeof value !== "string") return;
+      const v = value;
+      let changed = false;
+
+      setTabs((prev) =>
+        prev.map((t) => {
+          if (t.id !== activeTabId) return t;
+          if (t.content === v) return t;
+          changed = true;
+          return { ...t, content: v, dirty: true };
+        })
+      );
+
+      if (changed) refreshPreview(v);
     },
-    [activeTab, activeTabId, updateTab, refreshPreview]
+    [activeTabId, refreshPreview]
   );
 
   // ── file ops ───────────────────────────────────────────────────────────────
@@ -129,27 +139,7 @@ export function useEditor() {
 
   const saveFile = useCallback(
     async (filePath?: string) => {
-      let path = filePath ?? activeTab.filePath;
-
-      // For tabs opened without absolute path, try to resolve an existing path
-      // without opening any dialog.
-      if (!path) {
-        const byName = recentFiles.filter((p) => (p.split(/[/\\]/).pop() ?? "") === activeTab.label);
-        if (byName.length === 1) {
-          path = byName[0];
-        } else {
-          const candidates = [activeTab.label, `./${activeTab.label}`];
-          for (const candidate of candidates) {
-            try {
-              await Files.read(candidate);
-              path = candidate;
-              break;
-            } catch {
-              // Keep trying other candidates.
-            }
-          }
-        }
-      }
+      const path = filePath ?? activeTab.filePath;
 
       if (path) {
         await Files.write(path, activeTab.content);
@@ -164,40 +154,10 @@ export function useEditor() {
         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`;
-
-      // Try Tauri native save dialog first (desktop mode).
-      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,
-          browserHandle: null,
-          label: resolvedPath.split(/[/\\]/).pop() ?? resolvedPath,
-        });
-        Files.addRecent(resolvedPath)
-          .then(({ entries }) => setRecentFiles(entries))
-          .catch(console.error);
-        return;
-      } catch {
-        // Not running with Tauri dialog capability.
-      }
-
-      // Outside Tauri dialog capability, do nothing silently.
+      // No explicit path available: do nothing silently.
       return;
     },
-    [activeTab, activeTabId, recentFiles, updateTab]
+    [activeTab, activeTabId, updateTab]
   );
 
   const newTab = useCallback(() => {

scripts/dev-tauri.sh

@@ -0,0 +1,61 @@
+#!/usr/bin/env bash
+# dev-tauri.sh — Start FastAPI backend and Tauri desktop shell in development mode.
+
+set -euo pipefail
+
+ROOT="$(cd "$(dirname "$0")/.." && pwd)"
+PYTHON="${ROOT}/.venv/bin/python"
+FRONTEND="${ROOT}/frontend"
+BACKEND_URL="http://127.0.0.1:8000"
+PACKAGED_SIDECAR_PATTERN="Markdown Reader.app/Contents/MacOS/markdown-reader-backend"
+
+BACKEND_PID=""
+
+is_port_in_use() {
+  local port="$1"
+  lsof -nP -iTCP:"${port}" -sTCP:LISTEN >/dev/null 2>&1
+}
+
+cleanup() {
+  echo ""
+  echo "⏹ Stopping…"
+  if [ -n "${BACKEND_PID}" ]; then
+    kill "${BACKEND_PID}" 2>/dev/null || true
+  fi
+}
+
+trap cleanup SIGINT SIGTERM EXIT
+
+if pgrep -f "${PACKAGED_SIDECAR_PATTERN}" >/dev/null 2>&1; then
+  echo "▶ Found packaged sidecar backend process; stopping it for Tauri dev mode…"
+  pkill -f "${PACKAGED_SIDECAR_PATTERN}" || true
+fi
+
+if is_port_in_use 8000; then
+  if curl -fsS "${BACKEND_URL}/api/health" >/dev/null 2>&1; then
+    echo "▶ Reusing existing FastAPI backend on ${BACKEND_URL}"
+  else
+    echo "ERROR: Port 8000 is already in use, but it does not look like this app's backend."
+    echo "Please stop the process using port 8000, then run ./scripts/dev-tauri.sh again."
+    exit 1
+  fi
+else
+  echo "▶ Starting FastAPI backend on ${BACKEND_URL} …"
+  cd "$ROOT"
+  "$PYTHON" -m uvicorn backend.main:app --host 127.0.0.1 --port 8000 --reload &
+  BACKEND_PID=$!
+  echo "  Backend PID: $BACKEND_PID"
+  if ! kill -0 "$BACKEND_PID" 2>/dev/null; then
+    echo "ERROR: Backend failed to start."
+    exit 1
+  fi
+fi
+
+echo ""
+echo "▶ Launching Tauri desktop shell in development mode …"
+echo "   Frontend dev server will be started by Tauri via frontend/src-tauri/tauri.conf.json"
+echo "   Backend is fixed at ${BACKEND_URL} in debug mode"
+echo ""
+
+cd "$FRONTEND"
+npx tauri dev