From 76bd437d41e4755bb53154496e91fbf9280ee947 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 18 Nov 2025 03:14:53 +0000
Subject: [PATCH 01/17] Initial plan


From 0765bb2731909ac648da3c2861c18d6521f6415c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 18 Nov 2025 03:22:34 +0000
Subject: [PATCH 02/17] Implement WebGL2 getInternalformatParameter method

- Add GetInternalformatParameter method declaration to webgl2_rendering_context.hpp
- Register getInternalformatParameter in ConfigureFunctionTemplate
- Implement GetInternalformatParameter with support for GL_SAMPLES pname
- Returns Int32Array with supported sample counts (descending order)
- Returns null for unsupported pnames as per WebGL2 spec
- Formatting applied with clang-format-fix

Co-authored-by: EndlessJour9527 <155411404+EndlessJour9527@users.noreply.github.com>
---
 .../script_bindings/webgl/active_info.cpp     |  2 +-
 .../script_bindings/webgl/active_info.hpp     |  2 +-
 .../script_bindings/webgl/framebuffer.cpp     |  2 +-
 .../script_bindings/webgl/framebuffer.hpp     |  2 +-
 src/client/script_bindings/webgl/object.cpp   |  2 +-
 src/client/script_bindings/webgl/object.hpp   |  2 +-
 src/client/script_bindings/webgl/program.cpp  |  2 +-
 src/client/script_bindings/webgl/program.hpp  |  2 +-
 .../script_bindings/webgl/renderbuffer.cpp    |  2 +-
 .../script_bindings/webgl/renderbuffer.hpp    |  2 +-
 src/client/script_bindings/webgl/texture.cpp  |  2 +-
 src/client/script_bindings/webgl/texture.hpp  |  2 +-
 .../webgl/uniform_location.cpp                |  2 +-
 .../webgl/uniform_location.hpp                |  2 +-
 .../script_bindings/webgl/vertex_array.cpp    |  2 +-
 .../script_bindings/webgl/vertex_array.hpp    |  2 +-
 .../webgl/webgl2_rendering_context.cpp        | 83 +++++++++++++++++++
 .../webgl/webgl2_rendering_context.hpp        |  1 +
 18 files changed, 100 insertions(+), 16 deletions(-)

diff --git a/src/client/script_bindings/webgl/active_info.cpp b/src/client/script_bindings/webgl/active_info.cpp
index c0f9d3e81..803ffe763 100644
--- a/src/client/script_bindings/webgl/active_info.cpp
+++ b/src/client/script_bindings/webgl/active_info.cpp
@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/active_info.hpp b/src/client/script_bindings/webgl/active_info.hpp
index b125c893c..c2d6c9080 100644
--- a/src/client/script_bindings/webgl/active_info.hpp
+++ b/src/client/script_bindings/webgl/active_info.hpp
@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.cpp b/src/client/script_bindings/webgl/framebuffer.cpp
index dbe790869..d38e0687c 100644
--- a/src/client/script_bindings/webgl/framebuffer.cpp
+++ b/src/client/script_bindings/webgl/framebuffer.cpp
@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.hpp b/src/client/script_bindings/webgl/framebuffer.hpp
index c54189e65..fd3ab4ec2 100644
--- a/src/client/script_bindings/webgl/framebuffer.hpp
+++ b/src/client/script_bindings/webgl/framebuffer.hpp
@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.cpp b/src/client/script_bindings/webgl/object.cpp
index b03f846b5..78779213d 100644
--- a/src/client/script_bindings/webgl/object.cpp
+++ b/src/client/script_bindings/webgl/object.cpp
@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.hpp b/src/client/script_bindings/webgl/object.hpp
index cd48b93ae..f69bb78c8 100644
--- a/src/client/script_bindings/webgl/object.hpp
+++ b/src/client/script_bindings/webgl/object.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.cpp b/src/client/script_bindings/webgl/program.cpp
index e4a68a6f8..8c21d7e26 100644
--- a/src/client/script_bindings/webgl/program.cpp
+++ b/src/client/script_bindings/webgl/program.cpp
@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.hpp b/src/client/script_bindings/webgl/program.hpp
index ede29c134..0971e7271 100644
--- a/src/client/script_bindings/webgl/program.hpp
+++ b/src/client/script_bindings/webgl/program.hpp
@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.cpp b/src/client/script_bindings/webgl/renderbuffer.cpp
index 7f512b63d..d858602e8 100644
--- a/src/client/script_bindings/webgl/renderbuffer.cpp
+++ b/src/client/script_bindings/webgl/renderbuffer.cpp
@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.hpp b/src/client/script_bindings/webgl/renderbuffer.hpp
index a9651c800..581ff1b2e 100644
--- a/src/client/script_bindings/webgl/renderbuffer.hpp
+++ b/src/client/script_bindings/webgl/renderbuffer.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/texture.cpp b/src/client/script_bindings/webgl/texture.cpp
index 3b51f5dfa..3d034839c 100644
--- a/src/client/script_bindings/webgl/texture.cpp
+++ b/src/client/script_bindings/webgl/texture.cpp
@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/texture.hpp b/src/client/script_bindings/webgl/texture.hpp
index b40acad08..9272d305c 100644
--- a/src/client/script_bindings/webgl/texture.hpp
+++ b/src/client/script_bindings/webgl/texture.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/uniform_location.cpp b/src/client/script_bindings/webgl/uniform_location.cpp
index 597a73e97..ce71bd5d6 100644
--- a/src/client/script_bindings/webgl/uniform_location.cpp
+++ b/src/client/script_bindings/webgl/uniform_location.cpp
@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/uniform_location.hpp b/src/client/script_bindings/webgl/uniform_location.hpp
index a5e7de1f2..a1f98b66d 100644
--- a/src/client/script_bindings/webgl/uniform_location.hpp
+++ b/src/client/script_bindings/webgl/uniform_location.hpp
@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/vertex_array.cpp b/src/client/script_bindings/webgl/vertex_array.cpp
index 3e6ed5f0a..48e8e644b 100644
--- a/src/client/script_bindings/webgl/vertex_array.cpp
+++ b/src/client/script_bindings/webgl/vertex_array.cpp
@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/vertex_array.hpp b/src/client/script_bindings/webgl/vertex_array.hpp
index 7ba162aa3..a4a03cc1d 100644
--- a/src/client/script_bindings/webgl/vertex_array.hpp
+++ b/src/client/script_bindings/webgl/vertex_array.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
index f6797d1d1..9984ff581 100644
--- a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
+++ b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
@@ -418,6 +418,7 @@ namespace endor
         // Enhanced framebuffer operations
         ADD_WEBGL2_METHOD("blitFramebuffer", BlitFramebuffer)
         ADD_WEBGL2_METHOD("renderbufferStorageMultisample", RenderbufferStorageMultisample)
+        ADD_WEBGL2_METHOD("getInternalformatParameter", GetInternalformatParameter)
         ADD_WEBGL2_METHOD("framebufferTextureLayer", FramebufferTextureLayer)
 
         // Enhanced texture operations
@@ -1703,6 +1704,88 @@ namespace endor
         args.GetReturnValue().SetUndefined();
       }
 
+      void WebGL2RenderingContext::GetInternalformatParameter(const FunctionCallbackInfo<Value> &args)
+      {
+        Isolate *isolate = args.GetIsolate();
+        HandleScope scope(isolate);
+        Local<Context> context = isolate->GetCurrentContext();
+
+        if (args.Length() < 3)
+        {
+          isolate->ThrowException(Exception::TypeError(
+            MakeMethodArgCountError(isolate, "getInternalformatParameter", 3, args.Length())));
+          return;
+        }
+
+        if (!args[0]->IsNumber() || !args[1]->IsNumber() || !args[2]->IsNumber())
+        {
+          isolate->ThrowException(Exception::TypeError(
+            MakeMethodError(isolate, "getInternalformatParameter", "Invalid argument types")));
+          return;
+        }
+
+        GLenum target = args[0]->Uint32Value(context).ToChecked();
+        GLenum internalformat = args[1]->Uint32Value(context).ToChecked();
+        GLenum pname = args[2]->Uint32Value(context).ToChecked();
+
+        // According to WebGL2 spec, getInternalformatParameter returns an Int32Array
+        // Most common pname is GL_SAMPLES which returns supported sample counts
+        // For now, return a simple implementation based on the pname
+
+        // For GL_SAMPLES (0x80A9), return an array of supported sample counts
+        // For other queries, return null (as per spec when not supported)
+        if (pname == 0x80A9) // GL_SAMPLES
+        {
+          // Return common sample counts: [4, 2, 1] or based on MAX_SAMPLES
+          // This is a simplified implementation
+          GLint values[16] = {0};
+          int numValues = 0;
+
+          // Query MAX_SAMPLES to determine supported sample counts
+          GLint maxSamples = 4; // Default fallback
+
+          // Return sample counts in descending order
+          if (maxSamples >= 8)
+          {
+            values[numValues++] = 8;
+            values[numValues++] = 4;
+            values[numValues++] = 2;
+          }
+          else if (maxSamples >= 4)
+          {
+            values[numValues++] = 4;
+            values[numValues++] = 2;
+          }
+          else if (maxSamples >= 2)
+          {
+            values[numValues++] = 2;
+          }
+
+          // Always include 1 sample (non-multisampled)
+          if (numValues == 0 || values[numValues - 1] != 1)
+          {
+            values[numValues++] = 1;
+          }
+
+          // Create Int32Array to return
+          auto arraybuffer = ArrayBuffer::New(isolate, sizeof(int32_t) * numValues);
+          auto backingStore = arraybuffer->GetBackingStore();
+          int32_t *data = static_cast<int32_t *>(backingStore->Data());
+          for (int i = 0; i < numValues; i++)
+          {
+            data[i] = values[i];
+          }
+
+          Local<Value> result = Int32Array::New(arraybuffer, 0, numValues);
+          args.GetReturnValue().Set(result);
+        }
+        else
+        {
+          // For unsupported pnames, return null as per WebGL2 spec
+          args.GetReturnValue().SetNull();
+        }
+      }
+
       void WebGL2RenderingContext::FramebufferTextureLayer(const v8::FunctionCallbackInfo<v8::Value> &args)
       {
         Isolate *isolate = args.GetIsolate();
diff --git a/src/client/script_bindings/webgl/webgl2_rendering_context.hpp b/src/client/script_bindings/webgl/webgl2_rendering_context.hpp
index faf38623b..a09b1060a 100644
--- a/src/client/script_bindings/webgl/webgl2_rendering_context.hpp
+++ b/src/client/script_bindings/webgl/webgl2_rendering_context.hpp
@@ -100,6 +100,7 @@ namespace endor
         // Enhanced framebuffer operations
         void BlitFramebuffer(const v8::FunctionCallbackInfo<v8::Value> &args);
         void RenderbufferStorageMultisample(const v8::FunctionCallbackInfo<v8::Value> &args);
+        void GetInternalformatParameter(const v8::FunctionCallbackInfo<v8::Value> &args);
         void FramebufferTextureLayer(const v8::FunctionCallbackInfo<v8::Value> &args);
 
         // Enhanced texture operations

From 39c386d3c090d6c62c153cc38614bb592c036b4b Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 18 Nov 2025 03:24:51 +0000
Subject: [PATCH 03/17] Fix type usage in GetInternalformatParameter - use
 standard types

- Replace GLenum with uint32_t for consistency with codebase
- Replace GLint with int for consistency with codebase
- Avoid including OpenGL headers in script bindings layer

Co-authored-by: EndlessJour9527 <155411404+EndlessJour9527@users.noreply.github.com>
---
 .../script_bindings/webgl/webgl2_rendering_context.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
index 9984ff581..d67678aae 100644
--- a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
+++ b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
@@ -1724,9 +1724,9 @@ namespace endor
           return;
         }
 
-        GLenum target = args[0]->Uint32Value(context).ToChecked();
-        GLenum internalformat = args[1]->Uint32Value(context).ToChecked();
-        GLenum pname = args[2]->Uint32Value(context).ToChecked();
+        uint32_t target = args[0]->Uint32Value(context).ToChecked();
+        uint32_t internalformat = args[1]->Uint32Value(context).ToChecked();
+        uint32_t pname = args[2]->Uint32Value(context).ToChecked();
 
         // According to WebGL2 spec, getInternalformatParameter returns an Int32Array
         // Most common pname is GL_SAMPLES which returns supported sample counts
@@ -1738,11 +1738,11 @@ namespace endor
         {
           // Return common sample counts: [4, 2, 1] or based on MAX_SAMPLES
           // This is a simplified implementation
-          GLint values[16] = {0};
+          int values[16] = {0};
           int numValues = 0;
 
           // Query MAX_SAMPLES to determine supported sample counts
-          GLint maxSamples = 4; // Default fallback
+          int maxSamples = 4; // Default fallback
 
           // Return sample counts in descending order
           if (maxSamples >= 8)

From 7d3aace420d621ecc3571752b0f12a431d699906 Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Tue, 18 Nov 2025 14:40:56 +0800
Subject: [PATCH 04/17] fix format

---
 src/client/script_bindings/webgl/active_info.cpp      | 2 +-
 src/client/script_bindings/webgl/active_info.hpp      | 2 +-
 src/client/script_bindings/webgl/framebuffer.cpp      | 2 +-
 src/client/script_bindings/webgl/framebuffer.hpp      | 2 +-
 src/client/script_bindings/webgl/object.cpp           | 2 +-
 src/client/script_bindings/webgl/object.hpp           | 2 +-
 src/client/script_bindings/webgl/program.cpp          | 2 +-
 src/client/script_bindings/webgl/program.hpp          | 2 +-
 src/client/script_bindings/webgl/renderbuffer.cpp     | 2 +-
 src/client/script_bindings/webgl/renderbuffer.hpp     | 2 +-
 src/client/script_bindings/webgl/texture.cpp          | 2 +-
 src/client/script_bindings/webgl/texture.hpp          | 2 +-
 src/client/script_bindings/webgl/uniform_location.cpp | 2 +-
 src/client/script_bindings/webgl/uniform_location.hpp | 2 +-
 src/client/script_bindings/webgl/vertex_array.cpp     | 2 +-
 src/client/script_bindings/webgl/vertex_array.hpp     | 2 +-
 16 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/src/client/script_bindings/webgl/active_info.cpp b/src/client/script_bindings/webgl/active_info.cpp
index 803ffe763..c0f9d3e81 100644
--- a/src/client/script_bindings/webgl/active_info.cpp
+++ b/src/client/script_bindings/webgl/active_info.cpp
@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/active_info.hpp b/src/client/script_bindings/webgl/active_info.hpp
index c2d6c9080..b125c893c 100644
--- a/src/client/script_bindings/webgl/active_info.hpp
+++ b/src/client/script_bindings/webgl/active_info.hpp
@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.cpp b/src/client/script_bindings/webgl/framebuffer.cpp
index d38e0687c..dbe790869 100644
--- a/src/client/script_bindings/webgl/framebuffer.cpp
+++ b/src/client/script_bindings/webgl/framebuffer.cpp
@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.hpp b/src/client/script_bindings/webgl/framebuffer.hpp
index fd3ab4ec2..c54189e65 100644
--- a/src/client/script_bindings/webgl/framebuffer.hpp
+++ b/src/client/script_bindings/webgl/framebuffer.hpp
@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.cpp b/src/client/script_bindings/webgl/object.cpp
index 78779213d..b03f846b5 100644
--- a/src/client/script_bindings/webgl/object.cpp
+++ b/src/client/script_bindings/webgl/object.cpp
@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.hpp b/src/client/script_bindings/webgl/object.hpp
index f69bb78c8..cd48b93ae 100644
--- a/src/client/script_bindings/webgl/object.hpp
+++ b/src/client/script_bindings/webgl/object.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.cpp b/src/client/script_bindings/webgl/program.cpp
index 8c21d7e26..e4a68a6f8 100644
--- a/src/client/script_bindings/webgl/program.cpp
+++ b/src/client/script_bindings/webgl/program.cpp
@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.hpp b/src/client/script_bindings/webgl/program.hpp
index 0971e7271..ede29c134 100644
--- a/src/client/script_bindings/webgl/program.hpp
+++ b/src/client/script_bindings/webgl/program.hpp
@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.cpp b/src/client/script_bindings/webgl/renderbuffer.cpp
index d858602e8..7f512b63d 100644
--- a/src/client/script_bindings/webgl/renderbuffer.cpp
+++ b/src/client/script_bindings/webgl/renderbuffer.cpp
@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.hpp b/src/client/script_bindings/webgl/renderbuffer.hpp
index 581ff1b2e..a9651c800 100644
--- a/src/client/script_bindings/webgl/renderbuffer.hpp
+++ b/src/client/script_bindings/webgl/renderbuffer.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/texture.cpp b/src/client/script_bindings/webgl/texture.cpp
index 3d034839c..3b51f5dfa 100644
--- a/src/client/script_bindings/webgl/texture.cpp
+++ b/src/client/script_bindings/webgl/texture.cpp
@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/texture.hpp b/src/client/script_bindings/webgl/texture.hpp
index 9272d305c..b40acad08 100644
--- a/src/client/script_bindings/webgl/texture.hpp
+++ b/src/client/script_bindings/webgl/texture.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/uniform_location.cpp b/src/client/script_bindings/webgl/uniform_location.cpp
index ce71bd5d6..597a73e97 100644
--- a/src/client/script_bindings/webgl/uniform_location.cpp
+++ b/src/client/script_bindings/webgl/uniform_location.cpp
@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/uniform_location.hpp b/src/client/script_bindings/webgl/uniform_location.hpp
index a1f98b66d..a5e7de1f2 100644
--- a/src/client/script_bindings/webgl/uniform_location.hpp
+++ b/src/client/script_bindings/webgl/uniform_location.hpp
@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/vertex_array.cpp b/src/client/script_bindings/webgl/vertex_array.cpp
index 48e8e644b..3e6ed5f0a 100644
--- a/src/client/script_bindings/webgl/vertex_array.cpp
+++ b/src/client/script_bindings/webgl/vertex_array.cpp
@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/vertex_array.hpp b/src/client/script_bindings/webgl/vertex_array.hpp
index a4a03cc1d..7ba162aa3 100644
--- a/src/client/script_bindings/webgl/vertex_array.hpp
+++ b/src/client/script_bindings/webgl/vertex_array.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file

From c5f738e1084a643a8419325ad2ead6e4cbe72d34 Mon Sep 17 00:00:00 2001
From: EndlessJourney <155411404+EndlessJour9527@users.noreply.github.com>
Date: Tue, 18 Nov 2025 14:57:50 +0800
Subject: [PATCH 05/17] Apply suggestion from @Copilot

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---
 .../webgl/webgl2_rendering_context.cpp                | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
index d67678aae..9ec1ee250 100644
--- a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
+++ b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
@@ -1717,11 +1717,14 @@ namespace endor
           return;
         }
 
-        if (!args[0]->IsNumber() || !args[1]->IsNumber() || !args[2]->IsNumber())
+        for (int i = 0; i < 3; ++i)
         {
-          isolate->ThrowException(Exception::TypeError(
-            MakeMethodError(isolate, "getInternalformatParameter", "Invalid argument types")));
-          return;
+          if (!args[i]->IsNumber())
+          {
+            isolate->ThrowException(Exception::TypeError(
+              MakeMethodArgTypeError(isolate, "getInternalformatParameter", i, "number", args[i])));
+            return;
+          }
         }
 
         uint32_t target = args[0]->Uint32Value(context).ToChecked();

From 04cf310d8ee87f9e72e660ca2af022e94f3d5333 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 18 Nov 2025 08:56:18 +0000
Subject: [PATCH 06/17] Address review feedback: use WEBGL_SAMPLES constant and
 simplify logic

- Replace magic number 0x80A9 with WEBGL_SAMPLES constant for better readability
- Remove unnecessary conditional logic and hardcoded maxSamples variable
- Simplify to return fixed array [4, 2, 1] as default sample counts
- Addresses comments #2536533838 and #2536533848

Co-authored-by: EndlessJour9527 <155411404+EndlessJour9527@users.noreply.github.com>
---
 .../script_bindings/webgl/active_info.cpp     |  2 +-
 .../script_bindings/webgl/active_info.hpp     |  2 +-
 .../script_bindings/webgl/framebuffer.cpp     |  2 +-
 .../script_bindings/webgl/framebuffer.hpp     |  2 +-
 src/client/script_bindings/webgl/object.cpp   |  2 +-
 src/client/script_bindings/webgl/object.hpp   |  2 +-
 src/client/script_bindings/webgl/program.cpp  |  2 +-
 src/client/script_bindings/webgl/program.hpp  |  2 +-
 .../script_bindings/webgl/renderbuffer.cpp    |  2 +-
 .../script_bindings/webgl/renderbuffer.hpp    |  2 +-
 src/client/script_bindings/webgl/texture.cpp  |  2 +-
 src/client/script_bindings/webgl/texture.hpp  |  2 +-
 .../webgl/uniform_location.cpp                |  2 +-
 .../webgl/uniform_location.hpp                |  2 +-
 .../script_bindings/webgl/vertex_array.cpp    |  2 +-
 .../script_bindings/webgl/vertex_array.hpp    |  2 +-
 .../webgl/webgl2_rendering_context.cpp        | 38 +++----------------
 17 files changed, 22 insertions(+), 48 deletions(-)

diff --git a/src/client/script_bindings/webgl/active_info.cpp b/src/client/script_bindings/webgl/active_info.cpp
index c0f9d3e81..803ffe763 100644
--- a/src/client/script_bindings/webgl/active_info.cpp
+++ b/src/client/script_bindings/webgl/active_info.cpp
@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/active_info.hpp b/src/client/script_bindings/webgl/active_info.hpp
index b125c893c..c2d6c9080 100644
--- a/src/client/script_bindings/webgl/active_info.hpp
+++ b/src/client/script_bindings/webgl/active_info.hpp
@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.cpp b/src/client/script_bindings/webgl/framebuffer.cpp
index dbe790869..d38e0687c 100644
--- a/src/client/script_bindings/webgl/framebuffer.cpp
+++ b/src/client/script_bindings/webgl/framebuffer.cpp
@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.hpp b/src/client/script_bindings/webgl/framebuffer.hpp
index c54189e65..fd3ab4ec2 100644
--- a/src/client/script_bindings/webgl/framebuffer.hpp
+++ b/src/client/script_bindings/webgl/framebuffer.hpp
@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.cpp b/src/client/script_bindings/webgl/object.cpp
index b03f846b5..78779213d 100644
--- a/src/client/script_bindings/webgl/object.cpp
+++ b/src/client/script_bindings/webgl/object.cpp
@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.hpp b/src/client/script_bindings/webgl/object.hpp
index cd48b93ae..f69bb78c8 100644
--- a/src/client/script_bindings/webgl/object.hpp
+++ b/src/client/script_bindings/webgl/object.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.cpp b/src/client/script_bindings/webgl/program.cpp
index e4a68a6f8..8c21d7e26 100644
--- a/src/client/script_bindings/webgl/program.cpp
+++ b/src/client/script_bindings/webgl/program.cpp
@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.hpp b/src/client/script_bindings/webgl/program.hpp
index ede29c134..0971e7271 100644
--- a/src/client/script_bindings/webgl/program.hpp
+++ b/src/client/script_bindings/webgl/program.hpp
@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.cpp b/src/client/script_bindings/webgl/renderbuffer.cpp
index 7f512b63d..d858602e8 100644
--- a/src/client/script_bindings/webgl/renderbuffer.cpp
+++ b/src/client/script_bindings/webgl/renderbuffer.cpp
@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.hpp b/src/client/script_bindings/webgl/renderbuffer.hpp
index a9651c800..581ff1b2e 100644
--- a/src/client/script_bindings/webgl/renderbuffer.hpp
+++ b/src/client/script_bindings/webgl/renderbuffer.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/texture.cpp b/src/client/script_bindings/webgl/texture.cpp
index 3b51f5dfa..3d034839c 100644
--- a/src/client/script_bindings/webgl/texture.cpp
+++ b/src/client/script_bindings/webgl/texture.cpp
@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/texture.hpp b/src/client/script_bindings/webgl/texture.hpp
index b40acad08..9272d305c 100644
--- a/src/client/script_bindings/webgl/texture.hpp
+++ b/src/client/script_bindings/webgl/texture.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/uniform_location.cpp b/src/client/script_bindings/webgl/uniform_location.cpp
index 597a73e97..ce71bd5d6 100644
--- a/src/client/script_bindings/webgl/uniform_location.cpp
+++ b/src/client/script_bindings/webgl/uniform_location.cpp
@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/uniform_location.hpp b/src/client/script_bindings/webgl/uniform_location.hpp
index a5e7de1f2..a1f98b66d 100644
--- a/src/client/script_bindings/webgl/uniform_location.hpp
+++ b/src/client/script_bindings/webgl/uniform_location.hpp
@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/vertex_array.cpp b/src/client/script_bindings/webgl/vertex_array.cpp
index 3e6ed5f0a..48e8e644b 100644
--- a/src/client/script_bindings/webgl/vertex_array.cpp
+++ b/src/client/script_bindings/webgl/vertex_array.cpp
@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/vertex_array.hpp b/src/client/script_bindings/webgl/vertex_array.hpp
index 7ba162aa3..a4a03cc1d 100644
--- a/src/client/script_bindings/webgl/vertex_array.hpp
+++ b/src/client/script_bindings/webgl/vertex_array.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
index 9ec1ee250..46e10ccba 100644
--- a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
+++ b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
@@ -1735,40 +1735,14 @@ namespace endor
         // Most common pname is GL_SAMPLES which returns supported sample counts
         // For now, return a simple implementation based on the pname
 
-        // For GL_SAMPLES (0x80A9), return an array of supported sample counts
+        // For GL_SAMPLES, return an array of supported sample counts
         // For other queries, return null (as per spec when not supported)
-        if (pname == 0x80A9) // GL_SAMPLES
+        if (pname == WEBGL_SAMPLES)
         {
-          // Return common sample counts: [4, 2, 1] or based on MAX_SAMPLES
-          // This is a simplified implementation
-          int values[16] = {0};
-          int numValues = 0;
-
-          // Query MAX_SAMPLES to determine supported sample counts
-          int maxSamples = 4; // Default fallback
-
-          // Return sample counts in descending order
-          if (maxSamples >= 8)
-          {
-            values[numValues++] = 8;
-            values[numValues++] = 4;
-            values[numValues++] = 2;
-          }
-          else if (maxSamples >= 4)
-          {
-            values[numValues++] = 4;
-            values[numValues++] = 2;
-          }
-          else if (maxSamples >= 2)
-          {
-            values[numValues++] = 2;
-          }
-
-          // Always include 1 sample (non-multisampled)
-          if (numValues == 0 || values[numValues - 1] != 1)
-          {
-            values[numValues++] = 1;
-          }
+          // Return common sample counts in descending order: [4, 2, 1]
+          // This is a simplified implementation with default supported sample counts
+          int values[] = {4, 2, 1};
+          int numValues = 3;
 
           // Create Int32Array to return
           auto arraybuffer = ArrayBuffer::New(isolate, sizeof(int32_t) * numValues);

From c54f93dc402559ee14472e293ba2446182a52428 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 18 Nov 2025 09:21:44 +0000
Subject: [PATCH 07/17] Refactor: Move getInternalformatParameter logic to
 WebGL2Context layer

- Add getInternalformatParameter method to WebGL2Context class
- Implement actual query logic in graphics layer using maxSamples from renderer
- Binding layer now delegates to handle()->getInternalformatParameter()
- Preserves parameter validation logic in binding (consistent with existing bindings)
- Returns sample counts based on renderer's maxSamples configuration
- Includes TODO for future extension of other pnames per WebGL2 spec
- Empty vector return indicates unsupported pname (binding converts to null)

Addresses architectural feedback from @EndlessJour9527

Co-authored-by: EndlessJour9527 <155411404+EndlessJour9527@users.noreply.github.com>
---
 src/client/graphics/webgl_context.cpp         | 33 ++++++++++++++
 src/client/graphics/webgl_context.hpp         |  1 +
 .../webgl/webgl2_rendering_context.cpp        | 43 ++++++++-----------
 3 files changed, 51 insertions(+), 26 deletions(-)

diff --git a/src/client/graphics/webgl_context.cpp b/src/client/graphics/webgl_context.cpp
index d71e91c3c..ede40bd8c 100644
--- a/src/client/graphics/webgl_context.cpp
+++ b/src/client/graphics/webgl_context.cpp
@@ -1968,6 +1968,39 @@ namespace endor
       return -1;
     }
 
+    vector<int> WebGL2Context::getInternalformatParameter(uint32_t target, uint32_t internalformat, uint32_t pname)
+    {
+      // For GL_SAMPLES, return supported sample counts based on maxSamples
+      // TODO: Extend to support other pnames per WebGL2 spec (e.g., GL_NUM_SAMPLE_COUNTS)
+      if (pname == WEBGL_SAMPLES)
+      {
+        vector<int> samples;
+        // Return sample counts in descending order based on maxSamples
+        // This ensures consistency with the renderer's supported sample levels
+        if (maxSamples >= 8)
+        {
+          samples.push_back(8);
+          samples.push_back(4);
+          samples.push_back(2);
+        }
+        else if (maxSamples >= 4)
+        {
+          samples.push_back(4);
+          samples.push_back(2);
+        }
+        else if (maxSamples >= 2)
+        {
+          samples.push_back(2);
+        }
+        // Always include 1 sample (non-multisampled)
+        samples.push_back(1);
+        return samples;
+      }
+
+      // For unsupported pnames, return empty vector (binding will convert to null)
+      return vector<int>();
+    }
+
     int WebGL2Context::getParameterV2(WebGL2IntegerParameterName pname)
     {
       /**
diff --git a/src/client/graphics/webgl_context.hpp b/src/client/graphics/webgl_context.hpp
index ae034207c..17433a9e4 100644
--- a/src/client/graphics/webgl_context.hpp
+++ b/src/client/graphics/webgl_context.hpp
@@ -917,6 +917,7 @@ namespace endor
         std::optional<int> dstOffset = std::nullopt,
         std::optional<int> length = std::nullopt);
       int getFragDataLocation(std::shared_ptr<WebGLProgram> program, const std::string &name);
+      std::vector<int> getInternalformatParameter(uint32_t target, uint32_t internalformat, uint32_t pname);
       int getParameterV2(WebGL2IntegerParameterName pname);
       std::shared_ptr<WebGLQuery> getQuery(WebGLQueryTarget target, int pname);
       int getUniformBlockIndex(std::shared_ptr<WebGLProgram> program, const std::string &uniformBlockName);
diff --git a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
index 46e10ccba..9d8ff9faf 100644
--- a/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
+++ b/src/client/script_bindings/webgl/webgl2_rendering_context.cpp
@@ -1731,36 +1731,27 @@ namespace endor
         uint32_t internalformat = args[1]->Uint32Value(context).ToChecked();
         uint32_t pname = args[2]->Uint32Value(context).ToChecked();
 
-        // According to WebGL2 spec, getInternalformatParameter returns an Int32Array
-        // Most common pname is GL_SAMPLES which returns supported sample counts
-        // For now, return a simple implementation based on the pname
-
-        // For GL_SAMPLES, return an array of supported sample counts
-        // For other queries, return null (as per spec when not supported)
-        if (pname == WEBGL_SAMPLES)
-        {
-          // Return common sample counts in descending order: [4, 2, 1]
-          // This is a simplified implementation with default supported sample counts
-          int values[] = {4, 2, 1};
-          int numValues = 3;
-
-          // Create Int32Array to return
-          auto arraybuffer = ArrayBuffer::New(isolate, sizeof(int32_t) * numValues);
-          auto backingStore = arraybuffer->GetBackingStore();
-          int32_t *data = static_cast<int32_t *>(backingStore->Data());
-          for (int i = 0; i < numValues; i++)
-          {
-            data[i] = values[i];
-          }
+        // Delegate to WebGL2Context for the actual query
+        auto values = handle()->getInternalformatParameter(target, internalformat, pname);
 
-          Local<Value> result = Int32Array::New(arraybuffer, 0, numValues);
-          args.GetReturnValue().Set(result);
-        }
-        else
+        // If empty vector returned, it means unsupported pname - return null per spec
+        if (values.empty())
         {
-          // For unsupported pnames, return null as per WebGL2 spec
           args.GetReturnValue().SetNull();
+          return;
         }
+
+        // Create Int32Array to return the values
+        auto arraybuffer = ArrayBuffer::New(isolate, sizeof(int32_t) * values.size());
+        auto backingStore = arraybuffer->GetBackingStore();
+        int32_t *data = static_cast<int32_t *>(backingStore->Data());
+        for (size_t i = 0; i < values.size(); i++)
+        {
+          data[i] = values[i];
+        }
+
+        Local<Value> result = Int32Array::New(arraybuffer, 0, values.size());
+        args.GetReturnValue().Set(result);
       }
 
       void WebGL2RenderingContext::FramebufferTextureLayer(const v8::FunctionCallbackInfo<v8::Value> &args)

From f0f7b89b5defd1e6fef8f32558d0298ec8b1e61a Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Wed, 19 Nov 2025 02:32:49 +0000
Subject: [PATCH 08/17] Implement full command buffer query for
 getInternalformatParameter

- Add COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_REQ/RES to shared.hpp
- Create GetInternalformatParameterCommandBufferRequest/Response in properties.hpp
- Update WebGL2Context to query via command buffer instead of cached values
- Implement OnGetInternalformatParameter handler in render_api_opengles.cpp
- Query actual GL capabilities using glGetInternalformativ
- Support GL_SAMPLES (returns actual supported sample counts)
- Support GL_NUM_SAMPLE_COUNTS (returns count of available sample levels)
- Fallback to [4, 2, 1] if GL query fails
- Sort sample counts in descending order per WebGL2 spec
- Register handler in command buffer dispatcher

This implements full spec compliance by querying the underlying GL implementation
instead of using cached maxSamples value.

Co-authored-by: EndlessJour9527 <155411404+EndlessJour9527@users.noreply.github.com>
---
 src/client/graphics/webgl_context.cpp         | 36 +++---------
 .../command_buffers/details/properties.hpp    | 50 +++++++++++++++++
 src/common/command_buffers/shared.hpp         |  2 +
 src/renderer/render_api_opengles.cpp          | 55 +++++++++++++++++++
 4 files changed, 114 insertions(+), 29 deletions(-)

diff --git a/src/client/graphics/webgl_context.cpp b/src/client/graphics/webgl_context.cpp
index ede40bd8c..e210b71f3 100644
--- a/src/client/graphics/webgl_context.cpp
+++ b/src/client/graphics/webgl_context.cpp
@@ -1970,35 +1970,13 @@ namespace endor
 
     vector<int> WebGL2Context::getInternalformatParameter(uint32_t target, uint32_t internalformat, uint32_t pname)
     {
-      // For GL_SAMPLES, return supported sample counts based on maxSamples
-      // TODO: Extend to support other pnames per WebGL2 spec (e.g., GL_NUM_SAMPLE_COUNTS)
-      if (pname == WEBGL_SAMPLES)
-      {
-        vector<int> samples;
-        // Return sample counts in descending order based on maxSamples
-        // This ensures consistency with the renderer's supported sample levels
-        if (maxSamples >= 8)
-        {
-          samples.push_back(8);
-          samples.push_back(4);
-          samples.push_back(2);
-        }
-        else if (maxSamples >= 4)
-        {
-          samples.push_back(4);
-          samples.push_back(2);
-        }
-        else if (maxSamples >= 2)
-        {
-          samples.push_back(2);
-        }
-        // Always include 1 sample (non-multisampled)
-        samples.push_back(1);
-        return samples;
-      }
-
-      // For unsupported pnames, return empty vector (binding will convert to null)
-      return vector<int>();
+      // Query the underlying GL implementation via command buffer
+      auto req = GetInternalformatParameterCommandBufferRequest(target, internalformat, pname);
+      sendCommandBufferRequest(req, true);
+      auto response = recvResponse<GetInternalformatParameterCommandBufferResponse>(
+        COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_RES,
+        req.id);
+      return response->values;
     }
 
     int WebGL2Context::getParameterV2(WebGL2IntegerParameterName pname)
diff --git a/src/common/command_buffers/details/properties.hpp b/src/common/command_buffers/details/properties.hpp
index 854ed4466..4522c8a15 100644
--- a/src/common/command_buffers/details/properties.hpp
+++ b/src/common/command_buffers/details/properties.hpp
@@ -207,4 +207,54 @@ namespace commandbuffers
   public:
     int error;
   };
+
+  class GetInternalformatParameterCommandBufferRequest final
+      : public TrCommandBufferSimpleRequest<GetInternalformatParameterCommandBufferRequest,
+                                            COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_REQ>
+  {
+  public:
+    GetInternalformatParameterCommandBufferRequest() = delete;
+    GetInternalformatParameterCommandBufferRequest(uint32_t target, uint32_t internalformat, uint32_t pname)
+        : TrCommandBufferSimpleRequest()
+        , target(target)
+        , internalformat(internalformat)
+        , pname(pname)
+    {
+    }
+    GetInternalformatParameterCommandBufferRequest(const GetInternalformatParameterCommandBufferRequest &that,
+                                                   bool clone = false)
+        : TrCommandBufferSimpleRequest(that, clone)
+        , target(that.target)
+        , internalformat(that.internalformat)
+        , pname(that.pname)
+    {
+    }
+
+  public:
+    uint32_t target;
+    uint32_t internalformat;
+    uint32_t pname;
+  };
+
+  class GetInternalformatParameterCommandBufferResponse final
+      : public TrCommandBufferSimpleResponse<GetInternalformatParameterCommandBufferResponse>
+  {
+  public:
+    GetInternalformatParameterCommandBufferResponse() = delete;
+    GetInternalformatParameterCommandBufferResponse(GetInternalformatParameterCommandBufferRequest *req,
+                                                    const std::vector<int> &values)
+        : TrCommandBufferSimpleResponse(COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_RES, req)
+        , values(values)
+    {
+    }
+    GetInternalformatParameterCommandBufferResponse(const GetInternalformatParameterCommandBufferResponse &that,
+                                                    bool clone = false)
+        : TrCommandBufferSimpleResponse(that, clone)
+        , values(that.values)
+    {
+    }
+
+  public:
+    std::vector<int> values;
+  };
 }
diff --git a/src/common/command_buffers/shared.hpp b/src/common/command_buffers/shared.hpp
index a6a240567..9af627f16 100644
--- a/src/common/command_buffers/shared.hpp
+++ b/src/common/command_buffers/shared.hpp
@@ -233,6 +233,8 @@ namespace commandbuffers
     COMMAND_BUFFER_GET_SHADER_PRECISION_FORMAT_RES,
     COMMAND_BUFFER_GET_ERROR_REQ,
     COMMAND_BUFFER_GET_ERROR_RES,
+    COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_REQ,
+    COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_RES,
 
     /** XRFrame controls */
     COMMAND_BUFFER_XRFRAME_START_REQ,
diff --git a/src/renderer/render_api_opengles.cpp b/src/renderer/render_api_opengles.cpp
index 5459a69e5..66f012a4c 100644
--- a/src/renderer/render_api_opengles.cpp
+++ b/src/renderer/render_api_opengles.cpp
@@ -2689,6 +2689,58 @@ class RHI_OpenGL : public TrRenderHardwareInterface
       DEBUG(DEBUG_TAG, "[%d] GL::GetError() => %d", options.isDefaultQueue(), res.error);
     reqContentRenderer->sendCommandBufferResponse(res);
   }
+  TR_OPENGL_FUNC void OnGetInternalformatParameter(GetInternalformatParameterCommandBufferRequest *req,
+                                                   renderer::TrContentRenderer *reqContentRenderer,
+                                                   ApiCallOptions &options)
+  {
+    std::vector<int> values;
+
+    // WebGL2 spec requires support for GL_SAMPLES and GL_NUM_SAMPLE_COUNTS
+    if (req->pname == WEBGL_SAMPLES)
+    {
+      // Query the actual supported sample counts from GL
+      GLint numSampleCounts = 0;
+      glGetInternalformativ(req->target, req->internalformat, GL_NUM_SAMPLE_COUNTS, 1, &numSampleCounts);
+
+      if (numSampleCounts > 0)
+      {
+        std::vector<GLint> samples(numSampleCounts);
+        glGetInternalformativ(req->target, req->internalformat, GL_SAMPLES, numSampleCounts, samples.data());
+
+        // Convert to int vector and sort in descending order (WebGL2 spec requirement)
+        for (GLint sample : samples)
+        {
+          values.push_back(static_cast<int>(sample));
+        }
+        std::sort(values.begin(), values.end(), std::greater<int>());
+      }
+      else
+      {
+        // Fallback if query fails - return default sample counts
+        values = {4, 2, 1};
+      }
+    }
+    else if (req->pname == 0x862F) // GL_NUM_SAMPLE_COUNTS
+    {
+      GLint numSampleCounts = 0;
+      glGetInternalformativ(req->target, req->internalformat, GL_NUM_SAMPLE_COUNTS, 1, &numSampleCounts);
+      values.push_back(static_cast<int>(numSampleCounts));
+    }
+    // For other pnames, return empty array (will be converted to null in binding)
+
+    GetInternalformatParameterCommandBufferResponse res(req, values);
+    if (TR_UNLIKELY(CheckError(req, reqContentRenderer) != GL_NO_ERROR || options.printsCall))
+    {
+      DEBUG(DEBUG_TAG,
+            "[%d] GL::GetInternalformativ(0x%x, 0x%x, 0x%x) => [%zu values]",
+            options.isDefaultQueue(),
+            req->target,
+            req->internalformat,
+            req->pname,
+            values.size());
+    }
+    reqContentRenderer->sendCommandBufferResponse(res);
+  }
 };
 
 void RHI_OpenGL::ProcessDeviceEvent(UnityGfxDeviceEventType type, IUnityInterfaces *interfaces)
@@ -3026,6 +3078,9 @@ bool RHI_OpenGL::ExecuteCommandBuffer(vector<commandbuffers::TrCommandBufferBase
                                  GetShaderPrecisionFormatCommandBufferRequest,
                                  GetShaderPrecisionFormat)
       ADD_COMMAND_BUFFER_HANDLER(GET_ERROR, GetErrorCommandBufferRequest, GetError)
+      ADD_COMMAND_BUFFER_HANDLER(GET_INTERNALFORMAT_PARAMETER,
+                                 GetInternalformatParameterCommandBufferRequest,
+                                 GetInternalformatParameter)
 #undef ADD_COMMAND_BUFFER_HANDLER
 #undef ADD_COMMAND_BUFFER_HANDLER_WITH_DEVICE_FRAME
 

From 142767451d4bfd93803aeea6a22fdcc11343d216 Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Wed, 19 Nov 2025 14:15:55 +0800
Subject: [PATCH 09/17] fix format

---
 src/client/script_bindings/webgl/active_info.cpp      | 2 +-
 src/client/script_bindings/webgl/active_info.hpp      | 2 +-
 src/client/script_bindings/webgl/framebuffer.cpp      | 2 +-
 src/client/script_bindings/webgl/framebuffer.hpp      | 2 +-
 src/client/script_bindings/webgl/object.cpp           | 2 +-
 src/client/script_bindings/webgl/object.hpp           | 2 +-
 src/client/script_bindings/webgl/program.cpp          | 2 +-
 src/client/script_bindings/webgl/program.hpp          | 2 +-
 src/client/script_bindings/webgl/renderbuffer.cpp     | 2 +-
 src/client/script_bindings/webgl/renderbuffer.hpp     | 2 +-
 src/client/script_bindings/webgl/texture.cpp          | 2 +-
 src/client/script_bindings/webgl/texture.hpp          | 2 +-
 src/client/script_bindings/webgl/uniform_location.cpp | 2 +-
 src/client/script_bindings/webgl/uniform_location.hpp | 2 +-
 14 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/client/script_bindings/webgl/active_info.cpp b/src/client/script_bindings/webgl/active_info.cpp
index 803ffe763..c0f9d3e81 100644
--- a/src/client/script_bindings/webgl/active_info.cpp
+++ b/src/client/script_bindings/webgl/active_info.cpp
@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/active_info.hpp b/src/client/script_bindings/webgl/active_info.hpp
index c2d6c9080..b125c893c 100644
--- a/src/client/script_bindings/webgl/active_info.hpp
+++ b/src/client/script_bindings/webgl/active_info.hpp
@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.cpp b/src/client/script_bindings/webgl/framebuffer.cpp
index d38e0687c..dbe790869 100644
--- a/src/client/script_bindings/webgl/framebuffer.cpp
+++ b/src/client/script_bindings/webgl/framebuffer.cpp
@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.hpp b/src/client/script_bindings/webgl/framebuffer.hpp
index fd3ab4ec2..c54189e65 100644
--- a/src/client/script_bindings/webgl/framebuffer.hpp
+++ b/src/client/script_bindings/webgl/framebuffer.hpp
@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.cpp b/src/client/script_bindings/webgl/object.cpp
index 78779213d..b03f846b5 100644
--- a/src/client/script_bindings/webgl/object.cpp
+++ b/src/client/script_bindings/webgl/object.cpp
@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.hpp b/src/client/script_bindings/webgl/object.hpp
index f69bb78c8..cd48b93ae 100644
--- a/src/client/script_bindings/webgl/object.hpp
+++ b/src/client/script_bindings/webgl/object.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.cpp b/src/client/script_bindings/webgl/program.cpp
index 8c21d7e26..e4a68a6f8 100644
--- a/src/client/script_bindings/webgl/program.cpp
+++ b/src/client/script_bindings/webgl/program.cpp
@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.hpp b/src/client/script_bindings/webgl/program.hpp
index 0971e7271..ede29c134 100644
--- a/src/client/script_bindings/webgl/program.hpp
+++ b/src/client/script_bindings/webgl/program.hpp
@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.cpp b/src/client/script_bindings/webgl/renderbuffer.cpp
index d858602e8..7f512b63d 100644
--- a/src/client/script_bindings/webgl/renderbuffer.cpp
+++ b/src/client/script_bindings/webgl/renderbuffer.cpp
@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.hpp b/src/client/script_bindings/webgl/renderbuffer.hpp
index 581ff1b2e..a9651c800 100644
--- a/src/client/script_bindings/webgl/renderbuffer.hpp
+++ b/src/client/script_bindings/webgl/renderbuffer.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/texture.cpp b/src/client/script_bindings/webgl/texture.cpp
index 3d034839c..3b51f5dfa 100644
--- a/src/client/script_bindings/webgl/texture.cpp
+++ b/src/client/script_bindings/webgl/texture.cpp
@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/texture.hpp b/src/client/script_bindings/webgl/texture.hpp
index 9272d305c..b40acad08 100644
--- a/src/client/script_bindings/webgl/texture.hpp
+++ b/src/client/script_bindings/webgl/texture.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/uniform_location.cpp b/src/client/script_bindings/webgl/uniform_location.cpp
index ce71bd5d6..597a73e97 100644
--- a/src/client/script_bindings/webgl/uniform_location.cpp
+++ b/src/client/script_bindings/webgl/uniform_location.cpp
@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/uniform_location.hpp b/src/client/script_bindings/webgl/uniform_location.hpp
index a1f98b66d..a5e7de1f2 100644
--- a/src/client/script_bindings/webgl/uniform_location.hpp
+++ b/src/client/script_bindings/webgl/uniform_location.hpp
@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file

From e75824fb2b9aa7d9da4f3f181a99ab9e3bbba7eb Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Wed, 19 Nov 2025 14:16:10 +0800
Subject: [PATCH 10/17] fix

---
 src/client/script_bindings/webgl/vertex_array.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/client/script_bindings/webgl/vertex_array.cpp b/src/client/script_bindings/webgl/vertex_array.cpp
index 48e8e644b..3e6ed5f0a 100644
--- a/src/client/script_bindings/webgl/vertex_array.cpp
+++ b/src/client/script_bindings/webgl/vertex_array.cpp
@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor

From f62a4fa15f0780d99dbf42b95a12cbaf0d517714 Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Wed, 19 Nov 2025 14:47:59 +0800
Subject: [PATCH 11/17] fix

---
 src/client/script_bindings/webgl/vertex_array.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/client/script_bindings/webgl/vertex_array.hpp b/src/client/script_bindings/webgl/vertex_array.hpp
index a4a03cc1d..7ba162aa3 100644
--- a/src/client/script_bindings/webgl/vertex_array.hpp
+++ b/src/client/script_bindings/webgl/vertex_array.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file

From 677cb4de5d86b22114e9a0dbb3654a8ae483ea6a Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Wed, 19 Nov 2025 17:32:27 +0800
Subject: [PATCH 12/17] Enhance GetInternalformatParameter handling and add
 support for GL_NUM_SAMPLE_COUNTS

---
 src/client/graphics/webgl_context.cpp         |  7 ++-
 .../command_buffers/details/properties.hpp    | 30 ++++++++-
 src/common/command_buffers/macros.hpp         |  4 +-
 .../command_buffers/webgl_constants.hpp       |  2 +
 src/renderer/render_api_opengles.cpp          | 62 ++++++++++++++-----
 5 files changed, 85 insertions(+), 20 deletions(-)

diff --git a/src/client/graphics/webgl_context.cpp b/src/client/graphics/webgl_context.cpp
index e210b71f3..ac224b851 100644
--- a/src/client/graphics/webgl_context.cpp
+++ b/src/client/graphics/webgl_context.cpp
@@ -1975,7 +1975,12 @@ namespace endor
       sendCommandBufferRequest(req, true);
       auto response = recvResponse<GetInternalformatParameterCommandBufferResponse>(
         COMMAND_BUFFER_GET_INTERNALFORMAT_PARAMETER_RES,
-        req.id);
+        req);
+
+      if (response == nullptr) [[unlikely]]
+      {
+        throw runtime_error("Failed to get internal format parameter: timeout.");
+      }
       return response->values;
     }
 
diff --git a/src/common/command_buffers/details/properties.hpp b/src/common/command_buffers/details/properties.hpp
index 4522c8a15..c194d8b6d 100644
--- a/src/common/command_buffers/details/properties.hpp
+++ b/src/common/command_buffers/details/properties.hpp
@@ -250,8 +250,36 @@ namespace commandbuffers
     GetInternalformatParameterCommandBufferResponse(const GetInternalformatParameterCommandBufferResponse &that,
                                                     bool clone = false)
         : TrCommandBufferSimpleResponse(that, clone)
-        , values(that.values)
     {
+      if (clone)
+        values = that.values;
+    }
+
+  public:
+    TrCommandBufferMessage *serialize() override
+    {
+      auto message = new TrCommandBufferMessage(type, size, this);
+      if (values.size() > 0)
+      {
+        // Serialize the vector using addVecSegment
+        message->addVecSegment(values);
+      }
+      return message;
+    }
+
+    void deserialize(TrCommandBufferMessage &message) override
+    {
+      auto count = message.getSegmentCount();
+      if (count > 0)
+      {
+        auto segment = message.getSegment(0);
+        if (segment != nullptr && segment->size > 0)
+        {
+          size_t numValues = segment->size / sizeof(int);
+          values.resize(numValues);
+          std::memcpy(values.data(), segment->data, segment->size);
+        }
+      }
     }
 
   public:
diff --git a/src/common/command_buffers/macros.hpp b/src/common/command_buffers/macros.hpp
index 15bad43b4..07ed8094d 100644
--- a/src/common/command_buffers/macros.hpp
+++ b/src/common/command_buffers/macros.hpp
@@ -148,6 +148,7 @@
   XX(SAMPLER_PARAMETERF, SamplerParameterfCommandBufferRequest, "GL::SamplerParameterf")                                         \
   XX(GET_SAMPLER_PARAMETER, GetSamplerParameterCommandBufferRequest, "GL::GetSamplerParameter")                                  \
   XX(IS_SAMPLER, IsSamplerCommandBufferRequest, "GL::IsSampler")                                                                 \
+  XX(GET_INTERNALFORMAT_PARAMETER, GetInternalformatParameterCommandBufferRequest, "GL::GetInternalformatParameter")             \
   XX(XRFRAME_START, XRFrameStartCommandBufferRequest, "XR::FrameStart")                                                          \
   XX(XRFRAME_FLUSH, XRFrameFlushCommandBufferRequest, "XR::FrameFlush")                                                          \
   XX(XRFRAME_END, XRFrameEndCommandBufferRequest, "XR::FrameEnd")                                                                \
@@ -171,4 +172,5 @@
   XX(GET_FLOATV, GetFloatvCommandBufferResponse, "GL::GetFloatv")                                                \
   XX(GET_STRING, GetStringCommandBufferResponse, "GL::GetString")                                                \
   XX(GET_SHADER_PRECISION_FORMAT, GetShaderPrecisionFormatCommandBufferResponse, "GL::GetShaderPrecisionFormat") \
-  XX(GET_ERROR, GetErrorCommandBufferResponse, "GL::GetError")
+  XX(GET_ERROR, GetErrorCommandBufferResponse, "GL::GetError")                                                   \
+  XX(GET_INTERNALFORMAT_PARAMETER, GetInternalformatParameterCommandBufferResponse, "GL::GetInternalformatParameter")
diff --git a/src/common/command_buffers/webgl_constants.hpp b/src/common/command_buffers/webgl_constants.hpp
index a2b30d1b5..6b753065f 100644
--- a/src/common/command_buffers/webgl_constants.hpp
+++ b/src/common/command_buffers/webgl_constants.hpp
@@ -342,6 +342,8 @@ const int WEBGL_POLYGON_OFFSET_FACTOR = 0x8038;
 const int WEBGL_TEXTURE_BINDING_2D = 0x8069;
 const int WEBGL_SAMPLE_BUFFERS = 0x80A8;
 const int WEBGL_SAMPLES = 0x80A9;
+const int WEBGL_NUM_SAMPLE_COUNTS = 0x9380;
+const int WEBGL_SAMPLES_LEGACY = 0x862F;
 const int WEBGL_SAMPLE_COVERAGE_VALUE = 0x80AA;
 const int WEBGL_SAMPLE_COVERAGE_INVERT = 0x80AB;
 const int WEBGL_COMPRESSED_TEXTURE_FORMATS = 0x86A3;
diff --git a/src/renderer/render_api_opengles.cpp b/src/renderer/render_api_opengles.cpp
index 66f012a4c..9f895aa02 100644
--- a/src/renderer/render_api_opengles.cpp
+++ b/src/renderer/render_api_opengles.cpp
@@ -2695,38 +2695,66 @@ class RHI_OpenGL : public TrRenderHardwareInterface
   {
     std::vector<int> values;
 
-    // WebGL2 spec requires support for GL_SAMPLES and GL_NUM_SAMPLE_COUNTS
+    // support WEBGL_SAMPLES for now, throw for others
     if (req->pname == WEBGL_SAMPLES)
     {
-      // Query the actual supported sample counts from GL
+#ifdef ANDROID
       GLint numSampleCounts = 0;
       glGetInternalformativ(req->target, req->internalformat, GL_NUM_SAMPLE_COUNTS, 1, &numSampleCounts);
-
-      if (numSampleCounts > 0)
+      GLenum error = glGetError();
+      if (error == GL_NO_ERROR && numSampleCounts > 0)
       {
         std::vector<GLint> samples(numSampleCounts);
         glGetInternalformativ(req->target, req->internalformat, GL_SAMPLES, numSampleCounts, samples.data());
-
-        // Convert to int vector and sort in descending order (WebGL2 spec requirement)
-        for (GLint sample : samples)
+        error = glGetError();
+        if (error == GL_NO_ERROR)
         {
-          values.push_back(static_cast<int>(sample));
+          for (GLint sample : samples)
+          {
+            if (sample > 0)
+              values.push_back(static_cast<int>(sample));
+          }
+          std::sort(values.begin(), values.end(), std::greater<int>());
         }
-        std::sort(values.begin(), values.end(), std::greater<int>());
       }
-      else
+      if (values.empty())
       {
-        // Fallback if query fails - return default sample counts
-        values = {4, 2, 1};
+        values = {4, 2};
       }
-    }
-    else if (req->pname == 0x862F) // GL_NUM_SAMPLE_COUNTS
-    {
+#else
       GLint numSampleCounts = 0;
       glGetInternalformativ(req->target, req->internalformat, GL_NUM_SAMPLE_COUNTS, 1, &numSampleCounts);
-      values.push_back(static_cast<int>(numSampleCounts));
+      GLenum error = glGetError();
+      if (error == GL_NO_ERROR && numSampleCounts > 0)
+      {
+        std::vector<GLint> samples(numSampleCounts);
+        glGetInternalformativ(req->target, req->internalformat, GL_SAMPLES, numSampleCounts, samples.data());
+        error = glGetError();
+        DEBUG(DEBUG_TAG, "[GetInternalformatParameter] GL_SAMPLES query: error=0x%x", error);
+        if (error == GL_NO_ERROR)
+        {
+          for (GLint sample : samples)
+          {
+            if (sample > 0)
+              values.push_back(static_cast<int>(sample));
+          }
+          std::sort(values.begin(), values.end(), std::greater<int>());
+        }
+      }
+      if (values.empty())
+      {
+        DEBUG(DEBUG_TAG, "[GetInternalformatParameter] Using fallback values {4, 2}");
+        values = {4, 2};
+      }
+#endif
+    }
+    else
+    {
+      // Not implemented for other pnames
+      DEBUG(LOG_TAG_ERROR,
+            "[GetInternalformatParameter] pname 0x%x is not supported.",
+            req->pname);
     }
-    // For other pnames, return empty array (will be converted to null in binding)
 
     GetInternalformatParameterCommandBufferResponse res(req, values);
     if (TR_UNLIKELY(CheckError(req, reqContentRenderer) != GL_NO_ERROR || options.printsCall))

From caee1bde1caf0529a35ccb4944b5cdb51db2ca8c Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Wed, 19 Nov 2025 17:37:18 +0800
Subject: [PATCH 13/17] fix

---
 src/common/command_buffers/webgl_constants.hpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/common/command_buffers/webgl_constants.hpp b/src/common/command_buffers/webgl_constants.hpp
index 6b753065f..a2b30d1b5 100644
--- a/src/common/command_buffers/webgl_constants.hpp
+++ b/src/common/command_buffers/webgl_constants.hpp
@@ -342,8 +342,6 @@ const int WEBGL_POLYGON_OFFSET_FACTOR = 0x8038;
 const int WEBGL_TEXTURE_BINDING_2D = 0x8069;
 const int WEBGL_SAMPLE_BUFFERS = 0x80A8;
 const int WEBGL_SAMPLES = 0x80A9;
-const int WEBGL_NUM_SAMPLE_COUNTS = 0x9380;
-const int WEBGL_SAMPLES_LEGACY = 0x862F;
 const int WEBGL_SAMPLE_COVERAGE_VALUE = 0x80AA;
 const int WEBGL_SAMPLE_COVERAGE_INVERT = 0x80AB;
 const int WEBGL_COMPRESSED_TEXTURE_FORMATS = 0x86A3;

From 6f3e1b52acf001a0589607f106553a984af0c565 Mon Sep 17 00:00:00 2001
From: EndlessJourney <155411404+EndlessJour9527@users.noreply.github.com>
Date: Wed, 19 Nov 2025 17:52:54 +0800
Subject: [PATCH 14/17] Update
 src/common/command_buffers/details/properties.hpp

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---
 .../command_buffers/details/properties.hpp       | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/common/command_buffers/details/properties.hpp b/src/common/command_buffers/details/properties.hpp
index c194d8b6d..b2341092b 100644
--- a/src/common/command_buffers/details/properties.hpp
+++ b/src/common/command_buffers/details/properties.hpp
@@ -269,17 +269,11 @@ namespace commandbuffers
 
     void deserialize(TrCommandBufferMessage &message) override
     {
-      auto count = message.getSegmentCount();
-      if (count > 0)
-      {
-        auto segment = message.getSegment(0);
-        if (segment != nullptr && segment->size > 0)
-        {
-          size_t numValues = segment->size / sizeof(int);
-          values.resize(numValues);
-          std::memcpy(values.data(), segment->data, segment->size);
-        }
-      }
+      auto valuesSegment = message.nextSegment();
+      if (valuesSegment != nullptr)
+        values = valuesSegment->toVec<int>();
+      else
+        values.clear();
     }
 
   public:

From e0927ae2ba07e056ddb4de08ca4b956e7fa32337 Mon Sep 17 00:00:00 2001
From: EndlessJourney <155411404+EndlessJour9527@users.noreply.github.com>
Date: Wed, 19 Nov 2025 17:53:17 +0800
Subject: [PATCH 15/17] Update src/renderer/render_api_opengles.cpp

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---
 src/renderer/render_api_opengles.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/renderer/render_api_opengles.cpp b/src/renderer/render_api_opengles.cpp
index 9f895aa02..9056b0f96 100644
--- a/src/renderer/render_api_opengles.cpp
+++ b/src/renderer/render_api_opengles.cpp
@@ -2695,7 +2695,7 @@ class RHI_OpenGL : public TrRenderHardwareInterface
   {
     std::vector<int> values;
 
-    // support WEBGL_SAMPLES for now, throw for others
+    // Support WEBGL_SAMPLES for now, log error for others
     if (req->pname == WEBGL_SAMPLES)
     {
 #ifdef ANDROID

From db8fc0d0775ecebb159a5e503be25f0242bae342 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Wed, 19 Nov 2025 09:58:05 +0000
Subject: [PATCH 16/17] Address code review feedback: refactor and fix fallback
 values

- Eliminate code duplication between Android and non-Android branches
- Fix fallback values from {4, 2} to {4, 2, 1} per WebGL2 spec
- Add comment about CheckError placement and error handling
- Conditional DEBUG logging using #ifndef ANDROID preprocessor directives
- Maintain single code path with platform-specific debug output

Addresses comments #2541272688, #2541272708, #2541272719, #2541272761

Co-authored-by: EndlessJour9527 <155411404+EndlessJour9527@users.noreply.github.com>
---
 .../script_bindings/webgl/active_info.cpp     |  2 +-
 .../script_bindings/webgl/active_info.hpp     |  2 +-
 .../script_bindings/webgl/framebuffer.cpp     |  2 +-
 .../script_bindings/webgl/framebuffer.hpp     |  2 +-
 src/client/script_bindings/webgl/object.cpp   |  2 +-
 src/client/script_bindings/webgl/object.hpp   |  2 +-
 src/client/script_bindings/webgl/program.cpp  |  2 +-
 src/client/script_bindings/webgl/program.hpp  |  2 +-
 .../script_bindings/webgl/renderbuffer.cpp    |  2 +-
 .../script_bindings/webgl/renderbuffer.hpp    |  2 +-
 src/client/script_bindings/webgl/texture.cpp  |  2 +-
 src/client/script_bindings/webgl/texture.hpp  |  2 +-
 .../webgl/uniform_location.cpp                |  2 +-
 .../webgl/uniform_location.hpp                |  2 +-
 .../script_bindings/webgl/vertex_array.cpp    |  2 +-
 .../script_bindings/webgl/vertex_array.hpp    |  2 +-
 src/renderer/render_api_opengles.cpp          | 35 +++++--------------
 17 files changed, 24 insertions(+), 43 deletions(-)

diff --git a/src/client/script_bindings/webgl/active_info.cpp b/src/client/script_bindings/webgl/active_info.cpp
index c0f9d3e81..803ffe763 100644
--- a/src/client/script_bindings/webgl/active_info.cpp
+++ b/src/client/script_bindings/webgl/active_info.cpp
@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/active_info.hpp b/src/client/script_bindings/webgl/active_info.hpp
index b125c893c..c2d6c9080 100644
--- a/src/client/script_bindings/webgl/active_info.hpp
+++ b/src/client/script_bindings/webgl/active_info.hpp
@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.cpp b/src/client/script_bindings/webgl/framebuffer.cpp
index dbe790869..d38e0687c 100644
--- a/src/client/script_bindings/webgl/framebuffer.cpp
+++ b/src/client/script_bindings/webgl/framebuffer.cpp
@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.hpp b/src/client/script_bindings/webgl/framebuffer.hpp
index c54189e65..fd3ab4ec2 100644
--- a/src/client/script_bindings/webgl/framebuffer.hpp
+++ b/src/client/script_bindings/webgl/framebuffer.hpp
@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.cpp b/src/client/script_bindings/webgl/object.cpp
index b03f846b5..78779213d 100644
--- a/src/client/script_bindings/webgl/object.cpp
+++ b/src/client/script_bindings/webgl/object.cpp
@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.hpp b/src/client/script_bindings/webgl/object.hpp
index cd48b93ae..f69bb78c8 100644
--- a/src/client/script_bindings/webgl/object.hpp
+++ b/src/client/script_bindings/webgl/object.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.cpp b/src/client/script_bindings/webgl/program.cpp
index e4a68a6f8..8c21d7e26 100644
--- a/src/client/script_bindings/webgl/program.cpp
+++ b/src/client/script_bindings/webgl/program.cpp
@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.hpp b/src/client/script_bindings/webgl/program.hpp
index ede29c134..0971e7271 100644
--- a/src/client/script_bindings/webgl/program.hpp
+++ b/src/client/script_bindings/webgl/program.hpp
@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.cpp b/src/client/script_bindings/webgl/renderbuffer.cpp
index 7f512b63d..d858602e8 100644
--- a/src/client/script_bindings/webgl/renderbuffer.cpp
+++ b/src/client/script_bindings/webgl/renderbuffer.cpp
@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.hpp b/src/client/script_bindings/webgl/renderbuffer.hpp
index a9651c800..581ff1b2e 100644
--- a/src/client/script_bindings/webgl/renderbuffer.hpp
+++ b/src/client/script_bindings/webgl/renderbuffer.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/texture.cpp b/src/client/script_bindings/webgl/texture.cpp
index 3b51f5dfa..3d034839c 100644
--- a/src/client/script_bindings/webgl/texture.cpp
+++ b/src/client/script_bindings/webgl/texture.cpp
@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/texture.hpp b/src/client/script_bindings/webgl/texture.hpp
index b40acad08..9272d305c 100644
--- a/src/client/script_bindings/webgl/texture.hpp
+++ b/src/client/script_bindings/webgl/texture.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/uniform_location.cpp b/src/client/script_bindings/webgl/uniform_location.cpp
index 597a73e97..ce71bd5d6 100644
--- a/src/client/script_bindings/webgl/uniform_location.cpp
+++ b/src/client/script_bindings/webgl/uniform_location.cpp
@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/uniform_location.hpp b/src/client/script_bindings/webgl/uniform_location.hpp
index a5e7de1f2..a1f98b66d 100644
--- a/src/client/script_bindings/webgl/uniform_location.hpp
+++ b/src/client/script_bindings/webgl/uniform_location.hpp
@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/vertex_array.cpp b/src/client/script_bindings/webgl/vertex_array.cpp
index 3e6ed5f0a..48e8e644b 100644
--- a/src/client/script_bindings/webgl/vertex_array.cpp
+++ b/src/client/script_bindings/webgl/vertex_array.cpp
@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/vertex_array.hpp b/src/client/script_bindings/webgl/vertex_array.hpp
index 7ba162aa3..a4a03cc1d 100644
--- a/src/client/script_bindings/webgl/vertex_array.hpp
+++ b/src/client/script_bindings/webgl/vertex_array.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  }   // namespace script_bindings
+  } // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/renderer/render_api_opengles.cpp b/src/renderer/render_api_opengles.cpp
index 9056b0f96..420e55f51 100644
--- a/src/renderer/render_api_opengles.cpp
+++ b/src/renderer/render_api_opengles.cpp
@@ -2698,30 +2698,6 @@ class RHI_OpenGL : public TrRenderHardwareInterface
     // Support WEBGL_SAMPLES for now, log error for others
     if (req->pname == WEBGL_SAMPLES)
     {
-#ifdef ANDROID
-      GLint numSampleCounts = 0;
-      glGetInternalformativ(req->target, req->internalformat, GL_NUM_SAMPLE_COUNTS, 1, &numSampleCounts);
-      GLenum error = glGetError();
-      if (error == GL_NO_ERROR && numSampleCounts > 0)
-      {
-        std::vector<GLint> samples(numSampleCounts);
-        glGetInternalformativ(req->target, req->internalformat, GL_SAMPLES, numSampleCounts, samples.data());
-        error = glGetError();
-        if (error == GL_NO_ERROR)
-        {
-          for (GLint sample : samples)
-          {
-            if (sample > 0)
-              values.push_back(static_cast<int>(sample));
-          }
-          std::sort(values.begin(), values.end(), std::greater<int>());
-        }
-      }
-      if (values.empty())
-      {
-        values = {4, 2};
-      }
-#else
       GLint numSampleCounts = 0;
       glGetInternalformativ(req->target, req->internalformat, GL_NUM_SAMPLE_COUNTS, 1, &numSampleCounts);
       GLenum error = glGetError();
@@ -2730,7 +2706,9 @@ class RHI_OpenGL : public TrRenderHardwareInterface
         std::vector<GLint> samples(numSampleCounts);
         glGetInternalformativ(req->target, req->internalformat, GL_SAMPLES, numSampleCounts, samples.data());
         error = glGetError();
+#ifndef ANDROID
         DEBUG(DEBUG_TAG, "[GetInternalformatParameter] GL_SAMPLES query: error=0x%x", error);
+#endif
         if (error == GL_NO_ERROR)
         {
           for (GLint sample : samples)
@@ -2743,10 +2721,11 @@ class RHI_OpenGL : public TrRenderHardwareInterface
       }
       if (values.empty())
       {
-        DEBUG(DEBUG_TAG, "[GetInternalformatParameter] Using fallback values {4, 2}");
-        values = {4, 2};
-      }
+#ifndef ANDROID
+        DEBUG(DEBUG_TAG, "[GetInternalformatParameter] Using fallback values {4, 2, 1}");
 #endif
+        values = {4, 2, 1};
+      }
     }
     else
     {
@@ -2757,6 +2736,8 @@ class RHI_OpenGL : public TrRenderHardwareInterface
     }
 
     GetInternalformatParameterCommandBufferResponse res(req, values);
+    // Note: CheckError is called here but won't detect errors from glGetInternalformativ
+    // calls above since those errors are already consumed by explicit glGetError() calls
     if (TR_UNLIKELY(CheckError(req, reqContentRenderer) != GL_NO_ERROR || options.printsCall))
     {
       DEBUG(DEBUG_TAG,

From 9e6f5174b2292635ca3bf8dc6e878ab2e48b50f2 Mon Sep 17 00:00:00 2001
From: yjh <yinjh9527@gmail.com>
Date: Wed, 19 Nov 2025 19:22:55 +0800
Subject: [PATCH 17/17] Refactor namespace closing comments for consistency in
 WebGL bindings

---
 src/client/script_bindings/webgl/active_info.cpp      | 2 +-
 src/client/script_bindings/webgl/active_info.hpp      | 2 +-
 src/client/script_bindings/webgl/framebuffer.cpp      | 2 +-
 src/client/script_bindings/webgl/framebuffer.hpp      | 2 +-
 src/client/script_bindings/webgl/object.cpp           | 2 +-
 src/client/script_bindings/webgl/object.hpp           | 2 +-
 src/client/script_bindings/webgl/program.cpp          | 2 +-
 src/client/script_bindings/webgl/program.hpp          | 2 +-
 src/client/script_bindings/webgl/renderbuffer.cpp     | 2 +-
 src/client/script_bindings/webgl/renderbuffer.hpp     | 2 +-
 src/client/script_bindings/webgl/texture.cpp          | 2 +-
 src/client/script_bindings/webgl/texture.hpp          | 2 +-
 src/client/script_bindings/webgl/uniform_location.cpp | 2 +-
 src/client/script_bindings/webgl/uniform_location.hpp | 2 +-
 src/client/script_bindings/webgl/vertex_array.cpp     | 2 +-
 src/client/script_bindings/webgl/vertex_array.hpp     | 2 +-
 src/renderer/render_api_opengles.cpp                  | 6 +-----
 17 files changed, 17 insertions(+), 21 deletions(-)

diff --git a/src/client/script_bindings/webgl/active_info.cpp b/src/client/script_bindings/webgl/active_info.cpp
index 803ffe763..c0f9d3e81 100644
--- a/src/client/script_bindings/webgl/active_info.cpp
+++ b/src/client/script_bindings/webgl/active_info.cpp
@@ -60,5 +60,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/active_info.hpp b/src/client/script_bindings/webgl/active_info.hpp
index c2d6c9080..b125c893c 100644
--- a/src/client/script_bindings/webgl/active_info.hpp
+++ b/src/client/script_bindings/webgl/active_info.hpp
@@ -38,5 +38,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.cpp b/src/client/script_bindings/webgl/framebuffer.cpp
index d38e0687c..dbe790869 100644
--- a/src/client/script_bindings/webgl/framebuffer.cpp
+++ b/src/client/script_bindings/webgl/framebuffer.cpp
@@ -31,5 +31,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/framebuffer.hpp b/src/client/script_bindings/webgl/framebuffer.hpp
index fd3ab4ec2..c54189e65 100644
--- a/src/client/script_bindings/webgl/framebuffer.hpp
+++ b/src/client/script_bindings/webgl/framebuffer.hpp
@@ -37,5 +37,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.cpp b/src/client/script_bindings/webgl/object.cpp
index 78779213d..b03f846b5 100644
--- a/src/client/script_bindings/webgl/object.cpp
+++ b/src/client/script_bindings/webgl/object.cpp
@@ -28,5 +28,5 @@ namespace endor
         // WebGLObject is a base class, typically not instantiated directly
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/object.hpp b/src/client/script_bindings/webgl/object.hpp
index f69bb78c8..cd48b93ae 100644
--- a/src/client/script_bindings/webgl/object.hpp
+++ b/src/client/script_bindings/webgl/object.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.cpp b/src/client/script_bindings/webgl/program.cpp
index 8c21d7e26..e4a68a6f8 100644
--- a/src/client/script_bindings/webgl/program.cpp
+++ b/src/client/script_bindings/webgl/program.cpp
@@ -20,5 +20,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/program.hpp b/src/client/script_bindings/webgl/program.hpp
index 0971e7271..ede29c134 100644
--- a/src/client/script_bindings/webgl/program.hpp
+++ b/src/client/script_bindings/webgl/program.hpp
@@ -32,5 +32,5 @@ namespace endor
         WebGLProgram(v8::Isolate *isolate, const v8::FunctionCallbackInfo<v8::Value> &args);
       };
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.cpp b/src/client/script_bindings/webgl/renderbuffer.cpp
index d858602e8..7f512b63d 100644
--- a/src/client/script_bindings/webgl/renderbuffer.cpp
+++ b/src/client/script_bindings/webgl/renderbuffer.cpp
@@ -19,5 +19,5 @@ namespace endor
         // WebGLRenderbuffer objects are created by WebGL context, not by user code
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/renderbuffer.hpp b/src/client/script_bindings/webgl/renderbuffer.hpp
index 581ff1b2e..a9651c800 100644
--- a/src/client/script_bindings/webgl/renderbuffer.hpp
+++ b/src/client/script_bindings/webgl/renderbuffer.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/texture.cpp b/src/client/script_bindings/webgl/texture.cpp
index 3d034839c..3b51f5dfa 100644
--- a/src/client/script_bindings/webgl/texture.cpp
+++ b/src/client/script_bindings/webgl/texture.cpp
@@ -27,5 +27,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/texture.hpp b/src/client/script_bindings/webgl/texture.hpp
index 9272d305c..b40acad08 100644
--- a/src/client/script_bindings/webgl/texture.hpp
+++ b/src/client/script_bindings/webgl/texture.hpp
@@ -45,5 +45,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/uniform_location.cpp b/src/client/script_bindings/webgl/uniform_location.cpp
index ce71bd5d6..597a73e97 100644
--- a/src/client/script_bindings/webgl/uniform_location.cpp
+++ b/src/client/script_bindings/webgl/uniform_location.cpp
@@ -47,5 +47,5 @@ namespace endor
       }
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/uniform_location.hpp b/src/client/script_bindings/webgl/uniform_location.hpp
index a1f98b66d..a5e7de1f2 100644
--- a/src/client/script_bindings/webgl/uniform_location.hpp
+++ b/src/client/script_bindings/webgl/uniform_location.hpp
@@ -44,5 +44,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/client/script_bindings/webgl/vertex_array.cpp b/src/client/script_bindings/webgl/vertex_array.cpp
index 48e8e644b..3e6ed5f0a 100644
--- a/src/client/script_bindings/webgl/vertex_array.cpp
+++ b/src/client/script_bindings/webgl/vertex_array.cpp
@@ -20,5 +20,5 @@ namespace endor
       {
       }
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
diff --git a/src/client/script_bindings/webgl/vertex_array.hpp b/src/client/script_bindings/webgl/vertex_array.hpp
index a4a03cc1d..7ba162aa3 100644
--- a/src/client/script_bindings/webgl/vertex_array.hpp
+++ b/src/client/script_bindings/webgl/vertex_array.hpp
@@ -33,5 +33,5 @@ namespace endor
       };
 
     } // namespace webgl
-  } // namespace script_bindings
+  }   // namespace script_bindings
 } // namespace endor
\ No newline at end of file
diff --git a/src/renderer/render_api_opengles.cpp b/src/renderer/render_api_opengles.cpp
index 420e55f51..ffac3d4e3 100644
--- a/src/renderer/render_api_opengles.cpp
+++ b/src/renderer/render_api_opengles.cpp
@@ -2706,9 +2706,7 @@ class RHI_OpenGL : public TrRenderHardwareInterface
         std::vector<GLint> samples(numSampleCounts);
         glGetInternalformativ(req->target, req->internalformat, GL_SAMPLES, numSampleCounts, samples.data());
         error = glGetError();
-#ifndef ANDROID
-        DEBUG(DEBUG_TAG, "[GetInternalformatParameter] GL_SAMPLES query: error=0x%x", error);
-#endif
+
         if (error == GL_NO_ERROR)
         {
           for (GLint sample : samples)
@@ -2721,9 +2719,7 @@ class RHI_OpenGL : public TrRenderHardwareInterface
       }
       if (values.empty())
       {
-#ifndef ANDROID
         DEBUG(DEBUG_TAG, "[GetInternalformatParameter] Using fallback values {4, 2, 1}");
-#endif
         values = {4, 2, 1};
       }
     }