@nordcraft/runtime 1.0.75 → 1.0.77

package/src/editor-preview.main.ts

@@ -453,7 +453,7 @@ export const createRoot = (
 
             const node = getDOMNodeFromNodeId(selectedNodeId)
             const element =
-              component?.nodes[node?.getAttribute('data-node-id') ?? '']
+              component?.nodes?.[node?.getAttribute('data-node-id') ?? '']
             if (
               node &&
               element &&
@@ -544,7 +544,7 @@ export const createRoot = (
               return false
             }
             const nodeId = getNodeId(component, id.split('.').slice(1))
-            const node = nodeId ? component?.nodes[nodeId] : undefined
+            const node = nodeId ? component?.nodes?.[nodeId] : undefined
             if (!node) {
               return false
             }
@@ -562,7 +562,7 @@ export const createRoot = (
             if (message.data.metaKey) {
               // Figure out if the clicked element is a text element
               // or if one of its descendants is a text element
-              const root = component.nodes.root
+              const root = component.nodes?.root
               if (root && id) {
                 const nodeLookup = getNodeAndAncestors(component, root, id)
                 if (nodeLookup?.node.type === 'text') {
@@ -574,7 +574,7 @@ export const createRoot = (
                   const firstTextChild =
                     nodeLookup?.node.type === 'element'
                       ? nodeLookup.node.children.find(
-                          (c) => component?.nodes[c]?.type === 'text',
+                          (c) => component?.nodes?.[c]?.type === 'text',
                         )
                       : undefined
                   if (firstTextChild) {
@@ -603,7 +603,7 @@ export const createRoot = (
             mode === 'design'
           ) {
             // Figure out if the clicked element is a component
-            const root = component.nodes.root
+            const root = component.nodes?.root
             if (root) {
               const nodeLookup = getNodeAndAncestors(component, root, id)
               if (
@@ -659,7 +659,7 @@ export const createRoot = (
         }
         case 'introspect_qraphql_api': {
           const { apiKey } = message.data
-          const api = component?.apis[apiKey]
+          const api = component?.apis?.[apiKey]
           if (api && !isLegacyApi(api) && component) {
             const formulaContext: FormulaContext = {
               component,
@@ -737,7 +737,7 @@ export const createRoot = (
                     parent: parentDataId,
                     index: !isNaN(nextSiblingId)
                       ? nextSiblingId
-                      : component?.nodes[parentNodeId]?.children?.length,
+                      : component?.nodes?.[parentNodeId]?.children?.length,
                   })
                   dragState = null
                 })
@@ -913,38 +913,9 @@ export const createRoot = (
           })
           break
         case 'preview_style':
-          const { styles: previewStyleStyles, theme, resources } = message.data
+          const { styles: previewStyleStyles, theme } = message.data
           cancelAnimationFrame(previewStyleAnimationFrame)
           previewStyleAnimationFrame = requestAnimationFrame(() => {
-            // Allow for temporarily adding preview resources (e.g. fonts).
-            const resourceElements = Array.from(
-              document.head.querySelectorAll('[data-id="preview-resource"]'),
-            )
-            // Remove any resources that are no longer needed
-            resourceElements.forEach((el) => {
-              if (
-                !resources ||
-                resources.length === 0 ||
-                !resources.some((res) => res.href === el.getAttribute('href'))
-              ) {
-                el.remove()
-              }
-            })
-            resources
-              ?.filter(
-                (resource) =>
-                  !resourceElements.some(
-                    (el) => el.getAttribute('href') === resource.href,
-                  ),
-              )
-              .forEach((resource) => {
-                const resourceElement = document.createElement('link')
-                resourceElement.setAttribute('data-id', 'preview-resource')
-                resourceElement.rel = 'stylesheet'
-                resourceElement.href = resource.href
-                document.head.appendChild(resourceElement)
-              })
-
             // Update or create a new style tag and set the given styles with important priority
             let styleElement = document.head.querySelector(
               '[data-id="selected-node-styles"]',
@@ -965,20 +936,23 @@ export const createRoot = (
             // If style variant targets a pseudo-element, apply styles to it instead
             let pseudoElement = ''
             if (component && styleVariantSelection) {
-              const nodeLookup = getNodeAndAncestors(
-                component,
-                component.nodes.root,
-                styleVariantSelection.nodeId,
-              )
+              const rootNode = component.nodes?.root
+              if (rootNode) {
+                const nodeLookup = getNodeAndAncestors(
+                  component,
+                  rootNode,
+                  styleVariantSelection.nodeId,
+                )
 
-              if (
-                (nodeLookup?.node.type === 'element' ||
-                  nodeLookup?.node.type === 'component') &&
-                nodeLookup.node.variants?.[
-                  styleVariantSelection.styleVariantIndex
-                ].pseudoElement
-              ) {
-                pseudoElement = `::${nodeLookup.node.variants[styleVariantSelection.styleVariantIndex].pseudoElement}`
+                if (
+                  (nodeLookup?.node.type === 'element' ||
+                    nodeLookup?.node.type === 'component') &&
+                  nodeLookup.node.variants?.[
+                    styleVariantSelection.styleVariantIndex
+                  ].pseudoElement
+                ) {
+                  pseudoElement = `::${nodeLookup.node.variants[styleVariantSelection.styleVariantIndex].pseudoElement}`
+                }
               }
             }
 
@@ -1044,6 +1018,37 @@ export const createRoot = (
             }
           })
           break
+        case 'preview_resources': {
+          const { resources } = message.data
+          // Allow for temporarily adding preview resources (e.g. fonts).
+          const resourceElements = Array.from(
+            document.head.querySelectorAll('[data-id="preview-resource"]'),
+          )
+          // Remove any resources that are no longer needed
+          resourceElements.forEach((el) => {
+            if (
+              resources.length === 0 ||
+              !resources.some((res) => res.href === el.getAttribute('href'))
+            ) {
+              el.remove()
+            }
+          })
+          resources
+            .filter(
+              (resource) =>
+                !resourceElements.some(
+                  (el) => el.getAttribute('href') === resource.href,
+                ),
+            )
+            .forEach((resource) => {
+              const resourceElement = document.createElement('link')
+              resourceElement.setAttribute('data-id', 'preview-resource')
+              resourceElement.rel = 'stylesheet'
+              resourceElement.href = resource.href
+              document.head.appendChild(resourceElement)
+            })
+          break
+        }
         case 'preview_theme': {
           const { theme } = message.data
           if (theme) {
@@ -1078,7 +1083,7 @@ export const createRoot = (
     }
     if (mode === 'design') {
       if (selectedNodeId !== null) {
-        const root = _component?.nodes.root
+        const root = _component?.nodes?.root
         if (root) {
           const nodeLookup = getNodeAndAncestors(
             _component,
@@ -1106,7 +1111,7 @@ export const createRoot = (
         nodeId: selectedNodeId,
         styleVariantIndex: variantIndex,
       }
-      const root = component?.nodes.root
+      const root = component?.nodes?.root
       if (root && component) {
         const nodeLookup = getNodeAndAncestors(component, root, selectedNodeId)
         if (nodeLookup) {
@@ -1187,10 +1192,10 @@ export const createRoot = (
     if (
       fastDeepEqual(ctx?.component.attributes, _component.attributes) === false
     ) {
-      Attributes = mapObject(_component.attributes, ([name, { testValue }]) => [
-        name,
-        testValue,
-      ])
+      Attributes = mapObject(
+        _component.attributes ?? {},
+        ([name, { testValue }]) => [name, testValue],
+      )
     }
     if (
       _component.route &&
@@ -1234,10 +1239,10 @@ export const createRoot = (
         )
       }
 
-      Attributes = mapObject(_component.attributes, ([name, { testValue }]) => [
-        name,
-        testValue,
-      ])
+      Attributes = mapObject(
+        _component.attributes ?? {},
+        ([name, { testValue }]) => [name, testValue],
+      )
     }
     if (
       fastDeepEqual(
@@ -1282,7 +1287,7 @@ export const createRoot = (
             const formulaContext: FormulaContext = {
               data: {
                 Attributes: mapObject(
-                  providerComponent.attributes,
+                  providerComponent.attributes ?? {},
                   ([name, attr]) => [name, attr.testValue],
                 ),
                 // Recursively resolve contexts providers before their children to build up the fake context tree in preview mode
@@ -1316,7 +1321,7 @@ export const createRoot = (
               }
             }
             formulaContext.data.Variables = mapObject(
-              providerComponent.variables,
+              providerComponent.variables ?? {},
               ([name, variable]) => [
                 name,
                 applyFormula(variable.initialValue, formulaContext),
@@ -1350,7 +1355,7 @@ export const createRoot = (
       fastDeepEqual(_component.variables, ctx?.component.variables) === false
     ) {
       Variables = mapObject(
-        _component.variables,
+        _component.variables ?? {},
         ([name, { initialValue }]) => [
           name,
           applyFormula(initialValue, {
@@ -1388,7 +1393,7 @@ export const createRoot = (
     for (const api in newCtx.component.apis) {
       // check if the api has changed (ignoring onCompleted and onFailed).
       const apiInstance = newCtx.component.apis[api]
-      const previousApiInstance = ctx?.component.apis[api]
+      const previousApiInstance = ctx?.component.apis?.[api]
       if (isLegacyApi(apiInstance)) {
         if (
           fastDeepEqual(
@@ -1405,7 +1410,7 @@ export const createRoot = (
               Apis: omitKeys(data.Apis ?? {}, [
                 ...Object.keys(data.Apis ?? {}).filter(
                   // remove any data from an api that is not part of the component
-                  (key) => !newCtx.component.apis[key],
+                  (key) => !newCtx.component.apis?.[key],
                 ),
                 api,
               ]),
@@ -1605,7 +1610,7 @@ export const createRoot = (
   const updateConditionalElements = () => {
     const displayedNodes: string[] = []
     if (selectedNodeId && component) {
-      const root = component.nodes.root
+      const root = component.nodes?.root
       if (root) {
         const nodeLookup = getNodeAndAncestors(component, root, selectedNodeId)
         if (isNodeOrAncestorConditional(nodeLookup)) {
@@ -1721,7 +1726,7 @@ function getNodeId(component: Component, path: string[]) {
     if (nextChild === undefined || currentId === undefined) {
       return currentId ?? null
     }
-    const currentNode = component.nodes[currentId]
+    const currentNode = component.nodes?.[currentId]
     if (!currentNode?.children) {
       return null
     }

package/src/page.main.ts

@@ -162,7 +162,7 @@ export const createRoot = (domNode: HTMLElement) => {
     ...window.toddle.pageState,
     // Re-initialize variables since some of them might rely on client-side
     // state (e.g. localStorage, sensors etc.)
-    Variables: mapObject(component.variables, ([name, variable]) => [
+    Variables: mapObject(component.variables ?? {}, ([name, variable]) => [
       name,
       applyFormula(variable.initialValue, {
         data: window.toddle.pageState,
@@ -213,17 +213,19 @@ export const createRoot = (domNode: HTMLElement) => {
   }
 
   // Note: this function must run procedurally to ensure apis (which are in correct order) can reference each other
-  sortApiObjects(Object.entries(component.apis)).forEach(([name, api]) => {
-    if (isLegacyApi(api)) {
-      ctx.apis[name] = createLegacyAPI(api, ctx)
-    } else {
-      ctx.apis[name] = createAPI({
-        apiRequest: api,
-        ctx,
-        componentData: dataSignal.get(),
-      })
-    }
-  })
+  sortApiObjects(Object.entries(component.apis ?? {})).forEach(
+    ([name, api]) => {
+      if (isLegacyApi(api)) {
+        ctx.apis[name] = createLegacyAPI(api, ctx)
+      } else {
+        ctx.apis[name] = createAPI({
+          apiRequest: api,
+          ctx,
+          componentData: dataSignal.get(),
+        })
+      }
+    },
+  )
   // Trigger actions for all APIs after all of them are created.
   Object.values(ctx.apis)
     .filter(isContextApiV2)

package/src/styles/CustomPropertyStyleSheet.ts

@@ -1,4 +1,5 @@
 import type { MediaQuery } from '@nordcraft/core/dist/component/component.types'
+import type { Nullable } from '@nordcraft/core/dist/types'
 
 /**
  * CustomPropertyStyleSheet is a utility class that manages CSS custom properties
@@ -15,7 +16,10 @@ export class CustomPropertyStyleSheet {
   // Selector to rule index mapping
   private ruleMap: Map<string, CSSStyleRule | CSSNestedDeclarations> | undefined
 
-  constructor(root: Document | ShadowRoot, styleSheet?: CSSStyleSheet | null) {
+  constructor(
+    root: Document | ShadowRoot,
+    styleSheet?: Nullable<CSSStyleSheet>,
+  ) {
     if (styleSheet) {
       this.styleSheet = styleSheet
     } else {
@@ -30,10 +34,10 @@ export class CustomPropertyStyleSheet {
   public registerProperty(
     selector: string,
     name: string,
-    options?: {
-      mediaQuery?: MediaQuery
-      startingStyle?: boolean
-    },
+    options?: Nullable<{
+      mediaQuery?: Nullable<MediaQuery>
+      startingStyle?: Nullable<boolean>
+    }>,
   ): (newValue: string) => void {
     this.ruleMap ??= this.hydrateFromBase()
     const fullSelector = CustomPropertyStyleSheet.getFullSelector(
@@ -69,11 +73,11 @@ export class CustomPropertyStyleSheet {
   public unregisterProperty(
     selector: string,
     name: string,
-    options?: {
-      mediaQuery?: MediaQuery
-      startingStyle?: boolean
-      deepClean?: boolean
-    },
+    options?: Nullable<{
+      mediaQuery?: Nullable<MediaQuery>
+      startingStyle?: Nullable<boolean>
+      deepClean?: Nullable<boolean>
+    }>,
   ): void {
     if (!this.ruleMap) {
       return
@@ -161,10 +165,10 @@ export class CustomPropertyStyleSheet {
 
   private static getFullSelector(
     selector: string,
-    options?: {
-      mediaQuery?: MediaQuery
-      startingStyle?: boolean
-    },
+    options?: Nullable<{
+      mediaQuery?: Nullable<MediaQuery>
+      startingStyle?: Nullable<boolean>
+    }>,
   ) {
     let result =
       selector + (options?.startingStyle ? ' { @starting-style { }}' : ' { }')

package/src/utils/nodes.ts

@@ -38,7 +38,7 @@ export const getNodeAndAncestors = (
             nodeId: path.slice(0, i + 1).join('.'),
           })
         }
-        return component.nodes[node.children[childIndex]]
+        return component.nodes?.[node.children[childIndex]]
       default:
         return undefined
     }