@rahul05ranjan/dhruv-cli 1.8.1 → 1.8.2

package/CHANGELOG.md

@@ -1,9 +1,9 @@
-# [1.4.0](https://github.com/rahul05ranjan/dhruv-cli/compare/v1.8.0...v1.4.0) (2026-09-21)
+# [1.4.0](https://github.com/rahul05ranjan/dhruv-cli/compare/v1.8.1...v1.4.0) (2026-09-22)
 
 
 ### Bug Fixes
 
-* **health:** exit with code 1 on FAIL in text mode (closes [#93](https://github.com/rahul05ranjan/dhruv-cli/issues/93)) ([272fdd8](https://github.com/rahul05ranjan/dhruv-cli/commit/272fdd8e14d6f46227ee47756c1e556c49897c12))
+* **cli:** exit with code 1 on missing or unreadable input (closes [#94](https://github.com/rahul05ranjan/dhruv-cli/issues/94)) ([90c19fa](https://github.com/rahul05ranjan/dhruv-cli/commit/90c19fa926f1ef781c4137dc1713626854b2ca08))
 # Changelog
 
 ## [1.1.1](https://github.com/rahul05ranjan/dhruv-cli/compare/v1.1.0...v1.1.1) (2025-06-29)

package/__tests__/file-workflows.test.ts

@@ -231,4 +231,60 @@ describe('file analysis commands', () => {
     expect(client.requests[0].prompt).toContain('expected impact');
     expect(client.requests[0].prompt).toContain('trade-offs');
   });
+
+  it('exits with code 1 and skips AI call when review target does not exist', async () => {
+    process.exitCode = 0;
+    const nonExistent = path.join(root, 'nonexistent.ts');
+
+    await review(nonExistent);
+
+    expect(process.exitCode).toBe(1);
+    expect(client.requests).toHaveLength(0);
+    process.exitCode = 0;
+  });
+
+  it('exits with code 1 and skips AI call when review --diff has no uncommitted changes', async () => {
+    process.exitCode = 0;
+    const { execFileSync } = await import('child_process');
+    execFileSync('git', ['init'], { cwd: root });
+
+    await review(root, { diff: true });
+
+    expect(process.exitCode).toBe(1);
+    expect(client.requests).toHaveLength(0);
+    process.exitCode = 0;
+  });
+
+  it('exits with code 1 and skips AI call when security-check target does not exist', async () => {
+    process.exitCode = 0;
+    const nonExistent = path.join(root, 'nonexistent.ts');
+
+    await securityCheck(nonExistent);
+
+    expect(process.exitCode).toBe(1);
+    expect(client.requests).toHaveLength(0);
+    process.exitCode = 0;
+  });
+
+  it('exits with code 1 and skips AI call when generate target does not exist', async () => {
+    process.exitCode = 0;
+    const nonExistent = path.join(root, 'nonexistent.ts');
+
+    await generate('tests', nonExistent);
+
+    expect(process.exitCode).toBe(1);
+    expect(client.requests).toHaveLength(0);
+    process.exitCode = 0;
+  });
+
+  it('exits with code 1 and skips AI call when optimize target does not exist', async () => {
+    process.exitCode = 0;
+    const nonExistent = path.join(root, 'nonexistent.ts');
+
+    await optimize(nonExistent);
+
+    expect(process.exitCode).toBe(1);
+    expect(client.requests).toHaveLength(0);
+    process.exitCode = 0;
+  });
 });

package/dist/commands/generate.js

@@ -45,6 +45,7 @@ function extractTestCode(response) {
 export async function generate(type, target, options = {}) {
     if (!fs.existsSync(target)) {
         printError(`Target file "${target}" does not exist.`);
+        process.exitCode = 1;
         return;
     }
     const content = fs.readFileSync(target, 'utf-8');

package/dist/commands/optimize.js

@@ -21,10 +21,12 @@ function optimizationType(file) {
 export async function optimize(file) {
     if (!file || file.trim().length === 0) {
         printError('Please provide a file path to optimize.');
+        process.exitCode = 1;
         return;
     }
     if (!fs.existsSync(file)) {
         printError(`File "${file}" does not exist.`);
+        process.exitCode = 1;
         return;
     }
     let content;
@@ -33,6 +35,7 @@ export async function optimize(file) {
     }
     catch (err) {
         printError(`Error reading file "${file}": ${err.message}`);
+        process.exitCode = 1;
         return;
     }
     const type = optimizationType(file);

package/dist/commands/review.js

@@ -96,8 +96,10 @@ function readGitDiff(fileOrDir) {
 }
 export async function review(fileOrDir, options = {}) {
     const code = options.diff ? readGitDiff(fileOrDir) : readCode(fileOrDir);
-    if (code === undefined)
+    if (code === undefined) {
+        process.exitCode = 1;
         return;
+    }
     const projectRoot = fs.existsSync(fileOrDir) && fs.statSync(fileOrDir).isDirectory() ? fileOrDir : path.dirname(fileOrDir);
     const projectType = detectProjectType(projectRoot);
     const scope = options.diff ? 'the current uncommitted git diff' : 'the supplied source files';

package/dist/commands/security-check.js

@@ -128,8 +128,10 @@ function readDirectory(dir) {
 }
 export async function securityCheck(fileOrDir = '.', options = {}) {
     const code = readCode(fileOrDir);
-    if (code === undefined)
+    if (code === undefined) {
+        process.exitCode = 1;
         return;
+    }
     const findings = findHighConfidenceFindings(code);
     const safeCode = redactSensitiveContent(code);
     const findingSummary = findings.length === 0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rahul05ranjan/dhruv-cli",
-  "version": "1.8.1",
+  "version": "1.8.2",
   "description": "AI-powered CLI assistant for developers using Ollama",
   "license": "MIT",
   "keywords": [

package/src/commands/generate.ts

@@ -54,6 +54,7 @@ export interface GenerateOptions {
 export async function generate(type: string, target: string, options: GenerateOptions = {}) {
   if (!fs.existsSync(target)) {
     printError(`Target file "${target}" does not exist.`);
+    process.exitCode = 1;
     return;
   }
 

package/src/commands/optimize.ts

@@ -18,10 +18,12 @@ function optimizationType(file: string): string {
 export async function optimize(file: string) {
   if (!file || file.trim().length === 0) {
     printError('Please provide a file path to optimize.');
+    process.exitCode = 1;
     return;
   }
   if (!fs.existsSync(file)) {
     printError(`File "${file}" does not exist.`);
+    process.exitCode = 1;
     return;
   }
 
@@ -30,6 +32,7 @@ export async function optimize(file: string) {
     content = fs.readFileSync(file, 'utf-8');
   } catch (err) {
     printError(`Error reading file "${file}": ${(err as Error).message}`);
+    process.exitCode = 1;
     return;
   }
 

package/src/commands/review.ts

@@ -105,7 +105,10 @@ export interface ReviewOptions {
 
 export async function review(fileOrDir: string, options: ReviewOptions = {}) {
   const code = options.diff ? readGitDiff(fileOrDir) : readCode(fileOrDir);
-  if (code === undefined) return;
+  if (code === undefined) {
+    process.exitCode = 1;
+    return;
+  }
   const projectRoot = fs.existsSync(fileOrDir) && fs.statSync(fileOrDir).isDirectory() ? fileOrDir : path.dirname(fileOrDir);
   const projectType = detectProjectType(projectRoot);
   const scope = options.diff ? 'the current uncommitted git diff' : 'the supplied source files';

package/src/commands/security-check.ts

@@ -141,7 +141,10 @@ function readDirectory(dir: string): string | undefined {
 
 export async function securityCheck(fileOrDir: string = '.', options: SecurityCheckOptions = {}) {
   const code = readCode(fileOrDir);
-  if (code === undefined) return;
+  if (code === undefined) {
+    process.exitCode = 1;
+    return;
+  }
   const findings = findHighConfidenceFindings(code);
   const safeCode = redactSensitiveContent(code);
   const findingSummary = findings.length === 0