test: enforce provider construction ownership

app/ARCHITECTURE.md

@@ -28,6 +28,8 @@ background scheduler
 
 - `server.py` is the stable command/import facade. Runtime composition lives in
   `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
   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
@@ -36,7 +38,7 @@ background scheduler
 - `backend/features/<feature>/` owns the mechanically moved service, repository, HTTP, agent,
   or deterministic calculation code for that product area.
 - `backend/data/` owns provider construction, source policy, provenance, units, freshness,
-  coverage, and display-versus-calculation eligibility.
+  coverage, display-versus-calculation eligibility, and shared numeric normalization policies.
 - `backend/database/` owns connection management, ordered migrations, and narrow repository
   adapters. Root `database.py` remains the legacy schema/composition anchor and combines the
   feature repository mixins; do not add feature queries to it.
@@ -55,7 +57,11 @@ background scheduler
 
 Root modules such as `screener.py`, `tushare_client.py`, and `mentor_agent.py` are compatibility
 aliases to canonical modules. They contain no second implementation and remain only because
-the original public import surface is part of the preservation contract.
+the original public import surface is part of the preservation contract. Canonical backend
+modules must import other canonical modules directly rather than routing through these aliases.
+The remaining `api_access` import in `backend/application.py` and preserved lazy
+`sentiment_engine` import in the screener repository are registered transition boundaries;
+the root `database.py` remains the documented schema/composition anchor.
 
 ## Non-negotiable maintenance rules
 

app/backend/bootstrap/config.py

@@ -71,6 +71,10 @@ def normalize_date(value: str) -> str:
     return parsed.strftime("%Y%m%d")
 
 
+def display_compact_date(value: str) -> str:
+    return f"{value[:4]}-{value[4:6]}-{value[6:8]}" if len(value) == 8 else value
+
+
 def validate_stock_code(value: str) -> str:
     code = value.strip()
     if not re.fullmatch(r"\d{6}", code):

app/backend/bootstrap/container.py

@@ -9,10 +9,10 @@ from backend.database.repositories import RepositoryBundle, build_repository_bun
 from backend.features.alerts import AlertService
 from backend.features.mentor.agent import MentorSkillRegistry
 from backend.features.review import TradeJournalService
+from backend.features.screener.engine import ScreenerEngine
 from backend.features.screener.tracking import StrategyTrackingService
 from backend.jobs import InProcessJobRunner, JobRegistry, SQLiteJobRunRepository
 from database import ReviewDatabase
-from screener import ScreenerEngine
 from backend.data.providers.ifind_client import IfindHttpClient
 from backend.data.realtime import WebRealtimeAggregator
 from backend.features.market.charts import MarketChartClient

app/backend/data/numbers.py

@@ -0,0 +1,20 @@
+from __future__ import annotations
+
+import math
+from typing import Any
+
+
+def finite_number(value: Any, default: float = 0.0) -> float:
+    try:
+        number = float(value)
+        return number if math.isfinite(number) else default
+    except (TypeError, ValueError):
+        return default
+
+
+def non_nan_number(value: Any, default: float = 0.0) -> float:
+    try:
+        number = float(value)
+        return number if number == number else default
+    except (TypeError, ValueError):
+        return default

app/backend/data/providers/tushare_client.py

@@ -11,6 +11,8 @@ from datetime import datetime, time as dt_time, timedelta
 from threading import Lock
 from typing import Any, ClassVar
 
+from backend.bootstrap.config import display_compact_date as _display_date
+from backend.data.numbers import finite_number as _number
 from backend.features.sentiment.engine import apply_sentiment_to_dashboard
 
 
@@ -1812,14 +1814,6 @@ class TushareClient:
         }
 
 
-def _number(value: Any, default: float = 0.0) -> float:
-    try:
-        number = float(value)
-        return number if math.isfinite(number) else default
-    except (TypeError, ValueError):
-        return default
-
-
 def _text(value: Any) -> str:
     if isinstance(value, (list, tuple, set)):
         return "、".join(str(item).strip() for item in value if str(item).strip())
@@ -2014,10 +2008,6 @@ def _display_time(value: Any) -> str:
     return f"{raw[:2]}:{raw[2:4]}:{raw[4:6]}"
 
 
-def _display_date(value: str) -> str:
-    return f"{value[:4]}-{value[4:6]}-{value[6:8]}" if len(value) == 8 else value
-
-
 def _realtime_market_status(current_time: dt_time) -> str:
     if current_time < dt_time(9, 25):
         return "pre_open"

app/backend/features/market/insights.py

@@ -6,6 +6,7 @@ from datetime import datetime, time as dt_time, timedelta, timezone
 from statistics import median
 from typing import TYPE_CHECKING, Any, Callable
 
+from backend.data.numbers import non_nan_number as _number
 from backend.data.providers.ifind_client import IfindError, IfindHttpClient
 from backend.data.providers.tushare_client import TushareClient, TushareError
 
@@ -16,14 +17,6 @@ if TYPE_CHECKING:
 CHINA_TIMEZONE = timezone(timedelta(hours=8))
 
 
-def _number(value: Any, default: float = 0.0) -> float:
-    try:
-        number = float(value)
-        return number if number == number else default
-    except (TypeError, ValueError):
-        return default
-
-
 def _display_date(value: str) -> str:
     text = str(value or "").replace("-", "")
     if len(text) != 8:

app/backend/features/screener/compiler.py

@@ -4,7 +4,7 @@ import json
 from typing import Any
 
 from backend.llm import transport as llm_transport
-from screener import FACTOR_FIELDS, REGIMES
+from backend.features.screener.engine import FACTOR_FIELDS, REGIMES
 
 
 class LLMCompilerError(RuntimeError):

app/backend/features/screener/engine.py

@@ -8,10 +8,12 @@ from collections import defaultdict
 from datetime import datetime, timedelta
 from typing import Any
 
-from advanced_strategies import ADVANCED_CURATED_STRATEGIES
+from backend.bootstrap.config import display_compact_date as _display_date
+from backend.data.numbers import finite_number as _number
+from backend.data.providers.tushare_client import TushareClient, TushareError
+from backend.features.screener.strategies import ADVANCED_CURATED_STRATEGIES
 from database import ReviewDatabase
 from backend.features.sentiment.engine import build_sentiment_history, latest_contiguous_history
-from tushare_client import TushareClient, TushareError
 
 
 REGIMES = {
@@ -2199,15 +2201,3 @@ def _regime_reason(regime: str) -> str:
         "divergence": "指数或核心仍强，但广度、封板质量开始分化。",
         "retreat": "情绪指标继续走弱，应提高筛选门槛并接受无候选结果。",
     }.get(regime, "市场阶段待确认。")
-
-
-def _number(value: Any, default: float = 0.0) -> float:
-    try:
-        number = float(value)
-        return number if math.isfinite(number) else default
-    except (TypeError, ValueError):
-        return default
-
-
-def _display_date(value: str) -> str:
-    return f"{value[:4]}-{value[4:6]}-{value[6:8]}" if len(value) == 8 else value

app/backend/features/sentiment/engine.py

@@ -4,6 +4,8 @@ from copy import deepcopy
 from statistics import mean, median
 from typing import Any
 
+from backend.data.numbers import non_nan_number as _number
+
 
 COMPONENT_WEIGHTS = {
     "breadth": 20,
@@ -16,14 +18,6 @@ COMPONENT_WEIGHTS = {
 SENTIMENT_ENGINE_VERSION = 2
 
 
-def _number(value: Any, default: float = 0.0) -> float:
-    try:
-        number = float(value)
-        return number if number == number else default
-    except (TypeError, ValueError):
-        return default
-
-
 def _clamp(value: float, lower: float = 0.0, upper: float = 100.0) -> float:
     return min(upper, max(lower, value))
 

app/config/architecture-inventory.json

@@ -220,6 +220,41 @@
       "runtime_role": "index observation fallback"
     }
   ],
+  "provider_construction": [
+    {
+      "client": "TushareClient",
+      "owner": "backend/data/providers/tushare.py",
+      "compatibility_fallback": "backend/features/market/service.py"
+    },
+    {
+      "client": "IfindHttpClient",
+      "owner": "backend/data/gateway.py"
+    },
+    {
+      "client": "MarketChartClient",
+      "owner": "backend/data/gateway.py"
+    },
+    {
+      "client": "WebRealtimeAggregator",
+      "owner": "backend/data/gateway.py"
+    }
+  ],
+  "numeric_normalization": [
+    {
+      "function": "finite_number",
+      "path": "backend/data/numbers.py"
+    },
+    {
+      "function": "non_nan_number",
+      "path": "backend/data/numbers.py"
+    }
+  ],
+  "date_formatting": [
+    {
+      "function": "display_compact_date",
+      "path": "backend/bootstrap/config.py"
+    }
+  ],
   "llm_entrypoints": [
     {
       "function": "stream_with_mentor",
@@ -279,13 +314,13 @@
     },
     {
       "path": "backend/features/screener/engine.py",
-      "bytes": 108552,
-      "lines": 2213
+      "bytes": 108387,
+      "lines": 2203
     },
     {
       "path": "backend/data/providers/tushare_client.py",
-      "bytes": 94329,
-      "lines": 2175
+      "bytes": 94124,
+      "lines": 2165
     },
     {
       "path": "frontend/app.js",
@@ -314,8 +349,8 @@
     },
     {
       "path": "backend/features/market/insights.py",
-      "bytes": 58150,
-      "lines": 1314
+      "bytes": 57998,
+      "lines": 1307
     },
     {
       "path": "frontend/pages/market/runtime.js",

app/sync_data.py

@@ -3,7 +3,8 @@ from __future__ import annotations
 import argparse
 from datetime import date
 
-from server import SERVICE, normalize_date
+from backend.application import SERVICE
+from backend.bootstrap.config import normalize_date
 
 
 def main() -> None:

app/tests/preservation_helpers.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import ast
 import hashlib
 import re
 from pathlib import Path
@@ -33,6 +34,41 @@ def sha256(path: Path) -> str:
     return hashlib.sha256(path.read_bytes()).hexdigest()
 
 
+def function_contract(path: Path, name: str) -> tuple[str, str]:
+    tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
+    function = next(
+        node for node in tree.body if isinstance(node, ast.FunctionDef) and node.name == name
+    )
+    body = ast.Module(body=function.body, type_ignores=[])
+    return (
+        ast.dump(function.args, include_attributes=False),
+        ast.dump(body, include_attributes=False),
+    )
+
+
+def module_contract(
+    path: Path,
+    *,
+    excluded_definitions: set[str] | None = None,
+    excluded_import_modules: set[str] | None = None,
+    exclude_imports: bool = False,
+) -> str:
+    excluded_definitions = excluded_definitions or set()
+    excluded_import_modules = excluded_import_modules or set()
+    tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
+    tree.body = [
+        node
+        for node in tree.body
+        if not (exclude_imports and isinstance(node, (ast.Import, ast.ImportFrom)))
+        and not (
+            isinstance(node, ast.ImportFrom)
+            and node.module in excluded_import_modules
+        )
+        and getattr(node, "name", None) not in excluded_definitions
+    ]
+    return ast.dump(tree, include_attributes=False)
+
+
 def reassembled_frontend_runtime() -> str:
     chunks: dict[tuple[int, int], str] = {}
     for path in FRONTEND_ROOT.rglob("*.js"):

app/tests/test_data_gateway.py

@@ -1,7 +1,9 @@
 from __future__ import annotations
 
+import ast
 import unittest
 from datetime import datetime, timedelta
+from pathlib import Path
 
 from backend.data import (
     DataPolicyError,
@@ -45,8 +47,6 @@ class DataGatewayTests(unittest.TestCase):
         self.assertIs(gateway.chart_data.ifind, gateway.ifind)
 
     def test_server_has_no_direct_runtime_tushare_construction(self) -> None:
-        from pathlib import Path
-
         source = (
             Path(__file__).resolve().parents[1]
             / "backend"
@@ -57,6 +57,33 @@ class DataGatewayTests(unittest.TestCase):
         self.assertEqual(source.count("TushareClient(self.token)"), 1)
         self.assertIn("return gateway.tushare()", source)
 
+    def test_provider_construction_has_unique_declared_owners(self) -> None:
+        root = Path(__file__).resolve().parents[1]
+        owners = {
+            "EastmoneyChartClient": {"backend/data/gateway.py"},
+            "IfindHttpClient": {"backend/data/gateway.py"},
+            "IfindProvider": {"backend/data/gateway.py"},
+            "MarketChartClient": {"backend/data/gateway.py"},
+            "TushareClient": {"backend/features/market/service.py"},
+            "TushareProvider": {"backend/data/gateway.py"},
+            "WebRealtimeAggregator": {"backend/data/gateway.py"},
+        }
+        found = {name: set() for name in owners}
+        for path in (root / "backend").rglob("*.py"):
+            relative = path.relative_to(root).as_posix()
+            tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
+            for node in ast.walk(tree):
+                if not isinstance(node, ast.Call):
+                    continue
+                name = getattr(node.func, "id", None) or getattr(node.func, "attr", None)
+                if name in found:
+                    found[name].add(relative)
+        self.assertEqual(found, owners)
+        provider_source = (root / "backend/data/providers/tushare.py").read_text(
+            encoding="utf-8"
+        )
+        self.assertIn("client_factory: Callable[[str], TushareClient] = TushareClient", provider_source)
+
     def test_quality_gate_accepts_matching_daily_evidence(self) -> None:
         timezone = market_timezone()
         now = datetime(2026, 7, 29, 16, 0, tzinfo=timezone)

app/tests/test_feature_boundaries.py

@@ -20,12 +20,6 @@ class FeatureBoundaryTests(unittest.TestCase):
         }
         violations = []
         for path in FEATURES.rglob("*.py"):
-            # The screener engine is an exact-preservation move of the legacy
-            # calculation module.  Its provider dependency is covered by the
-            # slice equivalence tests and will be addressed only after the
-            # behavior-preserving migration is complete.
-            if path.relative_to(FEATURES).as_posix() == "screener/engine.py":
-                continue
             tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
             for node in ast.walk(tree):
                 names = []
@@ -38,6 +32,53 @@ class FeatureBoundaryTests(unittest.TestCase):
                         violations.append(f"{path.relative_to(ROOT)} -> {name}")
         self.assertEqual(violations, [])
 
+    def test_backend_uses_root_compatibility_modules_only_at_declared_boundaries(self) -> None:
+        compatibility_modules = {
+            "advanced_strategies",
+            "alert_service",
+            "api_access",
+            "app_config",
+            "assistant_agent",
+            "chart_data_provider",
+            "heaven_agent",
+            "heaven_engine",
+            "ifind_client",
+            "llm_strategy",
+            "llm_stream",
+            "market_insights",
+            "mentor_agent",
+            "realtime_aggregator",
+            "screener",
+            "security",
+            "sentiment_engine",
+            "server",
+            "strategy_tracking",
+            "trade_journal",
+            "tushare_client",
+        }
+        allowed = {
+            "backend/application.py": {"api_access"},
+            "backend/features/screener/repository.py": {"sentiment_engine"},
+        }
+        violations = []
+        for path in (ROOT / "backend").rglob("*.py"):
+            relative = path.relative_to(ROOT).as_posix()
+            tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
+            for node in ast.walk(tree):
+                names = []
+                if isinstance(node, ast.Import):
+                    names = [alias.name for alias in node.names]
+                elif isinstance(node, ast.ImportFrom) and node.module:
+                    names = [node.module]
+                for name in names:
+                    root_name = name.split(".")[0]
+                    if (
+                        root_name in compatibility_modules
+                        and root_name not in allowed.get(relative, set())
+                    ):
+                        violations.append(f"{relative} -> {name}")
+        self.assertEqual(violations, [])
+
     def test_legacy_service_modules_are_compatibility_exports_only(self) -> None:
         for filename in ("alert_service.py", "trade_journal.py", "strategy_tracking.py"):
             tree = ast.parse((ROOT / filename).read_text(encoding="utf-8"))

app/tests/test_numeric_normalization.py

@@ -0,0 +1,38 @@
+from __future__ import annotations
+
+import math
+import unittest
+
+from backend.data.numbers import finite_number, non_nan_number
+from backend.data.providers import tushare_client
+from backend.features.market import insights
+from backend.features.screener import engine as screener_engine
+from backend.features.sentiment import engine as sentiment_engine
+
+
+class NumericNormalizationTests(unittest.TestCase):
+    def test_consumers_use_their_declared_shared_policy(self) -> None:
+        self.assertIs(tushare_client._number, finite_number)
+        self.assertIs(screener_engine._number, finite_number)
+        self.assertIs(insights._number, non_nan_number)
+        self.assertIs(sentiment_engine._number, non_nan_number)
+
+    def test_finite_policy_preserves_existing_results(self) -> None:
+        self.assertEqual(finite_number("12.5"), 12.5)
+        self.assertEqual(finite_number(None), 0.0)
+        self.assertEqual(finite_number("invalid", 7.0), 7.0)
+        self.assertEqual(finite_number(math.nan, 7.0), 7.0)
+        self.assertEqual(finite_number(math.inf, 7.0), 7.0)
+        self.assertEqual(finite_number(-math.inf, 7.0), 7.0)
+
+    def test_non_nan_policy_keeps_infinity_but_rejects_nan(self) -> None:
+        self.assertEqual(non_nan_number("12.5"), 12.5)
+        self.assertEqual(non_nan_number(None), 0.0)
+        self.assertEqual(non_nan_number("invalid", 7.0), 7.0)
+        self.assertEqual(non_nan_number(math.nan, 7.0), 7.0)
+        self.assertEqual(non_nan_number(math.inf, 7.0), math.inf)
+        self.assertEqual(non_nan_number(-math.inf, 7.0), -math.inf)
+
+
+if __name__ == "__main__":
+    unittest.main()

app/tests/test_preservation_slice_market.py

@@ -11,12 +11,14 @@ import realtime_aggregator
 import tushare_client
 from backend.bootstrap import config as bootstrap_config
 from backend.data import realtime
+from backend.data.numbers import finite_number
 from backend.data.providers import ifind_client as canonical_ifind
 from backend.data.providers import tushare_client as canonical_tushare
 from backend.features.market import charts
 from tests.preservation_helpers import (
     assert_frontend_runtime_matches_audited_baseline,
     assert_moved_asset_matches,
+    function_contract,
 )
 
 
@@ -102,18 +104,6 @@ def top_level_definitions(path: Path) -> dict[str, str]:
     }
 
 
-def function_contract(path: Path, name: str) -> tuple[str, str]:
-    tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
-    function = next(
-        node for node in tree.body if isinstance(node, ast.FunctionDef) and node.name == name
-    )
-    body = ast.Module(body=function.body, type_ignores=[])
-    return (
-        ast.dump(function.args, include_attributes=False),
-        ast.dump(body, include_attributes=False),
-    )
-
-
 class MarketSliceSourceEquivalenceTests(unittest.TestCase):
     def test_market_service_methods_are_exact_original_ast(self) -> None:
         original = class_methods(ORIGINAL_ROOT / "server.py", "DashboardService")
@@ -154,10 +144,25 @@ class MarketSliceSourceEquivalenceTests(unittest.TestCase):
         )
         for original, migrated in exact_moves:
             self.assertEqual(sha256(ORIGINAL_ROOT / original), sha256(APP_ROOT / migrated))
+        original_tushare = top_level_definitions(ORIGINAL_ROOT / "tushare_client.py")
+        original_tushare.pop("_number")
+        original_tushare.pop("_display_date")
         self.assertEqual(
-            top_level_definitions(ORIGINAL_ROOT / "tushare_client.py"),
+            original_tushare,
             top_level_definitions(APP_ROOT / "backend/data/providers/tushare_client.py"),
         )
+        self.assertEqual(
+            function_contract(ORIGINAL_ROOT / "tushare_client.py", "_number"),
+            function_contract(APP_ROOT / "backend/data/numbers.py", "finite_number"),
+        )
+        self.assertIs(canonical_tushare._number, finite_number)
+        self.assertEqual(
+            function_contract(ORIGINAL_ROOT / "tushare_client.py", "_display_date"),
+            function_contract(
+                APP_ROOT / "backend/bootstrap/config.py", "display_compact_date"
+            ),
+        )
+        self.assertIs(canonical_tushare._display_date, bootstrap_config.display_compact_date)
         original_charts = top_level_definitions(ORIGINAL_ROOT / "chart_data_provider.py")
         original_charts.pop("_stock_market_code")
         self.assertEqual(

app/tests/test_preservation_slice_market_insights.py

@@ -6,11 +6,13 @@ import unittest
 from pathlib import Path
 
 import market_insights
+from backend.data.numbers import non_nan_number
 from backend.features.market import insights as canonical_insights
 from tests.preservation_helpers import (
     assert_frontend_runtime_matches_audited_baseline,
     assert_moved_asset_matches,
     assert_page_prefix_matches,
+    function_contract,
 )
 
 
@@ -106,6 +108,11 @@ class MarketInsightsSliceSourceEquivalenceTests(unittest.TestCase):
             MARKET_INSIGHT_METHODS,
         )
         self.assertIs(market_insights.MarketInsightsService, canonical_insights.MarketInsightsService)
+        self.assertEqual(
+            function_contract(ORIGINAL_ROOT / "market_insights.py", "_number"),
+            function_contract(APP_ROOT / "backend/data/numbers.py", "non_nan_number"),
+        )
+        self.assertIs(canonical_insights._number, non_nan_number)
 
     def test_dashboard_service_methods_are_exact_original_ast(self) -> None:
         original = ORIGINAL_ROOT / "server.py"

app/tests/test_preservation_slice_screener.py

@@ -9,12 +9,16 @@ import advanced_strategies
 import llm_strategy
 import screener
 import strategy_tracking
+from backend.bootstrap import config as bootstrap_config
+from backend.data.numbers import finite_number
 from backend.features.screener import compiler, engine, strategies, tracking
 from backend.features.screener import service as screener_service
 from tests.preservation_helpers import (
     assert_frontend_runtime_matches_audited_baseline,
     assert_moved_asset_matches,
     assert_page_prefix_matches,
+    function_contract,
+    module_contract,
 )
 
 
@@ -91,14 +95,6 @@ def top_level_definition(path: Path, name: str) -> str:
     return ast.dump(node, include_attributes=False)
 
 
-def module_without_imports(path: Path) -> str:
-    tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
-    tree.body = [
-        node for node in tree.body if not isinstance(node, (ast.Import, ast.ImportFrom))
-    ]
-    return ast.dump(tree, include_attributes=False)
-
-
 def sha256(path: Path) -> str:
     return hashlib.sha256(path.read_bytes()).hexdigest()
 
@@ -154,11 +150,29 @@ class ScreenerSliceSourceEquivalenceTests(unittest.TestCase):
 
     def test_engine_and_tracking_logic_match_the_original(self) -> None:
         self.assertEqual(
-            module_without_imports(ORIGINAL_ROOT / "screener.py"),
-            module_without_imports(
-                APP_ROOT / "backend" / "features" / "screener" / "engine.py"
+            module_contract(
+                ORIGINAL_ROOT / "screener.py",
+                excluded_definitions={"_display_date", "_number"},
+                exclude_imports=True,
+            ),
+            module_contract(
+                APP_ROOT / "backend" / "features" / "screener" / "engine.py",
+                excluded_definitions={"_display_date", "_number"},
+                exclude_imports=True,
             ),
         )
+        self.assertEqual(
+            function_contract(ORIGINAL_ROOT / "screener.py", "_number"),
+            function_contract(APP_ROOT / "backend/data/numbers.py", "finite_number"),
+        )
+        self.assertIs(engine._number, finite_number)
+        self.assertEqual(
+            function_contract(ORIGINAL_ROOT / "screener.py", "_display_date"),
+            function_contract(
+                APP_ROOT / "backend/bootstrap/config.py", "display_compact_date"
+            ),
+        )
+        self.assertIs(engine._display_date, bootstrap_config.display_compact_date)
         self.assertEqual(
             class_methods(
                 ORIGINAL_ROOT / "backend" / "features" / "screener" / "tracking.py",

app/tests/test_preservation_slice_sentiment_pools.py

@@ -6,11 +6,14 @@ import unittest
 from pathlib import Path
 
 import sentiment_engine
+from backend.data.numbers import non_nan_number
 from backend.features.sentiment import engine as canonical_engine
 from tests.preservation_helpers import (
     assert_frontend_runtime_matches_audited_baseline,
     assert_moved_asset_matches,
     assert_page_prefix_matches,
+    function_contract,
+    module_contract,
 )
 
 
@@ -94,10 +97,22 @@ class SentimentPoolSliceSourceEquivalenceTests(unittest.TestCase):
 
     def test_sentiment_engine_is_exact_original_with_legacy_alias(self) -> None:
         self.assertEqual(
-            sha256(ORIGINAL_ROOT / "sentiment_engine.py"),
-            sha256(APP_ROOT / "backend" / "features" / "sentiment" / "engine.py"),
+            module_contract(
+                ORIGINAL_ROOT / "sentiment_engine.py",
+                excluded_definitions={"_number"},
+            ),
+            module_contract(
+                APP_ROOT / "backend" / "features" / "sentiment" / "engine.py",
+                excluded_definitions={"_number"},
+                excluded_import_modules={"backend.data.numbers"},
+            ),
+        )
+        self.assertEqual(
+            function_contract(ORIGINAL_ROOT / "sentiment_engine.py", "_number"),
+            function_contract(APP_ROOT / "backend/data/numbers.py", "non_nan_number"),
         )
         self.assertIs(sentiment_engine, canonical_engine)
+        self.assertIs(canonical_engine._number, non_nan_number)
 
     def test_api_and_frontend_assets_are_unchanged(self) -> None:
         self.assertEqual(

app/tools/build_architecture_inventory.py

@@ -155,6 +155,19 @@ def build() -> dict[str, Any]:
             {"provider": "eastmoney", "path": "backend/data/realtime.py", "runtime_role": "isolated realtime observation"},
             {"provider": "tencent", "path": "backend/data/realtime.py", "runtime_role": "index observation fallback"},
         ],
+        "provider_construction": [
+            {"client": "TushareClient", "owner": "backend/data/providers/tushare.py", "compatibility_fallback": "backend/features/market/service.py"},
+            {"client": "IfindHttpClient", "owner": "backend/data/gateway.py"},
+            {"client": "MarketChartClient", "owner": "backend/data/gateway.py"},
+            {"client": "WebRealtimeAggregator", "owner": "backend/data/gateway.py"},
+        ],
+        "numeric_normalization": [
+            {"function": "finite_number", "path": "backend/data/numbers.py"},
+            {"function": "non_nan_number", "path": "backend/data/numbers.py"},
+        ],
+        "date_formatting": [
+            {"function": "display_compact_date", "path": "backend/bootstrap/config.py"},
+        ],
         "llm_entrypoints": [
             {"function": "stream_with_mentor", "path": "backend/features/mentor/agent.py"},
             {"function": "interpret_heaven", "path": "backend/features/heaven/agent.py"},

docs/governance/code-reduction.md

@@ -21,6 +21,10 @@
 | CR-01 | LLM供应商传输 | 问师、问天、复盘助手和策略编译各自构造HTTP请求、解析响应和读取错误 | 只保留`backend/llm/transport.py`一个网络出口 | 已完成 |
 | CR-02 | HTTP精确POST委托 | 27个端点重复使用“比较路径、调用无参数处理器、返回”三行分支 | 用公开/受保护两张显式映射统一委托，同时保留复杂路由的原控制流 | 已完成 |
 | CR-03 | 股票市场后缀转换 | Tushare业务与iFinD图表各保留一份完全相同的沪深京代码转换函数 | 图表复用`bootstrap/config.py::tushare_code`，只保留一份函数体 | 已完成 |
+| CR-04 | 数值归一化策略 | 四个业务模块分别保留两组完全相同的数值转换函数体 | 由`backend/data/numbers.py`集中拥有两种既有语义，消费者保留原局部别名 | 已完成 |
+| CR-05 | 根级兼容入口 | 正式后端仍有五处通过迁移兼容模块反向导入规范实现 | 正式代码改用规范路径；兼容入口只服务原公开导入契约 | 已完成 |
+| CR-06 | 紧凑日期显示 | Tushare与选股引擎各保留一份完全相同的`YYYYMMDD`显示转换 | 由`bootstrap/config.py`拥有唯一格式策略，消费者保留原局部别名 | 已完成 |
+| CR-07 | 数据Provider组装 | 核查网关、容器和业务服务是否重复创建外部数据客户端 | 固化唯一创建位置及兼容例外，不改数据源语义 | 已完成 |
 
 ## CR-01验收口径
 
@@ -89,3 +93,99 @@
 
 本批基线为`xiaobai-reduction-02-http-dispatch-20260801`；检查点为
 `xiaobai-reduction-03-market-symbol-20260801`。
+
+## CR-04验收口径
+
+- 只合并参数、函数体和运行结果完全一致的数值转换函数，不借本批改变任何业务计算或异常默认值。
+- `finite_number`继续拒绝`NaN`与正负无穷；`non_nan_number`继续只拒绝`NaN`并保留正负无穷。
+- Tushare与智能选股必须复用有限数策略；市场洞察与情绪引擎必须复用非NaN策略，并继续暴露原局部
+  `_number`名称以保持兼容。
+- 实时行情和图表转换器的空值、默认值或参数签名语义不同，必须继续独立保留，不能因名称相同而合并。
+- 原版函数参数和函数体分别与共享实现AST相等；四个消费者的局部别名必须指向对应的唯一函数对象。
+
+## CR-04结果
+
+- 删除Tushare、智能选股、市场洞察和情绪引擎中的四份重复函数体，新建两种明确命名的共享策略；生产
+  代码净减少约12行，全仓AST扫描不再发现完全相同的函数定义。
+- 将可复用的函数及模块AST契约归入测试辅助层，原有迁移保持性测试改为“未改范围保持相等、被替换
+  函数与共享实现相等、运行时唯一对象”三重断言，没有降低门禁。
+- 实时行情`backend/data/realtime.py::_number`与图表`backend/features/market/charts.py::_number`
+  被明确保留；它们不是本批重复实现，也未改变行为。
+- 58项定向测试、41项保持性/治理测试、原版231项、候选320项、纯`app/`导出257项和45项
+  Playwright通过；24个JavaScript文件、API/架构注册表、Git空白检查和SQLite完整性检查通过。
+- 本批不修改前端、CSS、接口、数据来源、行情口径、选股条件、数据库、LLM、权限或部署。
+
+本批基线为`xiaobai-reduction-03-market-symbol-20260801`；检查点为
+`xiaobai-reduction-04-numeric-normalization-20260801`。
+
+## CR-05验收口径
+
+- 逐项扫描根级Python入口、生产代码、测试、工具和动态导入；没有消费者或兼容责任的入口才能删除。
+- 规范后端不得经由`screener`、`advanced_strategies`、`tushare_client`或`server`兼容入口
+  间接访问已经归位的实现。
+- 所有根级模块继续保持原导入名称、导出对象及模块对象身份，既有启动命令和第三方维护脚本不受影响。
+- `api_access`、选股Repository的惰性`sentiment_engine`导入及根级`database.py`属于已登记边界，
+  分别留到HTTP、Repository阶段处理，不在本批跨边界修改。
+
+## CR-05结果
+
+- 审计确认21个根级兼容入口均有测试、工具、启动或原公开导入契约消费者，因此本批没有冒险删除文件。
+- 容器、策略编译器、选股引擎及数据同步命令的五处导入改为规范模块路径，正式代码不再通过四个根级
+  兼容模块反向进入实现；运行代码行数未增加。
+- 特性边界测试取消选股引擎旧例外，并新增全后端兼容导入门禁；只允许两项已登记过渡边界，后续代码
+  无法重新引入隐式根级依赖。
+- 候选321项、纯`app/`导出258项、24个JavaScript文件、API/架构注册表、Git空白检查和SQLite
+  完整性检查通过；本批不涉及页面、CSS或浏览器行为。
+- 本批不修改业务计算、策略公式、数据源、API、数据库、LLM、权限、前端或部署。
+
+本批基线为`xiaobai-reduction-04-numeric-normalization-20260801`；检查点为
+`xiaobai-reduction-05-compatibility-boundaries-20260801`。
+
+## CR-06验收口径
+
+- 只合并参数、函数体和异常行为完全相同的日期/文本转换；名称相似但空值、未来日期、错误文案或
+  输入格式不同的函数不得合并。
+- Tushare与选股引擎继续暴露局部`_display_date`名称，并分别指向唯一共享实现。
+- 原版两个`_display_date`函数必须分别与共享实现AST相等，所有原调用结果保持不变。
+- 市场洞察的日期显示函数会清理连字符并容忍空值，语义不同，必须继续独立保留。
+
+## CR-06结果
+
+- 删除Tushare与选股引擎内两个重复日期函数体，新增`display_compact_date`唯一策略；生产代码总
+  行数不增加，重复函数体由两份降为一份。
+- 架构清单登记日期格式唯一所有权；保持性测试改为未改范围AST相等、共享函数AST相等和运行时
+  对象身份三重契约，没有放宽原迁移门禁。
+- `normalize_date`、市场洞察日期显示、实时行情时间格式和会员日期边界因语义不同均原样保留。
+- 候选321项、纯`app/`导出258项、24个JavaScript文件、API/架构注册表、Git空白检查和SQLite
+  完整性检查通过；本批不涉及页面、CSS或浏览器行为。
+- 本批不修改日期输入规则、业务计算、选股结果、接口、数据库、数据源、LLM、权限或部署。
+
+本批基线为`xiaobai-reduction-05-compatibility-boundaries-20260801`；检查点为
+`xiaobai-reduction-06-date-formatting-20260801`。
+
+## CR-07验收口径
+
+- iFinD、图表、实时观察器和Provider适配器必须只在`build_data_gateway`创建，并由容器共享。
+- Tushare必须继续通过实时Token供应器按需创建，不能为了减少对象数量缓存过期Token。
+- 市场服务中为原版隔离测试桩保留的一处`TushareClient(self.token)`是明确兼容例外，不得被误判为
+  第二条正式数据链路。
+- 不得合并Tushare、iFinD、东方财富和腾讯的传输、缓存、重试或降级逻辑。
+
+## CR-07结果
+
+- 全后端构造点扫描确认iFinD、MarketChart、东方财富图表和实时观察器均只有网关一个创建位置；
+  `ApplicationContainer`暴露的是同一对象引用，没有第二份客户端。
+- Tushare Provider使用动态Token供应器，市场服务只有一处已登记测试兼容回退；本批没有发现可安全
+  删除的生产实现，因此不为追求行数强行修改运行代码。
+- 架构清单新增Provider创建所有权，自动测试会在未来出现第二个未登记构造点时失败。
+- 候选322项、纯`app/`导出259项、24个JavaScript文件、API/架构注册表、Git空白检查和SQLite
+  完整性检查通过；本批不涉及页面、CSS或浏览器行为。
+- 本批不修改请求频率、缓存、重试、Token更新、数据源选择、计算口径、API、数据库或前端。
+
+本批基线为`xiaobai-reduction-06-date-formatting-20260801`；检查点为
+`xiaobai-reduction-07-provider-ownership-20260801`。
+
+## 人工验收记录
+
+- 2026-08-01：用户检查CR-02与CR-03运行结果，确认未发现明显异常。本记录仅表示本轮可见功能与
+  页面使用未发现明显回归，不替代后续批次各自的自动测试和人工抽查。