fix: review fixes

Author: astandrik

src/utils/__test__/logs.test.ts

@@ -3,7 +3,7 @@ import {getLogsLink} from '../logs';
 describe('getLogsLink', () => {
     test('should insert dbName into logs URL query', () => {
         const loggingData = {
-            url: 'https://monitoring.yandex-team.ru/projects/kikimr/logs?from=now-1h&to=now&query=%7Bproject+%3D+%22kikimr%22%2C+service+%3D+%22ydb%22%2C+cluster+%3D+%22ydb-ru-prestable%22%7D',
+            url: 'https://logging.url/projects/kikimr/logs?from=now-1h&to=now&query=%7Bproject+%3D+%22kikimr%22%2C+service+%3D+%22ydb%22%2C+cluster+%3D+%22ydb-ru-prestable%22%7D',
         };
 
         const result = getLogsLink({
@@ -12,16 +12,14 @@ describe('getLogsLink', () => {
         });
 
         // The URL should contain the dbName in the query parameter
-        expect(result).toContain('database+%3D+%22testdb%22');
-        // Original query parts should still be present
-        expect(result).toContain('project+%3D+%22kikimr%22');
-        expect(result).toContain('service+%3D+%22ydb%22');
-        expect(result).toContain('cluster+%3D+%22ydb-ru-prestable%22');
+        expect(result).toBe(
+            'https://logging.url/projects/kikimr/logs?from=now-1h&to=now&query=%7Bproject+%3D+%22kikimr%22%2C+service+%3D+%22ydb%22%2C+cluster+%3D+%22ydb-ru-prestable%22%2C+database+%3D+%22testdb%22%7D',
+        );
     });
 
     test('should handle empty query parameters', () => {
         const loggingData = {
-            url: 'https://monitoring.yandex-team.ru/projects/kikimr/logs?from=now-1h&to=now&query=%7B%7D',
+            url: 'https://logging.url/projects/kikimr/logs?from=now-1h&to=now&query=%7B%7D',
         };
 
         const result = getLogsLink({
@@ -30,7 +28,9 @@ describe('getLogsLink', () => {
         });
 
         // Should add dbName to empty query
-        expect(result).toContain('database+%3D+%22testdb%22');
+        expect(result).toBe(
+            'https://logging.url/projects/kikimr/logs?from=now-1h&to=now&query=%7Bdatabase+%3D+%22testdb%22%7D',
+        );
     });
 
     test('should return empty string for invalid data', () => {

src/utils/logs.ts

@@ -21,9 +21,14 @@ export function getLogsLink({dbName, logging}: GetLogsLinkProps): string {
             if (queryParam) {
                 const decodedQuery = decodeURIComponent(queryParam);
 
+                // gets content between curly braces and replaces with {content, database = dbName}
                 const updatedQuery = decodedQuery.replace(/\{([^}]*)\}/, (_match, contents) => {
                     const trimmedContents = contents.trim();
-                    return `{${trimmedContents}${trimmedContents ? ', ' : ''}database = "${dbName}", level = "ERROR"}`;
+                    if (trimmedContents) {
+                        return `{${trimmedContents}, database = "${dbName}"}`;
+                    } else {
+                        return `{database = "${dbName}"}`;
+                    }
                 });
 
                 url.searchParams.set('query', updatedQuery);