fix: FileAccessor SavePath

[kochounoyume]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors FileAccessor to replace its constructor-based savePath with an abstract SavePath property, ensuring each implementation overrides the new property.

• Introduce protected internal abstract string SavePath { get; } and remove the primary constructor from FileAccessor
• Update DefaultFileAccessor and CryptoFileAccessor to override SavePath instead of passing it to the base
• Simplify LSAccessor constructor to assign SavePath directly

Reviewed Changes

Copilot reviewed 3 out of 7 changed files in this pull request and generated no comments.

File	Description
src/LocalPrefs/FileAccessor.cs	Removed primary constructor parameter and added abstract SavePath
src/LocalPrefs/CryptoFileAccessor.cs	Changed base call to override SavePath
src/LocalPrefs.Unity/Packages/jp.andantetribe.localprefs/Runtime/LSAccessor.cs	Simplified constructor and implemented SavePath override

Comments suppressed due to low confidence (3)

src/LocalPrefs/FileAccessor.cs:7

• The class XML summary still refers to a constructor parameter that no longer exists; update the summary to describe the new abstract SavePath property instead of the removed constructor argument.

public abstract class FileAccessor

src/LocalPrefs/FileAccessor.cs:53

• Consider adding a unit test to verify that ReadAllBytes returns an empty array when the file does not exist to cover this fallback behavior.

            File.Exists(SavePath) ? File.ReadAllBytes(SavePath) : [];

src/LocalPrefs.Unity/Packages/jp.andantetribe.localprefs/Runtime/LSAccessor.cs:22

• Add tests to confirm that LSAccessor.SavePath returns the provided path and that ReadAllBytes correctly delegates to LSUtils using SavePath.

        public LSAccessor(in string path) => SavePath = path;