fix(groups): 改判清空系列键与结算性能修复

Author: Aparencia

app/src-tauri/src/commands_settlement.rs

@@ -96,8 +96,14 @@ pub fn execute_settlement(
             }
         }
     }
-    // 核心提炼：组核心笔记（本地规则——笔记目录 + 碎片摘要，落组内）
-    let core_note_id = create_core_note(&state_clone, group_id)?;
+    // 核心提炼：组核心笔记（本地规则——笔记目录 + 碎片摘要，落组内）。
+    // 审查修复（2026-08-22）：无合并无归档的空转结算不产核心笔记——
+    // 防反复点击结算刷出重复笔记（仪式留痕靠 settlements 记录即可）。
+    let core_note_id = if merged > 0 || archived > 0 {
+        create_core_note(&state_clone, group_id)?
+    } else {
+        None
+    };
     // 留痕：结算记录 + 北极星埋点（组成③经历过结算）
     let stats = serde_json::json!({
         "merged": merged, "archived": archived, "coreNoteId": core_note_id,
@@ -144,9 +150,11 @@ fn settlement_plan_inner(state: &AppState, group_id: i64) -> Result<SettlementPl
         })
         .collect();
     let age_cutoff = now_secs - ARCHIVE_AGE_DAYS * 86_400;
+    // 审查修复（2026-08-22）：有卡绑定判定改单条 SQL 全集（替代逐碎片 N+1）
+    let card_bound = state.db.fragment_ids_with_cards().map_err(|e| e.to_string())?;
     let mut archive_candidates = Vec::new();
     for f in &fragments {
-        if f.created_at < age_cutoff && !state.db.fragment_has_card(f.id).map_err(|e| e.to_string())? {
+        if f.created_at < age_cutoff && !card_bound.contains(&f.id) {
             archive_candidates.push(ArchiveCandidateView { id: f.id, text: f.text.clone() });
         }
     }

app/src-tauri/src/db_note_groups.rs

@@ -144,6 +144,10 @@ impl Db {
 
     /// 路由改判（REQ-198 修改即记忆）：覆盖 kind/domain_tag，reason 追加来源，
     /// route_overridden 置 1——后续自动路由不得覆盖用户裁决。
+    ///
+    /// @ai-context: 审查修复（2026-08-22）：改判为非课程组时同步清空 series_key——
+    /// 残留的系列键会让后续同系列会话经 find_group_by_series_key 误归入
+    /// 已被改判的组（路由误判 ★★★★ 死法的改判侧漏洞）。
     pub fn override_group_route(
         &self,
         id: i64,
@@ -154,7 +158,9 @@ impl Db {
         self.with_conn(|conn| {
             let affected = conn.execute(
                 "UPDATE note_groups SET kind = ?1, domain_tag = ?2, route_reason = ?3,
-                 route_overridden = 1, updated_at = ?4 WHERE id = ?5",
+                 route_overridden = 1,
+                 series_key = CASE WHEN ?1 = 'course' THEN series_key ELSE NULL END,
+                 updated_at = ?4 WHERE id = ?5",
                 params![kind, domain_tag, reason, unix_seconds(), id],
             )?;
             Ok(affected > 0)

app/src-tauri/src/db_note_groups_tests.rs

@@ -131,6 +131,24 @@ fn override_route_marks_and_updates() {
     assert!(fetched.route_reason.unwrap().contains("用户改判"));
 }
 
+#[test]
+fn override_course_group_clears_series_key() {
+    // Arrange：课程组带系列键（审查修复回归：改判后系列键不得残留，
+    // 否则后续同系列会话经 find_group_by_series_key 误归入已改判的组）
+    let db = mem_db();
+    let mut course = group("零基础化妆");
+    course.kind = "course".to_string();
+    course.source = "series".to_string();
+    course.series_key = Some("零基础化妆".to_string());
+    let g = db.create_group(&course).expect("create");
+    // Act：改判为主题组
+    db.override_group_route(g.id, "topic", Some("beauty"), "用户改判").expect("override");
+    // Assert：系列键清空，系列查找不再命中
+    let fetched = db.get_group(g.id).expect("get").expect("exists");
+    assert_eq!(fetched.series_key, None);
+    assert!(db.find_group_by_series_key("零基础化妆").expect("find").is_none());
+}
+
 #[test]
 fn move_note_between_groups() {
     // Arrange

app/src-tauri/src/db_settlements.rs

@@ -38,6 +38,9 @@ impl Db {
     }
 
     /// 碎片是否有闪卡绑定（归档候选判据：有卡碎片不进归档——学习循环资产）。
+    /// 登记豁免 dead_code：计划构建已改走 fragment_ids_with_cards 批量口径，
+    /// 单查接口留给后续单碎片操作面（单测消费中）。
+    #[allow(dead_code)]
     pub fn fragment_has_card(&self, fragment_id: i64) -> Result<bool> {
         self.with_conn(|conn| {
             let count: i64 = conn.query_row(
@@ -48,6 +51,18 @@ impl Db {
             Ok(count > 0)
         })
     }
+
+    /// 有卡绑定的碎片 id 集合（审查修复 2026-08-22：结算计划批量判定替代
+    /// 逐碎片 fragment_has_card 的 N+1 查询——单条 SQL 全集）。
+    pub fn fragment_ids_with_cards(&self) -> Result<std::collections::HashSet<i64>> {
+        self.with_conn(|conn| {
+            let mut stmt =
+                conn.prepare("SELECT DISTINCT fragment_id FROM flashcards WHERE fragment_id IS NOT NULL")?;
+            let rows = stmt.query_map([], |r| r.get::<_, i64>(0))?;
+            let set: std::collections::HashSet<i64> = rows.collect::<rusqlite::Result<Vec<_>>>()?.into_iter().collect();
+            Ok(set)
+        })
+    }
 }
 
 /// 单测独立文件。