From 184c5fef369417360603322e7de0ca466127354b Mon Sep 17 00:00:00 2001 From: Germain Date: Tue, 1 Aug 2023 12:24:32 +0100 Subject: [PATCH 1/3] Transpile @vector-im/compound-web with babel --- webpack.config.js | 1 + 1 file changed, 1 insertion(+) diff --git a/webpack.config.js b/webpack.config.js index 676fc300ea1..419b2dec767 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -270,6 +270,7 @@ module.exports = (env, argv) => { // include node modules inside these modules, so we add 'src'. if (f.startsWith(reactSdkSrcDir)) return true; if (f.startsWith(jsSdkSrcDir)) return true; + if (f.includes("compound-web")) return true; // but we can't run all of our dependencies through babel (many of them still // use module.exports which breaks if babel injects an 'include' for its From 97fb4494f08dcc1622749f81d18f374245a6c78e Mon Sep 17 00:00:00 2001 From: Germain Date: Wed, 2 Aug 2023 10:31:36 +0100 Subject: [PATCH 2/3] Add context on why we need to transpile compound-web --- webpack.config.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/webpack.config.js b/webpack.config.js index 419b2dec767..c17e03121d9 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -270,6 +270,11 @@ module.exports = (env, argv) => { // include node modules inside these modules, so we add 'src'. if (f.startsWith(reactSdkSrcDir)) return true; if (f.startsWith(jsSdkSrcDir)) return true; + + // Some of the syntax in this package is not understood by + // either webpack or our babel setup. + // When we do get to upgrade our current setup, this should + // probably be removed. if (f.includes("compound-web")) return true; // but we can't run all of our dependencies through babel (many of them still From 0a7b14cb2944990946a4d01ca12c66c5cac5d016 Mon Sep 17 00:00:00 2001 From: Germain Date: Wed, 2 Aug 2023 10:45:08 +0100 Subject: [PATCH 3/3] Make naming more specific --- webpack.config.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webpack.config.js b/webpack.config.js index c17e03121d9..198be2059a7 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -275,7 +275,7 @@ module.exports = (env, argv) => { // either webpack or our babel setup. // When we do get to upgrade our current setup, this should // probably be removed. - if (f.includes("compound-web")) return true; + if (f.includes("@vector-im/compound-web")) return true; // but we can't run all of our dependencies through babel (many of them still // use module.exports which breaks if babel injects an 'include' for its