Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

Classify secondary and abuse rate limits separately from generic GitHub failures

Aberta
#2,219 1 comentário 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
76/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
github, go
Domínio
api, backend

Direção de pesquisa

Comece pelo classifier em pkg/errors/error.go, depois acompanhe o tratamento das respostas do GitHub por meio de pkg/utils/api.go e revise docs/error-handling.md. Adicione cobertura de unit tests mostrando que as respostas dos limites primary, secondary e abuse são classificadas de forma distinta, preservando os metadados de retry quando disponíveis; os testes devem passar em relação ao commit referenciado.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Problem
GitHub quota and abuse responses currently lose their specific classification in the shared error layer, so downstream tooling cannot distinguish them from generic 403/transport failures.

Why now
Rate limits are a routine operational condition for GitHub integrations. This server already documents structured error handling, but the central classifier only recognizes invalid token and insufficient scope today.

Evidence packet

  • Commit under test: 0c0ac794a443626d3a1517f8154f109f06a26e68
  • Runtime: macOS 15.3 / Darwin 25.3.0 arm64, Go 1.25.7
  • Relevant codepaths:
    • pkg/errors/error.go
    • pkg/utils/api.go
    • docs/error-handling.md
  • Minimal repro:
    1. Trigger a secondary rate limit or abuse-detection response from GitHub.
    2. Observe the error payload stored in middleware / tool results.
  • Expected: explicit classifications for secondary rate limits and abuse limits, ideally preserving retry timing when GitHub provides it.
  • Actual: classifyHTTPErrorCode only handles 401 invalid_token and some 403 insufficient_scope cases, leaving rate-limit-specific failures unclassified.

Scope
This is a platform contract issue in shared GitHub error mapping.

Validation target
Unit tests should prove that primary, secondary, and abuse limit responses classify distinctly and preserve retry metadata when available.

Linguagem predominante
Go
Estrelas
33.1k
Forks
5k
Merge médio
2d 1h
PRs com merge (30d)
25

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de github/github-mcp-server

Todas as issues de github/github-mcp-server

Issues semelhantes

Mais issues de Go

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.