[IMP] Do not change database structure at every startup

Instead, only load table definitions and constraints when the table
is created. This only impacts Odev's database, used for internal
operations.

Performance gains:
- Framework loaded: 5ms (1.3%)
- Framework started: 48ms (6.4%)
- Execution completed: 46ms (5.5%)

Author: brinkflew

odev/common/postgres.py

@@ -158,25 +158,24 @@ def __init__(self, database: PostgresDatabase, name: str | None = None):
         self.name: str = name or self.name
         """Name of the table in which data is stored, must be set in subclass."""
 
-        with self.database:
-            self.prepare_database_table()
-
         self.database.tables[self.name] = self
 
     def prepare_database_table(self):
         """Prepare the table and ensures it has the correct definition and constraints applied."""
         if self._columns is not None:
-            if not self.database.table_exists(self.name):
-                logger.debug(f"Creating table {self.name!r} in database {self.database!r}")
-                self.database.create_table(self.name, self._columns)
-            elif missing_columns := self.database.columns_exist(self.name, list(self._columns.keys())):
+            logger.debug(f"Creating table {self.name!r} in database {self.database!r}")
+            self.database.create_table(self.name, self._columns)
+
+            if missing_columns := self.database.columns_exist(self.name, list(self._columns.keys())):
                 for column in missing_columns:
                     self.__add_missing_column(column)
 
-        if self._constraints is not None:
-            for name, definition in self._constraints.items():
-                logger.debug(f"Creating constraint {name!r} on table {self.name!r} in database {self.database.name!r}")
-                self.database.constraint(self.name, name, definition)
+            if self._constraints is not None:
+                for name, definition in self._constraints.items():
+                    logger.debug(
+                        f"Creating constraint {name!r} on table {self.name!r} in database {self.database.name!r}"
+                    )
+                    self.database.constraint(self.name, name, definition)
 
     def clear(self):
         """Clear the table."""

odev/common/store/datastore.py

@@ -27,8 +27,14 @@ def __init__(self, name: str = "odev"):
         self.databases = DatabaseStore(self)
         self.history = HistoryStore(self)
         self.secrets = SecretStore(self)
+        self.__load_tables()
         self.__load_plugins_tables()
 
+    def __load_tables(self):
+        for table in self.tables.values():
+            if not self.table_exists(table.name):
+                table.prepare_database_table()
+
     def __load_plugins_tables(self):
         odev_path = Path(__file__).parents[2]
         plugins = [path for path in (odev_path / "plugins").glob("*/datastore") if path.is_dir()]