…933782887

chore: sync main (v9.6.5) into beta

1. Fix the bug mentioned in #956 and add index mentioned in #1014

Patch

…71343948

chore: update Unity package to beta version 9.6.6-beta.2

…es (#1036)

…14653622

chore: update Unity package to beta version 9.6.6-beta.3

SceneView.lastActiveSceneView is null in batch mode, so
stats_set_scene_debug correctly returns an error. Use Assume.That
to mark the test inconclusive rather than failing CI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…e count

- StatsSetSceneDebug_ValidMode test now asserts success when SceneView
  exists and correctly asserts failure when it doesn't (batch mode),
  instead of going inconclusive.
- Workflow now parses NUnit XML for failed=0 rather than trusting the
  runner exit code, which returns 2 for inconclusive tests (URP,
  ProBuilder, Volumes not installed in CI).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…#1045)

* Fix create_script validator false-positive on constructor invocations

The regex-based duplicate method signature check misidentified
`new Type(...)` constructor calls as method declarations, causing
valid C# like `new GameObject("A"); new GameObject("B");` to be
rejected. Capture the return-type token and skip when it is `new`.

Addresses #1044

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Address review: Ordinal comparison, add new-modifier + constructor test

Use string.Equals with StringComparison.Ordinal for the "new" check.
Add test combining `public new void Init()` with nearby constructor
invocations to guard against modifier vs return-type parsing regressions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…62948128

chore: update Unity package to beta version 9.6.6-beta.4

chore: bump version to 9.6.6