From 66801e8a6928677af3eb07aa85f6143fc9f65931 Mon Sep 17 00:00:00 2001
From: Aman Kumar <aman.kumar@contentstack.com>
Date: Fri, 5 Sep 2025 12:21:16 +0530
Subject: [PATCH 1/2] fix: Improper Handling of Optional Log Config and set log
 level issue

---
 package-lock.json                             |   8 +-
 packages/contentstack-config/README.md        |  19 +-
 packages/contentstack-config/package.json     |   2 +-
 .../src/commands/config/get/log.ts            |  40 +-
 .../src/commands/config/set/log.ts            |  67 ++-
 .../src/utils/log-config-defaults.ts          |  36 ++
 .../test/unit/commands/log.test.ts            | 456 ++++++++++++++----
 packages/contentstack-utilities/package.json  |   2 +-
 .../contentstack-utilities/src/logger/log.ts  |   8 +-
 .../src/logger/logger.ts                      |   5 +
 packages/contentstack/README.md               |  17 +-
 packages/contentstack/package.json            |   4 +-
 pnpm-lock.yaml                                |  42 +-
 13 files changed, 519 insertions(+), 187 deletions(-)
 create mode 100644 packages/contentstack-config/src/utils/log-config-defaults.ts

diff --git a/package-lock.json b/package-lock.json
index 203f4b4454..76f9bce6cd 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -25369,10 +25369,10 @@
         "@contentstack/cli-cm-migrate-rte": "~1.6.1",
         "@contentstack/cli-cm-seed": "~1.12.1",
         "@contentstack/cli-command": "~1.6.1",
-        "@contentstack/cli-config": "~1.15.1",
+        "@contentstack/cli-config": "~1.15.2",
         "@contentstack/cli-launch": "^1.9.2",
         "@contentstack/cli-migration": "~1.8.1",
-        "@contentstack/cli-utilities": "~1.14.0",
+        "@contentstack/cli-utilities": "~1.14.1",
         "@contentstack/cli-variants": "~1.3.1",
         "@contentstack/management": "~1.22.0",
         "@oclif/core": "^4.3.0",
@@ -26012,7 +26012,7 @@
     },
     "packages/contentstack-config": {
       "name": "@contentstack/cli-config",
-      "version": "1.15.1",
+      "version": "1.15.2",
       "license": "MIT",
       "dependencies": {
         "@contentstack/cli-command": "~1.6.1",
@@ -27708,7 +27708,7 @@
     },
     "packages/contentstack-utilities": {
       "name": "@contentstack/cli-utilities",
-      "version": "1.14.0",
+      "version": "1.14.1",
       "license": "MIT",
       "dependencies": {
         "@contentstack/management": "~1.22.0",
diff --git a/packages/contentstack-config/README.md b/packages/contentstack-config/README.md
index cabc0c8dea..19b10888a3 100644
--- a/packages/contentstack-config/README.md
+++ b/packages/contentstack-config/README.md
@@ -18,7 +18,7 @@ $ npm install -g @contentstack/cli-config
 $ csdx COMMAND
 running command...
 $ csdx (--version)
-@contentstack/cli-config/1.15.1 darwin-arm64 node-v22.14.0
+@contentstack/cli-config/1.15.2 darwin-arm64 node-v22.14.0
 $ csdx --help [COMMAND]
 USAGE
   $ csdx COMMAND
@@ -329,9 +329,10 @@ USAGE
   $ csdx config:set:log [--level debug|info|warn|error] [--path <value>] [--show-console-logs]
 
 FLAGS
-  --level=<option>          Set the log level for the CLI.
+  --level=<option>          Set the log level for the CLI. Defaults to "info" if not specified.
                             <options: debug|info|warn|error>
-  --path=<value>            Specify the file path where logs should be saved.
+  --path=<value>            Specify the directory path where logs should be saved. Supports both relative and absolute
+                            paths. Defaults to ~/.contentstack/logs if not specified.
   --[no-]show-console-logs  Enable console logging.
 
 DESCRIPTION
@@ -340,9 +341,19 @@ DESCRIPTION
 EXAMPLES
   $ csdx config:set:log
 
-  $ csdx config:set:log --level debug --path ./logs/app.log --show-console-logs
+  $ csdx config:set:log --level debug
+
+  $ csdx config:set:log --path ./logs
+
+  $ csdx config:set:log --level debug --path ./logs --show-console-logs
 
   $ csdx config:set:log --no-show-console-logs
+
+  $ csdx config:set:log --level warn --show-console-logs
+
+  $ csdx config:set:log --path ~/custom/logs
+
+  $ csdx config:set:log --path /var/log/contentstack
 ```
 
 _See code: [src/commands/config/set/log.ts](https://github.com/contentstack/cli/blob/main/packages/contentstack-config/src/commands/config/set/log.ts)_
diff --git a/packages/contentstack-config/package.json b/packages/contentstack-config/package.json
index 22277666d4..7406f5d815 100644
--- a/packages/contentstack-config/package.json
+++ b/packages/contentstack-config/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@contentstack/cli-config",
   "description": "Contentstack CLI plugin for configuration",
-  "version": "1.15.1",
+  "version": "1.15.2",
   "author": "Contentstack",
   "scripts": {
     "build": "npm run clean && npm run compile",
diff --git a/packages/contentstack-config/src/commands/config/get/log.ts b/packages/contentstack-config/src/commands/config/get/log.ts
index 12a97bf94c..8b2307a12c 100644
--- a/packages/contentstack-config/src/commands/config/get/log.ts
+++ b/packages/contentstack-config/src/commands/config/get/log.ts
@@ -1,5 +1,6 @@
 import { Command } from '@contentstack/cli-command';
-import { cliux, configHandler, messageHandler, TableHeader } from '@contentstack/cli-utilities';
+import { cliux, configHandler, TableHeader } from '@contentstack/cli-utilities';
+import { getEffectiveLogConfig } from '../../../utils/log-config-defaults';
 
 export default class LogGetCommand extends Command {
   static description = 'Get logging configuration for CLI';
@@ -9,26 +10,29 @@ export default class LogGetCommand extends Command {
   async run() {
     try {
       const currentLoggingConfig = configHandler.get('log') || {};
-      const logLevel = currentLoggingConfig?.level;
-      const logPath = currentLoggingConfig?.path;
+      const effectiveConfig = getEffectiveLogConfig(currentLoggingConfig);
 
-      if (logLevel || logPath) {
-        const logConfigList = [
-          {
-            'Log Level': logLevel || 'Not set',
-            'Log Path': logPath || 'Not set',
-          },
-        ];
+      const logConfigList = [
+        {
+          Setting: 'Log Level',
+          Value: effectiveConfig.level,
+        },
+        {
+          Setting: 'Log Path',
+          Value: effectiveConfig.path,
+        },
+        {
+          Setting: 'Show Console Logs',
+          Value: effectiveConfig.showConsoleLogs.toString(),
+        },
+      ];
 
-        const headers: TableHeader[] = [
-          { value: 'Log Level' },
-          { value: 'Log Path' },
-        ];
+      const headers: TableHeader[] = [{ value: 'Setting' }, { value: 'Value' }];
 
-        cliux.table(headers, logConfigList);
-      } else {
-        cliux.print(`error: ${messageHandler.parse('CLI_CONFIG_LOG_NO_CONFIG')}`, { color: 'red' });
-      }
+      cliux.table(headers, logConfigList);
+      cliux.print('\nNote: Absolute paths are displayed. Relative paths are resolved from current working directory.', {
+        color: 'dim',
+      });
     } catch (error) {
       cliux.error('error', error);
     }
diff --git a/packages/contentstack-config/src/commands/config/set/log.ts b/packages/contentstack-config/src/commands/config/set/log.ts
index 556d17661a..82449f3461 100644
--- a/packages/contentstack-config/src/commands/config/set/log.ts
+++ b/packages/contentstack-config/src/commands/config/set/log.ts
@@ -1,60 +1,75 @@
 import { Command } from '@contentstack/cli-command';
 import { cliux, flags, configHandler, FlagInput, messageHandler } from '@contentstack/cli-utilities';
-import { interactive } from '../../../utils';
+import { resolveLogPath } from '../../../utils/log-config-defaults';
+import * as path from 'path';
 
 export default class LogSetCommand extends Command {
   static description = 'Set logging configuration for CLI';
 
   static flags: FlagInput = {
-    'level': flags.string({
-      description: 'Set the log level for the CLI.',
+    level: flags.string({
+      description: 'Set the log level for the CLI. Defaults to "info" if not specified.',
       options: ['debug', 'info', 'warn', 'error'],
     }),
-    'path': flags.string({
-      description: 'Specify the file path where logs should be saved.',
+    path: flags.string({
+      description:
+        'Specify the directory path where logs should be saved. Supports both relative and absolute paths. Defaults to ~/.contentstack/logs if not specified.',
     }),
     'show-console-logs': flags.boolean({
       description: 'Enable console logging.',
       allowNo: true, // no-show-console-logs
       default: false,
-    })
+    }),
   };
 
-
   static examples = [
     'csdx config:set:log',
-    'csdx config:set:log --level debug --path ./logs/app.log --show-console-logs',
+    'csdx config:set:log --level debug',
+    'csdx config:set:log --path ./logs',
+    'csdx config:set:log --level debug --path ./logs --show-console-logs',
     'csdx config:set:log --no-show-console-logs',
+    'csdx config:set:log --level warn --show-console-logs',
+    'csdx config:set:log --path ~/custom/logs',
+    'csdx config:set:log --path /var/log/contentstack',
   ];
 
   async run() {
     try {
       const { flags } = await this.parse(LogSetCommand);
+      const currentLoggingConfig = configHandler.get('log') || {};
+      if (flags['level'] !== undefined) {
+        currentLoggingConfig.level = flags['level'];
+      }
 
-      let logLevel: string = flags['level'];
-      let logPath: string = flags['path'];
-      const showConsoleLogs: boolean = flags['show-console-logs'];
-
-      // Interactive prompts if not passed via flags
-      if (!logLevel) {
-        logLevel = await interactive.askLogLevel();
+      if (flags['path'] !== undefined) {
+        // Convert to absolute path and ensure it's a directory
+        let resolvedPath = resolveLogPath(flags['path']);
+        const pathExt = path.extname(resolvedPath);
+        if (pathExt && pathExt.length > 0) {
+          resolvedPath = path.dirname(resolvedPath);
+        }
+        
+        currentLoggingConfig.path = resolvedPath;
       }
 
-      if (!logPath) {
-        logPath = await interactive.askLogPath();
+      if (flags['show-console-logs'] !== undefined) {
+        currentLoggingConfig['show-console-logs'] = flags['show-console-logs'];
       }
+      configHandler.set('log', currentLoggingConfig);
 
-      const currentLoggingConfig = configHandler.get('log') || {};
-      if (logLevel) currentLoggingConfig.level = logLevel;
-      if (logPath) currentLoggingConfig.path = logPath;
-      currentLoggingConfig['show-console-logs'] = showConsoleLogs;
+      if (flags['level'] !== undefined) {
+        cliux.success(messageHandler.parse('CLI_CONFIG_LOG_LEVEL_SET', currentLoggingConfig.level));
+      }
 
-      configHandler.set('log', currentLoggingConfig);
+      if (flags['path'] !== undefined) {
+        cliux.success(messageHandler.parse('CLI_CONFIG_LOG_PATH_SET', currentLoggingConfig.path));
+      }
 
-      cliux.success(messageHandler.parse('CLI_CONFIG_LOG_LEVEL_SET', logLevel));
-      cliux.success(messageHandler.parse('CLI_CONFIG_LOG_PATH_SET', logPath));
-      cliux.success(messageHandler.parse('CLI_CONFIG_LOG_CONSOLE_SET', String(showConsoleLogs)));
-      cliux.print(messageHandler.parse('CLI_CONFIG_LOG_SET_SUCCESS'), { color: 'green' });
+      if (flags['show-console-logs'] !== undefined) {
+        cliux.success(
+          messageHandler.parse('CLI_CONFIG_LOG_CONSOLE_SET', String(currentLoggingConfig['show-console-logs'])),
+        );
+      }
     } catch (error) {
       cliux.error('error', error);
     }
diff --git a/packages/contentstack-config/src/utils/log-config-defaults.ts b/packages/contentstack-config/src/utils/log-config-defaults.ts
new file mode 100644
index 0000000000..c882d3b7ce
--- /dev/null
+++ b/packages/contentstack-config/src/utils/log-config-defaults.ts
@@ -0,0 +1,36 @@
+import * as path from 'path';
+import * as os from 'os';
+
+export const LOG_CONFIG_DEFAULTS = {
+  LEVEL: 'info',
+  PATH: path.join(os.homedir(), '.contentstack', 'logs'),
+  SHOW_CONSOLE_LOGS: false,
+} as const;
+
+/**
+ * Resolves a log path to an absolute path
+ * @param logPath - The path to resolve
+ * @returns Absolute path
+ */
+export function resolveLogPath(logPath: string): string {
+  if (!logPath) return LOG_CONFIG_DEFAULTS.PATH;
+
+  return path.isAbsolute(logPath) ? logPath : path.resolve(process.cwd(), logPath);
+}
+
+/**
+ * Gets the effective log configuration with defaults applied
+ * @param currentConfig - Current configuration from config handler
+ * @returns Configuration with defaults applied
+ */
+export function getEffectiveLogConfig(currentConfig: any = {}) {
+  const logLevel = currentConfig?.level || LOG_CONFIG_DEFAULTS.LEVEL;
+  const logPath = resolveLogPath(currentConfig?.path || LOG_CONFIG_DEFAULTS.PATH);
+  const showConsoleLogs = currentConfig?.['show-console-logs'] ?? LOG_CONFIG_DEFAULTS.SHOW_CONSOLE_LOGS;
+
+  return {
+    level: logLevel,
+    path: logPath,
+    showConsoleLogs,
+  };
+}
diff --git a/packages/contentstack-config/test/unit/commands/log.test.ts b/packages/contentstack-config/test/unit/commands/log.test.ts
index f1dbabc1bd..fa3d825721 100644
--- a/packages/contentstack-config/test/unit/commands/log.test.ts
+++ b/packages/contentstack-config/test/unit/commands/log.test.ts
@@ -1,10 +1,11 @@
 import { expect } from 'chai';
 import * as sinon from 'sinon';
-import { cliux, configHandler, messageHandler } from '@contentstack/cli-utilities';
+import * as path from 'path';
+import { cliux, configHandler } from '@contentstack/cli-utilities';
 
-import { interactive } from '../../../src/utils';
 import LogSetCommand from '../../../src/commands/config/set/log';
 import LogGetCommand from '../../../src/commands/config/get/log';
+import { LOG_CONFIG_DEFAULTS } from '../../../src/utils/log-config-defaults';
 
 describe('Log Commands', () => {
   describe('Log Set Command', () => {
@@ -26,13 +27,16 @@ describe('Log Commands', () => {
       sinon.restore();
     });
 
-    it('should set log level and path from flags', async () => {
+    it('should set log level and path from flags with absolute path resolution and file-to-directory conversion', async () => {
       const cmd = new LogSetCommand([], {} as any);
+      const relativePath = './logs/app.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), './logs'); // Directory, not file
+      
       sinon.stub(cmd as any, 'parse').resolves({
         flags: {
           level: 'debug',
-          path: './logs/app.log',
-          'show-console-logs': false, // Include default value
+          path: relativePath,
+          'show-console-logs': false,
         },
       });
 
@@ -44,7 +48,7 @@ describe('Log Commands', () => {
       expect(
         setStub.calledWith('log', {
           level: 'debug',
-          path: './logs/app.log',
+          path: expectedAbsolutePath, // Should be directory path, not file path
           'show-console-logs': false,
         }),
       ).to.be.true;
@@ -54,47 +58,58 @@ describe('Log Commands', () => {
       expect(successMessage[2]).to.include('CLI_CONFIG_LOG_CONSOLE_SET');
     });
 
-    it('should prompt for log level and path when flags are not provided', async () => {
+    it('should call set even when no flags are provided', async () => {
       const cmd = new LogSetCommand([], {} as any);
       sinon.stub(cmd as any, 'parse').resolves({
-        flags: {
-          'show-console-logs': false, // Include default value
-        },
+        flags: {},
       });
 
-      sinon.stub(interactive, 'askLogLevel').resolves('info');
-      sinon.stub(interactive, 'askLogPath').resolves('./custom/logs/app.log');
+      sinon.stub(configHandler, 'get').returns({ level: 'warn', path: './existing.log' });
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      // Should call set even when no flags are provided
+      expect(setStub.called).to.be.true;
+      expect(setStub.calledWith('log', { level: 'warn', path: './existing.log' })).to.be.true;
+
+      // Should not display any success messages when no flags are provided
+      expect(successMessage.length).to.equal(0);
+    });
+
+    it('should call set even when no flags are provided and no existing config', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: {},
+      });
 
       sinon.stub(configHandler, 'get').returns({});
       const setStub = sinon.stub(configHandler, 'set');
 
       await cmd.run();
 
-      expect(
-        setStub.calledWith('log', {
-          level: 'info',
-          path: './custom/logs/app.log',
-          'show-console-logs': false,
-        }),
-      ).to.be.true;
+      // Should call set even when no flags are provided
+      expect(setStub.called).to.be.true;
+      expect(setStub.calledWith('log', {})).to.be.true;
 
-      expect(successMessage[0]).to.include('CLI_CONFIG_LOG_LEVEL_SET');
-      expect(successMessage[1]).to.include('CLI_CONFIG_LOG_PATH_SET');
-      expect(successMessage[2]).to.include('CLI_CONFIG_LOG_CONSOLE_SET');
+      // Should not display any success messages when no flags are provided
+      expect(successMessage.length).to.equal(0);
     });
 
     it('should preserve existing config values when only setting level', async () => {
       const cmd = new LogSetCommand([], {} as any);
+      const existingPath = './existing/logs/app.log';
+      
       sinon.stub(cmd as any, 'parse').resolves({
         flags: {
           level: 'warn',
-          'show-console-logs': false, // Include default value
         },
       });
 
-      sinon.stub(interactive, 'askLogPath').resolves('./existing/logs/app.log');
-
-      sinon.stub(configHandler, 'get').returns({ path: './existing/logs/app.log' });
+      sinon.stub(configHandler, 'get').returns({ 
+        path: existingPath,
+        'show-console-logs': true 
+      });
       const setStub = sinon.stub(configHandler, 'set');
 
       await cmd.run();
@@ -102,28 +117,31 @@ describe('Log Commands', () => {
       expect(
         setStub.calledWith('log', {
           level: 'warn',
-          path: './existing/logs/app.log',
-          'show-console-logs': false,
+          path: existingPath,  // Should preserve existing path unchanged
+          'show-console-logs': true,  // Should preserve existing console logs setting
         }),
       ).to.be.true;
 
-      expect(successMessage[0]).to.include('CLI_CONFIG_LOG_LEVEL_SET');
-      expect(successMessage[1]).to.include('CLI_CONFIG_LOG_PATH_SET');
-      expect(successMessage[2]).to.include('CLI_CONFIG_LOG_CONSOLE_SET');
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.false;
     });
 
     it('should preserve existing config values when only setting path', async () => {
       const cmd = new LogSetCommand([], {} as any);
+      const newPath = './new/logs/app.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), './new/logs');
+      
       sinon.stub(cmd as any, 'parse').resolves({
         flags: {
-          path: './new/logs/app.log',
-          'show-console-logs': false, // Include default value
+          path: newPath,
         },
       });
 
-      sinon.stub(interactive, 'askLogLevel').resolves('error');
-
-      sinon.stub(configHandler, 'get').returns({ level: 'error' });
+      sinon.stub(configHandler, 'get').returns({ 
+        level: 'error',
+        'show-console-logs': false 
+      });
       const setStub = sinon.stub(configHandler, 'set');
 
       await cmd.run();
@@ -131,39 +149,38 @@ describe('Log Commands', () => {
       expect(
         setStub.calledWith('log', {
           level: 'error',
-          path: './new/logs/app.log',
+          path: expectedAbsolutePath,
           'show-console-logs': false,
         }),
       ).to.be.true;
 
-      expect(successMessage[0]).to.include('CLI_CONFIG_LOG_LEVEL_SET');
-      expect(successMessage[1]).to.include('CLI_CONFIG_LOG_PATH_SET');
-      expect(successMessage[2]).to.include('CLI_CONFIG_LOG_CONSOLE_SET');
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.false;
     });
 
-    it('should set show-console-logs flag to true', async () => {
+    it('should set show-console-logs flag only when explicitly provided', async () => {
       const cmd = new LogSetCommand([], {} as any);
       sinon.stub(cmd as any, 'parse').resolves({
         flags: { 'show-console-logs': true },
       });
 
-      sinon.stub(interactive, 'askLogLevel').resolves('info');
-      sinon.stub(interactive, 'askLogPath').resolves('./logs/app.log');
-
-      sinon.stub(configHandler, 'get').returns({});
+      sinon.stub(configHandler, 'get').returns({ level: 'debug', path: './existing.log' });
       const setStub = sinon.stub(configHandler, 'set');
 
       await cmd.run();
 
       expect(
         setStub.calledWith('log', {
-          level: 'info',
-          path: './logs/app.log',
+          level: 'debug', 
+          path: './existing.log',
           'show-console-logs': true,
         }),
       ).to.be.true;
 
-      expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
     });
 
     it('should set show-console-logs flag to false (--no-show-console-logs)', async () => {
@@ -172,10 +189,7 @@ describe('Log Commands', () => {
         flags: { 'show-console-logs': false },
       });
 
-      sinon.stub(interactive, 'askLogLevel').resolves('info');
-      sinon.stub(interactive, 'askLogPath').resolves('./logs/app.log');
-
-      sinon.stub(configHandler, 'get').returns({});
+      sinon.stub(configHandler, 'get').returns({ level: 'info' });
       const setStub = sinon.stub(configHandler, 'set');
 
       await cmd.run();
@@ -183,20 +197,24 @@ describe('Log Commands', () => {
       expect(
         setStub.calledWith('log', {
           level: 'info',
-          path: './logs/app.log',
           'show-console-logs': false,
         }),
       ).to.be.true;
 
-      expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
     });
 
-    it('should set all flags together (level, path, show-console-logs)', async () => {
+    it('should set all flags together (level, path, show-console-logs) with absolute path', async () => {
       const cmd = new LogSetCommand([], {} as any);
+      const relativePath = './logs/warnings.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), './logs'); 
+      
       sinon.stub(cmd as any, 'parse').resolves({
         flags: {
           level: 'warn',
-          path: './logs/warnings.log',
+          path: relativePath,
           'show-console-logs': true,
         },
       });
@@ -209,29 +227,29 @@ describe('Log Commands', () => {
       expect(
         setStub.calledWith('log', {
           level: 'warn',
-          path: './logs/warnings.log',
+          path: expectedAbsolutePath, 
           'show-console-logs': true,
         }),
       ).to.be.true;
 
-      expect(successMessage).to.have.length(4); // 3 individual + 1 overall success
+      expect(successMessage).to.have.length(3);
       expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.true;
       expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.true;
       expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
-      expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_SET_SUCCESS'))).to.be.true;
     });
 
-    it('should use default values when no flags provided', async () => {
+    it('should handle absolute paths correctly', async () => {
       const cmd = new LogSetCommand([], {} as any);
+      const absolutePath = '/tmp/cli.log';
+      const expectedDirectoryPath = '/tmp'; 
+      
       sinon.stub(cmd as any, 'parse').resolves({
         flags: {
-          'show-console-logs': false, // Include default value
+          path: absolutePath,
+          'show-console-logs': false,
         },
       });
 
-      sinon.stub(interactive, 'askLogLevel').resolves('info');
-      sinon.stub(interactive, 'askLogPath').resolves('./logs/app.log');
-
       sinon.stub(configHandler, 'get').returns({});
       const setStub = sinon.stub(configHandler, 'set');
 
@@ -239,13 +257,10 @@ describe('Log Commands', () => {
 
       expect(
         setStub.calledWith('log', {
-          level: 'info',
-          path: './logs/app.log',
+          path: expectedDirectoryPath,
           'show-console-logs': false,
         }),
       ).to.be.true;
-
-      expect(successMessage.some((msg) => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
     });
 
     it('should handle errors gracefully', async () => {
@@ -261,16 +276,13 @@ describe('Log Commands', () => {
   });
 
   describe('Log Get Command', () => {
-    let printMessage: string[] = [];
     let tableMessage: any[] = [];
     let errorMessage: any;
 
     beforeEach(() => {
-      printMessage = [];
       tableMessage = [];
       errorMessage = null;
 
-      sinon.stub(cliux, 'print').callsFake((msg: string) => printMessage.push(msg));
       sinon.stub(cliux, 'table').callsFake((headers: any, data: any) => {
         tableMessage.push({ headers, data });
       });
@@ -283,62 +295,141 @@ describe('Log Commands', () => {
       sinon.restore();
     });
 
-    it('should display log level and path when both are configured', async () => {
+    it('should display log level and path when both are configured with absolute path', async () => {
       const cmd = new LogGetCommand([], {} as any);
-      sinon.stub(configHandler, 'get').returns({ level: 'debug', path: './logs/app.log' });
+      const relativePath = './logs/app.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), relativePath);
+      
+      sinon.stub(configHandler, 'get').returns({ level: 'debug', path: relativePath });
 
       await cmd.run();
 
       expect(tableMessage).to.have.length(1);
-      expect(tableMessage[0].headers).to.deep.equal([{ value: 'Log Level' }, { value: 'Log Path' }]);
+      expect(tableMessage[0].headers).to.deep.equal([
+        { value: 'Setting' }, 
+        { value: 'Value' }
+      ]);
       expect(tableMessage[0].data).to.deep.equal([
         {
-          'Log Level': 'debug',
-          'Log Path': './logs/app.log',
+          'Setting': 'Log Level',
+          'Value': 'debug'
+        },
+        {
+          'Setting': 'Log Path',
+          'Value': expectedAbsolutePath
+        },
+        {
+          'Setting': 'Show Console Logs',
+          'Value': 'false'
         },
       ]);
     });
 
-    it('should display only log level when path is not configured', async () => {
+    it('should display configured level with default path', async () => {
       const cmd = new LogGetCommand([], {} as any);
       sinon.stub(configHandler, 'get').returns({ level: 'info' });
 
       await cmd.run();
 
       expect(tableMessage).to.have.length(1);
-      expect(tableMessage[0].headers).to.deep.equal([{ value: 'Log Level' }, { value: 'Log Path' }]);
+      expect(tableMessage[0].headers).to.deep.equal([
+        { value: 'Setting' }, 
+        { value: 'Value' }
+      ]);
       expect(tableMessage[0].data).to.deep.equal([
         {
-          'Log Level': 'info',
-          'Log Path': 'Not set',
+          'Setting': 'Log Level',
+          'Value': 'info'
+        },
+        {
+          'Setting': 'Log Path',
+          'Value': LOG_CONFIG_DEFAULTS.PATH
+        },
+        {
+          'Setting': 'Show Console Logs',
+          'Value': 'false'
         },
       ]);
     });
 
-    it('should display only log path when level is not configured', async () => {
+    it('should display configured path with default level', async () => {
       const cmd = new LogGetCommand([], {} as any);
-      sinon.stub(configHandler, 'get').returns({ path: './custom/logs/app.log' });
+      const customPath = './custom/logs/app.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), customPath);
+      
+      sinon.stub(configHandler, 'get').returns({ path: customPath });
 
       await cmd.run();
 
       expect(tableMessage).to.have.length(1);
-      expect(tableMessage[0].headers).to.deep.equal([{ value: 'Log Level' }, { value: 'Log Path' }]);
+      expect(tableMessage[0].headers).to.deep.equal([
+        { value: 'Setting' }, 
+        { value: 'Value' }
+      ]);
       expect(tableMessage[0].data).to.deep.equal([
         {
-          'Log Level': 'Not set',
-          'Log Path': './custom/logs/app.log',
+          'Setting': 'Log Level',
+          'Value': LOG_CONFIG_DEFAULTS.LEVEL
+        },
+        {
+          'Setting': 'Log Path',
+          'Value': expectedAbsolutePath
+        },
+        {
+          'Setting': 'Show Console Logs',
+          'Value': 'false'
         },
       ]);
     });
 
-    it('should display error message when no config is found', async () => {
+    it('should display all defaults when no config is found', async () => {
       const cmd = new LogGetCommand([], {} as any);
       sinon.stub(configHandler, 'get').returns({});
-      sinon.stub(messageHandler, 'parse').returns('No logging configuration found');
 
       await cmd.run();
 
-      expect(printMessage[0]).to.include('error: No logging configuration found');
+      expect(tableMessage).to.have.length(1);
+      expect(tableMessage[0].data).to.deep.equal([
+        {
+          'Setting': 'Log Level',
+          'Value': LOG_CONFIG_DEFAULTS.LEVEL
+        },
+        {
+          'Setting': 'Log Path',
+          'Value': LOG_CONFIG_DEFAULTS.PATH
+        },
+        {
+          'Setting': 'Show Console Logs',
+          'Value': 'false'
+        },
+      ]);
+    });
+
+    it('should display configured console logs setting', async () => {
+      const cmd = new LogGetCommand([], {} as any);
+      sinon.stub(configHandler, 'get').returns({ 
+        level: 'debug', 
+        path: '/tmp/cli.log',
+        'show-console-logs': true 
+      });
+
+      await cmd.run();
+
+      expect(tableMessage).to.have.length(1);
+      expect(tableMessage[0].data).to.deep.equal([
+        {
+          'Setting': 'Log Level',
+          'Value': 'debug'
+        },
+        {
+          'Setting': 'Log Path',
+          'Value': '/tmp/cli.log' 
+        },
+        {
+          'Setting': 'Show Console Logs',
+          'Value': 'true'
+        },
+      ]);
     });
 
     it('should handle errors gracefully', async () => {
@@ -351,4 +442,193 @@ describe('Log Commands', () => {
       expect(errorMessage).to.equal(testError);
     });
   });
+
+  describe('Log Set Command - New Functionality', () => {
+    let successMessage: string[] = [];
+    let errorMessage: any;
+
+    beforeEach(() => {
+      successMessage = [];
+      errorMessage = null;
+
+      sinon.stub(cliux, 'success').callsFake((msg: string) => successMessage.push(msg));
+      sinon.stub(cliux, 'print').callsFake((msg: string) => successMessage.push(msg));
+      sinon.stub(cliux, 'error').callsFake((_, err) => {
+        errorMessage = err;
+      });
+    });
+
+    afterEach(() => {
+      sinon.restore();
+    });
+
+    it('should use existing config values when available and no flags provided', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      const existingPath = '/existing/path/cli.log';
+      
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: { 'show-console-logs': false },
+      });
+
+      sinon.stub(configHandler, 'get').returns({ 
+        level: 'warn', 
+        path: existingPath 
+      });
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      expect(
+        setStub.calledWith('log', {
+          level: 'warn', 
+          path: existingPath, 
+          'show-console-logs': false,
+        }),
+      ).to.be.true;
+    });
+
+    it('should mix existing config with new flag values', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      const newPath = './new-logs/cli.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), './new-logs'); 
+      
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: { 
+          path: newPath,
+          'show-console-logs': true 
+        },
+      });
+
+      sinon.stub(configHandler, 'get').returns({ 
+        level: 'error' 
+      });
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      expect(
+        setStub.calledWith('log', {
+          level: 'error', 
+          path: expectedAbsolutePath, 
+          'show-console-logs': true, 
+        }),
+      ).to.be.true;
+    });
+
+    it('should handle Windows-style paths correctly with file-to-directory conversion', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      const windowsPath = 'C:\\logs\\cli.log';
+      const resolvedPath = path.resolve(process.cwd(), windowsPath);
+      const expectedPath = path.dirname(resolvedPath); 
+      
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: { 
+          path: windowsPath,
+          'show-console-logs': false 
+        },
+      });
+
+      sinon.stub(configHandler, 'get').returns({ level: 'debug' });
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      expect(
+        setStub.calledWith('log', {
+          level: 'debug', 
+          path: expectedPath, 
+          'show-console-logs': false,
+        }),
+      ).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.false;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
+    });
+
+    it('should override existing values when flags are provided with file-to-directory conversion', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      const newPath = './override/logs/cli.log';
+      const expectedAbsolutePath = path.resolve(process.cwd(), './override/logs'); // Directory, not file
+      
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: { 
+          level: 'error',
+          path: newPath,
+          'show-console-logs': true
+        },
+      });
+
+      sinon.stub(configHandler, 'get').returns({ 
+        level: 'debug', 
+        path: './old/path.log',
+        'show-console-logs': false
+      });
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      expect(
+        setStub.calledWith('log', {
+          level: 'error', 
+          path: expectedAbsolutePath,
+          'show-console-logs': true,  
+        }),
+      ).to.be.true;
+
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_LEVEL_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.true;
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_CONSOLE_SET'))).to.be.true;
+    });
+
+    it('should convert file paths to directory paths automatically', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      const filePath = './custom/logs/debug.log';
+      const expectedDirectoryPath = path.resolve(process.cwd(), './custom/logs');
+      
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: { 
+          path: filePath
+        },
+      });
+
+      sinon.stub(configHandler, 'get').returns({});
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      expect(
+        setStub.calledWith('log', {
+          path: expectedDirectoryPath,
+        }),
+      ).to.be.true;
+
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.true;
+    });
+
+    it('should keep directory paths unchanged', async () => {
+      const cmd = new LogSetCommand([], {} as any);
+      const directoryPath = './custom/logs';
+      const expectedDirectoryPath = path.resolve(process.cwd(), directoryPath);
+      
+      sinon.stub(cmd as any, 'parse').resolves({
+        flags: { 
+          path: directoryPath
+        },
+      });
+
+      sinon.stub(configHandler, 'get').returns({});
+      const setStub = sinon.stub(configHandler, 'set');
+
+      await cmd.run();
+
+      expect(
+        setStub.calledWith('log', {
+          path: expectedDirectoryPath, 
+        }),
+      ).to.be.true;
+
+      // Should show success message for path only
+      expect(successMessage.some(msg => msg.includes('CLI_CONFIG_LOG_PATH_SET'))).to.be.true;
+    });
+  });
 });
diff --git a/packages/contentstack-utilities/package.json b/packages/contentstack-utilities/package.json
index a00dbad67e..7daa516b92 100644
--- a/packages/contentstack-utilities/package.json
+++ b/packages/contentstack-utilities/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@contentstack/cli-utilities",
-  "version": "1.14.0",
+  "version": "1.14.1",
   "description": "Utilities for contentstack projects",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
diff --git a/packages/contentstack-utilities/src/logger/log.ts b/packages/contentstack-utilities/src/logger/log.ts
index 0d99ab21d0..d12a94d741 100644
--- a/packages/contentstack-utilities/src/logger/log.ts
+++ b/packages/contentstack-utilities/src/logger/log.ts
@@ -9,9 +9,15 @@ import { configHandler } from '..';
 let loggerInstance: Logger | null = null;
 
 function createLoggerInstance(): Logger {
+  const logConfig = configHandler.get('log');
+  const logLevel = logConfig?.level || 'info';
+  const showConsoleLogs = logConfig?.['show-console-logs'] ?? false;
+
   const config = {
     basePath: getLogPath(),
-    logLevel: configHandler.get('log.level') || 'info',
+    logLevel: logLevel,
+    consoleLoggingEnabled: showConsoleLogs,
+    consoleLogLevel: logLevel,
   };
   return new Logger(config);
 }
diff --git a/packages/contentstack-utilities/src/logger/logger.ts b/packages/contentstack-utilities/src/logger/logger.ts
index aff9067fb2..72cc94a412 100644
--- a/packages/contentstack-utilities/src/logger/logger.ts
+++ b/packages/contentstack-utilities/src/logger/logger.ts
@@ -115,6 +115,11 @@ export default class Logger {
   }
 
   private shouldLog(level: LogType, target: 'console' | 'file'): boolean {
+    // If console logging is disabled, don't log to console
+    if (target === 'console' && this.config.consoleLoggingEnabled === false) {
+      return false;
+    }
+    
     const configLevel = target === 'console' ? this.config.consoleLogLevel : this.config.logLevel;
     const minLevel = configLevel ? logLevels[configLevel] : 2;
     return logLevels[level] <= minLevel;
diff --git a/packages/contentstack/README.md b/packages/contentstack/README.md
index ca286fb227..0a5a656245 100644
--- a/packages/contentstack/README.md
+++ b/packages/contentstack/README.md
@@ -3574,9 +3574,10 @@ USAGE
   $ csdx config:set:log [--level debug|info|warn|error] [--path <value>] [--show-console-logs]
 
 FLAGS
-  --level=<option>          Set the log level for the CLI.
+  --level=<option>          Set the log level for the CLI. Defaults to "info" if not specified.
                             <options: debug|info|warn|error>
-  --path=<value>            Specify the file path where logs should be saved.
+  --path=<value>            Specify the directory path where logs should be saved. Supports both relative and absolute
+                            paths. Defaults to ~/.contentstack/logs if not specified.
   --[no-]show-console-logs  Enable console logging.
 
 DESCRIPTION
@@ -3585,9 +3586,19 @@ DESCRIPTION
 EXAMPLES
   $ csdx config:set:log
 
-  $ csdx config:set:log --level debug --path ./logs/app.log --show-console-logs
+  $ csdx config:set:log --level debug
+
+  $ csdx config:set:log --path ./logs
+
+  $ csdx config:set:log --level debug --path ./logs --show-console-logs
 
   $ csdx config:set:log --no-show-console-logs
+
+  $ csdx config:set:log --level warn --show-console-logs
+
+  $ csdx config:set:log --path ~/custom/logs
+
+  $ csdx config:set:log --path /var/log/contentstack
 ```
 
 _See code: [@contentstack/cli-config](https://github.com/contentstack/cli/blob/main/packages/contentstack-config/src/commands/config/set/log.ts)_
diff --git a/packages/contentstack/package.json b/packages/contentstack/package.json
index f5a0fa9453..8a23eb7dec 100755
--- a/packages/contentstack/package.json
+++ b/packages/contentstack/package.json
@@ -35,10 +35,10 @@
     "@contentstack/cli-cm-migrate-rte": "~1.6.1",
     "@contentstack/cli-cm-seed": "~1.12.1",
     "@contentstack/cli-command": "~1.6.1",
-    "@contentstack/cli-config": "~1.15.1",
+    "@contentstack/cli-config": "~1.15.2",
     "@contentstack/cli-launch": "^1.9.2",
     "@contentstack/cli-migration": "~1.8.1",
-    "@contentstack/cli-utilities": "~1.14.0",
+    "@contentstack/cli-utilities": "~1.14.1",
     "@contentstack/cli-variants": "~1.3.1",
     "@contentstack/management": "~1.22.0",
     "@oclif/core": "^4.3.0",
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index 2fd65920b1..ea416b06fd 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -25,10 +25,10 @@ importers:
       '@contentstack/cli-cm-migrate-rte': ~1.6.1
       '@contentstack/cli-cm-seed': ~1.12.1
       '@contentstack/cli-command': ~1.6.1
-      '@contentstack/cli-config': ~1.15.1
+      '@contentstack/cli-config': ~1.15.2
       '@contentstack/cli-launch': ^1.9.2
       '@contentstack/cli-migration': ~1.8.1
-      '@contentstack/cli-utilities': ~1.14.0
+      '@contentstack/cli-utilities': ~1.14.1
       '@contentstack/cli-variants': ~1.3.1
       '@contentstack/management': ~1.22.0
       '@oclif/core': ^4.3.0
@@ -2088,7 +2088,7 @@ packages:
     dependencies:
       '@apollo/client': 3.14.0_graphql@16.11.0
       '@contentstack/cli-command': 1.6.1_debug@4.4.1
-      '@contentstack/cli-utilities': 1.13.2_debug@4.4.1
+      '@contentstack/cli-utilities': 1.14.0_debug@4.4.1
       '@oclif/core': 4.5.2
       '@oclif/plugin-help': 6.2.32
       '@oclif/plugin-plugins': 5.4.46
@@ -2159,42 +2159,6 @@ packages:
       - debug
     dev: false
 
-  /@contentstack/cli-utilities/1.13.2_debug@4.4.1:
-    resolution: {integrity: sha512-jOiX99fs4XD811ipPo3N0yj/QBN63m7AIq/egzddKt39xhRTmrzdmQ/jaQdZphcl/65W6OennDdt/ThgBBtQhQ==}
-    dependencies:
-      '@contentstack/management': 1.22.0_debug@4.4.1
-      '@contentstack/marketplace-sdk': 1.4.0_debug@4.4.1
-      '@oclif/core': 4.5.2
-      axios: 1.11.0_debug@4.4.1
-      chalk: 4.1.2
-      cli-cursor: 3.1.0
-      cli-progress: 3.12.0
-      cli-table: 0.3.11
-      conf: 10.2.0
-      dotenv: 16.6.1
-      figures: 3.2.0
-      inquirer: 8.2.6
-      inquirer-search-checkbox: 1.0.0
-      inquirer-search-list: 1.2.6
-      js-yaml: 4.1.0
-      klona: 2.0.6
-      lodash: 4.17.21
-      mkdirp: 1.0.4
-      open: 8.4.2
-      ora: 5.4.1
-      papaparse: 5.5.3
-      recheck: 4.4.5
-      rxjs: 6.6.7
-      traverse: 0.6.11
-      tty-table: 4.2.3
-      unique-string: 2.0.0
-      uuid: 9.0.1
-      winston: 3.17.0
-      xdg-basedir: 4.0.0
-    transitivePeerDependencies:
-      - debug
-    dev: false
-
   /@contentstack/cli-utilities/1.14.0_debug@4.4.1:
     resolution: {integrity: sha512-dGTje9w6rGepJqJm8Fk8vIA8ja2x3AJFaNUTYIimdpMTBtA5N95STTUeV36sxKcjDaY/FmOsu0N/iZd0N2kZ5A==}
     dependencies:

From 851c54ac8daf38b30134571f97c2a40ce7b7c212 Mon Sep 17 00:00:00 2001
From: Aman Kumar <aman.kumar@contentstack.com>
Date: Fri, 5 Sep 2025 12:24:48 +0530
Subject: [PATCH 2/2] updated talismanrc file

---
 .talismanrc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.talismanrc b/.talismanrc
index 32b1415b81..d1f664b924 100644
--- a/.talismanrc
+++ b/.talismanrc
@@ -1,8 +1,8 @@
 fileignoreconfig:
 - filename: package-lock.json
-  checksum: b7a69ac42575d552fa99e0f7b62fa16485cdb8bb83efec3c5d46c22537e8bc37
+  checksum: 64a89b20371afdaaf3e901f82d7d82e342d331015efba222a32032a58063780b
 - filename: pnpm-lock.yaml
-  checksum: 62fc7744149464cc3a0b285544e90739d48e8e1384ac211d4799389c3c78db10
+  checksum: d9ad2eb48406e93e01b9db255da99bed98e29eb6db1fb5821b50f7e60b6f3efa
 - filename: packages/contentstack-import-setup/test/unit/backup-handler.test.ts
   checksum: 0582d62b88834554cf12951c8690a73ef3ddbb78b82d2804d994cf4148e1ef93
 - filename: packages/contentstack-import-setup/test/config.json