docs: select FTW delivery order and simple setup scope - #1214
Merged
Merged
Conversation
Signed-off-by: Fredrik Ahlgren <fredrik@sourceful-labs.com>
This was referenced Sep 12, 2026
Signed-off-by: Fredrik Ahlgren <fredrik@sourceful-labs.com>
frahlg
marked this pull request as ready for review
September 12, 2026 10:00
Contributor
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_4db26679-76db-4162-80c9-ffe66b61cd18) |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
FTW now has an owner-selected delivery order: close the confirmed control and Lua-host gaps, finish daily charging, then simplify commissioning and make live evidence useful to people and agents.
Record the source/test/browser baseline in the maintained roadmap, including the nine baseline regressions from #1170/#1199 (seven failed, two passed on merged Core f1a3b76). Keep existing capabilities, missing flows and missing physical evidence distinct.
The owner has removed the STRÅNG / roof geometry / panel drawing / related settings stack from Core scope. A future optional extension remains undecided; normal setup must work without irradiance-source selection, azimuth or panel layouts. VISION.md states that boundary. Close the old Python-optimizer implementation work while preserving useful requirements for current Energyplan evaluation and household-goal reporting. Independent benchmarks and development tools remain valid.
Validation: reviewed current code, relevant PR diffs and reviews, ran the recorded baseline regressions through a temporary overlay, and rendered the webapp simulator during the baseline audit. Documentation changes pass git diff --check and the planning-document gate. No runtime code or worker artifact changes. These are selected priorities and evidence requirements, not claims of physical commissioning or a new release.
Note
Low Risk
Documentation-only updates to vision and roadmap; no changes to dispatch, APIs, or shipped behavior.
Overview
This PR records owner-selected product scope and priorities in planning docs only—no runtime, worker, or client code changes.
VISION.mdnow states explicitly that STRÅNG, roof geometry, and panel drawing are outside Core scope, with normal setup required to work without irradiance-source choice, azimuth, or panel layout; a future optional extension remains undecided.docs/roadmap.mdgains a 12 September 2026 implementation review section that: pulls the STRÅNG/settings stack and retired Python-optimizer PRs out of the Core queue while keeping useful Energyplan requirements; pins a reviewed baseline (Core/webapp/native/drivers/website SHAs) and documents seven failing / two passing regression cases from #1170/#1199 on that baseline; lists confirmed legacy control gaps (Luadriver_command, dispatch load/SoC, curtail freshness); adds an area-by-area “what exists vs what remains” table; and sets a numbered delivery order (control gaps → daily charging → commissioning → live evidence → external authority/savings → later thermal).Reviewed by Cursor Bugbot for commit 79671c6. Bugbot is set up for automated code reviews on this repo. Configure here.