solid-js 1.5.6 → 1.6.0-beta.0

package/dist/solid.js

@@ -275,10 +275,6 @@ function createResource(pSource, pFetcher, pOptions) {
         loadedUnderTransition = false;
         runUpdates(() => {
           Transition.running = true;
-          if (!Transition.promises.size) {
-            Effects.push.apply(Effects, Transition.effects);
-            Transition.effects = [];
-          }
           completeLoad(v, error);
         }, false);
       } else completeLoad(v, error);
@@ -303,7 +299,7 @@ function createResource(pSource, pFetcher, pOptions) {
       createComputed(() => {
         track();
         if (pr) {
-          if (c.resolved && Transition) Transition.promises.add(pr);else if (!contexts.has(c)) {
+          if (c.resolved && Transition && Transition.running) Transition.promises.add(pr);else if (!contexts.has(c)) {
             c.increment();
             contexts.add(c);
           }
@@ -621,7 +617,9 @@ function updateComputation(node) {
     queueMicrotask(() => {
       runUpdates(() => {
         Transition && (Transition.running = true);
+        Listener = Owner = node;
         runComputation(node, node.tValue, time);
+        Listener = Owner = null;
       }, false);
     });
   }
@@ -737,36 +735,38 @@ function completeUpdates(wait) {
   }
   if (wait) return;
   let res;
-  if (Transition && Transition.running) {
-    if (Transition.promises.size || Transition.queue.size) {
+  if (Transition) {
+    if (!Transition.promises.size && !Transition.queue.size) {
+      const sources = Transition.sources;
+      const disposed = Transition.disposed;
+      Effects.push.apply(Effects, Transition.effects);
+      res = Transition.resolve;
+      for (const e of Effects) {
+        "tState" in e && (e.state = e.tState);
+        delete e.tState;
+      }
+      Transition = null;
+      runUpdates(() => {
+        for (const d of disposed) cleanNode(d);
+        for (const v of sources) {
+          v.value = v.tValue;
+          if (v.owned) {
+            for (let i = 0, len = v.owned.length; i < len; i++) cleanNode(v.owned[i]);
+          }
+          if (v.tOwned) v.owned = v.tOwned;
+          delete v.tValue;
+          delete v.tOwned;
+          v.tState = 0;
+        }
+        setTransPending(false);
+      }, false);
+    } else if (Transition.running) {
       Transition.running = false;
       Transition.effects.push.apply(Transition.effects, Effects);
       Effects = null;
       setTransPending(true);
       return;
     }
-    const sources = Transition.sources;
-    const disposed = Transition.disposed;
-    res = Transition.resolve;
-    for (const e of Effects) {
-      "tState" in e && (e.state = e.tState);
-      delete e.tState;
-    }
-    Transition = null;
-    runUpdates(() => {
-      for (const d of disposed) cleanNode(d);
-      for (const v of sources) {
-        v.value = v.tValue;
-        if (v.owned) {
-          for (let i = 0, len = v.owned.length; i < len; i++) cleanNode(v.owned[i]);
-        }
-        if (v.tOwned) v.owned = v.tOwned;
-        delete v.tValue;
-        delete v.tOwned;
-        v.tState = 0;
-      }
-      setTransPending(false);
-    }, false);
   }
   const e = Effects;
   Effects = null;
@@ -787,10 +787,6 @@ function scheduleQueue(queue) {
         runUpdates(() => {
           Transition.running = true;
           runTop(item);
-          if (!tasks.size) {
-            Effects.push.apply(Effects, Transition.effects);
-            Transition.effects = [];
-          }
         }, false);
         Transition && (Transition.running = false);
       });
@@ -1173,29 +1169,41 @@ function resolveSource(s) {
   return (s = typeof s === "function" ? s() : s) == null ? {} : s;
 }
 function mergeProps(...sources) {
-  return new Proxy({
-    get(property) {
-      for (let i = sources.length - 1; i >= 0; i--) {
-        const v = resolveSource(sources[i])[property];
-        if (v !== undefined) return v;
-      }
-    },
-    has(property) {
-      for (let i = sources.length - 1; i >= 0; i--) {
-        if (property in resolveSource(sources[i])) return true;
+  if (sources.some(s => s && ($PROXY in s || typeof s === "function"))) {
+    return new Proxy({
+      get(property) {
+        for (let i = sources.length - 1; i >= 0; i--) {
+          const v = resolveSource(sources[i])[property];
+          if (v !== undefined) return v;
+        }
+      },
+      has(property) {
+        for (let i = sources.length - 1; i >= 0; i--) {
+          if (property in resolveSource(sources[i])) return true;
+        }
+        return false;
+      },
+      keys() {
+        const keys = [];
+        for (let i = 0; i < sources.length; i++) keys.push(...Object.keys(resolveSource(sources[i])));
+        return [...new Set(keys)];
       }
-      return false;
-    },
-    keys() {
-      const keys = [];
-      for (let i = 0; i < sources.length; i++) keys.push(...Object.keys(resolveSource(sources[i])));
-      return [...new Set(keys)];
+    }, propTraps);
+  }
+  const target = {};
+  for (let i = 0; i < sources.length; i++) {
+    if (sources[i]) {
+      const descriptors = Object.getOwnPropertyDescriptors(sources[i]);
+      Object.defineProperties(target, descriptors);
     }
-  }, propTraps);
+  }
+  return target;
 }
 function splitProps(props, ...keys) {
   const blocked = new Set(keys.flat());
   const descriptors = Object.getOwnPropertyDescriptors(props);
+  const isProxy = ($PROXY in props);
+  if (!isProxy) keys.push(Object.keys(descriptors).filter(k => !blocked.has(k)));
   const res = keys.map(k => {
     const clone = {};
     for (let i = 0; i < k.length; i++) {
@@ -1211,17 +1219,19 @@ function splitProps(props, ...keys) {
     }
     return clone;
   });
-  res.push(new Proxy({
-    get(property) {
-      return blocked.has(property) ? undefined : props[property];
-    },
-    has(property) {
-      return blocked.has(property) ? false : property in props;
-    },
-    keys() {
-      return Object.keys(props).filter(k => !blocked.has(k));
-    }
-  }, propTraps));
+  if (isProxy) {
+    res.push(new Proxy({
+      get(property) {
+        return blocked.has(property) ? undefined : props[property];
+      },
+      has(property) {
+        return blocked.has(property) ? false : property in props;
+      },
+      keys() {
+        return Object.keys(props).filter(k => !blocked.has(k));
+      }
+    }, propTraps));
+  }
   return res;
 }
 function lazy(fn) {

package/html/dist/html.cjs

@@ -196,7 +196,7 @@ function createHTML(r, {
       templates[i].innerHTML = html[i];
       const nomarkers = templates[i].content.querySelectorAll("script,style");
       for (let j = 0; j < nomarkers.length; j++) {
-        const d = nomarkers[j].firstChild.data || "";
+        const d = nomarkers[j].firstChild?.data || "";
         if (d.indexOf(marker) > -1) {
           const parts = d.split(marker).reduce((memo, p, i) => {
             i && memo.push("");
@@ -405,15 +405,35 @@ function createHTML(r, {
       const isSVG = r.SVGElements.has(node.name);
       const isCE = node.name.includes("-");
       options.hasCustomElement = isCE;
-      for (let i = 0; i < keys.length; i++) {
-        const name = keys[i],
-              value = node.attrs[name];
-        if (value.includes("###")) {
-          delete node.attrs[name];
-          parseAttribute(tag, name, value, isSVG, isCE, options);
-        } else if (name === "###") {
-          delete node.attrs[name];
-          options.exprs.push(`r.spread(${tag},exprs[${options.counter++}],${isSVG},${!!node.children.length})`);
+      if (keys.includes("###")) {
+        const spreadArgs = [];
+        let current = "";
+        for (let i = 0; i < keys.length; i++) {
+          const name = keys[i],
+                value = node.attrs[name];
+          if (value.includes("###")) {
+            let count = options.counter++;
+            current += `${name}: ${name !== "ref" ? `typeof exprs[${count}] === "function" ? exprs[${count}]() : ` : ""}exprs[${count}],`;
+            delete node.attrs[name];
+          } else if (name === "###") {
+            if (current.length) {
+              spreadArgs.push(`()=>({${current}})`);
+              current = "";
+            }
+            spreadArgs.push(`exprs[${options.counter++}]`);
+            delete node.attrs[name];
+          }
+        }
+        if (current.length) spreadArgs.push(`()=>({${current}})`);
+        options.exprs.push(`r.spread(${tag},${spreadArgs.length === 1 ? `typeof ${spreadArgs[0]} === "function" ? r.mergeProps(${spreadArgs[0]}) : ${spreadArgs[0]}` : `r.mergeProps(${spreadArgs.join(",")})`},${isSVG},${!!node.children.length})`);
+      } else {
+        for (let i = 0; i < keys.length; i++) {
+          const name = keys[i],
+                value = node.attrs[name];
+          if (value.includes("###")) {
+            delete node.attrs[name];
+            parseAttribute(tag, name, value, isSVG, isCE, options);
+          }
         }
       }
       options.path = tag;

package/html/dist/html.js

@@ -194,7 +194,7 @@ function createHTML(r, {
       templates[i].innerHTML = html[i];
       const nomarkers = templates[i].content.querySelectorAll("script,style");
       for (let j = 0; j < nomarkers.length; j++) {
-        const d = nomarkers[j].firstChild.data || "";
+        const d = nomarkers[j].firstChild?.data || "";
         if (d.indexOf(marker) > -1) {
           const parts = d.split(marker).reduce((memo, p, i) => {
             i && memo.push("");
@@ -403,15 +403,35 @@ function createHTML(r, {
       const isSVG = r.SVGElements.has(node.name);
       const isCE = node.name.includes("-");
       options.hasCustomElement = isCE;
-      for (let i = 0; i < keys.length; i++) {
-        const name = keys[i],
-              value = node.attrs[name];
-        if (value.includes("###")) {
-          delete node.attrs[name];
-          parseAttribute(tag, name, value, isSVG, isCE, options);
-        } else if (name === "###") {
-          delete node.attrs[name];
-          options.exprs.push(`r.spread(${tag},exprs[${options.counter++}],${isSVG},${!!node.children.length})`);
+      if (keys.includes("###")) {
+        const spreadArgs = [];
+        let current = "";
+        for (let i = 0; i < keys.length; i++) {
+          const name = keys[i],
+                value = node.attrs[name];
+          if (value.includes("###")) {
+            let count = options.counter++;
+            current += `${name}: ${name !== "ref" ? `typeof exprs[${count}] === "function" ? exprs[${count}]() : ` : ""}exprs[${count}],`;
+            delete node.attrs[name];
+          } else if (name === "###") {
+            if (current.length) {
+              spreadArgs.push(`()=>({${current}})`);
+              current = "";
+            }
+            spreadArgs.push(`exprs[${options.counter++}]`);
+            delete node.attrs[name];
+          }
+        }
+        if (current.length) spreadArgs.push(`()=>({${current}})`);
+        options.exprs.push(`r.spread(${tag},${spreadArgs.length === 1 ? `typeof ${spreadArgs[0]} === "function" ? r.mergeProps(${spreadArgs[0]}) : ${spreadArgs[0]}` : `r.mergeProps(${spreadArgs.join(",")})`},${isSVG},${!!node.children.length})`);
+      } else {
+        for (let i = 0; i < keys.length; i++) {
+          const name = keys[i],
+                value = node.attrs[name];
+          if (value.includes("###")) {
+            delete node.attrs[name];
+            parseAttribute(tag, name, value, isSVG, isCE, options);
+          }
         }
       }
       options.path = tag;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "solid-js",
   "description": "A declarative JavaScript library for building user interfaces.",
-  "version": "1.5.6",
+  "version": "1.6.0-beta.0",
   "author": "Ryan Carniato",
   "license": "MIT",
   "homepage": "https://solidjs.com",
@@ -44,6 +44,13 @@
   ],
   "exports": {
     ".": {
+      "worker": {
+        "import": {
+          "types": "./types/index.d.ts",
+          "default": "./dist/server.js"
+        },
+        "require": "./dist/server.cjs"
+      },
       "browser": {
         "development": {
           "import": {
@@ -65,13 +72,6 @@
         },
         "require": "./dist/server.cjs"
       },
-      "worker": {
-        "import": {
-          "types": "./types/index.d.ts",
-          "default": "./dist/server.js"
-        },
-        "require": "./dist/server.cjs"
-      },
       "node": {
         "import": {
           "types": "./types/index.d.ts",
@@ -98,6 +98,13 @@
       "default": "./dist/solid.js"
     },
     "./store": {
+      "worker": {
+        "import": {
+          "types": "./store/types/index.d.ts",
+          "default": "./store/dist/server.js"
+        },
+        "require": "./store/dist/server.cjs"
+      },
       "browser": {
         "development": {
           "import": {
@@ -119,13 +126,6 @@
         },
         "require": "./store/dist/server.cjs"
       },
-      "worker": {
-        "import": {
-          "types": "./store/types/index.d.ts",
-          "default": "./store/dist/server.js"
-        },
-        "require": "./store/dist/server.cjs"
-      },
       "node": {
         "import": {
           "types": "./store/types/index.d.ts",
@@ -148,6 +148,13 @@
     },
     "./store/dist/*": "./store/dist/*",
     "./web": {
+      "worker": {
+        "import": {
+          "types": "./web/types/index.d.ts",
+          "default": "./web/dist/server.js"
+        },
+        "require": "./web/dist/server.cjs"
+      },
       "browser": {
         "development": {
           "import": {
@@ -169,13 +176,6 @@
         },
         "require": "./web/dist/server.cjs"
       },
-      "worker": {
-        "import": {
-          "types": "./web/types/index.d.ts",
-          "default": "./web/dist/server.js"
-        },
-        "require": "./web/dist/server.cjs"
-      },
       "node": {
         "import": {
           "types": "./web/types/index.d.ts",

package/store/package.json

@@ -12,6 +12,13 @@
   "sideEffects": false,
   "exports": {
     ".": {
+      "worker": {
+        "import": {
+          "types": "./types/index.d.ts",
+          "default": "./dist/server.js"
+        },
+        "require": "./dist/server.cjs"
+      },
       "browser": {
         "development": {
           "import": {
@@ -33,13 +40,6 @@
         },
         "require": "./dist/server.cjs"
       },
-      "worker": {
-        "import": {
-          "types": "./types/index.d.ts",
-          "default": "./dist/server.js"
-        },
-        "require": "./dist/server.cjs"
-      },
       "node": {
         "import": {
           "types": "./types/index.d.ts",

package/types/reactive/signal.d.ts

@@ -173,7 +173,7 @@ export declare function createEffect<Next, Init = Next>(fn: EffectFunction<Init
  * @description https://www.solidjs.com/docs/latest/api#createreaction
  */
 export declare function createReaction(onInvalidate: () => void, options?: EffectOptions): (tracking: () => void) => void;
-interface Memo<Prev, Next = Prev> extends SignalState<Next>, Computation<Next> {
+export interface Memo<Prev, Next = Prev> extends SignalState<Next>, Computation<Next> {
     tOwned?: Computation<Prev | Next, Next>[];
 }
 export interface MemoOptions<T> extends EffectOptions {

package/universal/dist/dev.cjs

@@ -4,12 +4,6 @@ Object.defineProperty(exports, '__esModule', { value: true });
 
 var solidJs = require('solid-js');
 
-function memo(fn, equals) {
-  return solidJs.createMemo(fn, undefined, !equals ? {
-    equals
-  } : undefined);
-}
-
 function createRenderer$1({
   createElement,
   createTextNode,
@@ -195,10 +189,10 @@ function createRenderer$1({
   }
   function spreadExpression(node, props, prevProps = {}, skipChildren) {
     props || (props = {});
-    if (!skipChildren && "children" in props) {
+    if (!skipChildren) {
       solidJs.createRenderEffect(() => prevProps.children = insertExpression(node, props.children, prevProps.children));
     }
-    props.ref && props.ref(node);
+    solidJs.createRenderEffect(() => props.ref && props.ref(node));
     solidJs.createRenderEffect(() => {
       for (const prop in props) {
         if (prop === "children" || prop === "ref") continue;
@@ -232,25 +226,15 @@ function createRenderer$1({
       setProperty(node, name, value, prev);
       return value;
     },
-    mergeProps,
+    mergeProps: solidJs.mergeProps,
     effect: solidJs.createRenderEffect,
-    memo,
+    memo: solidJs.createMemo,
     createComponent: solidJs.createComponent,
     use(fn, element, arg) {
       return solidJs.untrack(() => fn(element, arg));
     }
   };
 }
-function mergeProps(...sources) {
-  const target = {};
-  for (let i = 0; i < sources.length; i++) {
-    let source = sources[i];
-    if (typeof source === "function") source = source();
-    const descriptors = Object.getOwnPropertyDescriptors(source);
-    Object.defineProperties(target, descriptors);
-  }
-  return target;
-}
 
 function createRenderer(options) {
   const renderer = createRenderer$1(options);

package/universal/dist/dev.js

@@ -1,10 +1,4 @@
-import { createMemo, createRoot, createRenderEffect, createComponent, untrack, mergeProps as mergeProps$1 } from 'solid-js';
-
-function memo(fn, equals) {
-  return createMemo(fn, undefined, !equals ? {
-    equals
-  } : undefined);
-}
+import { createRoot, createRenderEffect, mergeProps, createMemo, createComponent, untrack } from 'solid-js';
 
 function createRenderer$1({
   createElement,
@@ -191,10 +185,10 @@ function createRenderer$1({
   }
   function spreadExpression(node, props, prevProps = {}, skipChildren) {
     props || (props = {});
-    if (!skipChildren && "children" in props) {
+    if (!skipChildren) {
       createRenderEffect(() => prevProps.children = insertExpression(node, props.children, prevProps.children));
     }
-    props.ref && props.ref(node);
+    createRenderEffect(() => props.ref && props.ref(node));
     createRenderEffect(() => {
       for (const prop in props) {
         if (prop === "children" || prop === "ref") continue;
@@ -230,27 +224,17 @@ function createRenderer$1({
     },
     mergeProps,
     effect: createRenderEffect,
-    memo,
+    memo: createMemo,
     createComponent,
     use(fn, element, arg) {
       return untrack(() => fn(element, arg));
     }
   };
 }
-function mergeProps(...sources) {
-  const target = {};
-  for (let i = 0; i < sources.length; i++) {
-    let source = sources[i];
-    if (typeof source === "function") source = source();
-    const descriptors = Object.getOwnPropertyDescriptors(source);
-    Object.defineProperties(target, descriptors);
-  }
-  return target;
-}
 
 function createRenderer(options) {
   const renderer = createRenderer$1(options);
-  renderer.mergeProps = mergeProps$1;
+  renderer.mergeProps = mergeProps;
   return renderer;
 }
 

package/universal/dist/universal.cjs

@@ -4,12 +4,6 @@ Object.defineProperty(exports, '__esModule', { value: true });
 
 var solidJs = require('solid-js');
 
-function memo(fn, equals) {
-  return solidJs.createMemo(fn, undefined, !equals ? {
-    equals
-  } : undefined);
-}
-
 function createRenderer$1({
   createElement,
   createTextNode,
@@ -195,10 +189,10 @@ function createRenderer$1({
   }
   function spreadExpression(node, props, prevProps = {}, skipChildren) {
     props || (props = {});
-    if (!skipChildren && "children" in props) {
+    if (!skipChildren) {
       solidJs.createRenderEffect(() => prevProps.children = insertExpression(node, props.children, prevProps.children));
     }
-    props.ref && props.ref(node);
+    solidJs.createRenderEffect(() => props.ref && props.ref(node));
     solidJs.createRenderEffect(() => {
       for (const prop in props) {
         if (prop === "children" || prop === "ref") continue;
@@ -232,25 +226,15 @@ function createRenderer$1({
       setProperty(node, name, value, prev);
       return value;
     },
-    mergeProps,
+    mergeProps: solidJs.mergeProps,
     effect: solidJs.createRenderEffect,
-    memo,
+    memo: solidJs.createMemo,
     createComponent: solidJs.createComponent,
     use(fn, element, arg) {
       return solidJs.untrack(() => fn(element, arg));
     }
   };
 }
-function mergeProps(...sources) {
-  const target = {};
-  for (let i = 0; i < sources.length; i++) {
-    let source = sources[i];
-    if (typeof source === "function") source = source();
-    const descriptors = Object.getOwnPropertyDescriptors(source);
-    Object.defineProperties(target, descriptors);
-  }
-  return target;
-}
 
 function createRenderer(options) {
   const renderer = createRenderer$1(options);

package/universal/dist/universal.js

@@ -1,10 +1,4 @@
-import { createMemo, createRoot, createRenderEffect, createComponent, untrack, mergeProps as mergeProps$1 } from 'solid-js';
-
-function memo(fn, equals) {
-  return createMemo(fn, undefined, !equals ? {
-    equals
-  } : undefined);
-}
+import { createRoot, createRenderEffect, mergeProps, createMemo, createComponent, untrack } from 'solid-js';
 
 function createRenderer$1({
   createElement,
@@ -191,10 +185,10 @@ function createRenderer$1({
   }
   function spreadExpression(node, props, prevProps = {}, skipChildren) {
     props || (props = {});
-    if (!skipChildren && "children" in props) {
+    if (!skipChildren) {
       createRenderEffect(() => prevProps.children = insertExpression(node, props.children, prevProps.children));
     }
-    props.ref && props.ref(node);
+    createRenderEffect(() => props.ref && props.ref(node));
     createRenderEffect(() => {
       for (const prop in props) {
         if (prop === "children" || prop === "ref") continue;
@@ -230,27 +224,17 @@ function createRenderer$1({
     },
     mergeProps,
     effect: createRenderEffect,
-    memo,
+    memo: createMemo,
     createComponent,
     use(fn, element, arg) {
       return untrack(() => fn(element, arg));
     }
   };
 }
-function mergeProps(...sources) {
-  const target = {};
-  for (let i = 0; i < sources.length; i++) {
-    let source = sources[i];
-    if (typeof source === "function") source = source();
-    const descriptors = Object.getOwnPropertyDescriptors(source);
-    Object.defineProperties(target, descriptors);
-  }
-  return target;
-}
 
 function createRenderer(options) {
   const renderer = createRenderer$1(options);
-  renderer.mergeProps = mergeProps$1;
+  renderer.mergeProps = mergeProps;
   return renderer;
 }