kinemotion 0.51.0__tar.gz → 0.52.0__tar.gz

kinemotion-0.52.0/.basic-memory/development/backend-granular-logging-implementation.md

@@ -0,0 +1,153 @@
+---
+title: Backend Granular Logging Implementation
+type: note
+permalink: development/backend-granular-logging-implementation
+---
+
+# Backend Granular Logging Implementation
+
+## Complete Logging Coverage
+
+All logging has been restored and enhanced with comprehensive instrumentation throughout the video analysis pipeline.
+
+## Logging Stages by Category
+
+### 1. Endpoint Level (routes/analysis.py) - 2 stages
+- **`analyzing_video_started`** - Before analysis begins (includes jump_type, quality, debug flags)
+- **`analyzing_video_completed`** - After analysis finishes (includes duration_ms, status_code)
+
+### 2. Validation & Setup (analysis_service.py) - 5 stages
+- **`validating_video_file`** - Starting video file validation
+- **`validating_video_file_completed`** - File validation complete (duration_ms)
+- **`validating_jump_type`** - Starting jump type validation (includes jump_type)
+- **`validating_jump_type_completed`** - Jump type validation complete (normalized_jump_type, duration_ms)
+- **`generating_storage_key`** - Starting storage key generation (includes filename)
+- **`generating_storage_key_completed`** - Key generation complete (storage_key, duration_ms)
+
+### 3. File Operations (analysis_service.py) - 2 stages
+- **`saving_uploaded_file`** - Starting file save to temp disk (temp_path)
+- **`saving_uploaded_file_completed`** - File save complete (file_size_mb, duration_ms)
+
+### 4. Video Processing Pipeline (analysis_service.py) - 2 main + N granular
+- **`video_processing_started`** - Starting core kinemotion analysis
+- **Individual pipeline stages** (via timer.get_metrics()) - Each stage from kinemotion library:
+  - `frame_read` - Reading video frames
+  - `frame_rotation` - Rotating frames for correct orientation
+  - `frame_conversion` - Converting frame formats
+  - `mediapipe_inference` - MediaPipe pose detection (DOMINANT: 88-94% of time)
+  - `landmark_extraction` - Extracting pose landmarks
+  - `smoothing_outlier_rejection` - Outlier rejection in smoothing
+  - `smoothing_bilateral` - Bilateral filter smoothing
+  - `smoothing_savgol` - Savitzky-Golay smoothing
+  - `parameter_auto_tuning` - Auto-tuning algorithm parameters
+  - `vertical_position_extraction` - Extracting Y-axis positions
+  - `phase_detection` - Detecting jump phases (CMJ-specific)
+  - `dj_detect_drop_start` - Detecting drop start (Drop Jump-specific)
+  - `dj_find_phases` - Finding drop jump phases
+  - `dj_identify_contact` - Ground contact detection
+  - `dj_analyze_flight` - Analyzing flight phase
+  - `dj_compute_velocity` - Computing velocity
+  - `dj_find_contact_frames` - Finding contact frames
+  - `cmj_compute_derivatives` - Computing derivatives
+  - `cmj_find_takeoff` - Finding takeoff point
+  - `cmj_find_lowest_point` - Finding lowest point
+  - `cmj_find_landing` - Finding landing point
+  - `cmj_find_standing_end` - Finding standing phase end
+  - `metrics_calculation` - Computing final metrics
+  - `quality_assessment` - Quality assessment
+  - `debug_video_generation` - Generating debug overlay video
+  - `debug_video_resize` - Resizing debug video frames
+  - `debug_video_write` - Writing debug video to disk
+  - `debug_video_copy` - Copying frames for debug video
+  - `debug_video_draw` - Drawing annotations on debug video
+- **`video_processing_completed`** - Processing complete (total_duration_s, duration_ms)
+
+### 5. Cloud Storage Operations (analysis_service.py) - 5 stages
+- **`uploading_original_video`** - Starting upload (storage_key)
+- **`original_video_uploaded`** - Upload complete (url)
+- **`uploading_analysis_results`** - Starting results JSON upload (storage_key)
+- **`r2_results_upload`** - Results upload complete (duration_ms, url, key)
+- **`uploading_debug_video`** - Starting debug video upload (storage_key)
+- **`r2_debug_video_upload`** - Debug video upload complete (duration_ms, url, key)
+- **`debug_video_empty_skipping_upload`** - Debug video empty, skipping upload
+
+### 6. Response Building & Cleanup (analysis_service.py) - 3 stages
+- **`response_serialization`** - Converting metrics to JSON response (duration_ms)
+- **`cleaning_up_temporary_files`** - Starting cleanup
+- **`temp_file_cleanup`** - Cleanup complete (duration_ms)
+
+### 7. Success Summary (analysis_service.py) - 1 stage
+- **`video_analysis_completed`** - Final summary (jump_type, duration_ms, metrics_count)
+
+### 8. Error Handling (routes/analysis.py + analysis_service.py) - 2 stages
+- **`analyze_endpoint_validation_error`** - (WARNING) Validation failed at endpoint
+- **`analyze_endpoint_error`** - (ERROR) Unexpected error at endpoint
+- **`video_analysis_validation_error`** - (ERROR) Validation failed in service
+- **`video_analysis_failed`** - (ERROR) Processing failed in service
+
+## Total Logging Coverage
+
+**Total granular logging events: 30+ individual events** including:
+- 2 endpoint-level events
+- 5 validation/setup events
+- 2 file operation events
+- 2 + N processing pipeline events (N = individual kinemotion stages)
+- 5 cloud storage events
+- 3 response/cleanup events
+- 1 success summary event
+- 4 error events
+
+## Key Features
+
+✅ **End-to-End Visibility** - Every major operation emits events
+✅ **Detailed Timing** - All operations include duration_ms where relevant
+✅ **Contextual Data** - Each log includes relevant identifiers (storage_key, filename, urls, etc.)
+✅ **Bottleneck Identification** - MediaPipe pose tracking consistently shows 88-94% of total time
+✅ **Test Coverage** - All 89 backend tests pass with instrumentation
+✅ **Code Quality** - Passes ruff linting and pyright type checking
+✅ **Request Tracing** - Can follow requests through entire pipeline via upload_id in error cases
+✅ **Performance Metrics** - Granular timing for optimization decisions
+
+## Log Output Examples
+
+### Successful Analysis Flow
+```
+analyzing_video_started jump_type=cmj quality=balanced debug=false
+validating_video_file
+validating_video_file_completed duration_ms=1.2
+validating_jump_type jump_type=cmj
+validating_jump_type_completed normalized_jump_type=cmj duration_ms=0.5
+generating_storage_key filename=video.mp4
+generating_storage_key_completed storage_key=abc123 duration_ms=0.8
+saving_uploaded_file temp_path=/tmp/video.mp4
+saving_uploaded_file_completed file_size_mb=45.23 duration_ms=250.4
+video_processing_started
+frame_read duration_ms=324.5
+mediapipe_inference duration_ms=5183.1
+smoothing_bilateral duration_ms=188.6
+metrics_calculation duration_ms=0.2
+video_processing_completed total_duration_s=5.72 duration_ms=5721.4
+uploading_original_video storage_key=abc123
+original_video_uploaded url=https://r2.example.com/videos/abc123.mp4
+uploading_analysis_results storage_key=abc123
+r2_results_upload duration_ms=234.0 url=https://r2.example.com/results/abc123.json key=results/abc123.json
+response_serialization duration_ms=1.5
+cleaning_up_temporary_files
+temp_file_cleanup duration_ms=45.2
+video_analysis_completed jump_type=cmj duration_ms=5721.4 metrics_count=12
+analyzing_video_completed duration_ms=6200.5 status_code=200
+```
+
+## Implementation Files
+
+- **routes/analysis.py** - Endpoint-level logging (analyzing_video_started/completed)
+- **services/analysis_service.py** - Core pipeline logging (validation, processing, storage, cleanup)
+- **services/video_processor.py** - Video processor logging wrapper
+
+## Testing
+
+✅ All 89 backend tests pass
+✅ All 16 API endpoint tests pass
+✅ Ruff linting: All checks passed
+✅ Pyright type checking: 0 errors
+✅ No breaking changes to existing functionality

{kinemotion-0.51.0 → kinemotion-0.52.0}/.github/workflows/deploy-backend.yml

@@ -7,20 +7,12 @@ on:
       - 'backend/**'
       - 'src/kinemotion/**'
       - '.github/workflows/deploy-backend.yml'
-      - '!frontend/**'
-      - '!docs/**'
-      - '!*.md' # ignore markdown files
-      - '!scripts/**'
-      - '!tests/**'
-      - '!.serena/**'
-      - '!.basic-memory/**'
-      - '!.cursor/**'
-      - '!.vscode/**'
-      - '!.gitignore'
-
-concurrency:
-  group: release-all
-  cancel-in-progress: false
+  workflow_dispatch:
+    inputs:
+      reason:
+        description: 'Reason for manual deployment'
+        required: false
+        type: string
 
 env:
   GCP_PROJECT_ID: kinemotion-backend
@@ -76,62 +68,9 @@ jobs:
           cd backend
           uv run ruff check .
 
-  release:
-    name: Create Release
-    needs: test
-    runs-on: ubuntu-latest
-    permissions:
-      contents: write
-    outputs:
-      version: ${{ steps.get_version.outputs.version }}
-      released: ${{ steps.get_version.outputs.released }}
-    steps:
-      - uses: actions/checkout@v6
-
-      - name: Set up Python
-        uses: actions/setup-python@v6
-        with:
-          python-version: '3.12'
-
-      - name: Install uv
-        run: pip install uv
-
-      - name: Install backend dependencies
-        run: |
-          cd backend
-          uv sync
-
-      - name: Create Release
-        env:
-          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        run: |
-          git config user.name "github-actions"
-          git config user.email "github-actions@github.com"
-          cd backend
-          uv run semantic-release version
-
-      - name: Determine version and release status
-        id: get_version
-        run: |
-          # Check if semantic-release created a new release (tagged commit)
-          if git describe --exact-match --tags HEAD 2>/dev/null; then
-            # Semantic release was created - use version from pyproject.toml
-            VERSION=$(grep '^version = ' backend/pyproject.toml | cut -d'"' -f2)
-            RELEASED="true"
-          else
-            # No semantic release - use fallback version (dev-{short_sha})
-            SHORT_SHA=$(git rev-parse --short HEAD)
-            VERSION="dev-${SHORT_SHA}"
-            RELEASED="false"
-          fi
-
-          echo "version=${VERSION}" >> $GITHUB_OUTPUT
-          echo "released=${RELEASED}" >> $GITHUB_OUTPUT
-          echo "Version: ${VERSION} (released: ${RELEASED})"
-
   build:
     name: Build and Push Docker Image to GCR
-    needs: release
+    needs: test
     runs-on: ubuntu-latest
     permissions:
       contents: read
@@ -160,7 +99,7 @@ jobs:
           context: .
           file: ./backend/Dockerfile
           push: true
-          tags: ${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:${{ needs.release.outputs.version }},${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:${{ github.sha }},${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:latest
+          tags: ${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:${{ github.sha }},${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:latest
           cache-from: type=registry,ref=${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:buildcache
           cache-to: type=registry,ref=${{ env.REGISTRY }}/${{ env.GCP_PROJECT_ID }}/${{ env.SERVICE_NAME }}:buildcache,mode=max
 

{kinemotion-0.51.0 → kinemotion-0.52.0}/.github/workflows/release.yml

@@ -8,20 +8,6 @@ on:
       - '.github/workflows/deploy-backend.yml'
       - 'backend/**'
       - 'frontend/**'
-      - '*.md' # ignore markdown files
-      - 'scripts/**'
-      - '.serena/**'
-      - '.basic-memory/**'
-      - '.cursor/**'
-      - '.vscode/**'
-      - '.gitignore'
-      - '.claude/**'
-      - '.gemini/**'
-      - '.mcp/**'
-
-concurrency:
-  group: release-all
-  cancel-in-progress: false
 
 jobs:
   test:

{kinemotion-0.51.0 → kinemotion-0.52.0}/.pre-commit-config.yaml

@@ -38,7 +38,7 @@ repos:
         additional_dependencies:
           - mdformat-gfm>=0.3.5  # GitHub Flavored Markdown
           - mdformat-tables  # Table formatting
-        exclude: (^CLAUDE\.md$|^CHANGELOG\.md$|^backend/CHANGELOG\.md$|^presentation/revealjs/slides\.md$|\.claude/agents/.*\.md$|\.basic-memory/.*\.md$|\.serena/.*\.md$)
+        exclude: (^CLAUDE\.md$|^CHANGELOG\.md$|^presentation/revealjs/slides\.md$|\.claude/agents/.*\.md$|\.basic-memory/.*\.md$|\.serena/.*\.md$)
 
   - repo: https://github.com/compilerla/conventional-pre-commit
     rev: v4.3.0

{kinemotion-0.51.0 → kinemotion-0.52.0}/CHANGELOG.md

@@ -7,63 +7,39 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 <!-- version list -->
 
-## v0.51.0 (2025-12-13)
+## v0.52.0 (2025-12-14)
 
-### Bug Fixes
-
-- Correct MetricsData instantiation and metrics_count calculation
-  ([`e2ebca1`](https://github.com/feniix/kinemotion/commit/e2ebca1d5614faa86cdb3c4e23ef9fa034cd8d2a))
-
-- Correct uv.lock path in backend semantic-release build command
-  ([`650c3bc`](https://github.com/feniix/kinemotion/commit/650c3bc7b1a18814f179e7f1e4835000186ee45c))
-
-- Remove invalid search_tag_format and fix tag_format syntax
-  ([`46c2a2a`](https://github.com/feniix/kinemotion/commit/46c2a2af5fa97a833256896253b300c307d3a528))
-
-- Update error handling tests to use 'status' field instead of 'status_code'
-  ([`890e54b`](https://github.com/feniix/kinemotion/commit/890e54b6ac7bead9dceb0f34fdaeb4789e3b3dbd))
-
-- **docs**: Enhance api module docstring with examples and analysis types
-  ([`94e32f1`](https://github.com/feniix/kinemotion/commit/94e32f1061505747c0345e8683a61bcf8dc60ee3))
-
-- **docs**: Improve app.py module docstring with comprehensive feature overview
-  ([`eab977c`](https://github.com/feniix/kinemotion/commit/eab977c237f2318f5dc0b94a32f4b736053bfc00))
+### Features
 
-- **docs**: Improve module docstrings in src/kinemotion and backend/src/
-  ([`f59d649`](https://github.com/feniix/kinemotion/commit/f59d6494c8735fa3e83f8bdbedefce0b6c405cf5))
+- Bump version
+  ([`6f0f055`](https://github.com/feniix/kinemotion/commit/6f0f055bd806f947371467dfa3b60720fddbff39))
 
-### Continuous Integration
 
-- Add semantic-release configuration to backend
-  ([`f8e447c`](https://github.com/feniix/kinemotion/commit/f8e447c86b03d7ad6cf9e40d67217371f642f254))
+## v0.51.0 (2025-12-14)
 
-- Improve workflow configuration and add backend versioning
-  ([`8d86933`](https://github.com/feniix/kinemotion/commit/8d8693398b6aec20c847ae0616488dd3b90c35af))
+### Chores
 
-- Use shared concurrency group to prevent race conditions between CLI and backend release workflows
-  ([`038e744`](https://github.com/feniix/kinemotion/commit/038e744ed13aa52c78557df8257995861de5308e))
+- Sync version with PyPI 0.51.0
+  ([`13a3806`](https://github.com/feniix/kinemotion/commit/13a380695f9499935c05f3088d7d409df6b7ccf3))
 
 ### Features
 
-- Add comprehensive granular logging for pipeline stage timing
-  ([`0489222`](https://github.com/feniix/kinemotion/commit/04892221faf2cb39f355867475242b8baafa34cd))
+- Bump version
+  ([`751e190`](https://github.com/feniix/kinemotion/commit/751e1908768fee2cf193d79c672174546955c445))
 
-- Implement continuous deployment with optional semantic releases
-  ([`3596e47`](https://github.com/feniix/kinemotion/commit/3596e47e055dd54dda130bba9d7e78c14c4b2819))
+- Sync versions
+  ([`03e6413`](https://github.com/feniix/kinemotion/commit/03e6413388142e3b04d535f9c07d653d5f73144e))
 
-### Refactoring
 
-- Decouple semantic-release configs with directory-specific filtering
-  ([`b7d4fef`](https://github.com/feniix/kinemotion/commit/b7d4fefcff2972d2d347dfef69a830b1f2473cfc))
+## v0.50.1 (2025-12-14)
 
-- Namespace semantic-release tags and reset backend to 0.1.0
-  ([`76d9c5b`](https://github.com/feniix/kinemotion/commit/76d9c5b4cedebfd56c5e5ec5272e260620520cd3))
+### Bug Fixes
 
-- Restore search_tag_format for decoupled version detection
-  ([`e9f163d`](https://github.com/feniix/kinemotion/commit/e9f163dd85aa81b0dce52d26339bcc858e9b6ef5))
+- Fix version
+  ([`e030ed2`](https://github.com/feniix/kinemotion/commit/e030ed2b76315bfcfa21ae61e0d65d99b7e8ba49))
 
-- Separate release job in backend deployment workflow
-  ([`521f03f`](https://github.com/feniix/kinemotion/commit/521f03f4d32c6bcf0b048c2fba3d54d74c7c0677))
+- **backend**: Use configured logger from logging_config
+  ([`e38e3bf`](https://github.com/feniix/kinemotion/commit/e38e3bff973ac47efa9c5735a7e45fead2b219f8))
 
 
 ## v0.50.0 (2025-12-13)

{kinemotion-0.51.0 → kinemotion-0.52.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinemotion
-Version: 0.51.0
+Version: 0.52.0
 Summary: Video-based kinematic analysis for athletic performance
 Project-URL: Homepage, https://github.com/feniix/kinemotion
 Project-URL: Repository, https://github.com/feniix/kinemotion

{kinemotion-0.51.0 → kinemotion-0.52.0}/backend/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kinemotion-backend"
-version = "0.1.1"
+version = "0.2.0"
 description = "FastAPI backend for Kinemotion video analysis"
 readme = "README.md"
 requires-python = ">=3.10,<3.13"
@@ -59,7 +59,6 @@ dev = [
     "httpx>=0.26.0",
     "ruff>=0.1.8",
     "pyright>=1.1.380",
-    "python-semantic-release>=9.8.2",
 ]
 
 [tool.pytest.ini_options]
@@ -154,32 +153,3 @@ reportUnusedClass = "warning"
 reportUnusedFunction = "warning"
 reportUnusedVariable = "warning"
 reportDuplicateImport = "warning"
-
-[tool.semantic_release]
-version_toml = ["pyproject.toml:project.version"]
-branch = "main"
-changelog_file = "CHANGELOG.md"
-tag_format = "kinemotion-backend-v{version}"
-build_command = """\
-    uv lock --upgrade-package "$PACKAGE_NAME"
-    git add ../uv.lock
-    uv build
-"""
-dist_path = "dist/"
-upload_to_vcs_release = true
-remove_dist = false
-patch_without_tag = false
-major_on_zero = false
-allow_zero_version = true
-commit_message = "chore(release): {version} [skip ci]"
-
-[tool.semantic_release.commit_parser_options]
-allowed_tags = ["feat", "fix", "docs", "style", "refactor", "perf", "test", "chore", "build", "ci"]
-minor_tags = ["feat"]
-patch_tags = ["fix", "perf"]
-
-[tool.semantic_release.publish]
-# Only process commits affecting backend/src/kinemotion_backend/ for backend releases
-# Tests (backend/tests/) are excluded since they are not part of the public API
-# This decouples backend versioning from root CLI versioning
-directories = ["backend/src/kinemotion_backend"]

{kinemotion-0.51.0 → kinemotion-0.52.0}/backend/src/kinemotion_backend/analysis_api.py

@@ -1,22 +1,4 @@
-"""API endpoints for analysis sessions and coach feedback.
-
-This module provides FastAPI endpoints for managing video analysis sessions and
-coach feedback. Key responsibilities:
-
-- Create and retrieve analysis sessions for uploaded videos
-- Process video analysis requests through the kinemotion engine
-- Store and retrieve coach feedback linked to analysis sessions
-- Handle authentication via Supabase JWT tokens
-- Manage session state and error handling
-
-Endpoints:
-    POST /api/analysis/sessions - Create a new analysis session
-    GET /api/analysis/sessions/{session_id} - Retrieve session details
-    POST /api/analysis/sessions/{session_id}/feedback - Add coach feedback
-    GET /api/analysis/sessions/{session_id}/feedback - Retrieve feedback
-
-All endpoints require JWT authentication via Authorization header.
-"""
+"""API endpoints for analysis sessions and coach feedback."""
 
 from typing import Any
 

{kinemotion-0.51.0 → kinemotion-0.52.0}/backend/src/kinemotion_backend/app.py

@@ -1,18 +1,6 @@
 """FastAPI application for Kinemotion video analysis backend.
 
-This module provides a REST API for analyzing jump mechanics from video files.
-It handles video uploads, processes them through the kinemotion analysis engine
-(drop jump and counter-movement jump analysis), stores results in Supabase, and
-manages video/result artifacts in Cloudflare R2 storage.
-
-Key features:
-- Video upload and processing for multiple jump types
-- Integration with MediaPipe pose tracking
-- Metrics calculation (RSI, flight time, jump height, etc.)
-- Result persistence in Supabase database
-- Artifact management in Cloudflare R2 storage
-- Rate limiting and request logging middleware
-- CORS support for frontend integration
+Real metrics integration with Cloudflare R2 storage for video and results management.
 """
 
 import os
@@ -90,7 +78,7 @@ class AnalysisResponse:
     def to_dict(self) -> dict[str, Any]:
         """Convert response to JSON-serializable dictionary."""
         result: dict[str, Any] = {
-            "status": self.status_code,
+            "status_code": self.status_code,
             "message": self.message,
             "processing_time_s": self.processing_time_s,
         }
@@ -1157,7 +1145,7 @@ async def http_exception_handler(request: Any, exc: HTTPException) -> JSONRespon
     return JSONResponse(
         status_code=exc.status_code,
         content={
-            "status": exc.status_code,
+            "status_code": exc.status_code,
             "message": "HTTP Error",
             "error": exc.detail,
         },
@@ -1177,7 +1165,7 @@ async def general_exception_handler(request: Any, exc: Exception) -> JSONRespons
     return JSONResponse(
         status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
         content={
-            "status": 500,
+            "status_code": 500,
             "message": "Internal server error",
             "error": error_detail,
         },

{kinemotion-0.51.0 → kinemotion-0.52.0}/backend/src/kinemotion_backend/models/responses.py

@@ -37,7 +37,7 @@ class MetricsData(BaseModel):
 class AnalysisResponse(BaseModel):
     """Response structure for video analysis results."""
 
-    status: int = Field(..., description="HTTP status code")
+    status_code: int = Field(..., description="HTTP status code")
     message: str = Field(..., description="Response message")
     metrics: MetricsData | None = Field(None, description="Analysis metrics")
     results_url: str | None = Field(None, description="URL to analysis results")

{kinemotion-0.51.0 → kinemotion-0.52.0}/backend/src/kinemotion_backend/routes/analysis.py

@@ -1,6 +1,7 @@
 """Video analysis routes for kinemotion backend."""
 
-import structlog
+import time
+
 from fastapi import APIRouter, File, Form, Header, Request, UploadFile
 from fastapi.responses import JSONResponse
 
@@ -12,11 +13,12 @@ except ImportError:
     fastapi_limiter_available = False
     RateLimiter = None  # type: ignore[assignment]
 
+from ..logging_config import get_logger
 from ..models.responses import AnalysisResponse
 from ..services import AnalysisService, validate_referer
 from ..utils import NoOpLimiter
 
-logger = structlog.get_logger()
+logger = get_logger(__name__)
 router = APIRouter(prefix="/api", tags=["Analysis"])
 
 # Rate limiting (use NoOpLimiter for testing or when fastapi_limiter unavailable)
@@ -56,8 +58,6 @@ async def analyze_video(
     Raises:
         HTTPException: If validation or processing fails
     """
-    import time
-
     start_time = time.time()
 
     # Validate referer (prevent direct API access)
@@ -70,12 +70,12 @@ async def analyze_video(
         # Convert debug string to boolean
         enable_debug = debug.lower() == "true"
 
+        # Log analysis start
         logger.info(
-            "analyze_endpoint_processing",
+            "analyzing_video_started",
             jump_type=jump_type,
             quality=quality,
             debug=enable_debug,
-            filename=file.filename,
         )
 
         # Perform analysis using service layer
@@ -87,13 +87,12 @@ async def analyze_video(
             user_id=None,  # TODO: Extract from auth when available
         )
 
-        # Log successful response
-        elapsed = time.time() - start_time
+        # Log analysis completion
+        analysis_duration = time.time() - start_time
         logger.info(
-            "analyze_endpoint_success",
-            status_code=result.status,
-            processing_time_s=round(elapsed, 2),
-            has_metrics=result.metrics is not None,
+            "analyzing_video_completed",
+            duration_ms=round(analysis_duration * 1000, 1),
+            status_code=result.status_code,
         )
 
         # Return JSON response
@@ -110,7 +109,7 @@ async def analyze_video(
 
         # Return validation error
         error_result = AnalysisResponse(
-            status=422,
+            status_code=422,
             message=str(e),
             error="validation_error",
             metrics=None,
@@ -138,7 +137,7 @@ async def analyze_video(
 
         # Return generic server error
         error_result = AnalysisResponse(
-            status=500,
+            status_code=500,
             message="Internal server error during analysis",
             error=str(e),
             metrics=None,