fixes #3 - support /* and */ and some edge cases (#4)

frankleng · web-flow · commit fd0942830379 · 2022-03-04T15:58:13.000-08:00
* fixes #3 - support `/*` and `*/` and some edge cases * up version
diff --git a/index.js b/index.js
@@ -3,8 +3,7 @@ const path = require("path");
 const fs = require('fs');
 
 function stripJsonComments (data) {
-  const re = new RegExp("\/\/(.*)","g");
-  return data.replace(re,'');
+  return data.replace(/\\"|"(?:\\"|[^"])*"|(\/\/.*|\/\*[\s\S]*?\*\/)/g, (m, g) => g ? "" : m);
 }
 
 module.exports = (relativeTsconfigPath = './tsconfig.json') => {
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "esbuild-ts-paths",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Transform TS path alias to absolute paths for esbuild",
   "main": "index.js",
   "repository": "git@github.com:frankleng/esbuild-ts-paths.git",

Original file line number	Diff line number	Diff line change
`@@ -3,8 +3,7 @@ const path = require("path");`
`3`	`3`	`const fs = require('fs');`
`4`	`4`
`5`	`5`	`function stripJsonComments (data) {`
`6`		`- const re = new RegExp("\/\/(.*)","g");`
`7`		`- return data.replace(re,'');`
	`6`	`+ return data.replace(/\\"\|"(?:\\"\|[^"])"\|(\/\/.\|\/\[\s\S]?\*\/)/g, (m, g) => g ? "" : m);`
`8`	`7`	`}`
`9`	`8`
`10`	`9`	`module.exports = (relativeTsconfigPath = './tsconfig.json') => {`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "esbuild-ts-paths",`
`3`		`- "version": "1.1.0",`
	`3`	`+ "version": "1.1.1",`
`4`	`4`	`"description": "Transform TS path alias to absolute paths for esbuild",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"repository": "git@github.com:frankleng/esbuild-ts-paths.git",`