📝 docs(adr): audit and update Status for 5 implemented ADRs #24

arcodange · 2026-05-03T13:31:48+02:00

arcodange commented

2026-05-03 13:31:48 +02:00

Summary

Audit of 7 ADRs marked "Proposed" → updates Status header for the 5 that are at least partially implemented. The full audit with file:line evidence is in audit-report.md (workspace, not in repo).

Updates

ADR	Old	New	Why
0018 user-management-auth-system	Proposed	Partially Implemented	auth/jwt/repos exist; middleware + greet integration missing
0019 postgresql-integration	Proposed	Partially Implemented	postgres repo exists; sqlite still present, not default
0022 rate-limiting-cache-strategy	Proposed	Implemented (Phase 1) - Phase 2 still Proposed	PRs #22 ratelimit + #23 cache; Redis/Dragonfly deferred
0024 bdd-test-organization-and-isolation	Proposed	Partially Implemented	domain dirs + scenario state; parallel exec opt-in only
0025 bdd-scenario-isolation-strategies	Proposed	Partially Implemented	schema-per-scenario opt-in via env; cache/user store isolation missing

Kept as "Proposed" (production code absent)

0021 jwt-secret-retention-policy : BDD scenarios test the proposed behavior but no ConfigManager / cleanup goroutine in production
0023 config-hot-reloading : testserver has reload logic but no Viper WatchConfig in pkg/config/

Audit method

Q-024 compliant: every status decision has at least 1 file:line of evidence (or git commit ref). Method documented in workspace.

Out of scope

Implementing the missing parts (each is a separate PR)
Updating the adr/README.md status table (if it exists - audit didn't touch it)

Migration context

ARCODANGE Phase 1 - autonomous run via Mistral Vibe with ICM workspace at ~/Work/Vibe/workspaces/adr-audit-status/. Cost €2.50 (full audit stage). Trainer finalized commit/PR.

🤖 Co-Authored-By: Mistral Vibe (devstral-2 / mistral-medium-3.5) + Claude Opus 4.7

## Summary Audit of 7 ADRs marked "Proposed" → updates Status header for the 5 that are at least partially implemented. The full audit with file:line evidence is in [audit-report.md](https://github.com/arcodange/dance-lessons-coach/blob/main/workspaces/adr-audit-status/stages/01-audit/output/audit-report.md) (workspace, not in repo). ## Updates | ADR | Old | New | Why | |---|---|---|---| | 0018 user-management-auth-system | Proposed | **Partially Implemented** | auth/jwt/repos exist; middleware + greet integration missing | | 0019 postgresql-integration | Proposed | **Partially Implemented** | postgres repo exists; sqlite still present, not default | | 0022 rate-limiting-cache-strategy | Proposed | **Implemented (Phase 1) - Phase 2 still Proposed** | PRs #22 ratelimit + #23 cache; Redis/Dragonfly deferred | | 0024 bdd-test-organization-and-isolation | Proposed | **Partially Implemented** | domain dirs + scenario state; parallel exec opt-in only | | 0025 bdd-scenario-isolation-strategies | Proposed | **Partially Implemented** | schema-per-scenario opt-in via env; cache/user store isolation missing | ## Kept as "Proposed" (production code absent) - **0021** jwt-secret-retention-policy : BDD scenarios test the proposed behavior but no ConfigManager / cleanup goroutine in production - **0023** config-hot-reloading : testserver has reload logic but no Viper WatchConfig in pkg/config/ ## Audit method Q-024 compliant: every status decision has at least 1 file:line of evidence (or git commit ref). Method documented in workspace. ## Out of scope - Implementing the missing parts (each is a separate PR) - Updating the adr/README.md status table (if it exists - audit didn't touch it) ## Migration context ARCODANGE Phase 1 - autonomous run via Mistral Vibe with ICM workspace at ~/Work/Vibe/workspaces/adr-audit-status/. Cost €2.50 (full audit stage). Trainer finalized commit/PR. 🤖 Co-Authored-By: Mistral Vibe (devstral-2 / mistral-medium-3.5) + Claude Opus 4.7

arcodange added 1 commit 2026-05-03 13:31:50 +02:00

📝 docs(adr): audit and update Status for 5 implemented ADRs 8c0e3830f8

Audits 7 ADRs marked "Proposed" against the actual code, updates the Status
field of 5 that are at least partially implemented. Keeps 2 as "Proposed"
because only test infrastructure exists (no production implementation).

Updated:
- 0018 user-management-auth-system : Partially Implemented (auth/jwt/repos exist; auth middleware + greet integration missing)
- 0019 postgresql-integration : Partially Implemented (postgres repo exists, BDD uses it; sqlite still present, not default)
- 0022 rate-limiting-cache-strategy : Implemented (Phase 1) - Phase 2 still Proposed (PRs #22 ratelimit, #23 cache; Redis/Dragonfly deferred)
- 0024 bdd-test-organization-and-isolation : Partially Implemented (domain dirs + scenario state isolation; parallel exec opt-in only)
- 0025 bdd-scenario-isolation-strategies : Partially Implemented (schema-per-scenario opt-in via BDD_SCHEMA_ISOLATION; cache/user store isolation missing)

Kept "Proposed" (production code not implemented, only test fixtures):
- 0021 jwt-secret-retention-policy (BDD scenarios exist but no ConfigManager / cleanup goroutine in pkg/)
- 0023 config-hot-reloading (testserver has reload, but no Viper WatchConfig in production)

Audit method: Q-024 compliant - every status decision has file:line evidence
documented in workspaces/adr-audit-status/stages/01-audit/output/audit-report.md.

Generated ~95% in autonomy by Mistral Vibe via ICM workspace
~/Work/Vibe/workspaces/adr-audit-status/. Cost €2.50 stage 01-audit (very thorough).
Trainer (Claude) finalized commit/PR (Mistral hit max-price).

🤖 Co-Authored-By: Mistral Vibe (devstral-2 / mistral-medium-3.5)
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>