|
| 1 | +#### Meeting from: July 22nd, 2020 |
| 2 | + |
| 3 | +# Open RFC Meeting (npm) |
| 4 | + |
| 5 | +### Attendees |
| 6 | +- Darcy Clarke (@darcyclarke) |
| 7 | +- Christian Siebmanns (@christian24) |
| 8 | +- Ruy Adorno (@ruyadorno) |
| 9 | +- Isaac Z. Schlueter (@isaacs) |
| 10 | +- Claudia Hernández (@claudiahdz) |
| 11 | +- Lukas Spieß (@lumaxis) |
| 12 | +- Wes Todd (@wesleytodd) |
| 13 | +- Zbyszek Tenerowicz (@naugtur) |
| 14 | + |
| 15 | +### Agenda |
| 16 | +1. **Housekeeping** |
| 17 | + 1. Introduction(s) |
| 18 | + 1. Code of Conduct Acknowledgement |
| 19 | + 1. Outline Intentions & Desired Outcomes |
| 20 | + 1. Announcements |
| 21 | +1. **PR**: [#165 RFC for parent package.json](https://github.com/npm/rfcs/pull/165) - @Christian24 |
| 22 | +1. **Issue**: [#160 [Poll] Preference on how to filter workspaces](https://github.com/npm/rfcs/issues/160) - @ruyadorno |
| 23 | +1. **PR**: [#150 RFC: Add file+pack dependency protocol](https://github.com/npm/rfcs/pull/150) - @RecuencoJones |
| 24 | +1. **PR**: [#129 RFC: overrides](https://github.com/npm/rfcs/pull/129) - @isaacs |
| 25 | +1. **PR**: [#126 RFC: Adding types information to the Package JSON in the registry](https://github.com/npm/rfcs/pull/126) - @orta |
| 26 | +1. **PR**: [#117 RFC: npm workspaces - Running Commands](https://github.com/npm/rfcs/pull/117) - @ruyadorno |
| 27 | +1. **PR**: [#114 RFC: Expand list of ignored files](https://github.com/npm/rfcs/pull/114) - @ruyadorno |
| 28 | +1. **PR**: [#96 RFC: Add publish confirmation prompt](https://github.com/npm/rfcs/pull/96) - @ruyadorno |
| 29 | +1. **PR**: [#18 npm audit resolve](https://github.com/npm/rfcs/pull/18) - @naugtur |
| 30 | +1. **PR**: [#185 Add ability to skip script hooks](https://github.com/npm/rfcs/pull/185) - @lumaxis |
| 31 | +1. **PR**: [#182 RFC: npm audit licenses](https://github.com/npm/rfcs/pull/182) - @bnb |
| 32 | + |
| 33 | +### Notes |
| 34 | + |
| 35 | +### Parent package.json |
| 36 | +- Problem description: managing multiple packages, would be great to reuse info across these |
| 37 | +- might be a very difficult problem to solve due to the complexities of the JavaScript ecosystem |
| 38 | +- workspaces in npm7 could also open up more possibilities (maybe challenges?) |
| 39 | +- implementation ideas: |
| 40 | + - no package.json extension at install time |
| 41 | + - disallow publishing of extensions keys to the registry |
| 42 | + - [read-package-json](https://github.com/npm/read-package-json) / [read-package-json-fast](https://github.com/npm/read-package-json-fast) |
| 43 | + - add logic to merge package.json during pack tarball time |
| 44 | + - we could ship it in a v7-minor release |
| 45 | + - need research to land in the exact key to use in package.json |
| 46 | + - TS currently uses "extends" |
| 47 | +- if we allow users to extend from published versions in the registry it gets way harder to build the ideal install tree (might need to add read-package-json as a pacote dep, etc) |
| 48 | +- should it be a ref to published package or to a file? |
| 49 | +- limiting consumption to exclude git-specs already simplifies by a lot the surface for end consumers that would not have to deal how package authors publish tags, etc |
| 50 | +- def have some details to be figured out |
| 51 | + |
| 52 | +### Preference on how to filter workspaces |
| 53 | +- Let's document the results in the RFC |
| 54 | + |
| 55 | +### Add publish confirmation prompt |
| 56 | +- RFC ratified :+1: |
| 57 | + |
| 58 | +### Audit resolve |
| 59 | +- @naugtur updated the RFC and updated JSON Schema |
| 60 | + |
| 61 | +### Add ability to skip script hooks |
| 62 | +- Fix to strange behavior of `--ignore-scripts` in `npm run-script` |
| 63 | +- When running `npm test --ignore-scripts` it's more intuitive to skip `pretest` and `posttest` |
| 64 | +- Could it be tweaked to ignore specific lifecycle (e.g `postinstall`) when running `npm install`? |
| 65 | + - ref: https://npm.community/t/add-ignore-script-scripts/4169/18 |
| 66 | + - That should be a diff RFC/discussion |
| 67 | +- RFC ratified :+1: |
| 68 | + |
| 69 | +### npm audit licenses |
| 70 | +- license checker built into npm cli itself |
| 71 | +- command: `npm audit license` |
| 72 | +- might interact with the audit resolve proposal on how to block/ignore/allow results |
| 73 | +- should be part of the default `audit`, some niceties like having results at install time |
| 74 | +- missing a better control at level, since some of the problems might not have a solution, same as current `audit` implementation (are probably going to be better once audit resolve is available) |
0 commit comments