Update: tryExtensions option is added (fixes #14)

And more compatibility to check published.

Author: mysticatea

docs/rules/no-missing-import.md

@@ -27,6 +27,41 @@ import existingFile from "./existing-file";
 import existingModule from "existing-module";
 ```
 
+### Options
+
+```json
+{
+    "rules": {
+        "node/no-missing-import": [2, {
+            "tryExtensions": [".js", ".json", ".node"]
+        }]
+    }
+}
+```
+
+#### `tryExtensions`
+
+When an import path does not exist, this rule checks whether or not any of `path.js`, `path.json`, and `path.node` exists.
+`tryExtensions` option is the extension list this rule uses at the time.
+
+Default is `[".js", ".json", ".node"]`.
+
+This option can be set by [shared settings](http://eslint.org/docs/user-guide/configuring.html#adding-shared-settings).
+Several rules have this option, but we can set this option at once.
+
+```json
+{
+    "settings": {
+        "node": {
+            "tryExtensions": [".js", ".json", ".node"]
+        }
+    },
+    "rules": {
+        "node/no-missing-import": 2
+    }
+}
+```
+
 ## When Not To Use It
 
 This rule should not be used in ES3/5 environments.

docs/rules/no-missing-require.md

@@ -33,6 +33,41 @@ var existingModule = require("existing-module");
 var foo = require(FOO_NAME);
 ```
 
+### Options
+
+```json
+{
+    "rules": {
+        "node/no-missing-require": [2, {
+            "tryExtensions": [".js", ".json", ".node"]
+        }]
+    }
+}
+```
+
+#### `tryExtensions`
+
+When an import path does not exist, this rule checks whether or not any of `path.js`, `path.json`, and `path.node` exists.
+`tryExtensions` option is the extension list this rule uses at the time.
+
+Default is `[".js", ".json", ".node"]`.
+
+This option can be set by [shared settings](http://eslint.org/docs/user-guide/configuring.html#adding-shared-settings).
+Several rules have this option, but we can set this option at once.
+
+```json
+{
+    "settings": {
+        "node": {
+            "tryExtensions": [".js", ".json", ".node"]
+        }
+    },
+    "rules": {
+        "node/no-missing-require": 2
+    }
+}
+```
+
 ## When Not To Use It
 
 If you don't want to be notified about usage of `require()`, then it's safe to disable this rule.

docs/rules/no-unpublished-import.md

@@ -36,6 +36,41 @@ import publishedFile from "./published-file";
 import dependedModule from "depended-module";
 ```
 
+### Options
+
+```json
+{
+    "rules": {
+        "node/no-missing-import": [2, {
+            "tryExtensions": [".js", ".json", ".node"]
+        }]
+    }
+}
+```
+
+#### `tryExtensions`
+
+When an import path does not exist, this rule checks whether or not any of `path.js`, `path.json`, and `path.node` exists.
+`tryExtensions` option is the extension list this rule uses at the time.
+
+Default is `[".js", ".json", ".node"]`.
+
+This option can be set by [shared settings](http://eslint.org/docs/user-guide/configuring.html#adding-shared-settings).
+Several rules have this option, but we can set this option at once.
+
+```json
+{
+    "settings": {
+        "node": {
+            "tryExtensions": [".js", ".json", ".node"]
+        }
+    },
+    "rules": {
+        "node/no-missing-import": 2
+    }
+}
+```
+
 ## When Not To Use It
 
 This rule should not be used in ES3/5 environments.

docs/rules/no-unpublished-require.md

@@ -38,6 +38,41 @@ var dependedModule = require("depended-module");
 var foo = require(FOO_NAME);
 ```
 
+### Options
+
+```json
+{
+    "rules": {
+        "node/no-missing-import": [2, {
+            "tryExtensions": [".js", ".json", ".node"]
+        }]
+    }
+}
+```
+
+#### `tryExtensions`
+
+When an import path does not exist, this rule checks whether or not any of `path.js`, `path.json`, and `path.node` exists.
+`tryExtensions` option is the extension list this rule uses at the time.
+
+Default is `[".js", ".json", ".node"]`.
+
+This option can be set by [shared settings](http://eslint.org/docs/user-guide/configuring.html#adding-shared-settings).
+Several rules have this option, but we can set this option at once.
+
+```json
+{
+    "settings": {
+        "node": {
+            "tryExtensions": [".js", ".json", ".node"]
+        }
+    },
+    "rules": {
+        "node/no-missing-import": 2
+    }
+}
+```
+
 ## When Not To Use It
 
 If you don't want to be notified about usage of `require()`, then it's safe to disable this rule.

lib/rules/no-missing-import.js

@@ -34,4 +34,19 @@ module.exports = function(context) {
     };
 };
 
-module.exports.schema = [];
+module.exports.schema = [
+    {
+        type: "object",
+        properties: {
+            tryExtensions: {
+                type: "array",
+                items: {
+                    type: "string",
+                    pattern: "^\\."
+                },
+                uniqueItems: true
+            }
+        },
+        additionalProperties: false
+    }
+];

lib/rules/no-missing-require.js

@@ -34,4 +34,19 @@ module.exports = function(context) {
     };
 };
 
-module.exports.schema = [];
+module.exports.schema = [
+    {
+        type: "object",
+        properties: {
+            tryExtensions: {
+                type: "array",
+                items: {
+                    type: "string",
+                    pattern: "^\\."
+                },
+                uniqueItems: true
+            }
+        },
+        additionalProperties: false
+    }
+];

lib/rules/no-unpublished-import.js

@@ -34,4 +34,19 @@ module.exports = function(context) {
     };
 };
 
-module.exports.schema = [];
+module.exports.schema = [
+    {
+        type: "object",
+        properties: {
+            tryExtensions: {
+                type: "array",
+                items: {
+                    type: "string",
+                    pattern: "^\\."
+                },
+                uniqueItems: true
+            }
+        },
+        additionalProperties: false
+    }
+];

lib/rules/no-unpublished-require.js

@@ -34,4 +34,19 @@ module.exports = function(context) {
     };
 };
 
-module.exports.schema = [];
+module.exports.schema = [
+    {
+        type: "object",
+        properties: {
+            tryExtensions: {
+                type: "array",
+                items: {
+                    type: "string",
+                    pattern: "^\\."
+                },
+                uniqueItems: true
+            }
+        },
+        additionalProperties: false
+    }
+];

lib/util/check-existence.js

@@ -38,10 +38,8 @@ module.exports = function checkForExistence(context, filePath, targets) {
         var target = targets[i];
 
         // Workaround for https://github.com/substack/node-resolve/issues/78
-        if (target.relative) {
-            var ext = path.extname(target.name);
-            var name = (ext === ".js") ? target.name : target.name + ".js";
-            if (exists(path.resolve(basedir, name))) {
+        if (target.filePath) {
+            if (exists(target.filePath)) {
                 continue;
             }
         } else {

lib/util/check-publish.js

@@ -13,7 +13,6 @@
 
 var path = require("path");
 var assign = require("object-assign");
-var exists = require("./exists");
 var getNpmignore = require("./get-npmignore");
 var getPackageJson = require("./get-package-json");
 
@@ -39,11 +38,7 @@ module.exports = function checkForPublish(context, filePath, targets) {
 
     var basedir = path.dirname(packageInfo.filePath);
     var toRelative = function(fullPath) { // eslint-disable-line func-style
-        var ext = path.extname(fullPath);
         var retv = path.relative(basedir, fullPath).replace(/\\/g, "/");
-        if (!ext && exists(fullPath + ".js")) {
-            retv += ".js";
-        }
         return retv;
     };
     var npmignore = getNpmignore(filePath);

lib/util/get-import-export-targets.js

@@ -12,6 +12,7 @@
 
 var path = require("path");
 var resolve = require("resolve");
+var getTryExtensions = require("./get-try-extensions");
 var ImportTarget = require("./import-target");
 
 //------------------------------------------------------------------------------
@@ -37,6 +38,7 @@ module.exports = function getImportExportTargets(context, programNode) {
     var retv = [];
     var basedir = path.dirname(path.resolve(context.getFilename()));
     var statements = programNode.body;
+    var extensions = getTryExtensions(context);
 
     for (var i = 0; i < statements.length; ++i) {
         var statement = statements[i];
@@ -47,9 +49,10 @@ module.exports = function getImportExportTargets(context, programNode) {
         }
 
         // Gets the target module.
-        var name = statement.source && statement.source.value;
+        var node = statement.source;
+        var name = node && node.value;
         if (name && !resolve.isCore(name)) {
-            retv.push(new ImportTarget(statement.source, name, basedir));
+            retv.push(new ImportTarget(node, name, basedir, extensions));
         }
     }