Define default usermods

[netmindz]

This pull request updates the platformio.ini configuration to centralize and standardize the definition of custom user modules across multiple build environments. The main change is the introduction of a default_usermods variable, which is then referenced throughout the configuration to ensure consistency and simplify future maintenance.

Centralization of usermod configuration:

• Added a new default_usermods variable under [common], listing audioreactive, multi_relay, and Temperature as default user modules.
• Updated all build environments to use custom_usermods = ${common.default_usermods} instead of specifying user modules individually, replacing previous direct assignments like custom_usermods = audioreactive. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14]

This change improves maintainability by ensuring that updates to the default user modules only need to be made in a single location.

Summary by CodeRabbit

• Chores
  • Standardized a shared default usermod list (now including audioreactive and Temperature) and updated multiple build environments to reference it, ensuring consistent inclusion of these core modules across builds.

[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

Run ID: 3ae6e984-acc7-4160-a173-e52b65ddd039

📥 Commits

Reviewing files that changed from the base of the PR and between 6acea21 and 1289777.

📒 Files selected for processing (1)

• platformio.ini

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

• platformio.ini

---

Walkthrough

Extracts a shared default_usermods variable into [common] in platformio.ini (set to audioreactive, Temperature) and updates multiple [env:*] sections to set custom_usermods = ${common.default_usermods} instead of hardcoding audioreactive.

Changes

PlatformIO Configuration

Layer / File(s)	Summary
Shared variable platformio.ini	Add [common] default_usermods = audioreactive, Temperature.
Environment wiring platformio.ini (env:nodemcuv2, env:nodemcuv2_160, env:esp8266_2m_160, env:esp32dev, env:esp32dev_8M, env:esp32dev_16M, env:esp32_eth, env:esp32_wrover, env:esp32c3dev, env:esp32s3dev_16MB_opi, env:esp32s3dev_8MB_opi, env:esp32S3_wroom2, env:esp32s3_4M_qspi, env:lolin_s2_mini)	Replace hardcoded custom_usermods = audioreactive with custom_usermods = ${common.default_usermods} across listed environments.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Usermod libs matrix #4592 — Changes PlatformIO usermod configuration variables and environment usermod wiring.

Suggested reviewers

• willmmiles

🚥 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 accurately and concisely describes the main change: introducing a shared default_usermods variable in platformio.ini for standardized configuration.
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.}

---

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

_{Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.}

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

[coderabbitai]

Actionable comments posted: 3

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@platformio.ini`:
- Line 427: The environment [env:esp01_1m_full_160] still overrides
custom_usermods with "audioreactive" instead of using the shared variable;
update its custom_usermods assignment to use ${common.default_usermods}
(matching the other envs) so changes to the central variable propagate, and
verify that the common.default_usermods definition exists and contains
"audioreactive" where intended.
- Around line 133-136: The global default_usermods list currently includes
audioreactive which is unsupported on ESP8266 and is pulled into ESP8266
environments; remove audioreactive from the global default_usermods, create a
new variable (e.g., common.default_usermods_esp32) that includes audioreactive
for ESP32/S2/S3/C3 envs, and update the ESP32-family environments to use
${common.default_usermods_esp32} while ensuring ESP8266 envs keep the original
default_usermods without audioreactive so CI/release builds for ESP8266 do not
include the unsupported module.
- Around line 133-136: The change adds a global default_usermods list
(default_usermods containing audioreactive, multi_relay, Temperature) in
platformio.ini which violates WLED guidelines and also makes audioreactive
active for ESP8266 where it is unsupported; revert the change in platformio.ini
and instead add these usermods to platformio_override.ini (or make them
conditional) so they only apply to intended environments, and if audioreactive
must be present ensure it is excluded from ESP8266 builds (e.g., move it behind
an ESP32-only override or conditional entry) and document any maintainer
approval for modifying global config on the PR.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: b75d98a4-d781-48ca-a27c-d3bd913cc3fe

📥 Commits

Reviewing files that changed from the base of the PR and between 02e593d and 6acea21.

📒 Files selected for processing (1)

• platformio.ini

[netmindz]

The list of mods is more of a proposal for discussion rather than final list

If we did include Temperature, we need to also change the usermod to be disabled by default

Do we think the MultiRelay is popular enough to include or are the stats being distorted by the early adopters? Should we hold that back for 16.1 ?

[DedeHai]

I would not add them by default. Multi-relay is used by quinled and maybe other hardware, it eats up 8k of flash (!) probably because it pulls in a library for I2C port expander. If we make it a default, it should be slimmed down to common use cases.

[softhack007]

ToDO: not matter which usermods we finally pick - we need to check them for "pin stealing" problems.

• modify usermod defaults, to make sure they all have default pins of "-1" = undefined
• check that usermods start up as "not enabled"