[olemarkus]

What changed

Dutch owners who enable their first device currently get a measured-peak question even though household grid charges depend on connection size. The setup step now explains that distinction and points to turning off Capacity limit while retaining price and solar features. The running default remains visible until the owner chooses; saved limits and market-neutral guidance stay intact.

Guided setup also waits for the complete device-setting operation, including fresh reads, queued edits and local commits. Closing device details during a slow save can no longer bypass that operation through Start managing; read failures block activation too.

Depends on #2969, which supplies the guided setup and solar discovery before export. Native daily Smart-task schedules are a separate follow-up.

Tariff basis: ACM household network charges.

Validation

The fixture and prop-contract repairs for #2969's own tests now live in #2969, so each layer passes by itself.

77 focused setup/save/limits unit tests and 13 onboarding/browser regression scenarios passed, including 320/390/480 px layouts. Commit lint and UI typechecks passed. SHS2 live-backend checks covered Dutch copy, solar discovery with no observed export, disabling Capacity limit, explicit review/activation, and 320 px rendering.

SHS2 browser checks used the built settings UI through a local SDK bridge connected to the deployed backend; the saved My Homey session could not access SHS2. Homey shell/socket behavior was not exercised by that smoke test. Main was reinstalled, original settings/geography verified, and the temporary Flow price row removed.

The full-stack pre-push gate passed: static checks, 824 related runtime tests, timezone checks, and all 2,695 settings UI unit tests.

Review budget

• Within 25 source files and 1,000 changed source lines, excluding tests and documentation.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: olemarkus/com.barelysufficient.pels
• Branch: feat/nl-onboarding-limits
• File: .coderabbit.yaml
• Commit: 554c6e1
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: fa248325-12a9-42fb-b7eb-d0fa0f80d39a

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.