lsh-framework 1.3.0 → 1.3.1

package/dist/services/secrets/secrets.js

@@ -543,6 +543,42 @@ API_KEY=
             process.exit(1);
         }
     });
+    /**
+     * Detect if file should use 'export' prefix based on file type
+     */
+    function shouldUseExport(filePath) {
+        const filename = path.basename(filePath);
+        const ext = path.extname(filePath);
+        // Shell script files - use export
+        if (['.sh', '.bash', '.zsh'].includes(ext)) {
+            return true;
+        }
+        // Shell profile/rc files - use export
+        if (['.bashrc', '.zshrc', '.profile', '.bash_profile', '.zprofile'].includes(filename)) {
+            return true;
+        }
+        // .envrc files (direnv) - use export
+        if (filename === '.envrc' || filename.endsWith('.envrc')) {
+            return true;
+        }
+        // .env files - do NOT use export
+        if (filename === '.env' || filename.startsWith('.env.')) {
+            return false;
+        }
+        // Default: no export (safest for most env files)
+        return false;
+    }
+    /**
+     * Format a line based on file type
+     */
+    function formatEnvLine(key, value, filePath) {
+        const needsQuotes = /[\s#]/.test(value);
+        const quotedValue = needsQuotes ? `"${value}"` : value;
+        const useExport = shouldUseExport(filePath);
+        return useExport
+            ? `export ${key}=${quotedValue}`
+            : `${key}=${quotedValue}`;
+    }
     /**
      * Set a single secret value
      */
@@ -563,12 +599,11 @@ API_KEY=
                     newLines.push(line);
                     continue;
                 }
-                const match = line.match(/^([^=]+)=(.*)$/);
+                // Match both with and without export
+                const match = line.match(/^(?:export\s+)?([^=]+)=(.*)$/);
                 if (match && match[1].trim() === key) {
-                    // Quote values with spaces or special characters
-                    const needsQuotes = /[\s#]/.test(value);
-                    const quotedValue = needsQuotes ? `"${value}"` : value;
-                    newLines.push(`${key}=${quotedValue}`);
+                    // Use appropriate format for this file type
+                    newLines.push(formatEnvLine(key, value, envPath));
                     found = true;
                 }
                 else {
@@ -579,9 +614,8 @@ API_KEY=
         }
         // If key wasn't found, append it
         if (!found) {
-            const needsQuotes = /[\s#]/.test(value);
-            const quotedValue = needsQuotes ? `"${value}"` : value;
-            content = content.trimRight() + `\n${key}=${quotedValue}\n`;
+            const formattedLine = formatEnvLine(key, value, envPath);
+            content = content.trimRight() + `\n${formattedLine}\n`;
         }
         fs.writeFileSync(envPath, content, 'utf8');
         console.log(`✅ Set ${key}`);
@@ -638,8 +672,8 @@ API_KEY=
                         // Skip comments and empty lines
                         if (trimmed.startsWith('#') || !trimmed)
                             continue;
-                        // Parse KEY=VALUE format
-                        const match = trimmed.match(/^([^=]+)=(.*)$/);
+                        // Parse KEY=VALUE format (with or without export)
+                        const match = trimmed.match(/^(?:export\s+)?([^=]+)=(.*)$/);
                         if (!match) {
                             errors.push(`Invalid format: ${trimmed}`);
                             continue;
@@ -671,15 +705,14 @@ API_KEY=
                                 newLines.push(line);
                                 continue;
                             }
-                            const match = line.match(/^([^=]+)=(.*)$/);
+                            // Match both with and without export
+                            const match = line.match(/^(?:export\s+)?([^=]+)=(.*)$/);
                             if (match) {
                                 const key = match[1].trim();
                                 if (newKeys.has(key)) {
-                                    // Update existing key
+                                    // Update existing key with appropriate format
                                     const value = newKeys.get(key);
-                                    const needsQuotes = /[\s#]/.test(value);
-                                    const quotedValue = needsQuotes ? `"${value}"` : value;
-                                    newLines.push(`${key}=${quotedValue}`);
+                                    newLines.push(formatEnvLine(key, value, envPath));
                                     newKeys.delete(key); // Mark as processed
                                     hasContent = true;
                                 }
@@ -696,13 +729,12 @@ API_KEY=
                     }
                     // Add new keys that weren't in the existing file
                     for (const [key, value] of newKeys.entries()) {
-                        const needsQuotes = /[\s#]/.test(value);
-                        const quotedValue = needsQuotes ? `"${value}"` : value;
+                        const formattedLine = formatEnvLine(key, value, envPath);
                         if (hasContent) {
-                            newLines.push(`${key}=${quotedValue}`);
+                            newLines.push(formattedLine);
                         }
                         else {
-                            newLines.push(`${key}=${quotedValue}`);
+                            newLines.push(formattedLine);
                             hasContent = true;
                         }
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lsh-framework",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "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": {