fix(review-feedback-1709): 补一个未命中敏感规则的异常回归测试

Author: ZhuLinsen

.env.example

@@ -354,7 +354,7 @@ SEARXNG_PUBLIC_INSTANCES_ENABLED=true
 # NEWS_INTEL_FETCH_TIMEOUT_SEC=8
 # 单次每个资讯源最多采集条数
 # NEWS_INTEL_MAX_ITEMS_PER_SOURCE=50
-# NewsNow HTTP API 基地址；可改为自建 NewsNow 实例，默认使用公开示例实例
+# NewsNow HTTP API 基地址；默认使用公开示例实例（可能存在不可用/限流），生产建议改为自建实例
 # NEWSNOW_BASE_URL=https://newsnow.busiyi.world
 # 注：本节仅控制资讯源采集与清理行为，不会改变 LLM / provider / base URL / 兼容回退语义。
 # 乖离率阈值（%），偏离 MA5 超过此值提示不追高；强势趋势股自动放宽到 1.5 倍

api/v1/endpoints/intelligence.py

@@ -39,7 +39,10 @@ def _not_found(message: str) -> HTTPException:
 def _internal_error(message: str, exc: Exception) -> HTTPException:
     sanitized_error = sanitize_diagnostic_text(str(exc), max_length=300) or "internal intelligence error"
     logger.error("%s: %s", message, sanitized_error)
-    return HTTPException(status_code=500, detail={"error": "internal_error", "message": f"{message}: {sanitized_error}"})
+    return HTTPException(
+        status_code=500,
+        detail={"error": "internal_error", "message": f"{message}: internal intelligence service error"},
+    )
 
 
 @router.post("/sources", response_model=IntelligenceSourceItem, responses={400: {"model": ErrorResponse}, 500: {"model": ErrorResponse}}, summary="Create intelligence source")

docs/CHANGELOG.md

@@ -45,6 +45,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 - [改进] #1707 补齐内置 RSS/Atom 资讯源模板后端入口，修复本地资讯拉取的 requests 参数兼容、请求阶段 DNS 校验、共享地址段拒绝、坏条目跳过与港股短代码匹配。
 - [文档] #1707 阐明情报池接入仅追加本地资讯消费，不改模型名/provider/base URL/默认模型策略/回退策略/保存前清理逻辑或运行时配置迁移；结构化风险提示若出现为关键词误报；回滚可采用 `revert` 本 PR 或停用/移除本地资讯接入入口与数据。
 - [文档] 本次 #1707 情报源基线仅新增 `NEWS_INTEL_*` 配置，不变更 LITELLM / ANSPIRE / Base URL / provider/model 清理与回退语义；回退策略为移除或禁用这三项情报源变量后恢复既有 LLM 配置行为。
+- [修复] #1707 默认禁用 `/api/v1/intelligence/sources/defaults` 新建源（未传 `enabled`），避免公开示例 NewsNow 实例被默认启用，同时统一 500 响应细节仅入日志、响应返回通用错误信息。
 
 - [新功能] 个股分析历史成功保存后会从最终报告 best-effort 提取 `DecisionSignal` 决策信号，复用现有信号去重、计划质量计算和脱敏契约。
 - [改进] 问股页移动端策略选择改为默认收起的按钮入口，展开后仍可多选策略并在发送后自动收起，减少对对话内容的遮挡。

docs/intelligence-sources.md

@@ -37,7 +37,12 @@ NEWS_INTEL_MAX_ITEMS_PER_SOURCE=50
 NEWSNOW_BASE_URL=https://newsnow.busiyi.world
 ```
 
-`NEWSNOW_BASE_URL` 用于拼出 `GET {NEWSNOW_BASE_URL}/api/s?id=<source_id>`。默认值来自 NewsNow MCP Server 示例配置；生产环境建议改成自建 NewsNow 实例，以减少公开示例实例不可用或限流带来的影响。
+`NEWSNOW_BASE_URL` 用于拼出 `GET {NEWSNOW_BASE_URL}/api/s?id=<source_id>`。当前默认值为公开示例实例 `https://newsnow.busiyi.world`，官方/上游链路与形态请以你实际接入实例文档为准；生产环境建议改成自建 NewsNow 实例，以避免公开示例实例不可用或限流。
+
+兼容性核验建议（非 mock）：
+
+- `curl -sS "${NEWSNOW_BASE_URL}/api/s?id=cls-hot" | python -c "import sys, json; data=json.load(sys.stdin); assert isinstance(data, dict); assert isinstance(data.get('items'), list); print('ok')"`
+- 若要手工核验 `status`、`id`、`items[].title`、`items[].url`/`mobileUrl`、`items[].pubDate`/`items[].extra.date` 这类字段，可与测试回归组合使用：`test_newsnow_source_fetches_json_items`。
 
 ## API
 
@@ -47,7 +52,7 @@ NEWSNOW_BASE_URL=https://newsnow.busiyi.world
 - `GET /sources`：查询资讯源。
 - `GET /sources/templates?market=hk`：查询内置资讯源模板。
 - `POST /sources/templates/{template_id}`：从内置模板创建资讯源，可覆盖名称、启用状态、作用域和说明。
-- `POST /sources/defaults`：一键创建全部内置默认源；接口幂等，已存在的同名源会返回 `created=false`，不会重复插入。请求体可传 `{ "enabled": false }` 控制创建后的启用状态。
+- `POST /sources/defaults`：一键创建全部内置默认源；接口幂等，已存在的同名源会返回 `created=false`，不会重复插入。默认不传 `enabled` 时以 `false` 创建；如需默认启用可传 `{ "enabled": true }`。
 - `POST /sources/test`：测试 payload，不落库。
 - `POST /sources/{source_id}/fetch?dry_run=false`：拉取单个源。
 - `POST /sources/fetch-enabled`：fail-open 拉取全部启用源。

src/services/intelligence_service.py

@@ -166,7 +166,8 @@ def create_source_from_template(self, template_id: str, overrides: Optional[Dict
         return self.create_source(payload)
 
     def create_default_sources(self, overrides: Optional[Dict[str, Any]] = None) -> Dict[str, Any]:
-        request_fields = overrides or {}
+        request_fields = dict(overrides or {})
+        request_fields.setdefault("enabled", False)
         created_count = 0
         items = []
         for template in self._builtin_source_templates():

tests/test_intelligence_api.py

@@ -128,9 +128,30 @@ def test_fetch_source_internal_error_is_sanitized(self) -> None:
         self.assertEqual(fetch_resp.status_code, 500)
         body = fetch_resp.json()
         self.assertEqual(body["error"], "internal_error")
-        self.assertTrue(body["message"].startswith("Fetch intelligence source failed"))
-        self.assertNotIn("token=secret", body["message"])
-        self.assertNotIn("abc12345", body["message"])
+        self.assertEqual(body["message"], "Fetch intelligence source failed: internal intelligence service error")
+
+    def test_fetch_source_internal_error_without_sensitive_pattern_is_generic(self) -> None:
+        create_resp = self.client.post("/api/v1/intelligence/sources", json={"name": "api-feed", "url": "https://feeds.example.com/rss.xml", "source_type": "rss", "scope_type": "market", "market": "cn"})
+        self.assertEqual(create_resp.status_code, 200)
+        source_id = create_resp.json()["id"]
+        with patch("src.services.intelligence_service.IntelligenceService.fetch_source", side_effect=RuntimeError("unexpected runtime assertion failure: pipeline context exhausted")):
+            fetch_resp = self.client.post(f"/api/v1/intelligence/sources/{source_id}/fetch")
+
+        self.assertEqual(fetch_resp.status_code, 500)
+        body = fetch_resp.json()
+        self.assertEqual(body["error"], "internal_error")
+        self.assertEqual(body["message"], "Fetch intelligence source failed: internal intelligence service error")
+        self.assertNotIn("pipeline context exhausted", body["message"])
+
+    def test_create_builtin_default_sources_are_disabled_by_default(self) -> None:
+        default_resp = self.client.post("/api/v1/intelligence/sources/defaults")
+        self.assertEqual(default_resp.status_code, 200)
+        newsnow_sources = [
+            item["source"] for item in default_resp.json()["items"]
+            if item["source"]["source_type"] == "newsnow"
+        ]
+        self.assertGreaterEqual(len(newsnow_sources), 5)
+        self.assertTrue(all(not item["enabled"] for item in newsnow_sources))
 
     def test_upstream_fetch_errors_do_not_expose_query_secret(self) -> None:
         secret_url = "https://feeds.example.com/rss.xml?token=super-secret"

tests/test_intelligence_service.py

@@ -296,6 +296,13 @@ def test_create_default_sources_is_idempotent(self) -> None:
         self.assertGreaterEqual(sources["total"], 3)
         self.assertTrue(all(not item["source"]["enabled"] for item in first["items"]))
 
+    def test_create_default_sources_are_disabled_by_default(self) -> None:
+        first = self.service.create_default_sources()
+        sources = self.service.list_sources()
+        self.assertEqual(first["created_count"], first["total"])
+        self.assertEqual(sources["total"], first["total"])
+        self.assertTrue(all(not item["enabled"] for item in sources["items"]))
+
     def test_same_url_can_be_saved_for_different_scopes(self) -> None:
         market = self.service.create_source({
             "name": "market-feed",