diff --git a/9f23-archive-services-setting.md b/9f23-archive-services-setting.md index a8a75b7..bb7b073 100644 --- a/9f23-archive-services-setting.md +++ b/9f23-archive-services-setting.md @@ -49,7 +49,14 @@ We need an `archive-services` setting that: - Created tests/archive_grasp_services.rs with 4 test scenarios - Tests matching/non-matching domains, multiple services, case-insensitivity - All 39 integration tests pass (525 total tests) -- Commits: 89b85d1 (feature), 25351fd (test helper fix), e815dbd (integration tests) +- Refactored for elegance and maintainability: + - Refactored `lists_service()` to `list_services()` returning Vec + - Now extracts domains from BOTH clone URLs AND relay URLs + - Updated `extract_domain()` to handle WebSocket URLs (ws://, wss://) + - Removed `clone_url_domains()` - no longer needed + - Squashed commits into single clean implementation + - All tests pass (39 integration, 18 config, 19 events validation) +- Final commit: e573c8b (feature + test helper fix + integration tests) ## Notes