Repository navigation
feat: expose running firmware version and native config hash - #28
Merged
flowcool merged 1 commit intoOct 20, 2026
Merged
Conversation
Owner
Author
|
@coderabbitai review |
✅ Action performedReview finished.
|
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.
Behavior
Publish ESPHome Version as a diagnostic using existing friendly-name conventions. Keep the native effective-configuration hash, omit the timestamp. This identifies the running image without local fleet metadata or HA migration.
Target: flowcool/Smart_Plant:upstream/staging-v2r1. Source
37d21f3, predecessorc16403e. Open after the preceding contribution merges so this PR contains only its owned change. The upstream baseline isbcf373144b2d5c7e3a712fd02698e2e98e45aeed. No PR to JGAguado is authorized.Verification
Both exact-source ESPHome 2026.9.0 full builds pass: API hash 0x67ba409c and MQTT hash 0xc5c12a50. Frozen synthetic fixtures and complete logs are retained under ignored tmp/profiles/37d21f3-*-build.log. Version platform schema accepts hide_hash:false. No deployment is inferred from a build or dashboard record.
Inspect the semantic diff and
git -c core.whitespace=cr-at-eol diff --check; preserve existing line endings. All tracked YAML parses. Firmware fixtures use local fonts/public images and synthetic credentials, never devices or shared builders. Local compilation does not prove deployment.Rollback: revert this isolated PR on staging. No automatic deployment, fleet/NAS/cache purge/MQTT/HA action.
AI assistance
Codex prepared the port or extraction. The author inspected the behavior, contribution boundary, actual-source tests, compatibility and diff. Existing staging OTA confirmation and native shutdown remain intact.
CodeRabbit review
Require current-head completed review and yaml-syntax before merge. Florent approved one manual review request per ignored PR for this series only. Documentation-only PRs use the standing docs gate. Record review/head/checks/merge in owner
infra-3rr.68.Summary by CodeRabbit