Add namespaces options for spec compliancy and better linting support.

Fixes #27.

Author: RReverser

README.md

@@ -37,6 +37,28 @@ var ast = acorn.parse(code, {
 });
 ```
 
+Note that official spec doesn't support mix of XML namespaces and object-style access in tag names (#27) like in `<namespace:Object.Property />`, so it was deprecated in `[email protected]`. If you still want to opt-in to support of such constructions, you can pass the following option:
+
+```javascript
+var ast = acorn.parse(code, {
+  plugins: {
+    jsx: { allowNamespacedObjects: true }
+  }
+});
+```
+
+Also, since most apps use pure React transformer, a new option was introduced that allows to prohibit namespaces completely:
+
+```javascript
+var ast = acorn.parse(code, {
+  plugins: {
+    jsx: { allowNamespaces: false }
+  }
+});
+```
+
+Note that by default `allowNamespaces` is enabled for spec compliancy.
+
 ## License
 
 This plugin is issued under the [MIT license](./LICENSE).

inject.js

@@ -198,7 +198,7 @@ module.exports = function(acorn) {
   pp.jsx_parseNamespacedName = function() {
     var startPos = this.start, startLoc = this.startLoc;
     var name = this.jsx_parseIdentifier();
-    if (!this.eat(tt.colon)) return name;
+    if (!this.options.plugins.jsx.allowNamespaces || !this.eat(tt.colon)) return name;
     var node = this.startNodeAt(startPos, startLoc);
     node.namespace = name;
     node.name = this.jsx_parseIdentifier();
@@ -211,6 +211,9 @@ module.exports = function(acorn) {
   pp.jsx_parseElementName = function() {
     var startPos = this.start, startLoc = this.startLoc;
     var node = this.jsx_parseNamespacedName();
+    if (this.type === tt.dot && node.type === 'JSXNamespacedName' && !this.options.plugins.jsx.allowNamespacedObjects) {
+      this.unexpected();
+    }
     while (this.eat(tt.dot)) {
       var newNode = this.startNodeAt(startPos, startLoc);
       newNode.object = node;
@@ -356,7 +359,20 @@ module.exports = function(acorn) {
     return this.jsx_parseElementAt(startPos, startLoc);
   };
 
-  acorn.plugins.jsx = function(instance) {
+  acorn.plugins.jsx = function(instance, opts) {
+    if (!opts) {
+      return;
+    }
+
+    if (typeof opts !== 'object') {
+      opts = {};
+    }
+
+    instance.options.plugins.jsx = {
+      allowNamespaces: opts.allowNamespaces !== false,
+      allowNamespacedObjects: !!opts.allowNamespacedObjects
+    };
+
     instance.extend('parseExprAtom', function(inner) {
       return function(refShortHandDefaultPos) {
         if (this.type === tt.jsxText)

package.json

@@ -2,7 +2,7 @@
   "name": "acorn-jsx",
   "description": "Alternative, faster React.js JSX parser",
   "homepage": "https://github.com/RReverser/acorn-jsx",
-  "version": "2.0.1",
+  "version": "3.0.0",
   "maintainers": [
     {
       "name": "Ingvar Stepanyan",

test/driver.js

@@ -25,7 +25,7 @@ exports.runTests = function(config, callback) {
       if (expected.onToken = testOpts.onToken) {
         testOpts.onToken = [];
       }
-      testOpts.plugins = { jsx: true };
+      testOpts.plugins = testOpts.plugins || { jsx: true };
       var ast = parse(test.code, testOpts);
       if (test.error) {
         if (config.loose) {

test/tests-jsx.js

@@ -3639,6 +3639,69 @@ if (typeof exports !== "undefined") {
 
 testFail("var x = <div>one</div><div>two</div>;", "Adjacent JSX elements must be wrapped in an enclosing tag (1:22)");
 
+testFail("<a:b.c />", "Unexpected token (1:4)");
+
+test("<a:b.c />", {
+  type: "Program",
+  range: [0, 9],
+  body: [{
+    type: "ExpressionStatement",
+    range: [0, 9],
+    expression: {
+      type: "JSXElement",
+      range: [0, 9],
+      openingElement: {
+        type: "JSXOpeningElement",
+        range: [0, 9],
+        attributes: [],
+        name: {
+          type: "JSXMemberExpression",
+          range: [1, 6],
+          object: {
+            type: "JSXNamespacedName",
+            range: [1, 4],
+            namespace: {
+              type: "JSXIdentifier",
+              range: [1, 2],
+              name: "a"
+            },
+            name: {
+              type: "JSXIdentifier",
+              range: [3, 4],
+              name: "b"
+            }
+          },
+          property: {
+            type: "JSXIdentifier",
+            range: [5, 6],
+            name: "c"
+          }
+        },
+        selfClosing: true
+      },
+      closingElement: null,
+      children: []
+    }
+  }]
+}, {
+  ranges: true,
+  plugins: {
+    jsx: { allowNamespacedObjects: true }
+  }
+});
+
+testFail('<ns:div />', 'Unexpected token (1:3)', {
+  plugins: {
+    jsx: { allowNamespaces: false }
+  }
+});
+
+testFail('<div ns:attr />', 'Unexpected token (1:7)', {
+  plugins: {
+    jsx: { allowNamespaces: false }
+  }
+});
+
 test('<a>{/* foo */}</a>', {}, {
   onToken: [
     {