Repository navigation
docs(evidence): re-verify render-blocking finding b8f6046e942a against current main and live (2026-08-11) #115
Changes from 10 commits
b00065f
b77690b
a4a5633
2eeca52
2543f1a
d13f76f
c3ab4e0
a92dbb4
76e5290
ed96dad
4114349
5c18f09
30b8e47
6911ea0
1380bce
17e7ab1
88a366e
23cb0b7
02f150a
a891e69
4ff4553
9b13c33
eacd77f
117bf7b
14fce2a
b64a39b
64448d0
9a5aaa5
947400c
d5866f5
819a578
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -221,3 +221,57 @@ b8f6046e942a ("Render-blocking resources on home") remains closed on the code | |||||||||||||||||||||||
| side (PRs #20 and #23), in CI (`npm run check:render-blocking`), and against | ||||||||||||||||||||||||
| the deployed site; this lane (2026-08-10) re-confirmed all three on the | ||||||||||||||||||||||||
| current head and found nothing further to change. | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| ### Closeout re-verification (added 2026-08-11) | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Re-verified against the current origin/main head (9302611, "fix(public): serve | ||||||||||||||||||||||||
| rel=icon favicon on every page so browsers stop 404ing /favicon.ico", merged | ||||||||||||||||||||||||
| 2026-08-11) after the commits that landed since the 2026-08-10 closeout — | ||||||||||||||||||||||||
| notably 9302611 (rel=icon favicon link), d4a2c30 (intake-field labels and | ||||||||||||||||||||||||
| document titles), 37ddaed (wrangler toolchain upgrade, no public assets), | ||||||||||||||||||||||||
| 2ae7504 (search-intent bridge), 1e78ecf and 5ab84ea (retired-Agent-Desk | ||||||||||||||||||||||||
| worker/check-site corrections), 6f85c61 and 0ad7481 (CSS layout/tap-target | ||||||||||||||||||||||||
| fixes), 1cc7a4e (canonicals and JSON-LD @ids), f9f0b0f (homepage footer link), | ||||||||||||||||||||||||
| plus the docs-only re-verification receipts. None of them introduced a | ||||||||||||||||||||||||
| render-blocking shape; the full test suite passes on this head (`npm test`, | ||||||||||||||||||||||||
| 92 tests across heading hierarchy, sitemap, worker contract, UI contract and | ||||||||||||||||||||||||
| product contract, plus the static source guards — "TinyStudio.io checks | ||||||||||||||||||||||||
|
Comment on lines
+236
to
+238
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
At the explicitly named revision 9302611, the five Node test files contain 6 heading, 7 sitemap, 42 worker, 16 UI, and 8 product-contract tests—79 total, not 92. Consequently this reported result cannot be reproduced on the revision the receipt claims to verify and suggests the output came from a different checkout; record the actual output from 9302611 or rerun the suite on the intended revision. Useful? React with 👍 / 👎. |
||||||||||||||||||||||||
| passed"). Two fresh measurements: | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| 1. `npm run check:render-blocking` on the current working tree at 9302611 | ||||||||||||||||||||||||
| (real Chromium, production CSP, css2 intercepted and delayed 2500ms, | ||||||||||||||||||||||||
| stubbed response): all six pages PASS — css2 non-blocking, | ||||||||||||||||||||||||
| first-contentful-paint never waits for it (homepage 224ms, audit 132ms, | ||||||||||||||||||||||||
| desk 216ms, pricing 176ms, specimen 160ms, brief-requested 160ms; the | ||||||||||||||||||||||||
| css2 response arrives at 2500ms, i.e. ~2.3s after first paint), no | ||||||||||||||||||||||||
| render-blocking resources other than the site's own same-origin | ||||||||||||||||||||||||
| stylesheets, promoted sheet applied. | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| 2. Live re-measurement of the deployed pages in real Chromium (unthrottled), | ||||||||||||||||||||||||
| served with the production CSP header emitted by the worker (verified via | ||||||||||||||||||||||||
| `curl -sI https://tinystudio.io/`): | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| | Page | css2 renderBlockingStatus | FCP (ms) | css2 responseEnd (ms) | Render-blocking resources | Fonts load (Karla / Fraunces) | Promoted sheet applied | | ||||||||||||||||||||||||
| |---|---|---|---|---|---|---| | ||||||||||||||||||||||||
| | index.html (home) | non-blocking | 388 | 473 | same-origin index.css only | yes / yes | yes | | ||||||||||||||||||||||||
| | audit.html | non-blocking | 216 | 82 | same-origin shared.css, audit.css | yes / yes | yes | | ||||||||||||||||||||||||
| | agents.html | non-blocking | 188 | 69 | same-origin shared.css, agents.css | yes / yes | yes | | ||||||||||||||||||||||||
| | pricing.html | non-blocking | 212 | 46 | same-origin shared.css, pricing.css | yes / yes | yes | | ||||||||||||||||||||||||
| | specimen.html | non-blocking | 172 | 61 | same-origin specimen.css, shared.css | yes / yes | yes | | ||||||||||||||||||||||||
| | brief-requested.html | non-blocking | 212 | 56 | same-origin shared.css, brief-requested.css | yes / yes | yes | | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| On the unthrottled live run the preloaded css2 (a non-blocking style preload, | ||||||||||||||||||||||||
| fetched at preload priority from the first byte) can finish before | ||||||||||||||||||||||||
| first-contentful-paint lands on the homepage (388ms FCP vs 473ms css2 end) — | ||||||||||||||||||||||||
| the same ordering the 2026-08-09 and 2026-08-10 runs showed (692 vs 491 and | ||||||||||||||||||||||||
| 1036 vs 631 on home). That is timing, not blocking: the deterministic | ||||||||||||||||||||||||
|
Comment on lines
+263
to
+267
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The stated ordering contradicts both the new measurement and the cited earlier runs: on 2026-08-11, FCP at 388 ms occurs before the css2 response ends at 473 ms, while the earlier 692/491 and 1036/631 measurements show css2 finishing before FCP. As written, the evidence receipt interprets the new numbers backward and incorrectly calls these opposite orderings the same, so either the table values or this explanation needs correction. Useful? React with 👍 / 👎. |
||||||||||||||||||||||||
| delayed-css2 run above paints ~2.3s before the css2 response arrives, so | ||||||||||||||||||||||||
| first paint cannot be waiting on it. | ||||||||||||||||||||||||
|
Comment on lines
+263
to
+269
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Correct the live timing explanation.
Proposed wording- On the unthrottled live run the preloaded css2 ... can finish before
- first-contentful-paint lands on the homepage (388ms FCP vs 473ms css2 end) —
- the same ordering the 2026-08-09 and 2026-08-10 runs showed ...
+ On the unthrottled live run, the homepage reached first-contentful-paint
+ 85ms before css2 completed (388ms FCP vs 473ms css2 end), unlike the
+ 2026-08-09 and 2026-08-10 runs. That timing is not evidence of blocking;
+ the deterministic delayed-css2 run above establishes the non-blocking behavior.📝 Committable suggestion
Suggested change
🧰 Tools🪛 LanguageTool[grammar] ~265-~265: Ensure spelling is correct (QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Same result as every earlier pass: no render-blocking scripts, no | ||||||||||||||||||||||||
| render-blocking external resources, first paint never waits for the font | ||||||||||||||||||||||||
| stylesheet, fonts still load and apply under the production CSP. Finding | ||||||||||||||||||||||||
| b8f6046e942a ("Render-blocking resources on home") remains closed on the code | ||||||||||||||||||||||||
| side (PRs #20 and #23), in CI (`npm run check:render-blocking`), and against | ||||||||||||||||||||||||
| the deployed site; this lane (2026-08-11) re-confirmed all three on the | ||||||||||||||||||||||||
| current head and found nothing further to change. | ||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This receipt calls 9302611 the current
origin/mainhead, but the reviewed commit is based on 883e106, which already contains nine later commits. In particular, 18128e8 changesbrief-requested.html, one of the six measured pages, and later commits changeaudit.html; therefore the recorded browser and live measurements do not verify the code this commit is documenting as the current head. Re-run the checks against the actual base or explicitly limit the conclusion to 9302611.Useful? React with 👍 / 👎.