This works as expected in a fresh empty project, but when trying to use it from an existing project I get an error on the require() line

const SortedArray = require('sorted-array');

Uncaught TypeError: this.insert is not a function
at Object.constructor (sorted-array.js?c258:9)

In Chrome I can see that when we reach line 9 'this' exists but only has 'array' and 'compare' in it, not 'insert'. (Its proto doesn't have 'insert' either).

I haven't been able to figure out what's happening. Why is require() executing a constructor (seems to happen at "module.exports = SortedArray" on line 74)? Is there a webpack/Babel setting I'm missing? Some new Javascript keyword?

This doesn't work when using a compare function? E.g.

var SortedArray = require("sorted-array");
var sorted = new SortedArray([{a:112}, {a: 99}, {a:4}], (a, b) => a.a - a.a);
console.log(sorted.array); // [ { a: 112 }, { a: 4 }, { a: 99 } ]

sorted-array.js line 6 states that var length = arguments.length;. However, to work as it does in the example, it needs to be array.length. Otherwise, it will only copy the first element into the internal array (or the first two if a compare function is supplied).

Is there any benefit to using this.search(element) instead of this.array.indexOf(element)?

If this.compare uses a nested property of element, there could be false positives.

I assumed remove() would simply remove element from this.array.

remove: function (element) {
  var index = this.array.indexOf(element);
  if (index >= 0) this.array.splice(index, 1);
  return this;
}

Oh, and thanks for making this library! 🍻