add no-redundant-path-segments rule Fixes import-js#471

src/index.js

@@ -30,6 +30,7 @@ export const rules = {
   'no-dynamic-require': require('./rules/no-dynamic-require'),
   'unambiguous': require('./rules/unambiguous'),
   'no-unassigned-import': require('./rules/no-unassigned-import'),
+  'no-redundant-path-segments': require('./rules/no-redundant-path-segments'),
 
   // metadata-based
   'no-deprecated': require('./rules/no-deprecated'),

src/rules/no-redundant-path-segments.js

@@ -0,0 +1,37 @@
+/**
+ * @fileOverview Ensures that there are no redundant path segments
+ * @author Thomas Grainger
+ */
+
+import path from 'path';
+import resolve from 'eslint-module-utils/resolve'
+import moduleVisitor from 'eslint-module-utils/moduleVisitor'
+
+module.exports = {
+  meta: {},
+
+  create: function (context) {
+    const currentPath = context.getFilename();
+
+    function checkSourceValue(source) {
+      const { value } = source;
+      const resolvedPath = resolve(source, context)
+      if (resolvedPath === undefined) {
+        return;
+      }
+
+      const expected = path.parse(path.relative(currentPath, resolvedPath));
+      const valueParsed = path.parse(value);
+
+      if (valueParsed.dir !== expected.dir) {
+        const proposed = path.format({ dir: expected.dir, name: valueParsed.name });
+        context.report({
+          node: source,
+          message: `Redundant paths for "${value}", should be ${proposed}`
+        })
+      }
+    }
+
+    return moduleVisitor(checkSourceValue, context.options[0])
+  },
+}