Can't use number as node name

I did the following in my code and expected to get a path [1, 2]. But it returned null.

route.addNode(1, { 2: 10, 3: 20 }); 
console.log(route.path(1, 2));

Turned out it doesn’t like me using numbers as the name of a node. It would be good if the library checks and raises an error, or make it work.

it('adds a node 2', () => {
      const route = new Graph();
      route.addNode(1, { 2: 10, 3: 20 }); 
      const node = route.graph.get(1);
      demand(node).be.instanceOf(Map);
      demand(node.get('2')).equal(10); //<--- this is fine
      demand(node.get(3)).equal(20); // <--- this fails
    }); 

[albertorestifo]

@kronick Good point.

That’s can be the sweet spot. Have the ability to pass Map to all the methods.

[kronick]

I ran into the same issue and came here to report it. This is a great library and clearing up this ambiguity would make it even better!

My nodes don’t have natural names so using numbers for the keys makes the most sense. Why not use a ES6 Map throughout and allow arbitrary types for all keys?