Merge pull request #9441 from keymanapp/chore/developer/report-fatal-compiler-errors-to-sentry

mcdurdin · web-flow · commit add9a43f9edd · 2023-08-09T18:35:53.000+10:00
chore(developer): improve kmc sentry reporting on fatal build errors
diff --git a/common/web/types/src/util/compiler-interfaces.ts b/common/web/types/src/util/compiler-interfaces.ts
@@ -6,6 +6,11 @@ export interface CompilerEvent {
   line?: number;
   code: number;
   message: string;
+  /**
+   * an internal error occurred that should be captured with a stack trace
+   * e.g. to the Keyman sentry instance by kmc
+   */
+  exceptionVar?: any;
 };
 
 export enum CompilerErrorSeverity {
@@ -399,7 +404,13 @@ export const defaultCompilerOptions: CompilerOptions = {
  * @param message
  * @returns
  */
-export const CompilerMessageSpec = (code: number, message: string) : CompilerEvent => { return { code, message } };
+export const CompilerMessageSpec = (code: number, message: string, exceptionVar?: any) : CompilerEvent => ({
+  code,
+  message: exceptionVar
+    ? (message ?? `Unexpected exception`) + `: ${exceptionVar.toString()}\n\nCall stack:\n${(exceptionVar instanceof Error ? exceptionVar.stack : (new Error()).stack)}` :
+    message,
+  exceptionVar
+});
 
 /**
  * @deprecated use `CompilerError.exceptionToString` instead
diff --git a/developer/src/kmc-analyze/src/messages.ts b/developer/src/kmc-analyze/src/messages.ts
@@ -1,4 +1,4 @@
-import { CompilerErrorNamespace, CompilerErrorSeverity, CompilerMessageSpec as m, compilerExceptionToString as exc } from "@keymanapp/common-types";
+import { CompilerErrorNamespace, CompilerErrorSeverity, CompilerMessageSpec as m } from "@keymanapp/common-types";
 
 const Namespace = CompilerErrorNamespace.Analyzer;
 const SevInfo = CompilerErrorSeverity.Info | Namespace;
@@ -8,7 +8,7 @@ const SevInfo = CompilerErrorSeverity.Info | Namespace;
 const SevFatal = CompilerErrorSeverity.Fatal | Namespace;
 
 export class AnalyzerMessages {
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, `Unexpected exception: ${exc(o.e)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x0001;
 
   static Info_ScanningFile = (o:{type: string, name: string}) => m(this.INFO_ScanningFile,
diff --git a/developer/src/kmc-keyboard-info/src/messages.ts b/developer/src/kmc-keyboard-info/src/messages.ts
@@ -8,8 +8,7 @@ const SevError = CompilerErrorSeverity.Error | Namespace;
 const SevFatal = CompilerErrorSeverity.Fatal | Namespace;
 
 export class KeyboardInfoCompilerMessages {
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException,
-    `Unexpected exception: ${(o.e ?? 'unknown error').toString()}\n\nCall stack:\n${(o.e instanceof Error ? o.e.stack : (new Error()).stack)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x0001;
 
   static Error_FileDoesNotExist = (o:{filename: string}) => m(this.ERROR_FileDoesNotExist, `File ${o.filename} does not exist.`);
diff --git a/developer/src/kmc-kmn/src/compiler/messages.ts b/developer/src/kmc-kmn/src/compiler/messages.ts
@@ -1,4 +1,4 @@
-import { CompilerErrorNamespace, CompilerErrorSeverity, CompilerEvent, CompilerMessageSpec as m, compilerExceptionToString as exc } from "@keymanapp/common-types";
+import { CompilerErrorNamespace, CompilerErrorSeverity, CompilerEvent, CompilerMessageSpec as m } from "@keymanapp/common-types";
 
 const Namespace = CompilerErrorNamespace.KmnCompiler;
 const SevInfo = CompilerErrorSeverity.Info | Namespace;
@@ -46,20 +46,21 @@ export const enum KmnCompilerMessageRanges {
   are reserved for kmcmplib messages.
 */
 export class CompilerMessages {
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, `Unexpected exception: ${exc(o.e)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x900;
 
-  static Fatal_MissingWasmModule = (o:{e?: any}) => m(this.FATAL_MissingWasmModule, `Could not instantiate WASM compiler module or initialization failed: ${exc(o.e)}`);
+  static Fatal_MissingWasmModule = (o:{e?: any}) => m(this.FATAL_MissingWasmModule,
+    `Could not instantiate WASM compiler module or initialization failed`, o.e ?? 'unknown error');
   static FATAL_MissingWasmModule = SevFatal | 0x901;
 
   // TODO: Is this now deprecated?
-  static Fatal_UnableToSetCompilerOptions = () => m(this.FATAL_UnableToSetCompilerOptions, `Unable to set compiler options`);
+  static Fatal_UnableToSetCompilerOptions = () => m(this.FATAL_UnableToSetCompilerOptions, null, `Unable to set compiler options`);
   static FATAL_UnableToSetCompilerOptions = SevFatal | 0x902;
 
-  static Fatal_CallbacksNotSet = () => m(this.FATAL_CallbacksNotSet, `Callbacks were not set with init`);
+  static Fatal_CallbacksNotSet = () => m(this.FATAL_CallbacksNotSet, null, `Callbacks were not set with init`);
   static FATAL_CallbacksNotSet = SevFatal | 0x903;
 
-  static Fatal_UnicodeSetOutOfRange = () => m(this.FATAL_UnicodeSetOutOfRange, `UnicodeSet buffer was too small`);
+  static Fatal_UnicodeSetOutOfRange = () => m(this.FATAL_UnicodeSetOutOfRange, null, `UnicodeSet buffer was too small`);
   static FATAL_UnicodeSetOutOfRange = SevFatal | 0x904;
 
   static Error_UnicodeSetHasStrings = () => m(this.ERROR_UnicodeSetHasStrings, `UnicodeSet contains strings, not allowed`);
@@ -72,19 +73,19 @@ export class CompilerMessages {
   static ERROR_UnicodeSetSyntaxError = SevError | 0x907;
 
   static Error_InvalidKvksFile = (o:{filename: string, e: any}) => m(this.ERROR_InvalidKvksFile,
-    `Error encountered parsing ${o.filename}: ${o.e}`);
+    `Error encountered parsing ${o.filename}: ${o.e ?? 'unknown error'}`); // Note, not fatal, not reporting to Sentry
   static ERROR_InvalidKvksFile = SevError | 0x908;
 
   static Warn_InvalidVkeyInKvksFile = (o:{filename: string, invalidVkey: string}) => m(this.WARN_InvalidVkeyInKvksFile,
     `Invalid virtual key ${o.invalidVkey} found in ${o.filename}`);
   static WARN_InvalidVkeyInKvksFile = SevWarn | 0x909;
 
   static Error_InvalidDisplayMapFile = (o:{filename: string, e: any}) => m(this.ERROR_InvalidDisplayMapFile,
-    `Error encountered parsing display map ${o.filename}: ${o.e}`);
+    `Error encountered parsing display map ${o.filename}: ${o.e ?? 'unknown error'}`); // Note, not fatal, not reporting to Sentry
   static ERROR_InvalidDisplayMapFile = SevError | 0x90A;
 
   static Error_InvalidKvkFile = (o:{filename: string, e: any}) => m(this.ERROR_InvalidKvkFile,
-    `Error encountered loading ${o.filename}: ${o.e}`);
+    `Error encountered loading ${o.filename}: ${o.e ?? 'unknown error'}`); // Note, not fatal, not reporting to Sentry
   static ERROR_InvalidKvkFile = SevError | 0x90B;
 };
 
diff --git a/developer/src/kmc-ldml/src/compiler/messages.ts b/developer/src/kmc-ldml/src/compiler/messages.ts
@@ -60,7 +60,7 @@ export class CompilerMessages {
   static ERROR_MustBeAtLeastOneLayerElement = SevError | 0x000E;
 
   static Fatal_SectionCompilerFailed = (o:{sect: string}) =>
-    m(this.FATAL_SectionCompilerFailed, `The compiler for '${o.sect}' failed unexpectedly.`);
+    m(this.FATAL_SectionCompilerFailed, null, `The compiler for '${o.sect}' failed unexpectedly.`);
   static FATAL_SectionCompilerFailed = SevFatal | 0x000F;
 
   static Error_DisplayIsRepeated = (o:{to: string}) =>
diff --git a/developer/src/kmc-model-info/src/messages.ts b/developer/src/kmc-model-info/src/messages.ts
@@ -8,8 +8,7 @@ const SevError = CompilerErrorSeverity.Error | Namespace;
 const SevFatal = CompilerErrorSeverity.Fatal | Namespace;
 
 export class ModelInfoCompilerMessages {
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException,
-    `Unexpected exception: ${(o.e ?? 'unknown error').toString()}\n\nCall stack:\n${(o.e instanceof Error ? o.e.stack : (new Error()).stack)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x0001;
 
   static Error_FileDoesNotExist = (o:{filename: string}) => m(this.ERROR_FileDoesNotExist, `File ${o.filename} does not exist.`);
diff --git a/developer/src/kmc-model/src/model-compiler-errors.ts b/developer/src/kmc-model/src/model-compiler-errors.ts
@@ -1,4 +1,4 @@
-import { CompilerErrorNamespace, CompilerErrorSeverity, CompilerEvent } from "@keymanapp/common-types";
+import { CompilerErrorNamespace, CompilerErrorSeverity, CompilerEvent, CompilerMessageSpec } from "@keymanapp/common-types";
 
 const Namespace = CompilerErrorNamespace.ModelCompiler;
 // const SevInfo = CompilerErrorSeverity.Info | Namespace;
@@ -7,12 +7,11 @@ const SevWarn = CompilerErrorSeverity.Warn | Namespace;
 const SevError = CompilerErrorSeverity.Error | Namespace;
 const SevFatal = CompilerErrorSeverity.Fatal | Namespace;
 
-const m = (code: number, message: string) : CompilerEvent => { return {
+const m = (code: number, message: string, exceptionVar?: any) : CompilerEvent => ({
+  ...CompilerMessageSpec(code, message, exceptionVar),
   line: ModelCompilerMessageContext.line,
   filename: ModelCompilerMessageContext.filename,
-  code,
-  message
-} };
+});
 
 export class ModelCompilerMessageContext {
   // Context added to all messages
@@ -22,8 +21,7 @@ export class ModelCompilerMessageContext {
 
 export class ModelCompilerMessages {
 
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException,
-    `Unexpected exception: ${(o.e ?? 'unknown error').toString()}\n\nCall stack:\n${(o.e instanceof Error ? o.e.stack : (new Error()).stack)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x0001;
 
   static Warn_MixedNormalizationForms = (o:{wordform: string}) => m(this.WARN_MixedNormalizationForms,
diff --git a/developer/src/kmc-package/src/compiler/messages.ts b/developer/src/kmc-package/src/compiler/messages.ts
@@ -8,8 +8,7 @@ const SevError = CompilerErrorSeverity.Error | Namespace;
 const SevFatal = CompilerErrorSeverity.Fatal | Namespace;
 
 export class CompilerMessages {
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException,
-    `Unexpected exception: ${(o.e ?? 'unknown error').toString()}\n\nCall stack:\n${(o.e instanceof Error ? o.e.stack : (new Error()).stack)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x0001;
 
   static Warn_AbsolutePath = (o:{filename: string}) => m(this.WARN_AbsolutePath, `File ${o.filename} has an absolute path, which is not portable.`);
diff --git a/developer/src/kmc/src/commands/analyze.ts b/developer/src/kmc/src/commands/analyze.ts
@@ -2,7 +2,7 @@ import * as fs from 'fs';
 import * as path from 'path';
 import { Command, Option } from 'commander';
 import { NodeCompilerCallbacks } from '../util/NodeCompilerCallbacks.js';
-import { InfrastructureMessages } from '../messages/messages.js';
+import { InfrastructureMessages } from '../messages/infrastructureMessages.js';
 import { CompilerCallbacks, CompilerLogLevel } from '@keymanapp/common-types';
 import { AnalyzeOskCharacterUse, AnalyzeOskRewritePua } from '@keymanapp/kmc-analyze';
 import { BaseOptions } from '../util/baseOptions.js';
diff --git a/developer/src/kmc/src/commands/build.ts b/developer/src/kmc/src/commands/build.ts
@@ -4,7 +4,7 @@ import { Command } from 'commander';
 import { buildActivities } from './buildClasses/buildActivities.js';
 import { BuildProject } from './buildClasses/BuildProject.js';
 import { NodeCompilerCallbacks } from '../util/NodeCompilerCallbacks.js';
-import { InfrastructureMessages } from '../messages/messages.js';
+import { InfrastructureMessages } from '../messages/infrastructureMessages.js';
 import { CompilerFileCallbacks, CompilerOptions, KeymanFileTypes } from '@keymanapp/common-types';
 import { BaseOptions } from '../util/baseOptions.js';
 import { expandFileLists } from '../util/fileLists.js';
@@ -80,13 +80,18 @@ If no input file is supplied, kmc will build the current folder.`)
 }
 
 async function build(filename: string, parentCallbacks: NodeCompilerCallbacks, options: CompilerOptions): Promise<boolean> {
+  try {
+    // TEST: allow command-line simulation of infrastructure fatal errors, and
+    // also for unit tests
+    if(process.env.SENTRY_CLIENT_TEST_BUILD_EXCEPTION == '1') {
+      throw new Error('Test exception from SENTRY_CLIENT_TEST_BUILD_EXCEPTION');
+    }
 
-  if(!fs.existsSync(filename)) {
-    parentCallbacks.reportMessage(InfrastructureMessages.Error_FileDoesNotExist({filename}));
-    return false;
-  }
+    if(!fs.existsSync(filename)) {
+      parentCallbacks.reportMessage(InfrastructureMessages.Error_FileDoesNotExist({filename}));
+      return false;
+    }
 
-  try {
     let builder = null;
 
     // If infile is a directory, then we treat that as a project and build it
@@ -136,3 +141,10 @@ async function build(filename: string, parentCallbacks: NodeCompilerCallbacks, o
     return false;
   }
 }
+
+/**
+ * these are exported only for unit tests, do not use
+ */
+export const unitTestEndpoints = {
+  build
+};
diff --git a/developer/src/kmc/src/commands/buildClasses/BuildKeyboardInfo.ts b/developer/src/kmc/src/commands/buildClasses/BuildKeyboardInfo.ts
@@ -3,7 +3,7 @@ import { BuildActivity } from './BuildActivity.js';
 import { CompilerCallbacks, CompilerOptions, KeymanDeveloperProject, KeymanFileTypes } from '@keymanapp/common-types';
 import { KeyboardInfoCompiler } from '@keymanapp/kmc-keyboard-info';
 import { loadProject } from '../../util/projectLoader.js';
-import { InfrastructureMessages } from '../../messages/messages.js';
+import { InfrastructureMessages } from '../../messages/infrastructureMessages.js';
 import { KmpCompiler } from '@keymanapp/kmc-package';
 import { calculateSourcePath } from '../../util/calculateSourcePath.js';
 
diff --git a/developer/src/kmc/src/commands/buildClasses/BuildModelInfo.ts b/developer/src/kmc/src/commands/buildClasses/BuildModelInfo.ts
@@ -4,7 +4,7 @@ import { CompilerCallbacks, CompilerOptions, KeymanFileTypes } from '@keymanapp/
 import { writeMergedModelMetadataFile } from '@keymanapp/kmc-model-info';
 import { KmpCompiler } from '@keymanapp/kmc-package';
 import { loadProject } from '../../util/projectLoader.js';
-import { InfrastructureMessages } from '../../messages/messages.js';
+import { InfrastructureMessages } from '../../messages/infrastructureMessages.js';
 import { calculateSourcePath } from '../../util/calculateSourcePath.js';
 
 export class BuildModelInfo extends BuildActivity {
diff --git a/developer/src/kmc/src/commands/buildClasses/BuildProject.ts b/developer/src/kmc/src/commands/buildClasses/BuildProject.ts
@@ -3,7 +3,7 @@ import * as fs from 'fs';
 import { CompilerCallbacks, CompilerFileCallbacks, CompilerOptions, KeymanDeveloperProject, KeymanDeveloperProjectFile, KeymanFileTypes } from '@keymanapp/common-types';
 import { BuildActivity } from './BuildActivity.js';
 import { buildActivities } from './buildActivities.js';
-import { InfrastructureMessages } from '../../messages/messages.js';
+import { InfrastructureMessages } from '../../messages/infrastructureMessages.js';
 import { loadProject } from '../../util/projectLoader.js';
 
 export class BuildProject extends BuildActivity {
diff --git a/developer/src/kmc/src/kmc.ts b/developer/src/kmc/src/kmc.ts
@@ -17,6 +17,10 @@ try {
   KeymanSentry.captureException(e);
 }
 
+// Ensure any messages reported to Sentry have had time to be uploaded before we
+// exit. In most cases, this will be a no-op so should not affect performance.
+await KeymanSentry.close();
+
 async function run() {
   /* Arguments */
 
diff --git a/developer/src/kmc/src/messages/infrastructureMessages.ts b/developer/src/kmc/src/messages/infrastructureMessages.ts
@@ -8,8 +8,7 @@ const SevError = CompilerErrorSeverity.Error | Namespace;
 const SevFatal = CompilerErrorSeverity.Fatal | Namespace;
 
 export class InfrastructureMessages {
-  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException,
-    `Unexpected exception: ${(o.e ?? 'unknown error').toString()}\n\nCall stack:\n${(o.e instanceof Error ? o.e.stack : (new Error()).stack)}`);
+  static Fatal_UnexpectedException = (o:{e: any}) => m(this.FATAL_UnexpectedException, null, o.e ?? 'unknown error');
   static FATAL_UnexpectedException = SevFatal | 0x0001;
 
   // For this message, we override the filename with the passed-in file. A bit of a hack but does the job
diff --git a/developer/src/kmc/src/util/KeymanSentry.ts b/developer/src/kmc/src/util/KeymanSentry.ts
@@ -4,6 +4,12 @@ import Sentry from "@sentry/node";
 import KEYMAN_VERSION from "@keymanapp/keyman-version";
 import { spawnChild } from "./spawnAwait.js";
 
+/**
+ * Maximum delay on shutdown of process to send pending events
+ * to Sentry, in msec
+ */
+const CLOSE_TIMEOUT = 2000;
+
 const cli = process.argv.join(' ');
 let isInit = false;
 
@@ -39,7 +45,7 @@ export class KeymanSentry {
       }
     } else if(cli.includes('event')) {
       const eventId = Sentry.captureMessage('Test message from -sentry-client-test-exception event');
-      await Sentry.close(2000);
+      await Sentry.close(CLOSE_TIMEOUT);
       console.log(`Captured test message with id ${eventId}`);
       process.exit(0);
     } else {
@@ -89,18 +95,34 @@ export class KeymanSentry {
     isInit = true;
   }
 
+  private static writeSentryMessage(eventId: string) {
+    process.stderr.write(`
+    This error has been automatically reported to the Keyman team.
+      Identifier:  ${eventId}
+      Application: Keyman Developer
+      Reported at: https://sentry.io/organizations/keyman/projects/keyman-developer/events/${eventId}/
+    `);
+  }
+
+  static async reportException(e: any, silent: boolean = true) {
+    if(isInit) {
+      const eventId = await Sentry.captureException(e);
+      if(!silent) {
+        this.writeSentryMessage(eventId);
+      }
+      return eventId;
+    }
+    return null;
+  }
+
   static async captureException(e: any) {
     if(isInit) {
       const eventId = Sentry.captureException(e);
       process.stderr.write(`
-Fatal error: ${(e??'').toString()}
-
-This error has been automatically reported to the Keyman team.
-  Identifier:  ${eventId}
-  Application: Keyman Developer
-  Reported at: https://sentry.io/organizations/keyman/projects/keyman-developer/events/${eventId}/
-`);
-      await Sentry.close(2000);
+      Fatal error: ${(e??'').toString()}
+      `);
+      this.writeSentryMessage(eventId);
+      this.close();
 
       // For local development, we don't want to bury the trace; we need the cast to avoid
       // TS2367 (comparison appears to be unintentional)
@@ -112,4 +134,10 @@ This error has been automatically reported to the Keyman team.
       throw e;
     }
   }
+
+  static async close() {
+    if(isInit) {
+      await Sentry.close(2000);
+    }
+  }
 }
diff --git a/developer/src/kmc/src/util/NodeCompilerCallbacks.ts b/developer/src/kmc/src/util/NodeCompilerCallbacks.ts
@@ -6,9 +6,10 @@ import { CompilerCallbacks, CompilerEvent,
          CompilerError,
          CompilerCallbackOptions,
          CompilerFileCallbacks} from '@keymanapp/common-types';
-import { InfrastructureMessages } from '../messages/messages.js';
+import { InfrastructureMessages } from '../messages/infrastructureMessages.js';
 import chalk from 'chalk';
 import supportsColor from 'supports-color';
+import { KeymanSentry } from './KeymanSentry.js';
 
 const color = chalk.default;
 const severityColors: {[value in CompilerErrorSeverity]: chalk.Chalk} = {
@@ -121,6 +122,14 @@ export class NodeCompilerCallbacks implements CompilerCallbacks {
 
     this.messages.push({...event});
 
+    // report fatal errors to Sentry, but don't abort; note, it won't be
+    // reported if user has disabled the Sentry setting
+    if(CompilerError.severity(event.code) == CompilerErrorSeverity.Fatal) {
+      // this is async so returns a Promise, we'll let it resolve in its own
+      // time, and it will emit a message to stderr with details at that time
+      KeymanSentry.reportException(event.exceptionVar, false);
+    }
+
     if(CompilerError.severity(event.code) < compilerLogLevelToSeverity[this.options.logLevel]) {
       // collect messages but don't print to console
       return;
diff --git a/developer/src/kmc/src/util/fileLists.ts b/developer/src/kmc/src/util/fileLists.ts
@@ -1,7 +1,7 @@
 import * as fs from 'fs';
 import * as path from 'path';
 import { CompilerCallbacks } from "@keymanapp/common-types";
-import { InfrastructureMessages } from "../messages/messages.js";
+import { InfrastructureMessages } from "../messages/infrastructureMessages.js";
 
 /**
  * Replaces each entry starting with `@` with the content of the file, with one
diff --git a/developer/src/kmc/src/util/projectLoader.ts b/developer/src/kmc/src/util/projectLoader.ts
@@ -2,7 +2,7 @@
 import * as path from 'path';
 import * as fs from 'fs';
 import { CompilerCallbacks, KeymanDeveloperProject, KeymanFileTypes, KPJFileReader } from "@keymanapp/common-types";
-import { InfrastructureMessages } from "../messages/messages.js";
+import { InfrastructureMessages } from "../messages/infrastructureMessages.js";
 
 export const isProject = (filename: string): boolean =>
   fs.existsSync(filename) && (
diff --git a/developer/src/kmc/test/test-infrastructureMessages.ts b/developer/src/kmc/test/test-infrastructureMessages.ts
diff --git a/developer/src/kmc/test/test-project-build.ts b/developer/src/kmc/test/test-project-build.ts
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
