Skip to content

Replay the Explorer comparison views and prompt catalog from core through d16a0761b - #29

Merged
joeharris76 merged 1 commit into
mainfrom
fix/site-replay-compare-views
Oct 14, 2026
Merged

joeharris76 merged 1 commit into
mainfrom
fix/site-replay-compare-views

Conversation

@joeharris76

Copy link
Copy Markdown
Contributor

Seeded-path replay of core 9adc8ef38 (BenchBox-dev/benchbox#2922) and d16a0761b (BenchBox-dev/benchbox#2974).

Core 9adc8ef38 broadens the Results Explorer comparison views:

  • Every run list now uses one compare button. It shows "Compare N selected" when runs are selected, and "Compare All Runs" when 2 to 10 runs match.
  • The compare page puts the summary charts first. The latency and speedup charts use one marker shape per platform and fit their axes to the data. Diverging bars are an alternative view of speedup.
  • A per-query page compares runs iteration by iteration and links to the query plans.
  • Normalized cost now shows list-price estimates for Snowflake and Databricks.
  • Run receipts show query-ID repairs and say when plans were recorded but not published.
  • Dashboards are removed.

The data side of these changes is already in the published bundle. This replay brings the matching UI.

Core d16a0761b updates the landing prompt catalog. The site copy now matches core's generated catalog, which also brings InfluxDB's managed deployment from core 5be6b2f61 into the site.

Site adaptations, recorded in inventory/replay-allowlist.json:

  • explorer/src/pages/Compare.tsx, explorer/src/pages/Query.tsx, explorer/src/lib/queryRunComparison.ts and explorer/src/components/RunReceipt.tsx resolve URLs against SITE_ORIGIN, not a hard-coded https://benchbox.dev, so preview and local builds keep their own origin. The repository-boundary check requires this.
  • explorer/e2e/captures/followup-usability-release.spec.ts keeps writing to explorer/captures, because the site has no _project/audits tree.

scripts/replay-fidelity.mjs used to report a file deleted in both core and the site as blocking. It now reports it as removed, and a test covers both cases.

Verified locally against bundle site-inputs-9adc8ef38 (sha256:55a41b19…):

  • replay-fidelity.mjs --since 06b65fa78 --ref d16a0761b: 154 of 154 files clean.
  • These checks pass: script tests, deploy tests, deploy tsc, lint:comments, lint:guard, bundle:verify --offline, explorer typecheck and e2e typecheck, website check, explorer and website unit tests, and the build.

…ough d16a0761b

Mirror core 9adc8ef38, which broadens the Results Explorer comparison
views, adds a per-query run comparison, shows list-price cost estimates
for Snowflake and Databricks and retires Dashboards, and core d16a0761b,
which updates the landing prompt catalog. The bundle already carries the
data these views read.

Four files resolve URLs against SITE_ORIGIN instead of the hard-coded
production origin, and one capture spec keeps the site's captures path;
each is listed with its reason in inventory/replay-allowlist.json.

replay-fidelity now treats a file removed in both core and the site as
clean, since that is a faithful replay of a core deletion.
@joeharris76
joeharris76 merged commit 816363f into main Oct 14, 2026
7 of 8 checks passed
@joeharris76
joeharris76 deleted the fix/site-replay-compare-views branch October 14, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant