Feature-Enforcement f�r Activity CSV-Import #37

New Issue

Lars · 2026-03-23T22:32:40+01:00

Lars commented

2026-03-23 22:32:40 +01:00

Priorit�t: Medium
Aufwand: 2-3h

Beschreibung

Activity CSV-Import (Apple Health) ist aktuell NICHT einschr�nkbar �ber das Feature-System. CSV-Import kann unbegrenzt Eintr�ge erstellen, Nutzer k�nnten Limits umgehen.

Problem

Endpoint: POST /api/activity/import-csv
Feature-ID: Keine (verwendet activity_entries nur f�r manuelle Eintr�ge)
Impact: Nutzer k�nnten Limits durch CSV-Import umgehen

L�sung

Backend

Entscheiden: Separates Feature activity_import oder activity_entries erweitern?
Bulk-Import-Logik: Limit-Check VOR Import oder Counter NACH Import?

Endpoint erweitern:

# In /api/activity/import-csv
access = check_feature_access(pid, "activity_entries")
if not access["allowed"]:
    raise HTTPException(403, detail="Activity Import limit reached")

# Nach Import:
increment_feature_usage(pid, "activity_entries", count=imported_count)

Frontend

Import-Zone deaktivieren oder nur Error-Message?
Usage-Badge auf Import-Panel hinzuf�gen
Alternativ: Nur Error-Message nach Versuch (aktuelles Verhalten)

Betroffene Dateien

backend/routers/activity.py (import-csv Endpoint)
frontend/src/pages/ActivityPage.jsx (Import UI)

Akzeptanzkriterien

CSV-Import respektiert Feature-Limit
UI zeigt Usage-Badge (optional)
Import-Zone deaktiviert bei Limit (optional)
Error-Message erkl�rt Problem

Quelle: .claude/docs/PENDING_FEATURES.md

**Priorit�t:** Medium **Aufwand:** 2-3h ## Beschreibung Activity CSV-Import (Apple Health) ist aktuell NICHT einschr�nkbar �ber das Feature-System. CSV-Import kann unbegrenzt Eintr�ge erstellen, Nutzer k�nnten Limits umgehen. ## Problem - **Endpoint:** `POST /api/activity/import-csv` - **Feature-ID:** Keine (verwendet `activity_entries` nur f�r manuelle Eintr�ge) - **Impact:** Nutzer k�nnten Limits durch CSV-Import umgehen ## L�sung ### Backend 1. Entscheiden: Separates Feature `activity_import` oder `activity_entries` erweitern? 2. Bulk-Import-Logik: Limit-Check VOR Import oder Counter NACH Import? 3. Endpoint erweitern: ```python # In /api/activity/import-csv access = check_feature_access(pid, "activity_entries") if not access["allowed"]: raise HTTPException(403, detail="Activity Import limit reached") # Nach Import: increment_feature_usage(pid, "activity_entries", count=imported_count) ``` ### Frontend - Import-Zone deaktivieren oder nur Error-Message? - Usage-Badge auf Import-Panel hinzuf�gen - Alternativ: Nur Error-Message nach Versuch (aktuelles Verhalten) ## Betroffene Dateien - `backend/routers/activity.py` (import-csv Endpoint) - `frontend/src/pages/ActivityPage.jsx` (Import UI) ## Akzeptanzkriterien - [ ] CSV-Import respektiert Feature-Limit - [ ] UI zeigt Usage-Badge (optional) - [ ] Import-Zone deaktiviert bei Limit (optional) - [ ] Error-Message erkl�rt Problem --- **Quelle:** `.claude/docs/PENDING_FEATURES.md`

Lars added the

feature

label 2026-03-23 22:32:40 +01:00

Lars commented

2026-07-23 10:32:49 +02:00

Umsetzung 2026-07-23 (nach Architektur-Abgleich)

Architektur-Check vor Implementierung

Phase C (Schreibpfad): laut ACTIVITY_PRODUCTION_ARCHITECTURE_AND_PHASES.md bereits erledigt (Sync abgestellt, Orchestrator als SSoT, keine Aufrufer von sync_column_backed_session_metrics). Keine Doppel-Implementierung.
Feature-ID: Issue-Vorgabe activity_entries (nicht separates activity_import) — konsistent mit create_activity und Universal-CSV-Modul-Check in csv_import.py.
Legacy-Endpoint bleibt: Frontend nutzt weiterhin POST /api/activity/import-csv (ActivityPage Apple-Health-Panel); Universal-Pfad ist parallel (ARCH §8.2).

Änderungen

Backend (routers/activity.py):

check_feature_access(pid, "activity_entries") vor Legacy-Import
HTTP 403 bei Limit (wie nutrition/import-csv)
increment_feature_usage pro neu eingefügter Zeile (inserted)

Frontend (ActivityPage.jsx ImportPanel):

UsageBadge am Import-Titel
Drop-Zone deaktiviert bei Limit
Usage-Reload nach Import

Tests: tests/test_activity_import_feature_enforcement.py (403 + Increment)

Regression INSERT-SQL: tests/test_activity_insert_sql.py (grün)

**Umsetzung 2026-07-23 (nach Architektur-Abgleich)** ## Architektur-Check vor Implementierung - **Phase C (Schreibpfad):** laut `ACTIVITY_PRODUCTION_ARCHITECTURE_AND_PHASES.md` bereits erledigt (Sync abgestellt, Orchestrator als SSoT, keine Aufrufer von `sync_column_backed_session_metrics`). Keine Doppel-Implementierung. - **Feature-ID:** Issue-Vorgabe `activity_entries` (nicht separates `activity_import`) — konsistent mit `create_activity` und Universal-CSV-Modul-Check in `csv_import.py`. - **Legacy-Endpoint bleibt:** Frontend nutzt weiterhin `POST /api/activity/import-csv` (`ActivityPage` Apple-Health-Panel); Universal-Pfad ist parallel (ARCH §8.2). ## Änderungen **Backend** (`routers/activity.py`): - `check_feature_access(pid, "activity_entries")` vor Legacy-Import - HTTP 403 bei Limit (wie `nutrition/import-csv`) - `increment_feature_usage` pro neu eingefügter Zeile (`inserted`) **Frontend** (`ActivityPage.jsx` ImportPanel): - `UsageBadge` am Import-Titel - Drop-Zone deaktiviert bei Limit - Usage-Reload nach Import **Tests:** `tests/test_activity_import_feature_enforcement.py` (403 + Increment) Regression INSERT-SQL: `tests/test_activity_insert_sql.py` (grün)

Lars closed this issue

2026-07-23 10:32:50 +02:00

Lars referenced this issue

2026-07-23 11:05:55 +02:00

Feature-Enforcement f�r Nutrition CSV-Import UI #38

Sign in to join this conversation.