From 1479cfd1a73246496980c9c9d2224bae8de88c1f Mon Sep 17 00:00:00 2001 From: Anten Skrabec Date: Tue, 28 Jul 2026 23:46:53 -0600 Subject: [PATCH] chore: clean up TODO.md after bug fix PRs #354-#359 Remove fixed bugs from TODO.md and add pre-existing orphan risk to Robustness section. Co-Authored-By: Claude Opus 4.6 (1M context) --- TODO.md | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/TODO.md b/TODO.md index ff0ac8b..4906741 100644 --- a/TODO.md +++ b/TODO.md @@ -9,14 +9,6 @@ ## Quick Wins - infinite use invite codes (no multi-use support, only single-use) -## Bugs -- restarting a headless client from the main headless page causes you to end up at that headless' info page -- headless client start/stop/restart buttons on `/quma/headless` go past the card length -- typo in update changelog: versions url should use `/#versions` hash anchor, not `/versions` path segment (current URL 404s on Forge) -- typo in update changelog: versions url should be like this `https://forge.sp-tarkov.com/mod/2310/wtt-commonlib/#versions` (`#version` is the important part) -- mod queue allows duplicate pending operations (operations on mods should be exlcusive in the queue) -- spt server profile creation is broken on user signup - ## Convoy - user config file sync - user specific mods @@ -43,6 +35,7 @@ - headless client numa scheduling section has shitty formatting ## Robustness +- SPT profile can be orphaned if QM DB transaction fails after SPT registration succeeds (`join.rs` — existing `TODO(debt)`) - no mutual exclusion on server start/stop/restart (`server.rs`) - TOCTOU on duplicate mod install/update check (`mods.rs` — task-manager dedup mitigates double-click, but concurrent installs from different paths can still race past the pre-spawn check) - no limit on concurrent SSE connections (`sse.rs`)