Added comment

Author: AndreasArvidsson

packages/cursorless-engine/src/core/commandRunner/CommandRunnerImpl.ts

@@ -213,6 +213,7 @@ export class CommandRunnerImpl implements CommandRunner {
         const action = this.actions[actionDescriptor.name];
 
         // Ensure we don't miss any new actions. Needed because we don't have input validation.
+        // FIXME: remove once we have schema validation (#983)
         if (action == null) {
           throw new Error(`Unknown action: ${actionDescriptor.name}`);
         }

packages/cursorless-engine/src/processTargets/MarkStageFactoryImpl.ts

@@ -49,6 +49,7 @@ export class MarkStageFactoryImpl implements MarkStageFactory {
         return new ExplicitMarkStage(mark);
       default: {
         // Ensure we don't miss any new marks. Needed because we don't have input validation.
+        // FIXME: remove once we have schema validation (#983)
         const _exhaustiveCheck: never = mark;
         const { type } = mark;
         throw new Error(`Unknown mark: ${type}`);

packages/cursorless-engine/src/processTargets/ModifierStageFactoryImpl.ts

@@ -111,6 +111,7 @@ export class ModifierStageFactoryImpl implements ModifierStageFactory {
         );
       default: {
         // Ensure we don't miss any new modifiers. Needed because we don't have input validation.
+        // FIXME: remove once we have schema validation (#983)
         const _exhaustiveCheck: never = modifier;
         const { type } = modifier;
         throw new Error(`Unknown modifier: ${type}`);