Updates astro add to ignore optional peer dependencies (#5192)

* checking peerDependenciesMeta before including peer deps * chore: add changeset
withastro · Oct 25, 2022 · 8728ee0 · 8728ee0
1 parent 44fa495
commit 8728ee0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/.changeset/smart-chicken-develop.md b/.changeset/smart-chicken-develop.md
@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+`astro add` no longer automatically installs optional peer dependencies
diff --git a/packages/astro/src/core/add/index.ts b/packages/astro/src/core/add/index.ts
@@ -705,8 +705,12 @@ export async function validateIntegrations(integrations: string[]): Promise<Inte
 				];
 
 				if (pkgJson['peerDependencies']) {
+					const meta = pkgJson['peerDependenciesMeta'] || {}
 					for (const peer in pkgJson['peerDependencies']) {
-						dependencies.push([peer, pkgJson['peerDependencies'][peer]]);
+						const optional = meta[peer]?.optional || false
+						if (!optional) {
+							dependencies.push([peer, pkgJson['peerDependencies'][peer]]);
+						}
 					}
 				}