fix(sqlite): keep shutdown database closed

Author: NathanFlurry

rivetkit-rust/packages/rivetkit-core/src/actor/sqlite.rs

@@ -1,7 +1,9 @@
 use std::collections::HashSet;
 use std::io::Cursor;
-#[cfg(feature = "sqlite-local")]
-use std::sync::Arc;
+use std::sync::{
+	Arc,
+	atomic::{AtomicBool, Ordering},
+};
 #[cfg(feature = "sqlite-local")]
 use std::time::Duration;
 
@@ -79,6 +81,7 @@ pub struct SqliteDb {
 	/// always sets up sqlite storage under the hood, so handle/actor_id are
 	/// not a reliable signal for whether the user opted in; this flag is.
 	enabled: bool,
+	shutdown_closed: Arc<AtomicBool>,
 	#[cfg(feature = "sqlite-local")]
 	// Forced-sync: native SQLite handles are used inside spawn_blocking and
 	// synchronous diagnostic accessors.
@@ -115,6 +118,7 @@ impl SqliteDb {
 			generation,
 			backend: select_sqlite_backend(enabled, remote_sqlite),
 			enabled,
+			shutdown_closed: Default::default(),
 			#[cfg(feature = "sqlite-local")]
 			db: Default::default(),
 			#[cfg(feature = "sqlite-local")]
@@ -143,17 +147,20 @@ impl SqliteDb {
 		&self,
 		request: protocol::SqliteGetPagesRequest,
 	) -> Result<protocol::SqliteGetPagesResponse> {
+		self.ensure_not_shutdown_closed()?;
 		self.handle()?.sqlite_get_pages(request).await
 	}
 
 	pub async fn commit(
 		&self,
 		request: protocol::SqliteCommitRequest,
 	) -> Result<protocol::SqliteCommitResponse> {
+		self.ensure_not_shutdown_closed()?;
 		self.handle()?.sqlite_commit(request).await
 	}
 
 	pub async fn open(&self) -> Result<()> {
+		self.ensure_not_shutdown_closed()?;
 		match self.backend {
 			SqliteBackend::LocalNative => {
 				#[cfg(feature = "sqlite-local")]
@@ -272,6 +279,7 @@ impl SqliteDb {
 	}
 
 	pub async fn exec(&self, sql: impl Into<String>) -> Result<QueryResult> {
+		self.ensure_not_shutdown_closed()?;
 		let sql = sql.into();
 		match self.backend {
 			SqliteBackend::LocalNative => self.local_exec(sql).await,
@@ -285,6 +293,7 @@ impl SqliteDb {
 		sql: impl Into<String>,
 		params: Option<Vec<BindParam>>,
 	) -> Result<QueryResult> {
+		self.ensure_not_shutdown_closed()?;
 		let sql = sql.into();
 		match self.backend {
 			SqliteBackend::LocalNative => self.local_query(sql, params).await,
@@ -300,6 +309,7 @@ impl SqliteDb {
 		sql: impl Into<String>,
 		params: Option<Vec<BindParam>>,
 	) -> Result<ExecResult> {
+		self.ensure_not_shutdown_closed()?;
 		let sql = sql.into();
 		match self.backend {
 			SqliteBackend::LocalNative => self.local_run(sql, params).await,
@@ -315,6 +325,7 @@ impl SqliteDb {
 		sql: impl Into<String>,
 		params: Option<Vec<BindParam>>,
 	) -> Result<ExecuteResult> {
+		self.ensure_not_shutdown_closed()?;
 		let sql = sql.into();
 		match self.backend {
 			SqliteBackend::LocalNative => self.local_execute(sql, params).await,
@@ -328,6 +339,7 @@ impl SqliteDb {
 		sql: impl Into<String>,
 		params: Option<Vec<BindParam>>,
 	) -> Result<ExecuteResult> {
+		self.ensure_not_shutdown_closed()?;
 		let sql = sql.into();
 		match self.backend {
 			SqliteBackend::LocalNative => self.local_execute_write(sql, params).await,
@@ -354,6 +366,7 @@ impl SqliteDb {
 	}
 
 	pub(crate) async fn cleanup(&self) -> Result<()> {
+		self.shutdown_closed.store(true, Ordering::SeqCst);
 		#[cfg(feature = "sqlite-local")]
 		{
 			self.stop_preload_hint_flush_task();
@@ -362,6 +375,13 @@ impl SqliteDb {
 		self.close().await
 	}
 
+	fn ensure_not_shutdown_closed(&self) -> Result<()> {
+		if self.shutdown_closed.load(Ordering::SeqCst) {
+			return Err(SqliteRuntimeError::Closed.build());
+		}
+		Ok(())
+	}
+
 	#[cfg(feature = "sqlite-local")]
 	fn ensure_preload_hint_flush_task(&self) -> Result<()> {
 		if !sqlite_optimization_flags().preload_hint_flush {