fix: ignore undefined values in "Globals.assign"

And categorize more globals as required

Author: aleclarson

packages/shared/src/globals.ts

@@ -1,4 +1,5 @@
 import { SpringInterpolator, InterpolatorConfig } from './types'
+import { ElementType } from 'react'
 
 declare const window: {
   requestAnimationFrame: (cb: (time: number) => void) => number
@@ -16,8 +17,12 @@ export interface AnimatedRef<T> {
 // Required
 //
 
+export let defaultElement: string | ElementType
+
 export let applyAnimatedValues: (node: any, props: Props) => boolean | void
 
+export let createAnimatedInterpolation: SpringInterpolator
+
 export let createStringInterpolator: (
   config: InterpolatorConfig<string>
 ) => (input: number) => string
@@ -35,28 +40,26 @@ export let frameLoop: {
 
 export let now = () => Date.now()
 
-export let colorNames: { [key: string]: number } | undefined
+export let colorNames: { [key: string]: number } | null = null as any
 
 export let skipAnimation = false
 
-export let defaultElement: any
-
-export let createAnimatedStyle: ((style: any) => any) | undefined
+export let createAnimatedStyle: ((style: any) => any) | null = null as any
 
-export let createAnimatedTransform: ((transform: any) => any) | undefined
-
-export let createAnimatedInterpolation: SpringInterpolator
+export let createAnimatedTransform:
+  | ((transform: any) => any)
+  | null = null as any
 
 export let createAnimatedRef: <T extends React.ElementType>(
   node: React.MutableRefObject<T>,
   mounted: React.MutableRefObject<boolean>,
   forceUpdate: () => void
 ) => T | AnimatedRef<T> = node => node.current
 
-export let requestAnimationFrame =
-  typeof window !== 'undefined' ? window.requestAnimationFrame : () => {}
+export let requestAnimationFrame: typeof window.requestAnimationFrame =
+  typeof window !== 'undefined' ? window.requestAnimationFrame : () => -1
 
-export let cancelAnimationFrame =
+export let cancelAnimationFrame: typeof window.cancelAnimationFrame =
   typeof window !== 'undefined' ? window.cancelAnimationFrame : () => {}
 
 //
@@ -110,5 +113,14 @@ export const assign = (globals: AnimatedGlobals): AnimatedGlobals =>
       requestAnimationFrame,
       cancelAnimationFrame,
     },
-    globals
+    pluckDefined(globals)
   ))
+
+// Ignore undefined values
+function pluckDefined(globals: any) {
+  const defined: any = {}
+  for (const key in globals) {
+    if (globals[key] !== void 0) defined[key] = globals[key]
+  }
+  return defined
+}