Skip to content

Commit bc697da

Browse files
committed
Fix ES6 import/require mixing in deployment script
- Use async/await pattern instead of mixing require with ES6 imports - Fix 'require is not defined' error in deployment - Maintain CRX generation logic with proper async handling
1 parent 0ae59d5 commit bc697da

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

scripts/deploy-chrome.js

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,18 @@ try {
4646
if (!fs.existsSync(crxPath)) {
4747
console.log('🔧 Generating CRX file from ZIP...');
4848
// For now, we'll use the existing package-chrome.js script to generate CRX
49-
const { execSync } = require('child_process');
5049
try {
51-
execSync('npm run package:chrome', { stdio: 'inherit' });
50+
await new Promise((resolve, reject) => {
51+
exec('npm run package:chrome', (error, stdout, stderr) => {
52+
if (error) {
53+
console.error('❌ Failed to generate CRX file:', error.message);
54+
reject(error);
55+
} else {
56+
console.log('✅ CRX file generated successfully');
57+
resolve();
58+
}
59+
});
60+
});
5261
} catch (error) {
5362
console.error('❌ Failed to generate CRX file:', error.message);
5463
throw error;

0 commit comments

Comments
 (0)