fix: 修复 AI Code Review 提出的3个阻塞问题

1. 模式切换失效：使用复合 triggerKey (libraryId_mode) 替代单一 libraryId 模式切换时 key 变化会重新触发自动匹配 2. 闭包陷阱：将 handleSmartMatch 从 smartMatch 解构并加入依赖项 handleSmartMatch 由 useCallback 包装，引用稳定 3. 依赖粒度：materials.items 替代 materials.items.length 素材内容变化也能正确触发重新匹配
2026-08-07 18:53:42 +08:00
parent 305f054d4c
commit 516aeab8de
1 changed files with 7 additions and 6 deletions
@@ -36,19 +36,20 @@ export function useStep2Materials({

  /* ── 自动触发智能匹配：选择视频库后自动调用 ── */
  const autoTriggeredRef = useRef<string>("")
+  const { handleSmartMatch } = smartMatch
  useEffect(() => {
    // 仅在 auto 模式 + 素材加载完成后自动触发
    if (materialMode !== "auto") return
    if (!selectedLibraryId) return
    if (materialsLoading) return
    if (materials.items.length === 0) return
-    // 防止同一视频库重复触发
-    if (autoTriggeredRef.current === selectedLibraryId) return
+    // 防止同一视频库+同一模式重复触发（模式切换时 key 变化会重新触发）
+    const triggerKey = `${selectedLibraryId}_${materialMode}`
+    if (autoTriggeredRef.current === triggerKey) return

-    autoTriggeredRef.current = selectedLibraryId
-    smartMatch.handleSmartMatch()
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [selectedLibraryId, materialMode, materialsLoading, materials.items.length])
+    autoTriggeredRef.current = triggerKey
+    handleSmartMatch()
+  }, [selectedLibraryId, materialMode, materialsLoading, materials.items, handleSmartMatch])

  /* ── 手动选择素材 ── */
  const handleToggleMaterial = useCallback(