Rid off Ramda

Author: artptr

Diff for: package.json

@@ -25,8 +25,7 @@
     "prepublish": "npm run build"
   },
   "dependencies": {
-    "co": "^4.6.0",
-    "ramda": "^0.18.0"
+    "co": "^4.6.0"
   },
   "devDependencies": {
     "babel-cli": "^6.3.17",
@@ -39,7 +38,6 @@
     "co": "^4.6.0",
     "mocha": "*",
     "prop-types": "^15.6.0",
-    "ramda": "^0.18.0",
     "react": "^16.1.1",
     "react-redux": "^4.0.0",
     "redux": "^3.0.0",

Diff for: src/index.js

@@ -1,8 +1,8 @@
 import PropTypes from 'prop-types';
-import R from 'ramda';
 import React from 'react';
 import co from 'co';
 import { aggregateSelectors } from './selector_utils';
+import { mapObj, pick, merge, mergeAll } from './utils';
 import { connect } from 'react-redux';
 
 const toDispatchSymbol = Symbol('toDispatch');
@@ -103,23 +103,23 @@ export function runControllerGenerator(propsGetter) {
  */
 export function controller(RootComponent, controllerGenerators, selectorBundles, controllerGeneratorRunner = runControllerGenerator) {
   // Combine selector bundles into one mapStateToProps function.
-  const mapStateToProps = aggregateSelectors(R.mergeAll(R.flatten([selectorBundles])));
+  const mapStateToProps = aggregateSelectors(mergeAll(selectorBundles));
   const selectorPropTypes = mapStateToProps.propTypes;
 
   // All the controller method propTypes should simply be "function" so we can
   // synthensize those.
-  const controllerMethodPropTypes = R.map(() => PropTypes.func.isRequired, controllerGenerators);
+  const controllerMethodPropTypes = mapObj(() => PropTypes.func.isRequired, controllerGenerators);
 
   // Declare the availability of all of the selectors and controller methods
   // in the React context for descendant components.
-  const contextPropTypes = R.merge(selectorPropTypes, controllerMethodPropTypes);
+  const contextPropTypes = merge(selectorPropTypes, controllerMethodPropTypes);
 
   class Controller extends React.Component {
     constructor(...constructorArgs) {
       super(...constructorArgs);
 
       const injectedControllerGeneratorRunner = controllerGeneratorRunner(() => this.props);
-      this.controllerMethods = R.map(controllerGenerator =>
+      this.controllerMethods = mapObj(controllerGenerator =>
         injectedControllerGeneratorRunner(controllerGenerator)
       , controllerGenerators);
 
@@ -136,8 +136,8 @@ export function controller(RootComponent, controllerGenerators, selectorBundles,
     getChildContext() {
       // Rather than injecting all of the RootComponent props into the context,
       // we only explictly pass selector and controller method props.
-      const selectorProps = R.pick(R.keys(selectorPropTypes), this.props);
-      const childContext = R.merge(selectorProps, this.controllerMethods);
+      const selectorProps = pick(Object.keys(selectorPropTypes), this.props);
+      const childContext = merge(selectorProps, this.controllerMethods);
       return childContext;
     }
 
@@ -148,7 +148,7 @@ export function controller(RootComponent, controllerGenerators, selectorBundles,
     }
   }
 
-  Controller.propTypes = R.merge(selectorPropTypes, RootComponent.propTypes || {});
+  Controller.propTypes = merge(selectorPropTypes, RootComponent.propTypes || {});
   Controller.childContextTypes = contextPropTypes;
 
   return connect(mapStateToProps)(Controller);

Diff for: src/selector_utils.js

@@ -1,4 +1,4 @@
-import R from 'ramda';
+import { mapObj } from './utils';
 
 /**
  * Combines bundle of selector functions into a single super selector function
@@ -44,8 +44,8 @@ import R from 'ramda';
  *   of the selector outputs.
  */
 export function aggregateSelectors(bundle) {
-  const combinedSelector = state => R.map(selectorFunction => selectorFunction(state), bundle);
-  combinedSelector.propTypes = R.map(selectorFunction => selectorFunction.propType, bundle);
+  const combinedSelector = state => mapObj(selectorFunction => selectorFunction(state), bundle);
+  combinedSelector.propTypes = mapObj(selectorFunction => selectorFunction.propType, bundle);
   return combinedSelector;
 }
 
@@ -57,8 +57,8 @@ export function aggregateSelectors(bundle) {
  *   annotated with a propType property.
  */
 export function disaggregateSuperSelector(superSelector) {
-  return R.mapObjIndexed((propType, propName) => {
-    const singleSelector = R.pipe(superSelector, R.prop(propName));
+  return mapObj((propType, propName) => {
+    const singleSelector = store => superSelector(store)[propName];
     singleSelector.propType = propType;
     return singleSelector;
   }, superSelector.propTypes);

Diff for: src/utils.js

@@ -0,0 +1,81 @@
+/**
+ * Map, but for objects. Creates an object with the same keys as `obj` and values
+ * generated by running each property of `obj` through `fn`. `fn` is passed three arguments: *(value, key, obj)*.
+ * @param {Function} fn A function called for each property in `obj`. Its return value will
+ *        become a new property on the return object.
+ * @param {Object} obj The object to iterate over.
+ * @return {Object} A new object with the same keys as `obj` and values that are the result
+ *         of running each property through `fn`.
+ */
+export function mapObj(fn, obj) {
+  const result = {};
+
+  const keys = Object.keys(obj);
+
+  for (let i = 0; i < keys.length; ++i) {
+    result[keys[i]] = fn(obj[keys[i]], keys[i], obj);
+  }
+
+  return result;
+}
+
+/**
+ * Returns a partial copy of an object containing only the keys specified.  If the key does not exist, the
+ * property is ignored.
+ * @param {String[]} keys An array of String property names to copy onto a new object.
+ * @param {Object} obj The object to copy from.
+ * @return {Object} A new object with only properties from `keys` on it.
+ */
+export function pick(keys, obj) {
+  const result = {};
+
+  for (let i = 0; i < keys.length; ++i) {
+    if (keys[i] in obj) {
+      result[keys[i]] = obj[keys[i]];
+    }
+  }
+
+  return result;
+}
+
+/**
+ * Create a new object with the own properties of `a`
+ * merged with the own properties of object `b`.
+ * @param {Object} a The first object.
+ * @param {Object} b The second object.
+ * @return {Object} A merged object.
+ */
+export function merge(a, b) {
+  const result = {};
+
+  for (let i = 0, keys = Object.keys(a); i < keys.length; ++i) {
+    result[keys[i]] = a[keys[i]];
+  }
+
+  for (let i = 0, keys = Object.keys(b); i < keys.length; ++i) {
+    result[keys[i]] = b[keys[i]];
+  }
+
+  return result;
+}
+
+/**
+ * Merges a list of objects together into one object.
+ * @param {Object|Object[]} objs An array of objects or a single object.
+ * @return {Object} A merged object.
+ */
+export function mergeAll(objs) {
+  const list = Array.isArray(objs) ? objs : [objs];
+
+  const result = {};
+
+  for (let i = 0; i < list.length; ++i) {
+    const keys = Object.keys(list[i]);
+
+    for (let j = 0; j < keys.length; ++j) {
+      result[keys[j]] = list[i][keys[j]];
+    }
+  }
+
+  return result;
+}

Diff for: yarn.lock

@@ -1760,10 +1760,6 @@ [email protected]:
   version "0.2.0"
   resolved "https://registry.yarnpkg.com/querystring/-/querystring-0.2.0.tgz#b209849203bb25df820da756e747005878521620"
 
-ramda@^0.18.0:
-  version "0.18.0"
-  resolved "https://registry.yarnpkg.com/ramda/-/ramda-0.18.0.tgz#c6e3c5d4b9ab1f7906727fdeeb039152a85d4db3"
-
 randomatic@^1.1.3:
   version "1.1.7"
   resolved "https://registry.yarnpkg.com/randomatic/-/randomatic-1.1.7.tgz#c7abe9cc8b87c0baa876b19fde83fd464797e38c"