Skip to content

[esphome] Modernise board spec, web server v3, remove legacy options#71

Open
bharvey88 wants to merge 4 commits intobetafrom
esphome-modernisation
Open

[esphome] Modernise board spec, web server v3, remove legacy options#71
bharvey88 wants to merge 4 commits intobetafrom
esphome-modernisation

Conversation

@bharvey88
Copy link
Contributor

Version: 25.7.18.1

What does this implement/fix?

A set of non-breaking modernisation changes to keep the MSR-1 firmware in line with current ESPHome best practices:

Core.yaml

  • Replace deprecated board: esp32-c3-devkitm-1 with explicit variant: esp32c3 + flash_size: 4MB — this is the current recommended way to target ESP32-C3 hardware with ESP-IDF
  • Enable web server version 3 (version: 3) for the modern ESPHome dashboard UI

MSR-1.yaml, MSR-1_BLE.yaml, MSR-1_Factory.yaml

  • Remove platformio_options: board_build.flash_mode: dio — no longer needed once the variant/flash_size spec is used

MSR-1_Factory.yaml

  • Remove legacy wifi: on_connect/on_disconnect BLE hooks — these are no longer required

Types of changes

  • Bugfix (fixed change that fixes an issue)
  • New feature (thanks!)
  • Breaking change (repair/feature that breaks existing functionality)
  • Dependency Update - Does not publish
  • Other - Does not publish
  • Website of github readme file update - Does not publish
  • Github workflows - Does not publish

Checklist / Checklijst:

  • The code change has been tested and works locally
  • The code change has not yet been tested

If user-visible functionality or configuration variables are added/modified:

  • Added/updated documentation for the web page

Exposes the device's current IP address as a diagnostic text sensor
in Home Assistant, making it easier to identify devices on the network.
Exposes the running ESPHome version and the Apollo firmware version as
diagnostic text sensors in Home Assistant, making it easy to verify
what firmware is installed without checking the ESPHome dashboard.
ESPHome renamed `services`/`service` to `actions`/`action` in newer
versions. Updates all three API entries (play_buzzer, calibrate_co2_value,
set_ld2410_bluetooth_password) to use the current syntax.

BREAKING CHANGE: Any Home Assistant automations or scripts that call
these as services must be updated to use the new action syntax.
…io options

- Replace deprecated board name with explicit variant/flash_size spec
  (esp32c3 + 4MB) — removes need for platformio_options flash_mode override
- Enable web server version 3 (modern UI)
- Remove platformio_options board_build.flash_mode: dio from all device YAMLs
- Remove legacy BLE on_connect/on_disconnect wifi hooks from Factory yaml
@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

Warning

Rate limit exceeded

@bharvey88 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 12 minutes and 25 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 8522ad8 and 71e4527.

📒 Files selected for processing (4)
  • Integrations/ESPHome/Core.yaml
  • Integrations/ESPHome/MSR-1.yaml
  • Integrations/ESPHome/MSR-1_BLE.yaml
  • Integrations/ESPHome/MSR-1_Factory.yaml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch esphome-modernisation

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants