Skip to content

[FIX] Enforce WGSL syntax highlighting in all WGSL chunks#8300

Merged
willeastcott merged 1 commit intomainfrom
glsl-wgsl
Dec 21, 2025
Merged

[FIX] Enforce WGSL syntax highlighting in all WGSL chunks#8300
willeastcott merged 1 commit intomainfrom
glsl-wgsl

Conversation

@willeastcott
Copy link
Contributor

@willeastcott willeastcott commented Dec 21, 2025

This pull request updates several shader chunk files to clarify that they use the WGSL shading language instead of GLSL. The change is made by updating the comment in the default export of each file from /* glsl */ to /* wgsl */. No functional or logic changes are introduced; this is a consistency and clarity improvement across the shader codebase.

Shader language annotation updates:

  • Updated the language annotation in the default export from /* glsl */ to /* wgsl */ in the following files to accurately reflect the use of WGSL:
    • src/scene/shader-lib/wgsl/chunks/lit/frag/lighting/lightEvaluation.js
    • src/scene/shader-lib/wgsl/chunks/lit/frag/lighting/shadowPCF3.js
    • src/scene/shader-lib/wgsl/chunks/lit/frag/metalnessModulate.js
    • src/scene/shader-lib/wgsl/chunks/lit/frag/reflectionSheen.js
    • src/scene/shader-lib/wgsl/chunks/lit/frag/spot.js
    • src/scene/shader-lib/wgsl/chunks/particle/frag/particle_blendNormal.js
    • src/scene/shader-lib/wgsl/chunks/particle/frag/particle_lighting.js

Checklist

  • I have read the contributing guidelines
  • My code follows the project's coding standards
  • This PR focuses on a single change

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request corrects the language annotation comments in WGSL shader chunk files from /* glsl */ to /* wgsl */ to accurately reflect the shader language being used. These comment annotations are typically used by code editors and IDEs for syntax highlighting and language support.

  • Updated language annotation comments from /* glsl */ to /* wgsl */ in 7 shader chunk files
  • All changes are within the src/scene/shader-lib/wgsl/chunks/ directory
  • No functional changes; this is purely a metadata/documentation improvement

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/scene/shader-lib/wgsl/chunks/particle/frag/particle_lighting.js Corrected language annotation from GLSL to WGSL
src/scene/shader-lib/wgsl/chunks/particle/frag/particle_blendNormal.js Corrected language annotation from GLSL to WGSL
src/scene/shader-lib/wgsl/chunks/lit/frag/spot.js Corrected language annotation from GLSL to WGSL
src/scene/shader-lib/wgsl/chunks/lit/frag/reflectionSheen.js Corrected language annotation from GLSL to WGSL
src/scene/shader-lib/wgsl/chunks/lit/frag/metalnessModulate.js Corrected language annotation from GLSL to WGSL
src/scene/shader-lib/wgsl/chunks/lit/frag/lighting/shadowPCF3.js Corrected language annotation from GLSL to WGSL
src/scene/shader-lib/wgsl/chunks/lit/frag/lighting/lightEvaluation.js Corrected language annotation from GLSL to WGSL

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@willeastcott willeastcott merged commit fc11e25 into main Dec 21, 2025
13 checks passed
@willeastcott willeastcott deleted the glsl-wgsl branch December 21, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants