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

Maximum call stack size exceeded when fallthroughAttributes: true #4761

Closed
ferferga opened this issue Aug 30, 2024 · 1 comment
Closed

Maximum call stack size exceeded when fallthroughAttributes: true #4761

ferferga opened this issue Aug 30, 2024 · 1 comment
Labels
bug Something isn't working good reproduction ✨ This issue provides a good reproduction, we will be able to investigate it first

Comments

@ferferga
Copy link
Contributor

Vue - Official extension or vue-tsc version

2.1.2

VSCode version

1.92.2

Vue version

3.4.37

TypeScript version

5.5.4

System Info

No response

Steps to reproduce

  • Clone the repo: git clone -b fallthrough-attrs-reproduction https://github.com/jellyfin/jellyfin-vue
  • Change to the appropiate workspace and install dependencies: cd frontend && npm ci
  • Run typecheck: npm run typecheck

What is expected?

Type checking passes successfully and fallthrough attributes are properly inferred.

What is actually happening?

The following stacktrace is printed:

/workspaces/jellyfin-vue/node_modules/vue-tsc/index.js:45
            throw err;
            ^

RangeError: Maximum call stack size exceeded
    at getObjectTypeInstantiation (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61411:38)
    at instantiateTypeWorker (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61670:16)
    at instantiateTypeWithAlias (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61650:20)
    at instantiateType (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61628:29)
    at getMappedType (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61307:67)
    at /workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61433:56
    at map (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:199:19)
    at getObjectTypeInstantiation (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61433:29)
    at instantiateTypeWorker (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61670:16)
    at instantiateTypeWithAlias (/workspaces/jellyfin-vue/node_modules/typescript/lib/tsc.js:61650:20)

Link to minimal reproduction

https://github.com/jellyfin/jellyfin-vue

Any additional comments?

  • Disabling fallthroughAttributes completes the typechecking (with type errors though) successfully.
@johnsoncodehk johnsoncodehk added bug Something isn't working good reproduction ✨ This issue provides a good reproduction, we will be able to investigate it first and removed performance labels Aug 31, 2024
@ferferga
Copy link
Contributor Author

ferferga commented Sep 1, 2024

@johnsoncodehk Thank you very much for the quick fix! Glad that the reproduction was useful (I was worried that it was not minimal at all, but couldn't reproduce it in isolation).

This was something I was really looking forward to, thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good reproduction ✨ This issue provides a good reproduction, we will be able to investigate it first
Projects
None yet
Development

No branches or pull requests

3 participants