@yasainet/eslint 0.0.21 → 0.0.23

package/README.md

@@ -111,11 +111,3 @@ git push --tags
 ```
 
 3. GitHub Actions will automatically publish to npm
-
-### With lazygit
-
-1. Stage and commit in lazygit
-2. Select the commit, press `T` to create a tag (e.g. `v1.0.0`)
-3. Press `P` to push the commit
-4. Switch to the Tags panel (`]`), select the tag, and press `P` to push it
-5. GitHub Actions will automatically publish to npm

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yasainet/eslint",
-  "version": "0.0.21",
+  "version": "0.0.23",
   "description": "ESLint",
   "type": "module",
   "exports": {

package/src/common/constants.mjs

@@ -22,12 +22,12 @@ const PROJECT_ROOT = findProjectRoot();
 
 const EXCLUDE_LIST = ["proxy.lib.ts"];
 
-/** @description Extract the base name from a .ts filename by stripping all extensions. */
+/** Extract the base name from a .ts filename by stripping all extensions. */
 function baseName(filename) {
   return filename.replace(/\..*$/, "");
 }
 
-/** @description Scan lib directory derived from featureRoot and build prefix-to-lib-relative-path mapping */
+/** Scan lib directory derived from featureRoot and build prefix-to-lib-relative-path mapping. */
 export function generatePrefixLibMapping(featureRoot) {
   const libRoot = featureRoot.replace(/features$/, "lib");
   const libDir = path.join(PROJECT_ROOT, libRoot);
@@ -67,7 +67,7 @@ export function generatePrefixLibMapping(featureRoot) {
   return mapping;
 }
 
-/** @description Build glob patterns scoped to the given feature root */
+/** Build glob patterns scoped to the given feature root. */
 export const featuresGlob = (featureRoot, subpath) => [
   `${featureRoot}/${subpath}`,
 ];

package/src/common/imports.mjs

@@ -110,7 +110,7 @@ function makeConfig(name, files, ...patternArrays) {
   };
 }
 
-/** @description Next.js-only: restrict @/lib imports to repositories */
+/** Next.js-only: restrict @/lib imports to repositories. */
 export const libBoundaryConfigs = [
   {
     name: "imports/lib-boundary",
@@ -128,7 +128,7 @@ export const libBoundaryConfigs = [
   },
 ];
 
-/** @description Scope import restriction rules to the given feature root */
+/** Scope import restriction rules to the given feature root. */
 export function createImportsConfigs(
   featureRoot,
   prefixLibMapping,

package/src/common/index.mjs

@@ -5,7 +5,7 @@ import { createLayersConfigs } from "./layers.mjs";
 import { createNamingConfigs } from "./naming.mjs";
 import { rulesConfigs } from "./rules.mjs";
 
-/** @description Build common configs scoped to the given feature root */
+/** Build common configs scoped to the given feature root. */
 export function createCommonConfigs(
   featureRoot,
   { banAliasImports = false } = {},

package/src/common/jsdoc.mjs

@@ -2,7 +2,7 @@ import jsdocPlugin from "eslint-plugin-jsdoc";
 
 import { featuresGlob } from "./constants.mjs";
 
-/** @description Scope JSDoc rules to the given feature root */
+/** Scope JSDoc rules to the given feature root. */
 export function createJsdocConfigs(featureRoot) {
   return [
     {

package/src/common/layers.mjs

@@ -1,4 +1,4 @@
-/** @description Scope layer rules to the given feature root */
+/** Scope layer rules to the given feature root. */
 export function createLayersConfigs(featureRoot) {
   const loggerSelector = "CallExpression[callee.object.name='logger']";
   const loggerMessage =

package/src/common/local-plugins/action-handle-service.mjs

@@ -1,6 +1,4 @@
-/**
- * @description In *.action.ts, `handleXxx` must call the service method `*.xxx()`.
- */
+/** In *.action.ts, handleXxx must call the service method *.xxx(). */
 export const actionHandleServiceRule = {
   meta: {
     type: "problem",

package/src/common/local-plugins/import-path-style.mjs

@@ -1,7 +1,7 @@
 import path from "path";
 
 /**
- * @description Enforce import path style within features:
+ * Enforce import path style within features:
  * - Same-feature imports must use relative paths
  * - Cross-feature imports must use @/ alias
  */
@@ -37,11 +37,9 @@ export const importPathStyleRule = {
     const featureName = afterRoot.split("/")[0];
     if (!featureName) return {};
 
-    // Absolute path of the current feature directory
     const featureDir =
       filename.slice(0, rootIdx + rootSep.length) + featureName;
 
-    // Alias prefix for same-feature: @/features/{featureName}
     const aliasBase = featureRoot.replace(/^src\//, "");
     const sameFeaturePrefix = `@/${aliasBase}/${featureName}/`;
     const sameFeatureExact = `@/${aliasBase}/${featureName}`;
@@ -50,7 +48,7 @@ export const importPathStyleRule = {
       if (!source || typeof source.value !== "string") return;
       const importPath = source.value;
 
-      // Case 1: @/features/{same-feature}/... → use relative path
+      // Same-feature alias should be a relative path for consistency.
       if (
         importPath.startsWith(sameFeaturePrefix) ||
         importPath === sameFeatureExact
@@ -63,7 +61,7 @@ export const importPathStyleRule = {
         return;
       }
 
-      // Case 2: relative path that exits current feature → use @/
+      // Cross-feature relative path should use @/ alias for clarity.
       if (importPath.startsWith(".")) {
         const resolved = path.resolve(path.dirname(filename), importPath);
         if (

package/src/common/local-plugins/index.mjs

@@ -1,7 +1,7 @@
 import { actionHandleServiceRule } from "./action-handle-service.mjs";
 import { importPathStyleRule } from "./import-path-style.mjs";
 
-/** @description Shared local plugin object to avoid ESLint "Cannot redefine plugin" errors */
+/** Single plugin object to avoid ESLint "Cannot redefine plugin" errors. */
 export const localPlugin = {
   rules: {
     "action-handle-service": actionHandleServiceRule,

package/src/common/naming.mjs

@@ -2,7 +2,7 @@ import { featuresGlob } from "./constants.mjs";
 import { localPlugin } from "./local-plugins/index.mjs";
 import { checkFile } from "./plugins.mjs";
 
-/** @description Scope naming rules to the given feature root */
+/** Scope naming rules to the given feature root. */
 export function createNamingConfigs(featureRoot, prefixLibMapping) {
   const prefixPattern = `@(${Object.keys(prefixLibMapping).join("|")})`;
   const sharedPrefixPattern = `@(shared|${Object.keys(prefixLibMapping).join("|")})`;
@@ -11,6 +11,7 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
     {
       name: "naming/services",
       files: featuresGlob(featureRoot, "**/services/*.ts"),
+      ignores: featuresGlob(featureRoot, "shared/services/*.ts"),
       plugins: { "check-file": checkFile },
       rules: {
         "check-file/filename-naming-convention": [
@@ -19,9 +20,21 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
         ],
       },
     },
+    {
+      name: "naming/services-shared",
+      files: featuresGlob(featureRoot, "shared/services/*.ts"),
+      plugins: { "check-file": checkFile },
+      rules: {
+        "check-file/filename-naming-convention": [
+          "error",
+          { "**/*.ts": `${sharedPrefixPattern}.service` },
+        ],
+      },
+    },
     {
       name: "naming/repositories",
       files: featuresGlob(featureRoot, "**/repositories/*.ts"),
+      ignores: featuresGlob(featureRoot, "shared/repositories/*.ts"),
       plugins: { "check-file": checkFile },
       rules: {
         "check-file/filename-naming-convention": [
@@ -30,6 +43,17 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
         ],
       },
     },
+    {
+      name: "naming/repositories-shared",
+      files: featuresGlob(featureRoot, "shared/repositories/*.ts"),
+      plugins: { "check-file": checkFile },
+      rules: {
+        "check-file/filename-naming-convention": [
+          "error",
+          { "**/*.ts": `${sharedPrefixPattern}.repo` },
+        ],
+      },
+    },
   ];
 
   configs.push(
@@ -59,12 +83,25 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
     },
     {
       name: "naming/schemas",
-      files: featuresGlob(featureRoot, "**/schemas/*.ts"),
+      files: featuresGlob(featureRoot, "*/schemas/*.schema.ts"),
+      ignores: featuresGlob(featureRoot, "shared/schemas/*.ts"),
       plugins: { "check-file": checkFile },
       rules: {
         "check-file/filename-naming-convention": [
           "error",
-          { "**/*.ts": `${prefixPattern}.schema` },
+          { "**/*/schemas/*.ts": "<1>" },
+          { ignoreMiddleExtensions: true },
+        ],
+      },
+    },
+    {
+      name: "naming/schemas-shared",
+      files: featuresGlob(featureRoot, "shared/schemas/*.ts"),
+      plugins: { "check-file": checkFile },
+      rules: {
+        "check-file/filename-naming-convention": [
+          "error",
+          { "**/*.ts": `${sharedPrefixPattern}.schema` },
         ],
       },
     },
@@ -119,6 +156,7 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
     {
       name: "naming/actions",
       files: featuresGlob(featureRoot, "**/actions/*.ts"),
+      ignores: featuresGlob(featureRoot, "shared/actions/*.ts"),
       plugins: { "check-file": checkFile },
       rules: {
         "check-file/filename-naming-convention": [
@@ -127,6 +165,17 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
         ],
       },
     },
+    {
+      name: "naming/actions-shared",
+      files: featuresGlob(featureRoot, "shared/actions/*.ts"),
+      plugins: { "check-file": checkFile },
+      rules: {
+        "check-file/filename-naming-convention": [
+          "error",
+          { "**/*.ts": `${sharedPrefixPattern}.action` },
+        ],
+      },
+    },
     {
       name: "naming/actions-export",
       files: featuresGlob(featureRoot, "**/actions/*.ts"),

package/src/common/plugins.mjs

@@ -3,7 +3,7 @@ import checkFile from "eslint-plugin-check-file";
 import jsdocPlugin from "eslint-plugin-jsdoc";
 import simpleImportSortPlugin from "eslint-plugin-simple-import-sort";
 
-/** @description Shared plugin instances used across ESLint configs */
+/** Shared plugin instances used across ESLint configs. */
 export const plugins = {
   "@stylistic": stylistic,
   "check-file": checkFile,

package/src/common/rules.mjs

@@ -2,7 +2,7 @@ import tseslint from "typescript-eslint";
 
 import { simpleImportSortPlugin, stylistic } from "./plugins.mjs";
 
-/** @description Base rule configs for code style and TypeScript checks */
+/** Base rule configs for code style and TypeScript checks. */
 export const rulesConfigs = [
   {
     name: "rules/shared",

package/src/deno/index.mjs

@@ -1,6 +1,6 @@
 import { createCommonConfigs } from "../common/index.mjs";
 
-/** @description Deno ESLint flat config entry point */
+/** Deno ESLint flat config entry point. */
 export const eslintConfig = [
   ...createCommonConfigs("supabase/functions/features", {
     banAliasImports: true,

package/src/next/directives.mjs

@@ -1,4 +1,4 @@
-/** @description Enforce "use server" / "use client" directives per file convention */
+/** Enforce "use server" / "use client" directives per file convention. */
 export const directivesConfigs = [
   {
     name: "directives/server-action",

package/src/next/imports.mjs

@@ -1,6 +1,6 @@
 import { localPlugin } from "../common/local-plugins/index.mjs";
 
-/** @description Next.js: enforce relative paths within same feature, @/ for cross-feature */
+/** Enforce relative paths within same feature, @/ for cross-feature in Next.js. */
 export const importPathStyleConfigs = [
   {
     name: "imports/path-style",

package/src/next/index.mjs

@@ -5,7 +5,7 @@ import { directivesConfigs } from "./directives.mjs";
 import { importPathStyleConfigs } from "./imports.mjs";
 import { namingConfigs } from "./naming.mjs";
 
-/** @description Next.js ESLint flat config entry point */
+/** Next.js ESLint flat config entry point. */
 export const eslintConfig = [
   ...createCommonConfigs("src/features"),
   ...libBoundaryConfigs,

package/src/next/naming.mjs

@@ -1,6 +1,6 @@
 import { checkFile } from "../common/plugins.mjs";
 
-/** @description Next.js-specific naming convention configs for hooks and components */
+/** Next.js-specific naming convention configs for hooks and components. */
 export const namingConfigs = [
   {
     name: "naming/hooks",

package/src/node/index.mjs

@@ -1,6 +1,6 @@
 import { createCommonConfigs } from "../common/index.mjs";
 
-/** @description Node.js ESLint flat config entry point */
+/** Node.js ESLint flat config entry point. */
 export const eslintConfig = [
   ...createCommonConfigs("scripts/features", { banAliasImports: true }),
 ];