diff --git a/engine/include/XCEngine/Rendering/AGENTS.md b/engine/include/XCEngine/Rendering/AGENTS.md index bdc2b291..6f4b09c6 100644 --- a/engine/include/XCEngine/Rendering/AGENTS.md +++ b/engine/include/XCEngine/Rendering/AGENTS.md @@ -152,3 +152,7 @@ Guardrails: `ScriptableRenderPipelineAsset`, owns renderer-data selection and runtime resource lifecycle itself, and creates `UniversalRenderPipeline` directly. The deleted bridge types must remain absent from the public API surface. +- Removed the public managed `SceneRenderInjectionPoint` type. Native scene + injection points remain internal built-in scene-recorder detail; managed SRP / + URP authors should use `RenderPassEvent`, renderer blocks, renderer-list draw + APIs, and RenderGraph declarations instead. diff --git a/managed/CMakeLists.txt b/managed/CMakeLists.txt index 768804cf..71998a14 100644 --- a/managed/CMakeLists.txt +++ b/managed/CMakeLists.txt @@ -198,7 +198,6 @@ set(XCENGINE_SCRIPT_CORE_SOURCES ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/RendererListDesc.cs ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/RendererListType.cs ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/RendererSortMode.cs - ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/SceneRenderInjectionPoint.cs ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/SceneRenderPhase.cs ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/ScriptableRenderContext.cs ${CMAKE_CURRENT_SOURCE_DIR}/XCEngine.ScriptCore/Rendering/Core/ScriptableRenderPipeline.cs diff --git a/managed/XCEngine.ScriptCore/Rendering/Core/SceneRenderInjectionPoint.cs b/managed/XCEngine.ScriptCore/Rendering/Core/SceneRenderInjectionPoint.cs deleted file mode 100644 index ad6ab707..00000000 --- a/managed/XCEngine.ScriptCore/Rendering/Core/SceneRenderInjectionPoint.cs +++ /dev/null @@ -1,12 +0,0 @@ -namespace XCEngine.Rendering -{ - public enum SceneRenderInjectionPoint - { - BeforeOpaque = 0, - AfterOpaque = 1, - BeforeSkybox = 2, - AfterSkybox = 3, - BeforeTransparent = 4, - AfterTransparent = 5 - } -} diff --git a/tests/scripting/test_mono_script_runtime.cpp b/tests/scripting/test_mono_script_runtime.cpp index 4ad74317..64ef5de0 100644 --- a/tests/scripting/test_mono_script_runtime.cpp +++ b/tests/scripting/test_mono_script_runtime.cpp @@ -1468,6 +1468,7 @@ TEST_F( bool hasRenderGraphRasterPassBuilderCommit = false; bool hasRenderGraphRasterContextType = false; bool hasRenderGraphRasterContextCommandBuffer = false; + bool hasSceneRenderInjectionPointType = false; bool hasRenderPassProtectedRecordColorScaleFullscreenPass = false; bool hasRenderPassProtectedRecordShaderVectorFullscreenPass = false; bool hasRenderPassProtectedRecordFinalColorFullscreenPass = false; @@ -1664,6 +1665,10 @@ TEST_F( selectionScript, "HasRenderGraphRasterContextCommandBuffer", hasRenderGraphRasterContextCommandBuffer)); + EXPECT_TRUE(runtime->TryGetFieldValue( + selectionScript, + "HasSceneRenderInjectionPointType", + hasSceneRenderInjectionPointType)); EXPECT_TRUE(runtime->TryGetFieldValue( selectionScript, "HasRenderPassProtectedRecordColorScaleFullscreenPass", @@ -1725,6 +1730,7 @@ TEST_F( EXPECT_TRUE(hasRenderGraphRasterPassBuilderCommit); EXPECT_TRUE(hasRenderGraphRasterContextType); EXPECT_TRUE(hasRenderGraphRasterContextCommandBuffer); + EXPECT_FALSE(hasSceneRenderInjectionPointType); EXPECT_FALSE(hasRenderPassProtectedRecordColorScaleFullscreenPass); EXPECT_FALSE(hasRenderPassProtectedRecordShaderVectorFullscreenPass); EXPECT_FALSE(hasRenderPassProtectedRecordFinalColorFullscreenPass);