Ai summary#7
Merged
Merged
Conversation
…nces ✅ Fixed Apple Intelligence context window overflow: - Reduced data sent to AI by 80-90% while maintaining accuracy - Compressed detailed set-by-set data into essential summaries - Streamlined last week's data to key metrics only 🎯 Improved workout name references: - Added workout split summary at prompt start for context - AI now properly references actual workout day names - Prevents hallucination of non-existent workout types 📊 Maintained all functionality: - Key Stats: Proper volume, sessions, PRs with percentages - Trends: Accurate trend analysis without duplication - Areas of Concern: Uses actual workout names from split - Recommendations: References correct workout day names 🔧 Data optimization: - THIS WEEK: Exercise summaries with volume, max weight, reps - LAST WEEK: Essential metrics for percentage calculations - Special techniques tracking (failure, drop sets, rest-pause) - Skipped exercises properly captured for recommendations 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Fixed critical bug where changing a set's type (warmup, drop set, etc.) would incorrectly modify a different set. The issue was caused by passing a 1-based display number as a 0-based array index, plus duplicate save logic. Changes: - Add setIndex computed property for correct 0-based array indexing - Pass setIndex instead of setDisplayNumber to SetTypeCell - Remove duplicate save logic from SetTypeCell (EditExerciseSetView handles saving on "Done") - Remove unused @Environment(\.modelContext) from SetTypeCell 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Fixed critical bug where AI summary was showing 91 sessions instead of 3 due to incorrect date range filtering. The issue was caused by string comparison of dates in "d MMMM yyyy" format (e.g., "9 October 2025" > "10 October 2025" fails because "9" > "1" as strings). Changes: - WorkoutDataFetcher: Convert date strings to Date objects before comparison for accurate filtering - WorkoutDataFetcher: Add locale to ensure consistent date parsing - WorkoutSummarizer: Add WorkoutSummaryError for better error handling - WorkoutSummarizer: Improve prompt to handle cases with no workout data - AISummaryView: Allow summary generation even with no workouts (for motivational feedback) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Added comprehensive screenshots to README.md and created detailed documentation: - Added screenshots gallery to README.md with app features - Created docs/architecture.md with architectural overview and diagrams - Created docs/user_guide.md with step-by-step user instructions - Added docs/images/ folder with 10 app screenshots 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
No description provided.