Audit finding from #300 (commit 679f9fe)
Severity: medium
Category: tech debt / maintenance
File: crates/web/src/components/message.rs (6 TODOs), crates/client/src/views.rs (6 TODOs), crates/web/src/components/chat.rs:476, crates/network/src/traits.rs:161, crates/client/src/listeners.rs:294
Obvious fix: no
Description
18 TODO/FIXME/HACK/XXX markers, all TODOs (no FIXME/HACK/XXX). Most are placeholders for unfinished features tied to spec docs (profile-card.md, whisper-mode.md, reactions-pins.md, sync-queue.md, multi-grove). They cluster in client/views.rs and web/components/message.rs, indicating these subsystems are partially implemented.
Impact / Threat
Maintenance — risk that placeholders silently diverge from spec and accumulate.
Suggested fix
Convert each TODO into a tracked GitHub issue with the spec link, and either gate the code paths behind a feature flag or remove the dead branches until the spec lands.
Verify
rg "TODO|FIXME|HACK|XXX" --type rust crates/ | wc -l
Audit finding from #300 (commit 679f9fe)
Severity: medium
Category: tech debt / maintenance
File: crates/web/src/components/message.rs (6 TODOs), crates/client/src/views.rs (6 TODOs), crates/web/src/components/chat.rs:476, crates/network/src/traits.rs:161, crates/client/src/listeners.rs:294
Obvious fix: no
Description
18 TODO/FIXME/HACK/XXX markers, all TODOs (no FIXME/HACK/XXX). Most are placeholders for unfinished features tied to spec docs (profile-card.md, whisper-mode.md, reactions-pins.md, sync-queue.md, multi-grove). They cluster in client/views.rs and web/components/message.rs, indicating these subsystems are partially implemented.
Impact / Threat
Maintenance — risk that placeholders silently diverge from spec and accumulate.
Suggested fix
Convert each TODO into a tracked GitHub issue with the spec link, and either gate the code paths behind a feature flag or remove the dead branches until the spec lands.
Verify