@fiverr/futile - npm Package Compare versions

Comparing version 1.8.0 to 1.9.0

lib/capitalize/test.js

 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -4,0 +4,0 @@ const capitalize = require('./');

lib/deepassign/benchmark.js

@@ -1,4 +0,2 @@
-'use strict';
-
-const BENCHMARK_COUNTER = 100000;
+const BENCHMARK_COUNTER = 1e5;
 const humanNumber = (num) => num.toString().replace(/(\d)(?=(\d{3})+$)/g, '$1,');
@@ -5,0 +3,0 @@ const benchmark = (method, count = BENCHMARK_COUNTER) => {

lib/flow/test.js

@@ -5,3 +5,3 @@ 'use strict';
 const assert = require('chai').assert;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -8,0 +8,0 @@ const flow = require('./');

lib/index.js

@@ -14,2 +14,3 @@ /**
  * @requires lib/interpolate
+ * @requires lib/interpolator
  * @requires lib/pluck
@@ -31,2 +32,3 @@ * @requires lib/resolve
     interpolate: require('./interpolate'),
+    interpolator: require('./interpolator'),
     pluck: require('./pluck'),
@@ -33,0 +35,0 @@ resolve: require('./resolve'),

lib/interpolate/index.js

 /**
  * @module lib/interpolate
  * @since 1.0.0
+ * @requires lib/interpolator
  */
 
-/**
- * @access private
- * @member {Array} Valid_result_types string, number
- */
-const VALID_RESULT_TYPES = Object.seal(['string', 'number']);
+const interpolator = require('../interpolator');
 
@@ -20,3 +17,9 @@ /**
  *
+ * @deprecated will be removed in version 2.0.0. Create an instance of interpolator instead
+ *
  * @example
+ * // After deprecation use lib/interpolator constructor instead
+ * const interpolate = interpolator(/\${([^{}]*)}/g, /%{([^{}]*)}/g, /{{([^{}]*)}}/g);
+ *
+ * @example
  * // JS flavour:
@@ -34,21 +37,6 @@ * interpolate('Hello, ${name}', {name: 'Martin'});
  */
-module.exports = (string = '', data) => {
-    if (typeof string !== 'string') {
-        throw new TypeError(`futile#interpolate expects first argument to be a string, got a ${typeof string}`);
-    }
-
-    if (!data) {
-        return string;
-    }
-
-    function replace(haystack, needle) {
-        const replacement = data[needle.trim()];
-        return ~VALID_RESULT_TYPES.indexOf(typeof replacement) ? replacement : haystack;
-    }
-
-    return string
-        .replace(/\${([^{}]*)}/g, replace)
-        .replace(/%{([^{}]*)}/g, replace)
-        .replace(/{{([^{}]*)}}/g, replace)
-        ;
-};
+module.exports = interpolator(
+    /\${([^{}]*)}/g,
+    /%{([^{}]*)}/g,
+    /{{([^{}]*)}}/g
+);

lib/interpolate/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -15,8 +15,2 @@ const interpolate = require('./');
 
-    it('interpolate is a function', () => interpolate.should.be.a('function'));
-
-    it('Interpolate can process string', () => expect(interpolate.bind(null, '')).to.not.throw(TypeError));
-
-    it('Interpolate can not process non string', () => expect(interpolate.bind(null, {})).to.throw(TypeError));
-
     it('Interpolate string with \'$\' notation (ES)', () => {
@@ -23,0 +17,0 @@ const res = interpolate('Hello, ${name}', {name}); // eslint-disable-line no-template-curly-in-string

lib/pluck/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const pluck = require('./');

lib/resolve/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const resolve = require('./');

lib/safe/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const safe = require('./');

lib/sequence/test.js

@@ -5,3 +5,3 @@ 'use strict';
 const assert = require('chai').assert;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -8,0 +8,0 @@ const sequence = require('./');

lib/shuffle/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const shuffle = require('./');

lib/uri/analyze/test.js

 'use strict';
 
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -5,0 +5,0 @@ const analyze = require('./');

lib/uri/benchmark.js

@@ -1,4 +0,2 @@
-'use strict';
-
-const BENCHMARK_COUNTER = 100000;
+const BENCHMARK_COUNTER = 1e5;
 const humanNumber = (num) => num.toString().replace(/(\d)(?=(\d{3})+$)/g, '$1,');
@@ -5,0 +3,0 @@ const benchmark = (method, count = BENCHMARK_COUNTER) => {

lib/uri/break-pair/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const breakPair = require('./');

lib/uri/index.js

@@ -78,8 +78,8 @@ /**
         }]
-        .concat(
-            props.map(
-                (prop) => ({[prop]: self.store[prop]})
+            .concat(
+                props.map(
+                    (prop) => ({[prop]: self.store[prop]})
+                )
             )
-        )
-    ));
+        ));
 
@@ -86,0 +86,0 @@ return self;

lib/uri/query-string-to-obj/test.js

 'use strict';
 
 const assert = require('chai').assert;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const queryStringToObj = require('./');

lib/uri/test.js

 'use strict';
 
 const expect = require('chai').expect;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const URI = require('./');

lib/uri/to-query-string/test.js

 'use strict';
 
 const assert = require('chai').assert;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
+require('chai').should();
 
@@ -6,0 +6,0 @@ const toQueryString = require('./');

lib/uri/valid-value/test.js

@@ -1,6 +0,3 @@
-'use strict';
-
 const expect = require('chai').expect;
 const assert = require('chai').assert;
-const should = require('chai').should(); // eslint-disable-line no-unused-vars
 
@@ -7,0 +4,0 @@ const validValue = require('./');

package.json

 {
   "name": "@fiverr/futile",
-  "version": "1.8.0",
+  "version": "1.9.0",
   "author": "Fiverr",
@@ -30,4 +30,4 @@ "license": "MIT",
     "docdash": "^0.4.0",
-    "eslint": "^3.19.0",
-    "eslint-plugin-react": "^6.10.3",
+    "eslint": "^4.0.0",
+    "eslint-plugin-react": "^7.1.0",
     "import-fresh": "^2.0.0",
@@ -38,5 +38,5 @@ "jsdoc": "^3.4.3",
     "mocha": "^3.4.2",
-    "nyc": "^10.3.2"
+    "nyc": "^11.0.2"
   },
   "dependencies": {}
 }

New alerts

Uses eval

Supply chain risk

Package uses dynamic code execution (e.g., eval()), which is a dangerous practice. This can prevent the code from running in certain environments and increases the risk that the code may contain exploits or malicious behavior.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

91006

increased by5.16%

Number of package files

54

increased by5.88%

Lines of code

2380

increased by6.92%

Worsened metrics

Number of medium supply chain risk alerts

2

increased byInfinity%

No dependency changes