@docusaurus/core 2.0.0-beta.15d451942 → 2.0.0-beta.16

package/lib/server/translations/translationsExtractor.js

@@ -1,22 +1,21 @@
 "use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.extractSourceCodeFileTranslations = exports.extractAllSourceCodeFileTranslations = exports.extractSiteSourceCodeTranslations = exports.globSourceCodeFilePaths = void 0;
-const tslib_1 = require("tslib");
 /**
  * Copyright (c) Facebook, Inc. and its affiliates.
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  */
-const fs_extra_1 = tslib_1.__importDefault(require("fs-extra"));
-const traverse_1 = tslib_1.__importDefault(require("@babel/traverse"));
-const generator_1 = tslib_1.__importDefault(require("@babel/generator"));
-const chalk_1 = tslib_1.__importDefault(require("chalk"));
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.extractSourceCodeFileTranslations = exports.extractAllSourceCodeFileTranslations = exports.extractSiteSourceCodeTranslations = exports.globSourceCodeFilePaths = void 0;
+const tslib_1 = require("tslib");
+const fs_extra_1 = (0, tslib_1.__importDefault)(require("fs-extra"));
+const traverse_1 = (0, tslib_1.__importDefault)(require("@babel/traverse"));
+const generator_1 = (0, tslib_1.__importDefault)(require("@babel/generator"));
+const logger_1 = (0, tslib_1.__importDefault)(require("@docusaurus/logger"));
 const core_1 = require("@babel/core");
-const lodash_1 = require("lodash");
-const path_1 = tslib_1.__importDefault(require("path"));
-const constants_1 = require("../../constants");
-const utils_1 = require("../utils");
+const path_1 = (0, tslib_1.__importDefault)(require("path"));
+const utils_1 = require("@docusaurus/utils");
+const utils_2 = require("../utils");
 // We only support extracting source code translations from these kind of files
 const TranslatableSourceCodeExtension = new Set([
     '.js',
@@ -24,7 +23,7 @@ const TranslatableSourceCodeExtension = new Set([
     '.ts',
     '.tsx',
     // TODO support md/mdx too? (may be overkill)
-    // need to compile the MDX to JSX first and remove frontmatter
+    // need to compile the MDX to JSX first and remove front matter
     // '.md',
     // '.mdx',
 ]);
@@ -32,13 +31,14 @@ function isTranslatableSourceCodePath(filePath) {
     return TranslatableSourceCodeExtension.has(path_1.default.extname(filePath));
 }
 function getSiteSourceCodeFilePaths(siteDir) {
-    return [path_1.default.join(siteDir, constants_1.SRC_DIR_NAME)];
+    return [path_1.default.join(siteDir, utils_1.SRC_DIR_NAME)];
 }
 function getPluginSourceCodeFilePaths(plugin) {
     var _a, _b, _c;
     // The getPathsToWatch() generally returns the js/jsx/ts/tsx/md/mdx file paths
-    // We can use this method as well to know which folders we should try to extract translations from
-    // Hacky/implicit, but do we want to introduce a new lifecycle method just for that???
+    // We can use this method as well to know which folders we should try to
+    // extract translations from. Hacky/implicit, but do we want to introduce a
+    // new lifecycle method just for that???
     const codePaths = (_b = (_a = plugin.getPathsToWatch) === null || _a === void 0 ? void 0 : _a.call(plugin)) !== null && _b !== void 0 ? _b : [];
     // We also include theme code
     const themePath = (_c = plugin.getThemePath) === null || _c === void 0 ? void 0 : _c.call(plugin);
@@ -48,28 +48,32 @@ function getPluginSourceCodeFilePaths(plugin) {
     return codePaths;
 }
 async function globSourceCodeFilePaths(dirPaths) {
-    const filePaths = await utils_1.safeGlobby(dirPaths);
+    const filePaths = await (0, utils_2.safeGlobby)(dirPaths);
     return filePaths.filter(isTranslatableSourceCodePath);
 }
 exports.globSourceCodeFilePaths = globSourceCodeFilePaths;
 async function getSourceCodeFilePaths(siteDir, plugins) {
     const sitePaths = getSiteSourceCodeFilePaths(siteDir);
     // The getPathsToWatch() generally returns the js/jsx/ts/tsx/md/mdx file paths
-    // We can use this method as well to know which folders we should try to extract translations from
-    // Hacky/implicit, but do we want to introduce a new lifecycle method for that???
-    const pluginsPaths = lodash_1.flatten(plugins.map(getPluginSourceCodeFilePaths));
+    // We can use this method as well to know which folders we should try to
+    // extract translations from. Hacky/implicit, but do we want to introduce a
+    // new lifecycle method for that???
+    const pluginsPaths = plugins.flatMap(getPluginSourceCodeFilePaths);
     const allPaths = [...sitePaths, ...pluginsPaths];
     return globSourceCodeFilePaths(allPaths);
 }
-async function extractSiteSourceCodeTranslations(siteDir, plugins, babelOptions) {
-    // Should we warn here if the same translation "key" is found in multiple source code files?
+async function extractSiteSourceCodeTranslations(siteDir, plugins, babelOptions, extraSourceCodeFilePaths = []) {
+    // Should we warn here if the same translation "key" is found in multiple
+    // source code files?
     function toTranslationFileContent(sourceCodeFileTranslations) {
-        return sourceCodeFileTranslations.reduce((acc, item) => {
-            return { ...acc, ...item.translations };
-        }, {});
+        return sourceCodeFileTranslations.reduce((acc, item) => ({ ...acc, ...item.translations }), {});
     }
     const sourceCodeFilePaths = await getSourceCodeFilePaths(siteDir, plugins);
-    const sourceCodeFilesTranslations = await extractAllSourceCodeFileTranslations(sourceCodeFilePaths, babelOptions);
+    const allSourceCodeFilePaths = [
+        ...sourceCodeFilePaths,
+        ...extraSourceCodeFilePaths,
+    ];
+    const sourceCodeFilesTranslations = await extractAllSourceCodeFileTranslations(allSourceCodeFilePaths, babelOptions);
     logSourceCodeFileTranslationsWarnings(sourceCodeFilesTranslations);
     return toTranslationFileContent(sourceCodeFilesTranslations);
 }
@@ -77,29 +81,31 @@ exports.extractSiteSourceCodeTranslations = extractSiteSourceCodeTranslations;
 function logSourceCodeFileTranslationsWarnings(sourceCodeFilesTranslations) {
     sourceCodeFilesTranslations.forEach(({ sourceCodeFilePath, warnings }) => {
         if (warnings.length > 0) {
-            console.warn(`Translation extraction warnings for file path=${sourceCodeFilePath}:\n- ${chalk_1.default.yellow(warnings.join('\n\n- '))}`);
+            logger_1.default.warn `Translation extraction warnings for file path=${sourceCodeFilePath}: ${warnings}`;
         }
     });
 }
 async function extractAllSourceCodeFileTranslations(sourceCodeFilePaths, babelOptions) {
-    return lodash_1.flatten(await Promise.all(sourceCodeFilePaths.map((sourceFilePath) => extractSourceCodeFileTranslations(sourceFilePath, babelOptions))));
+    return Promise.all(sourceCodeFilePaths.flatMap((sourceFilePath) => extractSourceCodeFileTranslations(sourceFilePath, babelOptions)));
 }
 exports.extractAllSourceCodeFileTranslations = extractAllSourceCodeFileTranslations;
 async function extractSourceCodeFileTranslations(sourceCodeFilePath, babelOptions) {
     try {
         const code = await fs_extra_1.default.readFile(sourceCodeFilePath, 'utf8');
-        const ast = core_1.parse(code, {
+        const ast = (0, core_1.parse)(code, {
             ...babelOptions,
             ast: true,
-            // filename is important, because babel does not process the same files according to their js/ts extensions
-            // see  see https://twitter.com/NicoloRibaudo/status/1321130735605002243
+            // filename is important, because babel does not process the same files
+            // according to their js/ts extensions.
+            // See https://twitter.com/NicoloRibaudo/status/1321130735605002243
             filename: sourceCodeFilePath,
         });
-        return await extractSourceCodeAstTranslations(ast, sourceCodeFilePath);
+        const translations = await extractSourceCodeAstTranslations(ast, sourceCodeFilePath);
+        return translations;
     }
-    catch (e) {
-        e.message = `Error while attempting to extract Docusaurus translations from source code file at path=${sourceCodeFilePath}\n${e.message}`;
-        throw e;
+    catch (err) {
+        logger_1.default.error `Error while attempting to extract Docusaurus translations from source code file at path=${sourceCodeFilePath}.`;
+        throw err;
     }
 }
 exports.extractSourceCodeFileTranslations = extractSourceCodeFileTranslations;
@@ -112,108 +118,143 @@ https://github.com/formatjs/formatjs/blob/main/packages/babel-plugin-formatjs/in
 https://github.com/pugjs/babel-walk
  */
 function extractSourceCodeAstTranslations(ast, sourceCodeFilePath) {
-    function staticTranslateJSXWarningPart() {
-        return 'Translate content could not be extracted.\nIt has to be a static string and use optional but static props, like <Translate id="my-id" description="my-description">text</Translate>.';
-    }
-    function sourceFileWarningPart(node) {
+    function sourceWarningPart(node) {
         var _a;
-        return `File=${sourceCodeFilePath} at line=${(_a = node.loc) === null || _a === void 0 ? void 0 : _a.start.line}`;
-    }
-    function generateCode(node) {
-        return generator_1.default(node).code;
+        return `File: ${sourceCodeFilePath} at line ${(_a = node.loc) === null || _a === void 0 ? void 0 : _a.start.line}
+Full code: ${(0, generator_1.default)(node).code}`;
     }
     const translations = {};
     const warnings = [];
-    // TODO we should check the presence of the correct @docusaurus imports here!
-    traverse_1.default(ast, {
-        JSXElement(path) {
-            if (!path
-                .get('openingElement')
-                .get('name')
-                .isJSXIdentifier({ name: 'Translate' })) {
+    let translateComponentName;
+    let translateFunctionName;
+    // First pass: find import declarations of Translate / translate.
+    // If not found, don't process the rest to avoid false positives
+    (0, traverse_1.default)(ast, {
+        ImportDeclaration(path) {
+            if (path.node.importKind === 'type' ||
+                path.get('source').node.value !== '@docusaurus/Translate') {
                 return;
             }
-            function evaluateJSXProp(propName) {
-                const attributePath = path
-                    .get('openingElement.attributes')
-                    .find((attr) => attr.isJSXAttribute() &&
-                    attr
-                        .get('name')
-                        .isJSXIdentifier({ name: propName }));
-                if (attributePath) {
-                    const attributeValue = attributePath.get('value');
-                    const attributeValueEvaluated = attributeValue.isJSXExpressionContainer()
-                        ? attributeValue.get('expression').evaluate()
-                        : attributeValue.evaluate();
-                    if (attributeValueEvaluated.confident &&
-                        typeof attributeValueEvaluated.value === 'string') {
-                        return attributeValueEvaluated.value;
-                    }
-                    else {
-                        warnings.push(`<Translate> prop=${propName} should be a statically evaluable object.\nExample: <Translate id="optional.id" description="optional description">Message</Translate>\nDynamically constructed values are not allowed, because they prevent translations to be extracted.\n${sourceFileWarningPart(path.node)}\n${generateCode(path.node)}`);
+            const importSpecifiers = path.get('specifiers');
+            const defaultImport = importSpecifiers.find((specifier) => specifier.node.type === 'ImportDefaultSpecifier');
+            const callbackImport = importSpecifiers.find((specifier) => specifier.node.type === 'ImportSpecifier' &&
+                (specifier.get('imported')
+                    .node.name === 'translate' ||
+                    specifier.get('imported')
+                        .node.value === 'translate'));
+            translateComponentName = defaultImport === null || defaultImport === void 0 ? void 0 : defaultImport.get('local').node.name;
+            translateFunctionName = callbackImport === null || callbackImport === void 0 ? void 0 : callbackImport.get('local').node.name;
+        },
+    });
+    (0, traverse_1.default)(ast, {
+        ...(translateComponentName && {
+            JSXElement(path) {
+                if (!path
+                    .get('openingElement')
+                    .get('name')
+                    .isJSXIdentifier({ name: translateComponentName })) {
+                    return;
+                }
+                function evaluateJSXProp(propName) {
+                    const attributePath = path
+                        .get('openingElement.attributes')
+                        .find((attr) => attr.isJSXAttribute() &&
+                        attr
+                            .get('name')
+                            .isJSXIdentifier({ name: propName }));
+                    if (attributePath) {
+                        const attributeValue = attributePath.get('value');
+                        const attributeValueEvaluated = attributeValue.isJSXExpressionContainer()
+                            ? attributeValue.get('expression').evaluate()
+                            : attributeValue.evaluate();
+                        if (attributeValueEvaluated.confident &&
+                            typeof attributeValueEvaluated.value === 'string') {
+                            return attributeValueEvaluated.value;
+                        }
+                        warnings.push(`<Translate> prop=${propName} should be a statically evaluable object.
+Example: <Translate id="optional id" description="optional description">Message</Translate>
+Dynamically constructed values are not allowed, because they prevent translations to be extracted.
+${sourceWarningPart(path.node)}`);
                     }
+                    return undefined;
                 }
-                return undefined;
-            }
-            // We only handle the optimistic case where we have a single non-empty content
-            const singleChildren = path
-                .get('children')
-                // Remove empty/useless text nodes that might be around our translation!
-                // Makes the translation system more reliable to JSX formatting issues
-                .filter((childrenPath) => !(childrenPath.isJSXText() &&
-                childrenPath.node.value.replace('\n', '').trim() === ''))
-                .pop();
-            if (singleChildren && singleChildren.isJSXText()) {
-                const message = singleChildren.node.value.trim().replace(/\s+/g, ' ');
                 const id = evaluateJSXProp('id');
                 const description = evaluateJSXProp('description');
-                translations[id !== null && id !== void 0 ? id : message] = {
-                    message,
-                    ...(description && { description }),
-                };
-            }
-            else if (singleChildren &&
-                singleChildren.isJSXExpressionContainer() &&
-                singleChildren.get('expression').evaluate().confident) {
-                const message = singleChildren.get('expression').evaluate().value;
-                const id = evaluateJSXProp('id');
-                const description = evaluateJSXProp('description');
-                translations[id !== null && id !== void 0 ? id : message] = {
-                    message,
-                    ...(description && { description }),
-                };
-            }
-            else {
-                warnings.push(`${staticTranslateJSXWarningPart()}\n${sourceFileWarningPart(path.node)}\n${generateCode(path.node)}`);
-            }
-        },
-        CallExpression(path) {
-            if (!path.get('callee').isIdentifier({ name: 'translate' })) {
-                return;
-            }
-            // console.log('CallExpression', path.node);
-            const args = path.get('arguments');
-            if (args.length === 1 || args.length === 2) {
-                const firstArgPath = args[0];
-                // evaluation allows translate("x" + "y"); to be considered as translate("xy");
-                const firstArgEvaluated = firstArgPath.evaluate();
-                // console.log('firstArgEvaluated', firstArgEvaluated);
-                if (firstArgEvaluated.confident &&
-                    typeof firstArgEvaluated.value === 'object') {
-                    const { message, id, description } = firstArgEvaluated.value;
+                let message;
+                const childrenPath = path.get('children');
+                // Handle empty content
+                if (!childrenPath.length) {
+                    if (!id) {
+                        warnings.push(`<Translate> without children must have id prop.
+Example: <Translate id="my-id" />
+${sourceWarningPart(path.node)}`);
+                    }
+                    else {
+                        translations[id] = {
+                            message: id,
+                            ...(description && { description }),
+                        };
+                    }
+                    return;
+                }
+                // Handle single non-empty content
+                const singleChildren = childrenPath
+                    // Remove empty/useless text nodes that might be around our
+                    // translation! Makes the translation system more reliable to JSX
+                    // formatting issues
+                    .filter((children) => !(children.isJSXText() &&
+                    children.node.value.replace('\n', '').trim() === ''))
+                    .pop();
+                const isJSXText = singleChildren && singleChildren.isJSXText();
+                const isJSXExpressionContainer = singleChildren &&
+                    singleChildren.isJSXExpressionContainer() &&
+                    singleChildren.get('expression').evaluate().confident;
+                if (isJSXText || isJSXExpressionContainer) {
+                    message = isJSXText
+                        ? singleChildren.node.value.trim().replace(/\s+/g, ' ')
+                        : singleChildren.get('expression').evaluate().value;
                     translations[id !== null && id !== void 0 ? id : message] = {
                         message,
                         ...(description && { description }),
                     };
                 }
                 else {
-                    warnings.push(`translate() first arg should be a statically evaluable object.\nExample: translate({message: "text",id: "optional.id",description: "optional description"}\nDynamically constructed values are not allowed, because they prevent translations to be extracted.\n${sourceFileWarningPart(path.node)}\n${generateCode(path.node)}`);
+                    warnings.push(`Translate content could not be extracted. It has to be a static string and use optional but static props, like <Translate id="my-id" description="my-description">text</Translate>.
+${sourceWarningPart(path.node)}`);
                 }
-            }
-            else {
-                warnings.push(`translate() function only takes 1 or 2 args\n${sourceFileWarningPart(path.node)}\n${generateCode(path.node)}`);
-            }
-        },
+            },
+        }),
+        ...(translateFunctionName && {
+            CallExpression(path) {
+                if (!path.get('callee').isIdentifier({ name: translateFunctionName })) {
+                    return;
+                }
+                const args = path.get('arguments');
+                if (args.length === 1 || args.length === 2) {
+                    const firstArgPath = args[0];
+                    // translate("x" + "y"); => translate("xy");
+                    const firstArgEvaluated = firstArgPath.evaluate();
+                    if (firstArgEvaluated.confident &&
+                        typeof firstArgEvaluated.value === 'object') {
+                        const { message, id, description } = firstArgEvaluated.value;
+                        translations[id !== null && id !== void 0 ? id : message] = {
+                            message: message !== null && message !== void 0 ? message : id,
+                            ...(description && { description }),
+                        };
+                    }
+                    else {
+                        warnings.push(`translate() first arg should be a statically evaluable object.
+Example: translate({message: "text",id: "optional.id",description: "optional description"}
+Dynamically constructed values are not allowed, because they prevent translations to be extracted.
+${sourceWarningPart(path.node)}`);
+                    }
+                }
+                else {
+                    warnings.push(`translate() function only takes 1 or 2 args
+${sourceWarningPart(path.node)}`);
+                }
+            },
+        }),
     });
     return { sourceCodeFilePath, translations, warnings };
 }

package/lib/server/utils.d.ts

@@ -1,4 +1,10 @@
-import { RouteConfig } from '@docusaurus/types';
-import globby from 'globby';
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+import type { RouteConfig } from '@docusaurus/types';
+import { Globby } from '@docusaurus/utils';
 export declare function getAllFinalRoutes(routeConfig: RouteConfig[]): RouteConfig[];
-export declare function safeGlobby(patterns: string[], options?: globby.GlobbyOptions): Promise<string[]>;
+export declare function safeGlobby(patterns: string[], options?: Globby.GlobbyOptions): Promise<string[]>;

package/lib/server/utils.js

@@ -1,23 +1,21 @@
 "use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.safeGlobby = exports.getAllFinalRoutes = void 0;
-const tslib_1 = require("tslib");
 /**
  * Copyright (c) Facebook, Inc. and its affiliates.
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  */
-const lodash_1 = require("lodash");
-const globby_1 = tslib_1.__importDefault(require("globby"));
-const path_1 = tslib_1.__importDefault(require("path"));
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.safeGlobby = exports.getAllFinalRoutes = void 0;
+const tslib_1 = require("tslib");
+const path_1 = (0, tslib_1.__importDefault)(require("path"));
 const utils_1 = require("@docusaurus/utils");
 // Recursively get the final routes (routes with no subroutes)
 function getAllFinalRoutes(routeConfig) {
     function getFinalRoutes(route) {
-        return route.routes ? lodash_1.flatMap(route.routes, getFinalRoutes) : [route];
+        return route.routes ? route.routes.flatMap(getFinalRoutes) : [route];
     }
-    return lodash_1.flatMap(routeConfig, getFinalRoutes);
+    return routeConfig.flatMap(getFinalRoutes);
 }
 exports.getAllFinalRoutes = getAllFinalRoutes;
 // Globby that fix Windows path patterns
@@ -25,7 +23,7 @@ exports.getAllFinalRoutes = getAllFinalRoutes;
 async function safeGlobby(patterns, options) {
     // Required for Windows support, as paths using \ should not be used by globby
     // (also using the windows hard drive prefix like c: is not a good idea)
-    const globPaths = patterns.map((dirPath) => utils_1.posixPath(path_1.default.relative(process.cwd(), dirPath)));
-    return globby_1.default(globPaths, options);
+    const globPaths = patterns.map((dirPath) => (0, utils_1.posixPath)(path_1.default.relative(process.cwd(), dirPath)));
+    return (0, utils_1.Globby)(globPaths, options);
 }
 exports.safeGlobby = safeGlobby;

package/lib/server/versions/index.d.ts

@@ -4,7 +4,6 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  */
-import { DocusaurusPluginVersionInformation } from '@docusaurus/types';
-export declare function getPackageJsonVersion(packageJsonPath: string): string | undefined;
-export declare function getPackageJsonName(packageJsonPath: string): string | undefined;
-export declare function getPluginVersion(pluginPath: string, siteDir: string): DocusaurusPluginVersionInformation;
+import type { DocusaurusPluginVersionInformation } from '@docusaurus/types';
+export declare function getPackageJsonVersion(packageJsonPath: string): Promise<string | undefined>;
+export declare function getPluginVersion(pluginPath: string, siteDir: string): Promise<DocusaurusPluginVersionInformation>;

package/lib/server/versions/index.js

@@ -6,45 +6,46 @@
  * LICENSE file in the root directory of this source tree.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getPluginVersion = exports.getPackageJsonName = exports.getPackageJsonVersion = void 0;
-const fs_extra_1 = require("fs-extra");
-const path_1 = require("path");
-function getPackageJsonVersion(packageJsonPath) {
-    if (fs_extra_1.existsSync(packageJsonPath)) {
+exports.getPluginVersion = exports.getPackageJsonVersion = void 0;
+const tslib_1 = require("tslib");
+const fs_extra_1 = (0, tslib_1.__importDefault)(require("fs-extra"));
+const path_1 = (0, tslib_1.__importDefault)(require("path"));
+async function getPackageJsonVersion(packageJsonPath) {
+    if (await fs_extra_1.default.pathExists(packageJsonPath)) {
         // eslint-disable-next-line @typescript-eslint/no-var-requires, import/no-dynamic-require, global-require
-        const { version } = require(packageJsonPath);
-        return typeof version === 'string' ? version : undefined;
+        return require(packageJsonPath).version;
     }
     return undefined;
 }
 exports.getPackageJsonVersion = getPackageJsonVersion;
-function getPackageJsonName(packageJsonPath) {
-    if (fs_extra_1.existsSync(packageJsonPath)) {
+async function getPackageJsonName(packageJsonPath) {
+    if (await fs_extra_1.default.pathExists(packageJsonPath)) {
         // eslint-disable-next-line @typescript-eslint/no-var-requires, import/no-dynamic-require, global-require
-        const { name } = require(packageJsonPath);
-        return typeof name === 'string' ? name : undefined;
+        return require(packageJsonPath).name;
     }
     return undefined;
 }
-exports.getPackageJsonName = getPackageJsonName;
-function getPluginVersion(pluginPath, siteDir) {
-    let potentialPluginPackageJsonDirectory = path_1.dirname(pluginPath);
+async function getPluginVersion(pluginPath, siteDir) {
+    let potentialPluginPackageJsonDirectory = path_1.default.dirname(pluginPath);
     while (potentialPluginPackageJsonDirectory !== '/') {
-        const packageJsonPath = path_1.join(potentialPluginPackageJsonDirectory, 'package.json');
-        if (fs_extra_1.existsSync(packageJsonPath) && fs_extra_1.lstatSync(packageJsonPath).isFile()) {
+        const packageJsonPath = path_1.default.join(potentialPluginPackageJsonDirectory, 'package.json');
+        if ((await fs_extra_1.default.pathExists(packageJsonPath)) &&
+            (await fs_extra_1.default.lstat(packageJsonPath)).isFile()) {
             if (potentialPluginPackageJsonDirectory === siteDir) {
-                // If the plugin belongs to the same docusaurus project, we classify it as local plugin.
+                // If the plugin belongs to the same docusaurus project, we classify it
+                // as local plugin.
                 return { type: 'project' };
             }
             return {
                 type: 'package',
-                name: getPackageJsonName(packageJsonPath),
-                version: getPackageJsonVersion(packageJsonPath),
+                name: await getPackageJsonName(packageJsonPath),
+                version: await getPackageJsonVersion(packageJsonPath),
             };
         }
-        potentialPluginPackageJsonDirectory = path_1.dirname(potentialPluginPackageJsonDirectory);
+        potentialPluginPackageJsonDirectory = path_1.default.dirname(potentialPluginPackageJsonDirectory);
     }
-    // In rare cases where a plugin is a path where no parent directory contains package.json, we can only classify it as local.
+    // In the case where a plugin is a path where no parent directory contains
+    // package.json (e.g. inline plugin), we can only classify it as local.
     return { type: 'local' };
 }
 exports.getPluginVersion = getPluginVersion;

package/lib/webpack/base.d.ts

@@ -4,9 +4,9 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  */
-import { Configuration } from 'webpack';
-import { Props } from '@docusaurus/types';
+import type { Configuration } from 'webpack';
+import type { Props } from '@docusaurus/types';
 export declare const clientDir: string;
 export declare function excludeJS(modulePath: string): boolean;
-export declare function getDocusaurusAliases(): Record<string, string>;
-export declare function createBaseConfig(props: Props, isServer: boolean, minify?: boolean): Configuration;
+export declare function getDocusaurusAliases(): Promise<Record<string, string>>;
+export declare function createBaseConfig(props: Props, isServer: boolean, minify?: boolean): Promise<Configuration>;