@rahul05ranjan/dhruv-cli 1.6.0 → 1.6.1

package/CHANGELOG.md

@@ -1,23 +1,9 @@
-# [1.4.0](https://github.com/rahul05ranjan/dhruv-cli/compare/v1.5.0...v1.4.0) (2026-09-18)
+# [1.4.0](https://github.com/rahul05ranjan/dhruv-cli/compare/v1.6.0...v1.4.0) (2026-09-21)
 
 
 ### Bug Fixes
 
-* **test:** allow health --json contract test to inspect stdout on offline service exit code ([6231027](https://github.com/rahul05ranjan/dhruv-cli/commit/623102722d5c3ac2604dfffa276868a3b7eecdb8))
-
-
-### Features
-
-* **ai:** ensure complete and streamable responses across commands (closes [#70](https://github.com/rahul05ranjan/dhruv-cli/issues/70)) ([285655e](https://github.com/rahul05ranjan/dhruv-cli/commit/285655e49eab1cbe87492cba2602a27a4bfecdfe))
-* **catalog:** unified command catalog, menu, and shell completions (closes [#79](https://github.com/rahul05ranjan/dhruv-cli/issues/79)) ([13a9c83](https://github.com/rahul05ranjan/dhruv-cli/commit/13a9c836a52933357b9ca187c787e3b9ae8f4948))
-* **cli:** standardize exit codes, timeouts, and cancellation (closes [#72](https://github.com/rahul05ranjan/dhruv-cli/issues/72)) ([d075d16](https://github.com/rahul05ranjan/dhruv-cli/commit/d075d16d1cfb3c000d156bcd238e3c261f0fc66a))
-* **cli:** standardize reliable json output contract (closes [#71](https://github.com/rahul05ranjan/dhruv-cli/issues/71)) ([e0de862](https://github.com/rahul05ranjan/dhruv-cli/commit/e0de86224b54dcf43652b5daf3f4b403d6aa7d8f))
-* **detection:** expand project detection and diagnostics (closes [#77](https://github.com/rahul05ranjan/dhruv-cli/issues/77)) ([221414a](https://github.com/rahul05ranjan/dhruv-cli/commit/221414a7c1156a650c9c746b5345b871e5274c57))
-* **diagnostics:** improve first-run model readiness and health diagnostics (closes [#78](https://github.com/rahul05ranjan/dhruv-cli/issues/78)) ([95cd3a8](https://github.com/rahul05ranjan/dhruv-cli/commit/95cd3a8d772024847a86ba53fb6391f8024ab962))
-* **generate:** previewable generation and multi-language test support (closes [#75](https://github.com/rahul05ranjan/dhruv-cli/issues/75)) ([c7f3dbe](https://github.com/rahul05ranjan/dhruv-cli/commit/c7f3dbe3902ee903220a40a4fae051b3a03797ac))
-* **metrics:** persistent invocation metrics with reset and export (closes [#76](https://github.com/rahul05ranjan/dhruv-cli/issues/76)) ([0b5c4db](https://github.com/rahul05ranjan/dhruv-cli/commit/0b5c4db69e85d10f6afeca1d26e8d9a089ce6e4c))
-* **review:** project-aware review and diff scanning (closes [#73](https://github.com/rahul05ranjan/dhruv-cli/issues/73)) ([290e4a1](https://github.com/rahul05ranjan/dhruv-cli/commit/290e4a18f21dfa9ea5a1f5fb51e2f8d98d83df53))
-* **security:** safe credential redaction and strict checks (closes [#74](https://github.com/rahul05ranjan/dhruv-cli/issues/74)) ([2467d37](https://github.com/rahul05ranjan/dhruv-cli/commit/2467d37f4404f7e16e2a10cfed860a76661d6c60))
+* **config:** keep one-off global flags in-memory and avoid writing local config (closes [#92](https://github.com/rahul05ranjan/dhruv-cli/issues/92)) ([3dd3f99](https://github.com/rahul05ranjan/dhruv-cli/commit/3dd3f99babe42d3804bfa438f38a3ddd6e1f1d72))
 # Changelog
 
 ## [1.1.1](https://github.com/rahul05ranjan/dhruv-cli/compare/v1.1.0...v1.1.1) (2025-06-29)

package/__tests__/cli-contract.test.ts

@@ -1,7 +1,10 @@
 import { describe, expect, it } from '@jest/globals';
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
-import { resolve } from 'node:path';
+import { resolve, join } from 'node:path';
+import { pathToFileURL } from 'node:url';
+import { mkdtempSync, rmSync, existsSync } from 'node:fs';
+import { tmpdir } from 'node:os';
 
 const execFileAsync = promisify(execFile);
 const repoRoot = resolve(__dirname, '..');
@@ -167,4 +170,34 @@ describe('CLI output contract', () => {
     expect(parsed).toHaveProperty('ok');
     expect(parsed.command).toBe('health');
   });
+
+  it('does not persist global flags (--json, --model, --verbose, --timeout) to .dhruv-config.json', async () => {
+    const tempDir = mkdtempSync(join(tmpdir(), 'dhruv-session-flags-'));
+    const tsNodeLoader = pathToFileURL(resolve(repoRoot, 'node_modules/ts-node/esm.mjs')).href;
+    try {
+      const result = await execFileAsync(
+        process.execPath,
+        ['--loader', tsNodeLoader, resolve(repoRoot, sourceEntry), 'metrics', '--json'],
+        {
+          cwd: tempDir,
+          env: {
+            ...process.env,
+            TS_NODE_PROJECT: resolve(repoRoot, 'tsconfig.json'),
+            DHRUV_METRICS_ENABLED: 'false',
+          },
+        },
+      );
+
+      const parsed = JSON.parse(result.stdout.trim()) as Record<string, unknown>;
+      expect(parsed).toMatchObject({
+        ok: true,
+        command: 'metrics',
+      });
+
+      const localConfigFile = join(tempDir, '.dhruv-config.json');
+      expect(existsSync(localConfigFile)).toBe(false);
+    } finally {
+      rmSync(tempDir, { recursive: true, force: true });
+    }
+  });
 });

package/__tests__/core.test.ts

@@ -5,7 +5,13 @@ import {
   setAIClient,
   InMemoryAIClient,
 } from '../src/core/ai';
-import { loadConfig, saveConfig } from '../src/config/config';
+import {
+  loadConfig,
+  saveConfig,
+  setSessionConfig,
+  resetSessionConfig,
+  getSessionConfig,
+} from '../src/config/config';
 import { createSpinner } from '../src/utils/ux';
 import { detectProjectType, detectProjectDetails } from '../src/utils/projectType';
 import { getSystemMessage } from '../src/core/prompts';
@@ -134,6 +140,58 @@ describe('Dhruv CLI Core Systems', () => {
         fs.unlinkSync(configPath);
       }
     });
+
+    it('should apply in-memory session overrides without writing to disk', () => {
+      resetSessionConfig();
+      const configPath = path.join(process.cwd(), '.dhruv-config.json');
+      const existedBefore = fs.existsSync(configPath);
+
+      setSessionConfig({
+        model: 'session-model',
+        responseFormat: 'json',
+        verbose: true,
+        timeoutMs: 12345,
+      });
+
+      expect(getSessionConfig()).toEqual({
+        model: 'session-model',
+        responseFormat: 'json',
+        verbose: true,
+        timeoutMs: 12345,
+      });
+
+      const loaded = loadConfig();
+      expect(loaded.model).toBe('session-model');
+      expect(loaded.responseFormat).toBe('json');
+      expect(loaded.verbose).toBe(true);
+      expect(loaded.timeoutMs).toBe(12345);
+
+      if (!existedBefore) {
+        expect(fs.existsSync(configPath)).toBe(false);
+      }
+
+      resetSessionConfig();
+      expect(getSessionConfig()).toEqual({});
+      const afterReset = loadConfig();
+      expect(afterReset.model).not.toBe('session-model');
+    });
+
+    it('should not persist session overrides when saving configuration', () => {
+      resetSessionConfig();
+      const configPath = path.join(process.cwd(), '.dhruv-config.json');
+      if (fs.existsSync(configPath)) fs.unlinkSync(configPath);
+
+      setSessionConfig({ responseFormat: 'json' });
+      saveConfig({ model: 'persistent-model' });
+
+      const fileContent = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+      expect(fileContent.model).toBe('persistent-model');
+      // Must not snapshot the in-memory responseFormat into the saved file
+      expect(fileContent.responseFormat).not.toBe('json');
+
+      if (fs.existsSync(configPath)) fs.unlinkSync(configPath);
+      resetSessionConfig();
+    });
   });
 
   describe('Project Type Detection', () => {

package/dist/config/config.d.ts

@@ -6,6 +6,9 @@ export interface DhruvConfig {
     timeoutMs: number;
     theme?: 'default' | 'dark' | 'light' | 'mono';
 }
+export declare function setSessionConfig(config: Partial<DhruvConfig>): void;
+export declare function resetSessionConfig(): void;
+export declare function getSessionConfig(): Partial<DhruvConfig>;
 export declare function loadConfig(): DhruvConfig;
 export declare function saveConfig(config: Partial<DhruvConfig>, options?: {
     scope?: ConfigScope;

package/dist/config/config.js

@@ -10,6 +10,16 @@ const defaultConfig = {
     timeoutMs: 45000,
     theme: 'default',
 };
+let sessionConfig = {};
+export function setSessionConfig(config) {
+    sessionConfig = { ...sessionConfig, ...config };
+}
+export function resetSessionConfig() {
+    sessionConfig = {};
+}
+export function getSessionConfig() {
+    return { ...sessionConfig };
+}
 function readConfigFile(file) {
     if (fs.existsSync(file)) {
         try {
@@ -25,11 +35,17 @@ function readConfigFile(file) {
     return defaultConfig;
 }
 export function loadConfig() {
-    if (fs.existsSync(LOCAL_CONFIG_FILE))
-        return readConfigFile(LOCAL_CONFIG_FILE);
-    if (fs.existsSync(GLOBAL_CONFIG_FILE))
-        return readConfigFile(GLOBAL_CONFIG_FILE);
-    return defaultConfig;
+    let base;
+    if (fs.existsSync(LOCAL_CONFIG_FILE)) {
+        base = readConfigFile(LOCAL_CONFIG_FILE);
+    }
+    else if (fs.existsSync(GLOBAL_CONFIG_FILE)) {
+        base = readConfigFile(GLOBAL_CONFIG_FILE);
+    }
+    else {
+        base = defaultConfig;
+    }
+    return validateAndMergeConfig({ ...base, ...sessionConfig });
 }
 function validateAndMergeConfig(config) {
     const validatedConfig = { ...defaultConfig };
@@ -57,7 +73,7 @@ function validateAndMergeConfig(config) {
 export function saveConfig(config, options = {}) {
     const scope = options.scope ?? 'local';
     const file = scope === 'global' ? GLOBAL_CONFIG_FILE : LOCAL_CONFIG_FILE;
-    const current = scope === 'global' ? readConfigFile(GLOBAL_CONFIG_FILE) : loadConfig();
+    const current = fs.existsSync(file) ? readConfigFile(file) : defaultConfig;
     if (scope === 'global')
         fs.mkdirSync(path.dirname(file), { recursive: true });
     fs.writeFileSync(file, JSON.stringify({ ...current, ...config }, null, 2));

package/dist/index.js

@@ -110,9 +110,9 @@ program
             config.responseFormat = 'json';
         if (opts.timeout)
             config.timeoutMs = Number(opts.timeout);
-        // Save config for session
+        // Set in-memory config overrides for the session
         const configModule = await import('./config/config.js');
-        configModule.saveConfig(config);
+        configModule.setSessionConfig(config);
     }
 });
 async function loadPlugins(program) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rahul05ranjan/dhruv-cli",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "description": "AI-powered CLI assistant for developers using Ollama",
   "keywords": [
     "ai",

package/src/config/config.ts

@@ -23,6 +23,20 @@ const defaultConfig: DhruvConfig = {
   theme: 'default',
 };
 
+let sessionConfig: Partial<DhruvConfig> = {};
+
+export function setSessionConfig(config: Partial<DhruvConfig>): void {
+  sessionConfig = { ...sessionConfig, ...config };
+}
+
+export function resetSessionConfig(): void {
+  sessionConfig = {};
+}
+
+export function getSessionConfig(): Partial<DhruvConfig> {
+  return { ...sessionConfig };
+}
+
 function readConfigFile(file: string): DhruvConfig {
   if (fs.existsSync(file)) {
     try {
@@ -38,9 +52,15 @@ function readConfigFile(file: string): DhruvConfig {
 }
 
 export function loadConfig(): DhruvConfig {
-  if (fs.existsSync(LOCAL_CONFIG_FILE)) return readConfigFile(LOCAL_CONFIG_FILE);
-  if (fs.existsSync(GLOBAL_CONFIG_FILE)) return readConfigFile(GLOBAL_CONFIG_FILE);
-  return defaultConfig;
+  let base: DhruvConfig;
+  if (fs.existsSync(LOCAL_CONFIG_FILE)) {
+    base = readConfigFile(LOCAL_CONFIG_FILE);
+  } else if (fs.existsSync(GLOBAL_CONFIG_FILE)) {
+    base = readConfigFile(GLOBAL_CONFIG_FILE);
+  } else {
+    base = defaultConfig;
+  }
+  return validateAndMergeConfig({ ...base, ...sessionConfig });
 }
 
 function validateAndMergeConfig(config: Partial<DhruvConfig>): DhruvConfig {
@@ -76,7 +96,7 @@ function validateAndMergeConfig(config: Partial<DhruvConfig>): DhruvConfig {
 export function saveConfig(config: Partial<DhruvConfig>, options: { scope?: ConfigScope } = {}) {
   const scope = options.scope ?? 'local';
   const file = scope === 'global' ? GLOBAL_CONFIG_FILE : LOCAL_CONFIG_FILE;
-  const current = scope === 'global' ? readConfigFile(GLOBAL_CONFIG_FILE) : loadConfig();
+  const current = fs.existsSync(file) ? readConfigFile(file) : defaultConfig;
   if (scope === 'global') fs.mkdirSync(path.dirname(file), { recursive: true });
   fs.writeFileSync(file, JSON.stringify({ ...current, ...config }, null, 2));
 }

package/src/index.ts

@@ -123,9 +123,9 @@ program
       if (opts.verbose) config.verbose = true;
       if (opts.json) config.responseFormat = 'json';
       if (opts.timeout) config.timeoutMs = Number(opts.timeout);
-      // Save config for session
+      // Set in-memory config overrides for the session
       const configModule = await import('./config/config.js');
-      configModule.saveConfig(config);
+      configModule.setSessionConfig(config);
     }
   });