Remove process.env refs from the browser artifact

Both the Node.js and the browser artifacts that we generate contain
dynamic references to entries in `process.env`. This does not play
nicely when bundling apps intended for the browser.

As of now, the only function that does this is `getBaseUrl`, which is
a function intended for internal development only. This change makes
sure that this function is defined for the browser in such a way that
does not reference `process.env` anymore and just returns the
Production base URL.

Author: jverce

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@pipedream/sdk",
-    "version": "2.0.12",
+    "version": "2.0.13",
     "private": false,
     "repository": "github:PipedreamHQ/pipedream-sdk-typescript",
     "type": "commonjs",
@@ -93,6 +93,8 @@
         "webpack": "^5.97.1"
     },
     "browser": {
+        "./dist/cjs/wrapper/utils/getBaseUrl.js": "./dist/cjs/wrapper/utils/getBaseUrl.browser.js",
+        "./dist/esm/wrapper/utils/getBaseUrl.mjs": "./dist/esm/wrapper/utils/getBaseUrl.browser.mjs",
         "fs": false,
         "os": false,
         "path": false,

src/version.ts

@@ -1 +1 @@
-export const SDK_VERSION = "2.0.12";
+export const SDK_VERSION = "2.0.13";

src/wrapper/Pipedream.ts

@@ -3,6 +3,7 @@ import { ProjectEnvironment } from "../api/index.js";
 import { Workflows } from "../api/resources/workflows/client/Client.js";
 import { PipedreamClient } from "../Client.js";
 import { PipedreamEnvironment } from "../environments.js";
+import { getBaseUrl } from "./utils/getBaseUrl.js";
 
 export type PipedreamClientOpts = {
     /**
@@ -43,17 +44,6 @@ export type PipedreamClientOpts = {
     workflowDomain?: string;
 };
 
-/**
- * Returns the base URL for the Pipedream API based on the provided environment.
- * It replaces any placeholders in the environment string with corresponding
- * environment variables.
- *
- * @param environment - The Pipedream environment string.
- * @returns The base URL for the Pipedream API.
- */
-const getBaseUrl = (environment: PipedreamEnvironment) =>
-    environment.replace(/\$\{(\w+)\}/g, (_, name) => process.env[name] ?? "");
-
 export class Pipedream extends PipedreamClient {
     private _workflowDomain?: string;
     private _workflows: Workflows | undefined;

src/wrapper/utils/getBaseUrl.browser.ts

@@ -0,0 +1,9 @@
+import { PipedreamEnvironment } from "../../environments.js";
+
+/**
+ * Returns the base URL for the Pipedream API based on the provided environment.
+ * Browser-optimized version that returns the production environment directly.
+ *
+ * @returns The base URL for the Pipedream API.
+ */
+export const getBaseUrl = () => PipedreamEnvironment.Prod;

src/wrapper/utils/getBaseUrl.ts

@@ -0,0 +1,12 @@
+import { PipedreamEnvironment } from "../../environments.js";
+
+/**
+ * Returns the base URL for the Pipedream API based on the provided environment.
+ * It replaces any placeholders in the environment string with corresponding
+ * environment variables.
+ *
+ * @param environment - The Pipedream environment string.
+ * @returns The base URL for the Pipedream API.
+ */
+export const getBaseUrl = (environment: PipedreamEnvironment) =>
+    environment.replace(/\$\{(\w+)\}/g, (_, name) => process.env[name] ?? "");

src/wrapper/utils/index.ts

@@ -0,0 +1 @@
+export { getBaseUrl } from "./getBaseUrl.js";