Feature request - consider changing the RequestContext abbreviated response name `res` to `response`

[nateiler]

The res abbreviated property within the RequestContext seems out of place. Recommending using the full response name instead.

Before

type RequestContext = {
  request: Request;
  event: APIGatewayProxyEvent;
  context: Context;
  res: Response;
};

After

type RequestContext = {
  request: Request;
  event: APIGatewayProxyEvent;
  context: Context;
  response: Response;
};

[dreamorosi]

Hi @nateiler, thank you for opening this issue.

I think we picked that name primarily for brevity's sake, but I agree it's kind of out of place since all the others are spelled out.

Personally I'm not against the change, but I'll let @svozza decide.

[nateiler]

Yeah, I figured as much. Normally wouldn't raise such a picky item, but since it's still experimental, I figured we could start the discussion.

Could also replace request with req but I would vote on the long form.

[svozza]

I would be more in fsvour of changing request to req. I initially chose res because that's what most other frameworks although weirdly some also use request rather than req but I prefer the symmetry of using short form here,

[dreamorosi]

In the next days we'll most likely add more properties to the object like params for path parameters, so shortening everything could make sense, but like I said I'm ambivalent.

@nateiler please keep the feedback coming; it's more than welcome!