refactor: move game phase into db

packages/cli/bin/cli.dart

@@ -117,21 +117,33 @@ Future<void> cliMain(List<String> args) async {
 
   logger.info('Welcome to Space Traders! 🚀');
 
-  if (results['selloff'] as bool) {
-    logger.err('Selling all ships!');
-    config = config = Config(GamePhase.selloff);
-  }
-
   // Use package:file to make things mockable.
   const fs = LocalFileSystem();
   final db = await defaultDatabase();
-  final agentName = Platform.environment['ST_AGENT'];
-  final email = Platform.environment['ST_EMAIL'];
-  final token =
-      await loadAuthTokenOrRegister(db, agentName: agentName, email: email);
 
-  // Api client should move to per-ship with a per-ship priority function.
-  final api = apiFromAuthToken(token, db);
+  final agentSymbol =
+      await db.getAgentSymbol() ?? Platform.environment['ST_AGENT'];
+  if (agentSymbol == null) {
+    throw StateError('No agent symbol found in database or environment.');
+  }
+  final api;
+  if (await db.getAuthToken() == null) {
+    final email = Platform.environment['ST_EMAIL'];
+    logger.info('No auth token found.');
+    // Otherwise, register a new user.
+    final token = await register(db, agentSymbol: agentSymbol, email: email);
+    await db.setAuthToken(token);
+    api = apiFromAuthToken(token, db);
+  } else {
+    api = await defaultApi(db);
+  }
+
+  if (results['selloff'] as bool) {
+    logger.err('Selling all ships!');
+    db.setGamePhase(GamePhase.selloff);
+  }
+
+  config = await Config.fromDb(db);
 
   final caches = await Caches.loadOrFetch(fs, api, db);
   final marketPricesCount = await db.marketPricesCount();

packages/cli/bin/squads.dart

@@ -18,8 +18,12 @@ Future<void> command(FileSystem fs, Database db, ArgResults argResults) async {
   final charting = ChartingCache(db);
   final ships = await ShipSnapshot.load(db);
   final shipyardShipCache = ShipyardShipCache.load(fs);
+  final agentSymbol = await db.getAgentSymbol();
+  if (agentSymbol == null) {
+    throw StateError('No agent symbol found in database.');
+  }
   // TODO(eseidel): Compute the current phase or read from db.
-  config = Config(GamePhase.construction);
+  config = Config(agentSymbol, GamePhase.construction);
 
   final squads = await assignShipsToSquads(
     db,

packages/cli/lib/central_command.dart

@@ -658,7 +658,10 @@ class CentralCommand {
     final ships = await ShipSnapshot.load(db);
     final phase = _determineGamePhase(ships);
     logger.info("$phase");
-    config = Config(phase);
+    if (phase != config.gamePhase) {
+      await db.setGamePhase(phase);
+      config = await Config.fromDb(db);
+    }
 
     final marketListings = await MarketListingSnapshot.load(db);
     final shipyardListings = await ShipyardListingSnapshot.load(db);

packages/cli/lib/config.dart

@@ -1,35 +1,20 @@
+import 'package:db/db.dart';
 import 'package:types/types.dart';
 
 /// The default max age for our caches is 3 days.
 /// This is used as a default argument and must be const.
 const defaultMaxAge = Duration(days: 3);
 
-/// Which game phase are we in.
-enum GamePhase with EnumIndexOrdering {
-  /// Initially just buying haulers and getting trading going.
-  bootstrap,
-
-  /// Focused on building the jumpgate.
-  construction,
-
-  /// Focused on exploring the galaxy to find better ships.
-  exploration,
-
-  /// Sell off all our ships and retire.
-  selloff
-}
-
 /// Class for holding our hard-coded configuration values.
 class Config {
   /// Create a new Config object.
-  Config(this.gamePhase);
+  Config(this.agentSymbol, this.gamePhase);
 
   /// Which phase are we in.
   final GamePhase gamePhase;
 
-  // TODO(eseidel): This should be configured at runtime.
   /// The symbol of the agent we are controlling.
-  final String agentSymbol = 'ESEIDEL';
+  final String agentSymbol;
 
   /// Whether or not we should enable mining behaviors.
   bool get enableMining => gamePhase < GamePhase.exploration;
@@ -259,9 +244,16 @@ class Config {
 
   /// Maximum markup we will tolerate when refueling (otherwise we will drift).
   final fuelMaxMarkup = 10.0;
+
+  static Future<Config> fromDb(Database db) async {
+    final agentSymbol = await db.getAgentSymbol();
+    if (agentSymbol == null) {
+      throw StateError('No agent symbol found in database.');
+    }
+    final gamePhase = await db.getGamePhase() ?? GamePhase.bootstrap;
+    return Config(agentSymbol, gamePhase);
+  }
 }
 
 /// Our global configuration object.
-// TODO(eseidel): Correctly detect when we've finished construction but not
-// bought our second probe yet (our first one is system-watching).
-Config config = Config(GamePhase.exploration);
+late Config config;

packages/cli/lib/net/register.dart

@@ -7,26 +7,6 @@ import 'package:cli/net/exceptions.dart';
 import 'package:db/db.dart';
 import 'package:types/types.dart';
 
-/// loadAuthTokenOrRegister loads the auth token from the given file system
-/// or registers a new user and returns the auth token.
-Future<String> loadAuthTokenOrRegister(
-  Database db, {
-  String? agentName,
-  String? email,
-}) async {
-  final token = await db.getAuthToken();
-  if (token != null) {
-    return token;
-  } else {
-    logger.info('No auth token found.');
-    // Otherwise, register a new user.
-    final name = agentName ?? logger.prompt('What is your agent name?');
-    final token = await register(db, agentName: name, email: email);
-    await db.setAuthToken(token);
-    return token;
-  }
-}
-
 Future<String> _tryRegister(
   DefaultApi api, {
   required String symbol,
@@ -48,7 +28,7 @@ Future<String> _tryRegister(
 /// associated with the call sign.
 Future<String> register(
   Database db, {
-  required String agentName,
+  required String agentSymbol,
   String? email,
   String? faction,
 }) async {
@@ -75,7 +55,7 @@ Future<String> register(
   try {
     return await _tryRegister(
       defaultApi,
-      symbol: agentName,
+      symbol: agentSymbol,
       faction: chosenFaction.symbol,
       email: email,
     );
@@ -94,7 +74,7 @@ Future<String> register(
     );
     return await _tryRegister(
       defaultApi,
-      symbol: agentName,
+      symbol: agentSymbol,
       faction: chosenFaction.symbol,
       email: email,
     );

packages/db/lib/db.dart

@@ -652,6 +652,38 @@ class Database {
     return result[0][0]! as int;
   }
 
+  Future<String?> getAgentSymbol() async {
+    final result = await executeSql(
+        'SELECT value FROM config WHERE key = \'agent_symbol\'');
+    if (result.isEmpty) {
+      return null;
+    }
+    return result[0][0] as String;
+  }
+
+  Future<void> setAgentSymbol(String symbol) async {
+    await executeSql(
+        'INSERT INTO config (key, value) VALUES (\'agent_symbol\', '
+        '\'$symbol\') ON CONFLICT (key) DO UPDATE SET value = EXCLUDED.value');
+  }
+
+  Future<GamePhase?> getGamePhase() async {
+    final result =
+        await executeSql('SELECT value FROM config WHERE key = \'game_phase\'');
+    if (result.isEmpty) {
+      return null;
+    }
+    return GamePhase.values.firstWhere(
+      (phase) => phase.name == result[0][0] as String,
+      orElse: () => throw ArgumentError('Unknown game phase: ${result[0][0]}'),
+    );
+  }
+
+  Future<void> setGamePhase(GamePhase phase) async {
+    await executeSql('INSERT INTO config (key, value) VALUES (\'game_phase\', '
+        '\'$phase\') ON CONFLICT (key) DO UPDATE SET value = EXCLUDED.value');
+  }
+
   Future<String?> getAuthToken() async {
     final result =
         await executeSql('SELECT value FROM config WHERE key = \'auth_token\'');

packages/types/lib/src/game_phase.dart

@@ -0,0 +1,16 @@
+import 'package:types/enum.dart';
+
+/// Which game phase are we in.
+enum GamePhase with EnumIndexOrdering {
+  /// Initially just buying haulers and getting trading going.
+  bootstrap,
+
+  /// Focused on building the jumpgate.
+  construction,
+
+  /// Focused on exploring the galaxy to find better ships.
+  exploration,
+
+  /// Sell off all our ships and retire.
+  selloff
+}

packages/types/lib/types.dart

@@ -10,6 +10,7 @@ export 'src/counts.dart';
 export 'src/deal.dart';
 export 'src/export.dart';
 export 'src/extraction.dart';
+export 'src/game_phase.dart';
 export 'src/jump_gate.dart';
 export 'src/market_listing.dart';
 export 'src/market_price.dart';