kinemotion 0.47.1__tar.gz → 0.47.3__tar.gz

{kinemotion-0.47.1 → kinemotion-0.47.3}/.claude/settings.local.json

@@ -65,7 +65,9 @@
       "WebFetch(domain:realpython.com)",
       "WebFetch(domain:sonarcloud.io)",
       "WebFetch(domain:www.pyopensci.org)",
-      "WebFetch(domain:www.sonarsource.com)"
+      "WebFetch(domain:www.sonarsource.com)",
+      "mcp__sonarqube__search_sonar_issues_in_projects",
+      "mcp__sonarqube__show_rule"
     ]
   },
   "enableAllProjectMcpServers": true

{kinemotion-0.47.1 → kinemotion-0.47.3}/.mcp.json

@@ -27,6 +27,19 @@
         "BASIC_MEMORY_KEBAB_FILENAMES": "true"
       }
     },
+    "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"
+      }
+    },
     "sequential-thinking": {
       "command": "uvx",
       "args": [

{kinemotion-0.47.1 → kinemotion-0.47.3}/CHANGELOG.md

@@ -7,6 +7,37 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 <!-- version list -->
 
+## v0.47.3 (2025-12-12)
+
+### Bug Fixes
+
+- **api**: Remove unused timer parameter from _assess_dropjump_quality
+  ([`31ac8ef`](https://github.com/feniix/kinemotion/commit/31ac8efc8b474383f9de72eccc50ea02d408eb86))
+
+### Refactoring
+
+- **api**: Split large api.py into modular architecture
+  ([`6eb3f06`](https://github.com/feniix/kinemotion/commit/6eb3f06ab52534e7f97ad85d6e9a361b657cabb8))
+
+
+## v0.47.2 (2025-12-12)
+
+### Bug Fixes
+
+- **telemetry**: Use structlog for robust error handling
+  ([`0c66c5f`](https://github.com/feniix/kinemotion/commit/0c66c5f486c4f034e5833ee0016d37fc31e9240d))
+
+### Documentation
+
+- Update CLAUDE.md with test count and coverage improvements
+  ([`1132324`](https://github.com/feniix/kinemotion/commit/1132324fe9bfe907bc6850397fd5370668b1bfa6))
+
+### Refactoring
+
+- Reduce cognitive complexity across validation, kinematics, and API
+  ([`04a88cb`](https://github.com/feniix/kinemotion/commit/04a88cbcec3b52bf1767e9647627b839ca842ecf))
+
+
 ## v0.47.1 (2025-12-12)
 
 ### Bug Fixes

{kinemotion-0.47.1 → kinemotion-0.47.3}/CLAUDE.md

@@ -45,7 +45,7 @@ uv run pytest --cov-report=html         # Generate HTML coverage report
 uv run ruff check --fix && uv run pyright  # Lint + type check
 ```
 
-**Note**: Test count reflects 207 test functions across 24 test files. Parameterized tests generate additional test instances at runtime. Total coverage: 78.67%.
+**Note**: Test count reflects 620 passing tests across 24 test files. Parameterized tests generate additional test instances at runtime. Total coverage: 80.86%.
 
 **Coverage Reports:**
 
@@ -98,7 +98,7 @@ src/kinemotion/
 │   └── validation_bounds.py    # Drop jump bounds (DropJumpBounds)
 └── [other modules]
 
-tests/                      # 519 tests (comprehensive coverage across all modules)
+tests/                      # 620 tests (comprehensive coverage across all modules)
 ├── conftest.py             # Shared fixtures (cli_runner, minimal_video, sample_video_path)
 ├── core/                   # Core module tests (9 files)
 ├── dropjump/               # Drop jump tests (6 files)
@@ -128,7 +128,7 @@ The project has evolved into a complete platform with three main components:
 │   ├── cli.py           # Main CLI commands
 │   ├── api.py           # Python API (used by backend)
 │   └── [modules]        # Core, dropjump, cmj
-└── tests/               # 519 comprehensive tests (78.67% coverage)
+└── tests/               # 620 comprehensive tests (80.86% coverage)
 ```
 
 **Data Flow:**
@@ -186,27 +186,29 @@ uv run pytest             # All tests (207 test functions)
 - Ruff (88 char lines, both linting and formatting)
 - Conventional Commits (see below)
 - **Code duplication target: < 3%**
-- **Test coverage: ≥ 50% (current: 78.67% with branch coverage)**
-- **Test count: 207 test functions across 24 test files**
+- **Test coverage: ≥ 50% (current: 80.86% with branch coverage)**
+- **Test count: 620 tests across 24 test files**
+- **SonarQube maintainability: 86% of issues resolved (6/7)**
 
 ### Coverage Summary
 
-**Current:** 78.67% (207 test functions, 2966 statements, 1056 branches)
+**Current:** 80.86% (620 tests, 3296 statements, 1088 branches)
 
 **Coverage by tier:**
 
 - Core algorithms: 89-100% ✅ (smoothing: 100%, filtering: 89%, analysis: 88-94%)
-- API/Integration: 72% ✅ (api.py)
-- CLI modules: 76-87% ✅ (dropjump: 87.17%, cmj: 76.12%)
+- API/Integration: 66% ✅ (api.py - improved with helper extraction)
+- CLI modules: 90-91% ✅ (dropjump: 90.24%, cmj: 91.26%)
 - Validation: 80-100% ✅ (validators, bounds)
-- Visualization: 10-36% ✅ (debug overlays - appropriate)
+- Kinematics: 92-93% ✅ (cmj: 92.25%, dropjump: 93.03%)
+- Visualization: 10-40% ✅ (debug overlays - appropriate)
 
 **Key metrics:**
 
-- All 519 tests pass
-- 0 type errors (pyright strict)
-- 0 linting errors (ruff)
-- 100% coverage on critical modules (smoothing, formatting, validation_bounds)
+- All 620 tests pass ✅
+- 0 type errors (pyright strict) ✅
+- 0 linting errors (ruff) ✅
+- 100% coverage on critical modules (smoothing, formatting, validation_bounds) ✅
 
 **Test Organization:**
 
@@ -227,6 +229,8 @@ View HTML report: `uv run pytest --cov-report=html && open htmlcov/index.html`
 ### Code Quality
 
 - **Duplication target:** < 3% (current: 2.96%)
+- **SonarQube maintainability:** 86% issues resolved
+- **Cognitive complexity:** Target ≤15 per function
 - **Check:** `npx jscpd src/kinemotion`
 
 **Principles:**
@@ -235,6 +239,13 @@ View HTML report: `uv run pytest --cov-report=html && open htmlcov/index.html`
 2. Use inheritance for shared behavior
 3. Create helper functions (testable, reusable)
 4. Use function composition (pass functions as parameters)
+5. Apply Extract Method pattern for complex functions (see recent refactoring)
+
+**Maintainability Patterns:**
+- Extract Method: Break down functions with complexity >15
+- Parameter Object: Bundle related parameters into dataclasses
+- Early Return: Reduce nesting by handling edge cases first
+- Helper Functions: Single Responsibility Principle for all helpers
 
 See [Testing Guide](docs/development/testing.md) for detailed duplication avoidance strategies.
 

{kinemotion-0.47.1 → kinemotion-0.47.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinemotion
-Version: 0.47.1
+Version: 0.47.3
 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.47.1 → kinemotion-0.47.3}/backend/pyproject.toml

@@ -30,12 +30,6 @@ dependencies = [
     "supabase>=2.10.0",
     "pyjwt>=2.8.0",
     "httpx>=0.27.0",
-    "opentelemetry-api>=1.20.0",
-    "opentelemetry-sdk>=1.20.0",
-    "opentelemetry-exporter-otlp>=1.20.0",
-    "opentelemetry-exporter-gcp-trace>=1.6.0",
-    "opentelemetry-instrumentation-fastapi>=0.40b0",
-    "opentelemetry-instrumentation-logging>=0.40b0",
     # Local development: use editable install via `uv sync`
     # Production deployment: use released package from PyPI
     "kinemotion>=0.37.0",

{kinemotion-0.47.1 → kinemotion-0.47.3}/backend/src/kinemotion_backend/app.py

@@ -27,8 +27,6 @@ from fastapi.responses import JSONResponse
 from kinemotion.api import process_cmj_video, process_dropjump_video
 from kinemotion.core.pose import PoseTracker
 from kinemotion.core.timing import (
-    CompositeTimer,
-    OpenTelemetryTimer,
     PerformanceTimer,
     Timer,
 )
@@ -37,7 +35,6 @@ from slowapi.util import get_remote_address
 
 from kinemotion_backend.logging_config import get_logger, setup_logging
 from kinemotion_backend.middleware import RequestLoggingMiddleware
-from kinemotion_backend.telemetry import setup_telemetry
 
 # Initialize structured logging
 setup_logging(
@@ -223,9 +220,6 @@ global_pose_trackers: dict[str, PoseTracker] = {}
 @asynccontextmanager
 async def lifespan(app: FastAPI):
     """Manage application lifecycle and global resources."""
-    # Initialize telemetry
-    app.state.telemetry_enabled = setup_telemetry(app)
-
     logger.info("initializing_pose_trackers")
     try:
         # Initialize trackers for each quality preset
@@ -614,12 +608,8 @@ async def analyze_video(
         # Process video with real kinemotion analysis
         analysis_start = time.time()
 
-        # Initialize timer (with optional telemetry)
-        base_timer = PerformanceTimer()
-        if getattr(request.app.state, "telemetry_enabled", False):
-            timer = CompositeTimer([base_timer, OpenTelemetryTimer()])
-        else:
-            timer = base_timer
+        # Initialize timer
+        timer = PerformanceTimer()
 
         # Select appropriate pre-initialized tracker from pool
         # Default to balanced if quality not found or trackers not initialized
@@ -866,12 +856,8 @@ async def analyze_local_video(
             )
 
         # Process video
-        # Initialize timer (with optional telemetry)
-        base_timer = PerformanceTimer()
-        if getattr(request.app.state, "telemetry_enabled", False):
-            timer = CompositeTimer([base_timer, OpenTelemetryTimer()])
-        else:
-            timer = base_timer
+        # Initialize timer
+        timer = PerformanceTimer()
 
         # Select appropriate pre-initialized tracker from pool
         tracker_key = quality.lower()

{kinemotion-0.47.1 → kinemotion-0.47.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kinemotion"
-version = "0.47.1"
+version = "0.47.3"
 description = "Video-based kinematic analysis for athletic performance"
 readme = "README.md"
 requires-python = ">=3.10,<3.13"

kinemotion-0.47.3/src/kinemotion/api.py

@@ -0,0 +1,38 @@
+"""Public API for programmatic use of kinemotion analysis.
+
+This module provides a unified interface for both drop jump and CMJ video analysis.
+The actual implementations have been moved to their respective submodules:
+- Drop jump: kinemotion.dropjump.api
+- CMJ: kinemotion.cmj.api
+"""
+
+# CMJ API
+from .cmj.api import (
+    CMJVideoConfig,
+    CMJVideoResult,
+    process_cmj_video,
+    process_cmj_videos_bulk,
+)
+from .cmj.kinematics import CMJMetrics
+
+# Drop jump API
+from .dropjump.api import (
+    DropJumpVideoConfig,
+    DropJumpVideoResult,
+    process_dropjump_video,
+    process_dropjump_videos_bulk,
+)
+
+__all__ = [
+    # Drop jump
+    "DropJumpVideoConfig",
+    "DropJumpVideoResult",
+    "process_dropjump_video",
+    "process_dropjump_videos_bulk",
+    # CMJ
+    "CMJMetrics",
+    "CMJVideoConfig",
+    "CMJVideoResult",
+    "process_cmj_video",
+    "process_cmj_videos_bulk",
+]

{kinemotion-0.47.1 → kinemotion-0.47.3}/src/kinemotion/cmj/__init__.py

@@ -1,5 +1,5 @@
 """Counter Movement Jump (CMJ) analysis module."""
 
-from kinemotion.cmj.kinematics import CMJMetrics
+from .kinematics import CMJMetrics
 
 __all__ = ["CMJMetrics"]

{kinemotion-0.47.1 → kinemotion-0.47.3}/src/kinemotion/cmj/analysis.py

@@ -480,7 +480,7 @@ def compute_average_hip_position(
 def find_standing_end(
     velocities: np.ndarray,
     lowest_point: float,
-    positions: np.ndarray | None = None,
+    _positions: np.ndarray | None = None,
     accelerations: np.ndarray | None = None,
 ) -> float | None:
     """
@@ -493,7 +493,7 @@ def find_standing_end(
     Args:
         velocities: Signed velocity array (for backward compatibility)
         lowest_point: Frame index of lowest point
-        positions: Position array (unused, kept for backward compatibility)
+        _positions: Intentionally unused - kept for backward compatibility
         accelerations: Acceleration array (if provided, uses
             acceleration-based detection)