kinemotion 0.47.4__tar.gz → 0.49.0__tar.gz

kinemotion-0.49.0/.basic-memory/development/commit-review-r2-video-storage-migration-fe74d1f.md

@@ -0,0 +1,243 @@
+---
+title: 'Commit Review: R2 Video Storage Migration (fe74d1f)'
+type: note
+permalink: development/commit-review-r2-video-storage-migration-fe74d1f
+---
+
+# Commit Review: R2 Video Storage Migration
+
+**Commit Hash:** fe74d1fc51103df2fb505a1cdc1f0e1f53845c3f
+**Date:** December 12, 2025
+**Author:** Sebastian Otaegui
+**Type:** feat (feature addition)
+
+## Overview
+
+This commit implements a complete migration from Vercel blob storage to Cloudflare R2 for serving original videos. The change addresses MVP requirement #12 and introduces a flexible URL generation strategy supporting both public and presigned URLs.
+
+## Key Changes
+
+### Backend (app.py)
+
+#### 1. **New `get_object_url()` Method** ✅
+- **Location:** `R2StorageClient` class (lines 161-172)
+- **Purpose:** Abstracts URL generation logic with fallback strategy
+- **Implementation:**
+  - Prefers public URL if `R2_PUBLIC_BASE_URL` is configured
+  - Falls back to presigned URL with configurable expiration
+- **Best Practice Alignment:** Matches Cloudflare R2 documentation patterns (from Exa context)
+- **Code Quality:** Clean separation of concerns, well-documented
+
+#### 2. **Configuration Enhancements** ✅
+- Added `public_base_url` attribute (line 115)
+- Added `presign_expiration_s` attribute with error handling (lines 117-122)
+- Default: 3600 seconds (1 hour) for presigned URLs
+- **Security:** Supports both public domains and custom R2 domains
+
+#### 3. **Video Upload Strategy** ✅
+- **UUID-based naming:** `{upload_id}{suffix}` instead of original filename (line 625)
+- **Benefits:**
+  - Prevents filename collisions
+  - Maintains file extension for proper MIME types
+  - Consistent naming across original/debug/results files
+- **R2 Path Structure:**
+  - Original videos: `videos/{jump_type}/{uuid}.mp4`
+  - Results: `results/{jump_type}/{uuid}_results.json`
+  - Debug videos: `debug_videos/{jump_type}/{uuid}_debug.mp4`
+
+#### 4. **AnalysisResponse Extension** ✅
+- Added `original_video_url` field (lines 62, 71, 92-93)
+- Properly serialized in `to_dict()` method
+- **Type Safety:** Optional field with `str | None`
+
+### Frontend (React/TypeScript)
+
+#### 1. **ResultsDisplay.tsx Changes** ✅
+- **Smart URL Prioritization:** R2 URL > local blob URL > undefined (line 231)
+- **useEffect Hook:** Proper cleanup of blob URLs to prevent memory leaks (lines 216-229)
+- **Conditional Rendering:** Shows original video when either URL is available (line 232)
+- **Download Links:** Both original and debug videos (lines 269-291)
+
+#### 2. **Type Definitions** ✅
+- Added `original_video_url?: string` to `AnalysisResponse` interface (frontend/src/types/api.ts)
+
+#### 3. **Test Updates** ✅
+- **ResultsDisplay.test.tsx:**
+  - Added test for R2 original video URL rendering
+  - Verified download link presence
+- **UploadForm.test.tsx:**
+  - Fixed React import (explicit import)
+  - Added proper TypeScript interface
+  - Fixed timestamp type (number vs ISO string)
+
+### Dependencies
+
+#### Frontend Package Updates ✅
+- Added `@types/jest@^30.0.0` for better test type safety
+- Updated yarn.lock with 366 new lines (Jest-related dependencies)
+
+## Code Quality Assessment
+
+### ✅ **Strengths**
+
+1. **Conventional Commits:** Properly formatted commit message with scope
+2. **Type Safety:** All new fields properly typed (TypeScript + Python)
+3. **Error Handling:** OSError exceptions with proper logging
+4. **Documentation:** Clear docstrings for `get_object_url()`
+5. **Flexibility:** Supports both public and presigned URL strategies
+6. **Memory Safety:** Proper blob URL cleanup in useEffect
+7. **Testing:** Comprehensive test coverage for new features
+8. **Logging:** Added URL logging for R2 uploads (lines 633-634)
+
+### ⚠️ **Minor Concerns**
+
+1. **Presigned URL Expiration:**
+   - Default 1 hour may be too short for long analysis sessions
+   - **Recommendation:** Consider 24 hours for production (86400s)
+   - **Justification:** Videos should remain accessible after analysis
+
+2. **Missing Backend Tests:**
+   - No explicit backend tests for `get_object_url()` method
+   - Search didn't find tests for `original_video_url` in backend
+   - **Recommendation:** Add unit tests for R2StorageClient methods
+
+3. **Public URL Configuration:**
+   - `R2_PUBLIC_BASE_URL` not documented in commit message
+   - **Missing:** Deployment documentation for environment variable
+   - **Recommendation:** Update deployment checklist
+
+4. **Frontend Type Import:**
+   - Explicit `import React` added to test files (lines 174, 300)
+   - This is unnecessary with React 17+ JSX transform
+   - **Note:** May be required by test setup (vitest)
+
+### 📋 **Compliance with Project Standards**
+
+| Standard | Status | Notes |
+|----------|--------|-------|
+| Conventional Commits | ✅ | `feat:` prefix, clear description |
+| Type Safety | ✅ | All new code properly typed |
+| Test Coverage | ⚠️ | Frontend tested, backend needs tests |
+| Documentation | ⚠️ | Code documented, deployment docs needed |
+| Error Handling | ✅ | Proper exception handling with logging |
+| Code Duplication | ✅ | No new duplication introduced |
+| Linting | ✅ | Follows project formatting standards |
+
+## Best Practices Validation
+
+### Cloudflare R2 Best Practices (from Exa research)
+
+1. **✅ Presigned URLs:** Correctly implemented with boto3 client
+2. **✅ Public URLs:** Supports custom domain configuration
+3. **✅ S3-Compatible API:** Uses standard boto3 S3 client
+4. **✅ Error Handling:** OSError wrapping for R2 operations
+
+### FastAPI Best Practices (from Ref documentation)
+
+1. **⚠️ Response Models:** Uses custom `AnalysisResponse` class instead of Pydantic
+   - **Current:** Manual `to_dict()` serialization
+   - **Better:** Use `@dataclass` with FastAPI's dataclass support
+   - **Trade-off:** Current approach works fine, but less idiomatic
+
+### React Best Practices
+
+1. **✅ useEffect Dependencies:** Correctly specified `[videoFile]`
+2. **✅ Cleanup Functions:** Revokes blob URLs on unmount
+3. **✅ Conditional Rendering:** Proper boolean coercion with `Boolean()`
+4. **✅ Test Coverage:** Tests for new URL prioritization logic
+
+## Performance Impact
+
+### Positive
+- **Reduced Vercel bandwidth:** Videos served from R2 (cheaper)
+- **Better CDN support:** R2 has global edge caching
+- **No frontend blob creation:** When R2 URL is available
+
+### Neutral
+- **Upload time:** Still uploads to R2 (no change)
+- **Analysis time:** No impact on kinemotion processing
+
+### Potential Issues
+- **Presigned URL expiration:** Short-lived URLs may break if page stays open
+  - **Mitigation:** Increase expiration to 24 hours
+
+## Security Considerations
+
+### ✅ **Good**
+- UUID-based filenames prevent enumeration attacks
+- Configurable expiration for presigned URLs
+- Supports both private (presigned) and public (CDN) strategies
+
+### ⚠️ **Consider**
+- Public URLs expose video content to anyone with the link
+- **Recommendation:** Document security trade-offs in deployment guide
+- **Future:** Consider signed cookies for private video access
+
+## Integration with MVP Goals
+
+**Alignment:** ✅ Excellent
+
+- Closes Issue #12 (MVP video serving migration)
+- Reduces infrastructure costs (Vercel → R2)
+- Improves scalability for MVP validation phase
+- Maintains backward compatibility (local blob URLs still work)
+
+## Recommendations
+
+### Immediate (Before Next Deploy)
+
+1. **Add Backend Tests:**
+```python
+# backend/tests/test_r2_storage.py
+def test_get_object_url_with_public_base():
+    """Test that public URL is preferred when configured."""
+    pass
+
+def test_get_object_url_fallback_to_presigned():
+    """Test fallback to presigned URL when no public base."""
+    pass
+```
+
+2. **Update Deployment Docs:**
+   - Add `R2_PUBLIC_BASE_URL` to environment variable checklist
+   - Document presigned URL expiration trade-offs
+   - Add R2 bucket CORS configuration steps
+
+3. **Increase Presigned Expiration:**
+   - Change default from 3600s (1 hour) to 86400s (24 hours)
+   - Or make it configurable per environment
+
+### Future Enhancements
+
+1. **Pydantic Response Models:**
+   - Migrate `AnalysisResponse` to use Pydantic's `@dataclass`
+   - Leverage FastAPI's automatic validation and serialization
+
+2. **URL Refresh Mechanism:**
+   - Frontend polling to refresh expiring presigned URLs
+   - Or use public URLs in production
+
+3. **Metrics Tracking:**
+   - Track R2 bandwidth usage
+   - Monitor presigned URL expiration issues
+
+## Conclusion
+
+**Overall Assessment:** ✅ **Excellent**
+
+This is a well-executed feature addition that successfully migrates video serving from Vercel to R2. The implementation is clean, flexible, and follows best practices for cloud storage integration.
+
+**Key Strengths:**
+- Flexible URL strategy (public vs presigned)
+- UUID-based naming prevents collisions
+- Proper error handling and logging
+- Good test coverage for frontend changes
+
+**Minor Improvements Needed:**
+- Add backend tests for `get_object_url()`
+- Update deployment documentation
+- Consider increasing presigned URL expiration
+
+**Recommendation:** ✅ **Approve with minor follow-ups**
+
+The commit is ready to merge. The recommended follow-ups (backend tests, deployment docs) can be addressed in subsequent commits without blocking this feature.

kinemotion-0.49.0/.basic-memory/development/feature-request-system-configuration.md

@@ -0,0 +1,97 @@
+---
+title: Feature Request System Configuration
+type: note
+permalink: development/feature-request-system-configuration
+---
+
+# Feature Request & Feedback System Configuration
+
+## Frontend Configuration
+
+### External Links Setup
+Edit `frontend/src/config/links.ts`:
+
+```typescript
+export const EXTERNAL_LINKS = {
+  // Replace with actual Google Forms URL
+  FEATURE_REQUEST: 'https://docs.google.com/forms/d/e/YOUR_FORM_ID/viewform',
+  GITHUB_ISSUES: 'https://github.com/feniix/kinemotion/issues',
+  DOCUMENTATION: 'https://github.com/feniix/kinemotion#readme',
+} as const
+
+export const UI_CONFIG = {
+  FEATURE_REQUEST: {
+    enabled: true,
+    buttonText: 'Request Feature',
+    buttonIcon: '💡',
+    tooltip: 'Share your ideas and suggestions for improving Kinemotion',
+    openInNewTab: true,
+  },
+} as const
+```
+
+### Google Forms Setup
+
+1. Create form at [Google Forms](https://forms.google.com)
+2. Include fields:
+   - Feature category (Analysis, UI, Performance, etc.)
+   - Feature description
+   - Use case/priority
+   - Additional comments
+3. Get form URL and update FEATURE_REQUEST constant
+
+## Component Usage
+
+### FeatureRequestButton Props
+```typescript
+<FeatureRequestButton
+  variant="primary"     // or "secondary"
+  size="medium"          // "small", "medium", "large"
+  showIcon={true}
+  className="custom-class"
+/>
+```
+
+### Button Styles
+- **Primary**: Blue background (`#6366f1`), white text
+- **Secondary**: Light gray background (`#f3f4f6`), dark text
+- **Sizes**: Small (0.75rem font), Medium (0.875rem), Large (1rem font)
+
+## Database Detection
+
+The frontend automatically detects backend database connectivity:
+
+```typescript
+const { status: dbStatus, loading: dbLoading } = useDatabaseStatus()
+
+// Conditional UI rendering
+{dbStatus?.database_connected && (
+  <button>Add Coach Feedback</button>
+)}
+
+{!dbStatus?.database_connected && (
+  <span>Database Offline - Feedback Unavailable</span>
+)}
+```
+
+## Files Created
+- `frontend/src/components/FeatureRequestButton.tsx`
+- `frontend/src/components/FeedbackForm.tsx`
+- `frontend/src/hooks/useDatabaseStatus.ts`
+- `frontend/src/config/links.ts`
+
+## Integration Points
+- **ResultsDisplay**: Dedicated feedback section
+- **Health Check**: Database status in `/health` endpoint
+- **Analytics**: Google Analytics tracking included
+
+## User Flow
+1. Analysis completes → Results display
+2. "Share Your Feedback" section appears
+3. Options based on database status:
+   - Database connected: Coach feedback + feature request + report issue
+   - Database offline: Feature request + report issue + error message
+4. Feature request opens Google Forms in new tab
+5. Coach feedback saves to Supabase (if connected)
+
+This system provides comprehensive user feedback collection while maintaining a seamless experience.

kinemotion-0.49.0/.basic-memory/development/follow-up-improvements-to-r2-video-storage-migration.md

@@ -0,0 +1,240 @@
+---
+title: Follow-up Improvements to R2 Video Storage Migration
+type: note
+permalink: development/follow-up-improvements-to-r2-video-storage-migration
+---
+
+# Follow-up Improvements to R2 Video Storage Migration
+
+**Date:** December 12, 2025
+**Parent Commit:** fe74d1fc (R2 video storage migration)
+**Improvements Implemented:** Default expiration + tests + documentation
+
+---
+
+## Summary of Changes
+
+Following the comprehensive commit review, three key improvements were implemented to address identified gaps:
+
+### 1. ✅ Changed Default Presigned URL Expiration
+
+**File:** `backend/src/kinemotion_backend/app.py`
+
+**Change:** Updated default presigned URL expiration from 1 hour (3600s) to 7 days (604800s)
+
+**Rationale:**
+- 1 hour is too short for production use cases
+- 7 days is the maximum allowed by S3 API
+- Users can still access videos after long analysis sessions
+- Videos remain accessible for download/playback even after page reloads
+
+**Code Change:**
+```python
+# Before
+self.presign_expiration_s = int(os.getenv("R2_PRESIGN_EXPIRATION_S") or "3600")
+
+# After (line 118-120)
+self.presign_expiration_s = int(os.getenv("R2_PRESIGN_EXPIRATION_S") or "604800")
+```
+
+### 2. ✅ Added Backend Tests for `get_object_url()`
+
+**File:** `backend/tests/test_r2_integration.py`
+
+**New Tests Added (5 tests):**
+
+1. **`test_r2_client_initialization_with_public_url()`**
+   - Verifies `R2_PUBLIC_BASE_URL` is correctly loaded from environment
+
+2. **`test_r2_client_initialization_strips_trailing_slash_from_public_url()`**
+   - Ensures trailing slashes are stripped for consistent URL formatting
+
+3. **`test_r2_client_initialization_custom_presign_expiration()`**
+   - Tests custom `R2_PRESIGN_EXPIRATION_S` configuration
+
+4. **`test_r2_client_initialization_invalid_presign_expiration()`**
+   - Verifies fallback to 7 days when invalid expiration provided
+
+5. **`test_get_object_url_with_public_base_url()`**
+   - Tests public URL is returned when `R2_PUBLIC_BASE_URL` is configured
+
+6. **`test_get_object_url_without_public_base_url()`**
+   - Tests presigned URL fallback when no public URL configured
+
+7. **`test_get_object_url_strips_leading_slash()`**
+   - Verifies key normalization (strips leading slash)
+
+8. **`test_get_object_url_with_custom_expiration()`**
+   - Tests that custom expiration is respected in presigned URLs
+
+**Updated Tests (3 tests):**
+- Updated expiration assertions from 3600s to 604800s in:
+  - `test_r2_upload_file_success()`
+  - `test_r2_put_object_success()`
+
+**Test Results:** All 28 tests pass ✅
+
+### 3. ✅ Documented R2 Configuration
+
+#### Backend README (`backend/README.md`)
+
+**Added comprehensive R2 configuration section:**
+
+```bash
+# R2 Storage (Optional)
+R2_ENDPOINT=https://abc123.r2.cloudflarestorage.com
+R2_ACCESS_KEY=your_access_key
+R2_SECRET_KEY=your_secret_key
+R2_BUCKET_NAME=kinemotion  # Default
+
+# Optional: Public URL strategy
+R2_PUBLIC_BASE_URL=https://kinemotion-public.example.com  # Custom domain
+# Or: https://kinemotion.abc123.r2.dev  # R2.dev public URL
+
+# Optional: Presigned URL expiration (default: 604800 = 7 days)
+R2_PRESIGN_EXPIRATION_S=604800
+```
+
+**Documented URL Strategy Trade-offs:**
+- **Public URLs:** Stable, long-lived, better for production (requires public bucket or custom domain)
+- **Presigned URLs:** Temporary access, expire after N seconds, no custom domain needed
+
+#### Deployment Checklist (`docs/guides/deployment-checklist.md`)
+
+**Added new section before deployment steps:**
+
+```markdown
+## Environment Configuration
+
+### Required Environment Variables
+#### R2 Storage (for video persistence)
+[Full configuration with examples]
+
+#### Optional R2 URL Strategy
+[Public URLs vs Presigned URLs with trade-offs]
+
+#### CORS Configuration
+[CORS setup for production]
+```
+
+**Updated Cloud Run deployment command** with R2 environment variables:
+```bash
+--set-env-vars "R2_ENDPOINT=..." \
+--set-env-vars "R2_ACCESS_KEY=..." \
+--set-env-vars "R2_SECRET_KEY=..." \
+--set-env-vars "R2_BUCKET_NAME=kinemotion" \
+--set-env-vars "R2_PUBLIC_BASE_URL=https://kinemotion-public.example.com"
+```
+
+---
+
+## Impact Analysis
+
+### Configuration Changes
+- **Default expiration:** 3600s → 604800s (168x increase)
+- **Backward compatible:** Can override with `R2_PRESIGN_EXPIRATION_S` env var
+- **No breaking changes:** Existing deployments continue to work
+
+### Test Coverage
+- **Before:** 23 tests in `test_r2_integration.py`
+- **After:** 28 tests (+5 new tests, +3 updated)
+- **Coverage improvement:** New tests cover all `get_object_url()` code paths
+
+### Documentation Completeness
+- **Backend README:** Comprehensive R2 configuration guide
+- **Deployment Checklist:** Production-ready environment variable setup
+- **Developer experience:** Clear guidance on public vs presigned URL strategies
+
+---
+
+## Validation
+
+### ✅ All Tests Pass
+```bash
+cd backend && uv run pytest tests/test_r2_integration.py -v
+===== 28 passed, 2 warnings in 0.49s =====
+```
+
+### ✅ No Linting Errors
+```bash
+uv run pyright backend/src/kinemotion_backend/app.py
+uv run ruff check backend/
+# No errors
+```
+
+### ✅ Type Safety Maintained
+- All new code follows strict type hints
+- Pyright strict mode passes
+
+---
+
+## Recommendations for Next Steps
+
+### Immediate (Before Deploy)
+1. **Update environment variables in Cloud Run:**
+   ```bash
+   gcloud run services update kinemotion-backend \
+     --set-env-vars "R2_PUBLIC_BASE_URL=https://kinemotion.YOUR_ACCOUNT.r2.dev" \
+     --set-env-vars "R2_PRESIGN_EXPIRATION_S=604800"
+   ```
+
+2. **Configure R2 Bucket for Public Access:**
+   - If using public URLs, enable public read access on R2 bucket
+   - Or configure custom domain with R2
+
+### Future Enhancements
+1. **URL Refresh Mechanism:**
+   - Frontend polling to refresh expiring presigned URLs (for 7-day+ sessions)
+   - Or fully migrate to public URLs in production
+
+2. **Monitoring:**
+   - Track presigned URL expiration issues in logs
+   - Monitor R2 bandwidth usage and costs
+
+3. **Security:**
+   - Consider signed cookies for private video access (enterprise feature)
+   - Document security implications of public vs presigned URLs
+
+---
+
+## Files Changed
+
+1. **backend/src/kinemotion_backend/app.py** (1 change)
+   - Line 118-120: Default expiration 3600 → 604800
+
+2. **backend/tests/test_r2_integration.py** (+8 new/updated tests)
+   - Lines 12-28: Updated initialization test + 4 new tests
+   - Lines 103-145: Added 5 new `get_object_url()` tests
+   - Lines 286-291: Updated expiration assertion
+
+3. **backend/README.md** (2 additions)
+   - Lines 53-80: Comprehensive R2 configuration section
+   - Lines 396-409: Production environment variables
+
+4. **docs/guides/deployment-checklist.md** (2 additions)
+   - Lines 95-135: New "Environment Configuration" section
+   - Lines 121-136: Updated deployment command with R2 vars
+
+---
+
+## Conclusion
+
+All three improvements from the commit review have been successfully implemented:
+
+1. ✅ **Default expiration changed to 7 days** (maximum allowed)
+2. ✅ **Backend tests added** (8 new/updated tests, 100% pass rate)
+3. ✅ **Documentation gap filled** (README + deployment checklist)
+
+**Ready for commit:** All changes are production-ready, tested, and documented.
+
+**Suggested commit message:**
+```
+feat(backend): improve R2 storage configuration and testing
+
+- Change default presigned URL expiration from 1 hour to 7 days (max allowed)
+- Add 8 new/updated tests for R2StorageClient.get_object_url()
+- Document R2_PUBLIC_BASE_URL and R2_PRESIGN_EXPIRATION_S in README
+- Update deployment checklist with R2 environment variables
+
+Closes follow-up from commit fe74d1f review
+```

kinemotion-0.49.0/.basic-memory/development/supabase-database-integration-setup.md

@@ -0,0 +1,104 @@
+---
+title: Supabase Database Integration Setup
+type: note
+permalink: development/supabase-database-integration-setup
+---
+
+# Supabase Database Integration for Coach Feedback
+
+## Database Schema
+
+Run this SQL in Supabase SQL Editor to create the required tables:
+
+```sql
+-- Analysis Sessions Table
+CREATE TABLE IF NOT EXISTS analysis_sessions (
+    id UUID PRIMARY KEY DEFAULT uuid_generate_v4(),
+    user_id UUID NOT NULL REFERENCES auth.users(id) ON DELETE CASCADE,
+    jump_type VARCHAR(50) NOT NULL CHECK (jump_type IN ('cmj', 'drop_jump')),
+    quality_preset VARCHAR(20) NOT NULL CHECK (quality_preset IN ('fast', 'balanced', 'accurate')),
+    original_video_url TEXT,
+    debug_video_url TEXT,
+    results_json_url TEXT,
+    analysis_data JSONB NOT NULL,
+    processing_time_s FLOAT,
+    upload_id VARCHAR(50),
+    created_at TIMESTAMP WITH TIME ZONE DEFAULT NOW(),
+    updated_at TIMESTAMP WITH TIME ZONE DEFAULT NOW()
+);
+
+-- Coach Feedback Table
+CREATE TABLE IF NOT EXISTS coach_feedback (
+    id UUID PRIMARY KEY DEFAULT uuid_generate_v4(),
+    analysis_session_id UUID NOT NULL REFERENCES analysis_sessions(id) ON DELETE CASCADE,
+    coach_user_id UUID NOT NULL REFERENCES auth.users(id) ON DELETE CASCADE,
+    notes TEXT,
+    rating INTEGER CHECK (rating >= 1 AND rating <= 5),
+    tags TEXT[] DEFAULT '{}',
+    created_at TIMESTAMP WITH TIME ZONE DEFAULT NOW(),
+    updated_at TIMESTAMP WITH TIME ZONE DEFAULT NOW()
+);
+
+-- Indexes and RLS policies included in full schema
+```
+
+## Backend Integration
+
+### New Files Created:
+- `backend/src/kinemotion_backend/database.py` - Async Supabase client
+- `backend/src/kinemotion_backend/analysis_api.py` - Feedback API endpoints
+- `backend/src/kinemotion_backend/models.py` - Pydantic models
+- `backend/supabase-schema.sql` - Complete database schema
+
+### API Endpoints:
+- `POST /api/analysis/sessions` - Create analysis sessions
+- `GET /api/analysis/sessions` - List user sessions
+- `GET /api/analysis/sessions/{id}` - Get session with feedback
+- `POST /api/analysis/sessions/{id}/feedback` - Add coach feedback
+- `GET /api/analysis/database-status` - Check database connection
+
+### Key Features:
+- Optional authentication (works with/without user login)
+- JSONB storage for flexible analysis data querying
+- Row Level Security for data privacy
+- Async database operations
+- Graceful error handling
+
+## Frontend Integration
+
+### Components Added:
+- `FeedbackForm.tsx` - Modal feedback form with rating, tags, notes
+- `FeatureRequestButton.tsx` - Reusable feature request button
+- `useDatabaseStatus.ts` - Hook for checking database connectivity
+
+### Configuration:
+- `config/links.ts` - External links and UI settings
+- Google Forms integration for feature requests
+- Conditional UI based on database status
+
+### User Experience:
+- Smart feedback section with multiple options
+- Database connection detection
+- Responsive design
+- Analytics tracking ready
+
+## Setup Requirements:
+1. Run database schema in Supabase
+2. Configure SUPABASE_URL and SUPABASE_ANON_KEY
+3. Update Google Forms URL in frontend config
+4. Test integration with authenticated/unauthenticated users
+
+## Example Queries:
+```sql
+-- Average jump height by user
+SELECT user_id, AVG((analysis_data->'data'->>'jump_height')::float) as avg_jump_height
+FROM analysis_sessions WHERE jump_type = 'cmj' GROUP BY user_id;
+
+-- Sessions with feedback
+SELECT s.*, f.rating, f.notes
+FROM analysis_sessions s
+LEFT JOIN coach_feedback f ON s.id = f.analysis_session_id
+WHERE f.id IS NOT NULL;
+```
+
+This system enables comprehensive coach feedback collection and athlete performance tracking.

{kinemotion-0.47.4 → kinemotion-0.49.0}/.cursor/mcp.json

@@ -56,6 +56,19 @@
         "-y",
         "@mettamatt/code-reasoning"
       ]
+    },
+    "sonarqube": {
+      "command": "direnv",
+      "args": [
+        "exec",
+        "/Users/feniix/src/personal/cursor/kinemotion",
+        "java",
+        "-jar",
+        "/tmp/sonarqube-mcp-server-1.4.0.1666.jar"
+      ],
+      "env": {
+        "STORAGE_PATH": "/tmp/sonarqube"
+      }
     }
   }
 }