Skip to content

Fix ESPHome dev build: use on_value trigger variable instead of Select .state - #23

Merged
TrevorSchirmer merged 51 commits into
devfrom
fix/select-state-dev-compat
Jul 7, 2026
Merged

TrevorSchirmer merged 51 commits into
devfrom
fix/select-state-dev-compat

Conversation

@TrevorSchirmer

Copy link
Copy Markdown
Member

Version: 26.7.7.1

What does this implement/fix?

Fixes the CI dev-channel build failure:

'class esphome::template_::TemplateSelect<...>' has no member named 'state'

ESPHome removed the deprecated std::string state member from select entities on the dev branch (deprecated in 2026.1.0, removed for 2026.7.0). The on_value condition lambdas of the firmware_selector select in TEMP_PRO-1_W.yaml and TEMP_PRO-1_ETH.yaml read id(firmware_selector).state, which no longer compiles against dev.

Since these lambdas run inside the select's own on_value trigger, the newly selected option is already available as the trigger variable x. Comparing x == "Ethernet" works on every ESPHome version back to this config's min_version: 2025.2.0 (x is std::string on older releases and StringRef on dev — both compare against string literals), so stable, beta, and dev all compile without raising the minimum version. Runtime behavior is unchanged.

Also bumps the firmware version to 26.7.7.1 per convention.

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

🤖 Generated with Claude Code

TrevorSchirmer and others added 30 commits December 12, 2024 01:01
…dates

Bumps the github-actions group with 3 updates in the / directory: [pozil/auto-assign-issue](https://github.com/pozil/auto-assign-issue), [actions/checkout](https://github.com/actions/checkout) and [esphome/build-action](https://github.com/esphome/build-action).


Updates `pozil/auto-assign-issue` from 1 to 2
- [Release notes](https://github.com/pozil/auto-assign-issue/releases)
- [Commits](pozil/auto-assign-issue@v1...v2)

Updates `actions/checkout` from 3 to 4
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

Updates `esphome/build-action` from 6 to 7
- [Release notes](https://github.com/esphome/build-action/releases)
- [Commits](esphome/build-action@v6...v7)

---
updated-dependencies:
- dependency-name: pozil/auto-assign-issue
  dependency-version: '2'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
- dependency-name: actions/checkout
  dependency-version: '4'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
- dependency-name: esphome/build-action
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/github-actions-09397a1a04

chore(ci): bump the github-actions group across 1 directory with 3 updates
TrevorSchirmer and others added 21 commits June 1, 2025 15:07
Bumps the github-actions group with 2 updates: [actions/checkout](https://github.com/actions/checkout) and [esphome/build-action](https://github.com/esphome/build-action).


Updates `actions/checkout` from 4.1.7 to 4.2.2
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4.1.7...v4.2.2)

Updates `esphome/build-action` from 6 to 7
- [Release notes](https://github.com/esphome/build-action/releases)
- [Commits](esphome/build-action@v6...v7)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 4.2.2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions
- dependency-name: esphome/build-action
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/github-actions-7223b173aa

chore(ci): bump the github-actions group with 2 updates
fix: initial_value over max_value
Bumps the github-actions group with 1 update: [actions/checkout](https://github.com/actions/checkout).


Updates `actions/checkout` from 4.2.2 to 5.0.0
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4.2.2...v5.0.0)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 5.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/github-actions-ee1c097272

chore(ci): bump actions/checkout from 4.2.2 to 5.0.0 in the github-actions group
…andon

Change auto-assign assignee from TrevorSchirmer to bharvey88
Bumps the github-actions group with 1 update: [actions/checkout](https://github.com/actions/checkout).


Updates `actions/checkout` from 5.0.0 to 6.0.2
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v5.0.0...v6.0.2)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 6.0.2
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/github-actions-e6ee9d7de3

chore(ci): bump actions/checkout from 5.0.0 to 6.0.2 in the github-actions group
Bumps the github-actions group with 1 update: [pozil/auto-assign-issue](https://github.com/pozil/auto-assign-issue).


Updates `pozil/auto-assign-issue` from 2 to 3
- [Release notes](https://github.com/pozil/auto-assign-issue/releases)
- [Commits](pozil/auto-assign-issue@v2...v3)

---
updated-dependencies:
- dependency-name: pozil/auto-assign-issue
  dependency-version: '3'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
…s/github-actions-3fa3f09d17

chore(ci): bump pozil/auto-assign-issue from 2 to 3 in the github-actions group
…t .state

ESPHome removed the deprecated std::string state member from select
entities on the dev branch (deprecated 2026.1.0, removal 2026.7.0),
which broke the CI dev-channel build:
'class esphome::template_::TemplateSelect<...>' has no member named 'state'

Inside the select's own on_value trigger the selected option is
available as the trigger variable x, which works on every ESPHome
version back to this config's min_version of 2025.2.0 (std::string
on older releases, StringRef on dev - both compare against string
literals). Runtime behavior is unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@TrevorSchirmer, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 59 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 04bbffb6-0178-4780-b854-b699b34c874d

📥 Commits

Reviewing files that changed from the base of the PR and between 7b6df83 and 4d4aee0.

📒 Files selected for processing (3)
  • Integrations/ESPHome/Core.yaml
  • Integrations/ESPHome/TEMP_PRO-1_ETH.yaml
  • Integrations/ESPHome/TEMP_PRO-1_W.yaml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/select-state-dev-compat

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.

@github-actions github-actions Bot added the bugfix Bug fix label Jul 7, 2026
@TrevorSchirmer
TrevorSchirmer changed the base branch from main to dev July 7, 2026 15:36
@TrevorSchirmer
TrevorSchirmer merged commit 3732856 into dev Jul 7, 2026
8 of 9 checks passed
@TrevorSchirmer
TrevorSchirmer deleted the fix/select-state-dev-compat branch July 7, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants