Tag the camera created by CreateScene with MainCamera

[nowsprinting]

Summary

• [CreateScene(camera: true)] created a camera named "Main Camera" without the built-in MainCamera tag, so Camera.main returned null and tests relying on it (e.g., Canvas.worldCamera setup, screenshot helpers) went inconclusive.
• Set tag = "MainCamera" on the created GameObject. The block is shared by the Play Mode and Edit Mode scene-creation paths, so one change covers both.
• Update the XML doc comment note to state that Camera.main returns the created camera.
• The GameObject.tag setter is a banned API (RS0030); suppressed at line scope with a why-not comment, since the ban text itself allows assignment outside the player loop.

Test plan

• New acceptance tests Attach_WithCamera_CameraMainIsCreatedCamera (Edit Mode and Play Mode) fail before the fix and pass after
• CreateSceneAttributeTest Play Mode 12/12, Edit Mode 2/2

🤖 Generated with Claude Code

[Copilot]

🟡 Changes recommended

Clarify the unloadOthers documentation and update the README to match the new Camera.main contract.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR tags cameras created by CreateSceneAttribute as MainCamera, enabling Camera.main, with Edit Mode and Play Mode coverage.

Changes:

• Tags generated cameras as MainCamera.
• Adds acceptance tests for both test modes.
• Updates XML documentation.

File summaries

File	Summary
Runtime/Attributes/CreateSceneAttribute.cs	Tags created cameras and updates documentation. Findings: moderate documentation issue regarding competing main cameras when unloadOthers is false; README documentation update requested as a nit.
Tests/Runtime/Attributes/CreateSceneAttributeTest.cs	Adds Play Mode coverage for Camera.main.
Tests/Editor/Attributes/CreateSceneAttributeTest.cs	Adds Edit Mode coverage for Camera.main.

Review details

Suppressed comments (1)

Runtime/Attributes/CreateSceneAttribute.cs:39

• unloadOthers is a supported option, but when it is false this leaves any existing enabled MainCamera cameras loaded. Unity does not guarantee that Camera.main selects the newly created camera in that situation, so this parameter documentation overstates the contract. Either qualify the documentation to the no-competing-camera case or explicitly define how competing main cameras are handled.

        /// <param name="camera">true: create main camera object (tagged <c>MainCamera</c>, so <c>Camera.main</c> returns it) in new scene</param>

• Files reviewed: 3/3 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

Code Metrics Report

	master (aa04e7e)	#217 (9d83bfa)	+/-
Coverage	84.5%	84.5%	0.0%
Code to Test Ratio	1:1.4	1:1.4	+0.0
Test Execution Time	6m22s	7m19s	+57s

Details

  |                     | master (aa04e7e) | #217 (9d83bfa) | +/-  |
  |---------------------|------------------|----------------|------|
  | Coverage            |            84.5% |          84.5% | 0.0% |
  |   Files             |               65 |             65 |    0 |
  |   Lines             |             2453 |           2453 |    0 |
  |   Covered           |             2073 |           2073 |    0 |
+ | Code to Test Ratio  |            1:1.4 |          1:1.4 | +0.0 |
  |   Code              |             4159 |           4161 |   +2 |
+ |   Test              |             6154 |           6175 |  +21 |
- | Test Execution Time |            6m22s |          7m19s | +57s |

Code coverage of files in pull request scope (100.0% → 100.0%, patch 100.0%)

Files	Coverage	+/-	Patch Coverage	Status
Runtime/Attributes/CreateSceneAttribute.cs	100.0%	0.0%	100.0%	modified

---

Reported by octocov