homelab-codex-ws/kb/decisions/ha-configs-as-code.md
oskar 01db57ab82 fix(kb): przepiecie wszystkich odwolan wewnetrznych po migracji
126 plikow (md, yaml, sh, py) odwolywalo sie do sciezek sprzed migracji.

  15  markdown-linkow [..](..) -> policzona sciezka WZGLEDNA wobec pliku
      odsylajacego (wczesniej czesc z nich byla repo-root-relative i nie
      rozwiazywala sie z katalogu, w ktorym lezala)
 200  odwolan tekstowych (backticki, proza, yaml, importy w kodzie)
      -> nowa sciezka repo-root-relative, zgodnie z konwencja repo
   5  linkow rodzenstwa (gole nazwy plikow, np. "](DEPLOY.md)") — dzialaly
      tylko w starym katalogu; przeliczone recznie

Objete m.in.: CLAUDE.md (scripts/onboard/README.md -> kb/runbooks/
node-onboarding-tool.md, docs/backlog.md -> kb/phases/backlog.md),
README.md, .claude/skills/, 20 session logow, kod jobow.

Ostatnie 5 odwolan pochodzi z tresci wciagnietej rebasem z origin/master
(session log 2026-07-31, override node-agenta na SOLARII, dwie pozycje
backlogu) — wskazywaly na docs/incidents/, docs/kb/modules/ i
services/narty27/README.md sprzed migracji.

Dodany wzajemny link miedzy kb/services/control-plane.md (stub kodu)
a kb/subsystems/control-plane.md (opis, deprecated) — dwa dokumenty o tym
samym systemie, latwe do pomylenia.

Weryfikacja na 790 plikach: 0 odwolan do starych sciezek,
0 martwych linkow markdown. Lint OKF: 190/190 plikow ZGODNE.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-08-04 16:53:57 +02:00

228 lines
13 KiB
Markdown

---
okf: "0.1"
type: decision
visibility: private
status: active
updated: 2026-07-30
links:
- ../incidents/2026-07-22-ha-dwie-instancje.md
---
# Home Assistant configs-as-code — design decisions
Status: **phase 1 (partial)**. `scripts/ha/deploy.sh` implements the write
path for the `api` adapter's automations/scripts/scenes scope (see "Deploy
path" and "Sync model" below); dashboards/helpers and the `docker-exec`
adapter have no write path yet. See `kb/phases/backlog.md` for the tracking
entry.
## Phasing
| Phase | Scope |
|---|---|
| **0 — Snapshot** | Import-only tooling (this skeleton). Pull `/config` + `.storage` from each instance into the repo, read-only. No deploy, no write path back to HA. |
| **1 — Repo + CC** | Repo is the reviewable source of truth. Changes are authored in the repo (by a human or Claude Code) and pushed manually via the docker-exec / api adapters described below. Deploy has a hard drift-abort (see Sync model). **Partially built**: `scripts/ha/deploy.sh` covers the `api` adapter's automations/scripts/scenes scope; `docker-exec` deploy and dashboards/helpers writes are still open. |
| **2 — MCP read-only** | Expose HA state (entities, areas, config) to agents via an MCP server in read-only mode, to let agents reason about the live instance without touching import/deploy paths. **2a done (operator decision 2026-07-30: own minimal MCP, not `hass-mcp`)**`services/ha-mcp/`, seven read-only tools over stdio, registered for Claude Code in the repo-root `.mcp.json`. Read-only by construction: GET + allowlisted WS `*_list` reads only; automations are read from the repo, not the API. Open: exposing it to anything other than CC on the dev station (2b). |
| **3 — Agents** | Agents propose changes (automations, scripts, scenes) through the same reviewable repo path used by humans; the human-in-the-loop approval flow from `services/control-plane/` (pending → approved → executed) governs anything destructive. Telegram becomes a first-class interface alongside CC. |
Each phase is a hard gate: no phase-N tooling depends on phase-(N+1) existing.
## Scope
Full `/config` tree per instance, plus a curated export of `.storage/*` JSON
files that are meaningfully version-controllable (registries, dashboards,
helpers) — not secrets, tokens, or runtime databases. See `.gitignore` below
for the exact exclusion list.
## Canonical format
All YAML committed to the repo passes through one normalization function
(`scripts/ha/lib/normalize.py`), used by every import path with no
per-caller variation:
- block style (no flow/inline collections)
- fixed indent width 2
- keys sorted (stable diffs)
- unlimited line width (no wrapping)
- UTF-8, no BOM
`.storage/*` files are JSON on disk in HA; on import they are parsed as JSON
and re-emitted through the same YAML normalizer so the whole repo — config
and storage-export alike — has one diff format.
## Deploy path: adapter per instance
There is no single deploy mechanism — each HA instance gets an adapter
behind a common interface (`import.sh`/eventual `deploy.sh <instance>`):
| Instance | Adapter | Why |
|---|---|---|
| `ken` (RPi4, HAOS, LAN `192.168.31.7:8123`) | **api** | Canonical home instance since the 2026-07-22 cutover (see Incident log). HAOS has no SSH access, so there is no `docker exec`/filesystem path — only the HA REST/websocket API is reachable. Full `/config` import is deferred until an alternative access path exists; for now the api adapter's import scope is limited to what the API exposes: automations, scripts, scenes, dashboards. |
| `ken-legacy` (piha, container `homeassistant5`) | **docker-exec over SSH** (archive-only) | Pre-migration container instance, superseded by `ken` at 31.7 (see Incident log) — same container filesystem access as the old `ken` entry (`ssh oskar@piha "docker exec homeassistant5 ..."`). Import only, for historical reference; never a deploy target. |
| `chelsty-ha` | **api** | Reachable over Tailscale at `100.70.180.90:8123` (confirmed working path — `kb/runbooks/ha-diag-agent-deploy.md` already curls this for health checks). Config-as-code deploy will reuse the same reachability, calling the HA REST/websocket API rather than shelling into the container. |
**Open**: a `file` adapter (direct bind-mount / SSH `rsync` to the config
directory, bypassing `docker exec`) is worth revisiting once SSH access to
the `chelsty-ha` VM itself is verified — see Open questions.
## Sync model: bidirectional, asymmetric safety
- **Deploy** (repo → instance): hard **drift-abort**. Before writing, the
adapter re-imports the instance's current state and diffs it against the
last-known-imported snapshot committed in the repo. Any unexpected
difference aborts the deploy with a non-zero exit and a diff printed —
never silently overwrites live drift. This applies unconditionally on
PIHA (`ken`); untested/undecided whether it should be relaxed for
`chelsty-ha` given its intermittent LTE uplink (see Open questions).
- **Import** (instance → repo): the reverse direction is intentionally
permissive. Running `import.sh <instance>` on a dev workstation pulls
current state and, if it differs from the last commit, the operator
commits it under a conventional message: `drift(<instance>): <summary>`.
This is how out-of-band UI changes (made directly in the HA web UI) get
captured back into git history instead of being silently overwritten by
the next deploy.
## Validation gate
`check_config` (HA's built-in config validator, invoked via the running
container/instance — `docker exec homeassistant5 python -m homeassistant
--script check_config -c /config` or the equivalent over the api adapter)
is a mandatory gate before any deploy. A failing `check_config` blocks the
deploy entirely; it is not a warning.
## Change classification: reload vs. restart
Deploys default to the least disruptive mechanism:
- **reload** — default for anything HA exposes a reload service for
(automations, scripts, scenes, input_* helpers, template entities, etc.)
via `homeassistant.reload_config_entry` / domain-specific `*.reload`
services.
- **restart** — only when the changed file requires it (e.g.
`configuration.yaml` core changes, new integrations, `.storage`
registry edits) **and** only when the deploy is invoked with an explicit
`--restart` flag. No implicit restarts, ever — an unattended restart on
`chelsty-ha` during a period of LTE unavailability would leave the site
without automation until someone is physically present.
## Split + normalization
On import, list-of-object YAML files are split one-object-per-file so diffs
stay scoped to what actually changed:
- `automations.yaml``config/<instance>/automations/<id>.yaml`
- `scripts.yaml``config/<instance>/scripts/<key>.yaml`
- `scenes.yaml``config/<instance>/scenes/<id>.yaml`
Every other `*.yaml` file under `/config` is copied through the normalizer
1:1 (same relative path, same filename). Non-YAML files under `/config`
(binaries, databases, `secrets.yaml`, logs — see `.gitignore`) are never
copied into the repo.
## Tokens
- A dedicated `deploy_agent` HA user account (admin rights, **local-only**
— never exposed through the public API/ingress) is created per instance,
mirroring the existing `diag_agent` account pattern documented in
`kb/runbooks/ha-diag-agent-deploy.md`. Reusing `diag_agent` is explicitly
rejected — deploy tooling and the diagnostic agent must be revocable
independently.
- Long-lived access tokens for `deploy_agent` live at
`~/.config/ha-deploy/<instance>.token` **on PIHA** (the control node
where import/deploy tooling runs), `chmod 600`.
- Tokens are never committed to the repo, never templated into
`env.example`-style files, and never logged. Import scripts that need a
token to hit `/api/states` fail soft (skip the fixtures step with a
message) if the token file is absent, rather than aborting the whole
import — see `scripts/ha/import.sh`.
## Interface
- Phase 0/1: Claude Code is the interface, with the operator's mobile CC
client acting as the bridge for approvals made away from a desk.
- Phase 3 (agents): Telegram joins CC as a first-class interface, reusing
the existing Telegram bot / approval-queue pattern from
`services/control-plane/`.
## Decyzje operatora po audycie 2026-07-23
Zobacz `docs/audyt-automatyzacji-2026-07-23.md` (sekcja "Do decyzji operatora",
17 punktów) — poniżej wyłącznie decyzje, które doprowadziły do zmian w
fix-pack 1 (`task/ha-fix-pack-1`) albo świadomie do braku zmian. Reszta
checklisty (baterie/re-pairing czujników, kalibracje TRV, xiaomi_miot,
konsolidacja aliasów, higiena 4.x) zostaje otwarta w `kb/phases/backlog.md`.
- **Pkt 6 (klima salon: sunset ubija też ręczne chłodzenie?)** — decyzja:
NIE. „Klima salon: wyłącz…" (`1784804668795`) ma teraz respektować
`input_boolean.klima_salon_auto` — gałąź sunset/balkon działa tylko gdy
`klima_salon_auto = on` (automat wyłącza tylko chłodzenie, którym sam
zarządza). Wyjątek świadomy: trigger „`klima_salon_auto` przechodzi w
`off`" nadal odpala suszenie parownika bez tego warunku (`choose` z osobną
gałęzią) — to jest graceful shutdown samego automatu, nie efekt trybu
auto, więc warunek `auto = on` by go zawsze ucinał (przełącznik właśnie
zgasł). Zweryfikowano na żywym `ken`: helpery klimy (`klima_salon_auto`,
`_temp_docelowa`, `_tolerancja`) **istnieją** (pkt 5 checklisty — obawa o
martwe helpery z 1.1 nieaktualna).
- **Pkt 7 (enforcer sleep mode gasi światła cyklicznie całą noc) i pkt 9
(konsolidacja czterech nocnych wyłączników)** — bez zmian w tym fix-packu.
Oba wchłania przyszły projekt „architektura night_mode" (patrz
`kb/phases/backlog.md`) — punktowa łatka tu tylko dodałaby kolejny wariant do
już przegęszczonego zestawu nakładających się automatyzacji (audyt 2.2).
- **Pkt 11 (OwnTracks: przywrócić czy skasować) i pkt 12 (Leave auto on:
batch 02 — włączyć z powrotem?)** — świadomie bez zmian; obie wymagają
decyzji operatora spoza zakresu tego fix-packu (integracja/urlop), nie
jednoliniowej poprawki.
- **On-leave alerty ruchu (1.2a)** — świadomie bez zmian: martwe przez
sprzęt (czujniki `unavailable`, pkt 1 checklisty), nie przez logikę
automatyzacji; poprawka logiki niczego by nie naprawiła bez wymiany
baterii/re-pairingu.
Zaimplementowane w tym fix-packu (bez decyzji operatora — jednoliniowe
poprawki oczywistych bugów, audyt 2.6 i 2.3):
- `1752086407230` „dry in Kuchnia": trigger `moist``not_moist`.
- `1752085965483` „dry in Lazienka": literówka `mesaage``message`.
- `1700832676138` „…at 3am": `time_pattern hours: '3'` → punktowy trigger
`time: 03:00:00` (było 60 odpaleń/noc, teraz 1).
- Pkt 8 (`1764190493305` „after 15 minutes anyway" ma respektować ręczne
„Disable AUTO off"?) — TAK: dodany warunek `state: automation.
turn_off_lights_in_kuchania = on` (double-click wyłącza tę automatyzację;
„anyway" teraz cofa się przed tym stanem zamiast go unieważniać po 15 min).
## Konwencje automatyzacji
Ustalone po audycie 2026-07-23 (`docs/audyt-automatyzacji-2026-07-23.md`, sekcja 6
"Spójność stylistyczna", pkt 17 checklisty operatora — decyzja: TAK). Obowiązuje dla
**nowych** automatyzacji od teraz; istniejące **nie są migrowane hurtowo** (patrz
sekcja 4.1 audytu — konsolidacja/rename `entity_id` wymaga osobnej mapy referencji
krzyżowych, to osobny task, nie efekt uboczny porządków).
- **`entity_id`, nie device automations.** `device_id`/encja-UID (32-znakowy hex) są
nieczytelne w YAML-u i kruche przy wymianie sprzętu — nowe urządzenie generuje nowy
`device_id`, a automatyzacja umiera po cichu (dokładnie ten mechanizm ubił parę
Tymka i ukrył rename `occusalon`, patrz audyt 4.1). Nowe automatyzacje używają
`action:`/`trigger:`/`condition:` z `entity_id:` jawnym.
- **Alias po polsku, z prefiksem funkcjonalnym.** Format `"<Funkcja>: <opis>"`, np.
`"Klima salon: włącz chłodzenie i synchronizuj cel"`, `"Pimirror: graceful shutdown
przed odcięciem zasilania"`. Jeden język w aliasie (nie mieszanka PL/EN jak w
automatyzacjach z audytu 6).
- **`description:` zawiera `managed-by: repo`.** Odróżnia automatyzacje autorskie
repo od tych z UI/migracji; przy okazji miejsce na kontekst (skąd odtworzone, jakie
encje zaadaptowano — patrz przykład `1785164185794.yaml`).
## Open questions
- What actually drives the phase-3 operational agent (a new agent process
vs. extending an existing one in `services/`)?
- ~~Own minimal MCP server vs. adopting `hass-mcp` for phase 2 read-only
access~~ — decided 2026-07-30: own minimal server (`services/ha-mcp/`).
Reusing `scripts/ha/lib/{ha_api,ha_ws}.py` keeps one token-handling and
one read-only-by-construction story for the whole HA toolchain, and lets
the tools answer from the repo (`read_automation`) and from
`instances.yaml` (offline `chelsty-ha`) — neither of which a generic
server knows about.
- Whether SSH access to `chelsty-ha` itself (not just its HA API) is
available/reliable enough to justify a `file` adapter there, which would
let phase-1 tooling treat `chelsty-ha` more like `ken` for drift-checking
purposes.