fix(website): reframe streaming claims as capability-anchored after S5 cross-validation#129
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Vercel preview readyPreview: https://pretable-16bz2io4z-cacheplane.vercel.app Updated automatically by the |
4 tasks
blove
added a commit
that referenced
this pull request
May 11, 2026
* docs(plans): B2 follow-up — homepage interaction wedge refresh plan Six-task plan: aggregator script + milestone summary, ComparisonTable rows + labels, tests, /bench Interactions section, repo-memory, gates + PR (no auto-merge). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * chore(bench): aggregator for sort/filter milestone summary * fix(website): ComparisonTable adds 3 interaction rows + trail-marker label refresh * test(website): update ComparisonTable trail-marker regex assertions Three labels changed in the prior commit (AG Grid + TanStack + MUI X) to reflect the interaction wedge from PR #131. Regex assertions updated to match the new label phrasings: - AG Grid: "1.7× slower scroll, 3× slower interaction; row-height drift" - TanStack: "Headless; ~2× slower interaction (filter-metadata ties pretable)" - MUI X: "Scroll-p95 parity; 2× slower interaction" The pretable "Recommended path" assertion is unchanged. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * feat(website): /bench page Interactions section with sort + filter comparator data Replaces the placeholder paragraph that said "comparative interaction evidence is on the roadmap" with a real section paralleling the H1 scroll layout. Driven by the new aggregated milestone summary at status/milestones/2026-05-10-b2-sort-filter-summary.json (built from PR #131's per-run summaries). New loader (loadInteractionSummary) + verdict helper (interactionVerdictFor) mirror the existing scroll-side patterns. The verdict helper computes per-script ratios against the fastest adapter and annotates the TanStack filter-metadata tie inline ("2.1–2.6× slower (filter-metadata ties pretable)"). Per-adapter latency table + two prose paragraphs match the H1 section's structure for visual consistency. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * docs(research): repo-memory entry for homepage interaction wedge refresh Documents the three editorial surfaces touched (ComparisonTable rows + trail-markers, /bench Interactions section, aggregated summary file), the deliberate non-goals (ReceiptsBand owned by PR #129, n=20 follow-up deferred), and the remaining open threads from B2. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * chore: prettier-format milestone JSON + plan/spec docs Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
…5 cross-validation PR #128's S5/S7 cross-validation matrix surfaced a finding: AG Grid Community matches pretable on every measured streaming numeric (frame p95, 25k/sec envelope, visible-row drift). The homepage's stub-era "purpose-built streaming pipeline" framing — and the implication that pretable is uniquely fast at streaming — is no longer supportable on hypothesis-scale numerics. The honest wedge is package surface: pretable ships the SSE → partial-JSON → batcher → applyTransaction pipeline as a single import; AG Grid expects you to wire it yourself. Three editorial edits: - ComparisonTable.tsx: streaming row renamed from "purpose-built streaming pipeline" to "streaming pipeline (SSE → partial JSON → batcher → applyTransaction)" — same yes/n/a/n/a/n/a shape, sharper capability claim. Header docblock updated to cite the S5/S7 cross-validation milestone alongside the existing B2 sources. - ReceiptsBand.tsx: replaced the "25k/s · max sustained update rate" hero stat (no longer pretable-unique) with "OpenAI · Anthropic · SSE · streaming sources, one import". Added a `compact: true` flag to the Stat interface so the longer label renders at 20–24 px instead of 44–56 px, preserving the four-cell grid without overflowing the hero font scale. - FeatureGrid.tsx: Stream-aware card — dropped "sustained from 100 to 25,000 updates/sec" tail; rewrote the description around the pipeline that ships as one import. Test added: ReceiptsBand.test.tsx regression-guards the new capability anchor (`streaming sources` + `openai`). Repo-memory entry appended (B2 follow-up #7); MEMORY.md index updated; project_b2_followups.md regenerated to reflect everything resolved except item #5 (open comparator interaction scripts). No source/package changes outside apps/website + the docs entry; all 190+ website tests pass. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
999679a to
964e2af
Compare
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.
Summary
PR #128's S5/S7 cross-validation matrix found that AG Grid Community matches pretable on every measured streaming numeric (frame p95 9.2 ms each, 25k/sec envelope each, AG Grid drift 0 vs pretable drift 1). The homepage's stub-era streaming claims — including the "25k/s · max sustained update rate" hero receipt and the vague "purpose-built streaming pipeline" comparison row — are no longer supportable as comparative numerics. This PR moves the streaming wedge to package surface: pretable ships the SSE → partial-JSON → batcher → applyTransaction pipeline as a single import; AG Grid expects you to wire that yourself.
No auto-merge. Editorial / prose changes — please review the wording before merging.
What changed
Prose draft for review
ComparisonTable streaming row
`streaming pipeline (SSE → partial JSON → batcher → applyTransaction)`: `yes` / `n/a` / `n/a` / `n/a`
ReceiptsBand fourth stat
`OpenAI · Anthropic · SSE` (small font, compact rendering) — caption `streaming sources, one import`.
FeatureGrid Stream-aware card
(Trail-marker label `Advanced — bring your own SSE` retained — pretable ships the post-SSE pipeline; the SSE source itself is consumer-supplied.)
What's NOT in this PR
Test plan
🤖 Generated with Claude Code