yes, our team also manages our opennext

yes, our team also manages our opennext integration, what's the question/issue? 🙂
14 Replies
Jorel
Jorel2mo ago
Ah cool! I created an issue 2 months ago, in the opennextjs/opennextjs-cloudflare repo, but havent gotten any feedback, so I'm wondering if that repo is the wrong place to post issues? 🙂 It's https://github.com/opennextjs/opennextjs-cloudflare/issues/760, essentially the Env typing was colliding in the opennext build if I had a service binding to another worker with types setup properly.
Dario
DarioOP2mo ago
That is the right repo for issues 🙂👍 Sorry for the issue not having been looked thus far, there aren't many engineers working on the open-next adapter and currently there are some high priority things that we're trying to address (specifically performance issues)
Jorel
Jorel2mo ago
Gotcha, that's perfectly understandable. Thanks Dario!
Dario
DarioOP2mo ago
anyways I just read your issue, are you sure that it is an open-next issue? does the same not apply when you use standard workers? 🤔
Jorel
Jorel2mo ago
Nope, it's the next build that fails. So something in the next build pipeline is wonky with the worker types. I have 3 workers, 2 of them work fine together The third (opennext) is the problem. I've tried to do the exact same binding just to verify, and got no issues between two TS workers
Dario
DarioOP2mo ago
I see 🤔 I'll try to have a look
Jorel
Jorel2mo ago
my repro repo should still be valid, but it's not changed since 2 months ago!
Dario
DarioOP2mo ago
ah... by the way... does disabling the Next.js typescript checking help?
Dario
DarioOP2mo ago
next.config.js: typescript | Next.js
Next.js reports TypeScript errors by default. Learn to opt-out of this behavior here.
Jorel
Jorel2mo ago
I believe I tried to do that, but it also failed... But my memory might fail me here. Can check!
Dario
DarioOP2mo ago
(ignoreBuildErrors) if you can that could be helpful, but I also can when looking into it 😄👍
Jorel
Jorel2mo ago
typescript: {
ignoreBuildErrors: true,
}
typescript: {
ignoreBuildErrors: true,
}
did indeed solve the problem! Or well, works as a workaround. I think what I remember trying is to exclude the other worker in the tsconfig, which didn't work. So must have not tried this. But then we lose all type safety during build, which perhaps is not the best 🙂 Could possible build a CI/CD pipe that has a manual typecheck step, but that still gets the type error from the other worker 🙁 Right now my workaround is to NOT type the worker and just call .fetch to it, building a manual request
Dario
DarioOP2mo ago
yeah I mean, ignoring the typescript at build is not really a good solution, I was just wondering where in the process fails 🙂
Jorel
Jorel2mo ago
Rest of next build succeeded with ignoreBuildErrors: true 👍

Did you find this page helpful?