Add SHTC3_v2 usermod to platformio.ini

[Will-wastelander]

Add SHTC3_v2 usermod for temperature sensing.

Summary by CodeRabbit

• New Features
  • Added support for SHTC3 v2 temperature and humidity sensors to the Waveshare ESP32-S3 Hub75 build.
  • Enables temperature and humidity readings from compatible SHTC3 v2 devices.

[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: ab83e23d-0c93-4088-b231-a6e9f95f0db8

📥 Commits

Reviewing files that changed from the base of the PR and between 4812bb0 and b5e2cc9.

📒 Files selected for processing (1)

• platformio.ini

🚧 Files skipped from review as they are similar to previous changes (1)

• platformio.ini

---

Walkthrough

The Waveshare ESP32-S3 32MB HUB75 build environment adds the SHTC3_v2 usermod repository to its existing custom usermod list.

Changes

SHTC3 usermod integration

Layer / File(s)	Summary
Add SHTC3_v2 to build environment platformio.ini	The waveshare_esp32s3_32MB_hub75 environment now includes the SHTC3_v2 Git repository in custom_usermods.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: netmindz

🚥 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 concisely describes the main change: adding the SHTC3_v2 usermod to platformio.ini.
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.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

🧹 Nitpick comments (1)

platformio.ini (1)

1168-1168: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick win

Pin the SHTC3_v2 dependency to an immutable commit.

The new external usermod tracks the repository’s moving default branch, so future builds may silently consume different code. Pin it to a commit SHA, consistent with the nearby audioreactive dependency.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@platformio.ini` at line 1168, Update the SHTC3_v2 dependency declaration to
reference a specific immutable commit SHA instead of the repository’s moving
default branch, following the nearby audioreactive dependency’s pinning format.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@platformio.ini`:
- Line 1168: Update the SHTC3_v2 dependency declaration to reference a specific
immutable commit SHA instead of the repository’s moving default branch,
following the nearby audioreactive dependency’s pinning format.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 3ca38a03-d879-4669-b14d-9d652194e67a

📥 Commits

Reviewing files that changed from the base of the PR and between 8db3d7a and e39e1ab.

📒 Files selected for processing (1)

• platformio.ini

[Will-wastelander]

Updated waveshare_esp32s3_32MB_hub75 build env to include SHTC3 UM, enabling the onboard temp/hum sensor.

[Will-wastelander]

@softhack007 They created this UM at my request, in #sensors on discord. I'm not sure if they have released it to WLED-AC yet or not, but I wanted to get this added to the build env, so it would have the support. Here is the link to their post in that channel.
https://discord.com/channels/473448917040758787/708677213830250527/1527357353056534579

[softhack007]

@lost-hope I've quickly looked through the UM code.

One line is a bit problematic

https://github.com/lost-hope/SHTC3_v2/blob/b8a648e7b71ac1bb6efca1bacdf6577aeb17b09c/SHTC3_v2.cpp#L134

this delay() runs in the main loop, it can lead to effect stuttering for higher framerates, especially above 60FPS.

[DedeHai]

a delay of 15ms is really bad, it should be a state machine instead. i.e. use timestamps and states, if its not time to read yet, just give the CPU back to other tasks.

[lost-hope]

@softhack007 @DedeHai it became a state machine now ;D ( made it before i saw your comment)
so the 16ms ( there was another delay of 1ms a few lines up) are gone now.
When you two are happy with it now, i will add it to the list in the kb.

[DedeHai]

When you two are happy with it now, i will add it to the list in the kb.

looks good, go ahead :)

[lost-hope]

Why are you referencing to a specific commit?

[Will-wastelander]

Why are you referencing to a specific commit?

@lost-hope Coderabbitai nitpick.. If it's not needed, then I can change it to main. I have 0 issues w/ that. That's what I originally had. But Coderabbitat complained.

#5756 (review)

[willmmiles]

Commit pinning is a must - otherwise we have no guarantees on what's getting built, opening us up to supply chain attacks.

[Will-wastelander]

Commit pinning is a must - otherwise we have no guarantees on what's getting built, opening us up to supply chain attacks.

@willmmiles That's what I was figuring, and other that coderabbitai nitpicking it, figured would be best. I have tested this commit, and know it works.