Bug: Method on numbers are not wrapped: `(1.2).toString()` becomes `1.2..toString()`

The following code:

(1.2).toString()

Should remain as-is, or be converted to 1.2.toString(), but instead is converted to the following:

1.2..toString();

Specifically, there is an extra period after the decimal number

Here is an AST representing this case:

{
  "type": "ExpressionStatement",
  "expression": {
	"type": "CallExpression",
	"callee": {
	  "type": "MemberExpression",
	  "object": {
		"type": "Literal",
		"value": 1.2,
		"raw": "1.2"
	  },
	  "property": {
		"type": "Identifier",
		"name": "toString"
	  },
	  "computed": false
	},
	"arguments": []
  }
}

You can try it out on the escodegen demo website

[agarand-godaddy]

Yea, I was trying it out on the demo website, which is still on version 0.0.16-dev for some reason

It does work as expected on the adequate version, my bad!

[papandreou]

The raw and extra values attached to the nodes are just extra information about the lexical tokens of the original souce. They don’t affect the semantics of the ast. I’m not the maintainer of this project, but I’m fairly sure that it’s a non-goal to do a perfect round-trip.