data-wellformed's message for a 100000-deep JSON file depends on the Python build ("malformed JSON" vs "Expecting value")

Aberta
  • Algo errado?
#290 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
45/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
python
Domínio
data

Direção de pesquisa

Start in ecosystems/data.py at _parse_json, which calls json.loads and falls back to the non-recursive check_json only on RecursionError. Reproduce with n_structure_100000_opening_arrays.json and n_structure_open_array_object.json from the JSONTestSuite corpus under two recursion limits. Done when one message is produced regardless of the limit, and the corpus expectations are updated to match.

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

Descrição

ecosystems/data.py::_parse_json tries json.loads first and falls back to the non-recursive check_json only on RecursionError. For the JSON test-suite files n_structure_100000_opening_arrays.json and n_structure_open_array_object.json (denoland/std jsonc/testdata/JSONTestSuite/, ocaml-community/yojson test/compliance/test_cases/), json.loads raises RecursionError on one machine (→ check_json → "1:100001: malformed JSON") and JSONDecodeError("Expecting value") on another (the GitHub runner, Python 3.14.10 → "1:100001: Expecting value" / "2:1: Expecting value"), so the corpus expectations for the two projects differ between the first local recording and the first CI run (corpus PR).

Expected: one message whatever the recursion limit, e.g. always parse with check_json (or report the C decoder's position only when it agrees with the non-recursive checker). The corpus records the runner's wording for now.

Found by the regression corpus's first CI run.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ByTXNqDWdM7HzezrFtEvQt

Linguagem predominante
Python
Estrelas
4
Forks
1
Merge médio
10h 41min
PRs com merge (30d)
79

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

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 cpp-best-practices/project_validator

Todas as issues de cpp-best-practices/project_validator

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

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