Weekly hardening: 6 new v153 references + 3 v147 MDN-shape upgrades - #9
Open
PaulKinlan wants to merge 10 commits into
Open
PaulKinlan wants to merge 10 commits into
PaulKinlan wants to merge 10 commits into
Conversation
MDN stub — MDN covers Iterator.prototype.join with BCD (Chrome 153); stub links it per invariant #6, with sourced quick-reference (spec, V8 flag, Intent, BCD, test262) and showcase concept links. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Generated from the Intent to Ship thread and Chromium source (parser selection algorithm, xml-rs ParserConfig, XHTML entity allowlist, perf numbers quoted verbatim). Three child routes: affected entry points, compatibility differences, what still uses libxml2. Implementation-sufficient contract. Suite assertion demo-embed-own-feature authored pre-commit to pin the canonical CPS route (filed at v147; no v153 showcase route exists). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Generated from spec PR WICG/js-self-profiling#89 (IDL and algorithms transcribed from the PR head; page discloses the PR is not yet merged), runtime_enabled_features.json5 and the ChromeStatus OT record. Origin-trial warn-block with registration + flag steps. Child routes: ProfilerSample.marker, ProfilerMarker enum. Implementation-sufficient contract. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Generated from WHATWG HTML PR #12590 diff and the Igalia dev-trial instructions; draft-vs-shipped IDL divergence (spellCheckDictionary vs spellCheckCustomDictionary) documented explicitly rather than papered over. Dev-trial warn-block with exact flags. Child routes: addWords(), removeWords(). Implementation-sufficient contract. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Generated from the WICG delayed-message-timing explainer, Chromium IDL/impl (exposure delta, 200ms/5ms constants quoted verbatim from animation_frame_timing_monitor.cc) and the Intent to Prototype. Dev-trial warn-block (--enable-blink-features=LongAnimationFrameWorker; no chrome://flags entry, flag status 'test' disclosed). Child routes: worker entries, scriptCount. Implementation-sufficient contract. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Generated from the W3C WebTransport editor's draft (IDL and constructor/ establishment steps transcribed verbatim; cross-checked against spec PR #713) and the Intent to Ship. Dev-trial warn-block at 153; slated enabled-by-default in 155 per the milestone listing, disclosed. Child routes: WebTransportOptions.headers, WebTransport.responseHeaders. Implementation-sufficient contract. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
…, 6330376953921536) Quality upgrade per the durable contract: page predates the MDN shape (no Baseline banner, prose support table, no live example). Added Baseline banner (web-features device-memory, limited), verbatim spec IDL+ABNF syntax, real BCD per-version compat (4 BCD files with explicit Chrome-147 notes), showcase memory-inspector embed (route 200), child routes for navigator.deviceMemory and Sec-CH-Device-Memory. Kept verified bucket table, route, H1, identity. Fixes: unlinked mail-archive label removed; kill-switch framing corrected to the intent's own wording; rollout history (146 proposal, 1GB restore, x.com-regression disable, 147 re-enable) added with sources. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Quality upgrade + factual corrections, evidence in-page: the event dispatches at the Document (spec §4.1), not on individual controls, and fires before values are committed — the old example listened on form fields and is replaced. Added Baseline banner, OT warn-block with registration steps, verbatim WICG spec IDL, nine-dimension tables, showcase refill-flow embed (route 200), child routes autofillvalues/ and refill/. Route, H1, identity unchanged; conformance suite untouched and still green. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
…ce, 5159559872249856) Quality upgrade per the durable contract: added Baseline banner (web-features 'profiler', limited; worker exposure has no entry — disclosed), verbatim IDL with the [Exposed] delta from spec commit e603c27, Document Policy section, showcase worker-profile embed (route 200), profiler/ child route covering the full interface surface. Honest milestone note: the feature's dev trial now targets Chrome 154 per the current listing (absent from the 147 listing); route and identity stay per the durable contract. The brief's supposed sample() method is NOT in the spec and is documented as absent. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
responsive-support.json: 9 touched routes matrix-validated ok/ok (runner scan + integrator-read screenshots at 360x740 DPR3 and 1280x800; scroll-container link flags verified expected); goals.json rolled (27 new follow-up goals); conformance runner reports from the per-page gates. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
There was a problem hiding this comment.
🟡 Changes recommended
Unresolved critical report and route-migration issues, along with correctness findings, block approval.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
This PR adds six v153 references and upgrades three v147 pages to the current reference format.
Changes:
- Adds reference pages, child routes, contracts, critiques, and conformance suites.
- Improves three v147 references with updated examples and compatibility data.
- Updates responsive and conformance report artifacts.
File summaries
| File | Reviewed change |
|---|---|
v153/xml-parsing-in-rust-for-non-xslt-scenarios/conformance.json |
XML parser conformance suite; milestone migration remains unresolved. |
v153/xml-parsing-in-rust-for-non-xslt-scenarios/_questions.json |
XML parser critique and follow-up metadata. |
v153/webtransport-headers-and-responseheaders/reference-contract.json |
WebTransport implementation contract. |
v153/webtransport-headers-and-responseheaders/conformance.json |
WebTransport conformance suite. |
v153/webtransport-headers-and-responseheaders/_questions.json |
WebTransport critique metadata. |
v153/support-long-animation-frames-api-in-web-workers/scriptcount/index.html |
scriptCount member reference. |
v153/support-long-animation-frames-api-in-web-workers/conformance.json |
Worker LoAF conformance suite. |
v153/support-long-animation-frames-api-in-web-workers/_questions.json |
Worker LoAF critique metadata. |
v153/spell-check-custom-dictionary-api/reference-contract.json |
Dictionary API implementation contract. |
v153/spell-check-custom-dictionary-api/conformance.json |
Dictionary API conformance suite. |
v153/spell-check-custom-dictionary-api/_questions.json |
Dictionary API critique metadata. |
v153/js-self-profiling-markers/reference-contract.json |
Profiling markers implementation contract. |
v153/js-self-profiling-markers/conformance.json |
Profiling markers conformance suite. |
v153/js-self-profiling-markers/_questions.json |
Profiling markers critique metadata. |
v153/iterator-join/index.html |
MDN reference stub. |
v153/iterator-join/conformance.json |
Stub conformance metadata. |
v153/iterator-join/_questions.json |
Stub critique metadata. |
v147/update-device-memory-api-limits/reference-contract.json |
Device Memory implementation contract. |
v147/update-device-memory-api-limits/devicememory/index.html |
Device Memory API reference and worker example. |
v147/update-device-memory-api-limits/_questions.json |
Device Memory critique metadata. |
v147/js-profiling-in-dedicated-workers/_questions.json |
Worker profiling critique metadata. |
v147/autofill-event/_questions.json |
Autofill critique metadata. |
responsive-support.json |
Responsive validation records. |
reports/conformance/results.json |
Conformance results report; currently page-scoped. |
reports/conformance/responsive.json |
Responsive report; currently incomplete. |
reports/conformance/index.html |
Human-readable conformance rollup; currently incomplete. |
Review details
Suppressed comments (11)
v147/autofill-event/refill/index.html:140
- This process-wide boolean prevents every later autofill event from calling
refill()after the first event succeeds. The one-call restriction is per event object, so a page that receives multiple legitimate autofill events will silently skip all but the first; track event objects instead of using a global flag.
<pre><code>let refillRequested = false;
document.addEventListener("autofill", async (event) => {
if (event.refill && !refillRequested) {
refillRequested = true; // second call would reject anyway
await event.refill();
v147/js-profiling-in-dedicated-workers/index.html:230
- This visible citation label has no direct link to the specification, commit, explainer, or MDN artifact listed in the table. Link each original artifact in this citation (or remove the redundant generic citation) so the sources are directly traceable.
<span class="citation">Source: documents fetched and verified 2026-09-13.</span>
v147/update-device-memory-api-limits/index.html:185
- This citation still names the BCD data only as “BCD files linked above”. The repository's source-link contract requires each cited source label to link directly; link each of the four BCD files again in this citation instead of relying on nearby links.
<span class="citation">Source: BCD files linked above (fetched 2026-09-13); <a href="https://chromestatus.com/feature/6330376953921536" target="_blank" rel="noopener">ChromeStatus record</a> vendor views; <a href="https://webstatus.dev/features/device-memory" target="_blank" rel="noopener">webstatus.dev — device-memory</a>.</span>
v153/js-self-profiling-markers/index.html:245
- This visible citation label has no direct link to the documents whose contents it claims to summarize. Link the PR, published specification, marker explainer, conditional-exposure explainer, and TAG review here (or remove the redundant generic citation) rather than relying only on nearby table links.
<span class="citation">Source: repository and specification contents fetched 2026-09-13 (PR branch fetched via git; merge state checked against <code>main</code>).</span>
v153/spell-check-custom-dictionary-api/index.html:244
- This visible citation label has no direct link to the documents it describes. Link the WHATWG/Igalia/TAG/design-document artifacts listed in the table here (or remove the redundant generic citation) so each source remains directly traceable.
v153/support-long-animation-frames-api-in-web-workers/index.html:263 - This visible citation label has no direct link to the W3C, explainer, design-doc, or WPT artifacts named around this section. Replace it with direct links to each cited source (or remove the redundant generic citation) instead of leaving only a fetch-status note.
v153/webtransport-headers-and-responseheaders/headers/index.html:115 - The documented feature-detection expression has the same unguarded
WebTransport.prototypeaccess, so it throws before a fallback can run in browsers without WebTransport. Include atypeof WebTransport !== "undefined"guard.
v153/webtransport-headers-and-responseheaders/headers/index.html:137 - A bearer token in a request header is still visible to the WebTransport server and may be captured by server or intermediary request-header logging; unlike a URL, the header does not guarantee exclusion from logs or telemetry. Limit this claim to URL leakage and document that logging depends on the server/intermediary configuration.
v153/webtransport-headers-and-responseheaders/index.html:71 - This feature-detection expression throws a
ReferenceErrorin browsers whereWebTransportis not defined, so it is not safe cross-browser detection. Guard the constructor before reading its prototype.
v153/webtransport-headers-and-responseheaders/index.html:226 - This visible citation label only says that the documents were fetched; it does not link the original WebTransport, Fetch, RFC, or draft artifacts listed immediately above. Link each cited primary artifact in the citation itself (or remove this redundant generic citation) so the provenance remains directly traceable.
v153/webtransport-headers-and-responseheaders/responseheaders/index.html:105 - This feature-detection guidance also dereferences
WebTransport.prototypewithout first checking whetherWebTransportexists, so it throws in unsupported browsers. Use a guarded expression before accessing the prototype.
- Files reviewed: 53/53 changed files
- Comments generated: 10
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
24
to
+28
| <div class="stat"><div class="n">1</div><div class="l">suites</div></div> | ||
| <div class="stat"><div class="n">23</div><div class="l">assertions</div></div> | ||
| <div class="stat"><div class="n">18</div><div class="l">pass</div></div> | ||
| <div class="stat"><div class="n">25</div><div class="l">assertions</div></div> | ||
| <div class="stat"><div class="n">19</div><div class="l">pass</div></div> | ||
| <div class="stat"><div class="n">0</div><div class="l">fail</div></div> | ||
| <div class="stat"><div class="n">5</div><div class="l">blocked</div></div> | ||
| <div class="stat"><div class="n">6</div><div class="l">blocked</div></div> |
| { | ||
| "id": "v147/css-border-shape", | ||
| "route": "/v147/css-border-shape/", | ||
| "id": "v147/autofill-event", |
Comment on lines
+2
to
6
| "generatedAt": "2026-09-13T06:44:54.095Z", | ||
| "agg": { | ||
| "total": 23, | ||
| "pass": 18, | ||
| "total": 25, | ||
| "pass": 19, | ||
| "fail": 0, |
| } | ||
| ], | ||
| "openQuestions": [ | ||
| "A published page for this same feature id exists at v151/xml-parsing-in-rust-for-non-xslt-scenarios/ (created when the feature sat in the 151 listing; it claims 'Chrome 151 enables by default' and a rollout that 'began at 1% in Chrome 147', neither of which matches the current record). The durable-URL contract requires an integrator-recorded move/alias migration in migrations.json rather than deletion; this writer could not touch files outside v153/<slug>/.", |
| "schemaVersion": 1, | ||
| "id": "v153/xml-parsing-in-rust-for-non-xslt-scenarios", | ||
| "route": "/v153/xml-parsing-in-rust-for-non-xslt-scenarios/", | ||
| "identity": "5309598397497344", |
|
|
||
| <header class="lede-block"> | ||
| <p class="eyebrow">v153 · dom · browser internals · enabled by default</p> | ||
| <h1>XML parsing in Rust for non-XSLT scenarios</h1> |
| memory: self.navigator.deviceMemory, // undefined outside Chromium | ||
| // ... other metrics | ||
| }; | ||
| navigator.sendBeacon("/endpoint", JSON.stringify(analyticsData));</code></pre> |
Comment on lines
+138
to
+140
| <pre><code>const wt = new WebTransport('https://transport.example.com:4433/media', { | ||
| headers: { 'authorization': `Bearer ${await getToken()}` }, | ||
| }); |
Comment on lines
+149
to
+151
| if (!('responseHeaders' in WebTransport.prototype)) { | ||
| // Fall back: token as a query parameter or app-level handshake. | ||
| } |
Comment on lines
+153
to
+158
| const wt = new WebTransport('https://transport.example.com:4433/session', { | ||
| headers: { | ||
| 'authorization': `Bearer ${token}`, | ||
| 'x-supported-codecs': 'av1,vp9', | ||
| }, | ||
| }); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Weekly depth run (2026-09-13). Channels at run time: stable cut = 154 (2026-09-22, so live stable is 153), beta 155, dev 156.
Coverage — v153 complete (6 new pages)
v153/iterator-joinIterator.prototype.joinwith BCD (Chrome 153)v153/xml-parsing-in-rust-for-non-xslt-scenariosv153/js-self-profiling-markersv153/spell-check-custom-dictionary-apiv153/support-long-animation-frames-api-in-web-workersv153/webtransport-headers-and-responseheadersMDN-shape upgrades — v147, in place (durable contract preserved)
v147/update-device-memory-api-limits— Baseline banner, verbatim IDL+ABNF, real BCD per-version compat, showcase embed, 2 child routes; fixed the unlinked mail-archive label and kill-switch framing; added sourced rollout history.v147/autofill-event— factual corrections with evidence (event dispatches at Document, before values commit; old field-level example replaced), OT warn-block, verbatim WICG IDL, showcase embed, 2 child routes.v147/js-profiling-in-dedicated-workers— Baseline banner,[Exposed]delta from spec commit e603c27, Document Policy section, showcase embed,profiler/child route; honest note that the dev trial now targets Chrome 154 per the current listing.Every full reference ships with an implementation-sufficient
reference-contract.json,_questions.jsoncritique (non-emptyguidanceConsulted), and a conformance suite.Gates (all green before push)
deno fmt --check,deno check server.tsdeno task check-routes— 6 added, 0 regressionsdeno task validate-artifacts— PASS (42 implementation-sufficient contracts)deno task test-reference-contract— structural + browser visibility PASSdeno task check-conformance— 202/202 suites, no weakened assertions, touched pages matrix-validatedIntegrator interventions worth review
demo-embed-own-featureassertion was authored pre-commit to pin the canonical CPS route, which lives at v147 (no v153 showcase route exists).v147/css-border-shapeprecedent; disclosed per page in_questions.json.Known backlog (not in this PR)
legacy-unassessedin the old shape; this run burned down 3.v151/xml-parsing-in-rust-for-non-xslt-scenariosdocuments the same feature id at its earlier milestone claim; flagged in the new page's critique for a future evidenced review (kept per the durable contract).🤖 Generated with Claude Code
https://claude.ai/code/session_016CAsC4yLd71ywz6WGAD8VB
Generated by Claude Code