Simplify features

crates/utils/Cargo.toml

@@ -23,20 +23,6 @@ workspace = true
 
 [features]
 default = [
-  "error",
-  "request",
-  "response",
-  "settings",
-  "misc",
-  "rate-limit",
-  "apub",
-  "cache-header",
-  "email",
-]
-full = ["default", "ts-rs"]
-ts-rs = ["dep:ts-rs"]
-error-type = ["dep:serde", "dep:strum"]
-error = [
   "error-type",
   "dep:serde_json",
   "dep:anyhow",
@@ -44,11 +30,9 @@ error = [
   "dep:diesel",
   "dep:http",
   "dep:actix-web",
-]
+  "dep:reqwest-middleware",
-request = ["dep:reqwest-middleware", "dep:tracing", "dep:strum"]
+  "dep:tracing",
-response = ["error", "dep:actix-web"]
+  "dep:actix-web",
-settings = [
-  "error",
   "dep:deser-hjson",
   "dep:regex",
   "dep:urlencoding",
@@ -56,34 +40,20 @@ settings = [
   "dep:url",
   "dep:once_cell",
   "dep:smart-default",
-]
-rate-limit = [
-  "error",
   "dep:enum-map",
-  "dep:tracing",
-  "dep:actix-web",
   "dep:futures",
   "dep:tokio",
-  "dep:once_cell",
+  "dep:openssl",
-]
-apub = ["dep:openssl"]
-cache-header = ["dep:actix-web"]
-email = [
-  "error",
-  "settings",
   "dep:html2text",
   "dep:lettre",
   "dep:uuid",
   "dep:rosetta-i18n",
-]
-misc = [
-  "settings",
   "dep:itertools",
   "dep:markdown-it",
-  "dep:tokio",
+
-  "dep:futures",
-  "dep:tracing",
 ]
+full = ["default", "dep:ts-rs"]
+error-type = ["dep:serde", "dep:strum"]
 
 [dependencies]
 regex = { workspace = true, optional = true }

crates/utils/src/error.rs

@@ -2,7 +2,7 @@ use cfg_if::cfg_if;
 use serde::{Deserialize, Serialize};
 use std::fmt::Debug;
 use strum_macros::{Display, EnumIter};
-#[cfg(feature = "ts-rs")]
+#[cfg(feature = "full")]
 use ts_rs::TS;
 
 #[derive(Display, Debug, Serialize, Deserialize, Clone, PartialEq, Eq, EnumIter, Hash)]
@@ -168,7 +168,7 @@ pub enum LemmyErrorType {
 }
 
 cfg_if! {
-  if #[cfg(feature = "error")] {
+  if #[cfg(feature = "default")] {
 
     use tracing_error::SpanTrace;
     use std::fmt;

crates/utils/src/lib.rs

@@ -1,22 +1,27 @@
-#[cfg(feature = "apub")]
+use cfg_if::cfg_if;
-pub mod apub;
+
-#[cfg(feature = "cache-header")]
+cfg_if! {
-pub mod cache_header;
+  if #[cfg(feature = "default")] {
-#[cfg(feature = "email")]
+    pub mod apub;
-pub mod email;
+    pub mod cache_header;
-#[cfg(feature = "error-type")]
+    pub mod email;
-pub mod error;
+    pub mod error;
-#[cfg(feature = "rate-limit")]
+    pub mod rate_limit;
-pub mod rate_limit;
+    pub mod request;
-#[cfg(feature = "request")]
+    pub mod response;
-pub mod request;
+    pub mod settings;
-#[cfg(feature = "response")]
+    pub mod utils;
-pub mod response;
+    pub mod version;
-#[cfg(feature = "settings")]
+  } else {
-pub mod settings;
+    mod error;
-#[cfg(feature = "misc")]
+  }
-pub mod utils;
+}
-pub mod version;
+
+cfg_if! {
+    if #[cfg(feature = "error-type")] {
+    pub use error::LemmyErrorType;
+  }
+}
 
 use std::time::Duration;
 
@@ -36,7 +41,7 @@ macro_rules! location_info {
   };
 }
 
-#[cfg(feature = "misc")]
+#[cfg(feature = "default")]
 /// tokio::spawn, but accepts a future that may fail and also
 /// * logs errors
 /// * attaches the spawned task to the tracing span of the caller for better logging