fix: incorrectly exiting with undefined when dependant function expects URI

TGTGamer · TGTGamer · commit 1f8bc15ccc14 · 2024-02-21T00:31:15.000Z
Signed-off-by: Jonathan Stevens &lt;jonathan.stevens@resnovas.com&gt;
diff --git a/packages/schema/src/utils/ast-utils.ts b/packages/schema/src/utils/ast-utils.ts
@@ -106,8 +106,7 @@ export function resolveImportUri(imp: ModelImport): URI | undefined {
         && !imp.path.startsWith('/') // Respect absolute paths (Unix)
         && !/^[a-zA-Z]:\\/.test(imp.path) // Respect absolute paths (Windows)
     ) {
-        imp.path = findNodeModulesFile(imp.path) ?? '';
-        if (!imp.path) return undefined; // If the path is empty, return undefined
+        imp.path = findNodeModulesFile(imp.path) ?? imp.path;
     } 
 
     const dirUri = Utils.dirname(getDocument(imp).uri);

Original file line number	Diff line number	Diff line change
`@@ -106,8 +106,7 @@ export function resolveImportUri(imp: ModelImport): URI \| undefined {`
`106`	`106`	`&& !imp.path.startsWith('/') // Respect absolute paths (Unix)`
`107`	`107`	`&& !/^[a-zA-Z]:\\/.test(imp.path) // Respect absolute paths (Windows)`
`108`	`108`	`) {`
`109`		`- imp.path = findNodeModulesFile(imp.path) ?? '';`
`110`		`- if (!imp.path) return undefined; // If the path is empty, return undefined`
	`109`	`+ imp.path = findNodeModulesFile(imp.path) ?? imp.path;`
`111`	`110`	`}`
`112`	`111`
`113`	`112`	`const dirUri = Utils.dirname(getDocument(imp).uri);`