Fix: Simplify wrangler.jsonc update script by removing comments and directly replacing values

Author: abdulkarim1422

.github/workflows/main.yml

@@ -32,33 +32,23 @@ jobs:
 
       - name: Update wrangler config with environment variables
         run: |
-          # Create script to update wrangler.jsonc (using .cjs extension for CommonJS)
+          # Create script to update wrangler.jsonc using string replacement
           cat > update_config.cjs << 'EOF'
           const fs = require('fs');
           
           try {
             // Read the file
             let content = fs.readFileSync('wrangler.jsonc', 'utf8');
             
-            // Remove single-line comments
-            content = content.replace(/\/\/.*$/gm, '');
+            // Replace specific values in the vars section using regex
+            content = content.replace(/"MEMBERSHIP_NUMBER_PREFIX":\s*"[^"]*"/, `"MEMBERSHIP_NUMBER_PREFIX": "${process.env.MEMBERSHIP_NUMBER_PREFIX}"`);
+            content = content.replace(/"MOODLE_API_URL":\s*"[^"]*"/, `"MOODLE_API_URL": "${process.env.MOODLE_API_URL}"`);
+            content = content.replace(/"SMTP_HOST":\s*"[^"]*"/, `"SMTP_HOST": "${process.env.SMTP_HOST}"`);
+            content = content.replace(/"SMTP_PORT":\s*"[^"]*"/, `"SMTP_PORT": "${process.env.SMTP_PORT}"`);
+            content = content.replace(/"SMTP_USER":\s*"[^"]*"/, `"SMTP_USER": "${process.env.SMTP_USER}"`);
             
-            // Remove multi-line comments
-            content = content.replace(/\/\*[\s\S]*?\*\//g, '');
-            
-            // Parse JSON
-            const config = JSON.parse(content);
-            
-            // Update vars
-            if (!config.vars) config.vars = {};
-            config.vars.MEMBERSHIP_NUMBER_PREFIX = process.env.MEMBERSHIP_NUMBER_PREFIX;
-            config.vars.MOODLE_API_URL = process.env.MOODLE_API_URL;
-            config.vars.SMTP_HOST = process.env.SMTP_HOST;
-            config.vars.SMTP_PORT = process.env.SMTP_PORT;
-            config.vars.SMTP_USER = process.env.SMTP_USER;
-            
-            // Write back as JSON (not JSONC)
-            fs.writeFileSync('wrangler.jsonc', JSON.stringify(config, null, 2));
+            // Write back the modified content
+            fs.writeFileSync('wrangler.jsonc', content);
             console.log('Successfully updated wrangler.jsonc with environment variables');
           } catch (error) {
             console.error('Error updating config:', error);