feat: Netcode log system

[EmandM]

Purpose of this PR

• Adds a ContextualLogger class that can hold references to a NetworkManager and a GameObject
• Adds a ILogContext interface to standardize how LogContexts can be written to a builder
• Adds a Context struct that is passed to the functions on the ContextualLogger to collect information about specific callsites
• Uses these new structs throughout NetworkManager, NetworkManagerHelper, and NetworkManagerEditor.
• Swaps out NetworkLog to use a system wide static version of the ContextualLogger
• The new logger will highlight the relevant GameObject in the scene view when clicking on logs!

Jira ticket

Changelog

• Added: Context aware logging

Documentation

• No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

• Manual testing done

Automated tests:

• Covered by existing automated tests
• Covered by new automated tests

Does the change require QA team to:

• Review automated tests?
• Execute manual tests?
• Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

[ShadauxCat]

I think this naming is a bit too vague. If I didn't have the context of a diff to see what it's replacing, I wouldn't be able to figure out what it's supposed to do. I propose naming it something like WithTag or WithLabel.

[ShadauxCat]

What happens here if someone changes scenes and creates a new NetworkManager rather than reusing the previous one (like, say, if they're switching from a lobby to an instanced game so they're changing to a new session)? Or if multiple NetworkManagers exist at once as in our tests?