fix: make filePaths platform agnostic

sanderson-ut · sanderson-ut · commit fba286567e2d · 2022-10-19T11:14:17.000+01:00
Use `path.sep` as the file separator instead of `/` so that this
works on both Windows and Unix.

Rename input parameter from `path` to `modulePath` so that it
doesn't override `node:path`
diff --git a/src/index.ts b/src/index.ts
@@ -250,8 +250,8 @@ const isFile = (path?: string | undefined): path is string => {
   }
 }
 
-const isModule = (path?: string | undefined): path is string => {
-  return !!path && isFile(`${path}/package.json`)
+const isModule = (modulePath?: string | undefined): modulePath is string => {
+  return !!modulePath && isFile(`${modulePath}${path.sep}package.json`)
 }
 
 /**
@@ -287,7 +287,7 @@ function getMappedPath(
         ]),
       )
       .flat(2)
-      .filter(path => isFile(path) || isModule(path))
+      .filter(mappedPath => isFile(mappedPath) || isModule(mappedPath))
   }
 
   if (retry && paths.length === 0) {

Original file line number	Diff line number	Diff line change
`@@ -250,8 +250,8 @@ const isFile = (path?: string \| undefined): path is string => {`
`250`	`250`	`}`
`251`	`251`	`}`
`252`	`252`
`253`		`-const isModule = (path?: string \| undefined): path is string => {`
`254`		- return !!path && isFile(`${path}/package.json`)
	`253`	`+const isModule = (modulePath?: string \| undefined): modulePath is string => {`
	`254`	+ return !!modulePath && isFile(`${modulePath}${path.sep}package.json`)
`255`	`255`	`}`
`256`	`256`
`257`	`257`	`/**`
`@@ -287,7 +287,7 @@ function getMappedPath(`
`287`	`287`	`]),`
`288`	`288`	`)`
`289`	`289`	`.flat(2)`
`290`		`- .filter(path => isFile(path) \|\| isModule(path))`
	`290`	`+ .filter(mappedPath => isFile(mappedPath) \|\| isModule(mappedPath))`
`291`	`291`	`}`
`292`	`292`
`293`	`293`	`if (retry && paths.length === 0) {`