Support tsconfig without baseUrl property

Related issue: #18

Author: unennhexium

.gitignore

@@ -1,4 +1,5 @@
 .vscode/settings.json
+.idea
 node_modules
 package-lock.json
 dist

index.ts

@@ -1,4 +1,4 @@
-import { join } from 'path';
+import { join, relative } from 'path';
 import { Plugin } from 'rollup';
 import {
 	CompilerOptions,
@@ -65,12 +65,19 @@ export const typescriptPaths = ({
 				return null;
 			}
 
-			const targetFileName = join(
-				outDir,
-				preserveExtensions
-					? resolvedFileName
-					: resolvedFileName.replace(/\.tsx?$/i, '.js'),
-			);
+			const processedFileName = preserveExtensions
+				? resolvedFileName
+				: resolvedFileName.replace(/\.tsx?$/i, '.js');
+
+			/* Do not use:
+			 *  - path.dirname(tsConfigPath) -> using abs. path to <proj_root>/test/<tsconfig>
+			 *  - __dirname -> using abs. path to compiled plugin files <proj_root>/dist
+			 *  - process.env.PWD -> non cross-platform
+			 * instead of process.cwd() -> using abs. path to <proj_root>/<tsconfig>, that is correct
+			 */
+			const targetFileName = typeof compilerOptions.baseUrl === 'undefined'
+				? join(outDir, relative(process.cwd(), processedFileName))
+				: join(outDir, processedFileName);
 
 			const resolved = absolute
 				? sys.resolvePath(targetFileName)

package.json

@@ -8,6 +8,7 @@
     "dist"
   ],
   "scripts": {
+    "test:wo-baseurl": "WO_BASEURL=true npm run test",
     "test": "npm run prepare && node test",
     "preversion": "npm test",
     "prepare": "rm -rf dist && tsc"

test/index.js

@@ -6,27 +6,31 @@ const typescriptPaths = require('../dist').default;
 
 const transform = (path) => path.replace(/\.js$/i, '.cjs.js');
 
+const tsConfigPath = process.env.WO_BASEURL
+	? 'tsconfig-wo-baseurl.json'
+	: 'tsconfig.json'
+
 const plugin = typescriptPaths({
-	tsConfigPath: resolve(__dirname, 'tsconfig.json'),
+	tsConfigPath: resolve(__dirname, tsConfigPath),
 });
 
 const pluginNonAbs = typescriptPaths({
-	tsConfigPath: resolve(__dirname, 'tsconfig.json'),
+	tsConfigPath: resolve(__dirname, tsConfigPath),
 	absolute: false,
 });
 
 const pluginNonRelative = typescriptPaths({
-	tsConfigPath: resolve(__dirname, 'tsconfig.json'),
+	tsConfigPath: resolve(__dirname, tsConfigPath),
 	nonRelative: true,
 });
 
 const pluginTransform = typescriptPaths({
-	tsConfigPath: resolve(__dirname, 'tsconfig.json'),
+	tsConfigPath: resolve(__dirname, tsConfigPath),
 	transform,
 });
 
 const pluginPreserveExtensions = typescriptPaths({
-	tsConfigPath: resolve(__dirname, 'tsconfig.json'),
+	tsConfigPath: resolve(__dirname, tsConfigPath),
 	preserveExtensions: true,
 });
 
@@ -86,11 +90,19 @@ try {
 	// skips non-relative paths unless enabled
 	strictEqual(plugin.resolveId('foo/bar', ''), null);
 
-	// resolves non-relative from baseUrl even if no path is matched
-	strictEqual(
-		pluginNonRelative.resolveId('foo/bar', ''),
-		join(__dirname, 'foo', 'bar.js'),
-	);
+	// resolves non-relative from baseUrl even if no path is matched, baseUrl is necessary in config
+	if (!process.env.WO_BASEURL) {
+		strictEqual(
+			pluginNonRelative.resolveId('foo/bar', ''),
+			join(__dirname, 'foo', 'bar.js'),
+		);
+	} else {
+		console.log(
+			'SKIP test forced nonRelative paths:\n',
+			'- Irrelevant since baseUrl is not provided in this case\n',
+			'- See WO_BASEURL environment variable in the test code'
+		)
+	}
 
 	// resolves as a relative path with option `absolute: false`
 	strictEqual(

test/tsconfig-wo-baseurl.json

@@ -0,0 +1,13 @@
+{
+	// comments should work
+	"compilerOptions": {
+		"paths": {
+			"@foobar": ["test/foo/bar"],
+			"@foobar-react": ["test/foo/bar-react"],
+			"@bar/*": ["test/bar/*"],
+			"bar/*": ["test/bar/*"],
+			"@js": ["test/js"],
+			"$/*": ["test/*"]
+		}
+	}
+}

tsconfig.json

@@ -5,6 +5,7 @@
 		"lib": ["esnext"],
 		"declaration": true,
 		"outDir": "dist",
+		"sourceMap": true,
 
 		"strict": true,
 		"noUnusedLocals": true,