fix(reporter): sort log tabs alphabetically instead of by creation time

[soloturn]

AI-assisted change proposal. Filed by agent driven by @soloturn via GDD.

Summary

• sortLogFiles() sorted by file creation time, reversed (newest first). With two log files from one session (Terasology-init.log, Terasology-menu.log) that reads as arbitrary - neither the order they were written in nor the order their names suggest - rather than a deliberate choice, per the original issue's own framing. Sorting by filename instead is deterministic and, for Terasology's own log naming, happens to match session order too.
• Also carries the GlobalProperties NPE guard from fix(reporter): include every log file in the PasteBin upload #56 - needed just to construct GlobalProperties() at all in cr-core's own test classpath, same reasoning as there. Will collapse to a no-op merge once fix(reporter): include every log file in the PasteBin upload #56 lands first, since it's the identical diff.

First item from #53 (1 of 5); items 2 and 3 are #56 and #58.

Test plan

• New ErrorMessagePanelTabOrderTest - writes two log files out of alphabetical order with a real gap between creation times, asserts the resulting tab order is alphabetical regardless.
• ./gradlew build - clean.

Related

• Fourth PR toward Reporter dialog: log tab order, PasteBin only uploads one log, dead/blank report links #53.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6c3ae4a1-d840-4ecc-b8fa-c53859abfe23

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}