feat(energy): price walking into Active energy from measured cadence (v77)

[DropTabl]

Unblocked. OpenStrap/analytics#52 merged; all three pin locations (pubspec.yaml ref, pubspec.lock, kAnalyticsPin beside kAlgoVersion) moved together to its main merge commit a077a4a, and the branch is rebased onto current main. The hop is exactly #52 (calories.dart + tests) — the number it moves IS this PR's change, priced in by the v77 bump + migration.

The bug (user report: "Walking calories are not counted")

A Walking workout records fine, its steps count fine, and the day's Active energy never moves. Not a wiring or sync failure: MOT-02's HR-flex gate deliberately bills nothing below the ACSM moderate floor (its own changelog: billed wake minutes → 4.9%, median daily active → 48 kcal), and a walk at ~95 bpm lives entirely below it. The session's bout number — which bills sub-gate samples at the resting rate — made the gap read like a sync bug.

The fix (edge half)

The analytics half (OpenStrap/analytics#52) prices sub-gate minutes from measured cadence — CADENCE-Adults' published 100/110/120/130 spm ↔ 3/4/5/6 METs line, the one gait signal this platform actually has after MT-05 ruled the 1 Hz accel out. This PR wires it:

• lib/compute/step_cadence.dart — the ONE windows→minutes cadence mapping, fed by the day's resolved live_coverage spans (credited, never raw rows, so band/phone overlap can't double-price a step). Partial coverage pro-rates down: a walk's boundary minute under-bills rather than half a minute of walking pricing a full MET-minute. Untouched minutes are null — unmeasured, not zero.
• Both energy passes go through that one mapping: the coordinator's canonical wakeDayEnergy and the pure pipeline's early-read mirror (DayBundleInput carries the spans across the isolate) — so Today's early number and the derived day bill a walk identically instead of the figure growing when the coordinator's pass lands.
• wakeDayEnergy filters the cadence series in the same pass as its off-skin HR filter; a dropped HR entry without its cadence would price every later cadence against the wrong minute (pinned by test).
• The TDEE block discloses the term (live_coverage_pedometer in inputs_used, walking kcal in the note) only on days it actually priced something.
• kAlgoVersion 76 → 77, changelog above the constant. A day with no pedometer coverage derives byte-identical to v76. No settings, no UI changes — Home's "Active energy" card and the Health export pick the figure up through the existing seam.

What it means for the reported walk

Tracked Walking workout, phone present (so the 100 Hz pedometer covered it), HR below the gate: the walk's minutes now bill (MET − 1) basal-minutes each — e.g. an hour at 110 spm ≈ 220 kcal for an 80 kg profile, previously 0. Walks with no live link stay unpriced (gen4 has no durable 24/7 cadence) — honest abstention, unchanged.

Tests

• step_cadence_test.dart — the mapping: whole/partial minutes, overlap summing, measured-zero vs null, degenerate spans.
• daily_energy_consistency_test.dart — the reported walk bills; total − active == basal (the Health-export invariant) survives the walking term; cadence stays aligned across the off-skin filter.
• flutter analyze clean; full suite green — 3,037 passed / 0 failed against the flipped a077a4a pin, rebased on current main.

Companion: OpenStrap/analytics#52 (the model + citations). Fixes the "Walking calories are not counted" report.

[coderabbitai]

Warning

Review limit reached

Next included review available in 24 minutes.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 62ead3d5-62c5-4be1-b73b-5dc349c21981

📥 Commits

Reviewing files that changed from the base of the PR and between 83375f2 and 916a54b.

⛔ Files ignored due to path filters (3)

• pubspec.lock is excluded by !**/*.lock
• test/daily_energy_consistency_test.dart is excluded by !test/**
• test/step_cadence_test.dart is excluded by !test/**

📒 Files selected for processing (4)

• lib/compute/derivation_engine.dart
• lib/compute/onehz_pipeline.dart
• lib/compute/step_cadence.dart
• pubspec.yaml

---

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.}