Add new items to ToDo list & add links to PRs

[JakobJingleheimer]

No description provided.

[GeoffreyBooth]

Do we want to distinguish between future items we're planning on doing and others that are tentative? Not sure if we have any tentative ones in this list though.

Losing the headings means we have no way to indicate an “in progress” item.

We should add an item for helper functions to reduce boilerplate. It's implied by the conclusion paragraph but since that was written I think we know we want to do that in some form.

[JakobJingleheimer]

Do we want to distinguish between future items we're planning on doing and others that are tentative? Not sure if we have any tentative ones in this list though.

Sure :) Cross that bridge when we come to it?

Losing the headings means we have no way to indicate an “in progress” item.

It'll have a PR link and not be checked off. Is that too subtle? (We also have a project in the nodejs/node repo)

We should add an item for helper functions to reduce boilerplate. It's implied by the conclusion paragraph but since that was written I think we know we want to do that in some form.

Oh! Yes, good shout. Will add. Done

[GeoffreyBooth]

The last paragraph repeats about the thread but this is fine.

[JakobJingleheimer]

Oh, actually now that you mention it, didn't Bradley already implement the channel communication thing? I'm thinking we just axe both of those from the final paragraph.

[GeoffreyBooth]

Oh, actually now that you mention it, didn't Bradley already implement the channel communication thing?

Not that I'm aware of? You can look at landed PRs that he's authored.

[cspotcode]

Pretty sure that a communication channel is already exposed to globalPreload code, if that's what you're talking about.
https://nodejs.org/dist/latest-v18.x/docs/api/esm.html#globalpreload

[JakobJingleheimer]

Pretty sure that a communication channel is already exposed to globalPreload code, if that's what you're talking about. https://nodejs.org/dist/latest-v18.x/docs/api/esm.html#globalpreload

Yes, exactly what I was thinking of (here's the PR nodejs/node#39240)