Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Excessive stack depth comparing types #178

Open
terravisit opened this issue Jul 14, 2023 · 2 comments
Open

Excessive stack depth comparing types #178

terravisit opened this issue Jul 14, 2023 · 2 comments

Comments

@terravisit
Copy link

Description:
I encountered an error while using the "prisma-nestjs-graphql" generator that resulted in an "Excessive stack depth comparing types" error. This error prevents nestjs from starting correctly.

Steps to reproduce:

  1. Clone the test repository: repo-test
  2. Follow the installation instructions in the README to set up the project.
  3. Run the project and trigger the scenario that causes the error.
  4. See the "Excessive stack depth comparing types" error in the console.

Expected behavior:
I expected the nestjs start without any error in the console

Additional information:

  • Version of the "prisma-nestjs-graphql" generator: ^18.0.2
  • Operating system: windows
  • Prisma version: ^4.15.0
  • NestJS version: ^9.0.0

I think the problem comes from typescript, it's certainly not a problem in the code itself. Strange thing:

  • I transform the type SupplierUpdateInput in the file app.service.ts into any.
  • I save, the server starts without errors
  • I change the type back to SupplierUpdateInput and save again, the server starts without error too.

So it seems to me that the problem is only with server startup. Of course, in a real project, transforming types into any everywhere in the code and putting them back is not an option.

Reproducible repository:
I have created a test repository where the issue can be reproduced. You can find it here: repo-test

@terravisit
Copy link
Author

Resolved here : microsoft/TypeScript#55027 (comment)

@unlight
Copy link
Owner

unlight commented Jul 15, 2023

Related #65

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants