|
| 1 | +#### Meeting from: September 1st, 2021 |
| 2 | + |
| 3 | +# Open RFC Meeting (npm) |
| 4 | + |
| 5 | +### Attendees |
| 6 | +- Darcy Clarke (@darcyclarke) |
| 7 | +- Gar (@wraithgar) |
| 8 | +- Vincent Bailly (@VincentBailly) |
| 9 | +- Isaac Z. Schlueter (@isaacs) |
| 10 | +- Nathan Fritz (@fritzy) |
| 11 | +- Owen Buckley (@) |
| 12 | +- Nathan LaFreniere (@nlf) |
| 13 | + |
| 14 | +### Previously... |
| 15 | + |
| 16 | +- [2021-08-25](https://github.com/npm/rfcs/blob/latest/meetings/2021-08-25.md) |
| 17 | + |
| 18 | +### Agenda |
| 19 | + |
| 20 | +1. **Housekeeping** |
| 21 | + 1. Introduction(s) |
| 22 | + 1. [Code of Conduct Acknowledgement](https://www.npmjs.com/policies/conduct) |
| 23 | + 1. Outline Intentions & Desired Outcomes |
| 24 | + 1. Announcements |
| 25 | +1. **`npm` v8 Plan:** https://github.com/npm/rfcs/issues/445 ~ @nlf |
| 26 | +1. **PR**: [#441 addendum: overrides apply if value matches, as well as key](https://github.com/npm/rfcs/pull/441) - @isaacs |
| 27 | +1. **PR**: [#437 RFC: Robust Lifecycle Scripts](https://github.com/npm/rfcs/pull/437) - @fritzy |
| 28 | +1. **PR**: [#436 new installation mode: pure-mode](https://github.com/npm/rfcs/pull/436) - @VincentBailly |
| 29 | +1. **PR**: [#422 RFC: audit assertions](https://github.com/npm/rfcs/pull/422) - @bnb |
| 30 | +1. **PR**: [#126 RFC: Adding types information to the Package JSON in the registry](https://github.com/npm/rfcs/pull/126) - @orta |
| 31 | + |
| 32 | +### Notes |
| 33 | + |
| 34 | +#### **`npm` v8 Plan:** https://github.com/npm/rfcs/issues/445 ~ @nlf |
| 35 | +- @nlf |
| 36 | + - want to set ourselves up for success in the future |
| 37 | + - `npm@8` should drop support for `node@10` |
| 38 | + - `npm@7` had a number of issues when consumers upgraded |
| 39 | + - `npm@8` should refuse to install a version of itself that is not compatible with the current `node` version for end-users |
| 40 | +- @ljharb |
| 41 | + - "What about the 'stricter peer deps' change becoming default?" |
| 42 | +- @nlf |
| 43 | + - we want to keep the number or breaking changes to a limited set |
| 44 | +- @isaacs |
| 45 | + - the "refuse to install a breaking version of the CLI" is a `npm@7` change we'll make to help with migration to `npm@8` |
| 46 | + - in terms of 'stricter peer deps' as |
| 47 | +- @wraithgar |
| 48 | + - the less that breaks, the more likely we are to land this version of `npm` into `node@8` |
| 49 | + |
| 50 | +#### **PR**: [#441 addendum: overrides apply if value matches, as well as key](https://github.com/npm/rfcs/pull/441) - @isaacs |
| 51 | +- @isaacs |
| 52 | + - we need to keep track of overriden nodes in the tree somehow |
| 53 | + - making this ammendment ensures that string or dot members can match on current node so that we can apply override nodes properly |
| 54 | + - there are some implications |
| 55 | +- **Actions:** |
| 56 | + - [ ] @isaacs to pull in changes to spec |
| 57 | + |
| 58 | +#### **PR**: [#437 RFC: Robust Lifecycle Scripts](https://github.com/npm/rfcs/pull/437) - @fritzy |
| 59 | +- @fritzy |
| 60 | + - no updates from last week |
| 61 | +- **Actions:** |
| 62 | + - [ ] @fritzy to flush out the rest of the RFC sections |
| 63 | + |
| 64 | +#### **PR**: [#436 new installation mode: pure-mode](https://github.com/npm/rfcs/pull/436) - @VincentBailly |
| 65 | +- @VincentBailly |
| 66 | + - based on last week's meeting notes/action items: |
| 67 | + - On SemVer: There isn't a need to make this a major version bump as it is an opt-in feature - it can wait to be introduced in a major if we _want_ to |
| 68 | + - On Breaking Changes: Packages that rely on the hoisting behavior for shadow dependencies, usually accidentally. |
| 69 | + - Examples of Broken Projects: Packages that do static analysis, Amazon Web Services & React Native projects do not support symlinks & would be broken by a strict-mode/symlinked mode |
| 70 | + - the examples of broken projects does not feel scary, personally, & is following a paved-path by other package managers (ex. `pnpm` & `yarn`) |
| 71 | +- @isaacs |
| 72 | + - had a sync w/ @fritzy & @vincentbailly |
| 73 | + - potential for lockfile & node_modules folder to not be accurate based on the initial implementation strategy we've discussed to transmute the tree between modes |
| 74 | + - that said, the hidden lockfile should always be accurate |
| 75 | + - there is a spearate conversation happening about the sharing of deps in a separate issue (#375) |
| 76 | +- @vincentbailly |
| 77 | + - there was a question about whether or not this pertains to just Workspace projects or to all projects |
| 78 | + - the answer to this is that this RFC, & corresponding work, should apply to **all** projects |
| 79 | + - seems to be some confusion in the vocabulary |
| 80 | +- **Actions:** |
| 81 | + - [ ] @vincentbailly will reframe the RFC to be more clear & direct people to #375 (ie. focus on motivation) |
| 82 | + - [ ] @vincentbailly will come up with options for a new name, as "Pure Mode" isn't well recieved by everyone (potentially run a poll using emojis) |
| 83 | + - [ ] @vincentbailly to review RFC to ensure it is clear that this mode applies to **all** `npm` projects |
| 84 | + |
| 85 | +#### **PR**: [#422 RFC: audit assertions](https://github.com/npm/rfcs/pull/422) - @bnb |
| 86 | +- @bnb |
| 87 | + - updated the RFC based on the last set of action items |
| 88 | + - there is some discussion around trust sources |
| 89 | + - not sure how important that feedback is to consider |
| 90 | +- @darcyclarke |
| 91 | + - has @asciimike jumped in & been able to help at all? |
| 92 | +- @bnb |
| 93 | + - yes |
| 94 | +- **Actions:** |
| 95 | + - [ ] @darcyclarke to add comments/feedback about API design (ie. `npm audit assert --module=<package spec>` vs. `npm audit asser <package spec>`) |
| 96 | + |
| 97 | +#### **PR**: [#126 RFC: Adding types information to the Package JSON in the registry](https://github.com/npm/rfcs/pull/126) - @orta |
| 98 | +- @wraithgar |
| 99 | + - no evaulation of `flow`, no one has asked for this |
| 100 | + - dropped `flow` |
| 101 | +- **Actions:** |
| 102 | + - [ ] @wraithgar to pull in [PR #103, adding `types` field to `read-package-json`](https://github.com/npm/read-package-json/pull/103) - future publishes will now have that metadata included |
0 commit comments