fix(truepbr): remove EnvmapData breaking MATOs

[soda3000]

Summary by CodeRabbit

• Bug Fixes
  • Corrected projected material color scaling in physically-based rendering mode for improved visual accuracy.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 10f418cf-789c-4cdd-8e80-34e5f51a1a62

📥 Commits

Reviewing files that changed from the base of the PR and between 88a5358 and b19c87b.

📒 Files selected for processing (1)

• package/Shaders/Lighting.hlsl

💤 Files with no reviewable changes (1)

• package/Shaders/Lighting.hlsl

---

📝 Walkthrough

Walkthrough

A single-line adjustment to the TRUE_PBR projected-material shader path replaces the base color scaling factor from EnvmapData.x to MaterialObjectRGBScale, directly affecting how projected UV colors blend into the final material base color computation.

Changes

Projected Material Base Color Scaling

Layer / File(s)	Summary
Projected material base color scaling with RGB scale package/Shaders/Lighting.hlsl	In the TRUE_PBR projected-UV handling path, projBaseColor.xyz is now scaled by MaterialObjectRGBScale (clamped with max(0, ...)) instead of EnvmapData.x, altering how projected UV colors contribute to the material's base color before subsequent RMAOS interpolation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• community-shaders/skyrim-community-shaders#2310: Both PRs update package/Shaders/Lighting.hlsl's TRUE_PBR projected-UV base color path to scale projBaseColor.xyz using MaterialObjectRGBScale instead of EnvmapData.x.
• community-shaders/skyrim-community-shaders#2046: Fixes how projected MATO fields including projectedMaterialBaseColorScale → MaterialObjectRGBScale are applied/cleared, directly supporting the scaling adjustment made in this PR's shader computation.
• community-shaders/skyrim-community-shaders#1950: Modifies the projected-material shading logic in the same TRUE_PBR/projection base color path for projBaseColor.xyz blending.

Suggested reviewers

• doodlum
• alandtse

Poem

A scale swap in shader light,
From EnvmapData to RGB bright,
Projected colors blend just right,
One line dances, makes it white— 🐰✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically identifies the main change: removing EnvmapData usage that was breaking MATO (material object) functionality in the TruePBR shader path.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

No actionable suggestions for changed features.

[soda3000]

Some additional info about why this change is necessary:

• Previous commit d026838 (PR fix(truepbr): apply MATO rbg scalars through cbuffer #2310) made it so PBR cpp no longer was overwriting EnvmapData.x with MATO slider red value, so it has whatever value the engine sets, or somewhere else in the pipeline.
• That value totally nukes the MATO final color, as it seems to change between 0 and something non-zero over the course of a few frames.
• PR fix(truepbr): apply MATO rbg scalars through cbuffer #2310 removed the need to apply MATO red to the final color, since RGB are all now provided through their own dedicated cbuffer constant.