feat(cli): make CLI not interactive in agentic context (#2319)

Synced from monorepo@abf8f7b089b6c55e6ba278675a574a735bc680b1

Author: divyenduz

.sync-commit

@@ -1 +1 @@
-055633d07c494cc7355400f5ec7dfc86997870ff
+abf8f7b089b6c55e6ba278675a574a735bc680b1

pnpm-lock.yaml

@@ -23,11 +23,13 @@ async function resolveCustomConfig(
   context: LocalContext,
   flags: Pick<Flags, 'custom-issuer' | 'custom-api-base-url' | 'custom-client-id' | 'custom-client-secret'>
 ): Promise<CustomConfig | undefined> {
-  const issuer = await context.enquirer.inputPrompt(context.isCI, 'Issuer URL', { flag: flags['custom-issuer'] });
-  const apiBaseUrl = await context.enquirer.inputPrompt(context.isCI, 'API base URL', {
+  const issuer = await context.enquirer.inputPrompt(context.isInteractive, 'Issuer URL', {
+    flag: flags['custom-issuer']
+  });
+  const apiBaseUrl = await context.enquirer.inputPrompt(context.isInteractive, 'API base URL', {
     flag: flags['custom-api-base-url']
   });
-  const clientSecret = await context.enquirer.inputPrompt(context.isCI, 'Client secret', {
+  const clientSecret = await context.enquirer.inputPrompt(context.isInteractive, 'Client secret', {
     flag: flags['custom-client-secret']
   });
 

src/commands/auth/login.ts

@@ -20,7 +20,7 @@ export async function implementation(this: LocalContext, { profile: profileFlag,
 
   if (!yes) {
     const confirmFromPrompt = await this.enquirer.confirmPrompt(
-      this.isCI,
+      this.isInteractive,
       `Do you want to log out of the profile ${profile}`
     );
     if (!confirmFromPrompt) {

src/commands/auth/logout.ts

@@ -25,7 +25,7 @@ export async function implementation(this: LocalContext, {}: Flags, profile?: st
       message: profile === activeProfile ? `${profile} (current)` : profile
     }));
 
-    selectedProfile = await this.enquirer.selectPrompt(this.isCI, 'Select profile to switch to:', choices);
+    selectedProfile = await this.enquirer.selectPrompt(this.isInteractive, 'Select profile to switch to:', choices);
 
     if (!selectedProfile) {
       console.log('No profile selected');

src/commands/auth/switch.ts

@@ -146,7 +146,11 @@ export async function getRegion(context: LocalContext, flags: { region?: string
 
   if (!flags.region) {
     const regionChoices = groupAndSortRegions(regions.regions);
-    const region = (await context.enquirer.selectPrompt(context.isCI, title, regionChoices)) as Flags['region'];
+    const region = (await context.enquirer.selectPrompt(
+      context.isInteractive,
+      title,
+      regionChoices
+    )) as Flags['region'];
     invariant(region, `Region should exist`);
     return region;
   }
@@ -171,7 +175,11 @@ export async function getReplicas(context: LocalContext, flags: { replicas?: str
   }
 
   if (!flags.replicas) {
-    const replicas = (await context.enquirer.selectPrompt(context.isCI, title, replicaChoices)) as Flags['replicas'];
+    const replicas = (await context.enquirer.selectPrompt(
+      context.isInteractive,
+      title,
+      replicaChoices
+    )) as Flags['replicas'];
     invariant(replicas, `Replicas should exist`);
     return replicas;
   }
@@ -203,7 +211,7 @@ export async function getInstanceType(
 
   if (!flags['instance-type']) {
     const instanceType = (await context.enquirer.selectPrompt(
-      context.isCI,
+      context.isInteractive,
       title,
       instanceChoices
     )) as Flags['instance-type'];
@@ -254,7 +262,9 @@ export async function getImage(
     const defaultIndex = sortedImages.findIndex((image) => image.name.startsWith('postgres:'));
     const initial = defaultIndex >= 0 ? defaultIndex : 0;
 
-    const image = (await context.enquirer.selectPrompt(context.isCI, title, imageChoices, { initial })) as string;
+    const image = (await context.enquirer.selectPrompt(context.isInteractive, title, imageChoices, {
+      initial
+    })) as string;
     invariant(image, `Postgres version should exist`);
     return image;
   } catch {
@@ -278,7 +288,9 @@ export async function implementation(this: LocalContext, flags: Flags) {
   const scaleToZeroChild = project.configuration.scaleToZero.childBranches.enabled;
   const inactivityPeriodChild = project.configuration.scaleToZero.childBranches.inactivityPeriodMinutes;
 
-  const branchName = await this.enquirer.inputPrompt(this.isCI, 'Please enter the branch name', { flag: flags.name });
+  const branchName = await this.enquirer.inputPrompt(this.isInteractive, 'Please enter the branch name', {
+    flag: flags.name
+  });
   if (!branchName) {
     this.process.stderr.write(chalk.red(`Expected input for flag --name`));
     this.process.exit(1);

src/commands/branch/create.ts

@@ -32,7 +32,7 @@ export async function implementation(this: LocalContext, flags: Flags, branchNam
 
   if (!flags.yes) {
     const confirmFromPrompt = await this.enquirer.confirmPrompt(
-      this.isCI,
+      this.isInteractive,
       `Are you sure you want to delete the branch ${branchToDelete.name} with id ${branchToDelete.id}?`
     );
     if (!confirmFromPrompt) {

src/commands/branch/delete.ts

@@ -35,7 +35,7 @@ export async function implementation(this: LocalContext, flags: Flags, branchNam
 
   if (!flags.yes) {
     const confirmed = await this.enquirer.confirmPrompt(
-      this.isCI,
+      this.isInteractive,
       `Rotate the password for PostgreSQL user ${username} on branch ${branch.name}? Existing connections will not be affected, but any new connections will need to use the new credentials.`
     );
     if (!confirmed) {

src/commands/branch/rotate-password.ts

@@ -113,45 +113,45 @@ export async function implementation(this: LocalContext, flags: Flags, fieldArg:
   if (!value) {
     value = await match(field)
       .with('name', async () => {
-        return await this.enquirer.inputPrompt(this.isCI, 'Please enter the new branch name');
+        return await this.enquirer.inputPrompt(this.isInteractive, 'Please enter the new branch name');
       })
       .with('replicas', async () => {
         return await this.enquirer.selectPrompt(
-          this.isCI,
+          this.isInteractive,
           'Please select number of replicas for the branch',
           replicaChoices
         );
       })
       .with('instance-type', async () => {
         return await this.enquirer.selectPrompt(
-          this.isCI,
+          this.isInteractive,
           'Please select the type of instance for this branch',
           instanceChoices
         );
       })
       .with('hibernate', async () => {
-        return await this.enquirer.selectPrompt(this.isCI, 'Please select hibernation status for the branch', [
+        return await this.enquirer.selectPrompt(this.isInteractive, 'Please select hibernation status for the branch', [
           { name: 'true', message: 'Hibernated' },
           { name: 'false', message: 'Not hibernated' }
         ]);
       })
       .with('scale-to-zero', async () => {
         return await this.enquirer.selectPrompt(
-          this.isCI,
+          this.isInteractive,
           `Please select scale to zero status for the branch (current: ${defaultScaleToZero ? 'Enabled' : 'Disabled'})`,
           [...scaleToZeroChoices]
         );
       })
       .with('inactivity-period', async () => {
         return await this.enquirer.selectPrompt(
-          this.isCI,
+          this.isInteractive,
           `Please select inactivity period for the branch (current: ${defaultInactivityPeriod} minutes)`,
           [...timeChoices]
         );
       })
       .with('postgres-version', async () => {
         return await this.enquirer.selectPrompt(
-          this.isCI,
+          this.isInteractive,
           `Please select the PostgreSQL version for this branch (current: ${branch.configuration.image})`,
           upgradeableImageChoices
         );

src/commands/branch/set.ts

@@ -63,7 +63,7 @@ type Flags = {
 };
 
 export async function implementation(this: LocalContext, flags: Flags) {
-  if (this.isCI && flags.mode !== 'auto') {
+  if (!this.isInteractive && flags.mode !== 'auto') {
     invariant(false, 'Only --mode `auto` is available in CI');
   }
   await checkBranchIsReachable(this, flags);
@@ -76,10 +76,14 @@ export async function implementation(this: LocalContext, flags: Flags) {
 
   let validationMode: ValidationMode = 'strict';
   if (flags['validation-mode'] === 'prompt') {
-    const validationModeResult = (await this.enquirer.selectPrompt(this.isCI, 'Do you want to anonymize data?', [
-      { name: 'strict', message: 'Yes' },
-      { name: 'relaxed', message: 'No' }
-    ])) as ValidationMode;
+    const validationModeResult = (await this.enquirer.selectPrompt(
+      this.isInteractive,
+      'Do you want to anonymize data?',
+      [
+        { name: 'strict', message: 'Yes' },
+        { name: 'relaxed', message: 'No' }
+      ]
+    )) as ValidationMode;
     if (validationModeResult) {
       validationMode = validationModeResult;
     }