From 58c607fd799a12802bb7bc7deac6239f8e6bae6c Mon Sep 17 00:00:00 2001 From: CleverThis Date: Fri, 24 Apr 2026 15:02:02 +0000 Subject: [PATCH] fix(providers): restore parse step matcher after openrouter regex steps The openrouter_provider_registry_steps.py file called use_step_matcher("re") at module level without restoring the default "parse" matcher at the end. This caused all step files loaded alphabetically after it (provider_registry_*, resource_*, session_*, etc.) to use the regex matcher instead of the parse matcher, breaking their {value}-style step patterns and causing unit_tests CI failures. Add use_step_matcher("parse") at the end of the file to restore the default matcher after the openrouter step definitions are registered. --- features/steps/openrouter_provider_registry_steps.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/features/steps/openrouter_provider_registry_steps.py b/features/steps/openrouter_provider_registry_steps.py index 3c1f4467a..d7bf5ae32 100644 --- a/features/steps/openrouter_provider_registry_steps.py +++ b/features/steps/openrouter_provider_registry_steps.py @@ -372,3 +372,8 @@ def step_or_assert_provider_not_configured(context: Any) -> None: info = context.or_provider_info assert info is not None, "Expected provider info to exist" assert info.is_configured is False, "Expected provider to not be configured" + + +# Restore the default step matcher so that step files loaded after this one +# are not affected by the regex matcher set above. +use_step_matcher("parse")