[danifischer]

Summary

The three cleanup items from the list, in three commits.

1. Timeline files are no longer written via the current directory (fix(logic))
TimelineCreator wrote index.html/statistics.html into the current directory first and then moved them, so two runs with the same output name overwrote each other (a new test with 8 parallel builds fails 3/3 on the old code). The files are now written next to the target and moved over it (a timeline that cannot be written completely never replaces a good one; the old file is no longer deleted up front).

2. RestSharp is gone (fix(app))
DpsReportUploadService used RestSharp 106. New DpsReportClient on HttpClient:

• multipart upload, token URL-encoded (it was concatenated raw)
• readable errors from dps.report (rate limit 429, "unsupported log", html error pages) instead of crashing on an unexpected answer
• the old code collected links in a plain List from Parallel.ForEach (race); now Parallel.ForEachAsync, a thread-safe collection, a stable output order, and a summary of how many logs failed
• boss filter ids are compared case-insensitively (0x3c4e works)
• RestSharp removed from the solution; 20 new tests (client with a stubbed handler, filter logic)

3. A fixture for every raid encounter (test)
14 new anonymized real-log fixtures for the encounters that had none, so all 24 definitions are covered. expected_columns.json holds the expected value of every column of every fixture, calculated from the raw json by make_expected_columns.py (a second implementation of the counting). BossMechanicsTests checks all columns, that every definition has a fixture, and that the renamed mechanics show up. Misspelling a mechanic in mechanics.json (tried on Xera, which had no fixture before) makes it fail.

• trim_ei_json.py now keeps players that cover every mechanic, and removes player names inside target names ("Slubling 2 "); a check over all fixtures against the character and account names of the 157 source logs finds no real name
• Only Dhuum Cone and Echo PU are 0 for everyone; they never happened in 8 real logs

Not in this PR

• Database project (finish or remove) still needs your decision; ParsingTest.cs and plan.md belong to it
• mechanics.local.json for your own encounters

Test plan

• 143 offline tests (110 Logic, 33 App), 0 warnings
• Regression check: restoring the old WriteHtmlFile makes the parallel test fail 3/3
• CI green

🤖 Generated with Claude Code