|
| 1 | +# Node.js Technical Steering Committee (TSC) Meeting 2024-08-21 |
| 2 | + |
| 3 | +## Links |
| 4 | + |
| 5 | +* **Recording**: <https://www.youtube.com/watch?v=1t6DuX4-Ae4> |
| 6 | +* **GitHub Issue**:<https://github.com/nodejs/TSC/issues/1607> |
| 7 | + |
| 8 | +## Present |
| 9 | + |
| 10 | +* Antoine du Hamel @aduh95 (voting member) |
| 11 | +* Yagiz Nizipli @anonrig (voting member) |
| 12 | +* Benjamin Gruenbaum @benjamingr (voting member) |
| 13 | +* Ruben Bridgewater @BridgeAR (voting member) |
| 14 | +* Joyee Cheung @joyeecheung (voting member) |
| 15 | +* Chengzhong Wu @legendecas (voting member) |
| 16 | +* Marco Ippolito @marco-ippolito (voting member) |
| 17 | +* Michael Dawson @mhdawson (voting member) |
| 18 | +* Moshe Atlow @MoLow (voting member) |
| 19 | +* Rafael Gonzaga @RafaelGSS (voting member) |
| 20 | +* Robert Nagy @ronag (voting member) |
| 21 | +* Ruy Adorno @ruyadorno (voting member) |
| 22 | +* Paolo Insogna @ShogunPanda (voting member) |
| 23 | + |
| 24 | +## Agenda |
| 25 | + |
| 26 | +### Announcements |
| 27 | + |
| 28 | +* Marco 20.17.0 will be released just after this meeting |
| 29 | +* Rafael, 22.7.0 will be released tomorrow. |
| 30 | + |
| 31 | +### Reminders |
| 32 | + |
| 33 | +* Remember to nominate people for the [contributor spotlight](https://github.com/nodejs/node/blob/main/doc/contributing/reconizing-contributors.md#bi-monthly-contributor-spotlight) |
| 34 | + |
| 35 | +### CPC and Board Meeting Updates |
| 36 | + |
| 37 | +*Extracted from **tsc-agenda** labeled issues and pull requests from the **nodejs org** prior to the meeting. |
| 38 | + |
| 39 | +* No updates this week |
| 40 | + |
| 41 | +### nodejs/node |
| 42 | + |
| 43 | +* lib: Symbol.dispose should be enabled with experimental flag [#54329](https://github.com/nodejs/node/pull/54329) |
| 44 | + * two issues |
| 45 | + * the whole ecmascript feature is unfinished, should there be an experimental warning? |
| 46 | + * for the future how do we want experimental global api additions |
| 47 | + * in v8 experimental features are hidden behind a flag while incomplete |
| 48 | + * the suggestion could be to apply SemVer major for each experimental feature exposed to |
| 49 | + the global scope, or basic TSC consensus for each one. |
| 50 | + * Marco, generally we should be cautious on polyfills, should likely mark as SemVer major as |
| 51 | + the default approach |
| 52 | + * Benjamin, in the specific case of Symbol.dispse, it implements the minimum to allow other |
| 53 | + tools to prototype and implement. In retrospective, agree it should go through the TSC, |
| 54 | + make it SemVer major by default. We have process/tooling to handle that. Last year warning |
| 55 | + might have made sense in this specific case, but not so much anymore, but don’t feel to |
| 56 | + strongly. |
| 57 | + * Michael any concern over policy of making SemVer major by default |
| 58 | + * Marco, agree that for specific instance don’t need to do anything, but in the future polyfills |
| 59 | + should be SemVer major. |
| 60 | + * Chengzhong, when exposed on Global scope -> SemVer major. In addition should it require |
| 61 | + an experimental flag to start if feature is incomplete |
| 62 | + * Antoine, if we use experimental flag, that is not SemVer major, right? |
| 63 | + * Chengzhong, agree without experimental flag SemVer major, but not if it has the |
| 64 | + experimental flag |
| 65 | + * Benjamin, believe it may depend on the situation, so probably better to mark always as |
| 66 | + SemVer major to have it go through the TSC for analysis/discussion. |
| 67 | + * Antoine, not sure if we need that for additions which are under a flag. So don’t think |
| 68 | + something under an experimental flag should need SemVer major |
| 69 | + * Benjamin, if there are cross-cutting concerns that originator is not aware on. |
| 70 | + * Antoine, would prefer to just make it informal. |
| 71 | + * Marco, not SemVer major in terms of breaking, but is in terms of consensus, so marking as |
| 72 | + SemVer major. |
| 73 | + * Antoine, not opposed to always be SemVer major if understanding is that it would be |
| 74 | + removed after TSC discussion. |
| 75 | + * Consensus is |
| 76 | + * that we will ask that experimental APIs on the Global scope will always be |
| 77 | + marked as SemVer major to start, expectation is that TSC will typically remove after |
| 78 | + discussion if its behind an experimental flag. |
| 79 | + * won’t add experimental warning to Symbol.dispose |
| 80 | + * document the new process in <https://github.com/nodejs/node/pull/54330> |
| 81 | + |
| 82 | +* timers: add sleep alias[#54404]( https://github.com/nodejs/node/pull/54404) |
| 83 | + * Paolo seems like its easy to mess up |
| 84 | + * Benjamin, main concern is that we already have 4-5 ways to schedule a timer |
| 85 | + * Paolo, we could remote the override, |
| 86 | + * Benjamin, might be better to remove scheduler.wait() |
| 87 | + * Michael, is it really worth it, lets document scheduler.wait() better |
| 88 | + * Paolo, agree we can just publish scheduler.wait(), but would be good to add warning if you |
| 89 | + call setTimeout without await after importing the async version. |
| 90 | + * Paolo, could this be SemVer minor. After some discussion seems reasonable to propose, |
| 91 | + * Chengzhong, echo scheduler api. Active discussion on experimental api, some discussion in |
| 92 | + <https://github.com/WICG/scheduling-apis/issues/95>, about the naming please comment there |
| 93 | + if you think it should change. |
| 94 | + |
| 95 | +### nodejs/admin |
| 96 | + |
| 97 | +* Conversion to Enterprise account [#905](https://github.com/nodejs/admin/issues/905) |
| 98 | + * Issue closed, nothing to discuss. |
| 99 | + |
| 100 | +## Strategic Initiatives |
| 101 | + |
| 102 | +## Upcoming Meetings |
| 103 | + |
| 104 | +* **Node.js Project Calendar**: <https://nodejs.org/calendar> |
| 105 | + |
| 106 | +Click `+GoogleCalendar` at the bottom right to add to your own Google calendar. |
0 commit comments