mrousavy / nitro

🔥 Insanely fast native C++, Swift or Kotlin modules with a statically compiled binding layer to JSI

https://nitro.margelo.com

MIT License

598 stars 15 forks source link

Nitro: RangeError: Maximum call stack size exceeded #90

Closed gtokman closed 2 months ago

gtokman commented 2 months ago

Description

I upgraded to 0.0.6 for nitro, and I'm seeing when running yarn run nitro-codegen in our spec.

    ⚙️   Generating specs for HybridObject "Send"...
        ❌  Failed to generate spec for Send! RangeError: Maximum call stack size exceeded
    ❌  No specs found in Send.nitro.ts!

mrousavy commented 2 months ago

Can you please create a simply reproduction?

gtokman commented 2 months ago

I'm having trouble reproducing it. Here's a video of me getting the error on 0.6.0 after adding the Send prefix to a model. https://github.com/user-attachments/assets/7a3dc39e-9672-44f7-a238-c963a46c0ea8

mrousavy commented 2 months ago

Is there another type called SendRequest?

mrousavy commented 2 months ago

Can you work around this issue for now by just avoiding the Send prefix? Then I'll consider it low priority and focus on other things until the release.

gtokman commented 2 months ago

Yeah, it's not a blocker. I can't reliably reproduce it either 😅