Improved ci behavior

bin/main.dart

@@ -7,16 +7,7 @@ import 'package:fvm/src/utils/context.dart';
 import 'package:scope/scope.dart';
 
 Future<void> main(List<String> args) async {
-  final editableArgs = List<String>.from(args);
-  final skipInput = editableArgs.remove('--fvm-skip-input');
-  final scope = Scope()
-    ..value(
-      contextKey,
-      FVMContext.create(
-        args: editableArgs,
-        skipInput: skipInput,
-      ),
-    );
+  final scope = Scope()..value(contextKey, FVMContext.create(args: args));
 
   await _flushThenExit(
     await scope.run(() async => FvmCommandRunner().run((args))),

lib/src/services/logger_service.dart

@@ -64,17 +64,17 @@ class LoggerService extends ContextService {
     return progress;
   }
 
-  bool confirm(String? message, {bool? defaultValue, bool? ciDefaultValue}) {
+  bool confirm(String? message, {required bool defaultValue}) {
     // When running tests, always return true.
     if (context.isTest) return true;
 
-    if (context.skipInput) {
-      if (ciDefaultValue != null) {
-        return ciDefaultValue;
-      } else if (defaultValue != null) {
-        return defaultValue;
-      }
-      exit(ExitCode.usage.code);
+    if (context.isCI || context.skipInput) {
+      logger.info(message ?? '');
+      logger
+        ..warn('Skipping input confirmation')
+        ..warn('Using default value of $defaultValue');
+
+      return defaultValue;
     }
 
     return interact.Confirm(prompt: message ?? '', defaultValue: defaultValue)
@@ -134,8 +134,6 @@ class LoggerService extends ContextService {
 final dot = '\u{25CF}'; // ●
 final rightArrow = '\u{2192}'; // →
 
-/// Logger for FVM
-/// Console controller instance
 final consoleController = ConsoleController();
 
 /// Console Controller

lib/src/utils/context.dart

@@ -79,7 +79,6 @@ class FVMContext with FVMContextMappable {
   static FVMContext create({
     String? id,
     List<String>? args,
-    bool? skipInput,
     AppConfig? configOverrides,
     String? workingDirectory,
     Map<Type, dynamic>? generatorOverrides,
@@ -95,13 +94,16 @@ class FVMContext with FVMContextMappable {
 
     final environment = {...Platform.environment, ...?environmentOverrides};
 
+    // Skips input if running in CI
+    final skipInput = args?.remove('--fvm-skip-input') ?? false;
+
     return FVMContext.base(
       id: id ?? 'MAIN',
       workingDirectory: workingDirectory,
       config: config,
       environment: environment,
       args: args ?? [],
-      skipInput: skipInput ?? false,
+      skipInput: skipInput,
       generators: {
         LoggerService: (context) => LoggerService(
               level: level,

lib/src/utils/deprecation_util.dart

@@ -21,6 +21,7 @@ void _warnDeprecatedEnvVars() {
     logger.info('Please use ${ConfigKeys.flutterUrl.envKey}');
     final confirmation = logger.confirm(
       'Do you want to proceed? This might impact the expected behavior.',
+      defaultValue: false,
     );
     if (!confirmation) {
       exit(ExitCode.success.code);

lib/src/workflows/ensure_cache.workflow.dart

@@ -75,7 +75,6 @@ Future<CacheFlutterVersion> ensureCacheWorkflow(
         final shouldInstallConfirmed = logger.confirm(
           'Would you like to install it now?',
           defaultValue: true,
-          ciDefaultValue: false,
         );
 
         if (!shouldInstallConfirmed) {

lib/src/workflows/use_version.workflow.dart

@@ -48,7 +48,10 @@ Future<void> useVersionWorkflow({
       logger
         ..spacer
         ..info('No pubspec.yaml detected in this directory');
-      final proceed = logger.confirm('Would you like to continue?');
+      final proceed = logger.confirm(
+        'Would you like to continue?',
+        defaultValue: true,
+      );
 
       if (!proceed) exit(ExitCode.success.code);
     }