From 6b9737a81757c8d762e3d955e514f9687a78d910 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Wed, 25 Sep 2024 22:46:04 +0200 Subject: [PATCH] doc: generate weekly retro (#36) Co-authored-by: Create or Update Pull Request Action --- retros/2024-09-15.md | 234 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 234 insertions(+) create mode 100644 retros/2024-09-15.md diff --git a/retros/2024-09-15.md b/retros/2024-09-15.md new file mode 100644 index 0000000..abfd3bd --- /dev/null +++ b/retros/2024-09-15.md @@ -0,0 +1,234 @@ +# Retrospective for `nodejs` from 2024-09-15 to 2024-09-22 + +Reporting on 30 Issues from 23 authors, 70 Pull Requests from 26 authors, and 10 Discussions from 9 authors. + + +### reliability + +#### Open Issues + +- CI Reliability 2024-09-22 ([#974](https://github.com/nodejs/reliability/issues/974)) +- CI Reliability 2024-09-21 ([#973](https://github.com/nodejs/reliability/issues/973)) +- CI Reliability 2024-09-20 ([#972](https://github.com/nodejs/reliability/issues/972)) +- CI Reliability 2024-09-19 ([#971](https://github.com/nodejs/reliability/issues/971)) + +### node + +#### Open Issues + +- Could not report code coverage for file in sourcemap ([#55054](https://github.com/nodejs/node/issues/55054)) +- Nominating @pmarchini to be a collaborator ([#55053](https://github.com/nodejs/node/issues/55053)) +- Exception during run (mocha) ([#55049](https://github.com/nodejs/node/issues/55049)) +- pipe() allow destroying source! ([#55010](https://github.com/nodejs/node/issues/55010)) +- Any idea when Single executable application feature will reach stable status? ([#55007](https://github.com/nodejs/node/issues/55007)) +- Support `net.BlockList` in `http.Agent` options ([#55000](https://github.com/nodejs/node/issues/55000)) +- http2 server ECONNRESET on http1.1 request ([#54998](https://github.com/nodejs/node/issues/54998)) + +#### Open Pull Requests + +- deps: patch V8 to 12.9.202.19 ([#55057](https://github.com/nodejs/node/pull/55057)) +- deps: update timezone to 2024b ([#55056](https://github.com/nodejs/node/pull/55056)) +- test_runner: throw on invalid source map ([#55055](https://github.com/nodejs/node/pull/55055)) +- src: add and use MaybeStackBuffer v8::LocalVector ([#55052](https://github.com/nodejs/node/pull/55052)) +- test: execute shell directly for refresh() ([#55051](https://github.com/nodejs/node/pull/55051)) +- lib: remove lib/internal/idna.js ([#55050](https://github.com/nodejs/node/pull/55050)) +- stream: construct correct prototype to transferred streams ([#55047](https://github.com/nodejs/node/pull/55047)) +- lib: prefer optional chaining ([#55045](https://github.com/nodejs/node/pull/55045)) +- lib: prefer logical assignment ([#55044](https://github.com/nodejs/node/pull/55044)) +- doc: correct `cleanup` option in stream.(promises.)finished ([#55043](https://github.com/nodejs/node/pull/55043)) +- tools: unlock versions of irrelevant DB deps ([#55042](https://github.com/nodejs/node/pull/55042)) +- crypto: ensure invalid SubtleCrypto JWK data import results in DataError ([#55041](https://github.com/nodejs/node/pull/55041)) +- benchmark: add webstorage benchmark ([#55040](https://github.com/nodejs/node/pull/55040)) +- test_runner: require `--enable-source-maps` for source map coverage ([#55039](https://github.com/nodejs/node/pull/55039)) +- test_runner: report error on missing sourcemap source ([#55037](https://github.com/nodejs/node/pull/55037)) +- test: make `test-runner-assert` more robust ([#55036](https://github.com/nodejs/node/pull/55036)) +- process: remove `process.assert` ([#55035](https://github.com/nodejs/node/pull/55035)) +- fs: check subdir correctly in cpSync ([#55033](https://github.com/nodejs/node/pull/55033)) +- meta: re-enable PR stalebot ([#55032](https://github.com/nodejs/node/pull/55032)) +- test: update tls test to support OpenSSL32 ([#55030](https://github.com/nodejs/node/pull/55030)) +- test,crypto: update WebCryptoAPI WPT ([#55029](https://github.com/nodejs/node/pull/55029)) +- test: do not assume `process.execPath` contains no spaces ([#55028](https://github.com/nodejs/node/pull/55028)) +- benchmark: use options.filename when, for esm, require.main.filename is undefined ([#55027](https://github.com/nodejs/node/pull/55027)) +- test: fix `test-vm-context-dont-contextify` when path contains a space ([#55026](https://github.com/nodejs/node/pull/55026)) +- build: display free disk space after build in the test-macOS workflow ([#55025](https://github.com/nodejs/node/pull/55025)) +- src: decode native error messages as UTF-8 ([#55024](https://github.com/nodejs/node/pull/55024)) +- doc: add missing EventSource docs to globals ([#55022](https://github.com/nodejs/node/pull/55022)) +- zlib: set zlib.bytesRead end-of-life ([#55020](https://github.com/nodejs/node/pull/55020)) +- crypto: runtime deprecate crypto.fips ([#55019](https://github.com/nodejs/node/pull/55019)) +- util: runtime deprecate toUSVString() ([#55018](https://github.com/nodejs/node/pull/55018)) +- url: runtime deprecate url.parse ([#55017](https://github.com/nodejs/node/pull/55017)) +- deps: update V8 to 13.0 ([#55014](https://github.com/nodejs/node/pull/55014)) +- test: remove interval and give more time to unsync ([#55006](https://github.com/nodejs/node/pull/55006)) +- test: deflake test-http-remove-header-stays-removed ([#55004](https://github.com/nodejs/node/pull/55004)) +- esm: do not interpret `"main"` as a URL ([#55003](https://github.com/nodejs/node/pull/55003)) + +#### Open Discussions + +- How to solve this error npm ERR! code EEXIST? ([#55002](https://github.com/orgs/nodejs/discussions/55002)) +- Warning potential scam emails ([#53726](https://github.com/orgs/nodejs/discussions/53726)) +- Import module from global NPM packages if it cannot find module in local package ([#54916](https://github.com/orgs/nodejs/discussions/54916)) +- try to build an extended version of node to log every undefined property visiting ([#46960](https://github.com/orgs/nodejs/discussions/46960)) +- Understanding mock.restoreAll() and mock.reset() in Node.js Test Runner ([#54887](https://github.com/orgs/nodejs/discussions/54887)) +- now that --experimental-network-imports has been removed, is there a work around to import files from a server? ([#54948](https://github.com/orgs/nodejs/discussions/54948)) + +#### Merged Pull Requests + +- test: fix `soucre` to `source` ([#55038](https://github.com/nodejs/node/pull/55038)) +- doc: cover --experimental-test-module-mocks flag ([#55021](https://github.com/nodejs/node/pull/55021)) +- test: adjust tls-set-ciphers for OpenSSL32 ([#55016](https://github.com/nodejs/node/pull/55016)) +- src: revert filesystem::path changes ([#55015](https://github.com/nodejs/node/pull/55015)) +- test_runner: replace ansi clear with ansi reset ([#55013](https://github.com/nodejs/node/pull/55013)) +- build: fix eslint makefile target ([#54999](https://github.com/nodejs/node/pull/54999)) + +#### Closed Issues + +- Deprecation Warning After Node Update ([#55046](https://github.com/nodejs/node/issues/55046)) +- Can not build on macOS 14.6.1 with clang 16 ([#55023](https://github.com/nodejs/node/issues/55023)) +- Please add native support of webrtc in node. ([#55012](https://github.com/nodejs/node/issues/55012)) +- JavaScript error ([#55011](https://github.com/nodejs/node/issues/55011)) +- `dot` reporter causes the terminal to clear since Node v22.8.0 ([#55009](https://github.com/nodejs/node/issues/55009)) +- [ invalid ] ([#55008](https://github.com/nodejs/node/issues/55008)) +- HTTPS CRL Usage not working ([#55005](https://github.com/nodejs/node/issues/55005)) + +#### Closed Pull Requests + +- test_runner: don't set coverage threshold exitCode until `test:summary` ([#55048](https://github.com/nodejs/node/pull/55048)) +- domain: EoL deprecate `MakeCallback` with `domain` property ([#55034](https://github.com/nodejs/node/pull/55034)) +- added test file ([#55031](https://github.com/nodejs/node/pull/55031)) +- test: add asserts to validate test assumptions ([#54997](https://github.com/nodejs/node/pull/54997)) + +### web-server-frameworks + +#### Open Issues + +- Node.js Web Server Frameworks Meeting 2024-09-24 ([#124](https://github.com/nodejs/web-server-frameworks/issues/124)) + +### loaders + +#### Open Issues + +- Node.js Loaders Team Meeting 2024-09-24 ([#227](https://github.com/nodejs/loaders/issues/227)) + +### build + +#### Open Pull Requests + +- promote: add expected assets for Node.js v23 ([#3915](https://github.com/nodejs/build/pull/3915)) + +#### Closed Issues + +- FreeBSD builds timing out ([#3914](https://github.com/nodejs/build/issues/3914)) + +### undici + +#### Open Pull Requests + +- feat: add new hooks api ([#3636](https://github.com/nodejs/undici/pull/3636)) +- fix(websocket): use binary operations instead of a Set for closeStatus ([#3635](https://github.com/nodejs/undici/pull/3635)) +- Update WPT ([#3634](https://github.com/nodejs/undici/pull/3634)) +- Remove symbols from web specs ([#3633](https://github.com/nodejs/undici/pull/3633)) +- build(deps): bump node from `3cb4748` to `83b4d7b` in /build ([#3621](https://github.com/nodejs/undici/pull/3621)) + +#### Open Discussions + +- What's the best way to reset the MockPool between tests? ([#3619](https://github.com/nodejs/undici/discussions/3619)) +- HTTP/2 client inside HTTP/1.1 proxy connection ([#3613](https://github.com/nodejs/undici/discussions/3613)) +- Does undici take care of consuming the response body if throwOnError: true? ([#3440](https://github.com/nodejs/undici/discussions/3440)) +- Backporting of PRs ([#3526](https://github.com/nodejs/undici/discussions/3526)) + +#### Merged Pull Requests + +- handle body errors ([#3632](https://github.com/nodejs/undici/pull/3632)) +- make cloned request inherit dispatcher ([#3631](https://github.com/nodejs/undici/pull/3631)) +- fix: fire close on failed WebSocket connection ([#3628](https://github.com/nodejs/undici/pull/3628)) +- Update WPT ([#3626](https://github.com/nodejs/undici/pull/3626)) +- append crlf to formdata body ([#3625](https://github.com/nodejs/undici/pull/3625)) +- Update WPT ([#3622](https://github.com/nodejs/undici/pull/3622)) + +#### Closed Issues + +- undici.Request.clone does not copy dispatcher ([#3630](https://github.com/nodejs/undici/issues/3630)) +- 🏁 RoSctober 2024: The Open Source Developer Challenge is about to start! 🎉 ([#3623](https://github.com/nodejs/undici/issues/3623)) +- When using fetch for FormData upload files, lack of"\r\n" ([#3624](https://github.com/nodejs/undici/issues/3624)) + +#### Closed Pull Requests + +- feat: new hooks ([#3629](https://github.com/nodejs/undici/pull/3629)) +- Revert "fix: fire `close` on failed WebSocket connection" ([#3627](https://github.com/nodejs/undici/pull/3627)) +- fix: dont crash on invalid brotli payload ([#3620](https://github.com/nodejs/undici/pull/3620)) + +### jenkins-alerts + +#### Closed Issues + +- test-rackspace-win2022_vs2022-x64-4 has low disk space ([#2877](https://github.com/nodejs/jenkins-alerts/issues/2877)) + +### nodejs-dist-indexer + +#### Open Issues + +- Test & Maybe Release workflow fails ([#32](https://github.com/nodejs/nodejs-dist-indexer/issues/32)) + +#### Merged Pull Requests + +- chore(deps): bump actions/setup-node from 4.0.3 to 4.0.4 ([#31](https://github.com/nodejs/nodejs-dist-indexer/pull/31)) + +### Release + +#### Open Issues + +- @nodejs/releasers health ([#1036](https://github.com/nodejs/Release/issues/1036)) +- Node.js 23 release plan ([#1034](https://github.com/nodejs/Release/issues/1034)) + +#### Open Pull Requests + +- chore: remove lts team ([#1037](https://github.com/nodejs/Release/pull/1037)) + +#### Merged Pull Requests + +- chore: add minutes for 2024-09-19 meeting ([#1035](https://github.com/nodejs/Release/pull/1035)) + +### readable-stream + +#### Open Issues + +- process is not defined ([#540](https://github.com/nodejs/readable-stream/issues/540)) + +### nodejs-dependency-vuln-assessments + +#### Open Issues + +- Plan to port the fixes to the openssl versions (CVE-2024-6119) in use for nodejs. ([#192](https://github.com/nodejs/nodejs-dependency-vuln-assessments/issues/192)) + +### node-auto-test + +#### Open Pull Requests + +- tools: add WPT updater for specific subsystems ([#73](https://github.com/nodejs/node-auto-test/pull/73)) + +### TSC + +#### Open Pull Requests + +- Choosing a marker to customize default exports returned by `require(esm)` ([#1622](https://github.com/nodejs/TSC/pull/1622)) +- doc: add minutes for meeting Sep 18 2024 ([#1620](https://github.com/nodejs/TSC/pull/1620)) + +#### Merged Pull Requests + +- Start a vote on require(esm) markers ([#1621](https://github.com/nodejs/TSC/pull/1621)) + +### branch-diff + +#### Open Pull Requests + +- chore(deps): bump actions/setup-node from 4.0.3 to 4.0.4 ([#73](https://github.com/nodejs/branch-diff/pull/73)) + +### node-core-utils + +#### Open Pull Requests + +- fix: followup fix for supported wpt jsons ([#855](https://github.com/nodejs/node-core-utils/pull/855)) + +#### Merged Pull Requests + +- chore(wpt): warn for non-existent wpt path ([#856](https://github.com/nodejs/node-core-utils/pull/856)) \ No newline at end of file