Improve error handling and robustness of validation scripts

- Use split() instead of replace() for version parsing
- Add better error handling with specific error types
- Improve error messages in CI workflows with fallbacks

Co-authored-by: longsizhuo <114939201+longsizhuo@users.noreply.github.com>

Author: Copilot

.github/workflows/content-check.yml

@@ -54,8 +54,12 @@ jobs:
           if ! git diff --exit-code pnpm-lock.yaml; then
             echo "❌ Error: pnpm-lock.yaml was modified after install"
             echo "This indicates a pnpm version mismatch or corrupted lockfile"
-            echo "Expected pnpm version: $(node -p 'require(\"./package.json\").packageManager')"
-            echo "Actual pnpm version: $(pnpm --version)"
+            echo ""
+            echo "Expected pnpm version from package.json:"
+            node -p "require('./package.json').packageManager || 'not specified'" || echo "Could not read package.json"
+            echo ""
+            echo "Actual pnpm version:"
+            pnpm --version || echo "pnpm not found"
             exit 1
           fi
           echo "✅ Lockfile is consistent"

.github/workflows/deploy.yml

@@ -43,8 +43,12 @@ jobs:
           if ! git diff --exit-code pnpm-lock.yaml; then
             echo "❌ Error: pnpm-lock.yaml was modified after install"
             echo "This indicates a pnpm version mismatch or corrupted lockfile"
-            echo "Expected pnpm version: $(node -p 'require(\"./package.json\").packageManager')"
-            echo "Actual pnpm version: $(pnpm --version)"
+            echo ""
+            echo "Expected pnpm version from package.json:"
+            node -p "require('./package.json').packageManager || 'not specified'" || echo "Could not read package.json"
+            echo ""
+            echo "Actual pnpm version:"
+            pnpm --version || echo "pnpm not found"
             exit 1
           fi
           echo "✅ Lockfile is consistent"

scripts/check-lockfile.mjs

@@ -58,7 +58,13 @@ try {
   process.exit(0);
 
 } catch (error) {
-  console.error('❌ Error checking lockfile:', error.message);
-  // Don't fail the commit, just warn
+  if (error.code === 'ENOENT') {
+    console.error('❌ Error: Git repository or command not found');
+  } else if (error.message.includes('not a git repository')) {
+    console.log('⚠️  Not in a git repository, skipping lockfile check');
+  } else {
+    console.error('❌ Error checking lockfile:', error.message);
+  }
+  // Don't fail the commit, just provide information
   process.exit(0);
 }

scripts/check-pnpm-version.mjs

@@ -30,7 +30,12 @@ try {
     process.exit(1);
   }
 
-  const expectedVersion = packageManager.replace('pnpm@', '');
+  const expectedVersion = packageManager.split('@')[1];
+  
+  if (!expectedVersion) {
+    console.error('❌ Error: Could not parse pnpm version from packageManager field');
+    process.exit(1);
+  }
 
   // Get actual pnpm version
   let actualVersion;