Surface tracked nutrient totals in daily nutrition and food log#14
Merged
Conversation
get_daily_nutrition returned the nutrient catalog instead of consumed amounts. Rewire it to report consumed totals for every tracked nutrient, and add the same summary to get_food_log. Closes #13
rwestergren
pushed a commit
that referenced
this pull request
Jun 15, 2026
* Add saturated_fat_g to add_custom_food Custom foods could not include saturated fat — the field had no parameter, so foods created via the API silently undercounted daily saturated fat totals (now surfaced in summaries since #14). Adds saturated_fat_g (per-serving, g, default 0) to add_custom_food, wired to nutrient ID 606 with the same per-100g scaling as other nutrients. Backward compatible — existing calls are unaffected. Field-tested for 3 weeks as a local patch driving a personal food library with cardiovascular-target tracking. The same pattern extends naturally to cholesterol, trans_fat, and omega_3/6 (already in NUTRIENT_IDS) if there's interest in a follow-up. * Reformat saturated_fat nutrient entry to satisfy ruff
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
1 participant
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.
get_daily_nutritionreturned the nutrient catalog instead of consumed amounts. Rewire it to report consumed totals for every tracked nutrient, and add the same summary to get_food_log.Closes #13