chore: apply @lavamoat/harden defaults#335
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Caution MetaMask internal reviewing guidelines:
|
| plugins: | ||
| - path: .yarn/plugins/@yarnpkg/plugin-allow-scripts.cjs | ||
| spec: 'https://raw.githubusercontent.com/LavaMoat/LavaMoat/main/packages/yarn-plugin-allow-scripts/bundles/@yarnpkg/plugin-allow-scripts.js' | ||
| spec: https://raw.githubusercontent.com/LavaMoat/LavaMoat/main/packages/yarn-plugin-allow-scripts/bundles/@yarnpkg/plugin-allow-scripts.js |
There was a problem hiding this comment.
you can now remove the old version of plugin-allow-scripts. The new one is the exact same functionality but readable.
| } | ||
| }, | ||
| "scriptsConfig": { | ||
| "#default": "lavamoat/scripts.loose.json" |
There was a problem hiding this comment.
we could add some stronger policies for specific scripts here.
I'll give it a go later.
There was a problem hiding this comment.
Would you prefer for this file to also strt with a dot? I'd consider it an inconsistency on my side.
There was a problem hiding this comment.
I don't think runner-plugin needs to start with a dot, since it's already inside a separate lavamoat folder. But either way is fine with me.
Examples