Skip to content

fix(plugin-content-docs): fix error message context (error cause) when doc processing fails #8234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
slorber merged 7 commits into from
Oct 26, 2022
Merged

fix(plugin-content-docs): fix error message context (error cause) when doc processing fails #8234

Changes from 2 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
eb1e867
fix: `await` docMetaData and handle error
shanpriyan Oct 21, 2022
a66b9c4
chore: add error cause
shanpriyan Oct 21, 2022
58f0932
add type
shanpriyan Oct 21, 2022
a532ed0
fix failed test
shanpriyan Oct 22, 2022
8e88351
Merge branch 'main' into fix/handle-metadata-error
slorber Oct 26, 2022
ff8fdfe
Jest expect error cause too!
slorber Oct 26, 2022
8c2eb58
fix lint
slorber Oct 26, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions packages/docusaurus-plugin-content-docs/src/docs.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -316,18 +316,17 @@ async function doProcessDocMetadata({
};
}

export function processDocMetadata(args: {
export async function processDocMetadata(args: {
docFile: DocFile;
versionMetadata: VersionMetadata;
context: LoadContext;
options: MetadataOptions;
env: DocEnv;
}): Promise<DocMetadataBase> {
try {
return doProcessDocMetadata(args);
return await doProcessDocMetadata(args);
} catch (err) {
logger.error`Can't process doc metadata for doc at path path=${args.docFile.filePath} in version name=${args.versionMetadata.versionName}`;
throw err;
throw new Error(`Can't process doc metadata for doc at path path=${args.docFile.filePath} in version name=${args.versionMetadata.versionName}`, {cause: err});
}
}

Expand Down