Ignore false matches from CoffeeScript codebase

danielbayley · Nicolas Porter · commit 2d3b2497ed7a · 2023-05-01T00:53:37.000-07:00
I'm not sure this is the best fix for this bug, which was hard to track down! But I have also attempted to address it here: https://github.com/evanw/node-source-map-support/pull/283… although looking at the activity on that repo, I don't have much hope of it being merged any time soon… Hence the reason for this PR. See also: evanw/node-source-map-support#254
diff --git a/lib/util.js b/lib/util.js
@@ -488,6 +488,8 @@ exports.compareByGeneratedPositionsInflated = compareByGeneratedPositionsInflate
  * JSON.
  */
 function parseSourceMapInput(str) {
+  // Ignore false matches from CoffeeScript codebase.
+  if (str.startsWith('zw(u')) return { version: 3, sources: [], mappings: [] }
   return JSON.parse(str.replace(/^\)]}'[^\n]*\n/, ""));
 }
 exports.parseSourceMapInput = parseSourceMapInput;

Original file line number	Diff line number	Diff line change
`@@ -488,6 +488,8 @@ exports.compareByGeneratedPositionsInflated = compareByGeneratedPositionsInflate`
`488`	`488`	`* JSON.`
`489`	`489`	`*/`
`490`	`490`	`function parseSourceMapInput(str) {`
	`491`	`+ // Ignore false matches from CoffeeScript codebase.`
	`492`	`+ if (str.startsWith('zw(u')) return { version: 3, sources: [], mappings: [] }`
`491`	`493`	`return JSON.parse(str.replace(/^\)]}'[^\n]*\n/, ""));`
`492`	`494`	`}`
`493`	`495`	`exports.parseSourceMapInput = parseSourceMapInput;`