From 6f77edde0e4e1d4d61dcefbf17ab5002d2b648c1 Mon Sep 17 00:00:00 2001 From: David Dal Busco Date: Fri, 26 May 2023 11:29:47 +0200 Subject: [PATCH] build: remove explicit import of lru-cache workaround --- frontend/src/routes/+layout.ts | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/frontend/src/routes/+layout.ts b/frontend/src/routes/+layout.ts index 8e78c615c21..a8d368ee1d5 100644 --- a/frontend/src/routes/+layout.ts +++ b/frontend/src/routes/+layout.ts @@ -1,16 +1,3 @@ -// There is a bug in Rollup that non-deterministically fails to detect cyclic -// dependencies. https://github.com/rollup/plugins/issues/1425 -// As a result 'lru-cache' is sometimes wrapped in a require* function and -// sometimes it is not. This makes our build non-deterministic. -// lru-caceh is not itself part of a circular dependency but -// semver/classes/range.js, which depends on lru-cache, is. -// By depending on lru-cache directly from our app, we change the order in which -// modules are analyzed, hopefully avoiding the buggy behavior. -// It probably doesn't matter where we put this import, so I put it as close to -// the root of the app as I could find. -// TODO: Remove when the Rollup issue is fixed. -import "lru-cache"; - export const prerender = true; // TODO: no ssr for local development until https://github.com/dfinity/ic-js/issues/238 solved