feat(e2e): enable strict oxlint plugins and burn down violations

Add import, node, and promise plugins with suspicious-as-error categories,
promote Playwright and type-aware unsafe-* rules to error, and fix the
resulting violations across helpers, specs, and support code.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: schultzp2020

e2e-tests/oxlint.config.ts

@@ -1,6 +1,19 @@
 import { defineConfig } from "oxlint";
 
 export default defineConfig({
+  plugins: [
+    "eslint",
+    "typescript",
+    "unicorn",
+    "oxc",
+    "import",
+    "node",
+    "promise",
+  ],
+  categories: {
+    correctness: "error",
+    suspicious: "error",
+  },
   options: {
     typeAware: true,
     typeCheck: true,
@@ -16,6 +29,11 @@ export default defineConfig({
   rules: {
     "typescript/no-floating-promises": "error",
     "typescript/await-thenable": "error",
+    "typescript/no-unsafe-assignment": "error",
+    "typescript/no-unsafe-member-access": "error",
+    "typescript/no-unsafe-call": "error",
+    "typescript/no-unsafe-return": "error",
+    "typescript/strict-void-return": "error",
     "check-file/filename-naming-convention": [
       "error",
       {
@@ -33,29 +51,67 @@ export default defineConfig({
     ],
     "playwright/no-wait-for-timeout": "error",
     "playwright/no-force-option": "error",
-    "playwright/expect-expect": "warn",
+    "playwright/expect-expect": "error",
     "playwright/valid-expect": "error",
-    "playwright/prefer-native-locators": "warn",
+    "playwright/prefer-native-locators": "error",
     "playwright/no-raw-locators": [
-      "warn",
+      "error",
       {
         allowed: [],
       },
     ],
     "playwright/no-skipped-test": [
-      "warn",
+      "error",
       {
         allowConditional: true,
       },
     ],
   },
   overrides: [
+    {
+      files: ["playwright/e2e/auth-providers/**/*.spec.ts"],
+      rules: {
+        "typescript/strict-void-return": "off",
+      },
+    },
     {
       files: ["**/*.spec.ts", "**/*.test.ts", "playwright/**/*.ts"],
       rules: {
         "playwright/valid-title": "off",
         "playwright/valid-describe-callback": "off",
         "playwright/no-wait-for-selector": "off",
+        "playwright/expect-expect": [
+          "error",
+          {
+            assertFunctionNames: [
+              "expect",
+              "toPass",
+              "verifyHeading",
+              "verifyQuickAccess",
+              "verifyLink",
+              "verifyRowsInTable",
+              "verifyDivHasText",
+              "verifyComponentInCatalog",
+              "verifyParagraph",
+              "verifyText",
+              "verifyTextinCard",
+              "verifyVisitedCardContent",
+              "verifyAboutCardIsDisplayed",
+              "verifyPRStatisticsRendered",
+              "verifyPRRows",
+              "verifyPRRowsPerPage",
+              "registerExistingComponent",
+              "validateLog",
+              "validateLogEvent",
+              "validateRbacLogEvent",
+              "checkRbacResponse",
+              "verifyTextInSelector",
+              "verifyPartialTextInSelector",
+              "loginAsGuest",
+              "waitForTitle",
+            ],
+          },
+        ],
       },
     },
   ],