lsh-framework 1.4.0 → 1.4.2

package/.env.example

@@ -1,5 +1,16 @@
-# LSH Shell - Environment Configuration
-# Copy this file to .env and update with your actual values
+# LSH Shell - Environment Configuration (DEPRECATED for LSH config)
+#
+# ⚠️  IMPORTANT: LSH configuration should now be in ~/.config/lsh/lshrc
+#
+# Use: lsh config              # Edit configuration
+#      lsh config set KEY val  # Set individual values
+#
+# This .env.example file is kept for reference and backward compatibility.
+# For new installations, use: lsh config init
+#
+# Priority: Environment variables > ~/.config/lsh/lshrc > defaults
+#
+# Use .env files ONLY for application secrets, NOT for LSH configuration
 
 # Core Configuration
 USER=your_username

package/README.md

@@ -698,6 +698,21 @@ npm run lint
 npm run lint:fix
 ```
 
+### Git Hooks
+
+Install git hooks for automatic linting before commits:
+
+```bash
+# Install hooks
+./scripts/install-git-hooks.sh
+
+# Hooks will run automatically on git commit
+# To skip hooks on a specific commit:
+git commit --no-verify
+```
+
+The pre-commit hook runs `npm run lint` and blocks commits if linting fails.
+
 ### From Source
 
 ```bash

package/dist/lib/config-manager.js

@@ -272,12 +272,16 @@ export class ConfigManager {
     }
     /**
      * Merge config with process.env
-     * Config file values take precedence over environment variables
+     * Config file values are loaded into process.env, but existing environment
+     * variables take precedence (config provides defaults)
      */
     mergeWithEnv() {
         for (const [key, value] of Object.entries(this.config)) {
             if (value !== undefined && value !== '') {
-                process.env[key] = value;
+                // Only set if not already in environment (env vars take precedence)
+                if (!process.env[key]) {
+                    process.env[key] = value;
+                }
             }
         }
     }

package/dist/lib/database-persistence.js

@@ -68,6 +68,16 @@ export class DatabasePersistence {
             return false;
         }
     }
+    /**
+     * Cleanup resources (stop timers, close connections)
+     * Call this when done to allow process to exit
+     */
+    async cleanup() {
+        if (this.localStorage) {
+            await this.localStorage.cleanup();
+        }
+        // Supabase client doesn't need cleanup (no persistent connections)
+    }
     /**
      * Save shell history entry
      */

package/dist/lib/secrets-manager.js

@@ -22,6 +22,13 @@ export class SecretsManager {
             this.gitInfo = getGitRepoInfo();
         }
     }
+    /**
+     * Cleanup resources (stop timers, close connections)
+     * Call this when done to allow process to exit
+     */
+    async cleanup() {
+        await this.persistence.cleanup();
+    }
     /**
      * Get default encryption key from environment or machine
      */

package/dist/services/secrets/secrets.js

@@ -16,15 +16,19 @@ export async function init_secrets(program) {
         .option('-e, --env <name>', 'Environment name (dev/staging/prod)', 'dev')
         .option('--force', 'Force push even if destructive changes detected')
         .action(async (options) => {
+        const manager = new SecretsManager();
         try {
-            const manager = new SecretsManager();
             await manager.push(options.file, options.env, options.force);
         }
         catch (error) {
             const err = error;
             console.error('❌ Failed to push secrets:', err.message);
+            await manager.cleanup();
             process.exit(1);
         }
+        finally {
+            await manager.cleanup();
+        }
     });
     // Pull secrets from cloud
     program
@@ -34,15 +38,19 @@ export async function init_secrets(program) {
         .option('-e, --env <name>', 'Environment name (dev/staging/prod)', 'dev')
         .option('--force', 'Overwrite without creating backup')
         .action(async (options) => {
+        const manager = new SecretsManager();
         try {
-            const manager = new SecretsManager();
             await manager.pull(options.file, options.env, options.force);
         }
         catch (error) {
             const err = error;
             console.error('❌ Failed to pull secrets:', err.message);
+            await manager.cleanup();
             process.exit(1);
         }
+        finally {
+            await manager.cleanup();
+        }
     });
     // List current local secrets
     program
@@ -262,8 +270,8 @@ API_KEY=
         .option('--load', 'Output eval-able export commands for loading secrets')
         .option('--force', 'Force sync even if destructive changes detected')
         .action(async (options) => {
+        const manager = new SecretsManager();
         try {
-            const manager = new SecretsManager();
             if (options.legacy) {
                 // Use legacy sync (suggestions only)
                 await manager.sync(options.file, options.env);
@@ -276,8 +284,12 @@ API_KEY=
         catch (error) {
             const err = error;
             console.error('❌ Failed to sync:', err.message);
+            await manager.cleanup();
             process.exit(1);
         }
+        finally {
+            await manager.cleanup();
+        }
     });
     // Status command - get detailed status info
     program

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lsh-framework",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "description": "Simple, cross-platform encrypted secrets manager with automatic sync and multi-environment support. Just run lsh sync and start managing your secrets.",
   "main": "dist/app.js",
   "bin": {