Conversation
Fork-submitted PRs get a read-only token on pull_request runs, so the label and assignee bots fail with 403. pull_request_target runs in the base repo context with a write token; safe here because neither job checks out or executes PR code. Build jobs stay on pull_request. Trim ci.yml permissions to what the builds need.
- The Firmware Update entity pointed at /H-1/artifact/manifest.json, which is a 404 (CI publishes to /H-1/firmware/) - the update system has never been able to offer an update - Trigger a manifest check once WiFi connects: the update component's first 6h poll fires before the network is up and skips silently, leaving the entity stale for ~6h after every boot - Fix firmare-d -> firmware-d typo in the H-1D publish directory
Home Assistant shows only the first 255 characters of an ESPHome release summary, so boilerplate is expensive and ## headings render oversized in the update dialog: - Render category titles and What's Changed in bold instead of H2 - Drop the star-the-repo footer - Drop the Full Changelog line: it semver-truncates 4-part versions (always links ...X.Y.Z.1) - the shared build workflow now appends a correct compare link instead
Fix label-check and auto-assign on fork PRs
Fix firmware update system (404 manifest URL, stale first check)
The README was overwritten with the H-2 one (title, photo link to a file this repo does not have, 24 LED / four button spec, H-2 charities and shop link). Rebuilt from the original README (d5137f0) and the firmware: 22 LEDs, ESP32-C3, three songs on one button. The H-1 shop page is gone, so the link points at the store root. Generated with [Claude Code](https://claude.com/claude-code)
Generated with [Claude Code](https://claude.com/claude-code)
# Conflicts: # .github/workflows/autoassign.yml # .github/workflows/ci.yml
Generated with [Claude Code](https://claude.com/claude-code)
2 tasks
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 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 |
Sync main into beta and bump to 26.8.23.1
Fix the H-1 README hardware details
ESPHome 2026.8.0 replaces rgb_order/is_rgbw/is_wrgb with a single channel_colors key on esp32_rmt_led_strip (esphome#18474).
`channel_colors` only exists from ESPHome 2026.8.0 (esphome#18474), but these YAMLs still declared an older min_version. Devices adopted through the ESPHome dashboard track this repo at HEAD with no pinned ref, so on an older ESPHome the next recompile fails with "Unknown option channel_colors" pointing into a file the user never wrote, instead of a clear version error. ESPHome merges remote packages and checks min_version in the preload phase, before component schemas load, so the bump produces the right message. No firmware change: min_version is validation-only and emits no code, so the compiled binary is unaffected. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Replace deprecated rgb_order with channel_colors
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
3 participants
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.
Merges beta into main for the 26.8.23.1 release. Changes include:
firmware-d(wasfirmare-d), so the "No Wifi" button on the classic installer and the Basic variant on install.apolloautomation.com get a manifestMerge commit, not squash. Merging triggers Build and Publish on main, which creates release 26.8.23.1 and publishes
firmware/andfirmware-d/.🤖 Generated with Claude Code