🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

Greptile Overview

Greptile Summary

This PR fixes a bug where the CLI would prompt users to select a hosting option even when the --domain flag was explicitly provided. The fix enables non-interactive login flows (e.g., for Terraform) by checking whether the domain flag was explicitly set using cmd.Flags().Changed("domain").

Key changes:

• Added domainFlagExplicitlySet boolean to detect when user explicitly provides --domain flag
• Modified shouldUsePresetDomain logic to bypass prompt when domain flag is explicitly set, even for US/EU cloud URLs
• Added check to prevent saving default cloud URLs (US/EU) to the config file's domains list

How it works:
The updated logic uses cmd.Flags().Changed("domain") to distinguish between:

1. User explicitly setting --domain "https://eu.infisical.com" → uses that domain directly without prompting
2. No --domain flag (uses default) → shows hosting selection prompt as before
3. --domain set to custom self-hosted URL → uses it directly (existing behavior preserved)

The fix properly handles the edge case where cloud URLs should be used when explicitly specified but not added to the persistent config file.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk - it's a focused bug fix with clear logic
• The implementation is straightforward and correctly addresses the issue. The boolean logic is sound, handling all edge cases properly (explicit flag vs default, cloud vs self-hosted). The fix doesn't introduce security vulnerabilities and maintains backward compatibility for users who don't use the --domain flag.
• No files require special attention

Important Files Changed