Anyone who to handle this PR?

@dougwilson @fengmk2

Is there any document talk about filed name can be a number?

@fengmk2 cnpm's model layer https://github.com/cnpm/cnpmjs.org/blob/master/models/download_total.js uses 'd1', 'd2'...

And I tried to use 1, 2...

It created successfully.

Sorry, apparently I never actually subscribed to this repository :) I don't see anything wrong with the concept, but I have a few comments on the change itself:

1. What if instead of stringifying only numbers, we just stringified anything that was not an array? Then it would let anything work.
2. The reassigning of an argument variable causes the function to deoptimize in v8. Please assign to a non-argument variable.

@dougwilson

For the first point, I don't think someone will expect escapeId(true) to be "true". But if you think it's necessary, I'm glad to stringify anything was not an array.

For the second one, I will commit later to declare a new variable.

For the first point, I don't think someone will expect escapeId(true) to be "true". But if you think it's necessary, I'm glad to stringify anything was not an array.

Sure, I tend to agree, but i.m.o that same argument would just be instead of even making this PR, just pass the number as a string in the first place... :)

@dougwilson e.g.

for (let i = 0; i < 10; i++) {
  escapeId(i);
}

Code above will crashed without this PR. But it's more convenient. Or the code should be like:

for (let i = 0; i < 10; i++) {
  escapeId(i.toString());
}