[EmandM]

Purpose of this PR

When we generate the serialization code for NetworkVariables, we add code at the assembly level. Mono, for whatever reason, didn't complain if there were private or protected types in the serialization generation code. CoreCLR throws an error if we try to generate serialization for private types.

This PR detects this case and logs a warning in Mono, and an error in CoreCLR. This gives users a human-readable error message with information on how to fix the issue, and lets the code generation safely continue.

Jira ticket

UUM-156224

Changelog

• Fixed: NetworkVariable serialization now logs a clear error message when using CoreCLR and generating serialization for private or protected types.

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.

Up-port

n/a

Backports

2.x doesn't need to support CoreCLR

[u-pr]

💡 Harness Review

The new diagnostics improve failure reporting, but the accessibility validation does not yet cover all supported serialization-generation contexts.

_{Reviewed commit 6624e66}

_{🤖 Helpful? 👍/👎}

[u-pr]

InitializeSerialization still fails on CoreCLR when NetworkVariable<Payload> uses a referenced base's protected internal nested enum without friend access. A derived NetworkBehaviour can legally declare that field using protected access. GetAllFieldsAndResolveGenerics collects Payload, but this predicate does not check IsNestedFamilyOrAssembly. The generated NetworkVariableSerializationHelper cannot access Payload because the helper is outside the base's assembly and inheritance hierarchy. Pass the consuming assembly into CollectTypesHiddenFromAssembly. Reject IsNestedFamilyOrAssembly types when that assembly lacks internal or friend access.

_{🤖 Helpful? 👍/👎}

[codecov-github-com]

Codecov Report

All modified and coverable lines are covered by tests ✅

@@                Coverage Diff                @@
##           develop-3.x.x    #4239      +/-   ##
=================================================
- Coverage          76.75%   76.74%   -0.01%     
=================================================
  Files                159      159              
  Lines              28078    28078              
=================================================
- Hits               21550    21549       -1     
- Misses              6528     6529       +1     

Flag	Coverage Δ
NGOv2_project_testproject_ubuntu	75.78% <ø> (-0.02%)	⬇️

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

see 1 file with indirect coverage changes

Components	Coverage Δ
com.unity.netcode.gameobjects	76.74% <ø> (-0.01%)	⬇️

ℹ️ Need help interpreting these results?

[NoelStephensUnity]

LGTM!