Skip to content
This repository has been archived by the owner on Apr 6, 2023. It is now read-only.

fix(kit): log module id to the console when import fails #8198

Merged
merged 4 commits into from Oct 15, 2022
Merged

fix(kit): log module id to the console when import fails #8198

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
1d613d3
feat(kit): Clearer error for requiring a module
Araneline Oct 14, 2022
a472efa
fix (kit): Use relative to display failed module path
Araneline Oct 14, 2022
6efdf4e
fix (kit): Move try-catch to install.ts
Araneline Oct 14, 2022
3889b25
fix (kit): Remove 'relative' import from node:path
Araneline Oct 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 8 additions & 3 deletions packages/kit/src/internal/cjs.ts
View file
Open in desktop
Expand Up @@ -118,10 +118,15 @@ export function requireModule (id: string, opts: RequireModuleOptions = {}) {
clearRequireCache(resolvedPath)
}

// Try to require
const requiredModule = _require(resolvedPath)
try {
// Try to require
const requiredModule = _require(resolvedPath)

return requiredModule
return requiredModule
} catch (error: unknown) {
console.error(`Error while requiring module \`${id.split('/').pop()}\`: ${error}`)
Araneline marked this conversation as resolved.
Show resolved Hide resolved
throw error
}
}

export function importModule (id: string, opts: RequireModuleOptions = {}) {
Expand Down