Merge pull request #1008 from NativeScript/tachev/fix-source-map-files

Dimitar Tachev · web-flow · commit 2b1bbf435705 · 2019-08-01T15:40:16.000+03:00
fix: require dependencies only in sourceFiles (avoid getting invalid `.js.map` files)
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nativescript-dev-webpack",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "main": "index",
   "description": "",
   "homepage": "http://www.telerik.com",
@@ -91,4 +91,4 @@
     "tns-core-modules": "next",
     "typescript": "~3.4.0"
   }
-}
+}
diff --git a/plugins/GenerateNativeScriptEntryPointsPlugin.js b/plugins/GenerateNativeScriptEntryPointsPlugin.js
@@ -72,7 +72,7 @@ exports.GenerateNativeScriptEntryPointsPlugin = (function () {
                 throw new Error(`${GenerationFailedError} File "${filePath}" not found for entry "${entryPointName}".`);
             }
 
-            if (!this.isHMRFile(filePath)) {
+            if (!this.isHMRFile(filePath) && this.isSourceFile(filePath)) {
                 const currFileDirRelativePath = path.dirname(filePath);
                 const pathToRootFromCurrFile = path.relative(currFileDirRelativePath, ".");
 

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "nativescript-dev-webpack",`
`3`		`- "version": "1.0.2",`
	`3`	`+ "version": "1.0.3",`
`4`	`4`	`"main": "index",`
`5`	`5`	`"description": "",`
`6`	`6`	`"homepage": "http://www.telerik.com",`
`@@ -91,4 +91,4 @@`
`91`	`91`	`"tns-core-modules": "next",`
`92`	`92`	`"typescript": "~3.4.0"`
`93`	`93`	`}`
`94`		`-}`
	`94`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ exports.GenerateNativeScriptEntryPointsPlugin = (function () {`
`72`	`72`	throw new Error(`${GenerationFailedError} File "${filePath}" not found for entry "${entryPointName}".`);
`73`	`73`	`}`
`74`	`74`
`75`		`- if (!this.isHMRFile(filePath)) {`
	`75`	`+ if (!this.isHMRFile(filePath) && this.isSourceFile(filePath)) {`
`76`	`76`	`const currFileDirRelativePath = path.dirname(filePath);`
`77`	`77`	`const pathToRootFromCurrFile = path.relative(currFileDirRelativePath, ".");`
`78`	`78`