@timmo001/oxlint-rules 0.1.4 → 0.2.1

package/dist/configs/effect.js

@@ -102,11 +102,67 @@ var timmoEffectPlugin = eslintCompatPlugin({
 });
 var effect_default = timmoEffectPlugin;
 
-// vendor/anti-slop/src/index.ts
+// vendor/anti-slop/src/effect/index.ts
 import { eslintCompatPlugin as eslintCompatPlugin2 } from "@oxlint/plugins";
 
-// vendor/anti-slop/src/rules/no-chained-type-assertions.ts
+// vendor/anti-slop/src/effect/rules/no-service-constructor-imports.ts
 import { defineRule as defineRule2 } from "@oxlint/plugins";
+var SERVICE_CONSTRUCTOR_NAME = /^make[A-Z]/u;
+var TEST_FILE = /\.(?:test|spec)\.[cm]?[jt]sx?$/u;
+function isProjectLocalImport(source) {
+  return source.startsWith("./") || source.startsWith("../");
+}
+function getImportedName(specifier) {
+  if (specifier.imported.type === "Identifier")
+    return specifier.imported.name;
+  return specifier.imported.value;
+}
+var noServiceConstructorImportsRule = defineRule2({
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Disallow project-local make<CapabilityName> imports outside test and spec files."
+    },
+    messages: {
+      serviceConstructorImport: 'Do not import Effect service constructor "{{name}}" into runtime code. Import the owning Layer, yield the contextual service, and allow its requirements to propagate to the composition root.'
+    }
+  },
+  create(context) {
+    const isTestFile = TEST_FILE.test(context.filename.replaceAll("\\", "/"));
+    return {
+      ImportDeclaration(node) {
+        if (isTestFile || !isProjectLocalImport(node.source.value))
+          return;
+        for (const specifier of node.specifiers) {
+          if (specifier.type !== "ImportSpecifier")
+            continue;
+          const importedName = getImportedName(specifier);
+          if (!SERVICE_CONSTRUCTOR_NAME.test(importedName))
+            continue;
+          context.report({
+            node: specifier,
+            messageId: "serviceConstructorImport",
+            data: { name: importedName }
+          });
+        }
+      }
+    };
+  }
+});
+
+// vendor/anti-slop/src/effect/index.ts
+var antiSlopEffectPlugin = eslintCompatPlugin2({
+  meta: { name: "anti-slop-effect" },
+  rules: {
+    "no-service-constructor-imports": noServiceConstructorImportsRule
+  }
+});
+var effect_default2 = antiSlopEffectPlugin;
+// vendor/anti-slop/src/index.ts
+import { eslintCompatPlugin as eslintCompatPlugin3 } from "@oxlint/plugins";
+
+// vendor/anti-slop/src/rules/no-chained-type-assertions.ts
+import { defineRule as defineRule3 } from "@oxlint/plugins";
 function isTypeAssertionExpression(node) {
   return node.type === "TSAsExpression" || node.type === "TSTypeAssertion";
 }
@@ -141,7 +197,7 @@ function isForbiddenAssertionChain(node) {
   }
   return assertionCount > 1 && hasNonConstAssertion;
 }
-var noChainedTypeAssertionsRule = defineRule2({
+var noChainedTypeAssertionsRule = defineRule3({
   meta: {
     type: "problem",
     docs: {
@@ -165,7 +221,7 @@ var noChainedTypeAssertionsRule = defineRule2({
 });
 
 // vendor/anti-slop/src/rules/no-conditional-empty-object-spread.ts
-import { defineRule as defineRule3 } from "@oxlint/plugins";
+import { defineRule as defineRule4 } from "@oxlint/plugins";
 function unwrapParentheses(node) {
   let current = node;
   while (current.type === "ParenthesizedExpression") {
@@ -180,7 +236,7 @@ function isConditionalEmptyObjectSpread(node) {
   const conditional = unwrapParentheses(node);
   return conditional.type === "ConditionalExpression" && (isEmptyObjectExpression(conditional.consequent) || isEmptyObjectExpression(conditional.alternate));
 }
-var noConditionalEmptyObjectSpreadRule = defineRule3({
+var noConditionalEmptyObjectSpreadRule = defineRule4({
   meta: {
     type: "suggestion",
     docs: {
@@ -204,7 +260,7 @@ var noConditionalEmptyObjectSpreadRule = defineRule3({
 });
 
 // vendor/anti-slop/src/rules/no-known-value-widening.ts
-import { defineRule as defineRule4 } from "@oxlint/plugins";
+import { defineRule as defineRule5 } from "@oxlint/plugins";
 
 // vendor/anti-slop/src/shared/lexical-type-parameters.ts
 function isNode(value) {
@@ -903,7 +959,7 @@ function isDictionaryAccumulatorTarget(destination) {
 function hasParentAssertion(node) {
   return node.parent?.type === "TSAsExpression" || node.parent?.type === "TSTypeAssertion";
 }
-var noKnownValueWideningRule = defineRule4({
+var noKnownValueWideningRule = defineRule5({
   meta: {
     type: "problem",
     docs: {
@@ -1016,7 +1072,7 @@ var noKnownValueWideningRule = defineRule4({
 });
 
 // vendor/anti-slop/src/rules/no-module-mocking.ts
-import { defineRule as defineRule5 } from "@oxlint/plugins";
+import { defineRule as defineRule6 } from "@oxlint/plugins";
 var moduleMockMethods = new Set(["doMock", "mock", "unstable_mockModule"]);
 function resolveVariable3(sourceCode, identifier) {
   let scope = sourceCode.getScope(identifier);
@@ -1061,7 +1117,7 @@ function moduleMockCall(sourceCode, callee) {
   const method = callee.computed ? property.type === "Literal" && (property.value === "doMock" || property.value === "mock" || property.value === "unstable_mockModule") ? property.value : null : property.type === "Identifier" ? property.name : null;
   return method !== null && moduleMockMethods.has(method);
 }
-var noModuleMockingRule = defineRule5({
+var noModuleMockingRule = defineRule6({
   meta: {
     type: "problem",
     docs: {
@@ -1085,8 +1141,8 @@ var noModuleMockingRule = defineRule5({
 });
 
 // vendor/anti-slop/src/rules/no-object-parameters.ts
-import { defineRule as defineRule6 } from "@oxlint/plugins";
-var noObjectParametersRule = defineRule6({
+import { defineRule as defineRule7 } from "@oxlint/plugins";
+var noObjectParametersRule = defineRule7({
   meta: {
     type: "problem",
     docs: {
@@ -1139,7 +1195,7 @@ var noObjectParametersRule = defineRule6({
 });
 
 // vendor/anti-slop/src/rules/no-reflect-apply.ts
-import { defineRule as defineRule7 } from "@oxlint/plugins";
+import { defineRule as defineRule8 } from "@oxlint/plugins";
 
 // vendor/anti-slop/src/shared/reflect-method.ts
 function resolveVariable4(sourceCode, identifier) {
@@ -1170,7 +1226,7 @@ function isGlobalReflectMethodCall(sourceCode, callee, methodName) {
 }
 
 // vendor/anti-slop/src/rules/no-reflect-apply.ts
-var noReflectApplyRule = defineRule7({
+var noReflectApplyRule = defineRule8({
   meta: {
     type: "problem",
     docs: {
@@ -1194,8 +1250,8 @@ var noReflectApplyRule = defineRule7({
 });
 
 // vendor/anti-slop/src/rules/no-reflect-get.ts
-import { defineRule as defineRule8 } from "@oxlint/plugins";
-var noReflectGetRule = defineRule8({
+import { defineRule as defineRule9 } from "@oxlint/plugins";
+var noReflectGetRule = defineRule9({
   meta: {
     type: "problem",
     docs: {
@@ -1219,7 +1275,7 @@ var noReflectGetRule = defineRule8({
 });
 
 // vendor/anti-slop/src/rules/no-runtime-typeof.ts
-import { defineRule as defineRule9 } from "@oxlint/plugins";
+import { defineRule as defineRule10 } from "@oxlint/plugins";
 function isRuntimeFunction(node) {
   return node.type === "ArrowFunctionExpression" || node.type === "FunctionDeclaration" || node.type === "FunctionExpression";
 }
@@ -1242,7 +1298,7 @@ function isExistenceProbe(node) {
   const other = parent.left === node ? parent.right : parent.left;
   return other.type === "Literal" && other.value === "undefined";
 }
-var noRuntimeTypeofRule = defineRule9({
+var noRuntimeTypeofRule = defineRule10({
   meta: {
     type: "problem",
     docs: {
@@ -1276,7 +1332,7 @@ var noRuntimeTypeofRule = defineRule9({
 });
 
 // vendor/anti-slop/src/rules/no-shape-in-symbol-names.ts
-import { defineRule as defineRule10 } from "@oxlint/plugins";
+import { defineRule as defineRule11 } from "@oxlint/plugins";
 var FORBIDDEN_SYMBOL_NAME = "shape";
 function containsForbiddenSymbolName(name) {
   return name.toLowerCase().includes(FORBIDDEN_SYMBOL_NAME);
@@ -1287,7 +1343,7 @@ function isBorrowedMemberName(node) {
     return false;
   return parent.property === node && parent.computed === false;
 }
-var noForbiddenTermInSymbolNamesRule = defineRule10({
+var noForbiddenTermInSymbolNamesRule = defineRule11({
   meta: {
     type: "problem",
     docs: {
@@ -1316,12 +1372,12 @@ var noForbiddenTermInSymbolNamesRule = defineRule10({
 });
 
 // vendor/anti-slop/src/rules/no-unknown-parameters.ts
-import { defineRule as defineRule11 } from "@oxlint/plugins";
+import { defineRule as defineRule12 } from "@oxlint/plugins";
 function isTypePredicateSubject(owner, parameterName) {
   const predicate = owner.returnType?.typeAnnotation;
   return predicate?.type === "TSTypePredicate" && predicate.parameterName.type === "Identifier" && predicate.parameterName.name === parameterName;
 }
-var noUnknownParametersRule = defineRule11({
+var noUnknownParametersRule = defineRule12({
   meta: {
     type: "problem",
     docs: {
@@ -1365,8 +1421,8 @@ var noUnknownParametersRule = defineRule11({
 });
 
 // vendor/anti-slop/src/rules/no-unknown-returns.ts
-import { defineRule as defineRule12 } from "@oxlint/plugins";
-var noUnknownReturnsRule = defineRule12({
+import { defineRule as defineRule13 } from "@oxlint/plugins";
+var noUnknownReturnsRule = defineRule13({
   meta: {
     type: "problem",
     docs: {
@@ -1419,8 +1475,8 @@ var noUnknownReturnsRule = defineRule12({
 });
 
 // vendor/anti-slop/src/rules/no-unknown-type-aliases.ts
-import { defineRule as defineRule13 } from "@oxlint/plugins";
-var noUnknownTypeAliasesRule = defineRule13({
+import { defineRule as defineRule14 } from "@oxlint/plugins";
+var noUnknownTypeAliasesRule = defineRule14({
   meta: {
     type: "problem",
     docs: {
@@ -1458,7 +1514,7 @@ var noUnknownTypeAliasesRule = defineRule13({
 });
 
 // vendor/anti-slop/src/rules/no-unsafe-dictionary-type.ts
-import { defineRule as defineRule14 } from "@oxlint/plugins";
+import { defineRule as defineRule15 } from "@oxlint/plugins";
 var typeNodeKinds = new Set([
   "JSDocNonNullableType",
   "JSDocNullableType",
@@ -1545,7 +1601,7 @@ function shouldReportType(node, environment) {
   }
   return true;
 }
-var noUnsafeDictionaryTypeRule = defineRule14({
+var noUnsafeDictionaryTypeRule = defineRule15({
   meta: {
     type: "problem",
     docs: {
@@ -1587,7 +1643,7 @@ var noUnsafeDictionaryTypeRule = defineRule14({
 });
 
 // vendor/anti-slop/src/rules/no-widen-then-assert.ts
-import { defineRule as defineRule15 } from "@oxlint/plugins";
+import { defineRule as defineRule16 } from "@oxlint/plugins";
 var functionBoundaryTypes = new Set([
   "ArrowFunctionExpression",
   "FunctionDeclaration",
@@ -1783,7 +1839,7 @@ function assertionIsNarrower(sourceText, broadKind, evidence, assertedType) {
     return isDefinitelyObjectType(assertedType);
   return isDefinitelyNarrowerRecordType(assertedType);
 }
-var noWidenThenAssertRule = defineRule15({
+var noWidenThenAssertRule = defineRule16({
   meta: {
     type: "problem",
     docs: {
@@ -1823,7 +1879,7 @@ var noWidenThenAssertRule = defineRule15({
 });
 
 // vendor/anti-slop/src/rules/require-safety-comment-for-type-assertion.ts
-import { defineRule as defineRule16 } from "@oxlint/plugins";
+import { defineRule as defineRule17 } from "@oxlint/plugins";
 var DEFAULT_SAFETY_MARKERS = ["SAFETY"];
 var commentOwnerKinds = new Set([
   "ExpressionStatement",
@@ -1866,7 +1922,7 @@ function hasSafetyComment(sourceCode, node, pattern) {
     current = current.parent;
   }
 }
-var requireSafetyCommentForTypeAssertionRule = defineRule16({
+var requireSafetyCommentForTypeAssertionRule = defineRule17({
   meta: {
     type: "problem",
     docs: {
@@ -1916,7 +1972,7 @@ var requireSafetyCommentForTypeAssertionRule = defineRule16({
 });
 
 // vendor/anti-slop/src/index.ts
-var antiSlopPlugin = eslintCompatPlugin2({
+var antiSlopPlugin = eslintCompatPlugin3({
   meta: { name: "anti-slop" },
   rules: {
     "no-chained-type-assertions": noChainedTypeAssertionsRule,
@@ -1937,62 +1993,74 @@ var antiSlopPlugin = eslintCompatPlugin2({
   }
 });
 var src_default = antiSlopPlugin;
-// vendor/anti-slop/src/effect/index.ts
-import { eslintCompatPlugin as eslintCompatPlugin3 } from "@oxlint/plugins";
+// src/generic/index.ts
+import { eslintCompatPlugin as eslintCompatPlugin4 } from "@oxlint/plugins";
 
-// vendor/anti-slop/src/effect/rules/no-service-constructor-imports.ts
-import { defineRule as defineRule17 } from "@oxlint/plugins";
-var SERVICE_CONSTRUCTOR_NAME = /^make[A-Z]/u;
-var TEST_FILE = /\.(?:test|spec)\.[cm]?[jt]sx?$/u;
-function isProjectLocalImport(source) {
-  return source.startsWith("./") || source.startsWith("../");
+// src/generic/rules/prefer-event-parameter-type.ts
+import { defineRule as defineRule18 } from "@oxlint/plugins";
+function nearestEnclosingFunction2(node) {
+  let current = node.parent;
+  while (current) {
+    if (current.type === "FunctionDeclaration" || current.type === "FunctionExpression" || current.type === "ArrowFunctionExpression") {
+      return current;
+    }
+    current = current.parent;
+  }
+  return null;
 }
-function getImportedName(specifier) {
-  if (specifier.imported.type === "Identifier")
-    return specifier.imported.name;
-  return specifier.imported.value;
+function assertedEventParameter(node) {
+  const expression = node.expression;
+  if (expression.type !== "MemberExpression" || expression.computed || expression.object.type !== "Identifier" || expression.property.type !== "Identifier" || expression.property.name !== "currentTarget" && expression.property.name !== "target") {
+    return null;
+  }
+  const parameterName = expression.object.name;
+  const property = expression.property.name;
+  const owner = nearestEnclosingFunction2(node);
+  if (!owner?.params.some((parameter) => parameter.type === "Identifier" && parameter.name === parameterName)) {
+    return null;
+  }
+  return {
+    parameter: parameterName,
+    property
+  };
 }
-var noServiceConstructorImportsRule = defineRule17({
+var preferEventParameterTypeRule = defineRule18({
   meta: {
-    type: "problem",
+    type: "suggestion",
     docs: {
-      description: "Disallow project-local make<CapabilityName> imports outside test and spec files."
+      description: "Prefer typing event target properties in the handler parameter instead of asserting them at use sites."
     },
     messages: {
-      serviceConstructorImport: 'Do not import Effect service constructor "{{name}}" into runtime code. Import the owning Layer, yield the contextual service, and allow its requirements to propagate to the composition root.'
+      typeEventParameter: "Type `{{parameter}}.{{property}}` in the function signature instead of asserting it at the use site."
     }
   },
   create(context) {
-    const isTestFile = TEST_FILE.test(context.filename.replaceAll("\\", "/"));
+    const checkAssertion = (node) => {
+      const eventParameter = assertedEventParameter(node);
+      if (!eventParameter)
+        return;
+      context.report({
+        node,
+        messageId: "typeEventParameter",
+        data: eventParameter
+      });
+    };
     return {
-      ImportDeclaration(node) {
-        if (isTestFile || !isProjectLocalImport(node.source.value))
-          return;
-        for (const specifier of node.specifiers) {
-          if (specifier.type !== "ImportSpecifier")
-            continue;
-          const importedName2 = getImportedName(specifier);
-          if (!SERVICE_CONSTRUCTOR_NAME.test(importedName2))
-            continue;
-          context.report({
-            node: specifier,
-            messageId: "serviceConstructorImport",
-            data: { name: importedName2 }
-          });
-        }
-      }
+      TSAsExpression: checkAssertion,
+      TSTypeAssertion: checkAssertion
     };
   }
 });
 
-// vendor/anti-slop/src/effect/index.ts
-var antiSlopEffectPlugin = eslintCompatPlugin3({
-  meta: { name: "anti-slop-effect" },
+// src/generic/index.ts
+var timmoPlugin = eslintCompatPlugin4({
+  meta: { name: "timmo" },
   rules: {
-    "no-service-constructor-imports": noServiceConstructorImportsRule
+    "prefer-event-parameter-type": preferEventParameterTypeRule
   }
 });
-var effect_default2 = antiSlopEffectPlugin;
+var generic_default = timmoPlugin;
+
 // node_modules/oxlint/dist/index.js
 function defineConfig(config) {
   return config;
@@ -2009,9 +2077,13 @@ var recommended = defineConfig({
     {
       name: "anti-slop",
       specifier: "@timmo001/oxlint-rules/upstream/anti-slop"
-    }
+    },
+    { name: "timmo", specifier: "@timmo001/oxlint-rules/generic" }
   ],
-  rules: enablePluginRules("anti-slop", src_default)
+  rules: {
+    ...enablePluginRules("anti-slop", src_default),
+    ...enablePluginRules("timmo", generic_default)
+  }
 });
 var recommended_default = recommended;
 

package/dist/configs/recommended.js

@@ -1833,6 +1833,74 @@ var antiSlopPlugin = eslintCompatPlugin({
   }
 });
 var src_default = antiSlopPlugin;
+// src/generic/index.ts
+import { eslintCompatPlugin as eslintCompatPlugin2 } from "@oxlint/plugins";
+
+// src/generic/rules/prefer-event-parameter-type.ts
+import { defineRule as defineRule16 } from "@oxlint/plugins";
+function nearestEnclosingFunction(node) {
+  let current = node.parent;
+  while (current) {
+    if (current.type === "FunctionDeclaration" || current.type === "FunctionExpression" || current.type === "ArrowFunctionExpression") {
+      return current;
+    }
+    current = current.parent;
+  }
+  return null;
+}
+function assertedEventParameter(node) {
+  const expression = node.expression;
+  if (expression.type !== "MemberExpression" || expression.computed || expression.object.type !== "Identifier" || expression.property.type !== "Identifier" || expression.property.name !== "currentTarget" && expression.property.name !== "target") {
+    return null;
+  }
+  const parameterName = expression.object.name;
+  const property = expression.property.name;
+  const owner = nearestEnclosingFunction(node);
+  if (!owner?.params.some((parameter) => parameter.type === "Identifier" && parameter.name === parameterName)) {
+    return null;
+  }
+  return {
+    parameter: parameterName,
+    property
+  };
+}
+var preferEventParameterTypeRule = defineRule16({
+  meta: {
+    type: "suggestion",
+    docs: {
+      description: "Prefer typing event target properties in the handler parameter instead of asserting them at use sites."
+    },
+    messages: {
+      typeEventParameter: "Type `{{parameter}}.{{property}}` in the function signature instead of asserting it at the use site."
+    }
+  },
+  create(context) {
+    const checkAssertion = (node) => {
+      const eventParameter = assertedEventParameter(node);
+      if (!eventParameter)
+        return;
+      context.report({
+        node,
+        messageId: "typeEventParameter",
+        data: eventParameter
+      });
+    };
+    return {
+      TSAsExpression: checkAssertion,
+      TSTypeAssertion: checkAssertion
+    };
+  }
+});
+
+// src/generic/index.ts
+var timmoPlugin = eslintCompatPlugin2({
+  meta: { name: "timmo" },
+  rules: {
+    "prefer-event-parameter-type": preferEventParameterTypeRule
+  }
+});
+var generic_default = timmoPlugin;
+
 // node_modules/oxlint/dist/index.js
 function defineConfig(config) {
   return config;
@@ -1849,9 +1917,13 @@ var recommended = defineConfig({
     {
       name: "anti-slop",
       specifier: "@timmo001/oxlint-rules/upstream/anti-slop"
-    }
+    },
+    { name: "timmo", specifier: "@timmo001/oxlint-rules/generic" }
   ],
-  rules: enablePluginRules("anti-slop", src_default)
+  rules: {
+    ...enablePluginRules("anti-slop", src_default),
+    ...enablePluginRules("timmo", generic_default)
+  }
 });
 var recommended_default = recommended;
 export {

package/dist/generic/index.js

@@ -0,0 +1,70 @@
+// src/generic/index.ts
+import { eslintCompatPlugin } from "@oxlint/plugins";
+
+// src/generic/rules/prefer-event-parameter-type.ts
+import { defineRule } from "@oxlint/plugins";
+function nearestEnclosingFunction(node) {
+  let current = node.parent;
+  while (current) {
+    if (current.type === "FunctionDeclaration" || current.type === "FunctionExpression" || current.type === "ArrowFunctionExpression") {
+      return current;
+    }
+    current = current.parent;
+  }
+  return null;
+}
+function assertedEventParameter(node) {
+  const expression = node.expression;
+  if (expression.type !== "MemberExpression" || expression.computed || expression.object.type !== "Identifier" || expression.property.type !== "Identifier" || expression.property.name !== "currentTarget" && expression.property.name !== "target") {
+    return null;
+  }
+  const parameterName = expression.object.name;
+  const property = expression.property.name;
+  const owner = nearestEnclosingFunction(node);
+  if (!owner?.params.some((parameter) => parameter.type === "Identifier" && parameter.name === parameterName)) {
+    return null;
+  }
+  return {
+    parameter: parameterName,
+    property
+  };
+}
+var preferEventParameterTypeRule = defineRule({
+  meta: {
+    type: "suggestion",
+    docs: {
+      description: "Prefer typing event target properties in the handler parameter instead of asserting them at use sites."
+    },
+    messages: {
+      typeEventParameter: "Type `{{parameter}}.{{property}}` in the function signature instead of asserting it at the use site."
+    }
+  },
+  create(context) {
+    const checkAssertion = (node) => {
+      const eventParameter = assertedEventParameter(node);
+      if (!eventParameter)
+        return;
+      context.report({
+        node,
+        messageId: "typeEventParameter",
+        data: eventParameter
+      });
+    };
+    return {
+      TSAsExpression: checkAssertion,
+      TSTypeAssertion: checkAssertion
+    };
+  }
+});
+
+// src/generic/index.ts
+var timmoPlugin = eslintCompatPlugin({
+  meta: { name: "timmo" },
+  rules: {
+    "prefer-event-parameter-type": preferEventParameterTypeRule
+  }
+});
+var generic_default = timmoPlugin;
+export {
+  generic_default as default
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timmo001/oxlint-rules",
-  "version": "0.1.4",
+  "version": "0.2.1",
   "description": "Shared Oxlint plugins and configs with optional Effect rules.",
   "type": "module",
   "license": "Apache-2.0",
@@ -14,6 +14,7 @@
   "exports": {
     "./upstream/anti-slop": "./dist/upstream/anti-slop.js",
     "./upstream/effect": "./dist/upstream/effect.js",
+    "./generic": "./dist/generic/index.js",
     "./effect": "./dist/effect/index.js",
     "./configs/recommended": "./dist/configs/recommended.js",
     "./configs/effect": "./dist/configs/effect.js"
@@ -21,6 +22,8 @@
   "files": [
     "dist",
     "src/effect",
+    "src/generic",
+    "!src/generic/**/*.test.ts",
     "!src/effect/**/*.test.ts",
     "vendor/anti-slop/src",
     "vendor/anti-slop/LICENSE",
@@ -37,7 +40,7 @@
     "format": "prettier --write \"{src,scripts}/**/*.ts\" \"skills/**/*.{md,mjs}\" \"*.md\" \"*.json\"",
     "format:check": "prettier --check \"{src,scripts}/**/*.ts\" \"skills/**/*.{md,mjs}\" \"*.md\" \"*.json\"",
     "lint": "oxlint src scripts",
-    "skills:validate": "bunx skills-ref validate skills/install-timmo-oxlint-rules && bunx skills-ref validate skills/add-oxlint-rule",
+    "skills:validate": "bunx skills-ref validate skills/install-timmo-oxlint-rules && bunx skills-ref validate skills/add-oxlint-rule && bunx skills-ref validate skills/release-oxlint-rules",
     "test": "node --experimental-strip-types scripts/test.ts",
     "typecheck": "tsc --noEmit"
   },

package/skills/add-oxlint-rule/SKILL.md

@@ -23,9 +23,13 @@ description: >-
    separately when Dylan Mulroy's plugin should own the behaviour.
 6. Register the rule in its plugin. Verify the matching config and copy command
    discover it from the plugin's `rules` map, then update the README rule list
-   and behaviour description. Do not add rule names to either skill.
+   and behaviour description. Keep skills workflow-only; do not duplicate rule
+   or plugin inventories or counts in them.
 7. Run `mise run check`, `mise run build`, `npm pack --dry-run`, and
-   `bunx jsr@0.14.3 publish --dry-run` in the central checkout.
+   `bunx jsr@0.14.3 publish --dry-run --allow-dirty` in the central checkout.
+8. If the user also asks to publish the change, load `release-oxlint-rules` and
+   hand off the release after the rule checks pass. Do not duplicate or bypass
+   its release workflow.
 
 Report the fixture contract, registration and docs changed, checks, and any
 consumer rollout left for a separate stage. Do not publish or assume a local

package/skills/install-timmo-oxlint-rules/SKILL.md

@@ -23,9 +23,9 @@ description: >-
 2. Detect Bun, npm, pnpm, or Yarn from the target's package manager declaration
    and lockfile. Add an exact development dependency through that package
    manager.
-3. Extend `@timmo001/oxlint-rules/configs/recommended`. Use `/configs/effect`
-   instead only when `effect` is a direct dependency or the user explicitly
-   requests it.
+3. Extend `@timmo001/oxlint-rules/configs/recommended`. The config owns plugin
+   registration and recommended severities. Use `/configs/effect` instead only
+   when `effect` is a direct dependency or the user explicitly requests it.
 4. Keep dependency and config edits visible. Do not delegate them to a script.
 
 ## Copy rules
@@ -36,11 +36,11 @@ description: >-
    meaningful differences before asking whether replacement is intended.
 3. Run `node scripts/copy.mjs <bun|npm|pnpm|yarn> <destination>`. Add `--force`
    only after explicit replacement approval.
-4. Register the three entry points printed by the command as `anti-slop`,
-   `anti-slop-effect`, and `timmo-effect`. Merge every printed general rule
-   setting into the target config. Merge every printed Effect rule setting only
-   for direct Effect use or an explicit request. Do not maintain a rule list in
-   this skill.
+4. Register every plugin entry point printed by the command. Merge every printed
+   general rule setting into the target config. Merge every printed Effect rule
+   setting only for direct Effect use or an explicit request. Treat the command
+   output as authoritative rather than maintaining plugin or rule inventories
+   in this skill.
 
 Run the target repository's normal lint, typecheck, tests, and build. Report
 package-manager changes, preserved local configuration, enabled rule groups,