🐛 Fix message reporting breaking on empty string (#1155)

packages/core/src/service/ErrorReporter.ts

@@ -16,6 +16,9 @@ export class ErrorReporter {
 		severity = ErrorSeverity.Error,
 		info?: LanguageErrorInfo,
 	): void {
+		if (message.trim() === '') {
+			throw new Error('Tried to report an error with no message')
+		}
 		this.errors.push(
 			LanguageError.create(message, Range.get(range), severity, info),
 		)

packages/locales/src/index.ts

@@ -29,7 +29,7 @@ type Parameter =
 export function localize(key: string, ...params: Parameter[]): string {
 	const value: string | undefined = Locales[language][key] ?? Locales.en[key]
 
-	return _resolveLocalePlaceholders(value, params) ?? ''
+	return _resolveLocalePlaceholders(value, params) ?? key
 }
 
 export function localeQuote(content: string) {

packages/locales/src/locales/en.json

@@ -65,6 +65,7 @@
   "linter-config-validator.name-convention.type": "Expects a string that contains a regular expression describing the name",
   "linter-config-validator.wrapper": "%0%. See [the documentation](%1) for more information",
   "long": "a long",
+  "long.between": "a long between %0% and %1%",
   "mcfunction.checker.command.data-modify-unapplicable-operation": "Operation %0% can only be used on %1%; the target path has type %2% instead",
   "mcfunction.completer.block.states.default-value": "Default: %0%",
   "mcfunction.parser.entity-selector.arguments.not-applicable": "%0% is not applicable here",
@@ -153,6 +154,7 @@
   "parser.integer.illegal": "Illegal integer that doesn't follow %0%",
   "parser.list.value": "a value",
   "parser.list.trailing-sep": "Trailing separation",
+  "parser.long.illegal": "Illegal long numeral that doesn't follow %0%",
   "parser.record.key": "a key",
   "parser.record.trailing-end": "Trailing separation",
   "parser.record.unexpected-char": "Unexpected character %0%",