Fix #57897 ignore object valued scripts

microsoft · Sep 17, 2018 · 295f04a · 295f04a
1 parent cd20626
commit 295f04a
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/extensions/npm/src/tasks.ts b/extensions/npm/src/tasks.ts
@@ -352,6 +352,7 @@ async function findAllScripts(buffer: string): Promise<StringMap> {
 
 	let visitor: JSONVisitor = {
 		onError(_error: ParseErrorCode, _offset: number, _length: number) {
+			console.log(_error);
 		},
 		onObjectEnd() {
 			if (inScripts) {
@@ -368,8 +369,10 @@ async function findAllScripts(buffer: string): Promise<StringMap> {
 			if (property === 'scripts') {
 				inScripts = true;
 			}
-			else if (inScripts) {
+			else if (inScripts && !script) {
 				script = property;
+			} else { // nested object which is invalid, ignore the script 
+				script = undefined;
 			}
 		}
 	};