Added Support colons in legacy GIDs

[tobias-tengler]

While preparing our Framework code for Hot Chocolate v14, I noticed that we have some (strongly typed) Ids that contain colons (:). This is problematic with the current implementation of the new Node Id Serializer, as it looks for : as the separator between Type name and Id value and doesn't consider that it might be dealing with a legacy Id that contains a colon.

This PR adds support for colons in Id values by looking for both the new : and old \n delimiter.

This of course impacts the hot path, but I think it's warranted. If someone were to implement a custom Id format (doesn't even have to be strongly typed, it could just be a string with another dalimiter) I would assume that the delimiter being chosen would be : 95% of the time. I'd therefore assume that there is probably at least one such Id in each larger code base. I think it would be really nice, if the new solution would be a drop-in replacement and wouldn't require any migration effort.

[codecov]

Codecov Report

Attention: Patch coverage is 91.30435% with 4 lines in your changes missing coverage. Please review.

Project coverage is 76.73%. Comparing base (df52cdf) to head (110bc81).

Files	Patch %	Lines
...pes/Relay/Serialization/DefaultNodeIdSerializer.cs	82.60%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7304      +/-   ##
==========================================
+ Coverage   75.83%   76.73%   +0.89%     
==========================================
  Files        2473     2771     +298     
  Lines      122307   137889   +15582     
==========================================
+ Hits        92755   105811   +13056     
- Misses      29552    32078    +2526     

Flag	Coverage Δ
unittests	76.73% <91.30%> (+0.89%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.