Skip to content

Let the page check target a deployed URL - #19

Merged
alexandrefimov merged 1 commit into
mainfrom
page-check-url
Aug 11, 2026
Merged

Let the page check target a deployed URL#19
alexandrefimov merged 1 commit into
mainfrom
page-check-url

Conversation

@alexandrefimov

Copy link
Copy Markdown
Owner

Summary

bench/check_page.py hardcoded the local server URL, so the same gate CI runs
against a fresh build could not be pointed at the live site. It now reads
QUERY_DOCTOR_PAGE_URL and falls back to the local build.

Used it to verify the deployment from #18 end to end against
https://alexandrefimov.github.io/Query-Doctor/ : boot 1898 ms on a cold cache,
103 KiB sample profile analyzed in 158 ms, 8 requests, no external hosts, no JS
errors.

Type

  • Tests only
  • Documentation

Safety Checklist

  • No raw production data, paths, hostnames, or secrets introduced.
  • No product behavior changes; this touches a check harness and its docs.

Validation

  • ruff check web/bench
  • Gate run against the deployed site: pass
  • Gate run against a local build: pass

Branch Hygiene

  • Targets a review branch, based on current main.

Assisted-by: Claude Opus 5 via Claude Code

@alexandrefimov
alexandrefimov merged commit 2f3243c into main Aug 11, 2026
12 checks passed
@alexandrefimov
alexandrefimov deleted the page-check-url branch August 11, 2026 11:49
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