feat(linter/plugins): oxlint export types

Author: overlookmotel

apps/oxlint/package.json

@@ -4,6 +4,7 @@
   "type": "module",
   "main": "dist/index.js",
   "bin": "dist/cli.js",
+  "types": "dist/index.d.ts",
   "scripts": {
     "build": "pnpm run build-napi-release && pnpm run build-js",
     "build-dev": "pnpm run build-napi && pnpm run build-js",
@@ -34,9 +35,13 @@
   "files": [
     "dist"
   ],
+  "dependencies": {
+    "@oxc-project/types": "workspace:^"
+  },
   "devDependencies": {
     "eslint": "^9.36.0",
     "execa": "^9.6.0",
+    "jiti": "^2.6.0",
     "tsdown": "0.15.1",
     "typescript": "catalog:",
     "vitest": "catalog:"

apps/oxlint/src-js/index.ts

@@ -2,6 +2,8 @@ import type { Context } from './plugins/context.ts';
 import type { CreateOnceRule, Plugin, Rule } from './plugins/load.ts';
 import type { BeforeHook, Visitor, VisitorWithHooks } from './plugins/types.ts';
 
+export type { Context, Plugin, Rule, Visitor, VisitorWithHooks };
+
 const { defineProperty, getPrototypeOf, hasOwn, setPrototypeOf, create: ObjectCreate } = Object;
 
 const dummyOptions: unknown[] = [],

apps/oxlint/test/fixtures/definePlugin/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-  "jsPlugins": ["./plugin.js"],
+  "jsPlugins": ["./plugin.ts"],
   "categories": { "correctness": "off" },
   "rules": {
     "define-plugin-plugin/create": "error",

apps/oxlint/test/fixtures/definePlugin/eslint.config.js

@@ -1,4 +1,4 @@
-import plugin from './plugin.js';
+import plugin from './plugin.ts';
 
 export default [
   {

apps/oxlint/test/fixtures/definePlugin/plugin.js renamed to apps/oxlint/test/fixtures/definePlugin/plugin.ts

@@ -1,5 +1,6 @@
 import { sep } from 'node:path';
 import { definePlugin } from '../../../dist/index.js';
+import type { Rule } from '../../../dist/index.js';
 
 // `loc` is required for ESLint
 const SPAN = {
@@ -14,10 +15,10 @@ const SPAN = {
 const DIR_PATH_LEN = import.meta.dirname.length + 1;
 
 const relativePath = sep === '/'
-  ? path => path.slice(DIR_PATH_LEN)
-  : path => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
+  ? (path: string) => path.slice(DIR_PATH_LEN)
+  : (path: string) => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
 
-const createRule = {
+const createRule: Rule = {
   create(context) {
     context.report({ message: `create body:\nthis === rule: ${this === createRule}`, node: SPAN });
 
@@ -36,16 +37,16 @@ const createRule = {
 // i.e. Oxlint calls `createOnce` directly, and not the `create` method that `defineRule` (via `definePlugin`)
 // adds to the rule.
 let createOnceCallCount = 0;
-const createOnceRule = {
+const createOnceRule: Rule = {
   createOnce(context) {
     createOnceCallCount++;
 
     // `fileNum` should be different for each file.
     // `identNum` should start at 1 for each file.
-    let fileNum = 0, identNum;
+    let fileNum = 0, identNum: number;
     // Note: Files are processed in unpredictable order, so `files/1.js` may be `fileNum` 1 or 2.
     // Therefore, collect all visits and check them in `after` hook of the 2nd file.
-    const visits = [];
+    const visits: { fileNum: number; identNum: number }[] = [];
 
     // `this` should be the rule object
     const topLevelThis = this;
@@ -105,7 +106,7 @@ const createOnceRule = {
 };
 
 // Tests that `before` hook returning `false` disables visiting AST for the file.
-const createOnceBeforeFalseRule = {
+const createOnceBeforeFalseRule: Rule = {
   createOnce(context) {
     return {
       before() {
@@ -138,7 +139,7 @@ const createOnceBeforeFalseRule = {
 
 // These 3 rules test that `createOnce` without `before` and `after` hooks works correctly.
 
-const createOnceBeforeOnlyRule = {
+const createOnceBeforeOnlyRule: Rule = {
   createOnce(context) {
     return {
       before() {
@@ -159,7 +160,7 @@ const createOnceBeforeOnlyRule = {
   },
 };
 
-const createOnceAfterOnlyRule = {
+const createOnceAfterOnlyRule: Rule = {
   createOnce(context) {
     return {
       Identifier(node) {
@@ -180,7 +181,7 @@ const createOnceAfterOnlyRule = {
   },
 };
 
-const createOnceNoHooksRule = {
+const createOnceNoHooksRule: Rule = {
   createOnce(context) {
     return {
       Identifier(node) {

apps/oxlint/test/fixtures/definePlugin_and_defineRule/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-  "jsPlugins": ["./plugin.js"],
+  "jsPlugins": ["./plugin.ts"],
   "categories": { "correctness": "off" },
   "rules": {
     "define-plugin-and-rule-plugin/create": "error",

apps/oxlint/test/fixtures/definePlugin_and_defineRule/eslint.config.js

@@ -1,4 +1,4 @@
-import plugin from './plugin.js';
+import plugin from './plugin.ts';
 
 export default [
   {

apps/oxlint/test/fixtures/definePlugin_and_defineRule/plugin.js renamed to apps/oxlint/test/fixtures/definePlugin_and_defineRule/plugin.ts

@@ -14,8 +14,8 @@ const SPAN = {
 const DIR_PATH_LEN = import.meta.dirname.length + 1;
 
 const relativePath = sep === '/'
-  ? path => path.slice(DIR_PATH_LEN)
-  : path => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
+  ? (path: string) => path.slice(DIR_PATH_LEN)
+  : (path: string) => path.slice(DIR_PATH_LEN).replace(/\\/g, '/');
 
 const createRule = defineRule({
   create(context) {
@@ -41,10 +41,10 @@ const createOnceRule = defineRule({
 
     // `fileNum` should be different for each file.
     // `identNum` should start at 1 for each file.
-    let fileNum = 0, identNum;
+    let fileNum = 0, identNum: number;
     // Note: Files are processed in unpredictable order, so `files/1.js` may be `fileNum` 1 or 2.
     // Therefore, collect all visits and check them in `after` hook of the 2nd file.
-    const visits = [];
+    const visits: { fileNum: number; identNum: number }[] = [];
 
     // `this` should be the rule object returned by `defineRule`
     const topLevelThis = this;

apps/oxlint/test/fixtures/defineRule/.oxlintrc.json

@@ -1,5 +1,5 @@
 {
-  "jsPlugins": ["./plugin.js"],
+  "jsPlugins": ["./plugin.ts"],
   "categories": { "correctness": "off" },
   "rules": {
     "define-rule-plugin/create": "error",

apps/oxlint/test/fixtures/defineRule/eslint.config.js

@@ -1,4 +1,4 @@
-import plugin from './plugin.js';
+import plugin from './plugin.ts';
 
 export default [
   {