The thing is, that test is depending on approximateSize. You cannot really make it run without approximateSize.
The big question is why it was not passing in the first place :(.

So, we need:

• fix these two tests due to level.js discrepancy on del.
• implement a good approximateSize based on variablesMask.

The failing tests are:
https://github.com/mcollina/node-levelgraph/blob/browser/test/triple_store_spec.js#L176-L204
https://github.com/mcollina/node-levelgraph/blob/browser/test/triple_store_spec.js#L126-L133

Ok, it is now completely working:

# tests 76 
# pass 76
# fail 0 

What is also missing is:

• a complete in-browser example.

@jez0990 could you please mess with the Makefile to write an automated build of the package?
Maybe something that also produces a minified version and a minified+gzipped one.

Ok, it just remains the README to be fixed!

I implemented the logic for building a working levelgraph bundle in the Makefile.
Everything is included, is 60K minified and gzipped.

However I do not like the end-user API:

var Leveljs = require("level-js");
var levelup = require("levelup");
var levelgraph = require("levelgraph");

db = levelgraph(levelup("test", { db: function(l) { return new Leveljs(l); } }));

I'll prefer to have a static bundle that do not expose require.

The whole LevelGraph with all dependencies is 60KB minified and gzipped. It's not that bad :).

This is great :) it sucks that I've not had any time to look at this over the last week though!

I didn't know about Browserify's external requires until reading your commit just now but I guess they don't lend themselves to an elegant module interface. How about just exporting a function in the provided build with the default options so you can do

var db levelgraph("test")

And advanced users can follow the Make step you wrote if you only put it in the readme...?

This would need a file levelgraphjs.js:

var Leveljs = require("level-js");
var levelup = require("levelup");
var levelgraph = require("./index.js");

module.exports = function (name) {
  return levelgraph(levelup(name, { db: function(l) { return new Leveljs(l); } }));
}

And the new Make step:

./node_modules/.bin/browserify ./levelgraphjs.js:levelgraphjs > build/levelgraph.js

Does that sound sensible?

Ha :)

ahahhaha we had the same idea mostly at the same time.
I am trying to merge this back inside LevelGraph standard API, with LevelDown instead of Level-js.

Looks great! May I delete the branch?

Done :)