update

Author: raju-opti

.github/workflows/javascript.yml

@@ -46,16 +46,22 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v3
-    - name: Set up Node
+    - name: Set up Node 20 (build)
       uses: actions/setup-node@v3
       with:
-        node-version: '18'
+        node-version: 20
         cache-dependency-path: ./package-lock.json
         cache: 'npm'
-    - name: Run CommonJS example
+    - name: Build and pack SDK
       run: |
         npm install
-        npm run test-commonjs
+        npm pack
+    - name: Set up Node 18 (runtime test)
+      uses: actions/setup-node@v3
+      with:
+        node-version: '18'
+    - name: Run CommonJS example
+      run: npm run test-commonjs -- --skip-build
 
   integration_tests:
     uses: optimizely/javascript-sdk/.github/workflows/integration_test.yml@master

package.json

@@ -76,7 +76,7 @@
     "build:win": "npm run validate-platform-isolation && npm run genmsg && tsc && rolldown -c rolldown.config.mjs && copy dist\\index.browser.d.ts dist\\index.d.ts",
     "build-browser-umd": "tsc && rolldown -c rolldown.config.mjs -- --config-umd",
     "coveralls": "nyc --reporter=lcov npm test",
-    "prepare": "npm run build",
+    "prepack": "npm run build",
     "prepublishOnly": "npm test",
     "genmsg": "./scripts/generate-messages.js ./lib/message/error_message.ts ./lib/message/log_message.ts",
     "test-typescript": "node ./scripts/run-ts-example.js",

scripts/run-cjs-example.js

@@ -53,16 +53,34 @@ function runQuiet(command, cwd) {
   }
 }
 
+function findTarball() {
+  const files = fs.readdirSync(rootDir).filter(f => f.startsWith('optimizely-optimizely-sdk-') && f.endsWith('.tgz'));
+  if (files.length === 0) {
+    console.error('No SDK tarball found. Run `npm pack` first.');
+    process.exit(1);
+  }
+  return path.join(rootDir, files[0]);
+}
+
 function main() {
+  const skipBuild = process.argv.includes('--skip-build');
+
   console.log('=== Building SDK and Running CommonJS Example ===\n');
 
-  console.log('Installing SDK dependencies...');
-  run('npm install', rootDir);
+  let tarballPath;
+
+  if (skipBuild) {
+    console.log('Skipping build (--skip-build), using existing tarball...');
+    tarballPath = findTarball();
+  } else {
+    console.log('Installing SDK dependencies...');
+    run('npm install', rootDir);
 
-  console.log('\nPacking SDK tarball...');
-  const packOutput = runQuiet('npm pack', rootDir);
-  const tarballPath = path.join(rootDir, packOutput);
-  console.log(`Created: ${packOutput}`);
+    console.log('\nPacking SDK tarball...');
+    const packOutput = runQuiet('npm pack', rootDir);
+    tarballPath = path.join(rootDir, packOutput);
+    console.log(`Created: ${packOutput}`);
+  }
 
   console.log('\nInstalling SDK tarball in cjs-example...');
   run(`npm install --no-save --no-package-lock ${tarballPath}`, exampleDir);
@@ -72,7 +90,7 @@ function main() {
 
   console.log('\nCleaning up tarball...');
   fs.unlinkSync(tarballPath);
-  console.log(`Removed: ${packOutput}`);
+  console.log(`Removed: ${path.basename(tarballPath)}`);
 
   console.log('\n=== Example completed successfully! ===\n');
 }