# Implementation Plan: [FEATURE] **Branch**: `[###-feature-name]` | **Date**: [DATE] | **Spec**: [link] **Input**: Feature specification from `/specs/[###-feature-name]/spec.md` **Note**: This template is filled in by the `/speckit.plan` command. See `.specify/templates/commands/plan.md` for the execution workflow. ## Summary This feature updates the GET /api/analysis/{analysis_id}/charts and GET /api/analysis/{analysis_id}/summary endpoints to automatically fetch and analyze workout files from a CentralDB. ## Technical Context **Language/Version**: Python 3.11 **Primary Dependencies**: FastAPI, pandas, numpy, scipy, matplotlib, fitparse, tcxparser, gpxpy, requests **Storage**: Ephemeral in-memory cache (last 5), CentralDB (long-term) **Testing**: pytest **Target Platform**: Linux server **Project Type**: web **Performance Goals**: 95% of cached requests < 500ms; new analyses < 45s for a 2-hour workout. **Constraints**: `analysis_id` must match CentralDB activity ID. **Scale/Scope**: 50 concurrent analysis requests. ## Constitution Check *GATE: Must pass before Phase 0 research. Re-check after Phase 1 design.* | Principle | Status | Justification (if violation) | |---|---|---| | I. Library-First | Aligned | The new functionality will be part of the existing library structure. | | II. CLI Interface | VIOLATION | The feature is API-only, which is the primary interface for this web service. | | III. Test-First (NON-NEGOTIABLE) | Aligned | The spec has acceptance scenarios that can be tested first. | | IV. Integration Testing | Aligned | The feature requires integration with CentralDB. | | V. Observability | Aligned | No new observability requirements for this feature. | | VI. Versioning & Breaking Changes | Aligned | No breaking changes are expected. | | VII. Simplicity | Aligned | The feature simplifies the user workflow by removing the need for manual file uploads. | ## Project Structure ### Documentation (this feature) ``` specs/[###-feature]/ ├── plan.md # This file (/speckit.plan command output) ├── research.md # Phase 0 output (/speckit.plan command) ├── data-model.md # Phase 1 output (/speckit.plan command) ├── quickstart.md # Phase 1 output (/speckit.plan command) ├── contracts/ # Phase 1 output (/speckit.plan command) └── tasks.md # Phase 2 output (/speckit.tasks command - NOT created by /speckit.plan) ``` ### Source Code (repository root) ``` src/ ├── core/ │ ├── batch_processor.py │ ├── chart_generator.py │ ├── file_parser.py │ ├── logger.py │ ├── report_generator.py │ ├── workout_analyzer.py │ └── workout_data.py ├── db/ │ ├── models.py │ └── session.py └── utils/ └── zone_calculator.py api/ ├── routers/ │ └── analysis.py ├── main.py └── schemas.py tests/ ├── unit/ ├── integration/ ├── contract/ └── performance/ ``` **Structure Decision**: The project will continue to use the existing structure, which separates the core logic, API, and tests into `src`, `api`, and `tests` directories respectively. ## Complexity Tracking *Fill ONLY if Constitution Check has violations that must be justified* | Violation | Why Needed | Simpler Alternative Rejected Because | |-----------|------------|-------------------------------------| | Violation | Why Needed | Simpler Alternative Rejected Because | |---|---|---| | II. CLI Interface | The primary interface for this feature is a REST API, as it is extending an existing web service. | A CLI is not the primary interface for this service and would add unnecessary complexity at this stage. |