Skip to main content

max / goingson

drop unused deps and collapse a nested if Remove serde from go-mcp and toml from src-tauri, both unused. strum in goingson-core stays: EnumString expands to code referencing strum::ParseError, which cargo-machete cannot see.
Co-Authored-By
Claude Opus 5 (1M context) <noreply@anthropic.com>
Author: Max Johnson <me@maxj.phd> · 2026-08-01 21:28 UTC
Signed with PGP, not checked
Commit: 499996560d33a2f51be31500940226172718820f
Parent: 990b6d0
4 files changed, +5 insertions, -28 deletions
M Cargo.lock +1 -22
@@ -2171,7 +2171,6 @@
2171 2171 "goingson-db-sqlite",
2172 2172 "kberg",
2173 2173 "painhours",
2174 - "serde",
2175 2174 "serde_json",
2176 2175 "sqlx",
2177 2176 "tokio",
@@ -2283,7 +2282,6 @@
2283 2282 "tokio",
2284 2283 "tokio-rustls",
2285 2284 "tokio-util",
2286 - "toml 1.1.3+spec-1.1.0",
2287 2285 "tracing",
2288 2286 "tracing-subscriber",
2289 2287 "uuid",
@@ -3109,8 +3107,6 @@
3109 3107 "thiserror 2.0.19",
3110 3108 "tokio",
3111 3109 "tokio-stream",
3112 - "tower",
3113 - "tower-http 0.7.0",
3114 3110 "tracing",
3115 3111 "uuid",
3116 3112 ]
@@ -4922,7 +4918,7 @@
4922 4918 "tokio-rustls",
4923 4919 "tokio-util",
4924 4920 "tower",
4925 - "tower-http 0.6.11",
4921 + "tower-http",
4926 4922 "tower-service",
4927 4923 "url",
4928 4924 "wasm-bindgen",
@@ -6987,23 +6983,6 @@
6987 6983 "url",
6988 6984 ]
6989 6985
6990 - [[package]]
6991 - name = "tower-http"
6992 - version = "0.7.0"
6993 - source = "registry+https://github.com/rust-lang/crates.io-index"
6994 - checksum = "b11f75e912b0c2be01b63d8cf8057b8c3f97cf34abb3d431a3a4c8675498e233"
6995 - dependencies = [
6996 - "bitflags 2.13.1",
6997 - "bytes",
6998 - "http",
6999 - "http-body",
7000 - "percent-encoding",
7001 - "pin-project-lite",
7002 - "tower-layer",
7003 - "tower-service",
7004 - "tracing",
7005 - ]
7006 -
7007 6986 [[package]]
7008 6987 name = "tower-layer"
7009 6988 version = "0.3.3"
@@ -99,7 +99,6 @@
99 99 # modal opens, so it cannot be materialised the way the stylesheet is.
100 100 makeover-webview = "0.9.0"
101 101 makeover-layout = "0.6.0"
102 - toml = { workspace = true }
103 102 # Browser opening
104 103 open = { workspace = true }
105 104
@@ -20,7 +20,6 @@
20 20
21 21 tokio = { workspace = true, features = ["rt-multi-thread", "macros", "signal"] }
22 22 sqlx = { workspace = true, features = ["sqlite"] }
23 - serde = { workspace = true }
24 23 serde_json = { workspace = true }
25 24 chrono = { workspace = true }
26 25 chrono-tz = { workspace = true }
@@ -102,10 +102,10 @@
102 102 /// value alone, which is the better contract, so the two ways the frontend
103 103 /// says this are reconciled here rather than in the crate.
104 104 fn current_value(&self) -> &str {
105 - if let Some(value) = self.value.as_deref() {
106 - if !value.is_empty() {
107 - return value;
108 - }
105 + if let Some(value) = self.value.as_deref()
106 + && !value.is_empty()
107 + {
108 + return value;
109 109 }
110 110 self.options
111 111 .as_deref()