Add explicity JsonConverters to all enums, so they are always serialized as strings

[KevinJump]

Not sure if this is the issue, but some reports that one of the enums in the ActionViews is returning to the user as a number not a string.

It's likely there is some global setting at play, but as with Newtonsoft, we can probibly overcome that by being explict with what to do with the enums.

[Copilot]

Pull request overview

This pull request adds explicit JsonStringEnumConverter attributes to all public enum declarations throughout the uSync codebase to ensure they are consistently serialized as strings (not numbers) when used in JSON API responses and SignalR messages. This addresses a reported issue where enums in ActionViews were being returned as numbers instead of strings to the client.

Changes:

• Added JsonStringEnumConverter<T> attributes to 11 enum types across the codebase
• Added necessary using System.Text.Json.Serialization; imports to affected files
• Ensures consistent string serialization for enums used in API models like uSyncActionView, which contains several of these enum types

Reviewed changes

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

Show a summary per file

File	Description
uSync.Core/uSyncContentState.cs	Added JsonConverter to uSyncContentState enum
uSync.Core/Tracking/SyncXmlTracker.cs	Added JsonConverter to TrackingDirection enum
uSync.Core/SyncActionType.cs	Added JsonConverter to SyncActionType enum
uSync.Core/Sync/SyncTreeType.cs	Added JsonConverter to SyncTreeType enum
uSync.Core/Models/uSyncChange.cs	Added JsonConverter to ChangeDetailType enum
uSync.Core/Dependency/uSyncDependency.cs	Added JsonConverter to DependencyMode enum
uSync.Core/ChangeType.cs	Added JsonConverter to ChangeType enum (used in API responses)
uSync.BackOffice/SyncHandlers/Models/HandlerActionNames.cs	Added JsonConverter to HandlerActions enum
uSync.BackOffice/Configuration/uSyncSettings.cs	Added JsonConverter to SyncFolderMode and SyncProcessingMode enums

Comments suppressed due to low confidence (1)

uSync.Core/uSyncContentState.cs:6

• Spelling error in comment: "thing" should be "think".

///  the state we thing a culture / content item should be in.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.