Skip to content

Ai summary#7

Merged
Rektoooooo merged 5 commits into
mainfrom
AI-summary
Oct 6, 2025
Merged

Ai summary#7
Rektoooooo merged 5 commits into
mainfrom
AI-summary

Conversation

@Rektoooooo

Copy link
Copy Markdown
Owner

No description provided.

Rektoooooo and others added 5 commits September 22, 2025 20:19
…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>
@Rektoooooo Rektoooooo merged commit a0c5db3 into main Oct 6, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant