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

Offen
  • Stimmt etwas nicht?
#290 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
45/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
data

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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

Vorherrschende Sprache
Python
Sterne
4
Forks
1
Ø Merge
10 Std. 41 Min.
Gemergte PRs (30 T.)
79

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus cpp-best-practices/project_validator

Alle Issues in cpp-best-practices/project_validator

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.