Issues · intendednull/willow · GitHub

Labels Milestones New issue

Codebase Audit — main @ 2f26d91 (2026-04-24)

#276

· intendednull opened

on Apr 24, 2026

[TEST-05] `state` crate has 90 unit tests but zero integration / persistence / crash-recovery tests

severity:medium

#274

· intendednull opened

on Apr 24, 2026

[GEN-13] Fire-and-forget `tokio::spawn` on `NeighborUp` / closures — unbounded on reconnect storms

severity:medium

#272

· intendednull opened

on Apr 24, 2026

[GEN-12] `tokio::time::sleep(50ms)` used to "wait for actor propagation" in ~10 `ClientHandle` methods

severity:medium

#271

· intendednull opened

on Apr 24, 2026

[GEN-10] HLC `wall_clock_ms` panics on backwards system clock (`SystemTime < UNIX_EPOCH`)

severity:medium

#270

· intendednull opened

on Apr 24, 2026

[GEN-09] `MessageDb::insert` swallows SQLite errors with `.ok();` — silent message loss

severity:medium

#269

· intendednull opened

on Apr 24, 2026

[GEN-08] `topological_sort()` runs O(N) on every `SyncRequest`, then truncates to 500

severity:medium

#268

· intendednull opened

on Apr 24, 2026

[GEN-02] No `timeout-minutes` on any CI / deploy / e2e job

severity:medium

#265

· intendednull opened

on Apr 24, 2026

[ARCH-12] `worker/src/actors/network.rs` is 728 LOC god-actor with silent `_ => Ignore` catch-all

severity:medium

#264

· intendednull opened

on Apr 24, 2026

[ARCH-11] `crates/state/src/tests.rs` is 3,726 LOC — single-file monolith, 44% of the crate

#263

· intendednull opened

on Apr 24, 2026

[ARCH-10] `willow-state` has no declared schema version — wire/disk migrations require hand coordination

#262

· intendednull opened

on Apr 24, 2026

[ARCH-09] `ServerState` name collision between `willow-state` and `willow-web`

severity:medium

#261

· intendednull opened

on Apr 24, 2026