Rewrite in N-API - #3
Conversation
|
OK, after lots of wailing and grinding of teeth I've got CI passing on all three platforms. This should obviate the need for #2 or #1 — all N-API modules are context-aware, and I'm imagining there will be fewer of those version-check fixes needed with N-API. And if we land this we should be able to bump the version we use in the |
|
I'm going back through and trying to figure out what the blockers are for various tasks. If we could land this, it would be a necessary step toward taking pulsar-edit/pulsar#1600 out of draft. For that reason, I tagged the two most active participants at this moment. @mauricioszabo, this one's more in your wheelhouse, but it's OK if you don't have time. Eventually I might make this a “speak now or forever hold…” type of PR, but I'll give it a shot at getting reviewed first. |
This needs updates to build against Node 22 (and Electron 32.3.3 because of its newer V8 version), so we might as well take this opportunity to migrate to N-API.
Specs are still in CoffeeScript, and I plan to address that next… but I wasn't going to decaf the specs until after this PR so we can more easily prove lack of regressions.