Remove four Config settings nothing reads

[dmccoystephenson]

Summary

• Removed Config.playerSaveFile, statsSaveFile, and timeServiceSaveFile — the multi-slot save layout resolves paths through SaveFileManager.get_save_path() instead, so these three have been dead since Multiple save files #51; a grep confirms every remaining match outside config.py/the test was a local as variable in fishE.py, never self.config.*.
• Removed Config.initialEnergy — player.py already documents that starting energy deliberately comes from the housing ladder, not a configured value, so the attribute had no reader. Updated that comment since it referenced the now-removed attribute by name.
• Trimmed the matching assertions out of tests/config/test_config.py (they were pinning dead values) and dropped the now-unused os import.

Test plan

• python3 -m compileall -q src
• SDL_VIDEODRIVER=dummy SDL_AUDIODRIVER=dummy python3 -m pytest — 729 passed
• grep -rn '\.playerSaveFile\|\.statsSaveFile\|\.timeServiceSaveFile\|\.initialEnergy' --include='*.py' . — no remaining references anywhere
• README/PLANNING.md grepped for the removed names — no doc drift

Closes #147

---

drafted by Claude on behalf of Daniel Stephenson

[dmccoystephenson]

Self-review rubric:

• Scope: PASS — only the three files needed to remove the four dead settings and update the one comment that named them.
• Tests-new: N/A — this removes dead surface rather than adding behavior; nothing new needs its own test.
• Tests-fix: N/A — not a bug fix.
• Sibling structure: N/A — no new files.
• Sibling renames: N/A — no renames.
• Docs: PASS — README.md/PLANNING.md grepped for the four removed names, no hits.
• Issue resolution: PASS — all four settings named in Config declares four settings nothing reads, and the tests assert they keep existing #147 (playerSaveFile, statsSaveFile, timeServiceSaveFile, initialEnergy) are removed, along with their test assertions, per the issue's suggested fix.
• CI: PASS — test check green on f6b7ae0 (https://github.com/Stephenson-Software/FishE/actions/runs/30771347132).
• Schema-sync: N/A — Config isn't a persisted model.
• Money-format: N/A.
• Deterministic-tests: N/A — no test touches random.
• Headless-pygame: N/A.
• camelCase: N/A — no new identifiers.

Summary: dead-code removal, all applicable items pass, CI green. Ready to merge.

---

drafted by Claude on behalf of Daniel Stephenson