[AOT] Clean up some AOT issues in Advanced Paste module

[moooyo]

Summary of the Pull Request

Clean up some AOT build issue. Currently, we can not build the Advanced Paste module with AOT enabled.

blocker:

1. We need to upgrade CommunityToolKit.WinUI to the latest preview version. It provide native AOT support. https://github.com/CommunityToolkit/Windows/releases/tag/v8.2.241112-preview1
2. CommunityToolKit.Mvvm provide more AOT diagnostic info in 8.4.0. But I'm not sure wether it provide Native AOT support for our project. https://github.com/CommunityToolkit/dotnet/releases
3. ReverseMarkdown does not support Native AOT

PR Checklist

• Closes: #xxx
• Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
• Tests: Added/updated and all pass
• Localization: All end user facing strings can be localized
• Dev docs: Added/updated
• New binaries: Added on the required places
  • JSON for signing for new binaries
  • WXS for installer for new binaries and localization folder
  • YML for CI pipeline for new test projects
  • YML for signed pipeline
• Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

[moooyo]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[jaimecbernardo]

Hi, we've added the "Don't Merge" tag to the PR, since we're trying to keep the repo stable for the release and possible hotfix if necessary. Please don't merge the PR while the tag is still in here. This allows people to review the PR and approve with less fear that it'll get merged 😄

[moooyo]

Hi, we've added the "Don't Merge" tag to the PR, since we're trying to keep the repo stable for the release and possible hotfix if necessary. Please don't merge the PR while the tag is still in here. This allows people to review the PR and approve with less fear that it'll get merged 😄

Thanks! We will wait until release complete

[snickler]

Suggested change

	public string ToJsonString() => JsonSerializer.Serialize(this, AdvancedPasteJsonSerializerContext.Default.PersistedCache);
	public string ToJsonString() => JsonSerializer.Serialize(this, AdvancedPasteJsonSerializerContext.Default.LogEvent);

Also, I believe the LogEvent class should be more defined.

Since we know telemetryEvent.CacheUsed, telemetryEvent.IsSavedQuery, telemetryEvent.PromptTokens, telemetryEvent.CompletionTokens, telemetryEvent.ModelName, telemetryEvent.ActionChain are the properties passed into the anonymous object, defining them in the class will make it cleaner and less prone to potential issues.

[moooyo]

Sure, I agree.

[moooyo]

Let me consider how to make this interface easy to use.

[moooyo]

@snickler
I've updated the LogEvent define. Could you please take a look?