feat(cli-repl): use mongosh_ prefix for logs in custom locations

Author: gagik

packages/cli-repl/src/cli-repl.spec.ts

@@ -1432,7 +1432,7 @@ describe('CliRepl', function () {
           });
 
           const customLogLocation = useTmpdir();
-          it('can set the log location', async function () {
+          it('can set the log location and uses a prefix', async function () {
             cliRepl.config.logLocation = customLogLocation.path;
             await cliRepl.start(await testServer.connectionString(), {});
 
@@ -1442,6 +1442,22 @@ describe('CliRepl', function () {
             expect(cliRepl.logWriter?.logFilePath).equals(
               path.join(
                 customLogLocation.path,
+                'mongosh_' + (cliRepl.logWriter?.logId as string) + '_log'
+              )
+            );
+          });
+
+          it('does not use a prefix if the custom location is the same as the home location', async function () {
+            const customLogHomePath = cliRepl.shellHomeDirectory.localPath('.');
+            cliRepl.config.logLocation = customLogHomePath;
+            await cliRepl.start(await testServer.connectionString(), {});
+
+            expect(await cliRepl.getConfig('logLocation')).equals(
+              customLogHomePath
+            );
+            expect(cliRepl.logWriter?.logFilePath).equals(
+              path.join(
+                customLogHomePath,
                 (cliRepl.logWriter?.logId as string) + '_log'
               )
             );

packages/cli-repl/src/cli-repl.ts

@@ -255,10 +255,12 @@ export class CliRepl implements MongoshIOProvider {
       throw new Error('Logging and telemetry not setup');
     }
 
+    const defaultDirectory = this.shellHomeDirectory.localPath('.');
+    const directory = (await this.getConfig('logLocation')) || defaultDirectory;
+
     this.logManager ??= new MongoLogManager({
-      directory:
-        (await this.getConfig('logLocation')) ||
-        this.shellHomeDirectory.localPath('.'),
+      directory,
+      prefix: directory !== defaultDirectory ? 'mongosh_' : undefined,
       retentionDays: await this.getConfig('logRetentionDays'),
       gzip: await this.getConfig('logCompressionEnabled'),
       maxLogFileCount: +(

packages/e2e-tests/test/e2e.spec.ts

@@ -1407,7 +1407,7 @@ describe('e2e', function () {
         }
         const logPath = path.join(
           customBasePath ?? logBasePath,
-          `${shell.logId}_log`
+          `${customBasePath ? 'mongosh_' : ''}${shell.logId}_log`
         );
         return readReplLogFile<T>(logPath);
       };