Amazon.Lambda.Annotations code generation explicit dependent on Amazon.Lambda.APIGatewayEvents

[Simonl9l]

Describe the bug

When experimenting with Amazon.Lambda.Annotations it seem that one also has to have Amazon.Lambda.APIGatewayEvents as a package reference in the project or the generated code can not compile.

This is neither noted in the Amazon.Lambda.Annotations README.md nor logically required if on just wants to deploy an HTTP Triggered Lambda without any API Gateway integration.

Expected Behavior

The generated code to compile without the need to add an explicit dependency, if this is required, that it should be a dependency of Amazon.Lambda.Annotations

Current Behavior

 "/Users/<user>/Projects/<path>/src/TestCGLambda/TestCGLambda.csproj" (default target) (1:7) ->
       (CoreCompile target) -> 
         /Users/<user>/Projects/<path>/TestCGLambda/Function.cs(11,5): error AWSLambda0104: Your project has a missing required package dependency. Please add a reference to the following package: Amazon.Lambda.APIGatewayEvents [/Users/l<user>/Projects/<path>/src/TestCGLambda/TestCGLambda.csproj]
         CSC : error AWSLambda0001: This is a bug. Please run the build with detailed verbosity (dotnet build --verbosity detailed) and file a bug at https://github.com/aws/aws-lambda-dotnet with the build output and stack trace Object reference not set to an instance of an object.   at Amazon.Lambda.Annotations.SourceGenerator.Models.TypeModelBuilder.Build(ITypeSymbol symbol, GeneratorExecutionContext context) [/Users/l<user>/Projects/<path>/src/TestCGLambda/TestCGLambda.csproj]

Reproduction Steps

build sample per README.md omitting Amazon.Lambda.APIGatewayEvents from referenced packages.

Possible Solution

No response

Additional Information/Context

No response

AWS .NET SDK and/or Package version used

Targeted .NET Platform

.NET 6

Operating System and version

OSX Monterey

[ashishdhingra]

@Simonl9l Please refer to the comment #1052 (comment) for the reasoning behind need to add explicit dependency in user project. Let me know if this resolves the confusion.

[Simonl9l]

@ashishdhingra thanks for the pointer...so how do I not need a dependency on that APIGatewayEvents package if I just want a raw HTTP Lambda with no specific binding? I don't now what in the code base has made this a requirement.

[ashishdhingra]

@ashishdhingra thanks for the pointer...so how do I not need a dependency on that APIGatewayEvents package if I just want a raw HTTP Lambda with no specific binding? I don't now what in the code base has made this a requirement.

@Simonl9l Could you please point me to sample which you are referring to in the issue description?

[Simonl9l]

Hi @ashishdhingra

So based on the Amazon.Lambda.Templates::6.2.0, I did a dotnet new lambda.EmptyFunction ... and then copied in the following sample fro the Amazon.Lambda.Annotation README.md into Function.cs:

public class Functions
{
    [LambdaFunction]
    [RestApi("/plus/{x}/{y}")]
    public int Plus(int x, int y)
    {
        return x + y;
    }
}

I noted that the RestAPI attribute syntax needed to be modified to [RestApi(LambdaHttpMethod.Get, "/plus/{x}/{y}")], and also modified that to be HttpApi.

That's it. The intent is not to have any need for the APIGatewayEvents so what do I need to change to eliminate ether dependency? Per the comment you reference It seems that I should not need this dependency not building a Lambda to connect with the API Gateway?

I apologies if Im making rookie mistakes here...and appreciate your continued assistance.

EDIT:
Seems commit fc7176398ca01e3e6d7b87f395b674bad57a6e06 added the dependency. This seems entirely unnecessary if one is not binding to APIGatewayEvents, and just providing an HTTP endpoint to say an ALB Target?

Whilst not need in this case, one assumes, in time, this will all be extended to support al the supported bindings?

[normj]

The generated code by the source generator use the types in Amazon.Lambda.APIGatewayEvents when using the RestApi or HttpApi attributes. Your code doesn't need to directly interact with the types Amazon.Lambda.APIGatewayEvents except for right now if you want to customize the response. Since your code is not interacting with Amazon.Lambda.APIGatewayEvents and the package is very small collection of types for serialization is there a reason you are asking to have no dependency on it?

[github-actions]

This issue has not received a response in 5 days. If you want to keep this issue open, please just leave a comment below and auto-close will be canceled.