Skip to content

Release 26.9.21.1 - Hotfix: out of memory fix when using Ethernet - #86

Merged
Bunton33 merged 11 commits into
mainfrom
beta
Sep 21, 2026
Merged

Bunton33 merged 11 commits into
mainfrom
beta

Conversation

@bharvey88

Copy link
Copy Markdown
Contributor

Version: 26.9.21.1

What does this implement/fix?

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

ESPHome dev changed http_request update's setter to
set_source_url(const char *), and it stores the pointer without
copying. The current lambda builds a std::string, which no longer
compiles, and adding .c_str() to a local std::string would leave the
component holding a dangling pointer once the lambda returns.

Select between two string literals instead. Literals have static
storage duration, so the stored pointer stays valid for the life of the
device. This also builds on 2026.8.x, where the setter still takes a
std::string and const char * converts implicitly, so no min_version
change is needed.

Version: 26.9.17.1

🤖 Generated with [Claude Code](https://claude.com/claude-code)
The dev leg was dropped on 2026-07-07 (1d5470b) with the note "dev
excluded until the TemplateSelect .state incompatibility is fixed". That
incompatibility was fixed the same day in 2d2d311, and again for the
wizmote selects on 2026-07-12 in 8a5ed17. Every select in the config now
uses current_option(); the remaining .state calls are on text, switch and
binary_sensor components, where it is still correct.

The leg was never turned back on, so two breaks against ESPHome dev went
unnoticed until a local dev build surfaced them: the OTA manifest lambda
no longer compiling against the new set_source_url signature, and the
esp32_improv deprecation. The first would have broken builds for anyone
compiling this YAML on the day 2026.10.0 released.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
The PR was opened on 2026-09-17 and merges on 2026-09-18.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
The 64KB data cache came in from the Home Assistant Voice PE config,
where it feeds microWakeWord. The CAST-1 has no microphone. On the S3 the
cache is carved out of internal SRAM, so dropping to the IDF default of
32KB returns 32KB to the internal heap.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
This merges to beta first.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
Drops the ESP32-S3 data cache from 64KB to the ESP-IDF default of 32KB.
The 64KB setting came from the Home Assistant Voice PE config, where it
serves microWakeWord; the CAST-1 has no microphone. The cache is carved
out of internal SRAM, so this returns 32KB to the internal heap that runs
short when streaming over Ethernet.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
ESPHome 2026.10 changes set_source_url to take a const char * and store
the pointer without copying. The OTA manifest lambda built a std::string,
which no longer compiles. It now selects between two string literals,
which stay valid for the life of the device and also build on 2026.8.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
Retargeting this PR from main to beta so main and beta stay in sync;
the next release carries it to main.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
Re-enable the dev leg of the ESPHome CI matrix
🤖 Generated with [Claude Code](https://claude.com/claude-code)
@bharvey88 bharvey88 added the bugfix Something isn't working label Sep 21, 2026
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 58 minutes.

Check out review usage here.

View limit details

Limit 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.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 04f6f70e-1a76-496c-92fb-58782a221f2e

📥 Commits

Reviewing files that changed from the base of the PR and between 8347dee and c2d5ab5.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • Integrations/ESPHome/Core.yaml

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.

@Bunton33
Bunton33 merged commit a9cd997 into main Sep 21, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants