@diegosouzacdv/jev-browser-mcp 0.6.2 → 0.7.1

package/mcp_servers/jev-browser-npm/src/config.mjs

@@ -170,11 +170,20 @@ function validProviderUrl(value) {
   return parsed.toString();
 }
 
-function profileDirectory(template, channel, override) {
-  const value = expandHomeDirectory(override?.trim() || template.replace("{browser}", channel));
-  if (!path.isAbsolute(value)) throw new JevBrowserError("JEV_BROWSER_PROFILE must be an absolute path");
-  return requireOutsidePackage(path.resolve(value), "browser profile");
-}
+function profileDirectory(template, channel, sessionId, override) {
+  const requestedOverride = override?.trim();
+  const unsupportedPlaceholder = requestedOverride?.replaceAll("{session}", "");
+  if (unsupportedPlaceholder?.includes("{") || unsupportedPlaceholder?.includes("}")) {
+    throw new JevBrowserError("JEV_BROWSER_PROFILE supports only the {session} placeholder");
+  }
+  const hasSessionPlaceholder = Boolean(requestedOverride?.includes("{session}"));
+  const profileBase = requestedOverride?.replaceAll("{session}", sessionId);
+  const value = expandHomeDirectory(requestedOverride
+    ? hasSessionPlaceholder ? profileBase : `${profileBase}-session-${sessionId}`
+    : template.replace("{browser}", channel).replace("{session}", sessionId));
+  if (!path.isAbsolute(value)) throw new JevBrowserError("JEV_BROWSER_PROFILE must be an absolute path");
+  return requireOutsidePackage(path.resolve(value), "browser profile");
+}
 
 export function loadSettings({ env = process.env, configPath = CONFIG_PATH } = {}) {
   let document;
@@ -213,12 +222,17 @@ export function loadSettings({ env = process.env, configPath = CONFIG_PATH } = {
   if (!new Set(["chrome", "msedge"]).has(channel)) {
     throw new JevBrowserError("JEV_BROWSER_CHANNEL must be chrome or msedge");
   }
-  const profileTemplate = document.browser.computer_user_data_dir;
-  const profileRemainder = typeof profileTemplate === "string" ? profileTemplate.replace("{browser}", "") : "";
-  if (typeof profileTemplate !== "string" || profileTemplate.split("{browser}").length !== 2
-    || profileRemainder.includes("{") || profileRemainder.includes("}")) {
-    throw new JevBrowserError("config/ui-testing.json browser.computer_user_data_dir must contain {browser}");
-  }
+  const profileTemplate = document.browser.computer_user_data_dir;
+  const profileRemainder = typeof profileTemplate === "string"
+    ? profileTemplate.replace("{browser}", "").replace("{session}", "") : "";
+  if (typeof profileTemplate !== "string" || profileTemplate.split("{browser}").length !== 2
+    || profileTemplate.split("{session}").length !== 2 || profileRemainder.includes("{") || profileRemainder.includes("}")) {
+    throw new JevBrowserError("config/ui-testing.json browser.computer_user_data_dir must contain {browser} and {session}");
+  }
+  const sessionId = (env.JEV_BROWSER_SESSION_ID?.trim() || `pid-${process.pid}`);
+  if (!/^[A-Za-z0-9][A-Za-z0-9_-]{0,63}$/.test(sessionId)) {
+    throw new JevBrowserError("JEV_BROWSER_SESSION_ID must contain 1-64 letters, numbers, underscores, or hyphens");
+  }
 
   const configuredProviderUrl = validProviderUrl(document.jev.provider_url);
   const providerUrl = validProviderUrl(env.JEV_PROVIDER_URL?.trim() || configuredProviderUrl);
@@ -296,7 +310,8 @@ export function loadSettings({ env = process.env, configPath = CONFIG_PATH } = {
     browser: Object.freeze({
       mode,
       channel,
-      profileDir: profileDirectory(profileTemplate, channel, env.JEV_BROWSER_PROFILE),
+      profileDir: profileDirectory(profileTemplate, channel, sessionId, env.JEV_BROWSER_PROFILE),
+      sessionId,
       maxFlowSteps: positiveNumber(document.browser.max_flow_steps, "browser.max_flow_steps", true),
       maxTextEntryChars: positiveNumber(document.browser.max_text_entry_chars, "browser.max_text_entry_chars", true),
       actionTimeoutMs: positiveNumber(browserOptions.max_action_timeout_seconds, "jev_browser_mcp.browser.max_action_timeout_seconds") * 1000,