Use weakRequire instead of ad-hoc evals

Author: lordofthelake

src/jsonUtils/nodeImpl/createStringMeasurer.ts

@@ -1,7 +1,8 @@
-import requireObjCBridge from './requireObjCBridge';
-
+import weakRequire from '../../utils/weakRequire';
 import { TextNode, Size } from '../../types';
 
+const bridge = weakRequire(module, 'node-sketch-bridge');
+
 export default function createStringMeasurer(textNodes: TextNode[], width: number): Size {
-  return requireObjCBridge().createStringMeasurer(textNodes, width);
+  return bridge.createStringMeasurer(textNodes, width);
 }

src/jsonUtils/nodeImpl/findFontName.ts

@@ -1,6 +1,8 @@
-import requireObjCBridge from './requireObjCBridge';
 import { TextStyle } from '../../types';
+import weakRequire from '../../utils/weakRequire';
+
+const bridge = weakRequire(module, 'node-sketch-bridge');
 
 export default function findFontName(style: TextStyle): string {
-  return requireObjCBridge().findFontName(style);
+  return bridge.findFontName(style);
 }

src/jsonUtils/nodeImpl/makeImageDataFromUrl.ts

@@ -6,7 +6,7 @@ import { createUniformBorder } from '../borders';
 import layerGroup from '../layerGroup';
 import { makePathsFromCommands, makeLineCapStyle } from './graphics/path';
 import { unionRects, scaleRect, makeBoundingRectFromCommands, resize } from './graphics/rect';
-import requireSvgModel from './requireSvgModel';
+import weakRequire from '../../utils/weakRequire';
 
 function makeLayerFromPathElement(pathElement, _parentFrame: FileFormat.Rect, scale: number) {
   const {
@@ -79,7 +79,7 @@ function makeLayerGroup(
 }
 
 export default function makeSvgLayer(layout: LayoutInfo, name: string, svg: string) {
-  const svgModel = requireSvgModel();
+  const svgModel = weakRequire(module, '@lona/svg-model');
 
   const {
     data: { params, children },

src/jsonUtils/nodeImpl/makeSvgLayer.ts

@@ -0,0 +1,9 @@
+/**
+ * Requires a module without having webpack including it in the bundle.
+ *
+ * @param sourceModule reference to the `module` instance from where this request originated.
+ * @param request name of the module to require.
+ */
+export default function weakRequire(sourceModule: NodeModule, request: string): any {
+  return 'require' in sourceModule ? sourceModule.require(request) : {};
+}