test(utils): add tests for compareKnowledgeBaseChange;

* test(utils): add tests for compareKnowledgeBaseChange; ci: run only
relevant tests in workflow

* ci: normalize changed paths for test selection to be relative to
working dir

* ci: Unit test triggering conditions

Author: EmbraceAn

.github/workflows/Essential-tests.yml

@@ -8,7 +8,7 @@ on:
   pull_request:
     branches: [test]
     paths:
-      - 'app/renderer/src/main/src/release/**'
+      - 'app/renderer/src/main/src/__tests__/**'
       - '.github/workflows/**'
 
 jobs:
@@ -35,6 +35,7 @@ jobs:
               app/renderer/engine-link-startup/yarn.lock
 
       - name: Cache Yarn cache
+        id: yarn-cache
         uses: actions/cache@v4
         with:
           path: ~/.cache/yarn
@@ -43,6 +44,7 @@ jobs:
             ${{ runner.os }}-yarn-cache-
 
       - name: Cache node_modules
+        id: node-mod-cache
         uses: actions/cache@v4
         with:
           path: app/renderer/src/main/node_modules
@@ -51,9 +53,14 @@ jobs:
             ${{ runner.os }}-node-modules-
 
       - name: Install dependencies (renderer main)
+        if: steps.node-mod-cache.outputs.cache-hit != 'true'
         working-directory: app/renderer/src/main
         run: yarn install --frozen-lockfile --prefer-offline --network-concurrency 1
 
+      - name: Skip install (node_modules cache hit)
+        if: steps.node-mod-cache.outputs.cache-hit == 'true'
+        run: echo "node_modules cache restored; skipping yarn install"
+
       - name: Verify installed packages (quick)
         working-directory: app/renderer/src/main
         run: |
@@ -70,8 +77,73 @@ jobs:
           echo "Show specific folder (if exists):"
           ls -la src/components/playground/Vitest__Test__ || true
 
+      - name: Determine relevant test files
+        id: test_selection
+        working-directory: app/renderer/src/main
+        shell: bash
+        run: |
+          # Get files changed in the PR. Use multiple fallbacks because the runner
+          # may have a shallow checkout and referenced commits might not exist.
+          CHANGED=""
+
+          # 1) Try the standard range if both commits are present
+          if git rev-parse --verify "${{ github.event.before }}" >/dev/null 2>&1 && git rev-parse --verify "${{ github.sha }}" >/dev/null 2>&1; then
+            CHANGED=$(git diff --name-only "${{ github.event.before }}" "${{ github.sha }}" || true)
+          fi
+
+          # 2) If still empty, try fetching the base branch (for PRs) and compare
+          if [ -z "$CHANGED" ] && [ -n "${{ github.event.pull_request.base.ref }}" ]; then
+            echo "Standard range not available — fetching base branch ${{ github.event.pull_request.base.ref }}"
+            git fetch --no-tags --depth=1 origin "${{ github.event.pull_request.base.ref }}" || true
+            BASE_REF="origin/${{ github.event.pull_request.base.ref }}"
+            CHANGED=$(git diff --name-only "$BASE_REF" "${{ github.sha }}" || true)
+          fi
+
+          # 3) Final fallback: use files from the last commit(s)
+          if [ -z "$CHANGED" ]; then
+            echo "Falling back to last commit changes"
+            CHANGED=$(git diff --name-only HEAD^ HEAD || git show --name-only --pretty= "" "${{ github.sha }}" || true)
+          fi
+
+          echo "Changed files:\n$CHANGED"
+
+          # Normalize changed paths to be relative to the working dir by
+          # taking the last occurrence of 'src/' (works for nested 'src' paths)
+          CHANGED_REL=$(printf "%s\n" "$CHANGED" | awk -F'src/' '{ if (NF>1) print "src/"$(NF) ; else print $0 }' | sort -u || true)
+          # remove empty lines
+          CHANGED_REL=$(printf "%s\n" "$CHANGED_REL" | sed '/^$/d' || true)
+          echo "Normalized changes:\n$CHANGED_REL"
+
+          # Prefer explicitly changed test files
+          TEST_FILES=$(printf "%s\n" "$CHANGED_REL" | grep -E 'src/.*\.(test|spec)\.(ts|tsx|js|jsx)' || true)
+          if [ -n "$TEST_FILES" ]; then
+            TF=$(echo "$TEST_FILES" | tr '\n' ' ')
+            echo "Found test files: $TF"
+            echo "tests=$TF" >> $GITHUB_OUTPUT
+            exit 0
+          fi
+
+          # Otherwise, map changed source dirs to test globs
+          DIRS=$(printf "%s\n" "$CHANGED_REL" | grep -E '^src/' || true)
+          if [ -n "$DIRS" ]; then
+            PATTERNS=$(printf "%s\n" "$DIRS" | sed -E 's|/[^/]+$||' | sort -u | awk '{print $0"/**/*.test.* "$0"/**/*.spec.*"}' | tr '\n' ' ')
+            echo "Derived patterns: $PATTERNS"
+            echo "tests=$PATTERNS" >> $GITHUB_OUTPUT
+            exit 0
+          fi
+
+          echo "No relevant tests found to run"
+          echo "tests=none" >> $GITHUB_OUTPUT
+
       - name: Run Vitest (renderer main)
         working-directory: app/renderer/src/main
         env:
           CI: true
-        run: yarn test:vitest -- --run --reporter verbose
+        run: |
+          TESTS="${{ steps.test_selection.outputs.tests }}"
+          if [ "$TESTS" = "none" ]; then
+            echo "No relevant tests changed, skipping vitest"
+            exit 0
+          fi
+          echo "Running tests: $TESTS"
+          yarn test:vitest -- $TESTS -- --run --reporter verbose

app/renderer/src/main/src/__tests__/playground/Vitest__Test__/utils.test.tsx

@@ -0,0 +1,31 @@
+import {describe, it, expect} from "vitest"
+import {compareKnowledgeBaseChange} from "@/components/playground/Vitest__Test__/utils"
+
+describe("compareKnowledgeBaseChange", () => {
+    it("returns true when prev or next is not an array", () => {
+        expect(compareKnowledgeBaseChange(null, [])).toBe(true)
+        expect(compareKnowledgeBaseChange([], undefined)).toBe(true)
+    })
+
+    it("detects a deleted item", () => {
+        const prev = [{ID: "1", name: "a"}, {ID: "2", name: "b"}]
+        const next = [{ID: "2", name: "b"}]
+
+        const res = compareKnowledgeBaseChange(prev as any, next as any)
+        expect(res).toEqual({delete: prev[0], increase: null})
+    })
+
+    it("detects an increased item", () => {
+        const prev = [{ID: "1", name: "a"}]
+        const next = [{ID: "1", name: "a"}, {ID: "2", name: "b"}]
+
+        const res = compareKnowledgeBaseChange(prev as any, next as any)
+        expect(res).toEqual({delete: null, increase: next[1]})
+    })
+
+    it("returns true when there is no change", () => {
+        const prev = [{ID: "1"}, {ID: "2"}]
+        const next = [{ID: "1"}, {ID: "2"}]
+        expect(compareKnowledgeBaseChange(prev as any, next as any)).toBe(true)
+    })
+})

app/renderer/src/main/src/components/playground/Vitest__Test__/index.tsx

@@ -1,4 +1,3 @@
-import {OperationRecordRes} from "@/components/yakitUI/YakitEditor/YakitEditorType"
 import {YakitSpin} from "@/components/yakitUI/YakitSpin/YakitSpin"
 import {getRemoteValue} from "@/utils/kv"
 import {useSafeState} from "ahooks"
@@ -11,47 +10,30 @@ interface VitestTestProps {
 const Vitest__Test__: FC<VitestTestProps> = (props) => {
     const {editorOperationRecord} = props
     const [typeLoading, setTypeLoading] = useSafeState<boolean>(true)
-    const [fontSize, setFontSize] = useSafeState<undefined | number>(12)
-    const [showLineBreaks, setShowLineBreaks] = useSafeState<boolean>(true)
-    const [noWordwrap, setNoWordwrap] = useSafeState(false)
 
     useEffect(() => {
-        if (!editorOperationRecord) {
-            setTypeLoading(false)
-            return
-        }
         setTypeLoading(true)
-        getRemoteValue(editorOperationRecord)
-            .then((data) => {
-                try {
-                    setTypeLoading(false)
-                    if (!data) return
-                    let obj: OperationRecordRes = JSON.parse(data)
-                    if (obj?.fontSize) {
-                        setFontSize(obj?.fontSize)
-                    }
-                    if (typeof obj?.showBreak === "boolean") {
-                        setShowLineBreaks(obj?.showBreak)
+        if (editorOperationRecord) {
+            getRemoteValue(editorOperationRecord)
+                .then((data) => {
+                    try {
+                        setTypeLoading(false)
+                        if (!data) return
+                    } catch (error) {
+                        setTypeLoading(false)
+                        fail(error + "")
                     }
-                    if (typeof obj?.noWordWrap === "boolean") {
-                        setNoWordwrap(obj?.noWordWrap)
-                    }
-                } catch (error) {
+                })
+                .finally(() => {
                     setTypeLoading(false)
-                    fail(error + "")
-                }
-            })
-            .finally(() => {
-                setTypeLoading(false)
-            })
+                })
+        } else {
+            setTypeLoading(false)
+        }
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [editorOperationRecord])
 
-    return (
-        <YakitSpin spinning={typeLoading}>
-            <div>Vitest__Test__</div>
-        </YakitSpin>
-    )
+    return <YakitSpin spinning={typeLoading}>vitest test page</YakitSpin>
 }
 
 export {Vitest__Test__}

app/renderer/src/main/src/components/playground/Vitest__Test__/utils.ts

@@ -0,0 +1,30 @@
+import {KnowledgeBaseItem} from "@/pages/KnowledgeBase/hooks/useKnowledgeBase"
+
+/**
+ * 对比两个知识库数组，判断新增或删除
+ * @param prev 上一次的数据
+ * @param next 当前的数据
+ */
+const compareKnowledgeBaseChange = (
+    prev: KnowledgeBaseItem[] | null | undefined,
+    next: KnowledgeBaseItem[] | null | undefined
+): {delete: KnowledgeBaseItem | null; increase: KnowledgeBaseItem | null} | true => {
+    // 如果任意一方为空，则无法比较，直接返回 true（表示无变化或无法判断）
+    if (!Array.isArray(prev) || !Array.isArray(next)) return true
+
+    const prevMap = new Map(prev.map((item) => [item.ID, item]))
+    const nextMap = new Map(next.map((item) => [item.ID, item]))
+
+    // 查找被删除的对象
+    const deleted = prev.find((item) => !nextMap.has(item.ID))
+    if (deleted) return {delete: deleted, increase: null}
+
+    // 查找新增的对象
+    const increased = next.find((item) => !prevMap.has(item.ID))
+    if (increased) return {delete: null, increase: increased}
+
+    // 没有变化
+    return true
+}
+
+export {compareKnowledgeBaseChange}