Merge 3e7d535 into 7a9187f

packages/solid/src/reactive/mutable.ts

@@ -54,7 +54,7 @@ function wrap<T extends StateNode>(value: T, name?: string): State<T> {
   let p = value[$PROXY];
   if (!p) {
     Object.defineProperty(value, $PROXY, { value: (p = new Proxy(value, proxyTraps)) });
-    let keys = Object.keys(value),
+    const keys = Object.keys(value),
       desc = Object.getOwnPropertyDescriptors(value);
     for (let i = 0, l = keys.length; i < l; i++) {
       const prop = keys[i];

packages/solid/src/reactive/scheduler.ts

@@ -89,8 +89,8 @@ function enqueue(taskQueue: Task[], task: Task) {
     let n = taskQueue.length - 1;
 
     while (m <= n) {
-      let k = (n + m) >> 1;
-      let cmp = task.expirationTime - taskQueue[k].expirationTime;
+      const k = (n + m) >> 1;
+      const cmp = task.expirationTime - taskQueue[k].expirationTime;
       if (cmp > 0) m = k + 1;
       else if (cmp < 0) n = k - 1;
       else return k;

packages/solid/src/reactive/signal.ts

@@ -85,9 +85,9 @@ export function createRoot<T>(fn: (dispose: () => void) => T, detachedOwner?: Ow
     root: Owner =
       fn.length === 0 && !"_SOLID_DEV_"
         ? UNOWNED
-        : { owned: null, cleanups: null, context: null, owner, attached: !!detachedOwner };
+        : { owned: null, cleanups: null, context: null, owner, attached: Boolean(detachedOwner) };
 
-  if ("_SOLID_DEV_" && owner) root.name = (owner as Computation<any>).name + "-r" + rootCount++;
+  if ("_SOLID_DEV_" && owner) root.name = `${(owner as Computation<any>).name}-r${rootCount++}`;
   Owner = root;
   Listener = null;
   let result: T;
@@ -288,7 +288,7 @@ export function createResource<T, U>(
     batch(() => {
       set(() => v);
       setLoading(false);
-      for (let c of contexts.keys()) c.decrement!();
+      for (const c of contexts.keys()) c.decrement!();
       contexts.clear();
     });
   }
@@ -313,7 +313,7 @@ export function createResource<T, U>(
   }
   function load() {
     setError((err = undefined));
-    let lookup = dynamic ? (source as () => U)() : (source as U);
+    const lookup = dynamic ? (source as () => U)() : (source as U);
     loadedUnderTransition = (Transition && Transition.running) as boolean;
     if (lookup == null || (lookup as any) === false) {
       loadEnd(pr, untrack(s)!);
@@ -386,7 +386,7 @@ export function createSelector<T, U>(
   fn: (a: U, b: T) => boolean = equalFn as any,
   options?: { name?: string }
 ): (key: U) => boolean {
-  let subs = new Map<U, Set<Computation<any>>>();
+  const subs = new Map<U, Set<Computation<any>>>();
   const node = createComputation(
     (p: T | undefined) => {
       const v = source();
@@ -476,7 +476,7 @@ export function on<T extends (() => any) | (() => any)[], U>(
   fn: (values: ReturnTypes<T>, prev: ReturnTypes<T>, prevResults?: U) => U,
   options?: { defer?: boolean }
 ): (prev?: U) => U | undefined {
-  let isArray = Array.isArray(deps);
+  const isArray = Array.isArray(deps);
   let prev: ReturnTypes<T>;
   let defer = options && options.defer;
   return prevResult => {
@@ -585,8 +585,8 @@ export function devComponent<T>(Comp: (props: T) => JSX.Element, props: T) {
 export function hashValue(v: any): string {
   const s = new Set();
   return (
-    "s" +
-    (typeof v === "string"
+    `s${
+    typeof v === "string"
       ? hash(v)
       : hash(
           JSON.stringify(v, (k, v) => {
@@ -596,7 +596,7 @@ export function hashValue(v: any): string {
             }
             return v;
           }) || ""
-        ))
+        )}`
   );
 }
 
@@ -605,7 +605,7 @@ export function registerGraph(name: string, value: { value: unknown }): string {
   if (Owner) {
     let i = 0;
     Owner.sourceMap || (Owner.sourceMap = {});
-    while (Owner.sourceMap[tryName]) tryName = name + "-" + ++i;
+    while (Owner.sourceMap[tryName]) tryName = `${name}-${++i}`;
     Owner.sourceMap[tryName] = value;
   }
   return tryName;
@@ -804,9 +804,9 @@ function createComputation<T>(
     if ("_SOLID_DEV_")
       c.name =
         (options && options.name) ||
-        ((Owner as Computation<any>).name || "c") +
-          "-" +
-          (Owner.owned || (Owner as Memo<T>).tOwned!).length;
+        `${(Owner as Computation<any>).name || "c"}-${
+          (Owner.owned || (Owner as Memo<T>).tOwned!).length
+        }`;
   }
   return c;
 }
@@ -1009,7 +1009,7 @@ function resolveChildren(children: any): unknown {
   if (Array.isArray(children)) {
     const results: any[] = [];
     for (let i = 0; i < children.length; i++) {
-      let result = resolveChildren(children[i]);
+      const result = resolveChildren(children[i]);
       Array.isArray(result) ? results.push.apply(results, result) : results.push(result);
     }
     return results;

packages/solid/src/reactive/state.ts

@@ -47,7 +47,7 @@ function wrap<T extends StateNode>(value: T, name?: string): State<T> {
   let p = value[$PROXY];
   if (!p) {
     Object.defineProperty(value, $PROXY, { value: (p = new Proxy(value, proxyTraps)) });
-    let keys = Object.keys(value),
+    const keys = Object.keys(value),
       desc = Object.getOwnPropertyDescriptors(value);
     for (let i = 0, l = keys.length; i < l; i++) {
       const prop = keys[i];
@@ -86,7 +86,7 @@ export function unwrap<T extends StateNode>(item: any, set = new Set()): T {
   } else {
     if (Object.isFrozen(item)) item = Object.assign({}, item);
     else set.add(item);
-    let keys = Object.keys(item),
+    const keys = Object.keys(item),
       desc = Object.getOwnPropertyDescriptors(item);
     for (let i = 0, l = keys.length; i < l; i++) {
       prop = keys[i];

packages/solid/src/reactive/stateModifiers.ts

@@ -21,7 +21,7 @@ function applyState(
   merge: boolean | undefined,
   key: string | null
 ) {
-  let previous = parent[property];
+  const previous = parent[property];
   if (target === previous) return;
   if (!isWrappable(target) || !isWrappable(previous) || (key && target[key] !== previous[key])) {
     target !== previous && setProperty(parent, property, target);

packages/solid/src/static/reactive.ts

@@ -76,8 +76,8 @@ export function on<T, U>(
   fn: (value: Array<T> | T, prev?: Array<T> | T, prevResults?: U) => U,
   options: { defer?: boolean } = {}
 ): (prev?: U) => U | undefined {
-  let isArray = Array.isArray(deps);
-  let defer = options.defer;
+  const isArray = Array.isArray(deps);
+  const defer = options.defer;
   return () => {
     if (defer) return undefined;
     let value: Array<T> | T;
@@ -151,7 +151,7 @@ function resolveChildren(children: any): unknown {
   if (Array.isArray(children)) {
     const results: any[] = [];
     for (let i = 0; i < children.length; i++) {
-      let result = resolveChildren(children[i]);
+      const result = resolveChildren(children[i]);
       Array.isArray(result) ? results.push.apply(results, result) : results.push(result);
     }
     return results;

packages/solid/src/static/rendering.ts

@@ -359,7 +359,7 @@ export function lazy(fn: () => Promise<{ default: any }>): (props: any) => strin
 }
 
 function suspenseComplete(c: SuspenseContextType) {
-  for (let r of c.resources.values()) {
+  for (const r of c.resources.values()) {
     if (r.loading) return false;
   }
   return true;
@@ -440,7 +440,7 @@ export function Suspense(props: { fallback: string; children: string }) {
   return sharedConfig.context!.async ? { t: `<#${id}#>` } : props.fallback;
 }
 
-const SUSPENSE_REPLACE = /<#([0-9\.]+)\#>/;
+const SUSPENSE_REPLACE = /<#([\d.]+)#>/;
 export function awaitSuspense(fn: () => any) {
   return new Promise(resolve => {
     const registry = new Set<string>();

packages/solid/test/mutable.spec.ts

@@ -95,7 +95,7 @@ describe("Simple update modes", () => {
 
 describe("Unwrapping Edge Cases", () => {
   test("Unwrap nested frozen state object", () => {
-    var state = createMutable({
+    const state = createMutable({
         data: Object.freeze({ user: { firstName: "John", lastName: "Snow" } })
       }),
       s = unwrap({ ...state });
@@ -105,7 +105,7 @@ describe("Unwrapping Edge Cases", () => {
     expect(s.data.user[$RAW]).toBeUndefined();
   });
   test("Unwrap nested frozen array", () => {
-    var state = createMutable({
+    const state = createMutable({
         data: [{ user: { firstName: "John", lastName: "Snow" } }]
       }),
       s = unwrap({ data: state.data.slice(0) });
@@ -115,7 +115,7 @@ describe("Unwrapping Edge Cases", () => {
     expect(s.data[0].user[$RAW]).toBeUndefined();
   });
   test("Unwrap nested frozen state array", () => {
-    var state = createMutable({
+    const state = createMutable({
         data: Object.freeze([{ user: { firstName: "John", lastName: "Snow" } }])
       }),
       s = unwrap({ ...state });
@@ -129,8 +129,8 @@ describe("Unwrapping Edge Cases", () => {
 describe("Tracking State changes", () => {
   test("Track a state change", () => {
     createRoot(() => {
-      var state = createMutable({ data: 2 }),
-        executionCount = 0;
+      const state = createMutable({ data: 2 })
+      let executionCount = 0;
 
       expect.assertions(2);
       createComputed(() => {
@@ -152,10 +152,10 @@ describe("Tracking State changes", () => {
 
   test("Track a nested state change", () => {
     createRoot(() => {
-      var state = createMutable({
+      const state = createMutable({
           user: { firstName: "John", lastName: "Smith" }
-        }),
-        executionCount = 0;
+        })
+      let executionCount = 0;
 
       expect.assertions(2);
       createComputed(() => {
@@ -178,15 +178,15 @@ describe("Tracking State changes", () => {
 describe("Handling functions in state", () => {
   test("Array Native Methods: Array.Filter", () => {
     createRoot(() => {
-      var state = createMutable({ list: [0, 1, 2] }),
+      const state = createMutable({ list: [0, 1, 2] }),
         getFiltered = createMemo(() => state.list.filter(i => i % 2));
       expect(getFiltered()).toStrictEqual([1]);
     });
   });
 
   test("Track function change", () => {
     createRoot(() => {
-      var state = createMutable<{ fn: () => number }>({
+      const state = createMutable<{ fn: () => number }>({
           fn: () => 1
         }),
         getValue = createMemo(() => state.fn());
@@ -199,7 +199,7 @@ describe("Handling functions in state", () => {
 describe("Setting state from Effects", () => {
   test("Setting state from signal", () => {
     createRoot(() => {
-      var [getData, setData] = createSignal("init"),
+      const [getData, setData] = createSignal("init"),
         state = createMutable({ data: "" });
       createComputed(() => (state.data = getData()));
       setData("signal");
@@ -209,7 +209,7 @@ describe("Setting state from Effects", () => {
 
   test("Select Promise", done => {
     createRoot(async () => {
-      var p = new Promise<string>(resolve => {
+      const p = new Promise<string>(resolve => {
           setTimeout(resolve, 20, "promised");
         }),
         state = createMutable({ data: "" });