@jterrazz/codestyle 1.2.2 → 1.2.4

package/README.md

@@ -12,7 +12,7 @@ Create `.oxlintrc.json`:
 
 ```json
 {
-    "extends": ["@jterrazz/codestyle/oxlint/node"]
+    "extends": ["node_modules/@jterrazz/codestyle/src/oxlint/node.json"]
 }
 ```
 
@@ -25,11 +25,12 @@ npx codestyle --fix    # Fix everything
 
 ## Configurations
 
-| Config                              | Use Case                          |
-| ----------------------------------- | --------------------------------- |
-| `@jterrazz/codestyle/oxlint/node`   | Node.js (requires .js extensions) |
-| `@jterrazz/codestyle/oxlint/expo`   | Expo / React Native               |
-| `@jterrazz/codestyle/oxlint/nextjs` | Next.js                           |
+| Config                                                                     | Use Case                           |
+| -------------------------------------------------------------------------- | ---------------------------------- |
+| `node_modules/@jterrazz/codestyle/src/oxlint/node.json`                    | Node.js (requires .js extensions)  |
+| `node_modules/@jterrazz/codestyle/src/oxlint/expo.json`                    | Expo / React Native                |
+| `node_modules/@jterrazz/codestyle/src/oxlint/nextjs.json`                  | Next.js                            |
+| `node_modules/@jterrazz/codestyle/src/oxlint/architectures/hexagonal.json` | Hexagonal architecture enforcement |
 
 ## What's Included
 
@@ -38,6 +39,7 @@ npx codestyle --fix    # Fix everything
 - Named exports enforcement (no default exports)
 - Performance warnings (spread in loops, etc.)
 - Style consistency (curly braces, comments, naming)
+- Custom rules: architecture boundaries, import extensions
 
 ## CLI
 
@@ -57,8 +59,8 @@ Enforce hexagonal architecture boundaries:
 ```json
 {
     "extends": [
-        "@jterrazz/codestyle/oxlint/node",
-        "@jterrazz/codestyle/oxlint/architectures/hexagonal"
+        "node_modules/@jterrazz/codestyle/src/oxlint/node.json",
+        "node_modules/@jterrazz/codestyle/src/oxlint/architectures/hexagonal.json"
     ]
 }
 ```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jterrazz/codestyle",
-  "version": "1.2.2",
+  "version": "1.2.4",
   "author": "Jean-Baptiste Terrazzoni <contact@jterrazz.com>",
   "bin": {
     "codestyle": "./src/codestyle.sh"
@@ -27,8 +27,7 @@
     "lint:fix": "oxlint --fix --ignore-pattern '**/fixtures/**' && oxfmt",
     "test": "vitest --run",
     "test:watch": "vitest",
-    "build": "# no build script",
-    "postinstall": "node src/postinstall.js"
+    "build": "# no build script"
   },
   "dependencies": {
     "@typescript/native-preview": "^7.0.0-dev",

package/src/oxlint/base.json

@@ -5,9 +5,9 @@
   "categories": {
     "correctness": "error",
     "suspicious": "error",
-    "perf": "warn",
+    "perf": "error",
     "pedantic": "off",
-    "style": "warn",
+    "style": "error",
     "nursery": "off"
   },
   "rules": {
@@ -80,21 +80,25 @@
     "no-continue": "off",
     "arrow-body-style": "off",
     "prefer-destructuring": "off",
+    "new-cap": "off",
+    "no-await-in-loop": "off",
     "typescript/consistent-type-definitions": "off",
     "typescript/array-type": "off",
+    "typescript/no-inferrable-types": "off",
     "unicorn/no-null": "off",
     "unicorn/no-array-sort": "off",
+    "import/no-unassigned-import": "off",
+    "import/no-named-default": "off",
 
     // ============================================
     // ENABLED - Code style enforcement
     // ============================================
 
-    "capitalized-comments": "warn",
-    "new-cap": "error",
+    "capitalized-comments": "error",
     "curly": "error",
     "no-nested-ternary": "error",
     "unicorn/catch-error-name": "error",
-    "unicorn/numeric-separators-style": "warn",
+    "unicorn/numeric-separators-style": "error",
 
     // ============================================
     // DISABLED - Conflicts with other rules

package/src/postinstall.js

@@ -1,106 +0,0 @@
-#!/usr/bin/env node
-
-import fs from "fs";
-import path from "path";
-import { fileURLToPath } from "url";
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-
-function findProjectRoot() {
-  let dir = process.cwd();
-  if (dir.includes("node_modules")) {
-    dir = dir.split("node_modules")[0];
-  }
-  return dir;
-}
-
-// Rule mappings based on extended configs
-const ruleMappings = [
-  {
-    pattern: "architectures/hexagonal",
-    rule: ["codestyle/arch-hexagonal", "error"],
-  },
-  {
-    pattern: "oxlint/node",
-    rule: ["codestyle/imports-with-ext", "error"],
-  },
-  {
-    pattern: "oxlint/expo",
-    rule: ["codestyle/imports-without-ext", "error"],
-  },
-  {
-    pattern: "oxlint/nextjs",
-    rule: ["codestyle/imports-without-ext", "error"],
-  },
-];
-
-function run() {
-  const projectRoot = findProjectRoot();
-  const configPath = path.join(projectRoot, ".oxlintrc.json");
-
-  if (!fs.existsSync(configPath)) {
-    return;
-  }
-
-  let config;
-  try {
-    config = JSON.parse(fs.readFileSync(configPath, "utf8"));
-  } catch {
-    console.log("[@jterrazz/codestyle] Could not parse .oxlintrc.json");
-    return;
-  }
-
-  const configStr = JSON.stringify(config);
-  const neededRules = new Map();
-  let needsPlugin = false;
-
-  for (const mapping of ruleMappings) {
-    if (configStr.includes(mapping.pattern)) {
-      neededRules.set(mapping.rule[0], mapping.rule[1]);
-      needsPlugin = true;
-    }
-  }
-
-  if (!needsPlugin) {
-    return;
-  }
-
-  let updated = false;
-  const updates = [];
-
-  // Add plugin
-  const pluginPath = "./node_modules/@jterrazz/codestyle/src/oxlint/plugins/codestyle.js";
-  config.jsPlugins = config.jsPlugins || [];
-  if (!config.jsPlugins.includes(pluginPath)) {
-    config.jsPlugins.push(pluginPath);
-    updated = true;
-    updates.push("  + jsPlugins: codestyle.js");
-  }
-
-  // Add rules
-  config.rules = config.rules || {};
-  for (const [ruleName, ruleLevel] of neededRules) {
-    if (!config.rules[ruleName]) {
-      config.rules[ruleName] = ruleLevel;
-      updated = true;
-      updates.push(`  + rule: ${ruleName}`);
-    }
-  }
-
-  if (!updated) {
-    return;
-  }
-
-  fs.writeFileSync(
-    configPath,
-    `${JSON.stringify(config, null, 2)}
-`,
-  );
-
-  console.log("[@jterrazz/codestyle] Updated .oxlintrc.json:");
-  for (const update of updates) {
-    console.log(update);
-  }
-}
-
-run();