Change RPE default from 1 to 5 to avoid misleading "Nothing at all" on new entries#19
Conversation
Greptile SummaryThis PR changes the default RPE value from Confidence Score: 5/5Safe to merge — minimal, well-scoped change with no logic errors or regressions. All three touch-points (default value, hasData sentinel, test fixture) are updated consistently. No P0 or P1 findings; the schema already accepts 5, form reset and draft restoration all behave correctly with the new default. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User opens app / clears form] --> B["getDefaultWorkoutValues()\nrpe: 5 ← NEW default"]
B --> C[form.reset with defaults]
C --> D{Has user changed RPE?}
D -- "rpe === 5 (default)" --> E["hasData = false\n(no indicator shown)"]
D -- "rpe ≠ 5 (user-modified)" --> F["hasData = true\n(indicator shown on Core Metrics)"]
F --> G[Markdown preview includes R: value]
E --> G
Reviews (1): Last reviewed commit: "Change RPE default from 1 to 5 to avoid ..." | Re-trigger Greptile |
https://claude.ai/code/session_01QFXvhJEFTL94eKgXU36Q3v