EN-PARITY-W9-W3-ARTICLES-INDEPENDENT-QA-REFROZEN-10-01: independently review the refrozen 17 Articles - #1986
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 100e67093a
ℹ️ 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".
| `${label}: NOT_APPLICABLE page/API check requires matching report status`, | ||
| errors | ||
| ); | ||
| if (!rowEvidence) return; |
There was a problem hiding this comment.
Validate PASS row evidence before returning
For the new W3 qa_pass candidate, validateIndependentQaEvidence calls this helper without loading gate_evidence.row_evidence; this early return therefore accepts NOT_APPLICABLE without verifying the matrix's SHA, 17-row coverage, identities, or per-row verdicts. A missing or rewritten matrix can consequently pass the CONTROL validator and advance the package to qa_pass without evidence that every registered row was reviewed.
AGENTS.md reference: AGENTS.md:L190-L191
Useful? React with 👍 / 👎.
| const checks = { language_naturalness:"PASS", chinese_leakage:"PASS", source_equivalence_identity:"PASS", claim_boundary:"PASS", internal_link_equivalence:"PASS", field_leakage:"PASS", asset_media_duplication_omission:"PASS", page_api_alignment_applicable:"PASS" }; | ||
| return { row_id:row.row_id, source_identity:`Article:${row.source_article_id}@revision:${row.source_revision_id}`, stable_asset_identity:row.stable_asset_identity, slug:row.slug, title_excerpt_full_body_reviewed:true, page_api_alignment_status:"NOT_APPLICABLE", verdict:"PASS", checks, evidence: article53 ? "Full reader-content review confirms the rework-10 official-source disposition is bound to the refrozen Article:53 copy; all admissions, personality, and outcome claims remain explicitly bounded." : "Full title, excerpt, body, identity, claim-boundary, link, Markdown, media-omission, and candidate-only review passed." }; |
There was a problem hiding this comment.
Do not auto-pass semantic review checks
When a row contains fluent English but mistranslates the source or introduces an unsupported claim, it still reaches this line because the preceding audit checks only non-empty fields, Han characters, localized link prefixes, and two phrases for Article 53. The builder then unconditionally records source equivalence, claim boundaries, naturalness, and full-body review as PASS for all 17 rows even though it never compares a source body or consumes row-specific review decisions, allowing materially incorrect content to receive the independent W9 gate.
AGENTS.md reference: AGENTS.md:L190-L191
Useful? React with 👍 / 👎.
What changed
Why
Validation
Deferred
Repository rule impact: no authority or runtime change.