From 60b257078af38be1e06ff7a35a85dd6fd7fcb16a Mon Sep 17 00:00:00 2001 From: Dominik Picheta Date: Wed, 2 Oct 2024 14:39:04 +0000 Subject: [PATCH] Don't include implicit stdlib snapshot imports in package snapshot key. --- src/pyodide/internal/snapshot.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/pyodide/internal/snapshot.ts b/src/pyodide/internal/snapshot.ts index b52739b5972..d65020735ec 100644 --- a/src/pyodide/internal/snapshot.ts +++ b/src/pyodide/internal/snapshot.ts @@ -275,12 +275,17 @@ function memorySnapshotDoImports(Module: Module): Array { const localModulePaths: Set = new Set( MetadataReader.getNames() ); + const SNAPSHOT_IMPORTS_SET = new Set(SNAPSHOT_IMPORTS); const importedModules: Array = ArtifactBundler.constructor // @ts-ignore parsePythonScriptImports is a static method. .parsePythonScriptImports(MetadataReader.getWorkerFiles('py')) .filter((module: string) => { const moduleFilename = module.replace('.', '/') + '.py'; - return !localModulePaths.has(moduleFilename) && module != 'js'; + return ( + !localModulePaths.has(moduleFilename) && + module != 'js' && + !SNAPSHOT_IMPORTS_SET.has(module) + ); }); const deduplicatedModules = [...new Set(importedModules)];