Fix TS builds, Lambda Runtime, Add Array input support, and fix tests#2
Fix TS builds, Lambda Runtime, Add Array input support, and fix tests#2deldrid1 wants to merge 8 commits intokdy1:masterfrom
Conversation
|
I did have to remove the nuxt example due to some peer dependency issues (I think the @next branch of @trpc/client isn't up to date with the fact that |
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
| "examples/with-interop", | ||
| "examples/with-serverless", | ||
| "examples/with-fastify", | ||
| "examples/with-nuxtjs" |
There was a problem hiding this comment.
Is removal of with-nuxtjs intentional?
There was a problem hiding this comment.
with-fastify wasn't, but nuxt was. See the comment up above
Unfortunately, there are some peer dependency issues that prevented me from getting that one going. Rather than have npm install problems, I opted to just take it out here for the time being.
| path: queryPath, | ||
| procedure, | ||
| }); | ||
| if(procedure._def.type === "mutation" || procedure._def.type === "query" ){ |
There was a problem hiding this comment.
This seems to exclude subscriptions from the cache. Is it your intention?
There was a problem hiding this comment.
Can you add some comments about the intention?
There was a problem hiding this comment.
I don't think WebSockets or Server-sent Events (SSE) are being properly handled by openapi-trpc? Happy to revert this if you don't think its a valid change.
| import type { RootConfig } from '@trpc/server/dist/core/internals/config'; | ||
| import { TRPC_ERROR_CODE_KEY } from '@trpc/server/rpc'; | ||
| import type { RouterDef } from '@trpc/server/src/core/router'; | ||
| import type { APIGatewayProxyEvent, APIGatewayProxyEventV2 } from 'aws-lambda'; |
There was a problem hiding this comment.
I think an import of aws-lambda should be moved to non-shared file.
There was a problem hiding this comment.
Its a type import so nothing runtime is affected - its only used during TypeScript builds of the project and is a devDependency. Can you justify the additional complexity of simply not listing it here?
Co-authored-by: Donny/강동윤 <kdy1997.dev@gmail.com>
deldrid1
changed the title
return an unknown error even if not TRPCError
3 participants
Hi - I ran into some issues when attempting to use this with
@trpc/serverv11, and after digging in was impressed anything was working at all given the number of TypeScript issues!I'm not 100% sure that I have everything exactly right, but tests are passing now and its working for my use case.