@builder.io/sdk-solid 0.1.11 → 0.1.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@builder.io/sdk-solid",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "",
   "type": "module",
   "main": "./solid-index.jsx",

package/src/blocks/symbol/symbol.jsx

@@ -1,4 +1,4 @@
-import { useContext, on, createEffect, createSignal } from "solid-js";
+import { useContext, onMount, on, createEffect, createSignal } from "solid-js";
 
 import RenderContent from "../../components/render-content/render-content.jsx";
 import BuilderContext from "../../context/builder.context.js";
@@ -6,10 +6,8 @@ import { getContent } from "../../functions/get-content/index.js";
 import { TARGET } from "../../constants/target";
 
 function Symbol(props) {
-  const [fetchedContent, setFetchedContent] = createSignal(null);
-
-  function className() {
-    return [
+  const [className, setClassName] = createSignal(
+    [
       ...(TARGET === "vue2" || TARGET === "vue3"
         ? Object.keys(props.attributes.class)
         : [props.attributes.class]),
@@ -20,18 +18,12 @@ function Symbol(props) {
         : undefined,
     ]
       .filter(Boolean)
-      .join(" ");
-  }
-
-  function contentToUse() {
-    return props.symbol?.content || fetchedContent();
-  }
-
-  const builderContext = useContext(BuilderContext);
+      .join(" ")
+  );
 
-  function onUpdateFn_0() {
-    const symbolToUse = props.symbol;
+  const [contentToUse, setContentToUse] = createSignal(props.symbol?.content);
 
+  function fetchContent() {
     /**
      * If:
      * - we have a symbol prop
@@ -42,25 +34,40 @@ function Symbol(props) {
      * then we want to re-fetch the symbol content.
      */
     if (
-      symbolToUse &&
-      !symbolToUse.content &&
-      !fetchedContent() &&
-      symbolToUse.model &&
+      !contentToUse() &&
+      props.symbol?.model &&
       // This is a hack, we should not need to check for this, but it is needed for Svelte.
       builderContext?.apiKey
     ) {
       getContent({
-        model: symbolToUse.model,
+        model: props.symbol.model,
         apiKey: builderContext.apiKey,
+        apiVersion: builderContext.apiVersion,
         query: {
-          id: symbolToUse.entry,
+          id: props.symbol.entry,
         },
-      }).then((response) => {
-        setFetchedContent(response);
-      });
+      })
+        .then((response) => {
+          if (response) {
+            setContentToUse(response);
+          }
+        })
+        .catch((err) => {
+          console.error("[Builder.io]: Could not fetch symbol content: ", err);
+        });
     }
   }
-  createEffect(on(() => [props.symbol, fetchedContent()], onUpdateFn_0));
+
+  const builderContext = useContext(BuilderContext);
+
+  onMount(() => {
+    fetchContent();
+  });
+
+  function onUpdateFn_0() {
+    fetchContent();
+  }
+  createEffect(on(() => [props.symbol], onUpdateFn_0));
 
   return (
     <div
@@ -71,13 +78,14 @@ function Symbol(props) {
       }}
     >
       <RenderContent
+        apiVersion={builderContext.apiVersion}
         apiKey={builderContext.apiKey}
         context={builderContext.context}
         customComponents={Object.values(builderContext.registeredComponents)}
         data={{
           ...props.symbol?.data,
           ...builderContext.state,
-          ...props.symbol?.content?.data?.state,
+          ...contentToUse()?.data?.state,
         }}
         model={props.symbol?.model}
         content={contentToUse()}

package/src/components/render-content/render-content.jsx

@@ -304,6 +304,7 @@ function RenderContent(props) {
           getContent({
             model: props.model,
             apiKey: props.apiKey,
+            apiVersion: props.apiVersion,
           }).then((content) => {
             if (content) {
               mergeNewContent(content);
@@ -318,21 +319,28 @@ function RenderContent(props) {
   });
 
   function onUpdateFn_0() {
+    if (props.content) {
+      mergeNewContent(props.content);
+    }
+  }
+  createEffect(on(() => [props.content], onUpdateFn_0));
+
+  function onUpdateFn_1() {
     evaluateJsCode();
   }
   createEffect(
-    on(() => [useContent?.data?.jsCode, contentState()], onUpdateFn_0)
+    on(() => [useContent?.data?.jsCode, contentState()], onUpdateFn_1)
   );
 
-  function onUpdateFn_1() {
+  function onUpdateFn_2() {
     runHttpRequests();
   }
-  createEffect(on(() => [useContent?.data?.httpRequests], onUpdateFn_1));
+  createEffect(on(() => [useContent?.data?.httpRequests], onUpdateFn_2));
 
-  function onUpdateFn_2() {
+  function onUpdateFn_3() {
     emitStateUpdate();
   }
-  createEffect(on(() => [contentState()], onUpdateFn_2));
+  createEffect(on(() => [contentState()], onUpdateFn_3));
 
   return (
     <builderContext.Provider
@@ -342,6 +350,7 @@ function RenderContent(props) {
         setState: setContextState,
         context: props.context || {},
         apiKey: props.apiKey,
+        apiVersion: props.apiVersion,
         registeredComponents: allRegisteredComponents(),
       }}
     >

package/src/context/builder.context.js

@@ -6,6 +6,7 @@ var stdin_default = createContext({
   setState() {
   },
   apiKey: null,
+  apiVersion: void 0,
   registeredComponents: {},
   inheritedStyles: {}
 });

package/src/functions/get-content/generate-content-url.js

@@ -28,12 +28,16 @@ const generateContentUrl = (options) => {
     model,
     apiKey,
     includeRefs = true,
-    locale
+    locale,
+    apiVersion = "v2"
   } = options;
   if (!apiKey) {
     throw new Error("Missing API key");
   }
-  const url = new URL(`https://cdn.builder.io/api/v2/content/${model}?apiKey=${apiKey}&limit=${limit}&noTraverse=${noTraverse}&includeRefs=${includeRefs}${locale ? `&locale=${locale}` : ""}`);
+  if (!["v2", "v3"].includes(apiVersion)) {
+    throw new Error(`Invalid apiVersion: expected 'v2' or 'v3', received '${apiVersion}'`);
+  }
+  const url = new URL(`https://cdn.builder.io/api/${apiVersion}/content/${model}?apiKey=${apiKey}&limit=${limit}&noTraverse=${noTraverse}&includeRefs=${includeRefs}${locale ? `&locale=${locale}` : ""}`);
   const queryOptions = __spreadValues(__spreadValues({}, getBuilderSearchParamsFromWindow()), normalizeSearchParams(options.options || {}));
   const flattened = flatten(queryOptions);
   for (const key in flattened) {

package/src/functions/get-content/generate-content-url.test.js

@@ -28,4 +28,46 @@ describe("Generate Content URL", () => {
     });
     expect(output).toMatchSnapshot();
   });
+  test("generate content url with apiVersion as v2", () => {
+    const output = generateContentUrl({
+      apiKey: testKey,
+      model: testModel,
+      query: { id: testId },
+      options,
+      apiVersion: "v2"
+    });
+    expect(output).toMatchSnapshot();
+  });
+  test("generate content url with apiVersion as v3", () => {
+    const output = generateContentUrl({
+      apiKey: testKey,
+      model: testModel,
+      query: { id: testId },
+      options,
+      apiVersion: "v3"
+    });
+    expect(output).toMatchSnapshot();
+  });
+  test("throw error when trying to generate content url with apiVersion as v1", () => {
+    expect(() => {
+      generateContentUrl({
+        apiKey: testKey,
+        model: testModel,
+        query: { id: testId },
+        options,
+        apiVersion: "v1"
+      });
+    }).toThrow(`Invalid apiVersion: expected 'v2' or 'v3', received 'v1'`);
+  });
+  test("throw error when trying to generate content url with an invalid apiVersion value", () => {
+    expect(() => {
+      generateContentUrl({
+        apiKey: testKey,
+        model: testModel,
+        query: { id: testId },
+        options,
+        apiVersion: "INVALID_API_VERSION"
+      });
+    }).toThrow(`Invalid apiVersion: expected 'v2' or 'v3', received 'INVALID_API_VERSION'`);
+  });
 });