Skip to content

chore(citations,#17712): g7-b0-ci-organs -- residuel 1 ligne (Tell c.820 dans detect_oss_signature) - #19119

Merged
myia-ai-01 merged 3 commits into
mainfrom
chore/17712-tellc-detect-oss-signature
Oct 4, 2026
Merged

myia-ai-01 merged 3 commits into
mainfrom
chore/17712-tellc-detect-oss-signature

Conversation

@jsboige

@jsboige jsboige commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/docs — lane myia-po-2025:CoursIA — prev: DEEP/qc #19106

Relai g7-b0-ci-organs de #17712 : retrait du résiduel « Tell c.820 » de la famille b0-ci-organs, fichier non couvert par la passe g3 (#18079, 6 fichiers — scripts/ci/detect_oss_signature.py absent).

Ce que la PR livre

1 fichier, +1/−1, docstring seule (scripts/ci/detect_oss_signature.py l.7) : ...LTAI prefix). Tell c.820 / → ...LTAI prefix). — la phrase qui suit (« secrets-hygiene rule 1: a presigned Signature IS a secret... ») reste cohérente, le « / » n'était que le pivot de la citation. Aucun code touché.

Vérifications (relancées sur la tête de la branche)

  • git grep -n "Tell c\." scripts/ci/detect_oss_signature.py : 0 occurrence résiduelle (1 sur origin/main avant — ligne 7, mesuré à l'instant).
  • python -m pytest scripts/tests/test_detect_oss_signature.py -q : 10 passed (75 s).

Contexte anti-collision

#18079 (g3, MERGED) n'a pas touché ce fichier (liste files vérifiée) — la citation vivait encore sur origin/main. Précédents de résiduels une-ligne post-famille : #18351 (g4), #18353 (g5), #18538 (relais c9).

See #17712

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

@jsboige

jsboige commented Oct 4, 2026

Copy link
Copy Markdown
Owner Author

Rouge Scripts Tests (CPU) = base-inherited (preuve), hors périmètre du diff

Run 37214855946 (job 111473180091), deux échecs, aucun lié à la PR (diff = 1 docstring dans scripts/ci/detect_oss_signature.py ; aucun des deux tests ne référence ce fichier) :

  1. scripts/tests/test_characterize_runner_variance.py::test_main_json_shape (assert 0 == 1, jobs_total) — rouge de main depuis 13:00Z sur toute PR qui relance la jambe (cas mesurés dans le body de fix(ci,#15574): figer l'horloge de test_main_json_shape (rouge de main depuis 13:00Z) #19111 : Fix(gate,#19093): re-jeu B.0 sur un dossier BLOCKED dont b0 est l'unique motif #19105 vert 12:49Z, feat(translation,#1650): FineTuning FT-01 + FT-03 EN -- 23 cellules md + 2 rendus _en #18968 rouge 13:26Z, même échec unique). Fix en cours : fix(ci,#15574): figer l'horloge de test_main_json_shape (rouge de main depuis 13:00Z) #19111 (figer l'horloge), OPEN.
  2. scripts/translation/tests/test_check_translation_parity.py::test_full_repo_state_passes_parity — medical_chatbot_en CODE_DRIFT (cellule d0d7a23d, outputs 48 vs 41) — fallout du merge fix(genai,#14755): F7 CaseStudies+Security — substitution modeles obsoletes + re-executions (medical FR/EN, receipe, tooling MCP) #18844, fix en cours : fix(genai,#18844): re-rendre medical_chatbot_en par T4 -- parite FR/EN cassee sur main #19115, OPEN.

Protocole : quand #19111 et #19115 sont sur main → update-branch (sans conflit attendu) + rejeu de la jambe. Le plancher DWELL (commit 15:43Z, élude ~17:43Z) sera depuis longtemps écoulé à ce moment.

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Hermes] VERDICT: LGTM — #19119 retrait du résiduel de citation « Tell c.820 » (docstring seule, +1/−1), head 4a222de094a4.

Vérifications réelles :

  1. Résiduel confirmé éliminé : grep -n "Tell" sur scripts/ci/detect_oss_signature.py au head → 0 occurrence ; la phrase suivante (« secrets-hygiene rule 1: a presigned Signature IS a secret… », l.9-11) reste complète et cohérente — le / retiré n'était que le pivot de la citation, comme le dit le body.
  2. Portée exacte : 1 fichier, docstring seulement, aucun chemin de code touché — le pattern détecteur (3-tuple signature) est intact.
  3. CI rouge au head = hérité de main, pas causé par cette PR : Scripts Tests (CPU) échoue aussi sur main (f224f81016, 33d65baabc). Joints lus : (a) test_full_repo_state_passes_parity → CODE_DRIFT medical_chatbot_en — c'est précisément l'objet de #19115 ; (b) test_main_json_shape assert 0 == 1 — objet de #19111. Ces deux rouges préexistent au 1-ligne docstring et seront résorbés par les PRs de fix dédiées. PR gate FAIL = agrégat de ce Scripts Tests, pas un défaut propre.

[Hermes hermes-pr-review, cycle :18 04/10, host f6be46d1b7a3, sig=6e9f100a]

@jsboige

jsboige commented Oct 4, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 19119
head: 69eb002
complete: true
body: read
comments-reviewed: 2
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: a7f4d75228af1f056e127f7f09559f3203c41e88bd2660accdc44152552147a0
diff-files: 1
diff-additions: 1
diff-deletions: 1
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

@myia-ai-01
myia-ai-01 merged commit ddac1e1 into main Oct 4, 2026
19 checks passed
@jsboige
jsboige deleted the chore/17712-tellc-detect-oss-signature branch October 7, 2026 07:41
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.

3 participants