feat: add GitHub templates (CODEOWNERS, PR template, issue templates) (#10)

* feat: add GitHub templates (CODEOWNERS, PR template, issue templates)

- CODEOWNERS for automatic reviewer assignment
- Pull request template with checklist
- Issue templates for bugs and feature requests

Stu Mason + AI <me@stumason.dev>

* style: fix ruff formatting on 4 Python files

Ran ruff format to fix formatting issues that were causing CI failure.

Stu Mason + AI <me@stumason.dev>

* fix: prefix unused user_id argument with underscore

Fixes ruff ARG002 (unused method argument) lint error while keeping
backwards compatibility.

Stu Mason + AI <me@stumason.dev>

* fix: use noqa comment instead of underscore prefix

The underscore prefix broke backwards compatibility for callers using
user_id as a keyword argument. Using noqa: ARG002 to suppress the lint
error while keeping the public API stable.

Stu Mason + AI <me@stumason.dev>

Author: StuMason

.github/CODEOWNERS

@@ -0,0 +1 @@
+* @StuMason

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -0,0 +1,26 @@
+name: Bug Report
+description: Report something that's broken
+labels: ["bug"]
+body:
+  - type: textarea
+    id: description
+    attributes:
+      label: What happened?
+    validations:
+      required: true
+  - type: textarea
+    id: steps
+    attributes:
+      label: Steps to reproduce
+    validations:
+      required: true
+  - type: textarea
+    id: expected
+    attributes:
+      label: Expected behaviour
+    validations:
+      required: true
+  - type: textarea
+    id: context
+    attributes:
+      label: Additional context

.github/ISSUE_TEMPLATE/config.yml

@@ -0,0 +1 @@
+blank_issues_enabled: true

.github/ISSUE_TEMPLATE/feature_request.yml

@@ -0,0 +1,20 @@
+name: Feature Request
+description: Suggest a new feature or enhancement
+labels: ["enhancement"]
+body:
+  - type: textarea
+    id: problem
+    attributes:
+      label: Problem or motivation
+    validations:
+      required: true
+  - type: textarea
+    id: solution
+    attributes:
+      label: Proposed solution
+    validations:
+      required: true
+  - type: textarea
+    id: alternatives
+    attributes:
+      label: Alternatives considered

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,16 @@
+## Summary
+
+
+## Changes
+
+-
+
+## Test plan
+
+- [ ] Tests added/updated
+- [ ] Manually tested locally
+
+## Checklist
+
+- [ ] Code follows project conventions
+- [ ] Documentation updated if needed

src/polar_flow/endpoints/activity_samples.py

@@ -50,11 +50,15 @@ async def list(
             ```
         """
         # Determine date range
-        end = date.today() if to_date is None else (
-            date.fromisoformat(to_date) if isinstance(to_date, str) else to_date
+        end = (
+            date.today()
+            if to_date is None
+            else (date.fromisoformat(to_date) if isinstance(to_date, str) else to_date)
         )
-        start = end - timedelta(days=days - 1) if from_date is None else (
-            date.fromisoformat(from_date) if isinstance(from_date, str) else from_date
+        start = (
+            end - timedelta(days=days - 1)
+            if from_date is None
+            else (date.fromisoformat(from_date) if isinstance(from_date, str) else from_date)
         )
 
         path = f"/v3/users/activities/samples?from={start}&to={end}"

src/polar_flow/endpoints/sleep.py

@@ -63,7 +63,7 @@ async def get(self, user_id: str, date: str | date) -> SleepData:
 
     async def list(
         self,
-        user_id: str | None = None,  # Keep for backwards compat but not used
+        user_id: str | None = None,  # noqa: ARG002 - Keep for backwards compat but not used
         *,
         days: int | None = None,
         since: str | None = None,

src/polar_flow/models/biosensing.py

@@ -21,17 +21,11 @@ class SpO2Result(BaseModel):
     test_status: str = Field(description="Test completion status")
     blood_oxygen_percent: int = Field(ge=0, le=100, description="SpO2 percentage")
     spo2_class: str = Field(description="Classification (NORMAL, LOW, etc.)")
-    spo2_value_deviation_from_baseline: str = Field(
-        description="Deviation from user's baseline"
-    )
-    spo2_quality_average_percent: float = Field(
-        description="Signal quality percentage"
-    )
+    spo2_value_deviation_from_baseline: str = Field(description="Deviation from user's baseline")
+    spo2_quality_average_percent: float = Field(description="Signal quality percentage")
     average_heart_rate_bpm: int = Field(description="Average HR during test")
     heart_rate_variability_ms: float = Field(description="HRV in milliseconds")
-    spo2_hrv_deviation_from_baseline: str = Field(
-        description="HRV deviation from baseline"
-    )
+    spo2_hrv_deviation_from_baseline: str = Field(description="HRV deviation from baseline")
     altitude_meters: float | None = Field(default=None, description="Altitude if available")
 
     @property
@@ -65,16 +59,10 @@ class ECGResult(BaseModel):
     time_zone_offset: int = Field(description="Timezone offset in minutes")
     average_heart_rate_bpm: int = Field(description="Average HR during test")
     heart_rate_variability_ms: float = Field(description="HRV in milliseconds (RMSSD)")
-    heart_rate_variability_level: str = Field(
-        description="HRV classification (LOW, NORMAL, HIGH)"
-    )
+    heart_rate_variability_level: str = Field(description="HRV classification (LOW, NORMAL, HIGH)")
     rri_ms: float = Field(description="R-R interval in milliseconds")
-    pulse_transit_time_systolic_ms: float | None = Field(
-        default=None, description="PTT systolic"
-    )
-    pulse_transit_time_diastolic_ms: float | None = Field(
-        default=None, description="PTT diastolic"
-    )
+    pulse_transit_time_systolic_ms: float | None = Field(default=None, description="PTT systolic")
+    pulse_transit_time_diastolic_ms: float | None = Field(default=None, description="PTT diastolic")
     pulse_transit_time_quality_index: float | None = Field(
         default=None, description="PTT quality index"
     )
@@ -103,9 +91,7 @@ class TemperatureSample(BaseModel):
     """Single temperature measurement sample."""
 
     temperature_celsius: float = Field(description="Temperature in Celsius")
-    recording_time_delta_milliseconds: int = Field(
-        description="Time offset from period start (ms)"
-    )
+    recording_time_delta_milliseconds: int = Field(description="Time offset from period start (ms)")
 
 
 class BodyTemperaturePeriod(BaseModel):
@@ -156,12 +142,8 @@ def max_temperature(self) -> float | None:
 class SkinTemperature(BaseModel):
     """Sleep skin temperature with baseline deviation."""
 
-    sleep_time_skin_temperature_celsius: float = Field(
-        description="Skin temperature during sleep"
-    )
-    deviation_from_baseline_celsius: float = Field(
-        description="Deviation from user's baseline"
-    )
+    sleep_time_skin_temperature_celsius: float = Field(description="Skin temperature during sleep")
+    deviation_from_baseline_celsius: float = Field(description="Deviation from user's baseline")
     sleep_date: str = Field(description="Date of sleep (YYYY-MM-DD)")
 
     @property

src/polar_flow/models/sleep.py

@@ -19,37 +19,52 @@ class SleepData(BaseModel):
     sleep_start_time: dt.datetime = Field(description="When sleep started")
     sleep_end_time: dt.datetime = Field(description="When sleep ended")
     device_id: str = Field(description="Device ID that recorded the sleep")
-    continuity: float = Field(description="Sleep continuity score (0.0-5.0, 0=no data)", ge=0.0, le=5.0)
-    continuity_class: int = Field(description="Sleep continuity classification (0-5, 0=no data)", ge=0, le=5)
+    continuity: float = Field(
+        description="Sleep continuity score (0.0-5.0, 0=no data)", ge=0.0, le=5.0
+    )
+    continuity_class: int = Field(
+        description="Sleep continuity classification (0-5, 0=no data)", ge=0, le=5
+    )
     light_sleep: int = Field(description="Light sleep duration in seconds", ge=0)
     deep_sleep: int = Field(description="Deep sleep duration in seconds", ge=0)
     rem_sleep: int = Field(description="REM sleep duration in seconds", ge=0)
     unrecognized_sleep_stage: int = Field(
         description="Unrecognized sleep stage duration in seconds", ge=0
     )
-    sleep_score: int = Field(description="Overall sleep quality score (0-100, 0=no data)", ge=0, le=100)
+    sleep_score: int = Field(
+        description="Overall sleep quality score (0-100, 0=no data)", ge=0, le=100
+    )
     total_interruption_duration: int = Field(
         description="Total interruption duration in seconds", ge=0
     )
-    sleep_charge: int | None = Field(default=None, description="Sleep charge score (-1=unavailable, 1-100)", ge=-1, le=100)
+    sleep_charge: int | None = Field(
+        default=None, description="Sleep charge score (-1=unavailable, 1-100)", ge=-1, le=100
+    )
     sleep_goal: int | None = Field(default=None, description="Sleep goal in seconds", ge=0)
-    sleep_rating: int | None = Field(default=None, description="User's subjective sleep rating (0-5, 0=not rated)", ge=0, le=5)
+    sleep_rating: int | None = Field(
+        default=None, description="User's subjective sleep rating (0-5, 0=not rated)", ge=0, le=5
+    )
     short_interruption_duration: int | None = Field(
         default=None, description="Short interruption duration in seconds", ge=0
     )
     long_interruption_duration: int | None = Field(
         default=None, description="Long interruption duration in seconds", ge=0
     )
     sleep_cycles: int | None = Field(default=None, description="Number of sleep cycles", ge=0)
-    group_duration_score: float | None = Field(default=None, description="Group duration score", ge=0.0, le=100.0)
-    group_solidity_score: float | None = Field(default=None, description="Group solidity score", ge=0.0, le=100.0)
+    group_duration_score: float | None = Field(
+        default=None, description="Group duration score", ge=0.0, le=100.0
+    )
+    group_solidity_score: float | None = Field(
+        default=None, description="Group solidity score", ge=0.0, le=100.0
+    )
     group_regeneration_score: float | None = Field(
         default=None, description="Group regeneration score", ge=0.0, le=100.0
     )
 
     # Sample data (time → value mappings)
     hypnogram: dict[str, int] | None = Field(
-        default=None, description="Hypnogram data (time → sleep stage mapping: 0=awake, 1=light, 3=deep, 4=REM)"
+        default=None,
+        description="Hypnogram data (time → sleep stage mapping: 0=awake, 1=light, 3=deep, 4=REM)",
     )
     heart_rate_samples: dict[str, int] | None = Field(
         default=None, description="Heart rate samples (time → BPM mapping, ~5 min intervals)"

tests/test_biosensing_endpoint.py

@@ -102,9 +102,7 @@ class TestBiosensingSpO2:
     """Tests for SpO2 endpoint."""
 
     @pytest.mark.asyncio
-    async def test_get_spo2_success(
-        self, httpx_mock: HTTPXMock, spo2_response: list[dict]
-    ) -> None:
+    async def test_get_spo2_success(self, httpx_mock: HTTPXMock, spo2_response: list[dict]) -> None:
         """Test successful SpO2 data fetch."""
         httpx_mock.add_response(json=spo2_response)
 
@@ -123,9 +121,7 @@ async def test_get_spo2_with_date_range(self, httpx_mock: HTTPXMock) -> None:
         httpx_mock.add_response(json=[])
 
         async with PolarFlow(access_token="test_token") as client:
-            await client.biosensing.get_spo2(
-                from_date="2024-01-01", to_date="2024-01-07"
-            )
+            await client.biosensing.get_spo2(from_date="2024-01-01", to_date="2024-01-07")
 
         request = httpx_mock.get_request()
         assert "from=2024-01-01" in str(request.url)
@@ -155,9 +151,7 @@ class TestBiosensingECG:
     """Tests for ECG endpoint."""
 
     @pytest.mark.asyncio
-    async def test_get_ecg_success(
-        self, httpx_mock: HTTPXMock, ecg_response: list[dict]
-    ) -> None:
+    async def test_get_ecg_success(self, httpx_mock: HTTPXMock, ecg_response: list[dict]) -> None:
         """Test successful ECG data fetch."""
         httpx_mock.add_response(json=ecg_response)
 
@@ -199,9 +193,7 @@ async def test_get_body_temperature(
         assert len(results[0].samples) == 3
 
     @pytest.mark.asyncio
-    async def test_body_temp_computed_properties(
-        self, body_temp_response: list[dict]
-    ) -> None:
+    async def test_body_temp_computed_properties(self, body_temp_response: list[dict]) -> None:
         """Test body temperature computed properties."""
         result = BodyTemperaturePeriod.model_validate(body_temp_response[0])
 
@@ -227,9 +219,7 @@ async def test_get_skin_temperature(
         assert results[0].sleep_time_skin_temperature_celsius == 35.8
 
     @pytest.mark.asyncio
-    async def test_skin_temp_elevation_detection(
-        self, skin_temp_response: list[dict]
-    ) -> None:
+    async def test_skin_temp_elevation_detection(self, skin_temp_response: list[dict]) -> None:
         """Test skin temperature elevation detection."""
         normal = SkinTemperature.model_validate(skin_temp_response[0])
         elevated = SkinTemperature.model_validate(skin_temp_response[1])