u3 package is broken

[vans163]

[!] (plugin commonjs) SyntaxError: Deleting local variable in strict mode (8:8) in 
../node_modules/u3/lib/cache.js
node_modules/u3/lib/cache.js (8:8)

It seems the u3 package used is outdated and does not follow modern javascript rules.

Also is there a way to build a static non-browser build (so no window. is included)?

[mehtaphysical]

It doesn't seem like this codebase uses u3. It's not in our yarn.lock. Are you maybe using a different dependency that uses it?

[thanhlmm]

@vans163 I face the same problem, so I need to override the u3 package by using patch-package

Here is the modified one

var cache = function (fn) {
    var called = false,
        store;

    if (!(fn instanceof Function)) {
        called = true;
        store = fn;
        fn = null;
    }

    return function () {
        if (!called) {
            called = true;
            store = fn.apply(this, arguments);
            fn = null;
        }
        return store;
    };
};

module.exports = cache;

[volovyks]

@mehtaphysical looks like it is in our lock file and dependency tree. Try npm list --depth=1 to find it. It's a dependency of error-polyfill, but both error-polyfill and u3 have the latest versions from npm.

[mehtaphysical]

@volovyk-s I see. We should consider removing this dependency. According to their GitHub repo it is:

INACTIVE (unstable) - Javascript Error Polyfill - Implementing the V8 Error API in non-V8 environments as much as possible

[volovyks]

@vans163 error-polyfill was updated and now we are using the latest version. The same goes for u3.
https://www.npmjs.com/package/u3
https://www.npmjs.com/package/error-polyfill
Please, reopen if you are still encountering any problems with these dependencies.