Repository navigation
Conversation
Copied again from srcfl/ftw web/components/ftw-energy-flow.js at b4d2488d41ff046356a6710b1d173e38cdc96a64 (srcfl/ftw#1862). fmtKw prints the size of a power; the word under the bubble, such as "discharging", says the direction. The signed kW the component sums is unchanged. The provenance line is stamped with that commit and the digest is re-recorded with RECORD_VENDOR_DIGESTS=1, as tests/vendored.test.ts asks. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
ftw-webapp | 0a80d02 | Commit Preview URL Branch Preview URL |
Oct 26 2026, 07:16 AM |
frahlg
marked this pull request as draft
October 26, 2026 06:53
The box's second review round: Home use in the middle keeps its sign, the planets keep printing sizes, and a battery FTW only observes says so under its title while the line under its size keeps the direction. Copied byte for byte; digests re-recorded. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
frahlg
marked this pull request as ready for review
October 26, 2026 07:17
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.
Paired with srcfl/ftw#1862, which removes raw minus signs from the box page. The box's
<ftw-energy-flow>printed a discharging battery as "−1.20 kW". It now prints "1.20 kW" with "discharging" under it, and still sums the signed value when it folds several packs.This copies
src/vendor/ftw/ftw-energy-flow.jsbyte for byte from the box branch (stampedb4d2488d) and re-recordssrc/vendor/ftw/digests.json. The web app no longer draws this component on Now, because it draws the ring (#110). The copy stays in step so the vendored check holds.Merge together with srcfl/ftw#1862. If that branch changes before merge, take the copy again.
Contract-pair: srcfl/ftw@cd68b44
Evidence:
npm run verifypasses, with svelte-check at 0 errors and 1178 tests plus 48 stats tests passing.tests/vendored.test.tsfailed before the re-record and passes after it. No token drift against the box branch.🤖 Generated with Claude Code