@markuplint/file-resolver 5.0.0-rc.0 → 5.0.0-rc.2

package/CHANGELOG.md

@@ -3,6 +3,22 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [5.0.0-rc.2](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.1...v5.0.0-rc.2) (2026-04-15)
+
+- feat(file-resolver)!: remove deprecated autoLoad parameter from resolveRules ([71cab2d](https://github.com/markuplint/markuplint/commit/71cab2d5fe5bedb3e28b162666cbad3d0a2773ff))
+
+### BREAKING CHANGES
+
+- The autoLoad parameter has been removed from resolveRules().
+  Rules are now always auto-loaded unconditionally.
+
+# [5.0.0-rc.1](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.0...v5.0.0-rc.1) (2026-03-27)
+
+### Bug Fixes
+
+- **file-resolver:** handle ERR_PACKAGE_PATH_NOT_EXPORTED in generalImport ([d563d60](https://github.com/markuplint/markuplint/commit/d563d6000c4c38be54c17ea015b0f26c80c5229c)), closes [#3516](https://github.com/markuplint/markuplint/issues/3516)
+- **file-resolver:** resolve package subpaths before import to avoid runtime-specific errors ([cb2b641](https://github.com/markuplint/markuplint/commit/cb2b641e25cd9de487d395b6f219dd8b60ecd306))
+
 # [5.0.0-rc.0](https://github.com/markuplint/markuplint/compare/v5.0.0-alpha.3...v5.0.0-rc.0) (2026-03-12)
 
 ### Bug Fixes

package/lib/auto-load-rules.d.ts

@@ -1,7 +1,4 @@
 import type { Ruleset, AnyMLRule } from '@markuplint/ml-core';
-/**
- * @deprecated
- */
 export declare function autoLoadRules(ruleset: Ruleset): Promise<{
     rules: AnyMLRule[];
     errors: unknown[];

package/lib/auto-load-rules.js

@@ -1,8 +1,5 @@
 import { MLRule } from '@markuplint/ml-core';
 const cache = new Map();
-/**
- * @deprecated
- */
 export async function autoLoadRules(ruleset) {
     const rules = [];
     const errors = [];

package/lib/config-provider.js

@@ -2,8 +2,7 @@ import path from 'node:path';
 import { mergeConfig } from '@markuplint/ml-config';
 import { ConfigParserError } from '@markuplint/parser-utils';
 import { InvalidSelectorError, createSelector } from '@markuplint/selector';
-import { nonNullableFilter, toNoEmptyStringArrayFromStringOrArray } from '@markuplint/shared';
-import { ConfigLoadError } from './config-load-error.js';
+import { nonNullableFilter, toNoEmptyStringArrayFromStringOrArray, ConfigLoadError } from '@markuplint/shared';
 import { load as loadConfig, search } from './cosmiconfig.js';
 import { log } from './debug.js';
 import { generalImport } from './general-import.js';

package/lib/cosmiconfig.d.ts

@@ -1,5 +1,5 @@
 import type { LoaderSync } from 'cosmiconfig';
-import { ConfigLoadError } from './config-load-error.js';
+import { ConfigLoadError } from '@markuplint/shared';
 type CosmiConfig = ReturnType<LoaderSync>;
 export declare function search<T = CosmiConfig>(filePath: string, cacheClear: boolean): Promise<{
     filePath: string;

package/lib/cosmiconfig.js

@@ -2,7 +2,7 @@ import path from 'node:path';
 import { ConfigParserError } from '@markuplint/parser-utils';
 import { cosmiconfig, defaultLoaders } from 'cosmiconfig';
 import { jsonc } from 'jsonc';
-import { ConfigLoadError } from './config-load-error.js';
+import { ConfigLoadError } from '@markuplint/shared';
 import { log } from './debug.js';
 const searchLog = log.extend('search');
 const jsoncLoader = (path, content) => {

package/lib/general-import.js

@@ -2,6 +2,7 @@ import fs from 'node:fs/promises';
 import { createRequire } from 'node:module';
 import path from 'node:path';
 import { pathToFileURL } from 'node:url';
+import { isFatalError } from '@markuplint/shared';
 import { resolve } from 'import-meta-resolve';
 import { log } from './debug.js';
 import { fromFileURL } from './path-utils.js';
@@ -14,6 +15,17 @@ export async function generalImport(name) {
     if (cache.has(name)) {
         return cache.get(name);
     }
+    // When the specifier is a package subpath (e.g., "@scope/pkg/file.json"),
+    // the package's exports map may not include it. Instead of relying on
+    // error codes (which differ between Node.js and vitest/vite), resolve
+    // the package directory upfront and convert to an absolute path.
+    const resolved = resolvePackageSubpath(name);
+    if (resolved) {
+        gLog('Resolved package subpath "%s" to "%s"', name, resolved);
+        const result = await generalImport(resolved);
+        cache.set(name, result);
+        return result;
+    }
     try {
         // Convert absolute paths to file:// URL format
         let importPath = name;
@@ -64,14 +76,110 @@ export async function generalImport(name) {
             }
         }
         if (path.isAbsolute(name) && name.endsWith('.json')) {
-            const file = await fs.readFile(name, 'utf8');
-            const json = JSON.parse(file);
-            gLogSuccess('Success by readFile("%s") and JSON.parse: %O', name, json);
-            cache.set(name, json);
-            return json;
+            try {
+                const file = await fs.readFile(name, 'utf8');
+                const json = JSON.parse(file);
+                gLogSuccess('Success by readFile("%s") and JSON.parse: %O', name, json);
+                cache.set(name, json);
+                return json;
+            }
+            catch (readError) {
+                gLogError('Error in readFile("%s"): %O', name, readError);
+            }
         }
         gLogError('Error in `import()`: %O', error);
         cache.set(name, null);
         return null;
     }
 }
+/**
+ * Detects a bare package subpath specifier (e.g., `@scope/pkg/file.json`)
+ * and resolves it to an absolute file path when the package's exports map
+ * does not include the subpath.
+ *
+ * Returns the absolute path if resolution succeeds, or `null` if:
+ * - The specifier is not a package subpath (absolute path, relative path, no subpath)
+ * - The package's exports map already includes the subpath (no bypass needed)
+ * - The package cannot be found at all
+ */
+function resolvePackageSubpath(name) {
+    if (path.isAbsolute(name) || name.startsWith('.')) {
+        return null;
+    }
+    const packageName = name.startsWith('@') ? name.split('/').slice(0, 2).join('/') : name.split('/')[0];
+    if (!packageName) {
+        return null;
+    }
+    const subpath = name.slice(packageName.length + 1);
+    if (!subpath) {
+        return null;
+    }
+    const pkgDir = resolvePackageDir(packageName);
+    if (!pkgDir) {
+        return null;
+    }
+    try {
+        const pkgJson = require(path.join(pkgDir, 'package.json'));
+        if (!pkgJson.exports) {
+            return null; // No exports map — import() should work fine
+        }
+        // If the exports map explicitly includes this subpath, let import() handle it
+        const exportKey = `./${subpath}`;
+        if (typeof pkgJson.exports === 'object' && !Array.isArray(pkgJson.exports) && exportKey in pkgJson.exports) {
+            return null;
+        }
+        // Subpath not in exports — resolve to absolute path to bypass the restriction
+        return path.join(pkgDir, subpath);
+    }
+    catch (error) {
+        if (isFatalError(error)) {
+            throw error;
+        }
+        return null;
+    }
+}
+/**
+ * Resolves the root directory of a package by name.
+ */
+function resolvePackageDir(packageName) {
+    // Try CJS require.resolve first — it can often resolve package.json
+    // even when the ESM exports map doesn't include it
+    try {
+        return path.dirname(require.resolve(`${packageName}/package.json`));
+    }
+    catch (error) {
+        if (isFatalError(error)) {
+            throw error;
+        }
+        // Fall through
+    }
+    // Fall back to import-meta-resolve to find the main entry, then walk up
+    try {
+        const mainUrl = resolve(packageName, import.meta.url);
+        const mainPath = mainUrl.startsWith('file:') ? fromFileURL(mainUrl) : mainUrl;
+        let dir = path.dirname(mainPath);
+        for (let i = 0; i < 10; i++) {
+            try {
+                require(path.join(dir, 'package.json'));
+                return dir;
+            }
+            catch (innerError) {
+                if (isFatalError(innerError)) {
+                    throw innerError;
+                }
+                const parent = path.dirname(dir);
+                if (parent === dir) {
+                    break; // Reached filesystem root
+                }
+                dir = parent;
+            }
+        }
+    }
+    catch (error) {
+        if (isFatalError(error)) {
+            throw error;
+        }
+        // Package not found at all
+    }
+    return null;
+}

package/lib/resolve-rules.d.ts

@@ -6,13 +6,6 @@ import type { AnyMLRule, Ruleset, Plugin } from '@markuplint/ml-core';
  * @param plugins - The resolved plugins that may provide custom rules
  * @param ruleset - The current ruleset (used for auto-loading)
  * @param importPreset - Whether to import the built-in preset rules from `@markuplint/rules`
- * @param autoLoad - Whether to auto-load rules referenced in the ruleset
  * @returns An array of all resolved MLRule instances
- *
- * @deprecated The `autoLoad` parameter is deprecated
- */
-export declare function resolveRules(plugins: readonly Plugin[], ruleset: Ruleset, importPreset: boolean, 
-/**
- * @deprecated
  */
-autoLoad: boolean): Promise<Readonly<AnyMLRule>[]>;
+export declare function resolveRules(plugins: readonly Plugin[], ruleset: Ruleset, importPreset: boolean): Promise<Readonly<AnyMLRule>[]>;

package/lib/resolve-rules.js

@@ -8,16 +8,9 @@ let cachedPresetRules = null;
  * @param plugins - The resolved plugins that may provide custom rules
  * @param ruleset - The current ruleset (used for auto-loading)
  * @param importPreset - Whether to import the built-in preset rules from `@markuplint/rules`
- * @param autoLoad - Whether to auto-load rules referenced in the ruleset
  * @returns An array of all resolved MLRule instances
- *
- * @deprecated The `autoLoad` parameter is deprecated
- */
-export async function resolveRules(plugins, ruleset, importPreset, 
-/**
- * @deprecated
  */
-autoLoad) {
+export async function resolveRules(plugins, ruleset, importPreset) {
     const rules = importPreset ? await importPresetRules() : [];
     for (const plugin of plugins) {
         if (!plugin.rules) {
@@ -31,11 +24,9 @@ autoLoad) {
             rules.push(rule);
         }
     }
-    if (autoLoad) {
-        const { rules: additionalRules } = await autoLoadRules(ruleset);
-        for (const rule of additionalRules) {
-            rules.push(rule);
-        }
+    const { rules: additionalRules } = await autoLoadRules(ruleset);
+    for (const rule of additionalRules) {
+        rules.push(rule);
     }
     // Clone
     return [...rules];

package/package.json

@@ -1,8 +1,12 @@
 {
 	"name": "@markuplint/file-resolver",
-	"version": "5.0.0-rc.0",
+	"version": "5.0.0-rc.2",
 	"description": "The file resolver of markuplint",
-	"repository": "git@github.com:markuplint/markuplint.git",
+	"repository": {
+		"type": "git",
+		"url": "https://github.com/markuplint/markuplint.git",
+		"directory": "packages/@markuplint/file-resolver"
+	},
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
 	"license": "MIT",
 	"engines": {
@@ -30,22 +34,22 @@
 		"@types/node": "24.5.1"
 	},
 	"dependencies": {
-		"@markuplint/html-parser": "5.0.0-rc.0",
-		"@markuplint/ml-ast": "5.0.0-rc.0",
-		"@markuplint/ml-config": "5.0.0-rc.0",
-		"@markuplint/ml-core": "5.0.0-rc.0",
-		"@markuplint/ml-spec": "5.0.0-rc.0",
-		"@markuplint/parser-utils": "5.0.0-rc.0",
-		"@markuplint/pretenders": "5.0.0-rc.0",
-		"@markuplint/selector": "5.0.0-rc.0",
-		"@markuplint/shared": "5.0.0-rc.0",
+		"@markuplint/html-parser": "5.0.0-rc.2",
+		"@markuplint/ml-ast": "5.0.0-rc.2",
+		"@markuplint/ml-config": "5.0.0-rc.2",
+		"@markuplint/ml-core": "5.0.0-rc.2",
+		"@markuplint/ml-spec": "5.0.0-rc.2",
+		"@markuplint/parser-utils": "5.0.0-rc.2",
+		"@markuplint/pretenders": "5.0.0-rc.2",
+		"@markuplint/selector": "5.0.0-rc.2",
+		"@markuplint/shared": "5.0.0-rc.2",
 		"cosmiconfig": "9.0.1",
 		"debug": "4.4.3",
 		"glob": "13.0.6",
 		"ignore": "7.0.5",
 		"import-meta-resolve": "4.2.0",
 		"jsonc": "2.0.0",
-		"minimatch": "10.2.4"
+		"minimatch": "10.2.5"
 	},
-	"gitHead": "ebf4d7cfca0c259aead3b292c6b8a202db4cd802"
+	"gitHead": "e43763858d9234c417053becc73dbd088c1e7ea6"
 }

package/lib/config-load-error.d.ts

@@ -1,6 +0,0 @@
-export declare class ConfigLoadError extends Error {
-    filePath: string;
-    name: string;
-    referrer: string;
-    constructor(message: string, filePath: string, referrer: string);
-}

package/lib/config-load-error.js

@@ -1,10 +0,0 @@
-export class ConfigLoadError extends Error {
-    filePath;
-    name = 'ConfigLoadError';
-    referrer;
-    constructor(message, filePath, referrer) {
-        super(message + ` in ${referrer}`);
-        this.filePath = filePath;
-        this.referrer = referrer;
-    }
-}