fix(shared): resolve circular dependencies

`shared/helpers` wants entire `shared/globals` inside `isAnimatedString`, `shared/globals` wants `noop` from `shared/globals` to assign exported variable.

If `shared/helpers` is executed before `shared/globals`, `willAdvance` is initialized by given value. Otherwise `willAdvance` is initialized after `noop` inside `shared/helpers` and have no defined value.

The safest way is keep `shared/helpers` pure and move `isAnimatedString` to own module.

Author: khmm12

packages/shared/src/helpers.ts

@@ -1,5 +1,4 @@
 import { Lookup, Arrify, AnyFn } from '@react-spring/types'
-import * as G from './globals'
 
 export const noop = () => {}
 
@@ -34,11 +33,6 @@ export function isEqual(a: any, b: any) {
   return a === b
 }
 
-// Not all strings can be animated (eg: {display: "none"})
-export const isAnimatedString = (value: unknown): value is string =>
-  is.str(value) &&
-  (value[0] == '#' || /\d/.test(value) || !!(G.colors && G.colors[value]))
-
 type Eachable<Value, Key> = {
   forEach: (cb: (value: Value, key: Key) => void, ctx?: any) => void
 }

packages/shared/src/index.ts

@@ -15,5 +15,6 @@ export * from './createInterpolator'
 export * from './stringInterpolation'
 export * from './deprecations'
 export * from './helpers'
+export * from './isAnimatedString'
 
 export * from 'fluids'

packages/shared/src/isAnimatedString.ts

@@ -0,0 +1,10 @@
+import * as G from './globals'
+import { is } from './helpers'
+
+// Not all strings can be animated (eg: {display: "none"})
+export function isAnimatedString(value: unknown): value is string {
+  return (
+    is.str(value) &&
+    (value[0] == '#' || /\d/.test(value) || !!(G.colors && G.colors[value]))
+  )
+}