Copy site-packages for Pyodide only when needed

FeodorFitsner · FeodorFitsner · commit eb0bf8d19543 · 2026-03-05T16:39:48.000-08:00
Add a guard to only copy site-packages for the Pyodide platform if there are requirements to include (requirements.isNotEmpty). This prevents unnecessary copying of the site-packages directory when no requirements are specified, avoiding wasted I/O and archive bloat.
diff --git a/src/serious_python/bin/package_command.dart b/src/serious_python/bin/package_command.dart
@@ -430,7 +430,7 @@ class PackageCommand extends Command {
       }
 
       // copy site packages to temp dir for web platform
-      if (platform == "Pyodide") {
+      if (platform == "Pyodide" && requirements.isNotEmpty) {
         final sitePackagesSrcDir = Directory(sitePackagesRoot);
         if (await sitePackagesSrcDir.exists()) {
           stdout.writeln("Copying site packages to app archive");

Original file line number	Diff line number	Diff line change
`@@ -430,7 +430,7 @@ class PackageCommand extends Command {`
`430`	`430`	`}`
`431`	`431`
`432`	`432`	`// copy site packages to temp dir for web platform`
`433`		`- if (platform == "Pyodide") {`
	`433`	`+ if (platform == "Pyodide" && requirements.isNotEmpty) {`
`434`	`434`	`final sitePackagesSrcDir = Directory(sitePackagesRoot);`
`435`	`435`	`if (await sitePackagesSrcDir.exists()) {`
`436`	`436`	`stdout.writeln("Copying site packages to app archive");`