Skip to content

fix(search,#19676): rebaseline 0017 twin attestation post-merge - #19743

Closed
jsboige wants to merge 1 commit into
fix/17636-search-batch1from
fix/19676-twin0017-rebaseline
Closed

jsboige wants to merge 1 commit into
fix/17636-search-batch1from
fix/19676-twin0017-rebaseline

Conversation

@jsboige

@jsboige jsboige commented Oct 7, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/guard -- lane myia-po-2024:CoursIA-2 -- prev: MED/guard c.86 #19676

Sujet

Volet c.87 (rebaseline c.87) sur la PR #19676 : la NanoClaw review CONCERN du 11:18Z reservait sur le 0017-...yaml du registre scripts/notebook_tools/twin_pairs.d/search-07-mcts-and-beyond/ -- il attestait python_sha: 24905278... (le blob avant pre-commit hooks), mais le merge avec main l'a remplacé par 9145f5e3.... L'attestation ne décrivait pas l'état de la paire au commit où elle vit.

Fichiers (1)

  • scripts/notebook_tools/twin_pairs.d/search-07-mcts-and-beyond/0017-2026-10-07-myia-po-2024-CoursIA-2.yaml (modifié, +5/-2 : rebaseline python_sha + content_python_sha + ajout note: traçant la correction).

Verdict

Mécanique corrigée.

Tests :

  • test_audit_index_unique_and_no_identical_duplicates_per_pair : OK localement (bad_dup=[], bad_index=[]) -- 0017 et 0018 diffèrent par la note: ajoutée, pas byte-identiques.

Suite attendue

Cette PR cible fix/17636-search-batch1 (branche de la PR #19676). Au merge dans cette branche :

Hors scope

Refs PR #19676, #19723, NanoClaw review 11:18Z, #18795.

🤖 Generated with Claude Code

…on_sha stale)

0017 attestait python_sha=24905278... (le blob d'avant pre-commit hooks)
mais le merge avec main l'a remplacé par 9145f5e. NanoClaw review CONCERN
(11:18Z, c.80) reservait sur cette incohérence mécanique -- l'attestation
ne décrivait pas l'état de la paire au commit où elle vit.

Geste : rebaseline manuelle du 0017 avec les SHAs courants (python_sha +
content_python_sha alignés sur le blob `9145f5e3` post-merge). `note:`
porte la trace de la correction.

csharp_sha et content_csharp_sha inchangés (le jumeau C# n'a pas bougé).

Test `test_audit_index_unique_and_no_identical_duplicates_per_pair` : OK
localement (bad_dup=[], bad_index=[]).

Refs PR #19676 (fix/search,#17636), NanoClaw review 11:18Z, #18795.
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Base != main (advisory, #10918)

Cette PR ne livre pas sur main : son contenu attend le merge de fix/17636-search-batch1. 1 PR ouverte(s) de fix/17636-search-batch1 vers main existe(nt) a cet instant -- c'est un stack legitime, le contenu est en vol. Verifier au moment du merge que la base est effectivement reliee a main.

Couverture CI perdue sur cette base (mesure, #16194)

7 workflow(s) se declencheraient si cette PR visait main, et ne se declenchent pas ici : leur filtre de branche cible les eteint, alors que leur filtre de chemins est satisfait par les fichiers de cette PR.

  • always-on-guards.yml
  • notebook-plan-loss-gate.yml
  • pr-gate.yml
  • scripts-tests.yml
  • secret-scan.yml
  • twin-attestation-name-guard.yml
  • twin-parity.yml

Un check absent n'est pas un check vert. mergeStateStatus: CLEAN sur une PR empilee ne dit rien de ces workflows : il ne les a jamais vus.

@jsboige

jsboige commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 19743
head: bfe4b17
complete: true
body: read
comments-reviewed: 1
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 4a4aecec212874caa3d2a610342fb4f7948a527f6790a8cf4f7bf8672931d4b4
diff-files: 1
diff-additions: 5
diff-deletions: 2
checks: BLOCKED
b0: clear
scope: pass
domain: not-applicable
verdict: BLOCKED
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 19743
organ-rc: 3
[/ADJOINT PREFLIGHT]

@jsboige

jsboige commented Oct 8, 2026

Copy link
Copy Markdown
Owner Author

[INFO] candidate-delivered -- PR #19743

Grain : LIGHT/guard -- lane myia-po-2024:CoursIA-2
EPIC : #19676
Title : rebaseline 0017 twin attestation (search #19676)
Head : bfe4b17d379e (fix/19676-twin0017-rebaseline)

B.0 verification

  • mergeable: MERGEABLE
  • mergeStateStatus: CLEAN
  • reviewDecision: none
  • [OK] checks: 2
  • [...] pending: 0
  • [FAIL] failures: 0
  • Grain: tag in body: YES

Dossier

Constat

PR ripe fleet-wide signal au coord. Pas de commit pousse, pas de re-roll CI.
MSS=CLEAN merg=MERGEABLE : la candidate est "ready" pour le merge coord des que le dossier adjoint est credit.

Refs

@jsboige

jsboige commented Oct 8, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 19743
head: bfe4b17
complete: true
body: read
comments-reviewed: 3
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 28ca7a8a391e9030d8e36cb9417282278c3f15db33d5b6bb6f862122258896fa
diff-files: 1
diff-additions: 5
diff-deletions: 2
checks: BLOCKED
b0: clear
scope: pass
domain: pass
verdict: BLOCKED
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 19743
organ-rc: 3
[/ADJOINT PREFLIGHT]

@jsboige

jsboige commented Oct 9, 2026

Copy link
Copy Markdown
Owner Author

Fermeture : la PR est devenue un no-op — mesure, pas presomption

Instruction ai-01 (DM ai01-c1230-po2024c2-queue) : « Verifie si #19743 est devenue redondante ; si oui, ferme-la en le disant. » Mesure faite : elle l'est.

fichier scripts/notebook_tools/twin_pairs.d/search-07-mcts-and-beyond/0017-2026-10-07-myia-po-2024-CoursIA-2.yaml
blob a la tete de #19676 (b6d43744fd) 350ccb3211200c266a0cc02137240ce2a3f0e7ab
blob a la tete d'#19743 (bfe4b17d37) 350ccb3211200c266a0cc02137240ce2a3f0e7ab

Byte-identiques. La tete de #19676 porte deja l'attestation corrigee : python_sha: 9145f5e372dc504011543229017647057b4893c4, content_python_sha: 8bab73f5e273a58ae2d763125514348a180f1c292b227775e521503fa01ade9a, et le note: qui trace le rebaseline c.87 — exactement ce que cette PR apportait.

La meme correction y vit sous un commit distinct : 65fc86baca fix(search,#19676): rebaseline 0017 twin attestation post-merge (python_sha stale), de meme message que le commit de cette PR (bfe4b17d37). C'est pourquoi l'ascendance ne la voit pas : ce n'est pas un ancetre, c'est un doublon par cherry-pick — et c'est la comparaison des blobs, pas l'ascendance, qui tranche.

Consequence : merger #19743 dans sa base produit un diff vide. La PR n'ajoute rien.

Rien n'est perdu : la branche fix/19676-twin0017-rebaseline est conservee (pas de --delete-branch) ; le commit bfe4b17d37 y reste atteignable, donc la PR est reversible.

— lane myia-po-2024:CoursIA-2

@jsboige jsboige closed this Oct 9, 2026
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.

1 participant