@yasainet/eslint 0.0.14 → 0.0.16

package/README.md

@@ -100,22 +100,22 @@ export default [...eslintConfig];
 
 ## Release
 
-1. Update `version` in `package.json`
-2. Commit and push to `main`
-3. Create and push a tag:
+Version is derived from the Git tag. CI automatically sets `package.json` version before publishing.
+
+1. Commit and push to `main`
+2. Create and push a tag:
 
 ```sh
 git tag v1.0.0
 git push --tags
 ```
 
-4. GitHub Actions will automatically publish to npm
+3. GitHub Actions will automatically publish to npm
 
 ### With lazygit
 
-1. Update `version` in `package.json`
-2. Stage and commit in lazygit
-3. Select the commit, press `T` to create a tag (e.g. `v1.0.0`)
-4. Press `P` to push the commit
-5. Switch to the Tags panel (`]`), select the tag, and press `P` to push it
-6. GitHub Actions will automatically publish to npm
+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.14",
+  "version": "0.0.16",
   "description": "ESLint",
   "type": "module",
   "exports": {

package/src/common/layers.mjs

@@ -1,6 +1,24 @@
 /** @description Scope layer rules to the given feature root */
 export function createLayersConfigs(featureRoot) {
+  const loggerSelector = "CallExpression[callee.object.name='logger']";
+  const loggerMessage =
+    "logger is not allowed outside actions. Logging belongs in actions.";
+
   return [
+    // Logger/console: all features except actions
+    {
+      name: "layers/logger",
+      files: [`${featureRoot}/**/*.ts`],
+      ignores: [`${featureRoot}/**/actions/*.ts`],
+      rules: {
+        "no-console": "error",
+        "no-restricted-syntax": [
+          "error",
+          { selector: loggerSelector, message: loggerMessage },
+        ],
+      },
+    },
+    // Repositories: try-catch + if + logger
     {
       name: "layers/repositories",
       files: [`${featureRoot}/**/repositories/*.ts`],
@@ -17,9 +35,11 @@ export function createLayersConfigs(featureRoot) {
             message:
               "if statements are not allowed in repositories. Conditional logic belongs in services.",
           },
+          { selector: loggerSelector, message: loggerMessage },
         ],
       },
     },
+    // Services: try-catch + logger
     {
       name: "layers/services",
       files: [`${featureRoot}/**/services/*.ts`],
@@ -31,6 +51,7 @@ export function createLayersConfigs(featureRoot) {
             message:
               "try-catch is not allowed in services. Error handling belongs in actions.",
           },
+          { selector: loggerSelector, message: loggerMessage },
         ],
       },
     },

package/src/common/naming.mjs

@@ -51,7 +51,7 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
     rules: {
       "check-file/filename-naming-convention": [
         "error",
-        { "**/*.ts": "shared.service" },
+        { "**/*.ts": "@(shared).service" },
       ],
     },
   });
@@ -64,7 +64,7 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
     rules: {
       "check-file/filename-naming-convention": [
         "error",
-        { "**/*.ts": "shared.repo" },
+        { "**/*.ts": "@(shared).repo" },
       ],
     },
   });
@@ -159,7 +159,7 @@ export function createNamingConfigs(featureRoot, prefixLibMapping) {
       rules: {
         "check-file/filename-naming-convention": [
           "error",
-          { "**/*.ts": "shared.action" },
+          { "**/*.ts": "@(shared).action" },
         ],
       },
     },