refactor(server): rename sql flag to raw_sql

Author: Larkooo

crates/cli/src/options.rs

@@ -336,7 +336,7 @@ pub struct ServerOptions {
         default_value_t = true,
         help = "Enable the SQL playground and query endpoint at /sql."
     )]
-    pub sql_api_enabled: bool,
+    pub raw_sql: bool,
 
     /// Path to the SSL certificate file (.pem)
     #[arg(
@@ -368,7 +368,7 @@ impl Default for ServerOptions {
             http_addr: DEFAULT_HTTP_ADDR,
             http_port: DEFAULT_HTTP_PORT,
             http_cors_origins: None,
-            sql_api_enabled: true,
+            raw_sql: true,
             tls_cert_path: None,
             tls_key_path: None,
             mkcert: false,

crates/grpc/server/src/lib.rs

@@ -1183,7 +1183,7 @@ impl<P: Provider + Sync + Send + 'static> proto::world::world_server::World for
         &self,
         request: Request<proto::types::SqlQueryRequest>,
     ) -> Result<Response<proto::types::SqlQueryResponse>, Status> {
-        if !self._config.sql_api_enabled {
+        if !self._config.raw_sql {
             return Err(Status::permission_denied("SQL endpoint is disabled."));
         }
 
@@ -1231,7 +1231,7 @@ pub struct GrpcConfig {
     pub http2_keepalive_interval: Duration,
     pub http2_keepalive_timeout: Duration,
     pub max_message_size: usize,
-    pub sql_api_enabled: bool,
+    pub raw_sql: bool,
 }
 
 impl Default for GrpcConfig {
@@ -1243,7 +1243,7 @@ impl Default for GrpcConfig {
             http2_keepalive_interval: Duration::from_secs(30),
             http2_keepalive_timeout: Duration::from_secs(10),
             max_message_size: 16 * 1024 * 1024,
-            sql_api_enabled: true,
+            raw_sql: true,
         }
     }
 }

crates/runner/src/lib.rs

@@ -740,7 +740,7 @@ impl Runner {
                     self.args.grpc.http2_keepalive_timeout,
                 ),
                 max_message_size: self.args.grpc.max_message_size,
-                sql_api_enabled: self.args.server.sql_api_enabled,
+                raw_sql: self.args.server.raw_sql,
             },
             Some(grpc_bind_addr),
         )
@@ -758,7 +758,7 @@ impl Runner {
             None,
             absolute_path.clone(),
             Arc::new(readonly_pool.clone()),
-            self.args.server.sql_api_enabled,
+            self.args.server.raw_sql,
             storage.clone(),
             provider.clone(),
             self.version_spec.clone(),
@@ -839,7 +839,7 @@ impl Runner {
         info!(target: LOG_TARGET, endpoint = %addr, protocol = %protocol, "Starting torii endpoint.");
         info!(target: LOG_TARGET, endpoint = %grpc_addr, "Serving gRPC endpoint.");
         info!(target: LOG_TARGET, endpoint = %gql_endpoint, "Serving Graphql playground.");
-        if self.args.server.sql_api_enabled {
+        if self.args.server.raw_sql {
             info!(target: LOG_TARGET, endpoint = %sql_endpoint, "Serving SQL playground.");
         } else {
             info!(target: LOG_TARGET, "SQL endpoint is disabled.");

crates/server/src/proxy.rs

@@ -167,7 +167,7 @@ impl<P: Provider + Sync + Send + Debug + 'static> Proxy<P> {
         graphql_addr: Option<SocketAddr>,
         artifacts_dir: Utf8PathBuf,
         pool: Arc<SqlitePool>,
-        sql_api_enabled: bool,
+        raw_sql: bool,
         storage: Arc<S>,
         provider: P,
         version_spec: String,
@@ -186,7 +186,7 @@ impl<P: Provider + Sync + Send + Debug + 'static> Proxy<P> {
             Box::new(GrpcHandler::new(grpc_addr, grpc_proxy_client.clone())),
             Box::new(McpHandler::new(pool.clone())),
             Box::new(MetadataHandler::new(storage.clone(), provider)),
-            Box::new(SqlHandler::new(pool.clone(), sql_api_enabled)),
+            Box::new(SqlHandler::new(pool.clone(), raw_sql)),
             Box::new(StaticHandler::new(artifacts_dir, (*pool).clone())),
         ];