fixing linting

popuz · popuz · commit f19f07a5e69a · 2026-07-20T12:38:56.000+02:00
diff --git a/Explorer/Assets/DCL/Infrastructure/Global/Dynamic/DynamicWorldContainer.cs b/Explorer/Assets/DCL/Infrastructure/Global/Dynamic/DynamicWorldContainer.cs
@@ -87,8 +87,6 @@ public class DynamicWorldContainer : DCLWorldContainer<DynamicWorldSettings>
 
         public IGlobalRealmController RealmController { get; }
 
-        public IRealmNavigator RealmNavigator { get; }
-
         public GlobalWorldFactory GlobalWorldFactory { get; }
 
         public IReadOnlyList<IDCLGlobalPlugin> GlobalPlugins { get; }
@@ -132,7 +130,6 @@ private DynamicWorldContainer(
         {
             this.uiShellContainer = uiShellContainer;
             RealmController = realmController;
-            RealmNavigator = realmNavigator;
             GlobalWorldFactory = globalWorldFactory;
             GlobalPlugins = globalPlugins;
             WorldPlugins = worldPlugins;
diff --git a/Explorer/Assets/DCL/McpServer/Components/McpMovementOverride.cs b/Explorer/Assets/DCL/McpServer/Components/McpMovementOverride.cs
@@ -6,7 +6,7 @@ namespace DCL.McpServer.Components
 {
     /// <summary>
     ///     Held movement input requested by the MCP walk tool. While present on the player entity,
-    ///     <see cref="McpInputOverrideSystem" /> re-asserts it into <see cref="MovementInputComponent" /> every frame.
+    ///     <see cref="DCL.McpServer.Systems.McpInputOverrideSystem" /> re-asserts it into <see cref="MovementInputComponent" /> every frame.
     /// </summary>
     public struct McpMovementOverride
     {
diff --git a/Explorer/Assets/DCL/McpServer/Components/McpPointerClickIntent.cs b/Explorer/Assets/DCL/McpServer/Components/McpPointerClickIntent.cs
@@ -7,7 +7,7 @@ namespace DCL.McpServer.Components
 {
     /// <summary>
     ///     Present on the player entity while an agent-requested pointer click is in flight.
-    ///     <see cref="McpPointerClickSystem" /> validates the aim with a physics raycast each frame,
+    ///     <see cref="DCL.McpServer.Systems.McpPointerClickSystem" /> validates the aim with a physics raycast each frame,
     ///     delivers the press through the target's <see cref="PBPointerEvents.AppendPointerEventResultsIntent" />
     ///     and removes the component once the click completes or fails.
     /// </summary>
diff --git a/Explorer/Assets/DCL/McpServer/Core/McpJsonSchema.cs b/Explorer/Assets/DCL/McpServer/Core/McpJsonSchema.cs
@@ -11,7 +11,7 @@ namespace DCL.McpServer.Core
     public sealed class McpJsonSchema
     {
         private readonly JObject properties = new ();
-        private readonly JArray required = new ();
+        private readonly JArray requiredNames = new ();
 
         private McpJsonSchema() { }
 
@@ -72,8 +72,8 @@ public JObject Build()
                 ["properties"] = properties,
             };
 
-            if (required.Count > 0)
-                schema["required"] = required;
+            if (requiredNames.Count > 0)
+                schema["required"] = requiredNames;
 
             return schema;
         }
@@ -106,7 +106,7 @@ private McpJsonSchema AddField(string name, JObject field, bool isRequired)
             properties[name] = field;
 
             if (isRequired)
-                required.Add(name);
+                requiredNames.Add(name);
 
             return this;
         }
diff --git a/Explorer/Assets/DCL/McpServer/Core/McpToolsRegistry.cs b/Explorer/Assets/DCL/McpServer/Core/McpToolsRegistry.cs
@@ -32,7 +32,7 @@ public McpToolsRegistry Build()
             {
                 JObject inputSchema = tool.InputSchema;
 
-                if (inputSchema == null || !IsObjectSchema(inputSchema))
+                if (!IsObjectSchema(inputSchema))
                     throw new InvalidOperationException($"MCP tool '{tool.Name}' produced an invalid input schema: expected a JSON Schema object (\"type\": \"object\"). Build it with McpJsonSchema.");
 
                 if (tool.OutputSchema != null && !IsObjectSchema(tool.OutputSchema))
diff --git a/Explorer/Assets/DCL/McpServer/Tests/FakeMcpTool.cs b/Explorer/Assets/DCL/McpServer/Tests/FakeMcpTool.cs
@@ -27,8 +27,6 @@ internal sealed class FakeMcpTool : IMcpTool
 
         public JObject? LastArguments { get; private set; }
 
-        public CancellationToken LastCancellationToken { get; private set; }
-
         private FakeMcpTool(string name, string description, JObject inputSchema, McpToolAnnotations annotations,
             Func<JObject, CancellationToken, McpToolResult> execute)
         {
@@ -61,7 +59,6 @@ public UniTask<McpToolResult> ExecuteAsync(JObject arguments, CancellationToken
         {
             CallCount++;
             LastArguments = arguments;
-            LastCancellationToken = ct;
             return UniTask.FromResult(execute(arguments, ct));
         }
     }
diff --git a/Explorer/Assets/DCL/McpServer/Tests/GetPlayerStateToolShould.cs b/Explorer/Assets/DCL/McpServer/Tests/GetPlayerStateToolShould.cs
@@ -1,7 +1,6 @@
 using Arch.Core;
 using DCL.Character.Components;
 using DCL.CharacterCamera;
-using DCL.CharacterCamera.Components;
 using DCL.McpServer.Core;
 using DCL.McpServer.Tools;
 using ECS.SceneLifeCycle.CurrentScene;
@@ -41,20 +40,20 @@ public void TearDown()
         [Test]
         public void DeclareAnObjectOutputSchema()
         {
-            Assert.That(tool.OutputSchema!["type"]!.Value<string>(), Is.EqualTo("object"));
+            Assert.That(tool.OutputSchema["type"]!.Value<string>(), Is.EqualTo("object"));
         }
 
         [Test]
         public void ModelTheAddressAsANullableString()
         {
-            var addressType = (JArray)tool.OutputSchema!["properties"]!["address"]!["type"]!;
+            var addressType = (JArray)tool.OutputSchema["properties"]!["address"]!["type"]!;
             Assert.That(addressType.ToObject<string[]>(), Is.EqualTo(new[] { "string", "null" }));
         }
 
         [Test]
         public void ModelTheCameraAsANestedObject()
         {
-            JToken camera = tool.OutputSchema!["properties"]!["camera"]!;
+            JToken camera = tool.OutputSchema["properties"]!["camera"]!;
 
             Assert.That(camera["type"]!.Value<string>(), Is.EqualTo("object"));
             Assert.That(camera["properties"]!["mode"]!["type"]!.Value<string>(), Is.EqualTo("string"));
@@ -67,7 +66,7 @@ public void KeepTheOutputSchemaInSyncWithTheStructuredPayload()
             var structured = (JObject)Execute().Payload["structuredContent"]!;
 
             // Assert
-            McpSchemaAssert.KeysMatch(tool.OutputSchema!, structured);
+            McpSchemaAssert.KeysMatch(tool.OutputSchema, structured);
         }
 
         private McpToolResult Execute() =>
diff --git a/Explorer/Assets/DCL/McpServer/Tests/GetSceneStateToolShould.cs b/Explorer/Assets/DCL/McpServer/Tests/GetSceneStateToolShould.cs
@@ -64,7 +64,7 @@ public void ReportAnAbsentSceneAsAJsonNull()
         public void DeclareAnObjectOutputSchemaThatAdmitsANullScene()
         {
             // Act
-            JObject schema = tool.OutputSchema!;
+            JObject schema = tool.OutputSchema;
 
             // Assert
             Assert.That(schema["type"]!.Value<string>(), Is.EqualTo("object"));
@@ -96,7 +96,7 @@ public void KeepTheOutputSchemaInSyncWithTheStructuredPayload()
             var structured = (JObject)Execute().Payload["structuredContent"]!;
 
             // Assert
-            McpSchemaAssert.KeysMatch(tool.OutputSchema!, structured);
+            McpSchemaAssert.KeysMatch(tool.OutputSchema, structured);
         }
 
         private McpToolResult Execute() =>
diff --git a/Explorer/Assets/DCL/McpServer/Tests/ListSceneEntitiesToolShould.cs b/Explorer/Assets/DCL/McpServer/Tests/ListSceneEntitiesToolShould.cs
@@ -85,7 +85,7 @@ public void KeepTheOutputSchemaInSyncWithTheStructuredPayload()
             var structured = (JObject)Execute(limit: 200).Payload["structuredContent"]!;
 
             // Assert
-            McpSchemaAssert.KeysMatch(tool.OutputSchema!, structured);
+            McpSchemaAssert.KeysMatch(tool.OutputSchema, structured);
         }
 
         private McpToolResult Execute(int limit) =>
diff --git a/Explorer/Assets/DCL/McpServer/Tests/McpJsonRpcDispatcherShould.cs b/Explorer/Assets/DCL/McpServer/Tests/McpJsonRpcDispatcherShould.cs
@@ -78,8 +78,8 @@ public void CallAKnownToolAndWrapItsResult()
             Assert.That(tool.LastArguments!["value"]!.Value<string>(), Is.EqualTo("hi"));
 
             var content = (JArray)result["content"]!;
-            Assert.That(content[0]!["type"]!.Value<string>(), Is.EqualTo("text"));
-            Assert.That(content[0]!["text"]!.Value<string>(), Is.EqualTo("done"));
+            Assert.That(content[0]["type"]!.Value<string>(), Is.EqualTo("text"));
+            Assert.That(content[0]["text"]!.Value<string>(), Is.EqualTo("done"));
             Assert.That(result.ContainsKey("isError"), Is.False);
         }
 
diff --git a/Explorer/Assets/DCL/McpServer/Tests/McpToolResultShould.cs b/Explorer/Assets/DCL/McpServer/Tests/McpToolResultShould.cs
@@ -15,8 +15,8 @@ public void WrapTextInASingleTextContentItemWithoutAnErrorFlag()
 
             var content = (JArray)payload["content"]!;
             Assert.That(content.Count, Is.EqualTo(1));
-            Assert.That(content[0]!["type"]!.Value<string>(), Is.EqualTo("text"));
-            Assert.That(content[0]!["text"]!.Value<string>(), Is.EqualTo("hello"));
+            Assert.That(content[0]["type"]!.Value<string>(), Is.EqualTo("text"));
+            Assert.That(content[0]["text"]!.Value<string>(), Is.EqualTo("hello"));
             Assert.That(payload.ContainsKey("isError"), Is.False);
         }
 
@@ -33,8 +33,8 @@ public void CarryBothAMirroringTextItemAndStructuredContent()
 
             var content = (JArray)payload["content"]!;
             Assert.That(content.Count, Is.EqualTo(1));
-            Assert.That(content[0]!["type"]!.Value<string>(), Is.EqualTo("text"));
-            Assert.That(content[0]!["text"]!.Value<string>(), Is.EqualTo("mirror"));
+            Assert.That(content[0]["type"]!.Value<string>(), Is.EqualTo("text"));
+            Assert.That(content[0]["text"]!.Value<string>(), Is.EqualTo("mirror"));
 
             var structuredContent = (JObject)payload["structuredContent"]!;
             Assert.That(structuredContent["count"]!.Value<int>(), Is.EqualTo(3));
@@ -50,8 +50,8 @@ public void FlagErrorsWithIsErrorAndCarryTheMessageAsText()
             Assert.That(payload["isError"]!.Value<bool>(), Is.True);
 
             var content = (JArray)payload["content"]!;
-            Assert.That(content[0]!["type"]!.Value<string>(), Is.EqualTo("text"));
-            Assert.That(content[0]!["text"]!.Value<string>(), Is.EqualTo("it broke"));
+            Assert.That(content[0]["type"]!.Value<string>(), Is.EqualTo("text"));
+            Assert.That(content[0]["text"]!.Value<string>(), Is.EqualTo("it broke"));
         }
 
         [Test]
@@ -64,12 +64,12 @@ public void EncodeImageBytesAsBase64AlongsideACaption()
             var content = (JArray)payload["content"]!;
             Assert.That(content.Count, Is.EqualTo(2));
 
-            var image = (JObject)content[0]!;
+            var image = (JObject)content[0];
             Assert.That(image["type"]!.Value<string>(), Is.EqualTo("image"));
             Assert.That(image["mimeType"]!.Value<string>(), Is.EqualTo("image/png"));
             Assert.That(Convert.FromBase64String(image["data"]!.Value<string>()!), Is.EqualTo(bytes));
 
-            var caption = (JObject)content[1]!;
+            var caption = (JObject)content[1];
             Assert.That(caption["type"]!.Value<string>(), Is.EqualTo("text"));
             Assert.That(caption["text"]!.Value<string>(), Is.EqualTo("a screenshot"));
             Assert.That(payload.ContainsKey("isError"), Is.False);
diff --git a/Explorer/Assets/DCL/McpServer/Tests/McpToolsRegistryShould.cs b/Explorer/Assets/DCL/McpServer/Tests/McpToolsRegistryShould.cs
@@ -50,7 +50,7 @@ public void FailRegistrationNamingTheToolWithAnInvalidSchema()
 
             // Act & Assert
             InvalidOperationException error = Assert.Throws<InvalidOperationException>(() => registry.Build());
-            Assert.That(error!.Message, Does.Contain("broken"));
+            Assert.That(error.Message, Does.Contain("broken"));
         }
 
         [Test]
diff --git a/Explorer/Assets/DCL/McpServer/Tools/GetPlayerStateTool.cs b/Explorer/Assets/DCL/McpServer/Tools/GetPlayerStateTool.cs
@@ -29,7 +29,7 @@ public class GetPlayerStateTool : IMcpTool
 
         public JObject InputSchema => McpJsonSchema.Object().Build();
 
-        public JObject? OutputSchema =>
+        public JObject OutputSchema =>
             McpJsonSchema.Object()
                           .Object("position", JObjectExtensions.VectorSchema())
                           .Object("rotationEuler", JObjectExtensions.VectorSchema())
diff --git a/Explorer/Assets/DCL/McpServer/Tools/GetSceneStateTool.cs b/Explorer/Assets/DCL/McpServer/Tools/GetSceneStateTool.cs
@@ -26,7 +26,7 @@ public class GetSceneStateTool : IMcpTool
 
         public JObject InputSchema => McpJsonSchema.Object().Build();
 
-        public JObject? OutputSchema =>
+        public JObject OutputSchema =>
             McpJsonSchema.Object()
                           .Object("currentParcel", JObjectExtensions.ParcelSchema())
                           .String("loadingStage")
diff --git a/Explorer/Assets/DCL/McpServer/Tools/ListSceneEntitiesTool.cs b/Explorer/Assets/DCL/McpServer/Tools/ListSceneEntitiesTool.cs
@@ -33,7 +33,7 @@ public class ListSceneEntitiesTool : IMcpTool
                           .Integer("limit", "Maximum ids to return. Default 200.")
                           .Build();
 
-        public JObject? OutputSchema =>
+        public JObject OutputSchema =>
             McpJsonSchema.Object()
                           .Integer("total")
                           .Integer("returned")
diff --git a/Explorer/Assets/DCL/McpServer/Tools/ScreenshotTool.cs b/Explorer/Assets/DCL/McpServer/Tools/ScreenshotTool.cs
@@ -171,7 +171,7 @@ private async UniTask<McpToolResult> CaptureAsync(int maxWidth, bool asPng, bool
 
                     using (NativeArray<byte> encodedNative = asPng
                                ? ImageConversion.EncodeNativeArrayToPNG(rawPixels, downscaled.graphicsFormat, (uint)width, (uint)height)
-                               : ImageConversion.EncodeNativeArrayToJPG(rawPixels, downscaled.graphicsFormat, (uint)width, (uint)height, 0, JPG_QUALITY))
+                               : ImageConversion.EncodeNativeArrayToJPG(rawPixels, downscaled.graphicsFormat, (uint)width, (uint)height))
                         encoded = encodedNative.ToArray();
 
                     mimeType = asPng ? "image/png" : "image/jpeg";
diff --git a/Explorer/Assets/DCL/PerformanceAndDiagnostics/Diagnostics/Container/DiagnosticsContainer.cs b/Explorer/Assets/DCL/PerformanceAndDiagnostics/Diagnostics/Container/DiagnosticsContainer.cs
@@ -14,8 +14,9 @@ public class DiagnosticsContainer : IDisposable
         private const int DEFAULT_REPORT_HANDLERS_COUNT = 2; // DebugLog + Sentry
 
         private ILogHandler defaultLogHandler;
-        public ReportHubLogger ReportHubLogger { get; private set; }
-        public SentryReportHandler? Sentry { get; private set; }
+        private ReportHubLogger reportHubLogger;
+
+        public SentryReportHandler? Sentry { get; private init; }
         public IReportsHandlingSettings Settings { get; private set; }
         public SentrySampler? SentrySampler { get; private set; }
 
@@ -57,14 +58,14 @@ public static DiagnosticsContainer Create(IReportsHandlingSettings settings, par
             // Enable Hub static accessors
             ReportHub.Initialize(logger);
 
-            return new DiagnosticsContainer { ReportHubLogger = logger, defaultLogHandler = defaultLogHandler, Sentry = sentryReportHandler, Settings = settings, SentrySampler = sentrySampler };
+            return new DiagnosticsContainer { reportHubLogger = logger, defaultLogHandler = defaultLogHandler, Sentry = sentryReportHandler, Settings = settings, SentrySampler = sentrySampler };
         }
 
         public void AddDebugConsoleHandler(DebugMenuConsoleLogEntryBus sceneDebugConsoleMessageBus)
         {
             SceneDebugConsoleReportHandler reportHandler = AddDebugConsoleReportHandler(sceneDebugConsoleMessageBus);
             ReportHub.EnforceUnconditionalVerboseLogs = true;
-            ReportHubLogger.AddHandler(reportHandler);
+            reportHubLogger.AddHandler(reportHandler);
         }
 
         private static SceneDebugConsoleReportHandler AddDebugConsoleReportHandler(DebugMenuConsoleLogEntryBus sceneDebugConsoleMessageBus)
diff --git a/Explorer/Assets/DCL/PerformanceAndDiagnostics/Diagnostics/ReportsHandling/ReportCategory.cs b/Explorer/Assets/DCL/PerformanceAndDiagnostics/Diagnostics/ReportsHandling/ReportCategory.cs
@@ -1,6 +1,4 @@
-﻿using UnityEngine;
-
-namespace DCL.Diagnostics
+﻿namespace DCL.Diagnostics
 {
     /// <summary>
     ///     Contains names of report categories, IDs must be constant so they can be specified in the attribute
diff --git a/Explorer/Explorer.sln.DotSettings b/Explorer/Explorer.sln.DotSettings
@@ -508,6 +508,7 @@
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=OTP/@EntryIndexedValue">OTP</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=PB/@EntryIndexedValue">PB</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=PBR/@EntryIndexedValue">PBR</s:String>
+	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=POV/@EntryIndexedValue">POV</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=SFX/@EntryIndexedValue">SFX</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=SO/@EntryIndexedValue">SO</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/Abbreviations/=UISFX/@EntryIndexedValue">UISFX</s:String>

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ namespace DCL.McpServer.Components`
`6`	`6`	`{`
`7`	`7`	`/// <summary>`
`8`	`8`	`/// Held movement input requested by the MCP walk tool. While present on the player entity,`
`9`		`- /// <see cref="McpInputOverrideSystem" /> re-asserts it into <see cref="MovementInputComponent" /> every frame.`
	`9`	`+ /// <see cref="DCL.McpServer.Systems.McpInputOverrideSystem" /> re-asserts it into <see cref="MovementInputComponent" /> every frame.`
`10`	`10`	`/// </summary>`
`11`	`11`	`public struct McpMovementOverride`
`12`	`12`	`{`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ namespace DCL.McpServer.Components`
`7`	`7`	`{`
`8`	`8`	`/// <summary>`
`9`	`9`	`/// Present on the player entity while an agent-requested pointer click is in flight.`
`10`		`- /// <see cref="McpPointerClickSystem" /> validates the aim with a physics raycast each frame,`
	`10`	`+ /// <see cref="DCL.McpServer.Systems.McpPointerClickSystem" /> validates the aim with a physics raycast each frame,`
`11`	`11`	`/// delivers the press through the target's <see cref="PBPointerEvents.AppendPointerEventResultsIntent" />`
`12`	`12`	`/// and removes the component once the click completes or fails.`
`13`	`13`	`/// </summary>`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public McpToolsRegistry Build()`
`32`	`32`	`{`
`33`	`33`	`JObject inputSchema = tool.InputSchema;`
`34`	`34`
`35`		`- if (inputSchema == null \|\| !IsObjectSchema(inputSchema))`
	`35`	`+ if (!IsObjectSchema(inputSchema))`
`36`	`36`	`throw new InvalidOperationException($"MCP tool '{tool.Name}' produced an invalid input schema: expected a JSON Schema object (\"type\": \"object\"). Build it with McpJsonSchema.");`
`37`	`37`
`38`	`38`	`if (tool.OutputSchema != null && !IsObjectSchema(tool.OutputSchema))`
Original file line number	Diff line number	Diff line change
`@@ -27,8 +27,6 @@ internal sealed class FakeMcpTool : IMcpTool`
`27`	`27`
`28`	`28`	`public JObject? LastArguments { get; private set; }`
`29`	`29`
`30`		`- public CancellationToken LastCancellationToken { get; private set; }`
`31`		`-`
`32`	`30`	`private FakeMcpTool(string name, string description, JObject inputSchema, McpToolAnnotations annotations,`
`33`	`31`	`Func<JObject, CancellationToken, McpToolResult> execute)`
`34`	`32`	`{`
`@@ -61,7 +59,6 @@ public UniTask<McpToolResult> ExecuteAsync(JObject arguments, CancellationToken`
`61`	`59`	`{`
`62`	`60`	`CallCount++;`
`63`	`61`	`LastArguments = arguments;`
`64`		`- LastCancellationToken = ct;`
`65`	`62`	`return UniTask.FromResult(execute(arguments, ct));`
`66`	`63`	`}`
`67`	`64`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,6 @@`
`1`	`1`	`using Arch.Core;`
`2`	`2`	`using DCL.Character.Components;`
`3`	`3`	`using DCL.CharacterCamera;`
`4`		`-using DCL.CharacterCamera.Components;`
`5`	`4`	`using DCL.McpServer.Core;`
`6`	`5`	`using DCL.McpServer.Tools;`
`7`	`6`	`using ECS.SceneLifeCycle.CurrentScene;`
`@@ -41,20 +40,20 @@ public void TearDown()`
`41`	`40`	`[Test]`
`42`	`41`	`public void DeclareAnObjectOutputSchema()`
`43`	`42`	`{`
`44`		`- Assert.That(tool.OutputSchema!["type"]!.Value<string>(), Is.EqualTo("object"));`
	`43`	`+ Assert.That(tool.OutputSchema["type"]!.Value<string>(), Is.EqualTo("object"));`
`45`	`44`	`}`
`46`	`45`
`47`	`46`	`[Test]`
`48`	`47`	`public void ModelTheAddressAsANullableString()`
`49`	`48`	`{`
`50`		`- var addressType = (JArray)tool.OutputSchema!["properties"]!["address"]!["type"]!;`
	`49`	`+ var addressType = (JArray)tool.OutputSchema["properties"]!["address"]!["type"]!;`
`51`	`50`	`Assert.That(addressType.ToObject<string[]>(), Is.EqualTo(new[] { "string", "null" }));`
`52`	`51`	`}`
`53`	`52`
`54`	`53`	`[Test]`
`55`	`54`	`public void ModelTheCameraAsANestedObject()`
`56`	`55`	`{`
`57`		`- JToken camera = tool.OutputSchema!["properties"]!["camera"]!;`
	`56`	`+ JToken camera = tool.OutputSchema["properties"]!["camera"]!;`
`58`	`57`
`59`	`58`	`Assert.That(camera["type"]!.Value<string>(), Is.EqualTo("object"));`
`60`	`59`	`Assert.That(camera["properties"]!["mode"]!["type"]!.Value<string>(), Is.EqualTo("string"));`
`@@ -67,7 +66,7 @@ public void KeepTheOutputSchemaInSyncWithTheStructuredPayload()`
`67`	`66`	`var structured = (JObject)Execute().Payload["structuredContent"]!;`
`68`	`67`
`69`	`68`	`// Assert`
`70`		`- McpSchemaAssert.KeysMatch(tool.OutputSchema!, structured);`
	`69`	`+ McpSchemaAssert.KeysMatch(tool.OutputSchema, structured);`
`71`	`70`	`}`
`72`	`71`
`73`	`72`	`private McpToolResult Execute() =>`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public void ReportAnAbsentSceneAsAJsonNull()`
`64`	`64`	`public void DeclareAnObjectOutputSchemaThatAdmitsANullScene()`
`65`	`65`	`{`
`66`	`66`	`// Act`
`67`		`- JObject schema = tool.OutputSchema!;`
	`67`	`+ JObject schema = tool.OutputSchema;`
`68`	`68`
`69`	`69`	`// Assert`
`70`	`70`	`Assert.That(schema["type"]!.Value<string>(), Is.EqualTo("object"));`
`@@ -96,7 +96,7 @@ public void KeepTheOutputSchemaInSyncWithTheStructuredPayload()`
`96`	`96`	`var structured = (JObject)Execute().Payload["structuredContent"]!;`
`97`	`97`
`98`	`98`	`// Assert`
`99`		`- McpSchemaAssert.KeysMatch(tool.OutputSchema!, structured);`
	`99`	`+ McpSchemaAssert.KeysMatch(tool.OutputSchema, structured);`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`private McpToolResult Execute() =>`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ public void KeepTheOutputSchemaInSyncWithTheStructuredPayload()`
`85`	`85`	`var structured = (JObject)Execute(limit: 200).Payload["structuredContent"]!;`
`86`	`86`
`87`	`87`	`// Assert`
`88`		`- McpSchemaAssert.KeysMatch(tool.OutputSchema!, structured);`
	`88`	`+ McpSchemaAssert.KeysMatch(tool.OutputSchema, structured);`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`private McpToolResult Execute(int limit) =>`
Original file line number	Diff line number	Diff line change
`@@ -78,8 +78,8 @@ public void CallAKnownToolAndWrapItsResult()`
`78`	`78`	`Assert.That(tool.LastArguments!["value"]!.Value<string>(), Is.EqualTo("hi"));`
`79`	`79`
`80`	`80`	`var content = (JArray)result["content"]!;`
`81`		`- Assert.That(content[0]!["type"]!.Value<string>(), Is.EqualTo("text"));`
`82`		`- Assert.That(content[0]!["text"]!.Value<string>(), Is.EqualTo("done"));`
	`81`	`+ Assert.That(content[0]["type"]!.Value<string>(), Is.EqualTo("text"));`
	`82`	`+ Assert.That(content[0]["text"]!.Value<string>(), Is.EqualTo("done"));`
`83`	`83`	`Assert.That(result.ContainsKey("isError"), Is.False);`
`84`	`84`	`}`
`85`	`85`