Fix duplicate property declaration in ScriptGenerationResult record

[Copilot]

The ScriptGenerationResult record had a duplicate property declaration causing compilation failure. Positional record parameters automatically generate properties, making explicit declarations redundant.

Changes:

• Removed explicit DataFileName property declaration (lines 20-23)
• Added XML doc <param> comment to document the positional parameter

Before:

public sealed record ScriptGenerationResult(string DataFileName)
{
    /// <summary>
    /// The data file name (without extension) used to generate the script.
    /// </summary>
    public string DataFileName { get; init; } = DataFileName;  // Duplicate!
    ...
}

After:

/// <param name="DataFileName">The data file name (without extension) used to generate the script.</param>
public sealed record ScriptGenerationResult(string DataFileName)
{
    // Property automatically generated from positional parameter
    ...
}

Validation:

• Verified all refactored services maintain original functionality
• Confirmed script generation produces correct output for all game versions (TR2, TR3, TR4, TRNG, TombEngine)
• Validated LuaStudio and ClassicScriptStudio integration paths

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.