@kadi.build/file-manager 1.0.0 → 1.1.0

package/README.md

@@ -202,7 +202,27 @@ Configuration is loaded from multiple sources (in priority order):
 
 1. Constructor options
 2. Environment variables (prefixed with `KADI_`)
-3. `.env` file
+3. `config.yml` file (walks up from CWD — looks for `file-manager:` section)
+4. `.env` file (walks up from CWD — legacy fallback)
+
+### config.yml
+
+Place a `config.yml` in your project root (or any parent directory):
+
+```yaml
+file-manager:
+  local_root: ./
+  default_upload_directory: ./uploads
+  default_download_directory: ./downloads
+  default_temp_directory: ./temp
+  max_file_size: 1073741824
+  watch_debounce_ms: 100
+  watch_max_watchers: 10
+  compression_format: zip
+  compression_level: 6
+```
+
+### Environment Variables (overrides)
 
 | Variable | Description | Default |
 |----------|-------------|---------|

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kadi.build/file-manager",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "Complete local and remote file management with watching and compression",
   "main": "src/index.js",
   "type": "module",
@@ -38,6 +38,9 @@
     "ssh2": "^1.15.0",
     "ssh2-sftp-client": "^9.1.0"
   },
+  "optionalDependencies": {
+    "js-yaml": "^4.1.0"
+  },
   "devDependencies": {
     "fs-extra": "^11.2.0"
   },

package/src/ConfigManager.js

@@ -3,12 +3,28 @@
  *
  * Handles configuration for file operations only.
  * Simplified from the monolith ConfigManager - no tunnel/server config.
+ *
+ * Configuration resolution order (later wins):
+ *   1. Built-in defaults
+ *   2. Walk-up config.yml → "file-manager" section
+ *   3. Walk-up .env file → fallback when no config.yml found
+ *   4. Environment variables (KADI_<KEY> or bare <KEY>)
+ *   5. Passed options (constructor/load)
  */
 
 import { promises as fs } from 'fs';
+import { readFileSync, existsSync } from 'fs';
 import path from 'path';
 import os from 'os';
 
+let yaml;
+try {
+  const m = await import('js-yaml');
+  yaml = m.default || m;
+} catch {
+  yaml = null;
+}
+
 class ConfigManager {
   constructor() {
     this.config = {};
@@ -72,14 +88,11 @@ class ConfigManager {
     // Apply defaults
     this.config = { ...this.defaults };
 
-    // Load from .env file if it exists
-    try {
-      const envPath = path.join(process.cwd(), '.env');
-      const envContent = await fs.readFile(envPath, 'utf8');
-      this._parseEnvContent(envContent);
-    } catch (error) {
-      // .env file not found, that's fine
-    }
+    // Load from config.yml (walk-up discovery) — "file-manager" section
+    this._loadFromConfigYml();
+
+    // Load from .env file if it exists (walk-up fallback)
+    await this._loadFromEnvFile();
 
     // Load from environment variables
     this._loadFromEnvironment();
@@ -106,6 +119,65 @@ class ConfigManager {
     return this;
   }
 
+  /**
+   * Walk up from CWD looking for config.yml and load the "file-manager" section.
+   */
+  _loadFromConfigYml() {
+    if (!yaml) return;
+
+    const configPath = this._walkUpFind('config.yml');
+    if (!configPath) return;
+
+    try {
+      const parsed = yaml.load(readFileSync(configPath, 'utf8'));
+      const section = parsed && parsed['file-manager'];
+      if (!section || typeof section !== 'object') return;
+
+      // Map YAML snake_case keys to UPPER_CASE config keys
+      for (const [ymlKey, ymlVal] of Object.entries(section)) {
+        const upperKey = ymlKey.toUpperCase();
+        if (this.defaults.hasOwnProperty(upperKey)) {
+          this.config[upperKey] = this._parseValue(String(ymlVal), this.defaults[upperKey]);
+        }
+      }
+    } catch {
+      // config.yml parse failure — continue with other sources
+    }
+  }
+
+  /**
+   * Walk up from CWD looking for a .env file and load it.
+   */
+  async _loadFromEnvFile() {
+    const envPath = this._walkUpFind('.env');
+    if (!envPath) return;
+
+    try {
+      const envContent = await fs.readFile(envPath, 'utf8');
+      this._parseEnvContent(envContent);
+    } catch {
+      // .env file not found or unreadable, that's fine
+    }
+  }
+
+  /**
+   * Walk up from startDir looking for a filename.
+   * @param {string} filename
+   * @param {string} [startDir]
+   * @returns {string|null}
+   */
+  _walkUpFind(filename, startDir) {
+    let dir = startDir || process.cwd();
+    while (true) {
+      const candidate = path.join(dir, filename);
+      if (existsSync(candidate)) return candidate;
+      const parent = path.dirname(dir);
+      if (parent === dir) break;
+      dir = parent;
+    }
+    return null;
+  }
+
   _parseEnvContent(content) {
     const lines = content.split('\n');
     for (const line of lines) {