6 Commits (266325addd82eba5480da6fe3dbbef20bc27956c)

Author	SHA1	Message	Date
Strycher	3b39a16edd	chore(#462): remove em-dash characters from test/ (and lib top-up) ... Follow-up to #457. Swept the em-dash character (U+2014) out of the test tree (test descriptions and comments) and cleaned 8 em-dashes that landed in lib comments via the #456 refactor after #457 merged, so the tree is back to zero. Same rules: replaced with commas/colons/periods, preserved the lone "no data" glyph placeholders, left non-English ARB untouched. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2 months ago
Strycher	48a9d79d5a	refactor(#456): capability-section schema (v2); broker enabled not a profile field ... - ConfigProfile becomes sectioned: wifi + mqtt (region/status_interval/brokers). schema_version -> 2. Parser/enumerator read sections; same firmware-key writes, so apply/screens unchanged. Future radio/repeater/companion sections slot in. - Broker 'enabled' removed from the profile schema: enabling a broker is the operator's runtime call (firmware ships slots disabled, opt-in #262). Apply preserves the device's current enabled state; parser rejects broker 'enabled'. 689 tests pass, analyze clean. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2 months ago
Strycher	900c168c61	refactor(#456): restructure config-profile schema into capability sections ... Profile is now a container of capability-scoped sections (schema_version 2): - wifi (WifiConfig) — any wifi device - mqtt (MqttSection = region + status_interval + brokers) — observer capability so MQTT is defined once and shared by observer / observer-repeater / observer-companion, never redone per device. Future radio/repeater/companion/ display sections slot in alongside. Parser reads the sectioned YAML and rejects the old flat v1 layout with a clear message. Enumerator reads from sections but emits the SAME firmware keys, so apply/diff/screens are unchanged. 45 config-profile tests updated + green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2 months ago
Strycher	b96f20140f	chore(#457): remove em-dash characters from source ... Ben's style rule: no em-dash character in copy, docs, or code comments (it reads as an AI tell), and the repo is going public. Replaced the em-dashes in code comments and English UI strings with commas, colons, or periods, whichever reads best. User-facing strings were hand-tuned for natural punctuation rather than a blanket comma. Preserved the lone "no data" glyph placeholders (a standalone dash used as a not-available indicator in status displays); those are a design element, not prose. Regenerated app_localizations*.dart from app_en.arb (the English fallback for untranslated keys propagates to every locale's generated file). Non-English ARB translations left untouched. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2 months ago
Strycher	d3fd110509	fix(#403): tighten profile validation after Gemini review ... Address 3 review findings (all confirmed real): thread section context through the optional-field accessors so nested errors name the broker (brokers[i]."port"), reject broker ports outside 1..65535, and reject negative integer fields (status_interval, jwt_refresh) via a shared _optUint. +3 tests (14 total). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2 months ago
Strycher	e73c2b5a10	feat(#403): parse YAML into a config-profile model ... Adds the yaml dependency and parseConfigProfile(): YAML -> ConfigProfile (#402). Strict by design since profiles are untrusted input (#139) — unknown keys, wrong types, out-of-range/duplicate broker slots, and unknown transport/auth values all throw ConfigProfileFormatException with a user-facing message. Only keys present populate the model. 11 unit tests. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2 months ago