@mslmio/eslint-config 1.2.0 → 2.0.1

package/README.md

@@ -38,7 +38,7 @@ UI half behind `/ui` means the root entry resolves neither `@shadcn/lint` nor
 | `no-upward-import` | A package may not import a tier above it, or a sibling leaf. Off until configured — see below |
 | `no-property-shorthand` | Always `{ key: key }`; exempts destructuring and imports |
 | `no-lib-process-env` | A library never reads `process.env` — configuration is passed in |
-| `no-module-scope-init` | No client/config built at module scope; build lazily and memoise |
+| `no-module-scope-init` | No client/config built at module scope; build lazily and memoize |
 | `no-express-json-spaces` | Express's `json spaces` indents but drops the trailing newline |
 | `prefer-copy-icon` | lucide `Copy`, not `Clipboard` |
 | `us-spelling` | US spelling in prose and identifiers |
@@ -78,13 +78,6 @@ A file's namespace is its top-level directory under that root, so neither the pa
 checked out at nor the package's own name changes the verdict. Outside a workspace there is no root,
 and a scoped package's scope is its namespace.
 
-## Ignores
-
-`house`, and so every preset, globally ignores `.design-sync/**`, `.ds-sync/**` and
-`ds-bundle/**`. Flat config does not read `.gitignore`, and these hold symlinks into app
-components — one moved component leaves a dangling link and `eslint .` exits `ENOENT` instead of
-reporting anything.
-
 ## License
 
 MIT

package/index.js

@@ -4,13 +4,6 @@ import tsPlugin from "@typescript-eslint/eslint-plugin";
 
 const TS_GLOBS = ["**/*.ts", "**/*.tsx", "**/*.mts", "**/*.js", "**/*.mjs"];
 
-// design-sync scratch and its generated bundle. All three are gitignored, and `ds-src` holds
-// symlinks that dangle the moment a component moves - which makes `eslint .` exit on ENOENT
-// rather than report. Flat config does not read .gitignore, so every consumer needs this.
-export const designSyncIgnores = {
-    ignores: [".design-sync/**", ".ds-sync/**", "ds-bundle/**"],
-};
-
 // The @typescript-eslint rule set. Without this the parser parses TypeScript and
 // NOTHING lints it - no unused vars, no floating promises, no unsafe any. The rules
 // go in `house` (so every consumer gets them) while the PARSER stays out of it, since
@@ -42,7 +35,6 @@ typescriptRules.push({
 
 // Every rule here encodes a documented house rule. Adding one means editing
 export const house = [
-    designSyncIgnores,
     ...typescriptRules,
     {
         files: TS_GLOBS,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mslmio/eslint-config",
-  "version": "1.2.0",
+  "version": "2.0.1",
   "description": "Shareable ESLint flat config and house rules: US spelling, library hygiene, workspace layering, and Tailwind/React conventions.",
   "keywords": [
     "eslint",
@@ -9,7 +9,7 @@
     "eslint-plugin",
     "flat-config",
     "typescript",
-    "monorepo",
+    "workspace",
     "layering",
     "tailwind",
     "react"

package/plugin.js

@@ -9,7 +9,7 @@ import noUpwardImport from "./rules/no-upward-import.js";
 import preferTableFacets from "./rules/prefer-table-facets.js";
 
 export default {
-    meta: { name: "@mslmio/eslint-plugin", version: "2.5.0" },
+    meta: { name: "@mslmio/eslint-plugin", version: "2.0.1" },
     rules: {
         "no-lib-process-env": noLibProcessEnv,
         "no-module-scope-init": noModuleScopeInit,

package/rules/no-lib-process-env.js

@@ -28,8 +28,9 @@ export default {
     create(context) {
         const docs = docsSuffix(context);
         const entryPoints = context.options[0]?.entryPoints ?? DEFAULT_ENTRY_POINTS;
-        const filename = context.filename ?? context.getFilename();
-        if (entryPoints.some((e) => filename.endsWith(e))) {
+        const filename = (context.filename ?? context.getFilename()).replaceAll("\\", "/");
+        // Whole trailing path segments, so `main.ts` exempts `src/main.ts` and never `domain.ts`.
+        if (entryPoints.some((e) => `/${filename}`.endsWith(`/${e}`))) {
             return {};
         }
         return {

package/rules/no-module-scope-init.js

@@ -36,7 +36,7 @@ export default {
             moduleScopeInit:
                 "`{{name}}` is constructed at module scope, so it runs on import - before any caller " +
                 "can configure it, and in every process that merely imports this file. Build it lazily " +
-                "inside a function and memoise.{{docs}}",
+                "inside a function and memoize.{{docs}}",
         },
     },
     create(context) {