docs(parity): verdict relecteur — 22 CLOSED, 33 rouverts (raison = critère de clôture), 14 contrats annotés - #22
Conversation
…avec raison-critère, 0/14 contrats signés annotés - 22 P0 signés → CLOSED (reviewer OpenAI-Codex, reviewCommit 57febea). - 33 P0 refusés → OPEN avec reviewVerdict REFUSED, refusalType (DESACCORD/PREUVE_INSUFFISANTE/NON_REPRODUCTIBLE/A_PRECISER) et la raison du refus comme conditionDeCloture ; 11 raisons verbatim transmises, 22 marquées A_PRECISER (aucune raison fabriquée). - 14 contrats §2.3 annotés reviewVerdict REFUSED + critère de clôture. - REMEDIATION_VERDICT_20260720.md : 5 corrections rapides / 5 chantiers / 1 bloquant (rapport détaillé du relecteur pour 22 refus). - Vues régénérées ; validateurs verts ; niveau inchangé contractsPresent / NOT_APPROVED ; 4 PROVEN hors lot intacts. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 33fa6f6c6e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| |---|---|---|---| | ||
| | R1 | P0-V4-1, P0-V4-2, P0-V3-02 | hashes Gallery obsolètes (fad9… vs 1f5f…) | re-capturer le rendu Gallery, re-hasher, mettre à jour SOURCE_REGISTRY + citations, rejouer les vérifs | | ||
| | R2 | P0-LS-14 | claim absolu « no model selector anywhere » | reformuler en observation BORNÉE (surfaces observées + date + hash), purger le claim absolu partout | | ||
| | R3 | P0-A2-10 | Gallery est DECIDED, pas OPEN | aligner DEC-GALLERY-NO-SELF-PUBLISH → DECIDED dans DECISION_REGISTRY + le P0 | |
There was a problem hiding this comment.
Target the Gallery option-B decision in R3
This remediation points at the wrong decision: P0-A2-10's proof references DEC-OWNER-GALLERY-OPTION-B, which is already DECIDED, while DEC-GALLERY-NO-SELF-PUBLISH is a separate, intentionally OPEN product decision about curated intake versus self-service publishing. Following R3 would close that unresolved decision without an owner verdict and would not correct the stale P0 proof that caused the refusal.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
Cette PR met à jour la documentation/parité pour refléter le verdict de relecture (lot 57febeab) : 22 P0 signés et clôturés, 33 P0 refusés et rouverts avec critères de clôture explicites, et 14 contrats annotés comme refusés avec critère de remédiation.
Changes:
- Annotation des contrats et documents de parité avec
reviewer,reviewVerdictetreviewCloseCriterion(verdict refusé tant que non durci + signature). - Mise à jour des registres/statuts générés (P0 registry, approval status, parity status, document manifest) pour refléter 22 CLOSED / 39 OPEN / 4 PROVEN.
- Ajout d’un plan de remédiation priorisé (
REMEDIATION_VERDICT_20260720.md) et traçage dans le changelog.
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/parity/SECURITY_PRIVACY_COMPLIANCE.md | Ajoute métadonnées de review + critère de clôture pour le contrat sécurité/privacy. |
| docs/parity/RUNTIME_NIX_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat Nix runtime. |
| docs/parity/REMEDIATION_VERDICT_20260720.md | Nouveau document: plan de remédiation post-verdict, priorisé. |
| docs/parity/RELEASE_PUBLISH_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat release/publish. |
| docs/parity/PROJECT_MANIFEST_SCHEMA.json | Ajoute champs x-reviewVerdict / x-reviewCloseCriterion au schéma. |
| docs/parity/PROJECT_FACTORY_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat project factory. |
| docs/parity/PARITY_STATUS.md | Met à jour la vue générée (niveaux + compteurs) suite aux réouvertures/refus. |
| docs/parity/P0_REGISTRY.yaml | Met à jour les statuts P0 (OPEN/CLOSED) + verdicts/refus + critères de clôture. |
| docs/parity/OPERATIONS_DR.md | Ajoute métadonnées de review (refus) pour ops/DR. |
| docs/parity/IMPORT_REMIX_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat import/remix. |
| docs/parity/IDENTITY_COLLABORATION_CONTRACT.md | Ajoute métadonnées de review (refus) pour identité/collaboration. |
| docs/parity/IAM_POLICY_BASELINE.md | Ajoute métadonnées de review (refus) pour le baseline IAM. |
| docs/parity/GALLERY_COMMUNITY_CONTRACT.md | Ajoute métadonnées de review (refus) pour gallery/community. |
| docs/parity/DOMAIN_MODEL.md | Ajoute métadonnées de review (refus) pour le modèle de domaine. |
| docs/parity/DOCUMENT_MANIFEST.yaml | Met à jour le manifest généré (fileCount + hashes) incluant le nouveau doc de remédiation. |
| docs/parity/DEPLOYMENT_TYPES_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat des types de déploiement. |
| docs/parity/CHECKPOINT_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat checkpoint. |
| docs/parity/CHANGELOG_AUDIT.md | Ajoute une entrée d’audit décrivant le verdict et ses effets sur les statuts. |
| docs/parity/BILLING_LEDGER_CONTRACT.md | Ajoute métadonnées de review (refus) pour le contrat ledger/billing. |
| docs/parity/APPROVAL_STATUS.json | Met à jour le statut d’approbation généré (raisons/rollups) selon les nouveaux états. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| reviewCloseCriterion: durcir le contenu puis obtenir la signature du relecteur ; raison détaillée du refus à consigner verbatim dès transmission du rapport | ||
| Schémas JSON exécutables: `docs/parity/schemas/domain/*.schema.json`. |
| reviewCloseCriterion: durcir le contenu puis obtenir la signature du relecteur ; raison détaillée du refus à consigner verbatim dès transmission du rapport | ||
| Règle d'or: ce qui n'a jamais été TESTÉ en réel est marqué UNTESTED — un plan |
| reviewCloseCriterion: durcir le contenu puis obtenir la signature du relecteur ; raison détaillée du refus à consigner verbatim dès transmission du rapport | ||
| Décisions détaillées: `docs/NIX_V2_DECISION.md` + `docs/DEPLOY_REPRODUCIBLE_PIPELINE.md` |
| reviewCloseCriterion: durcir le contenu puis obtenir la signature du relecteur ; raison détaillée du refus à consigner verbatim dès transmission du rapport | ||
| Source de vérité prix: `docs/parity/RATE_CARD.json` est GÉNÉRÉ depuis |
| | parityBaselineReady | ❌ FAIL (surface SRF-IDE-FILE-HISTORY not done … +10) | | ||
|
|
||
| **Compteurs (source unique)** : P0 65 (6 OPEN · 37 PROVEN · 22 CLOSED) · P1 40 · surfaces déclarées 10 (univers 159/159 importé, 1 évaluées, 56 services) · e2e 12/12 · constats 336 → 99 work items · claims non ancrées 0 · uiGaps [publish, rollback] | ||
| **Compteurs (source unique)** : P0 65 (39 OPEN · 4 PROVEN · 22 CLOSED) · P1 40 · surfaces déclarées 10 (univers 159/159 importé, 1 évaluées, 56 services) · e2e 12/12 · constats 336 → 99 work items · claims non ancrées 0 · uiGaps [publish, rollback] |
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Quoi (verdict OpenAI-Codex, lot 57febea)
reviewer: OpenAI-Codex,reviewCommit: 57febeab) — la garde CI existante exige commit+reviewer+proof pour CLOSED, satisfaite.reviewVerdict: REFUSED, unrefusalType(DÉSACCORD / PREUVE_INSUFFISANTE / NON_REPRODUCTIBLE / A_PRECISER) et la raison du refus commeconditionDeCloture. 11 raisons verbatim (WIF 3 chemins, générateur IMPLEMENTATION_STATUS absent, code Import ≠ contrat, CI --check sans génération, ledger double-entrée manquant, prix sans contexte, claim « no model selector » trop fort, Gallery DECIDED, hashes Gallery obsolètes ×3) ; 22 refus sans raison détaillée transmise →A_PRECISER(aucune raison fabriquée — à compléter verbatim dès réception du rapport).reviewVerdict: REFUSED+ critère de clôture (durcir puis signature).docs/parity/REMEDIATION_VERDICT_20260720.md(5 corrections rapides / 5 chantiers / 1 bloquant relecteur).contractsPresent/NOT_APPROVED.🤖 Generated with Claude Code