|
| 1 | +# Node.js Technical Steering Committee (TSC) Meeting 2025-04-23 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: <https://www.youtube.com/watch?v=YH_67jufU0s> |
| 6 | +* **GitHub Issue**: <https://github.com/nodejs/TSC/issues/1728> |
| 7 | + |
| 8 | +## Present |
| 9 | + |
| 10 | +* Antoine du Hamel @aduh95 (voting member) |
| 11 | +* Yagiz Nizipli @anonrig (voting member) |
| 12 | +* Ruben Bridgewater @BridgeAR (voting member) |
| 13 | +* Joyee Cheung @joyeecheung (voting member) |
| 14 | +* Michael Dawson @mhdawson (voting member) |
| 15 | +* Rafael Gonzaga @RafaelGSS (voting member) |
| 16 | +* Robert Nagy @ronag (voting member) |
| 17 | +* Ruy Adorno @ruyadorno (voting member) |
| 18 | + |
| 19 | +## Agenda |
| 20 | + |
| 21 | +### Announcements |
| 22 | + |
| 23 | +* No announcements this week |
| 24 | +* Antoine - Node.js 24.0.0 will not be shipped today, due to delay in landing V8 update |
| 25 | + |
| 26 | +### Reminders |
| 27 | + |
| 28 | +* Remember to nominate people for the [contributor spotlight](https://github.com/nodejs/node/blob/main/doc/contributing/reconizing-contributors.md#bi-monthly-contributor-spotlight) |
| 29 | + |
| 30 | +### CPC and Board Meeting Updates |
| 31 | + |
| 32 | +*Extracted from **tsc-agenda** labeled issues and pull requests from the **nodejs org** prior to the meeting. |
| 33 | + |
| 34 | +* No updates this week. |
| 35 | + |
| 36 | +### nodejs/node |
| 37 | + |
| 38 | +* doc: doc that proj has access to linked in [#57401](https://github.com/nodejs/node/pull/57401) |
| 39 | + * Kylie responded and Michael will follow up with her. |
| 40 | + |
| 41 | +* Deadlock at process shutdown [#54918](https://github.com/nodejs/node/issues/54918) |
| 42 | + * Robert, another project posted on the issue their solution so might be something to consider. |
| 43 | + * Joyee, initial implementation of custom v8 platform noted that related to task runner it would |
| 44 | + not be an issue now, but would be if tasks posted on background post another foreground task and deadlocks would |
| 45 | + happen. There was already an attempt not to drain the posted tasks, but that resulted in |
| 46 | + other failures in the test suite. Our implementation has not been implementing the platform |
| 47 | + correctly from the start. Needs someone knowledgeable. Joyee might be able to |
| 48 | + spend some cycles but not guarantee as have other things I need to do. Learned a bit |
| 49 | + when working on issues with V8 update. Chromium handles for people who we might follow |
| 50 | + up with are: ahaas and mlippautz |
| 51 | +* Robert, betting Ben involved. Michael believe we need to find funds to be able to get him |
| 52 | + involved, still not sure were we’d get funds from. |
| 53 | +* Joyee, in terms of tests, adding a gc in the shutdown may be a work around in terms of flaky |
| 54 | + tests, and it was proved out on one of the tests - <https://github.com/nodejs/node/pull/57930>. |
| 55 | + generally make sure there is no gc task on isolate disposal. |
| 56 | + |
| 57 | +### nodejs/nodejs.org |
| 58 | + |
| 59 | +* feat: supporter page and updated home page [#7552](https://github.com/nodejs/nodejs.org/pull/7552) |
| 60 | + * still waiting on board for a proposal on what would be a workable approach. |
| 61 | + |
| 62 | +### nodejs/TSC |
| 63 | + |
| 64 | +* Let's talk about the CI situation [#1614](https://github.com/nodejs/TSC/issues/1614) |
| 65 | + * Covered in discussion on Deadlock at process shutdown [#54918](https://github.com/nodejs/node/issues/54918) |
| 66 | + |
| 67 | +* 24 release delayed |
| 68 | + * Joyee |
| 69 | + * a while since we ran the full CI |
| 70 | + * one flake is blocked on CL that almost is landing in V8 |
| 71 | + * remaining issues are |
| 72 | + * smartos, patch exists, not sure if plan is to upstream to V8 |
| 73 | + * still a few failures on windows, crashes, don’t fully understand issues there not sure if |
| 74 | + anybody is looking at those |
| 75 | + * some simd crashes on arm V7, not sure anybody looking at that, managed to make it go |
| 76 | + away with a revert to disable a garbage collection option by default. Might be a work |
| 77 | + around |
| 78 | + * apparch from Windows failures, the rest may be fixable/work aroundable |
| 79 | + * Michael |
| 80 | + * options for shipping |
| 81 | + * joyee, main issues the 2 failing tests on windows. |
| 82 | + * Joyee next steps |
| 83 | + * get V8 CLs landed |
| 84 | + * Delay until at least next week |
| 85 | + * Michael - see where we are next week to see if we can land the V8 update and ship even |
| 86 | + if there are a few outstanding issue. |
| 87 | + * Joyee do you know why there is a conflict with simdutf |
| 88 | + * Yagiz we need to use wither use V8 version or our version. |
| 89 | + * Joyee - seems like in current V8 update branch it removes our version. |
| 90 | + * Yagiz - may make sense to use our version instead of V8 one since we don’t update |
| 91 | + V8 all that much. |
| 92 | + * Joyee, to get the least problems we may want to have two copies, one prefixed so they |
| 93 | + Co-exist. There is a PR to the V8 node version to use 2 copies. We should probably do |
| 94 | + the same. |
| 95 | + * Yagiz it is a large library so will increase size likely in the order of 2-3 meg |
| 96 | + * Joyee, template based so should the resulting size be based on what you use? And we |
| 97 | + don’t use a lot so should it not be able to trim out unused stuff. We can see if it |
| 98 | + has a lot of size impact. |
| 99 | + * Michael, would changing between 1 or 2 copies be SemVer major? |
| 100 | + * Yagiz, should not be as headers are not visible |
| 101 | + * Joyee, so should be able to just ship what is easiest to start and then adjust. For now |
| 102 | + just continue to remove in the V8 update branch and then decide what to do. |
| 103 | + |
| 104 | +## Strategic Initiatives |
| 105 | + |
| 106 | +* Antoine core promises - No update |
| 107 | +* Joyee - startup snapshot |
| 108 | +* Michael - Next-10 survey questions should be landed soon and survey go out. |
| 109 | + |
| 110 | +## Upcoming Meetings |
| 111 | + |
| 112 | +* **Node.js Project Calendar**: <https://nodejs.org/calendar> |
| 113 | + |
| 114 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
0 commit comments