docs: describe the collision stub as what N4E actually shipped

The job description still said the stub occupies both NetworkTime and NetworkTimeSystem, which
is what the stub looked like before 6013f45 corrected it. N4E 6.7.0 sub-namespaced NetworkTime
into Unity.Netcode.NetcodeTime and left NetworkTimeSystem in the shared root, so only one name
is occupied and only that one expectation inverts.

Author: NoelStephensUnity

.yamato/api-updater-test.yml

@@ -1,51 +1,52 @@
-{% metadata_file .yamato/project.metafile %}
----
-
-# DESCRIPTION--------------------------------------------------------------------------
-  # This job validates the NGO 2.x -> 3.x upgrade path for user scripts.
-  # NGO 3.0 renamed the editor assembly and its namespaces (Unity.Netcode.Editor ->
-  # Unity.Netcode.GameObjects.Editor) and moved the runtime timing types (Unity.Netcode.NetworkTime
-  # and friends -> Unity.Netcode.GameObjects.Timing), and every relocated public type carries a
-  # [MovedFrom] so that Unity's API updater rewrites a 2.x project's scripts automatically on upgrade.
-  # apiupdaterproject holds code written against the 2.x API; the job imports it with
-  # -accept-apiupdate and asserts that every 2.x type reference was rewritten and none survived.
-  # See apiupdaterproject/README.md.
-  #
-  # The second run adds an assembly that occupies Unity.Netcode.NetworkTime and NetworkTimeSystem,
-  # standing in for another package taking those names, and inverts the expectation for exactly those
-  # two: the updater is driven by resolution failure, so a name that still resolves cannot be
-  # migrated. NetworkTickSystem is absent from the stub and must still migrate, so a pass proves both
-  # halves. This is the regression test for why the timing types were moved at all.
-
-
-# TECHNICAL CONSIDERATIONS---------------------------------------------------------------
-  # apiupdaterproject/Packages/manifest.json references the package by relative path
-  # (file:../../com.unity.netcode.gameobjects), so the job tests the package as it sits in the repo
-  # and needs no package-pack dependency.
-  # The script restores the 2.x sources when it finishes, so the checkout is left unmodified and the
-  # job is safe to re-run on the same agent.
-  # --clean purges Library first: the assertion is meaningless against a warm Library that already
-  # holds rewritten sources from a previous run. That is also why the two runs are sequential
-  # commands rather than one - each needs its own cold import, and the script removes the stub and
-  # restores the sources on every exit path, so the second run starts from the same state as the first.
-
-{% for platform in test_platforms.default -%}
-{% for editor in validation_editors.default -%}
-api_updater_test_{{ platform.name }}_{{ editor }}:
-  name : API Updater Test - NGO 2.x editor scripts upgrade [{{ platform.name }}, {{ editor }}]
-  agent:
-    type: {{ platform.type }}
-    image: {{ platform.image }}
-    flavor: {{ platform.flavor }}
-  commands:
-    - unity-downloader-cli --fast --wait -u {{ editor }} -c Editor # Installing basic editor for the import
-    - python apiupdaterproject/run_upgrade_test.py --unity .Editor --clean
-    - python apiupdaterproject/run_upgrade_test.py --unity .Editor --clean --collision-stub
-  artifacts:
-    logs:
-      paths:
-        - "apiupdaterproject/upgrade-test.log"
-  dependencies:
-    - .yamato/_run-all.yml#run_quick_checks # initial checks to perform fast validation of common errors
-{% endfor -%}
-{% endfor -%}
+{% metadata_file .yamato/project.metafile %}
+---
+
+# DESCRIPTION--------------------------------------------------------------------------
+  # This job validates the NGO 2.x -> 3.x upgrade path for user scripts.
+  # NGO 3.0 renamed the editor assembly and its namespaces (Unity.Netcode.Editor ->
+  # Unity.Netcode.GameObjects.Editor) and moved the runtime timing types (Unity.Netcode.NetworkTime
+  # and friends -> Unity.Netcode.GameObjects.Timing), and every relocated public type carries a
+  # [MovedFrom] so that Unity's API updater rewrites a 2.x project's scripts automatically on upgrade.
+  # apiupdaterproject holds code written against the 2.x API; the job imports it with
+  # -accept-apiupdate and asserts that every 2.x type reference was rewritten and none survived.
+  # See apiupdaterproject/README.md.
+  #
+  # The second run adds an assembly that occupies Unity.Netcode.NetworkTimeSystem, which is what
+  # Netcode for Entities does as of 6.7.0 - it sub-namespaced NetworkTime out of the shared root but
+  # left NetworkTimeSystem in it - and inverts the expectation for that one name: the updater is
+  # driven by resolution failure, so a name that still resolves cannot be migrated. NetworkTime and
+  # NetworkTickSystem are absent from the stub and must still migrate, so a pass proves both halves.
+  # This is the regression test for why the timing types were moved at all.
+
+
+# TECHNICAL CONSIDERATIONS---------------------------------------------------------------
+  # apiupdaterproject/Packages/manifest.json references the package by relative path
+  # (file:../../com.unity.netcode.gameobjects), so the job tests the package as it sits in the repo
+  # and needs no package-pack dependency.
+  # The script restores the 2.x sources when it finishes, so the checkout is left unmodified and the
+  # job is safe to re-run on the same agent.
+  # --clean purges Library first: the assertion is meaningless against a warm Library that already
+  # holds rewritten sources from a previous run. That is also why the two runs are sequential
+  # commands rather than one - each needs its own cold import, and the script removes the stub and
+  # restores the sources on every exit path, so the second run starts from the same state as the first.
+
+{% for platform in test_platforms.default -%}
+{% for editor in validation_editors.default -%}
+api_updater_test_{{ platform.name }}_{{ editor }}:
+  name : API Updater Test - NGO 2.x editor scripts upgrade [{{ platform.name }}, {{ editor }}]
+  agent:
+    type: {{ platform.type }}
+    image: {{ platform.image }}
+    flavor: {{ platform.flavor }}
+  commands:
+    - unity-downloader-cli --fast --wait -u {{ editor }} -c Editor # Installing basic editor for the import
+    - python apiupdaterproject/run_upgrade_test.py --unity .Editor --clean
+    - python apiupdaterproject/run_upgrade_test.py --unity .Editor --clean --collision-stub
+  artifacts:
+    logs:
+      paths:
+        - "apiupdaterproject/upgrade-test.log"
+  dependencies:
+    - .yamato/_run-all.yml#run_quick_checks # initial checks to perform fast validation of common errors
+{% endfor -%}
+{% endfor -%}