Add AppState: Sync constraint to factors

Signed-off-by: Lann Martin <[email protected]>

Author: lann

crates/factors-executor/src/lib.rs

@@ -34,10 +34,7 @@ impl<T: RuntimeFactors, U: Send + 'static> FactorsExecutor<T, U> {
     }
 }
 
-impl<T: RuntimeFactors, U: Send + 'static> FactorsExecutor<T, U>
-where
-    T::AppState: Sync,
-{
+impl<T: RuntimeFactors, U: Send + 'static> FactorsExecutor<T, U> {
     /// Adds the given [`ExecutorHooks`] to this executor.
     ///
     /// Hooks are run in the order they are added.
@@ -84,7 +81,6 @@ where
 pub trait ExecutorHooks<T, U>: Send + Sync
 where
     T: RuntimeFactors,
-    T::AppState: Sync,
 {
     /// Configure app hooks run immediately after [`RuntimeFactors::configure_app`].
     async fn configure_app(&mut self, configured_app: &ConfiguredApp<T>) -> anyhow::Result<()> {
@@ -143,10 +139,7 @@ impl<T: RuntimeFactors, U: Send + 'static> FactorsExecutorApp<T, U> {
     }
 }
 
-impl<T: RuntimeFactors, U: Send + 'static> FactorsExecutorApp<T, U>
-where
-    T::AppState: Sync,
-{
+impl<T: RuntimeFactors, U: Send + 'static> FactorsExecutorApp<T, U> {
     /// Returns an instance builder for the given component ID.
     pub fn prepare(&self, component_id: &str) -> anyhow::Result<FactorsInstanceBuilder<T, U>> {
         let app_component = self

crates/factors/src/factor.rs

@@ -17,7 +17,7 @@ pub trait Factor: Any + Sized {
     /// The application state of this factor.
     ///
     /// This state *may* be cached by the runtime across multiple requests.
-    type AppState;
+    type AppState: Sync;
 
     /// The builder of instance state for this factor.
     type InstanceBuilder: FactorInstanceBuilder;

crates/factors/src/runtime_factors.rs

@@ -32,7 +32,7 @@ use crate::{factor::FactorInstanceState, App, ConfiguredApp, Factor};
 /// ```
 pub trait RuntimeFactors: Sized + 'static {
     /// The per application state of all the factors.
-    type AppState;
+    type AppState: Sync;
     /// The per instance state of the factors.
     type InstanceState: RuntimeFactorsInstanceState;
     /// The collection of all the `InstanceBuilder`s of the factors.

crates/trigger/src/cli/sqlite_statements.rs

@@ -69,7 +69,6 @@ impl SqlStatementExecutorHook {
 impl<F, U> ExecutorHooks<F, U> for SqlStatementExecutorHook
 where
     F: RuntimeFactors,
-    F::AppState: Sync,
 {
     async fn configure_app(
         &mut self,