Fix recognition of API Gateway V1 events (#30)

Correctly handle gateway V1 events with version

Author: christiangaetano

.changeset/serious-fishes-act.md

@@ -0,0 +1,5 @@
+---
+'@as-integrations/aws-lambda': patch
+---
+
+Correctly recognize gateway v1 events

src/__tests__/mockAPIGatewayV1Server.ts

@@ -30,6 +30,8 @@ function v1EventFromRequest(
 
   // simplify the V1 event down to what our integration actually cares about
   const event: Partial<APIGatewayProxyEvent> = {
+    // @ts-expect-error (version actually can exist on v1 events, this seems to be a typing error)
+    version: "1.0",
     httpMethod: req.method!,
     headers: Object.fromEntries(
       Object.entries(req.headers).map(([name, value]) => {

src/index.ts

@@ -102,7 +102,11 @@ function normalizeGatewayEvent(event: GatewayEvent): HTTPGraphQLRequest {
 }
 
 function isV1Event(event: GatewayEvent): event is APIGatewayProxyEvent {
-  return !('version' in event);
+  // APIGatewayProxyEvent incorrectly omits `version` even though API Gateway v1
+  // events may include `version: "1.0"`
+  return (
+    !('version' in event) || ('version' in event && event.version === '1.0')
+  );
 }
 
 function isV2Event(event: GatewayEvent): event is APIGatewayProxyEventV2 {