[C2] Add ProjectManager.update_document_content + editor save

Author: inliver233

issues/2026-02-10_18-23-04-ui-autosave-streaming-v1.csv

@@ -1,6 +1,6 @@
 ID,Title,Description,Acceptance,Test_Method,Tools,Dev_Status,Review1_Status,Regression_Status,Files,Dependencies,Notes
 C1,Unify bottom status UI (remove duplicate editor status frame),"Remove EditorPanel internal bottom status frame to avoid double bottom bars and overlapping lines; ensure stats/cursor/doc status still update via main EnhancedStatusBar; adjust layout/styling if needed; update Qt layout test accordingly.","Only one bottom status area is shown; no overlapping/stacked lines; status info still updates; Qt layout test passes.","python -m pytest -q tests/test_status_bar_layout.py",none,DONE,DONE,TODO,"src/gui/editor/editor_panel.py | src/gui/status/status_bar.py | tests/test_status_bar_layout.py",none,"done_at:2026-02-10 | test:.tmp\\ane0305-venv-311\\Scripts\\python.exe -m pytest -q tests/test_status_bar_layout.py"
-C2,Add ProjectManager.update_document_content + editor save,"Implement ProjectManager.update_document_content wrapper (persist content via update_document); update IntelligentTextEditor manual save and autosave paths to persist when bound; add unit test covering update_document_content behavior and ensuring no AttributeError/crash.","Autosave/manual save persist content via ProjectManager.update_document_content when project/doc bound; unit test passes.","python -m pytest -q tests/test_project_update_document_content.py",none,TODO,TODO,TODO,"src/core/project.py | src/gui/editor/text_editor.py | tests/test_project_update_document_content.py",none,none
+C2,Add ProjectManager.update_document_content + editor save,"Implement ProjectManager.update_document_content wrapper (persist content via update_document); update IntelligentTextEditor manual save and autosave paths to persist when bound; add unit test covering update_document_content behavior and ensuring no AttributeError/crash.","Autosave/manual save persist content via ProjectManager.update_document_content when project/doc bound; unit test passes.","python -m pytest -q tests/test_project_update_document_content.py",none,DONE,DONE,TODO,"src/core/project.py | src/gui/editor/text_editor.py | tests/test_project_update_document_content.py",none,"done_at:2026-02-10 | test:.tmp\\ane0305-venv-311\\Scripts\\python.exe -m pytest -q tests/test_project_update_document_content.py"
 C3,Bind EditorPanel documents to ProjectManager (autosave works),"Add EditorPanel.set_project_manager and ensure all editors created for project documents are bound to the active ProjectManager + correct document_id; update MainWindow wiring so selecting a document loads/updates via ProjectManager and manual save works; avoid users typing into an unbound scratch doc when a project is open; add integration test.","Editing a project document triggers autosave and persists into project DB; reopening project shows saved content; manual save works; integration test passes.","python -m pytest -q tests/test_editor_project_persistence.py",none,TODO,TODO,TODO,"src/gui/editor/editor_panel.py | src/gui/main_window_parts/ui.py | src/gui/main_window_parts/integrations.py | tests/test_editor_project_persistence.py",C2,none
 C4,Restore last session (reopen recent project & last document),"Persist and restore last-open project/document when enabled: wire SettingsDialog restore_session/auto_save prefs into Config; on startup, if enabled, auto-open most recent project and re-open last document (fallback to first scene).","Restart restores the previous project and last document content instead of showing a blank scratch doc (when restore_session enabled).",manual,none,TODO,TODO,TODO,"src/main.py | src/core/config_schema.py | src/gui/dialogs/settings_dialog.py | src/core/config.py | src/gui/main_window_parts/ui.py | src/gui/main_window_parts/integrations.py",C3,"manual_checklist: 1) Create/open a project, edit a scene, wait for autosave, close app. 2) Reopen app; expect project auto-open and edited content present. 3) Disable restore_session in settings; restart; expect no auto-open."
 C5,Streaming dispatch respects stream_response toggle,"Plumb config ai.stream_response (default on) through AICompletionService/AIRequestDispatcher to use QtAIClient.complete_stream_async; include context/request_id in chunk signals; ensure cancellation prevents stale chunks from updating UI; add unit test for dispatch selection.","When stream_response enabled, streaming client method is used and chunk signals are emitted; when disabled, non-stream method used; unit test passes.","python -m pytest -q tests/test_streaming_dispatcher.py",none,TODO,TODO,TODO,"src/application/ai_completion_service.py | src/core/ai_qt_client.py | src/gui/ai/enhanced_ai_manager.py | src/gui/main_window_parts/integrations.py | tests/test_streaming_dispatcher.py",none,none

src/core/project.py

@@ -441,6 +441,14 @@ def update_document(self, doc_id: str, save: bool = True, **kwargs) -> Optional[
             logger.info(f"Document updated: {doc.name}")
         return doc
 
+    def update_document_content(self, doc_id: str, content: str, save: bool = True) -> bool:
+        """更新文档内容（便捷方法，供编辑器自动保存/手动保存使用）"""
+        if not self._current_project or doc_id not in self._current_project.documents:
+            return False
+
+        doc = self.update_document(doc_id, save=save, content=content)
+        return doc is not None
+
     def get_document(self, doc_id: str) -> Optional[ProjectDocument]:
         if self._current_project:
             return self._current_project.documents.get(doc_id)

src/gui/editor/text_editor.py

@@ -886,14 +886,29 @@ def is_modified(self) -> bool:
         """检查是否已修改"""
         return self._is_modified
 
-    def save_document(self):
-        """保存文档"""
-        if self._is_modified:
-            content = self.toPlainText()
-            self._last_save_content = content
-            self._is_modified = False
-            self.autoSaveTriggered.emit(content)
-            logger.info("Document saved manually")
+    def save_document(self) -> bool:
+        """保存文档（优先持久化到项目数据库）。"""
+        if not self._is_modified:
+            return True
+
+        content = self.toPlainText()
+
+        # If bound to a project document, persist via ProjectManager.
+        if self._current_document_id and self._project_manager:
+            try:
+                success = bool(self._project_manager.update_document_content(self._current_document_id, content))
+            except Exception as e:  # noqa: BLE001
+                logger.warning(f"Failed to save via project manager: {e}")
+                success = False
+
+            if not success:
+                return False
+
+        self._last_save_content = content
+        self._is_modified = False
+        self.autoSaveTriggered.emit(content)
+        logger.info("Document saved manually")
+        return True
 
     def insert_text_at_cursor(self, text: str):
         """在光标位置插入文本"""

tests/test_project_update_document_content.py

@@ -0,0 +1,70 @@
+from __future__ import annotations
+
+import os
+from pathlib import Path
+
+import pytest
+
+os.environ.setdefault("QT_QPA_PLATFORM", "offscreen")
+
+pytest.importorskip("PyQt6")
+pytest.importorskip("pytestqt")
+
+from core.config import Config
+from core.project import ProjectManager
+from core.shared import Shared
+from gui.editor.text_editor import IntelligentTextEditor
+
+
+def test_project_manager_update_document_content_persists(tmp_path: Path) -> None:
+    config = Config()
+    shared = Shared(config)
+    pm = ProjectManager(config=config, shared=shared)
+
+    project_dir = tmp_path / "proj"
+    assert pm.create_project("Proj", str(project_dir)) is True
+    assert pm._current_project is not None
+    assert pm._current_project.documents
+
+    doc_id = next(iter(pm._current_project.documents.keys()))
+
+    assert pm.update_document_content(doc_id, "hello") is True
+    assert pm.get_document_content(doc_id) == "hello"
+
+    pm.close_project()
+    assert pm.open_project(str(project_dir)) is True
+    assert pm.get_document_content(doc_id) == "hello"
+
+
+class _DummyProjectManager:
+    def __init__(self) -> None:
+        self.calls: list[tuple[str, str]] = []
+
+    def update_document_content(self, doc_id: str, content: str) -> bool:  # noqa: D401
+        self.calls.append((doc_id, content))
+        return True
+
+
+def test_text_editor_save_and_autosave_use_project_manager(qtbot) -> None:
+    config = Config()
+    shared = Shared(config)
+    editor = IntelligentTextEditor(config=config, shared=shared)
+    qtbot.addWidget(editor)
+
+    dummy_pm = _DummyProjectManager()
+    editor.set_project_manager(dummy_pm)
+    editor.set_document_content("start", "doc1")
+
+    editor.setPlainText("changed")
+    assert editor.is_modified() is True
+
+    assert editor.save_document() is True
+    assert dummy_pm.calls == [("doc1", "changed")]
+    assert editor.is_modified() is False
+
+    editor.setPlainText("changed again")
+    assert editor.is_modified() is True
+
+    editor._trigger_auto_save()
+    assert dummy_pm.calls[-1] == ("doc1", "changed again")
+    assert editor.is_modified() is False