eslint-plugin-import-boundaries 0.2.0 → 0.2.2

package/README.md

@@ -116,8 +116,8 @@ Prevent violations of your architecture:
 {
   dir: 'application',
   alias: '@application',
-  allowImportsFrom: ['@domain'],  // Only allow imports from @domain
-  denyImportsFrom: ['@infrastructure'],  // Deny imports from @infrastructure (dependency inversion)
+  allowImportsFrom: ['@domain'],  // Only allow imports from @domain (deny-all by default)
+  // Note: denyImportsFrom is redundant here - anything not in allowImportsFrom is already denied
 }
 ```
 
@@ -151,7 +151,7 @@ Boundaries can be nested, and each boundary must explicitly declare its import r
       dir: 'interface',
       alias: '@interface',
       allowImportsFrom: ['@application', '@public-use-cases'],
-      denyImportsFrom: ['@use-cases'], // Can allow parent and specific child, but deny intermediate boundary
+      denyImportsFrom: ['@use-cases'], // Deny specific sub-boundary even though parent @application is allowed
     },
   ],
 }
@@ -159,11 +159,19 @@ Boundaries can be nested, and each boundary must explicitly declare its import r
 
 **Key behaviors:**
 
-- Each boundary must explicitly declare its rules (no inheritance)
-- Files in boundaries without rules resolve to their nearest ancestor with rules (or are rejected if no ancestor has rules)
+- Each boundary has rules: explicit (via `allowImportsFrom`/`denyImportsFrom`) or implicit "deny all" (if neither is specified)
+- Each boundary uses its own rules directly (no inheritance from parent boundaries)
 - Rules work the same regardless of nesting depth (flat rule checking)
 - You can selectively allow/deny specific nested boundaries
-- A boundary is considered to have rules if it defines `allowImportsFrom`, `denyImportsFrom`, or `allowTypeImportsFrom` (even if empty arrays)
+- Files resolve to their most specific boundary (longest matching path), which determines the rules to apply
+
+**Rule semantics:**
+
+- If both `allowImportsFrom` and `denyImportsFrom` exist: `allowImportsFrom` takes precedence (items in allow list are allowed even if also in deny list)
+- If only `allowImportsFrom`: deny-all by default (only items in allow list are allowed)
+- If only `denyImportsFrom`: allow-all by default (everything except deny list is allowed)
+- If neither: deny-all by default (strictest)
+- **Important**: When `allowImportsFrom` is specified, `denyImportsFrom` can deny specific sub-boundaries (e.g. deny `@utils` within allowed `@application`), but is otherwise redundant since anything not in the allow list is already denied by default. Note that this works recursively: It is possible to allow a boundary within a denied boundary within an allowed boundary, and so on.
 
 ### 4. Type-Only Imports
 
@@ -220,8 +228,8 @@ import { something } from "@application"; // When inside @application boundary
     {
       dir: 'application',
       alias: '@application',
-      allowImportsFrom: ['@domain'], // Application uses domain
-      denyImportsFrom: ['@infrastructure', '@interface', '@composition'], // Dependency inversion
+      allowImportsFrom: ['@domain'], // Application uses domain (deny-all by default)
+      // Note: denyImportsFrom is redundant here - those boundaries are already denied
     },
     {
       dir: 'infrastructure',

package/eslint-plugin-import-boundaries.js

@@ -87,18 +87,16 @@ function checkAliasSubpath(spec, boundaries) {
 	return { isSubpath: false };
 }
 /**
-* Resolve a file to the nearest boundary that has rules specified.
-* If no boundaries with rules are found, returns null.
+* Resolve a file/path to the nearest boundary (regardless of rules).
+* Used for target boundaries - returns the boundary if it exists, even without rules.
 *
 * @param filename - Absolute filename
 * @param boundaries - Array of all boundaries
-* @returns The nearest boundary with rules, or null if none found
+* @returns The nearest boundary, or null if none found
 */
-function resolveToSpecifiedBoundary(filename, boundaries) {
-	const specifiedBoundaries = boundaries.filter((b) => isInsideDir(b.absDir, filename)).filter((b) => b.allowImportsFrom !== void 0 || b.denyImportsFrom !== void 0 || b.allowTypeImportsFrom !== void 0);
-	if (specifiedBoundaries.length > 0) return specifiedBoundaries.sort((a, b) => b.absDir.length - a.absDir.length)[0];
-	const ancestors = boundaries.filter((b) => b.allowImportsFrom !== void 0 || b.denyImportsFrom !== void 0 || b.allowTypeImportsFrom !== void 0).filter((b) => isInsideDir(b.absDir, filename));
-	if (ancestors.length > 0) return ancestors.sort((a, b) => b.absDir.length - a.absDir.length)[0];
+function resolveToBoundary(filename, boundaries) {
+	const matchingBoundaries = boundaries.filter((b) => isInsideDir(b.absDir, filename));
+	if (matchingBoundaries.length > 0) return matchingBoundaries.sort((a, b) => b.absDir.length - a.absDir.length)[0];
 	return null;
 }
 /**
@@ -114,7 +112,7 @@ function getFileData(filename, boundaries) {
 	return {
 		isValid: true,
 		fileDir: path.dirname(filename),
-		fileBoundary: resolveToSpecifiedBoundary(filename, boundaries)
+		fileBoundary: resolveToBoundary(filename, boundaries)
 	};
 }
 
@@ -286,7 +284,7 @@ function calculateCorrectImportPath(rawSpec, fileDir, fileBoundary, boundaries,
 	".cjs"
 ]) {
 	const { targetAbs, targetDir } = resolveTargetPath(rawSpec, fileDir, boundaries, rootDir, cwd, barrelFileName, fileExtensions);
-	const targetBoundary = resolveToSpecifiedBoundary(targetAbs, boundaries);
+	const targetBoundary = resolveToBoundary(targetAbs, boundaries);
 	if (!fileBoundary || targetBoundary !== fileBoundary) {
 		if (targetBoundary) {
 			if (crossBoundaryStyle === "absolute") return path.join(rootDir, targetBoundary.dir).replace(/\\/g, "/");
@@ -371,7 +369,7 @@ function handleImport(options) {
 			}
 		}
 	}
-	const targetBoundary = resolveToSpecifiedBoundary(targetAbs, boundaries);
+	const targetBoundary = resolveToBoundary(targetAbs, boundaries);
 	if (!skipBoundaryRules && fileBoundary && targetBoundary && fileBoundary !== targetBoundary) {
 		const violation = checkBoundaryRules(fileBoundary, targetBoundary, boundaries, isTypeOnly);
 		if (violation) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-import-boundaries",
   "type": "module",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Enforce architectural boundaries with deterministic import paths",
   "author": "ClassicalMoser",
   "license": "ISC",
@@ -55,7 +55,7 @@
     "format:check": "prettier --check .",
     "validate:fix": "pnpm run lint:fix && pnpm run format && pnpm run typecheck",
     "validate:check": "pnpm run lint && pnpm run format:check && pnpm run typecheck",
-    "prepublishOnly": "npm run build && npm run test && npm run typecheck"
+    "prepublishOnly": "npm run build && npm run test && npm run validate:check"
   },
   "peerDependencies": {
     "eslint": ">=9.0.0"