@arela/uploader 1.0.4 → 1.0.5

package/docs/CROSS_PLATFORM_PATH_HANDLING.md

@@ -37,7 +37,7 @@ C:\Users\Docs → C:/Users/Docs (stored) → scan_palco_local_c_users_docs
 ### cli_registry Table
 - `company_slug`: Company identifier
 - `server_id`: Server identifier  
-- `base_path_full`: Absolute path with forward slashes (`C:/Users/Documents/2023`)
+- `base_path_full`: Absolute path (preserves original format for Windows paths like `O:\expediente\archivos`)
 - `table_name`: Sanitized version (`scan_palco_local_c_users_documents_2023`)
 - **Removed**: `base_path_label` (was redundant - same as base_path_full)
 
@@ -46,18 +46,22 @@ C:\Users\Docs → C:/Users/Docs (stored) → scan_palco_local_c_users_docs
 - `relative_path`: Relative to base with forward slashes  
 - `absolute_path`: Absolute path with forward slashes
 
-**Note**: All paths stored with forward slashes for consistency, but work on all platforms.
+**Note**: Paths are stored consistently. Windows absolute paths (with drive letters) are preserved when the CLI runs on the same Windows machine or a different OS (macOS/Linux).
 
 ## Key Features
 
-### 1. Cross-Platform Path Normalization
+### 1. Cross-Platform Path Detection
 
-All paths are normalized to POSIX format (forward slashes) for consistency:
+The CLI correctly identifies Windows absolute paths even when running on macOS/Linux:
 
-- **Windows paths**: `C:\Users\Documents` → `/Users/Documents`
-- **Unix paths**: `/home/user/docs` → `/home/user/docs`
-- **Relative paths**: `../parent/folder` → `../parent/folder`
-- **Network drives**: `O:/data/files` → `/data/files`
+- **Windows paths on any OS**: `O:\expediente\archivos` → recognized as absolute, preserved as-is
+- **Unix paths**: `/home/user/docs` → recognized as absolute, normalized
+- **Relative paths**: `./sample` → resolved against cwd
+
+This allows:
+- Running `arela scan` on Windows server → creates table with Windows path in `base_path_full`
+- Running `arela identify` on macOS (for testing) → correctly uses the same Windows path
+- Both resolve to the same table lookup
 
 ### 2. Full Path in Table Names
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arela/uploader",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "CLI to upload files/directories to Arela",
   "bin": {
     "arela": "./src/index.js"

package/src/config/config.js

@@ -34,10 +34,10 @@ class Config {
       const __dirname = path.dirname(__filename);
       const packageJsonPath = path.resolve(__dirname, '../../package.json');
       const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf-8'));
-      return packageJson.version || '1.0.4';
+      return packageJson.version || '1.0.5';
     } catch (error) {
       console.warn('⚠️ Could not read package.json version, using fallback');
-      return '1.0.4';
+      return '1.0.5';
     }
   }
 
@@ -257,11 +257,13 @@ class Config {
     if (!basePathLabel && process.env.UPLOAD_BASE_PATH) {
       const basePath = process.env.UPLOAD_BASE_PATH;
       // Resolve to absolute path (handles ../sample vs ./sample correctly)
+      // Note: toAbsolutePath handles Windows paths (O:\...) even on macOS/Linux
       basePathLabel = PathNormalizer.toAbsolutePath(basePath);
     }
 
     // If basePathLabel is provided, ensure it's absolute
-    if (basePathLabel && !path.isAbsolute(basePathLabel)) {
+    // Use PathNormalizer.isAbsolutePath for cross-platform Windows path detection
+    if (basePathLabel && !PathNormalizer.isAbsolutePath(basePathLabel)) {
       basePathLabel = PathNormalizer.toAbsolutePath(basePathLabel);
     }
 

package/src/utils/PathNormalizer.js

@@ -10,31 +10,83 @@ import { fileURLToPath } from 'url';
  * 2. Store absolute paths in base_path_full
  * 3. Normalize ONLY for table name generation (sanitize special chars)
  * 4. Keep full path structure in table names (hash if too long)
+ *
+ * Cross-platform support:
+ * - Recognizes Windows paths (C:\, D:\, etc.) even when running on macOS/Linux
+ * - Preserves Windows paths as-is when detected (for remote server scenarios)
+ * - This allows CLI running on macOS to work with paths stored from Windows servers
  */
 export class PathNormalizer {
+  /**
+   * Check if a path is a Windows-style absolute path (has drive letter)
+   * This works on any OS, not just Windows
+   *
+   * Examples:
+   * - C:\Users\Documents -> true
+   * - O:\expediente\archivos -> true
+   * - /home/user/docs -> false
+   * - ./relative/path -> false
+   *
+   * @param {string} inputPath - Path to check
+   * @returns {boolean} True if path has a Windows drive letter
+   */
+  static isWindowsAbsolutePath(inputPath) {
+    if (!inputPath || typeof inputPath !== 'string') return false;
+    // Match drive letter followed by colon and backslash or forward slash
+    // e.g., C:\, D:/, O:\
+    return /^[A-Za-z]:[/\\]/.test(inputPath);
+  }
+
+  /**
+   * Check if a path is absolute (works cross-platform)
+   * Recognizes both Unix and Windows absolute paths regardless of current OS
+   *
+   * @param {string} inputPath - Path to check
+   * @returns {boolean} True if path is absolute (Unix or Windows style)
+   */
+  static isAbsolutePath(inputPath) {
+    if (!inputPath) return false;
+    // Check native absolute path first, then Windows-style
+    return path.isAbsolute(inputPath) || this.isWindowsAbsolutePath(inputPath);
+  }
+
   /**
    * Resolve a path to an absolute path
    * This ensures ../sample and ./sample are treated as different paths
    *
+   * Cross-platform behavior:
+   * - Windows paths (C:\, O:\, etc.) are preserved as-is, even on macOS/Linux
+   * - Unix paths are normalized using native path.resolve
+   * - Relative paths are resolved against basePath or cwd
+   *
    * Examples:
    * - ../sample (from /data/project) -> /data/sample
    * - ./sample (from /data/project) -> /data/project/sample
-   * - C:\Users\Documents -> C:\Users\Documents (on Windows)
+   * - C:\Users\Documents -> C:\Users\Documents (preserved on any OS)
+   * - O:\expediente\archivos -> O:\expediente\archivos (preserved on any OS)
    * - /home/user/docs -> /home/user/docs (on Unix)
    *
    * @param {string} inputPath - Path to resolve
    * @param {string} basePath - Base path for resolving relative paths (defaults to cwd)
-   * @returns {string} Absolute path in native OS format
+   * @returns {string} Absolute path (native format for Unix, preserved for Windows)
    */
   static toAbsolutePath(inputPath, basePath = null) {
     if (!inputPath) return '';
 
-    // If already absolute, return as-is
+    // Check for Windows absolute path first (works on any OS)
+    // This prevents macOS from treating "O:\path" as relative
+    if (this.isWindowsAbsolutePath(inputPath)) {
+      // Normalize Windows path but preserve the format
+      // Replace forward slashes with backslashes for consistency
+      return inputPath.replace(/\//g, '\\');
+    }
+
+    // If native absolute path, return normalized
     if (path.isAbsolute(inputPath)) {
       return path.normalize(inputPath);
     }
 
-    // Resolve relative path
+    // Resolve relative path against base
     const base = basePath || process.cwd();
     return path.resolve(base, inputPath);
   }