⚙️ Add husky pre-push hook and fix e2e test code style

Author: CodFrm

.husky/pre-push

@@ -0,0 +1,35 @@
+#!/bin/sh
+
+# Skip checks: SKIP_PRE_PUSH=1 git push or git push --no-verify
+if [ "$SKIP_PRE_PUSH" = "1" ]; then
+  echo "⏭ SKIP_PRE_PUSH=1, skipping pre-push checks"
+  exit 0
+fi
+
+# Only run checks when pushing to main or release/* branches
+remote="$1"
+need_check=0
+
+while read local_ref local_sha remote_ref remote_sha; do
+  branch=$(echo "$remote_ref" | sed 's|refs/heads/||')
+
+  if [ "$branch" = "main" ] || echo "$branch" | grep -q "^release"; then
+    need_check=1
+    echo "🔍 Detected push target: $branch"
+  fi
+done
+
+if [ "$need_check" = "0" ]; then
+  exit 0
+fi
+
+echo ""
+echo "▶ Running lint..."
+pnpm run lint || exit 1
+
+echo ""
+echo "▶ Running tests..."
+pnpm run test:ci || exit 1
+
+echo ""
+echo "✅ All checks passed! (build and e2e tests will run in CI)"

e2e/gm-api.spec.ts

@@ -43,7 +43,7 @@ const test = base.extend<{
     });
     // Ensure service worker is registered before handing context to fixtures,
     // preventing extensionId fixture from timing out with the global 10s timeout.
-    let [sw] = context.serviceWorkers();
+    const [sw] = context.serviceWorkers();
     if (!sw) await context.waitForEvent("serviceworker", { timeout: 30_000 });
     await use(context);
     await context.close();

e2e/utils.ts

@@ -49,11 +49,9 @@ export async function installScriptByCode(context: BrowserContext, extensionId:
   await page.evaluate((text) => navigator.clipboard.writeText(text), code);
   await page.keyboard.press("ControlOrMeta+v");
   // Wait for Monaco to finish rendering the pasted content (content will differ from template)
-  await page.waitForFunction(
-    (init) => document.querySelector(".view-lines")?.textContent !== init,
-    initialText,
-    { timeout: 10_000 }
-  );
+  await page.waitForFunction((init) => document.querySelector(".view-lines")?.textContent !== init, initialText, {
+    timeout: 10_000,
+  });
   // Save
   await page.keyboard.press("ControlOrMeta+s");
   // Wait for save: try arco-message first, then verify via script list

package.json

@@ -7,6 +7,7 @@
   "private": true,
   "scripts": {
     "preinstall": "pnpm dlx only-allow pnpm",
+    "prepare": "husky",
     "test": "vitest --test-timeout=500 --no-coverage --isolate=false --reporter=verbose",
     "test:ci": "vitest run --test-timeout=500 --no-coverage --isolate=false --reporter=default --reporter.summary=false",
     "coverage": "vitest run --coverage",
@@ -84,6 +85,7 @@
     "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-userscripts": "^0.5.6",
     "globals": "^16.5.0",
+    "husky": "^9.1.7",
     "iconv-lite": "^0.7.2",
     "jsdom": "^26.1.0",
     "jszip": "^3.10.1",
@@ -104,4 +106,4 @@
     "**/*.scss",
     "**/*.less"
   ]
-}
+}