fix(connection-form): use a double dash for separating appName identi…

…fiers A single `-` can be hard to separate in cases such a vscode where the defaultAppName is `mongodb-vscode` as well as with UUIDs so we use `--` to make this separation clearer.
mongodb-js · Mar 7, 2025 · 39b2b47 · 39b2b47
1 parent 5bd08a4
commit 39b2b47
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/connection-form/src/utils/set-app-name-if-missing.spec.ts b/packages/connection-form/src/utils/set-app-name-if-missing.spec.ts
@@ -57,7 +57,7 @@ describe('setAppNameParamIfMissing', function () {
         connectionString: 'mongodb://atlas/',
       })
     ).to.deep.equal({
-      connectionString: 'mongodb://atlas/?appName=defaultAppName-789-123',
+      connectionString: 'mongodb://atlas/?appName=defaultAppName--789--123',
     });
   });
 });
diff --git a/packages/connection-form/src/utils/set-app-name-if-missing.ts b/packages/connection-form/src/utils/set-app-name-if-missing.ts
@@ -24,8 +24,8 @@ export function setAppNameParamIfMissing({
     if (!searchParams.has('appName') && defaultAppName !== undefined) {
       const appName = isAtlas
         ? `${defaultAppName}${
-            telemetryAnonymousId ? `-${telemetryAnonymousId}` : ''
-          }-${connectionId}`
+            telemetryAnonymousId ? `--${telemetryAnonymousId}` : ''
+          }--${connectionId}`
         : defaultAppName;
 
       searchParams.set('appName', appName);