PR changes

Author: agusayerza

package-lock.json

@@ -16,7 +16,6 @@ import { nangoConfigFile } from '@nangohq/nango-yaml';
 
 import { initAI } from './ai/init.js';
 import { generate, getVersionOutput, tscWatch } from './cli.js';
-import { MissingArgumentError } from './errors.js';
 import { migrateToZeroYaml } from './migrations/toZeroYaml.js';
 import { compileAllFiles } from './services/compile.service.js';
 import { parse } from './services/config.service.js';
@@ -28,6 +27,7 @@ import { create } from './services/function-create.service.js';
 import { directoryMigration, endpointMigration, v1toV2Migration } from './services/migration.service.js';
 import { generateTests } from './services/test.service.js';
 import verificationService from './services/verification.service.js';
+import { MissingArgumentError } from './utils/errors.js';
 import { NANGO_INTEGRATIONS_LOCATION, getNangoRootPath, isCI, printDebug, upgradeAction } from './utils.js';
 import { checkAndSyncPackageJson } from './zeroYaml/check.js';
 import { compileAll } from './zeroYaml/compile.js';
@@ -55,6 +55,13 @@ class NangoCommand extends Command {
 
             // opts.interactive is true by default (from the option default), or false if --no-interactive is passed.
             // We also disable it if we are in a CI environment.
+            if (isCI && opts.interactive) {
+                console.warn(
+                    chalk.yellow(
+                        "CI environment detected. Interactive mode has been automatically disabled to prevent hanging. Pass '--no-interactive' to silence this warning."
+                    )
+                );
+            }
             opts.interactive = opts.interactive && !isCI;
 
             printDebug(`Running in ${opts.interactive ? 'interactive' : 'non-interactive'} mode.`, opts.debug);

packages/cli/lib/errors.ts

@@ -7,7 +7,7 @@ import {
     promptForIntegrationName,
     promptForProjectPath
 } from './interactive.service.js';
-import { MissingArgumentError } from '../errors.js';
+import { MissingArgumentError } from '../utils/errors.js';
 
 import type { FunctionType } from '../types.js';
 

packages/cli/lib/index.ts

@@ -2,7 +2,7 @@ import { describe, expect, it, vi } from 'vitest';
 
 import { Ensure } from './ensure.service.js';
 import * as interactive from './interactive.service.js';
-import { MissingArgumentError } from '../errors.js';
+import { MissingArgumentError } from '../utils/errors.js';
 
 describe('Ensure', () => {
     it('should return current value if it exists', async () => {
@@ -27,7 +27,7 @@ describe('Ensure', () => {
     it('should throw error if prompt fails', async () => {
         const ensure = new Ensure(true);
         vi.spyOn(interactive, 'promptForProjectPath').mockRejectedValue(new Error('Prompt failed'));
-        await expect(ensure.projectPath(undefined)).rejects.toThrow('Interactive prompt cancelled.');
+        await expect(ensure.projectPath(undefined)).rejects.toThrow('Prompt failed');
     });
 
     it('should throw TTY error if prompt fails with isTtyError', async () => {

packages/cli/lib/services/ensure.service.ts

@@ -43,3 +43,10 @@ export class CLIError extends Error {
         this.code = code;
     }
 }
+
+export class MissingArgumentError extends Error {
+    constructor(message: string) {
+        super(message);
+        this.name = 'MissingArgumentError';
+    }
+}

packages/cli/lib/services/ensure.service.unit.test.ts

@@ -1,6 +1,6 @@
 {
     "name": "nango",
-    "version": "0.70.0",
+    "version": "0.69.20",
     "description": "Nango's CLI tool.",
     "type": "module",
     "bin": {
@@ -84,7 +84,7 @@
         "@types/commander": "2.12.5",
         "@types/ejs": "3.1.5",
         "@types/figlet": "1.5.6",
-        "@types/inquirer": "^9.0.7",
+        "@types/inquirer": "9.0.7",
         "@types/jscodeshift": "17.3.0",
         "@types/json-schema": "7.0.15",
         "@types/node": "22.15.29",