Conversation
esphome/esphome#19264 renamed the esp32_improv component to improv_ble and decoupled it from the ESP32 platform. The old top-level key still works through an alias, but every build now logs: The 'esp32_improv:' top-level key is deprecated; rename it to 'improv_ble:'. Removed in 2027.4.0. Only the key and the internal C++ names changed. Every option this config uses (id, authorizer) is unchanged, as are identify_duration, authorized_duration, wifi_timeout, status_indicator, next_url and the on_* triggers. improv_serial keeps its own key and options. improv_ble is not in any release tag yet, so min_version goes to 2026.10.0. That only affects people compiling the YAML themselves; precompiled and OTA users are unaffected. Version: 26.9.17.2 🤖 Generated with [Claude Code](https://claude.com/claude-code)
|
Warning Review limit reachedNext included review available in 59 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
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. Comment |
10 tasks
bharvey88
added a commit
that referenced
this pull request
Sep 18, 2026
The CAST-1 flash runs on the generic driver, so the vendor drivers linked beside it never run but keep their code in IRAM. flash_chip: generic drops them, freeing about 5KB of internal RAM (4,784 B measured at runtime). Needs ESPHome 2026.10.0; the min_version bump is in #81. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version: 26.9.17.2
What does this implement/fix?
esphome/esphome#19264 renamed the
esp32_improvcomponent toimprov_bleand decoupled it from the ESP32 platform. The old top-level key still resolves through an alias, but every build now logs:This renames the key and its id. I checked what else changed in #19264, since the title also says "decouple from ESP32":
id,authorizer,status_indicator,identify_duration,authorized_duration,wifi_timeout,next_urland theon_provisioned/on_provisioning/on_start/on_state/on_stoptriggers, with the same defaults.USE_*defines, the log tag, and thedump_configheading. None of that is reachable from this config, which has no lambdas touching the component.DEPENDENCIES = ["wifi", "esp32"]with a backend table pluscv.only_on(["esp32"]). ESP32 is still the only BLE server backend, so behaviour on the CAST-1 is identical.improv_serial:is unaffected. Its key, options and schema are unchanged.min_version and CI
improv_bleis in no release tag yet. The latest release is 2026.9.0 and the rename ships in 2026.10.0, somin_versiongoes from 2026.8.0 to 2026.10.0.This PR will sit with failing CI until 2026.10.0 is released.
ci.ymlruns thestableandbetalegs only, and both are on versions older than 2026.10.0, so both fail themin_versioncheck. Thebetaleg will go green as soon as 2026.10.0b1 is published, thestableleg once 2026.10.0 lands. Same pattern as AIR-1 #124. Validated locally against an ESPHome dev checkout:INFO Configuration is valid!with no deprecation warning.The
min_versionbump is not a breaking change for users. It only affects people compiling this YAML themselves; precompiled and OTA users are unaffected.Types of changes
Checklist / Checklijst:
If user-visible functionality or configuration variables are added/modified:
🤖 Generated with Claude Code