fix telemetry gating for OpenRouter headers

- keep default OpenRouter provider extra_headers unset
- rely on facade injection so telemetry opt-out suppresses attribution
- align default provider test with the gated behavior

Author: eric-tramel

packages/data-designer-config/src/data_designer/config/utils/constants.py

@@ -307,6 +307,8 @@ class NordColor(Enum):
     "X-OpenRouter-Categories": "programming-app",
 }
 
+# OpenRouter attribution is injected in the engine so telemetry opt-out can
+# suppress it cleanly for both default and existing provider configs.
 PREDEFINED_PROVIDERS = [
     {
         "name": NVIDIA_PROVIDER_NAME,
@@ -325,7 +327,6 @@ class NordColor(Enum):
         "endpoint": "https://openrouter.ai/api/v1",
         "provider_type": "openai",
         "api_key": OPENROUTER_API_KEY_ENV_VAR_NAME,
-        "extra_headers": OPENROUTER_ATTRIBUTION_HEADERS,
     },
 ]
 

packages/data-designer-config/tests/config/test_default_model_settings.py

@@ -110,11 +110,7 @@ def test_get_builtin_model_providers():
     assert builtin_model_providers[2].endpoint == "https://openrouter.ai/api/v1"
     assert builtin_model_providers[2].provider_type == "openai"
     assert builtin_model_providers[2].api_key == "OPENROUTER_API_KEY"
-    assert builtin_model_providers[2].extra_headers == {
-        "HTTP-Referer": "https://github.com/NVIDIA-NeMo/DataDesigner",
-        "X-OpenRouter-Title": "NeMo Data Designer",
-        "X-OpenRouter-Categories": "programming-app",
-    }
+    assert builtin_model_providers[2].extra_headers is None
 
 
 def test_get_default_model_configs_path_exists(tmp_path: Path):

packages/data-designer-engine/tests/engine/models/test_facade.py

@@ -276,7 +276,7 @@ def test_consolidate_kwargs_openrouter_attribution(
 ) -> None:
     """OpenRouter-specific attribution headers are injected when provider is openrouter."""
     stub_model_facade.model_provider.name = "openrouter"
-    stub_model_facade.model_provider.extra_headers = None  # simulates existing user config
+    stub_model_facade.model_provider.extra_headers = None
     result = stub_model_facade.consolidate_kwargs()
     assert result["extra_headers"] == {
         "X-Title": "NeMo Data Designer",