Skip to content

Navigation Menu

Sign in
Sign up

docs: define RTL audit evidence policy - #6046

Draft
nynexman4464 wants to merge 1 commit into
main from
nynexman4464/docs/rtl-audit-evidence
Draft

docs: define RTL audit evidence policy #6046
nynexman4464 wants to merge 1 commit into
main from
nynexman4464/docs/rtl-audit-evidence

Conversation

@nynexman4464

@nynexman4464 nynexman4464 commented Sep 4, 2026

Copy link
×ばつ24 expansion is removed because D7 does not currently assert size. " data-view-component="true"> Copy Markdown
Contributor

Problem

The RTL audit currently mixes implemented checks, proposed verified-N/A handling, and proposed known-debt handling. An all-N/A result can be mistaken for positive RTL evidence, while source lint, browser geometry, interaction checks, and screenshots prove different things.

Changes

Add a draft architecture:rtl-audit-evidence record that:

  • distinguishes measured, verified-N/A, coverage-gap, behavior-failure, stale-evidence, and proposed known-debt states;
  • keeps source lint, rendered geometry, and interaction evidence within their actual proof boundaries;
  • makes the live component roster the coverage owner;
  • states that behavior failures have no allowlist;
  • separates affected-component PR scope from the full weekly sweep;
  • requires mutation proof for new audit dimensions;
  • records today's enforcement gaps instead of describing proposed behavior as shipped.

The draft explicitly records that D7 checks hit-testing and center alignment but does not yet enforce target size. It also records that the reason-only verified-N/A registry detects newly applicable dimensions but does not bind evidence to source/story bytes.

Decisions needed before promotion

  1. Admit verified N/A with fail-closed evidence binding, or require executable rendered measurement for every component.
  2. Admit a visible removal-only baseline for pre-existing unexplained gaps, or keep every gap failing.
  3. Define the clean-state and stability prerequisites for making RTL a required merge check.

The draft also calls out the missing architecture:bidirectional-layout owner for shared RTL implementation mechanics; this evidence record does not claim that ownership.

This PR is intentionally draft and remains non-authoritative until those policy choices are approved.

Test plan

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 4, 2026

vercel Bot commented Sep 4, 2026
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
astryx Ready Ready Preview Sep 4, 2026 8:29pm UTC

Request Review

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

PR Analysis Report

Preview availability: Storybook and Sandbox were not published for this CI run.

No new or modified components detected.

Bundle Size Summary

No component packages changed.

Accessibility Audit

Status: No accessibility violations detected.


Generated by PR Enrichment workflow | View full report

cixzhang commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

/approve-spec a1a63a9

@cixzhang cixzhang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for separating measured RTL evidence, verified non-applicability, historical coverage debt, and merge-gate policy instead of treating one implementation as already decided. Approved as the canonical RTL audit-evidence record at exact head a1a63a9238fe5d382cb49b3b3c493b6d48e83ed3.

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

Reviewers

@cixzhang cixzhang cixzhang approved these changes
@imdreamrunner imdreamrunner Awaiting requested review from imdreamrunner imdreamrunner will be requested when the pull request is marked ready for review imdreamrunner is a code owner
@josephfarina josephfarina Awaiting requested review from josephfarina josephfarina will be requested when the pull request is marked ready for review josephfarina is a code owner

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /