fix: address code review findings from CR session

- H-1: revert yarn cache strategy to actions/cache@v4 + hashFiles
  package.json since yarn.lock is gitignored; cache: yarn requires
  yarn.lock to generate a stable cache key
- H-2: add fetch-depth: 0 to sync-catalog.yml checkout step to ensure
  git-auto-commit-action@v5 works correctly on workflow_dispatch
- M-1: fix DEVELOPMENT.md cache description to match actual ci.yml impl
- M-2: change lychee upload-artifact if: always() to if: failure() to
  avoid spurious warnings when no broken links are found

Author: zhaojing1987

.github/workflows/ci.yml

@@ -60,7 +60,7 @@ jobs:
 
       - name: Upload lychee results
         uses: actions/upload-artifact@v4
-        if: always()
+        if: failure()
         with:
           name: lychee-report
           path: lychee/out.md
@@ -77,7 +77,14 @@ jobs:
     - uses: actions/setup-node@v4
       with:
         node-version: 20
-        cache: 'yarn'
+
+    - name: Cache node_modules
+      uses: actions/cache@v4
+      with:
+        path: node_modules
+        key: ${{ runner.os }}-node-${{ hashFiles('**/package.json') }}
+        restore-keys: |
+          ${{ runner.os }}-node-
 
     - name: Docusaurus build
       run: |

.github/workflows/sync-catalog.yml

@@ -15,6 +15,8 @@ jobs:
 
     steps:
       - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
 
       - name: Setup Python
         uses: actions/setup-python@v5

DEVELOPMENT.md

@@ -40,7 +40,7 @@ yarn run write-translations -- --locale zh-cn
 yarn upgrade @docusaurus/core@latest @docusaurus/preset-classic@latest
 ```
 
-> **注意**：`yarn.lock` 已在 `.gitignore` 中，**不提交**。CI 使用 `package.json` hash 作为缓存键。
+> **注意**：`yarn.lock` 已在 `.gitignore` 中，**不提交**。CI 通过 `actions/cache@v4` + `hashFiles('**/package.json')` 缓存 `node_modules/`。
 
 ---