fix(typescript): sourcemap generated as null (rollup#276)

* Fix typescript sourcemap generated as null

* test(typescript): add inlineSources test

Author: ziofat

packages/typescript/src/options/normalize.ts

@@ -38,6 +38,9 @@ export function normalizeCompilerOptions(
     // Default to using source maps.
     // If the plugin user sets sourceMap to false we keep that option.
     compilerOptions.sourceMap = true;
+    // Using inlineSources to make sure typescript generate source content
+    // instead of source path.
+    compilerOptions.inlineSources = true;
     autoSetSourceMap = true;
   }
 

packages/typescript/test/test.js

@@ -662,6 +662,22 @@ test('prevents errors due to conflicting `sourceMap`/`inlineSourceMap` options',
   );
 });
 
+test('should not emit null sourceContent', async (t) => {
+  const bundle = await rollup({
+    input: 'fixtures/basic/main.ts',
+    plugins: [
+      typescript({
+        tsconfig: 'fixtures/basic/tsconfig.json'
+      })
+    ],
+    onwarn
+  });
+  const output = await getCode(bundle, { format: 'esm', sourcemap: true }, true);
+  const sourcemap = output[0].map;
+  // eslint-disable-next-line no-undefined
+  t.false(sourcemap.sourcesContent.includes(undefined));
+});
+
 test('should not fail if source maps are off', async (t) => {
   await t.notThrowsAsync(
     rollup({

util/test.js

@@ -6,8 +6,8 @@ const getCode = async (bundle, outputOptions, allFiles = false) => {
   const { output } = await bundle.generate(outputOptions || { format: 'cjs' });
 
   if (allFiles) {
-    return output.map(({ code, fileName, source }) => {
-      return { code, fileName, source };
+    return output.map(({ code, fileName, source, map }) => {
+      return { code, fileName, source, map };
     });
   }
   const [{ code }] = output;