Refactor: Rename loading to _loading and optimize createResource logic to prevent conflicts and improve resource management.

Author: dennev

.changeset/moody-clowns-vanish.md

@@ -0,0 +1,5 @@
+---
+"solid-js": patch
+---
+
+Refactor: Rename `loading` to `_loading` and optimize `createResource` logic to prevent conflicts and improve resource management.

packages/solid/src/server/rendering.ts

@@ -450,34 +450,48 @@ export function createResource<T, S>(
       return resource.ref;
     }
   }
-  const read = () => {
+  const prepareResource = () => {
     if (error) throw error;
     const resolved =
       options.ssrLoadFrom !== "initial" &&
       sharedConfig.context!.async &&
       "data" in sharedConfig.context!.resources[id];
     if (!resolved && resourceContext) resourceContext.push(id);
-    if (!resolved && read.loading) {
+    if (!resolved && read._loading) {
       const ctx = useContext(SuspenseContext);
       if (ctx) {
-        ctx.resources.set(id, read);
+        ctx.resources.set(id, read as unknown as { loading: boolean; error: any });
         contexts.add(ctx);
       }
     }
-    return resolved ? sharedConfig.context!.resources[id].data : value;
+    return resolved;
+  };
+  const read = () => {
+    return prepareResource() ? sharedConfig.context!.resources[id].data : value;
   };
-  read.loading = false;
+  const loading = () => {
+    prepareResource();
+    return read._loading;
+  };
+  read._loading = false;
   read.error = undefined as any;
   read.state = "initialValue" in options ? "ready" : "unresolved";
-  Object.defineProperty(read, "latest", {
-    get() {
-      return read();
+  Object.defineProperties(read, {
+    latest: {
+      get() {
+        return read();
+      }
+    },
+    loading: {
+      get() {
+        return loading();
+      }
     }
   });
   function load() {
     const ctx = sharedConfig.context!;
     if (!ctx.async)
-      return (read.loading = !!(typeof source === "function" ? (source as () => S)() : source));
+      return (read._loading = !!(typeof source === "function" ? (source as () => S)() : source));
     if (ctx.resources && id in ctx.resources && "data" in ctx.resources[id]) {
       value = ctx.resources[id].data;
       return;
@@ -495,19 +509,19 @@ export function createResource<T, S>(
       p = (fetcher as ResourceFetcher<S, T>)(lookup, { value });
     }
     if (p != undefined && typeof p === "object" && "then" in p) {
-      read.loading = true;
+      read._loading = true;
       read.state = "pending";
       p = p
         .then(res => {
-          read.loading = false;
+          read._loading = false;
           read.state = "ready";
           ctx.resources[id].data = res;
           p = null;
           notifySuspense(contexts);
           return res;
         })
         .catch(err => {
-          read.loading = false;
+          read._loading = false;
           read.state = "errored";
           read.error = error = castError(err);
           p = null;
@@ -523,7 +537,10 @@ export function createResource<T, S>(
     return ctx.resources[id].data;
   }
   if (options.ssrLoadFrom !== "initial") load();
-  const ref = [read, { refetch: load, mutate: (v: T) => (value = v) }] as ResourceReturn<T>;
+  const ref = [
+    read as unknown as Resource<T>,
+    { refetch: load, mutate: (v: T) => (value = v) }
+  ] as ResourceReturn<T>;
   if (p) resource.ref = ref;
   return ref;
 }