fix(#5): convert all kinds of file path to posix style

Author: dukeluo

lib/rules/filename-naming-convention.js

@@ -8,6 +8,7 @@ const {
   getFilename,
   getBasename,
   getPathFromRepositoryRoot,
+  toPosixPath,
 } = require('../utils/filename');
 const {
   checkSettings,
@@ -70,8 +71,8 @@ module.exports = {
         }
 
         const pathFromRepositoryRoot = getPathFromRepositoryRoot(
-          context.getPhysicalFilename(),
-          context.getCwd()
+          toPosixPath(context.getPhysicalFilename()),
+          toPosixPath(context.getCwd())
         );
         const filename = getFilename(pathFromRepositoryRoot);
 

lib/rules/folder-match-with-fex.js

@@ -4,7 +4,11 @@
  */
 'use strict';
 
-const { getFolderPath, getFilename } = require('../utils/filename');
+const {
+  getFolderPath,
+  getFilename,
+  toPosixPath,
+} = require('../utils/filename');
 const { checkSettings, globPatternValidator } = require('../utils/settings');
 const { getDocUrl } = require('../utils/doc');
 const { matchRule } = require('../utils/match');
@@ -55,7 +59,7 @@ module.exports = {
           return;
         }
 
-        const filenameWithPath = context.getPhysicalFilename();
+        const filenameWithPath = toPosixPath(context.getPhysicalFilename());
         const filename = getFilename(filenameWithPath);
         const folderPath = getFolderPath(filenameWithPath);
 

lib/rules/folder-naming-convention.js

@@ -9,6 +9,7 @@ const {
   getFolderPath,
   getSubPaths,
   getAllFolders,
+  toPosixPath,
 } = require('../utils/filename');
 const {
   checkSettings,
@@ -62,7 +63,7 @@ module.exports = {
           return;
         }
 
-        const filenameWithPath = context.getPhysicalFilename();
+        const filenameWithPath = toPosixPath(context.getPhysicalFilename());
         const folderPath = getFolderPath(filenameWithPath);
         const subPaths = getSubPaths(folderPath);
 

lib/rules/no-index.js

@@ -5,7 +5,7 @@
 'use strict';
 
 const { getDocUrl } = require('../utils/doc');
-const { getFilename, getBasename } = require('../utils/filename');
+const { getFilename, getBasename, toPosixPath } = require('../utils/filename');
 
 /**
  * @type {import('eslint').Rule.RuleModule}
@@ -26,7 +26,7 @@ module.exports = {
   create(context) {
     return {
       Program: (node) => {
-        const filenameWithPath = context.getPhysicalFilename();
+        const filenameWithPath = toPosixPath(context.getPhysicalFilename());
         const filename = getFilename(filenameWithPath);
         const basename = getBasename(filename);
 

lib/utils/filename.js

@@ -8,15 +8,15 @@ const path = require('path');
 
 /**
  * @type {string} filename without path
- * @param {string} p filename concat with path
+ * @param {string} p filename concat with path in posix style
  */
-const getFilename = (p) => path.basename(p);
+const getFilename = (p) => path.posix.basename(p);
 
 /**
  * @type {string} path of folder
- * @param {string} p filename concat with path
+ * @param {string} p filename concat with path in posix style
  */
-const getFolderPath = (p) => path.join(path.dirname(p), path.sep);
+const getFolderPath = (p) => path.join(path.posix.dirname(p), path.posix.sep);
 
 /**
  * @type {string} base name
@@ -31,14 +31,14 @@ const getBasename = (filename, ignoreMiddleExtensions = false) =>
 
 /**
  * @type {string[]} all folders
- * @param {string} p path of folder
+ * @param {string} p path of folder in posix style
  */
 const getAllFolders = (p) =>
-  p.split(path.sep).filter((folder) => folder !== '');
+  p.split(path.posix.sep).filter((folder) => folder !== '');
 
 /**
  * @type {string[]} all sub paths
- * @param {string} p path of folder
+ * @param {string} p path of folder in posix style
  */
 const getSubPaths = (p) => {
   const folders = getAllFolders(p);
@@ -49,8 +49,8 @@ const getSubPaths = (p) => {
       if (folder) {
         acc.push(
           index === 0
-            ? path.join(folder, path.sep)
-            : path.join(acc[acc.length - 1], folder, path.sep)
+            ? path.join(folder, path.posix.sep)
+            : path.join(acc[acc.length - 1], folder, path.posix.sep)
         );
       }
       return acc;
@@ -65,11 +65,17 @@ const getSubPaths = (p) => {
 
 /**
  * @type {string} path from repository root
- * @param {string} fullPath filename with full path
- * @param {string} repositoryRoot path of repository root
+ * @param {string} fullPath filename with full path in posix style
+ * @param {string} repositoryRoot path of repository root in posix style
  */
 const getPathFromRepositoryRoot = (fullPath, repositoryRoot) =>
-  fullPath.replace(path.join(repositoryRoot, path.sep), '');
+  fullPath.replace(path.join(repositoryRoot, path.posix.sep), '');
+
+/**
+ * @type {string} file path in posix style
+ * @param {string} p file path based on the operating system
+ */
+const toPosixPath = (p) => p.split(path.sep).join(path.posix.sep);
 
 module.exports = {
   getFolderPath,
@@ -78,4 +84,5 @@ module.exports = {
   getSubPaths,
   getAllFolders,
   getPathFromRepositoryRoot,
+  toPosixPath,
 };