Skip to content

fix(pipeline): учитывать реальный GraphQL-бюджет - #71

Merged
ivanarama merged 2 commits into
mainfrom
fix/graphql-rate-budget-sampler
Sep 17, 2026
Merged

ivanarama merged 2 commits into
mainfrom
fix/graphql-rate-budget-sampler

Conversation

@ivanarama

@ivanarama ivanarama commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Что изменено

  • core и search берутся из REST GET /rate_limit;
  • GraphQL remaining, limit, used, resetAt и viewer берутся из реального GraphQL rateLimit;
  • при повреждённом/противоречивом bucket, GraphQL errors или несовпадении viewer с настроенным trusted_account допуск закрывается с безопасной диагностикой без сырого ответа и токена;
  • отсутствие rate-limit telemetry не ломает legacy-профили без github_budget;
  • документация уточняет, что выборка GraphQL обычно расходует один GraphQL point.

Проверка

  • python -m pytest -q -W error — 522 passed;
  • реальный gh api graphql --input - подтвердил форму ответа (viewer, limit, remaining, resetAt, used) без вывода чувствительных данных.

Репликация REVIEW и планировщик намеренно не входят в этот PR.

@ivanarama
ivanarama merged commit b7bced9 into main Sep 17, 2026
4 checks passed
@ivanarama
ivanarama deleted the fix/graphql-rate-budget-sampler branch September 17, 2026 12:21
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