refactor(linter): move valid-title rule to shared and crate own rule for vitest

apps/oxlint/fixtures/cli/overrides_with_plugin/.oxlintrc.json

@@ -5,9 +5,9 @@
         "plugins": ["jest", "vitest"],
         "rules": {
           "jest/valid-title": "deny",
+          "vitest/valid-title": "deny",
           "no-unused-vars": "off"
         }
       }
     ]
   }
-

apps/oxlint/src/snapshots/fixtures__cli__issue_10394_-c .oxlintrc.json@oxlint.snap

@@ -14,7 +14,15 @@ working directory: fixtures/cli/issue_10394
    `----
   help: Write a meaningful title for your test
 
-Found 1 warning and 0 errors.
+  ! eslint-plugin-vitest(valid-title): Should not have an empty title
+   ,-[foo.test.ts:1:10]
+ 1 | describe("", () => {
+   :          ^^
+ 2 |   //
+   `----
+  help: Write a meaningful title for your test
+
+Found 2 warnings and 0 errors.
 Finished in <variable>ms on 1 file with 93 rules using 1 threads.
 ----------
 CLI result: LintSucceeded

apps/oxlint/src/snapshots/fixtures__cli__overrides_with_plugin_-c .oxlintrc.json@oxlint.snap

@@ -14,6 +14,14 @@ working directory: fixtures/cli/overrides_with_plugin
    `----
   help: Write a meaningful title for your test
 
+  x eslint-plugin-vitest(valid-title): Should not have an empty title
+   ,-[index.test.ts:1:10]
+ 1 | describe("", () => {
+   :          ^^
+ 2 |   // ^ jest/no-valid-title error as explicitly set in the `.test.ts` override
+   `----
+  help: Write a meaningful title for your test
+
   ! eslint-plugin-jest(expect-expect): Test has no assertions
    ,-[index.test.ts:4:3]
  3 | 
@@ -32,6 +40,15 @@ working directory: fixtures/cli/overrides_with_plugin
    `----
   help: Write a meaningful title for your test
 
+  x eslint-plugin-vitest(valid-title): Should not have an empty title
+   ,-[index.test.ts:4:6]
+ 3 | 
+ 4 |   it("", () => {});
+   :      ^^
+ 5 |   // ^ jest/no-valid-title error as explicitly set in the `.test.ts` override
+   `----
+  help: Write a meaningful title for your test
+
   ! eslint(no-unused-vars): Variable 'foo' is declared but never used. Unused variables should start with a '_'.
    ,-[index.ts:1:7]
  1 | const foo = 123;
@@ -41,7 +58,7 @@ working directory: fixtures/cli/overrides_with_plugin
    `----
   help: Consider removing this declaration.
 
-Found 2 warnings and 2 errors.
+Found 2 warnings and 4 errors.
 Finished in <variable>ms on 2 files with 93 rules using 1 threads.
 ----------
 CLI result: LintFoundErrors

crates/oxc_linter/src/rules.rs

@@ -717,6 +717,7 @@ pub(crate) mod vitest {
     pub mod prefer_to_be_object;
     pub mod prefer_to_be_truthy;
     pub mod require_local_test_context_for_concurrent_snapshots;
+    pub mod valid_title;
     pub mod warn_todo;
 }
 
@@ -749,6 +750,10 @@ pub(crate) mod vue {
     pub mod valid_define_props;
 }
 
+pub(crate) mod shared {
+    pub mod valid_title;
+}
+
 // Re-export RuleEnum, RULES, and all rule type aliases from generated code
 pub use crate::generated::rules_enum::*;