Merge pull request #363 from leoafarias/feature/detached-notice

Command detach check

lib/constants.dart

@@ -50,3 +50,7 @@ String get kFvmHome {
 
 /// Flutter Channels
 const kFlutterChannels = ['master', 'stable', 'dev', 'beta'];
+/// List of Flutter/Dart commands that need to run detached to avoid fvm errors.
+const kDetachedCommands = [
+  'pub cache repair',
+];

lib/src/utils/commands.dart

@@ -72,7 +72,8 @@ Future<int> _runCmd(
   Map<String, String>? environment,
 }) async {
   // Project again a non executable path
-  await Guards.canExecute(execPath);
+  await Guards.canExecute(execPath, args);
+
   final processManager = ProcessManager();
 
   // Switch off line mode

lib/src/utils/guards.dart

@@ -1,13 +1,49 @@
+import 'dart:io' show Platform;
+
 import 'package:io/io.dart';
 
+import '../../constants.dart';
 import '../../exceptions.dart';
+import 'logger.dart';
 
 /// Guards against certain action by validatin and throwing errors
 class Guards {
+  Guards._();
+
   /// Check if can execute path or throws error
-  static Future<void> canExecute(String execPath) async {
+  static Future<void> canExecute(String execPath, List<String> args) async {
     if (!await isExecutable(execPath)) {
       throw FvmInternalError('Cannot execute $execPath');
     }
+    if (Guards.shouldRunDetached(args)) {
+      logger.trace(Platform.script.path);
+      FvmLogger.spacer();
+      FvmLogger.info(
+        'This command ${args.join(" ")} will modify FVM installation.',
+      );
+      FvmLogger.info(
+        '''Because of that is suggested you run the following command in your terminal directly''',
+      );
+      FvmLogger.spacer();
+      FvmLogger.fine("$execPath ${args.join(' ')}");
+
+      FvmLogger.spacer();
+
+      FvmLogger.info(
+        '''If after this command FVM cannot be found in your terminal. Please run the following:''',
+      );
+
+      FvmLogger.spacer();
+      FvmLogger.fine("$execPath pub global activate fvm");
+
+      throw FvmUsageException('Command needs to run outside of FVM proxy');
+    }
+  }
+
+  /// Check if command needs to be run detached
+  static bool shouldRunDetached(List<String> args) {
+    final argString = args.join(' ');
+
+    return kDetachedCommands.any(argString.contains);
   }
 }