Handle skipping invalid enum values

Author: luttje

__tests__/api-writer/glua-api-writer.spec.ts

@@ -170,12 +170,17 @@ describe('GLua API Writer', () => {
         },
         {
           key: 'MATERIAL_FOG_LINEAR_BELOW_FOG_Z',
-          value: '2',
+          value: '-2147483648', // test large negative number
+        },
+        {
+          // Should be skipped
+          key: 'MATERIAL_FOG_NEW_FAKE',
+          value: 'TODO',
         }
       ],
     });
 
-    expect(api).toEqual(`---@alias MATERIAL_FOG 0|1|2\n--- No fog\nMATERIAL_FOG_NONE = 0\n--- Linear fog\nMATERIAL_FOG_LINEAR = 1\nMATERIAL_FOG_LINEAR_BELOW_FOG_Z = 2\n\n\n`);
+    expect(api).toEqual(`---@alias MATERIAL_FOG 0|1|-2147483648\n--- No fog\nMATERIAL_FOG_NONE = 0\n--- Linear fog\nMATERIAL_FOG_LINEAR = 1\nMATERIAL_FOG_LINEAR_BELOW_FOG_Z = -2147483648\n\n\n`);
   });
 
   it('should create enums for table enumerations', () => {

src/api-writer/glua-api-writer.ts

@@ -240,7 +240,9 @@ export class GluaApiWriter {
     } else {
       // Until LuaLS supports global enumerations (https://github.com/LuaLS/lua-language-server/issues/2721) we
       // will use @alias as a workaround
-      const validEnumerations = _enum.items.map(item => item.value).join('|');
+      const validEnumerations = _enum.items.map(item => item.value)
+        .filter(value => !isNaN(Number(value)))
+        .join('|');
       api += `---@alias ${_enum.name} ${validEnumerations}\n`;
     }
 
@@ -255,6 +257,12 @@ export class GluaApiWriter {
         return;
       }
 
+      if (isNaN(Number(item.value.trim()))) {
+        // Happens for TODO value in NAV_MESH_BLOCKED_LUA in https://wiki.facepunch.com/gmod/Enums/NAV_MESH
+        console.warn(`Enum ${_enum.name} has a TODO value for key ${key}. Skipping.`);
+        return;
+      }
+
       if (isContainedInTable) {
         key = key.split('.')[1];