Runtime Error in snowpack "TypeError: callBind is not a function" #3011
Replies: 9 comments 16 replies
-
|
I'm seeing this as well. If I open the source inside the browser I see this, with an error on line 41 indicating |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for this suggestion. Unfortunately, that did not fix it for me either. If I step into the debugger inside Chrome, I see below as captured in my screenshot. I don't fully understand what The name seems pretty clear: get the default export, one way or another. What I see in the live code and things look very strange to me:
Maybe this works correctly in certain situations, but in this case, it looks like it is not finding the function which can easily be retrieved from n using the "default" property name.
|
Beta Was this translation helpful? Give feedback.
-
|
Ok. If I manually change |
Beta Was this translation helpful? Give feedback.
-
|
But, I cannot for the life of me figure out where that function comes from; it is not in the snowpack codebase. |
Beta Was this translation helpful? Give feedback.
-
|
Looks like it comes from |
Beta Was this translation helpful? Give feedback.
-
|
I filed an issue for this #3295 and I think we have a pretty good understanding of what the problem is, but I'm not sure what the best way to fix it would be. Check out #3295 and if you have any opinions on the best way to fix it, I should be able to submit a PR to snowpack with the fix. |
Beta Was this translation helpful? Give feedback.
-
|
Did you find a solution? |
Beta Was this translation helpful? Give feedback.
-
|
Nope, and it resulted in me having to use a different web packer. |
Beta Was this translation helpful? Give feedback.
-
|
Ah, that's a shame. I spent over 12 hrs trying to get snowpack working on a large project yesterday with multiple problems, back to webpack for me too, for now. It was wishful thinking in retrospect. Thanks for the quick reply |
Beta Was this translation helpful? Give feedback.
-
|
Unfortunately, I had the same results, several attempts at getting snowpack
working, but in the end, reverted back to using webpack.
…On Sun, May 16, 2021 at 9:02 PM PaulBunker ***@***.***> wrote:
Ah, that's a shame. I spent over 12 hrs trying to get snowpack working on
a large project yesterday with multiple problems, back to webpack for me
too, for now. It was wishful thinking in retrospect. Thanks for the quick
reply
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#3011 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAEFKGN5VAFCWAJKSGDEIDTOCIM3ANCNFSM4Z3REYUA>
.
--
Chris Dawson
971-533-8335
Human potential, travel and entrepreneurship: http://webiphany.com/
Check out Teddy Hyde, Android editor for GitHub (especially Jekyll blogs):
bit.ly/16BL13z
|
Beta Was this translation helpful? Give feedback.
-
|
This happened to me recently after updating to 3.4.0. Any word on a fix? I have reverted to 3.0.10 (the previous version) for the time being. |
Beta Was this translation helpful? Give feedback.
-
|
I'm having the same issue after updating from 3.0.13 (which I had locked the version to because I had issue updating after this point) |
Beta Was this translation helpful? Give feedback.
-
|
Looks like this issue might be related? |
Beta Was this translation helpful? Give feedback.
-
|
Months later and I still am stuck an version |
Beta Was this translation helpful? Give feedback.
-
|
Chiming to mention that I've run into this issue trying to include the blueprintjs React framework, which includes call-bind somewhere down the dependency tree. Has anybody found a workaround for this other than downgrading to 3.0.13, or are many ubiquitous JS libraries just not usable at all with the current version of snowpack? |
Beta Was this translation helpful? Give feedback.
-
|
I am actually using blueprint and have the same issue. |
Beta Was this translation helpful? Give feedback.
-
|
I did some digging into this and am confident that the issue crept in when snowpack upgraded The issue has to do with how @rollup/plugin-commonjs handles the resolution of commonjs modules, i.e. what do you get back when you In v15, the logic was simply to look for a key called 'default' on the module and, if found, return its value as the result of require(). In v16, @rollup/plugin-commonjs added another option for resolution behavior and made this configurable via a parameter called Now, call-bind is just such a module. It returns a default export, a function, but it ALSO attaches another function called From object-is/index.js Because snowpack sets It seems to me that setting requireReturnsDefault to 'auto' with the switch to v16 probably just seemed like a sensible default and this change in behavior was unintended. Since snowpack 3.0.13, any project which includes I also commented on this in #3295 |
Beta Was this translation helpful? Give feedback.
-
|
I created PR #3660 to change requireReturnsDefault to 'preferred'. In the meantime, I've published a package of my fork with this change, which you can use instead of downgrading to 3.0.13 if you're having this issue. |
Beta Was this translation helpful? Give feedback.
-
|
Hello, I'm still getting the issue from your fork. I don't get it from 3.0.13. Perhaps check |
Beta Was this translation helpful? Give feedback.
-
|
For those on yarn2 you can just use it's patch feature to store temporary fixes: https://yarnpkg.com/cli/patch for myself, the two patch files are:
diff --git a/lib/config.js b/lib/config.js
index 2800a023fba265f818137b492a6986c6355fc466..07e44222117f09df2aa44cf5e0e4700f5ac2495e 100644
--- a/lib/config.js
+++ b/lib/config.js
@@ -154,6 +154,7 @@ const configSchema = {
type: 'array',
items: { type: 'string' },
},
+ requireReturnsDefault: {type: ['string', 'boolean']},
},
},
},
diff --git a/lib/sources/local-install.js b/lib/sources/local-install.js
index ffe32fce29d57fd502e01864cfcf9d80f08e2a82..f8b4cbce456faa451472910992c30fb4a51e6ddc 100644
--- a/lib/sources/local-install.js
+++ b/lib/sources/local-install.js
@@ -35,6 +35,7 @@ async function installPackages({ config, isDev, isSSR, installOptions, installTa
...installOptions,
stats: false,
rollup: {
+ requireReturnsDefault: installOptions?.rollup?.requireReturnsDefault,
plugins: [
...((_b = (_a = installOptions === null || installOptions === void 0 ? void 0 : installOptions.rollup) === null || _a === void 0 ? void 0 : _a.plugins) !== null && _b !== void 0 ? _b : []),
{
diff --git a/lib/index.d.ts b/lib/index.d.ts
index d1b8b7282d4e0b009cc5c16211eeba82fc1aba92..014054fe61f9820d8c0ee038c27a4a79df8ed81a 100644
--- a/lib/index.d.ts
+++ b/lib/index.d.ts
@@ -25,6 +25,7 @@ interface InstallOptions {
context?: string;
plugins?: RollupPlugin[];
dedupe?: string[];
+ requireReturnsDefault?: string | boolean;
};
}
declare type PublicInstallOptions = Partial<InstallOptions>;
diff --git a/lib/index.js b/lib/index.js
index f4b4799f71ab80ce5efb7eade7bbd8c0f32824b7..768f45b1708684d7c27d87135d8be2866f559171 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -274,7 +274,9 @@ ${colors.dim(` At least one dependency must have an ESM "module" entrypoint. Yo
esmExternals: Array.isArray(externalEsm)
? (id) => externalEsm.some((packageName) => isImportOfPackage(id, packageName))
: externalEsm,
- requireReturnsDefault: 'auto',
+ requireReturnsDefault: userDefinedRollup.requireReturnsDefault == undefined
+ ? 'auto'
+ : userDefinedRollup.requireReturnsDefault,
}),
rollup_plugin_wrap_install_targets_1.rollupPluginWrapInstallTargets(!!isTreeshake, installTargets, logger),
stats && rollup_plugin_stats_1.rollupPluginDependencyStats((info) => (dependencyStats = info)),you shoud follow the yarn 2 patch documentation about which instructs you to run make sure you look at the exact semver requirements of the above two packages. If (just added |
Beta Was this translation helpful? Give feedback.
-
|
meh, after patching snowpack, i still get |
Beta Was this translation helpful? Give feedback.
-
|
ok probs because my patch missed out on the diff --git a/lib/cjs/config.js b/lib/cjs/config.js
index fa97f84768729c80665be6e51ea9f8021ecbf1fe..e73cec2c5bef90ee2894484a2167423f54143d45 100644
--- a/lib/cjs/config.js
+++ b/lib/cjs/config.js
@@ -156,6 +156,7 @@ const configSchema = {
type: 'array',
items: { type: 'string' },
},
+ requireReturnsDefault: {type: ['string', 'boolean']},
},
},
},
diff --git a/lib/cjs/sources/local-install.js b/lib/cjs/sources/local-install.js
index ffe32fce29d57fd502e01864cfcf9d80f08e2a82..f8b4cbce456faa451472910992c30fb4a51e6ddc 100644
--- a/lib/cjs/sources/local-install.js
+++ b/lib/cjs/sources/local-install.js
@@ -35,6 +35,7 @@ async function installPackages({ config, isDev, isSSR, installOptions, installTa
...installOptions,
stats: false,
rollup: {
+ requireReturnsDefault: installOptions?.rollup?.requireReturnsDefault,
plugins: [
...((_b = (_a = installOptions === null || installOptions === void 0 ? void 0 : installOptions.rollup) === null || _a === void 0 ? void 0 : _a.plugins) !== null && _b !== void 0 ? _b : []),
{
diff --git a/lib/esm/config.js b/lib/esm/config.js
index ed1fa4115c2bd14ad92064ae621896ba216e22aa..83cf5fc963759e686afb9849758db657df0ea17a 100644
--- a/lib/esm/config.js
+++ b/lib/esm/config.js
@@ -150,6 +150,7 @@ const configSchema = {
type: 'array',
items: { type: 'string' },
},
+ requireReturnsDefault: {type: ['string', 'boolean']},
},
},
},
diff --git a/lib/esm/sources/local-install.js b/lib/esm/sources/local-install.js
index 29e78b2de244bde5c244869994889f7bb772b368..af01085eaf2ba9871905678922b68d2f9ed98ef1 100644
--- a/lib/esm/sources/local-install.js
+++ b/lib/esm/sources/local-install.js
@@ -29,6 +29,7 @@ export async function installPackages({ config, isDev, isSSR, installOptions, in
...installOptions,
stats: false,
rollup: {
+ requireReturnsDefault: installOptions?.rollup?.requireReturnsDefault,
plugins: [
...((_b = (_a = installOptions === null || installOptions === void 0 ? void 0 : installOptions.rollup) === null || _a === void 0 ? void 0 : _a.plugins) !== null && _b !== void 0 ? _b : []),
{
|
Beta Was this translation helpful? Give feedback.
-
|
@gpascale Thanks for your change. I'm not getting the |
Beta Was this translation helpful? Give feedback.
-
Snowpack has been awesome for building and serving our system, but after a recent update I'm completely stuck. The runtime appears to be looking for a hashed file name that doesn't match the files installed.
I've tried:
My
snowpack.config.js-- it used to have more in it, but this still repro's the problem:Browser console error:
Error output from snowpack command:
(note that the hash of the file on disk doesn't match what snowpack is looking for).
Haalp?
Beta Was this translation helpful? Give feedback.
All reactions