@builder.io/sdk 2.2.7-0 → 2.2.8

package/.yarnrc.yml

@@ -0,0 +1 @@
+npmPublishAccess: "public"

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @builder.io/sdk
 
+## 2.2.8
+
+### Patch Changes
+
+- 11e118c: Fix: serialize all functions within registered component info.
+
+## 2.2.7
+
+### Patch Changes
+
+- b965695: Fix: reverts `v2.2.5` change to `userAttributes` parsing logic, as it caused breaking changes in certain cases.
+
 ## 2.2.6
 
 ### Patch Changes

package/dist/index.browser.js

@@ -152,7 +152,7 @@
             throw new Error("Property name \"".concat(name, "\" is not allowed"));
     }
 
-    var version = "2.2.6";
+    var version = "2.2.8";
 
     var Subscription = /** @class */ (function () {
         function Subscription(listeners, listener) {
@@ -1441,36 +1441,25 @@
             enumerable: false,
             configurable: true
         });
+        Builder.serializeComponentInfo = function (info) {
+            var serializeFn = function (fnValue) {
+                var fnStr = fnValue.toString().trim();
+                // we need to account for a few different fn syntaxes:
+                // 1. `function name(args) => {code}`
+                // 2. `name(args) => {code}`
+                // 3. `(args) => {}`
+                var appendFunction = !fnStr.startsWith('function') && !fnStr.startsWith('(');
+                return "return (".concat(appendFunction ? 'function ' : '').concat(fnStr, ").apply(this, arguments)");
+            };
+            return JSON.parse(JSON.stringify(info, function (key, value) {
+                if (typeof value === 'function') {
+                    return serializeFn(value);
+                }
+                return value;
+            }));
+        };
         Builder.prepareComponentSpecToSend = function (spec) {
-            return __assign(__assign(__assign({}, spec), (spec.inputs && {
-                inputs: spec.inputs.map(function (input) {
-                    var _a;
-                    // TODO: do for nexted fields too
-                    // TODO: probably just convert all functions, not just
-                    // TODO: put this in input hooks: { onChange: ..., showIf: ... }
-                    var keysToConvertFnToString = ['onChange', 'showIf'];
-                    for (var _i = 0, keysToConvertFnToString_1 = keysToConvertFnToString; _i < keysToConvertFnToString_1.length; _i++) {
-                        var key = keysToConvertFnToString_1[_i];
-                        if (input[key] && typeof input[key] === 'function') {
-                            var fn = input[key];
-                            input = __assign(__assign({}, input), (_a = {}, _a[key] = "return (".concat(fn.toString(), ").apply(this, arguments)"), _a));
-                        }
-                    }
-                    return input;
-                }),
-            })), { hooks: Object.keys(spec.hooks || {}).reduce(function (memo, key) {
-                    var value = spec.hooks && spec.hooks[key];
-                    if (!value) {
-                        return memo;
-                    }
-                    if (typeof value === 'string') {
-                        memo[key] = value;
-                    }
-                    else {
-                        memo[key] = "return (".concat(value.toString(), ").apply(this, arguments)");
-                    }
-                    return memo;
-                }, {}), class: undefined });
+            return __assign(__assign({}, this.serializeComponentInfo(spec)), { class: undefined });
         };
         Builder.registerBlock = function (component, options) {
             this.registerComponent(component, options);
@@ -1861,20 +1850,15 @@
         };
         Builder.prototype.setTestsFromUrl = function () {
             var search = this.getLocation().search;
-            try {
-                var params = QueryString.parseDeep(this.modifySearch(search || '').substr(1));
-                var tests = params.builder && params.builder.tests;
-                if (tests && typeof tests === 'object') {
-                    for (var key in tests) {
-                        if (tests.hasOwnProperty(key)) {
-                            this.setTestCookie(key, tests[key]);
-                        }
+            var params = QueryString.parseDeep(this.modifySearch(search || '').substr(1));
+            var tests = params.builder && params.builder.tests;
+            if (tests && typeof tests === 'object') {
+                for (var key in tests) {
+                    if (tests.hasOwnProperty(key)) {
+                        this.setTestCookie(key, tests[key]);
                     }
                 }
             }
-            catch (e) {
-                console.debug('Error parsing tests from URL', e);
-            }
         };
         Builder.prototype.resetOverrides = function () {
             // Ugly - pass down instances per request instead using react context
@@ -1892,46 +1876,41 @@
         };
         Builder.prototype.getOverridesFromQueryString = function () {
             var location = this.getLocation();
-            try {
-                var params = QueryString.parseDeep(this.modifySearch(location.search || '').substr(1));
-                var builder = params.builder;
-                if (builder) {
-                    var userAttributes = builder.userAttributes, overrides = builder.overrides, env = builder.env, host = builder.host, api = builder.api, cachebust = builder.cachebust, noCache = builder.noCache, preview = builder.preview, editing = builder.editing, frameEditing = builder.frameEditing, options = builder.options, overrideParams = builder.params;
-                    if (userAttributes) {
-                        this.setUserAttributes(userAttributes);
-                    }
-                    if (options) {
-                        // picking only locale, includeRefs, and enrich for now
-                        this.queryOptions = __assign(__assign(__assign({}, (options.locale && { locale: options.locale })), (options.includeRefs && { includeRefs: options.includeRefs })), (options.enrich && { enrich: options.enrich }));
-                    }
-                    if (overrides) {
-                        this.overrides = overrides;
-                    }
-                    if (validEnvList.indexOf(env || api) > -1) {
-                        this.env = env || api;
-                    }
-                    if (Builder.isEditing) {
-                        var editingModel = frameEditing || editing || preview;
-                        if (editingModel && editingModel !== 'true') {
-                            this.editingModel = editingModel;
-                        }
-                    }
-                    if (cachebust) {
-                        this.cachebust = true;
-                    }
-                    if (noCache) {
-                        this.noCache = true;
-                    }
-                    if (preview) {
-                        this.preview = true;
-                    }
-                    if (params) {
-                        this.overrideParams = overrideParams;
+            var params = QueryString.parseDeep(this.modifySearch(location.search || '').substr(1));
+            var builder = params.builder;
+            if (builder) {
+                var userAttributes = builder.userAttributes, overrides = builder.overrides, env = builder.env; builder.host; var api = builder.api, cachebust = builder.cachebust, noCache = builder.noCache, preview = builder.preview, editing = builder.editing, frameEditing = builder.frameEditing, options = builder.options, overrideParams = builder.params;
+                if (userAttributes) {
+                    this.setUserAttributes(userAttributes);
+                }
+                if (options) {
+                    // picking only locale, includeRefs, and enrich for now
+                    this.queryOptions = __assign(__assign(__assign({}, (options.locale && { locale: options.locale })), (options.includeRefs && { includeRefs: options.includeRefs })), (options.enrich && { enrich: options.enrich }));
+                }
+                if (overrides) {
+                    this.overrides = overrides;
+                }
+                if (validEnvList.indexOf(env || api) > -1) {
+                    this.env = env || api;
+                }
+                if (Builder.isEditing) {
+                    var editingModel = frameEditing || editing || preview;
+                    if (editingModel && editingModel !== 'true') {
+                        this.editingModel = editingModel;
                     }
                 }
-            }
-            catch (e) {
-                console.debug('Error parsing overrides from URL', e);
+                if (cachebust) {
+                    this.cachebust = true;
+                }
+                if (noCache) {
+                    this.noCache = true;
+                }
+                if (preview) {
+                    this.preview = true;
+                }
+                if (params) {
+                    this.overrideParams = overrideParams;
+                }
             }
         };
         Builder.prototype.messageFrameLoaded = function () {