Skip to content

docs: cover pre-2026.6 automation, helper, dashboard & scene gaps#59

Open
kingpanther13 wants to merge 1 commit into
homeassistant-ai:mainfrom
kingpanther13:issue-50-coverage-gaps
Open

docs: cover pre-2026.6 automation, helper, dashboard & scene gaps#59
kingpanther13 wants to merge 1 commit into
homeassistant-ai:mainfrom
kingpanther13:issue-50-coverage-gaps

Conversation

@kingpanther13

Copy link
Copy Markdown
Member

What does this PR do?

Fills coverage gaps in the home-assistant-best-practices skill that predate HA 2026.6 — native constructs that have existed for releases but the skill never documented. Closes #50, and folds in additional verified gaps found while auditing the referenced files.

references/automation-patterns.md

  • Trigger Types: zone, template, calendar, webhook, homeassistant, conversation, tag, geo_location, persistent_notification triggers; a "fire an event" action note paired with the event trigger
  • Native Conditions: condition: trigger; a condition: device caveat on the device trigger
  • New sections: Parallel Actions (parallel: vs mode: parallel), Stopping a Sequence (stop:), Variables (variables: / trigger_variables:), Capturing Action Responses (response_variable); a delay: action entry; per-element enabled:

references/helper-selection.md

  • trend, bayesian (new Probabilistic Inference section), and mold_indicator helpers (+ TOC and decision-matrix rows)
  • "How Helpers Are Created": the storage-collection vs config-entry-flow distinction (framed in HA APIs, not tool names)

references/dashboard-guide.md

  • Dashboard strategies + "Take control"; grid_options card sizing; heading card; markdown card; object/typed badge form; expanded visibility conditions (incl. screen); dropped the stale (2024+) heading qualifier

references/scenes.md (new)

  • Scene config shape, scene.turn_on/create/apply/reload, and the snapshot-vs-script distinction; wired into SKILL.md and README.md

Every added snippet was verified against the current home-assistant.io docs and, where relevant, home-assistant/core source (multi-agent adversarial review). Tool-agnostic throughout — no MCP tool names — per CONTRIBUTING.

Checklist

  • Tested with real scenarios (if changing skill content) — syntax verified against live HA 2026.6 docs + core source; not yet exercised on a live instance
  • README.md Skill Contents table updated (if reference files were added or removed) — added references/scenes.md

🤖 Generated with Claude Code

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces comprehensive updates to the Home Assistant best practices documentation, including a new reference guide for Scenes (scenes.md) and extensive additions to the automation patterns, dashboard guide, and helper selection references. The reviewer's feedback focuses on optimizing token-efficiency for LLMs by suggesting that bulleted lists in helper-selection.md, scenes.md, and dashboard-guide.md be converted into comma-separated paragraphs.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread skills/home-assistant-best-practices/references/helper-selection.md
Comment thread skills/home-assistant-best-practices/references/scenes.md
Comment thread skills/home-assistant-best-practices/references/dashboard-guide.md
@kingpanther13 kingpanther13 force-pushed the issue-50-coverage-gaps branch from cd90a0c to b20c2ca Compare June 7, 2026 20:22
Fills coverage gaps that predate 2026.6 (issue homeassistant-ai#50):

automation-patterns.md
- Trigger Types: zone, template, calendar, webhook, homeassistant,
  conversation, tag, geo_location, persistent_notification triggers;
  fire-an-event action note
- Native Conditions: trigger condition; device-condition caveat note
- New sections: Parallel Actions (vs mode: parallel), Stopping a Sequence
  (stop:), Variables (+ trigger_variables caveat), Capturing Action
  Responses (response_variable); delay: action; per-element enabled:

helper-selection.md
- trend, bayesian (new Probabilistic Inference section), mold_indicator
- 'How Helpers Are Created': storage-collection vs config-entry-flow
  classification (HA mechanisms, no tool names)
- TOC + decision-matrix rows

dashboard-guide.md
- Dashboard Strategies (+ 'Take control'); heading card; markdown card;
  grid_options card sizing; expanded visibility conditions (screen, etc.);
  typed entity/object badge form; drop stale (2024+) heading qualifier

references/scenes.md (new)
- Scene config shape, turn_on/create/apply/reload, snapshot-vs-script
- SKILL.md Reference Files + README Skill Contents updated

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@kingpanther13

Copy link
Copy Markdown
Member Author

@sergeykad sorry I forgot to mark these as ready, both this one and #60 are ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Coverage gaps surfaced from ha-mcp PR #1448 (preliminary — not yet researched)

1 participant