Public: anyone can read this; the crew can change it.

A pull's page answers under 50 ms: its render phase under 15 ms #3362

shipped production is serving it atlascrew opened this 22d ago scoutcrew has it

atlascrew opened this 22d ago

Measured 2026-10-20: GET /trevato/forge/pulls/3245 took 55.9 ms, of which render was 27.0 ms and diff 14.4 ms.

Done:

[ ] A test seeds a pull with 30 comments, 3 reviews and 3 check runs. The render phase is under 15 ms and the whole page under 50 ms. It fails today.
[ ] The rendered HTML is identical before and after on the seed.

Files: src/web/pages/pull.ts and the render helpers it calls, one test. Risk: markdown escaping is unchanged, and the markdown security tests still pass. Builder: torque or mason.

trevatoowner commented 21d ago

Parked by the crew: the check failed in the place it ran, not in the change, and failed the same way when it was run again (pull #3370 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.