Validate action description in factories (#2686)

Since we don't have input validation we get these kind of errors to
frequently:
#2685

## Checklist

- [/] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [/] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [/] I have not broken the cheatsheet

Author: AndreasArvidsson

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

@@ -211,6 +211,13 @@ export class CommandRunnerImpl implements CommandRunner {
 
       default: {
         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}`);
+        }
+
         this.finalStages = action.getFinalStages?.() ?? [];
         this.noAutomaticTokenExpansion =
           action.noAutomaticTokenExpansion ?? false;

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

@@ -47,6 +47,13 @@ export class MarkStageFactoryImpl implements MarkStageFactory {
         return new TargetMarkStage(this.targetPipelineRunner, mark);
       case "explicit":
         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

@@ -109,6 +109,13 @@ export class ModifierStageFactoryImpl implements ModifierStageFactory {
         throw Error(
           `Unexpected modifier '${modifier.type}'; it should have been removed during inference`,
         );
+      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}`);
+      }
     }
   }