code-gen: error when JS keywords are used as group names (#638)

I made the same mistake twice already. The backend generators don't mind it at all, but in the react-query generator, we also dump a `useApi()` and something like the following is not allowed:

```ts
const { static } = useApi();
```

Closes #636

gen/code-gen.js

@@ -74,11 +74,15 @@ export function applyCodeGenStructure(app) {
             key: "sqlThrowingValidators",
           },
           {
-            key: "duplicateShortName",
+            key: "sqlDuplicateShortName",
             shortName: T.string(),
             firstName: T.string(),
             secondName: T.string(),
           },
+          {
+            key: "coreReservedGroupName",
+            groupName: T.string(),
+          },
         ),
       ],
     }),

packages/code-gen/src/generator/errors.js

@@ -33,9 +33,11 @@ export function exitOnErrorsOrReturn(context) {
       str += `Relation from '${error.referencedByType}' is missing the inverse 'T.oneToMany()' on '${error.typeName}'.
   Add 'T.oneToMany("${error.relationOwnKey}", T.reference("${error.referencedByGroup}", "${error.referencedByType}"))' to the 'relations()' call on '${error.typeName}'.
 `;
-    } else if (error.key === "duplicateShortName") {
+    } else if (error.key === "sqlDuplicateShortName") {
       str += `Short name '${error.shortName}' is used by both '${error.firstName}' and '${error.secondName}'.
   These short name values should be unique. Please call '.shortName()' on one or both of these types to set a custom value.`;
+    } else if (error.key === "coreReservedGroupName") {
+      str += `Group '${error.groupName}' is a JavaScript or TypeScript reserved keyword. Please use another group name.`;
     } else {
       str += `[${error.key}]: ${JSON.stringify(error, null, 2)}`;
     }

packages/code-gen/src/generator/index.js

@@ -97,6 +97,9 @@ export async function generate(logger, options, structure) {
   setupMemoizedTypes(context);
   exitOnErrorsOrReturn(context);
 
+  checkReservedGroupNames(context);
+  exitOnErrorsOrReturn(context);
+
   // Do initial sql checks and load in the types
   // This way the validators are generated
   if (context.options.enabledGenerators.indexOf("sql") !== -1) {
@@ -230,3 +233,71 @@ export function shouldGenerateModules(logger) {
     return true;
   }
 }
+
+/**
+ * Check if a group uses one of the reserved keywords
+ * Sourced from
+ * https://github.com/microsoft/TypeScript/blob/66ecfcbd04b8234855a673adb85e5cff3f8458d4/src/compiler/types.ts#L112
+ *
+ * @param {CodeGenContext} context
+ */
+function checkReservedGroupNames(context) {
+  const keywords = [
+    // Reserved words
+    "break",
+    "case",
+    "catch",
+    "class",
+    "const",
+    "continue",
+    "debugger",
+    "default",
+    "delete",
+    "do",
+    "else",
+    "enum",
+    "export",
+    "extends",
+    "false",
+    "final",
+    "for",
+    "function",
+    "if",
+    "import",
+    "in",
+    "instanceof",
+    "new",
+    "null",
+    "return",
+    "super",
+    "switch",
+    "this",
+    "throw",
+    "true",
+    "try",
+    "typeof",
+    "var",
+    "void",
+    "while",
+    "with",
+    // And strict mode included
+    "implements",
+    "interface",
+    "let",
+    "package",
+    "private",
+    "protected",
+    "public",
+    "static",
+    "yield",
+  ];
+
+  for (const group of Object.keys(context.structure)) {
+    if (keywords.indexOf(group.toLowerCase()) !== -1) {
+      context.errors.push({
+        key: "coreReservedGroupName",
+        groupName: group,
+      });
+    }
+  }
+}

packages/code-gen/src/generator/sql/utils.js

@@ -18,7 +18,7 @@ export function addShortNamesToQueryEnabledObjects(context) {
 
     if (knownShortNames[type.shortName]) {
       context.errors.push({
-        key: "duplicateShortName",
+        key: "sqlDuplicateShortName",
         shortName: type.shortName,
         firstName: type.name,
         secondName: knownShortNames[type.shortName],