Merge pull request #96 from adzerk/akirasato/sc-68087/decision-js-sdk…

…-using-nodejs-deps-client Skip require http/https during webpack build
adzerk · Jan 31, 2025 · 38f1623 · 38f1623
2 parents 02a7762 + e05de5f
commit 38f1623
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@adzerk/decision-sdk",
-  "version": "1.0.0-beta.25",
+  "version": "1.0.0-beta.26",
   "description": "SDK for interacting with Adzerk's Decision API",
   "main": "lib/index.js",
   "clientSideApp": "dist/adzerk-decision-sdk.js",

diff --git a/src/client.ts b/src/client.ts
@@ -26,6 +26,8 @@ const log = debug('adzerk-decision-sdk:client');
 const isNode = typeof process !== 'undefined' && process.title !== 'browser';
 const isReactNative =
   typeof navigator !== 'undefined' && navigator.product === 'ReactNative';
+/** @ts-ignore */
+const isWebpack = typeof __webpack_require__ === 'function'
 const deprecatedPlacementFields: Array<Array<string>> = [
   ['ecpmPartition', 'ecpmPartitions'],
 ];
@@ -398,7 +400,7 @@ export class Client {
 
     this._path = opts.path;
 
-    if (isNode && !isReactNative) {
+    if (isNode && !isReactNative && !isWebpack) {
       let { Agent } = require(protocol);
       this._agent =
         opts.agent ||