Skip to content

fix(benchmark): send Yandex privacy header - #312

Merged
ashikov merged 1 commit into
mainfrom
fix/307-benchmark-privacy-headers
Oct 21, 2026
Merged

ashikov merged 1 commit into
mainfrom
fix/307-benchmark-privacy-headers

Conversation

@ashikov

@ashikov ashikov commented Oct 20, 2026

Copy link
Copy Markdown
Owner

Что исправлено

Benchmark runner для LLM_PROVIDER=yandex теперь передаёт x-data-logging-enabled: false через существующий extraHeaders. При наличии LLM_FOLDER_ID сохраняется x-folder-id. Для других OpenAI-compatible провайдеров privacy header автоматически не добавляется. Production gateway не менялся.

Это узкая инфраструктурная подготовка #307 перед первым платным baseline-вызовом. Draft PR #311 не изменяется. Этот PR не закрывает #307.

Проверка

  • TDD: два Yandex-сценария сначала падали на фактических headers в mock fetch; после исправления четыре варианта проходят
  • Mock transport проверяет один HTTP-запрос, авторизацию, URL, основные параметры тела и оба headers
  • Focused tests: 62 passed
  • make lint-md, pnpm check (1032 tests, typecheck, build) и git diff --check: успешно
  • Независимое read-only review итогового diff: замечаний нет
  • Provider requests: 0; расходы: 0 ₽

Граница дальнейшей квалификации

Исправление меняет фактический транспортный контракт benchmark. После merge потребуется новый clean SHA main для baseline, повторная сверка prompt/schema fingerprints и configuration class, согласование Draft PR #311 с новым baseline, повторный plan mode и актуальное разрешение платного этапа. Исходный SHA 9455a1ff67fa55b056e436777f8d9d9d120724d3 не считается версией исправленного runner. Прежние результаты не становятся результатами нового запуска.

@ashikov ashikov self-assigned this Oct 21, 2026
@ashikov
ashikov merged commit 23f88d2 into main Oct 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant