BUGFIX: Fix array of strings parsing for coding contracts

[rocket3989]

When a contract expects an array of strings, and the user manually submits it, parsing fails for arrays with spaces after the commas

e.g:

["()()()", "(())()"]
          ^

This is problematic because the examples for the coding contract show the strings formatted in this way:

Most common scripting languages will also by default format arrays in this way, so this error will happen to anyone directly pasting from them

The issue is that removeQuotesFromString will only remove quotes if they are either the first or last char in the string, so if there is whitespace at the start of the string before a quote, it will not remove it.

bitburner-src/src/data/codingcontracttypes.ts

Lines 40 to 50 in ae8f26f

	function removeQuotesFromString(str: string): string {
	let strCpy: string = str;
	if (strCpy.startsWith('"') || strCpy.startsWith("'")) {
	strCpy = strCpy.slice(1);
	}
	if (strCpy.endsWith('"') || strCpy.endsWith("'")) {
	strCpy = strCpy.slice(0, -1);
	}
	return strCpy;
	}

The current flow for answer validation is:
split on ',' -> remove quote at start and/or end -> remove whitespace

This PR swaps the order to:

split on ',' -> remove whitespace -> remove quote at start and/or end

This fixes the array spacing issue without any major refactor to removeQuotesFromString

Video of built app running with the fixed validation:

testContract.mp4

closes #616

• npm run format
• npm run lint