Skip to content

detect_code_in_markdown_cells --json : la ligne d'identite casse la purete du JSON sur stdout #12858

Description

@myia-ai-01

La ligne d'identite pollue la sortie --json

Releve par Hermes en review de #12779 (concern mineur, declare non-bloquant, et ouvert avant le merge conformement a CLAUDE.md §B.0 : ce qui reporte doit exister avant gh pr merge, pas etre nomme apres).

detect_code_in_markdown_cells.py imprime desormais une ligne d'identite baseline: <path> (<n> entries) pour tous les modes non-update, --json compris. La sortie --json n'est donc plus du JSON pur sur stdout : la premiere ligne est cette ligne de prose.

Pourquoi ce n'est pas bloquant aujourd'hui : Hermes a cherche les consommateurs (search/code + grep sur scripts/ et .github/) — aucun consommateur --json n'existe dans le depot. Rien ne casse.

Pourquoi ca merite une issue quand meme : le premier | jq qu'on branchera dessus cassera sans que personne ne comprenne pourquoi, et la cause sera enfouie dans une ligne ajoutee pour une raison sans rapport. C'est moins cher a fermer maintenant qu'a diagnostiquer plus tard.

Fix

Gater la ligne sur not args.json, ou l'emettre sur stderr en mode json (preferable : l'identite reste visible a l'humain, stdout reste pipeable).

Acceptance

  1. --json rend du JSON parsable : python scripts/notebook_tools/detect_code_in_markdown_cells.py --json ... | jq . reussit.
  2. La ligne d'identite reste visible en mode non-json — c'est l'acquis de detect_code_in_markdown_cells --check sans --baseline compare a un ensemble vide : FAIL fantome (6 constats) sur un main vert #12585, on ne le perd pas.
  3. Controle positif : montrer le jq qui echoue avant le fix et reussit apres, dans la meme invocation. Un jq qui reussit seul ne prouve pas qu'il echouait.

See #12585

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingcandidate-deliveredReferenced by a merged PR with no post-merge activity -- candidate for close triage (#10466)

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions