refactor: move sector phase persistence to heaven repository

app/ARCHITECTURE.md

@@ -30,8 +30,9 @@ background scheduler
   `backend/application.py` and `backend/bootstrap/`.
 - `backend/bootstrap/` owns process configuration, dependency construction, startup, and
   shared input/display-format contracts. It does not own feature behavior.
-- `backend/http/` owns common authentication, request IDs, responses, static delivery, and
+- `backend/http/` owns common authentication, request IDs, JSON/NDJSON responses, static
-  error normalization. Feature-specific transport handlers live beside their feature.
+  delivery, streaming connection lifecycle, and error normalization. Feature-specific
+  transport handlers live beside their feature.
   Exact POST endpoints that only delegate to one of those handlers use the explicit maps in
   `backend/application.py`; endpoints with path parameters, body handling, or special error
   semantics remain visible control flow in `RequestHandler`.

app/backend/application.py

@@ -472,38 +472,6 @@ class DashboardService(
             **self.database.status(),
         }
 
-
-
-
-
-
-    @staticmethod
-    def _ifind_field(row: dict[str, Any], tokens: tuple[str, ...]) -> Any:
-        for key, value in row.items():
-            label = str(key or "")
-            if any(token.casefold() == label.casefold() for token in tokens):
-                return value
-        for key, value in row.items():
-            label = str(key or "")
-            if any(token in label for token in tokens):
-                return value
-        return None
-
-    @classmethod
-    def _ifind_row_code(cls, row: dict[str, Any]) -> str:
-        value = cls._ifind_field(row, ("股票代码", "证券代码", "代码", "thscode"))
-        match = re.search(r"(?<!\d)(\d{6})(?!\d)", str(value or ""))
-        if match:
-            return match.group(1)
-        for value in row.values():
-            match = re.search(r"(?<!\d)(\d{6})\.(?:SH|SZ|BJ)(?![A-Z])", str(value or ""), re.I)
-            if match:
-                return match.group(1)
-        return ""
-
-
-
-
 SERVICE = DashboardService()
 
 
@@ -1020,16 +988,6 @@ class RequestHandler(
             return
         self.send_json({"error": "Not found"}, HTTPStatus.NOT_FOUND)
 
-
-
-    def _write_stream_event(self, payload: dict[str, Any]) -> None:
-        self.wfile.write(
-            (json.dumps(payload, ensure_ascii=False, separators=(",", ":")) + "\n").encode("utf-8")
-        )
-        self.wfile.flush()
-
-
-
     def save_reason(self) -> None:
         try:
             body = self.read_json_body()

app/backend/features/heaven/repository.py

@@ -7,6 +7,35 @@ from typing import Any
 
 
 class HeavenRepositoryMixin:
+    def list_sector_phase_overrides(self) -> dict[str, str]:
+        with self.connect() as connection:
+            rows = connection.execute(
+                "SELECT name, element FROM sector_phase_overrides ORDER BY updated_at DESC, name"
+            ).fetchall()
+        return {row["name"]: row["element"] for row in rows}
+
+    def save_sector_phase_override(self, name: str, element: str) -> None:
+        now = datetime.now().astimezone().isoformat(timespec="seconds")
+        with self.connect() as connection:
+            connection.execute(
+                """
+                INSERT INTO sector_phase_overrides (name, element, updated_at)
+                VALUES (?, ?, ?)
+                ON CONFLICT(name) DO UPDATE SET
+                    element = excluded.element,
+                    updated_at = excluded.updated_at
+                """,
+                (name, element, now),
+            )
+
+    def delete_sector_phase_override(self, name: str) -> bool:
+        with self.connect() as connection:
+            cursor = connection.execute(
+                "DELETE FROM sector_phase_overrides WHERE name = ?",
+                (name,),
+            )
+            return cursor.rowcount > 0
+
     @staticmethod
     def _heaven_reading_dict(row: sqlite3.Row | None) -> dict[str, Any] | None:
         if not row:

app/backend/features/mentor/http.py

@@ -14,19 +14,4 @@ class MentorHttpMixin:
         except (ValueError, json.JSONDecodeError) as exc:
             self.send_json({"error": str(exc)}, HTTPStatus.BAD_REQUEST)
             return
-        self.send_response(HTTPStatus.OK)
+        self.send_ndjson_stream(stream, (ValueError, MentorAgentError))
-        self.send_header("Content-Type", "application/x-ndjson; charset=utf-8")
-        self.send_header("Cache-Control", "no-cache, no-transform")
-        self.send_header("X-Accel-Buffering", "no")
-        self.send_header("Connection", "close")
-        self.end_headers()
-        try:
-            for event in stream:
-                self._write_stream_event(event)
-            self._write_stream_event({"type": "done"})
-        except (ValueError, MentorAgentError) as exc:
-            self._write_stream_event({"type": "error", "error": str(exc)})
-        except (BrokenPipeError, ConnectionResetError):
-            pass
-        finally:
-            self.close_connection = True

app/backend/features/pools/service.py

@@ -117,6 +117,30 @@ class PoolServiceMixin:
         finally:
             self._ifind_event_lock.release()
 
+    @staticmethod
+    def _ifind_field(row: dict[str, Any], tokens: tuple[str, ...]) -> Any:
+        for key, value in row.items():
+            label = str(key or "")
+            if any(token.casefold() == label.casefold() for token in tokens):
+                return value
+        for key, value in row.items():
+            label = str(key or "")
+            if any(token in label for token in tokens):
+                return value
+        return None
+
+    @classmethod
+    def _ifind_row_code(cls, row: dict[str, Any]) -> str:
+        value = cls._ifind_field(row, ("股票代码", "证券代码", "代码", "thscode"))
+        match = re.search(r"(?<!\d)(\d{6})(?!\d)", str(value or ""))
+        if match:
+            return match.group(1)
+        for value in row.values():
+            match = re.search(r"(?<!\d)(\d{6})\.(?:SH|SZ|BJ)(?![A-Z])", str(value or ""), re.I)
+            if match:
+                return match.group(1)
+        return ""
+
     @staticmethod
     def _normalize_ifind_event_time(value: Any) -> str:
         text = str(value or "").strip()

app/backend/features/review/http.py

@@ -26,22 +26,8 @@ class ReviewHttpMixin:
         except (ValueError, json.JSONDecodeError) as exc:
             self.send_json({"error": str(exc)}, HTTPStatus.BAD_REQUEST)
             return
-        self.send_response(HTTPStatus.OK)
+        events = ({"type": "delta", "content": chunk} for chunk in stream)
-        self.send_header("Content-Type", "application/x-ndjson; charset=utf-8")
+        self.send_ndjson_stream(events, (ValueError, ReviewAssistantError))
-        self.send_header("Cache-Control", "no-cache, no-transform")
-        self.send_header("X-Accel-Buffering", "no")
-        self.send_header("Connection", "close")
-        self.end_headers()
-        try:
-            for chunk in stream:
-                self._write_stream_event({"type": "delta", "content": chunk})
-            self._write_stream_event({"type": "done"})
-        except (ValueError, ReviewAssistantError) as exc:
-            self._write_stream_event({"type": "error", "error": str(exc)})
-        except (BrokenPipeError, ConnectionResetError):
-            pass
-        finally:
-            self.close_connection = True
 
     def save_watchlist(self) -> None:
         try:

app/backend/http/handler.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 import json
 import mimetypes
 import secrets
+from collections.abc import Iterable
 from http import HTTPStatus
 from http.cookies import SimpleCookie
 from typing import Any
@@ -142,5 +143,33 @@ class HttpTransportMixin:
         self.end_headers()
         self.wfile.write(content)
 
+    def _write_stream_event(self, payload: dict[str, Any]) -> None:
+        self.wfile.write(
+            (json.dumps(payload, ensure_ascii=False, separators=(",", ":")) + "\n").encode("utf-8")
+        )
+        self.wfile.flush()
+
+    def send_ndjson_stream(
+        self,
+        events: Iterable[dict[str, Any]],
+        error_types: tuple[type[Exception], ...],
+    ) -> None:
+        self.send_response(HTTPStatus.OK)
+        self.send_header("Content-Type", "application/x-ndjson; charset=utf-8")
+        self.send_header("Cache-Control", "no-cache, no-transform")
+        self.send_header("X-Accel-Buffering", "no")
+        self.send_header("Connection", "close")
+        self.end_headers()
+        try:
+            for event in events:
+                self._write_stream_event(event)
+            self._write_stream_event({"type": "done"})
+        except error_types as exc:
+            self._write_stream_event({"type": "error", "error": str(exc)})
+        except (BrokenPipeError, ConnectionResetError):
+            pass
+        finally:
+            self.close_connection = True
+
     def log_message(self, format_string: str, *args: Any) -> None:
         print(f"[{self.log_date_time_string()}] {format_string % args}")

app/config/architecture-inventory.json

@@ -287,6 +287,16 @@
       "path": "backend/llm/transport.py"
     }
   ],
+  "http_transport": [
+    {
+      "function": "send_json",
+      "path": "backend/http/handler.py"
+    },
+    {
+      "function": "send_ndjson_stream",
+      "path": "backend/http/handler.py"
+    }
+  ],
   "css_layers": [
     "/shared/tokens.css?v=20260729-1",
     "/styles/styles.css",
@@ -364,8 +374,8 @@
     },
     {
       "path": "backend/application.py",
-      "bytes": 48749,
+      "bytes": 47552,
-      "lines": 1129
+      "lines": 1087
     },
     {
       "path": "frontend/styles/theme.css",
@@ -374,8 +384,8 @@
     },
     {
       "path": "database.py",
-      "bytes": 33284,
+      "bytes": 32073,
-      "lines": 746
+      "lines": 716
     }
   ]
 }

app/database.py

@@ -2,7 +2,7 @@ from __future__ import annotations
 
 import json
 import sqlite3
-from datetime import datetime, timezone
+from datetime import datetime
 from pathlib import Path
 from typing import Any
 
@@ -665,36 +665,6 @@ class ReviewDatabase(
             )
             MigrationRunner().apply(connection, MIGRATIONS)
 
-
-
-    def list_sector_phase_overrides(self) -> dict[str, str]:
-        with self.connect() as connection:
-            rows = connection.execute(
-                "SELECT name, element FROM sector_phase_overrides ORDER BY updated_at DESC, name"
-            ).fetchall()
-        return {row["name"]: row["element"] for row in rows}
-
-    def save_sector_phase_override(self, name: str, element: str) -> None:
-        now = datetime.now().astimezone().isoformat(timespec="seconds")
-        with self.connect() as connection:
-            connection.execute(
-                """
-                INSERT INTO sector_phase_overrides (name, element, updated_at)
-                VALUES (?, ?, ?)
-                ON CONFLICT(name) DO UPDATE SET
-                    element = excluded.element,
-                    updated_at = excluded.updated_at
-                """,
-                (name, element, now),
-            )
-
-    def delete_sector_phase_override(self, name: str) -> bool:
-        with self.connect() as connection:
-            cursor = connection.execute(
-                "DELETE FROM sector_phase_overrides WHERE name = ?",
-                (name,),
-            )
-            return cursor.rowcount > 0
     def list_wencai_saved_queries(
         self, user_id: int, limit: int = 30
     ) -> list[dict[str, Any]]:

app/tests/test_http_streaming.py

@@ -0,0 +1,85 @@
+from __future__ import annotations
+
+import io
+import json
+import unittest
+
+from backend.http.handler import HttpTransportMixin
+
+
+class StreamError(RuntimeError):
+    pass
+
+
+class TransportStub(HttpTransportMixin):
+    def __init__(self) -> None:
+        self.statuses: list[int] = []
+        self.response_headers: list[tuple[str, str]] = []
+        self.wfile = io.BytesIO()
+        self.close_connection = False
+
+    def send_response(self, status: int) -> None:
+        self.statuses.append(int(status))
+
+    def send_header(self, name: str, value: str) -> None:
+        self.response_headers.append((name, value))
+
+    def end_headers(self) -> None:
+        pass
+
+
+class HttpStreamingTests(unittest.TestCase):
+    def test_stream_transport_preserves_headers_events_and_completion(self) -> None:
+        handler = TransportStub()
+        handler.send_ndjson_stream(
+            ({"type": "delta", "content": value} for value in ("甲", "乙")),
+            (StreamError,),
+        )
+
+        self.assertEqual(handler.statuses, [200])
+        self.assertEqual(
+            dict(handler.response_headers),
+            {
+                "Content-Type": "application/x-ndjson; charset=utf-8",
+                "Cache-Control": "no-cache, no-transform",
+                "X-Accel-Buffering": "no",
+                "Connection": "close",
+            },
+        )
+        events = [
+            json.loads(line)
+            for line in handler.wfile.getvalue().decode("utf-8").splitlines()
+        ]
+        self.assertEqual(
+            events,
+            [
+                {"type": "delta", "content": "甲"},
+                {"type": "delta", "content": "乙"},
+                {"type": "done"},
+            ],
+        )
+        self.assertTrue(handler.close_connection)
+
+    def test_stream_transport_preserves_feature_error_event(self) -> None:
+        def events():
+            yield {"type": "delta", "content": "partial"}
+            raise StreamError("stream failed")
+
+        handler = TransportStub()
+        handler.send_ndjson_stream(events(), (StreamError,))
+        payloads = [
+            json.loads(line)
+            for line in handler.wfile.getvalue().decode("utf-8").splitlines()
+        ]
+        self.assertEqual(
+            payloads,
+            [
+                {"type": "delta", "content": "partial"},
+                {"type": "error", "error": "stream failed"},
+            ],
+        )
+        self.assertTrue(handler.close_connection)
+
+
+if __name__ == "__main__":
+    unittest.main()

app/tests/test_preservation_slice_accounts.py

@@ -40,6 +40,8 @@ class AccountSliceStructureTests(unittest.TestCase):
             "require_access",
             "serve_static",
             "send_json",
+            "_write_stream_event",
+            "send_ndjson_stream",
         ):
             self.assertNotIn(method, RequestHandler.__dict__)
             self.assertIn(method, HttpTransportMixin.__dict__)

app/tests/test_preservation_slice_heaven.py

@@ -38,6 +38,9 @@ HEAVEN_SERVICE_METHODS = {
 }
 
 HEAVEN_REPOSITORY_METHODS = {
+    "list_sector_phase_overrides",
+    "save_sector_phase_override",
+    "delete_sector_phase_override",
     "_heaven_reading_dict",
     "save_heaven_reading",
     "list_heaven_readings",

app/tests/test_preservation_slice_sentiment_pools.py

@@ -29,6 +29,8 @@ POOL_METHODS = {
     "_apply_reason_overrides",
     "_schedule_ifind_event_enrichment",
     "_refresh_ifind_event_enrichment",
+    "_ifind_field",
+    "_ifind_row_code",
     "_normalize_ifind_event_time",
     "_merge_ifind_event_enrichment",
 }

app/tools/build_architecture_inventory.py

@@ -179,6 +179,10 @@ def build() -> dict[str, Any]:
             {"function": "chat_completion", "path": "backend/llm/transport.py"},
             {"function": "stream_chat_completion", "path": "backend/llm/transport.py"},
         ],
+        "http_transport": [
+            {"function": "send_json", "path": "backend/http/handler.py"},
+            {"function": "send_ndjson_stream", "path": "backend/http/handler.py"},
+        ],
         "css_layers": css_layers(html),
         "code_hotspots": code_hotspots(),
     }

docs/governance/code-reduction.md

@@ -25,6 +25,9 @@
 | CR-05 | 根级兼容入口 | 正式后端仍有五处通过迁移兼容模块反向导入规范实现 | 正式代码改用规范路径；兼容入口只服务原公开导入契约 | 已完成 |
 | CR-06 | 紧凑日期显示 | Tushare与选股引擎各保留一份完全相同的`YYYYMMDD`显示转换 | 由`bootstrap/config.py`拥有唯一格式策略，消费者保留原局部别名 | 已完成 |
 | CR-07 | 数据Provider组装 | 核查网关、容器和业务服务是否重复创建外部数据客户端 | 固化唯一创建位置及兼容例外，不改数据源语义 | 已完成 |
+| CR-08 | NDJSON流式传输 | 问师与复盘助手重复维护响应头、事件写入、完成、断线及关闭流程 | HTTP共享层拥有唯一流式连接生命周期 | 已完成 |
+| CR-09 | 应用服务门面 | 两个仅服务股池iFinD补全的方法仍错位在全局`DashboardService` | 原函数体机械归位到`PoolServiceMixin` | 已完成 |
+| CR-10 | Repository所有权 | 五行行业阶段覆盖的三项持久化方法仍错位在根级数据库门面 | 原函数体机械归位到问天Repository，兼容数据继续保留 | 已完成 |
 
 ## CR-01验收口径
 
@@ -185,6 +188,65 @@
 本批基线为`xiaobai-reduction-06-date-formatting-20260801`；检查点为
 `xiaobai-reduction-07-provider-ownership-20260801`。
 
+## CR-08验收口径
+
+- 只合并NDJSON响应头、事件序列化、完成事件、业务错误事件、客户端断线和连接关闭这些传输行为。
+- 问师继续直接输出原事件字典；复盘助手继续把文本分片包装为`delta/content`事件。
+- 两个功能各自的业务异常类型、请求体错误状态码、错误正文和流创建时机保持不变。
+- `_write_stream_event`与连接生命周期必须只由`backend/http/handler.py`拥有，应用大类和功能HTTP
+  模块不再保留第二份实现。
+
+## CR-08结果
+
+- 删除问师和复盘助手各16行重复流式控制流，并将应用大类中的10行事件写入方法归入HTTP共享层；
+  新共享实现29行、两个调用适配共3行，生产代码净减少10行。
+- 新增专项测试固定四个响应头、中文NDJSON序列化、增量顺序、完成事件、业务错误事件和关闭状态。
+- 候选324项、纯`app/`导出261项及45项Playwright通过；24个JavaScript文件、API/架构注册表、
+  Git空白检查和SQLite完整性检查通过。
+- 本批不修改提示词、模型选择、会员计次、流式正文、前端解析、API路径、数据库或数据源。
+
+本批基线为`xiaobai-reduction-07-provider-ownership-20260801`；检查点为
+`xiaobai-reduction-08-ndjson-transport-20260801`。
+
+## CR-09验收口径
+
+- 只有消费者全部属于单一领域、且能够按原函数体机械移动的方法才从应用门面移出。
+- `_ifind_field`与`_ifind_row_code`继续保持静态/类方法签名、字段优先级、大小写规则和代码正则。
+- 账号委托属于稳定公开门面；系统设置属于跨领域协调；后台刷新留到CR-11，本批均不得删除或重写。
+- 移动后`DashboardService`必须继续通过Mixin解析同名方法，调用点和返回值不变。
+
+## CR-09结果
+
+- 将iFinD字段匹配和股票代码提取两个方法从应用大类机械移动到股池服务，原版与迁移方法AST逐项
+  相等；应用大类不再直接拥有股池专属实现。
+- 连同迁移期遗留空行，`backend/application.py`减少32行，股池服务增加24行，生产代码净减少8行。
+- 候选324项、纯`app/`导出261项、24个JavaScript文件、API/架构注册表、Git空白检查和SQLite
+  完整性检查通过；本批不涉及页面、CSS或浏览器行为。
+- 本批不修改字段匹配、涨跌停原因补全、接口、数据源、缓存、数据库、权限或前端。
+
+本批基线为`xiaobai-reduction-08-ndjson-transport-20260801`；检查点为
+`xiaobai-reduction-09-service-facade-20260801`。
+
+## CR-10验收口径
+
+- 只移动调用方、数据表和业务含义均明确属于单一领域的方法；数据库连接、事务和返回值必须保持不变。
+- `list_sector_phase_overrides`、`save_sector_phase_override`与`delete_sector_phase_override`必须由
+  `backend/features/heaven/repository.py`拥有，并继续通过`ReviewDatabase`的Mixin解析。
+- 不修改表结构、迁移顺序、时间格式、排序、冲突更新或删除结果语义。
+- `wencai_saved_queries`及其三个方法属于已登记的账户隔离兼容数据；即使前端入口已取消，也必须保留。
+
+## CR-10结果
+
+- 将五行行业阶段覆盖的查询、保存和删除三个方法从根级`database.py`机械移动到问天Repository；
+  调用名称、SQL、事务边界、时间值和返回结果均未改变。
+- 根级数据库门面不再直接拥有问天领域的持久化实现，问财历史兼容表和方法完整保留，未扩大删除范围。
+- 34项Repository、问天、账户隔离、清理契约及迁移定向测试通过；候选324项、纯`app/`导出261项、
+  24个JavaScript文件、API/架构注册表、Git空白检查和SQLite完整性检查通过。
+- 本批不修改页面、CSS、API、数据库结构、行情、数据源、业务计算、LLM、权限或后台任务。
+
+本批基线为`xiaobai-reduction-09-service-facade-20260801`；检查点为
+`xiaobai-reduction-10-repository-ownership-20260801`。
+
 ## 人工验收记录
 
 - 2026-08-01：用户检查CR-02与CR-03运行结果，确认未发现明显异常。本记录仅表示本轮可见功能与