feat: prompt to update CLI if version is older than latest published …

…version
invertase · Mar 30, 2022 · a88ade1 · a88ade1
1 parent 991d5a4
commit a88ade1
Show file tree

Hide file tree

Showing 5 changed files with 41 additions and 13 deletions.
diff --git a/packages/flutterfire_cli/bin/flutterfire.dart b/packages/flutterfire_cli/bin/flutterfire.dart
@@ -1,3 +1,4 @@
+// ignore_for_file: avoid_print
 /*
  * Copyright (c) 2020-present Invertase Limited & Contributors
  *
@@ -22,13 +23,35 @@ import 'package:flutterfire_cli/src/common/exception.dart';
 import 'package:flutterfire_cli/src/common/utils.dart' as utils;
 import 'package:flutterfire_cli/src/flutter_app.dart';
 import 'package:flutterfire_cli/version.g.dart';
+import 'package:pub_updater/pub_updater.dart';
 
 Future<void> main(List<String> arguments) async {
   if (arguments.contains('--version') || arguments.contains('-v')) {
-    // ignore: avoid_print
     print(cliVersion);
+    // No version checks on CIs.
+    if (utils.isCI) return;
+
+    // Check for updates.
+    final pubUpdater = PubUpdater();
+    const packageName = 'flutterfire_cli';
+    final isUpToDate = await pubUpdater.isUpToDate(
+      packageName: packageName,
+      currentVersion: cliVersion,
+    );
+    if (!isUpToDate) {
+      final latestVersion = await pubUpdater.getLatestVersion(packageName);
+      final shouldUpdate = utils.promptBool(
+        'There is a new version of $packageName available ($latestVersion). Would you like to update?',
+      );
+      if (shouldUpdate) {
+        await pubUpdater.update(packageName: packageName);
+      }
+      print('$packageName has been updated to version $latestVersion.');
+    }
+
     return;
   }
+
   try {
     final flutterApp = await FlutterApp.load(Directory.current);
     await FlutterFireCommandRunner(flutterApp).run(arguments);

diff --git a/packages/flutterfire_cli/lib/src/common/utils.dart b/packages/flutterfire_cli/lib/src/common/utils.dart
@@ -98,6 +98,16 @@ String listAsPaddedTable(List<List<String>> table, {int paddingSize = 1}) {
   return output.join('\n');
 }
 
+bool promptBool(
+  String prompt, {
+  bool defaultValue = true,
+}) {
+  return interact.Confirm(
+    prompt: prompt,
+    defaultValue: defaultValue,
+  ).interact();
+}
+
 int promptSelect(
   String prompt,
   List<String> choices, {

diff --git a/packages/flutterfire_cli/lib/src/firebase/firebase_app_id_file.dart b/packages/flutterfire_cli/lib/src/firebase/firebase_app_id_file.dart
@@ -19,7 +19,6 @@ import 'dart:convert';
 import 'dart:io';
 
 import 'package:ansi_styles/ansi_styles.dart';
-import 'package:interact/interact.dart' as interact;
 import 'package:path/path.dart';
 
 import '../common/exception.dart';
@@ -65,11 +64,9 @@ class FirebaseAppIDFile {
       // Only prompt overwrite if values are different.
       if (existingAppId != appId ||
           existingFirebaseProjectId != firebaseProjectId) {
-        final shouldOverwrite = interact.Confirm(
-          prompt:
-              'Generated FirebaseAppID file ${AnsiStyles.cyan(appIDFilePath)} already exists (for app id "$existingAppId" on Firebase Project "$existingFirebaseProjectId"), do you want to override it?',
-          defaultValue: true,
-        ).interact();
+        final shouldOverwrite = promptBool(
+          'Generated FirebaseAppID file ${AnsiStyles.cyan(appIDFilePath)} already exists (for app id "$existingAppId" on Firebase Project "$existingFirebaseProjectId"), do you want to override it?',
+        );
         if (!shouldOverwrite) {
           throw FirebaseAppIDAlreadyExistsException(appIDFilePath);
         }

diff --git a/packages/flutterfire_cli/lib/src/firebase/firebase_configuration_file.dart b/packages/flutterfire_cli/lib/src/firebase/firebase_configuration_file.dart
@@ -18,7 +18,6 @@
 import 'dart:io';
 
 import 'package:ansi_styles/ansi_styles.dart';
-import 'package:interact/interact.dart' as interact;
 import 'package:path/path.dart';
 
 import '../common/exception.dart';
@@ -58,11 +57,9 @@ class FirebaseConfigurationFile {
       final existingFileContents = await outputFile.readAsString();
       // Only prompt overwrite if contents have changed.
       if (existingFileContents != newFileContents) {
-        final shouldOverwrite = interact.Confirm(
-          prompt:
-              'Generated FirebaseOptions file ${AnsiStyles.cyan(outputFilePath)} already exists, do you want to override it?',
-          defaultValue: true,
-        ).interact();
+        final shouldOverwrite = promptBool(
+          'Generated FirebaseOptions file ${AnsiStyles.cyan(outputFilePath)} already exists, do you want to override it?',
+        );
         if (!shouldOverwrite) {
           throw FirebaseOptionsAlreadyExistsException(outputFilePath);
         }

diff --git a/packages/flutterfire_cli/pubspec.yaml b/packages/flutterfire_cli/pubspec.yaml
@@ -16,6 +16,7 @@ dependencies:
   meta: ^1.7.0
   path: ^1.8.0
   platform: ^3.0.2
+  pub_updater: ^0.2.2
   pubspec: ^2.0.1
   xml: ^5.3.1