Skip to content

Harden documentation deployment verification - #33

Merged
stefan-jansen merged 1 commit into
mainfrom
agent/fix-doc-release-verification
Aug 9, 2026
Merged

stefan-jansen merged 1 commit into
mainfrom
agent/fix-doc-release-verification

Conversation

@stefan-jansen

Copy link
Copy Markdown
Contributor

What changed

  • move public documentation identity verification into a tested CI script
  • send an explicit release-verifier user agent accepted by ml4trading.io
  • use a distinct cache-busting query for each retry
  • report the last observed payload or request error on failure

Root cause

The stable release deployed both documentation trees successfully, but Python's default urllib
user agent receives HTTP 403 from ml4trading.io. The inline workflow swallowed that response and
timed out without diagnostics.

Impact

Stable releases can verify the deployed revision before publishing the already-qualified candidate
to PyPI.

Validation

  • uv run pytest -q (408 passed)
  • uv run ruff check and uv run ruff format --check on changed Python files
  • uv run ty check scripts/ci/verify_docs_deployment.py
  • end-to-end verifier request against the deployed 0.1.0 stable and versioned markers

@stefan-jansen
stefan-jansen marked this pull request as ready for review August 9, 2026 13:19
Copilot AI lite review requested due to automatic review settings August 9, 2026 13:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@stefan-jansen
stefan-jansen merged commit a90e7a0 into main Aug 9, 2026
21 checks passed
@stefan-jansen
stefan-jansen deleted the agent/fix-doc-release-verification branch August 11, 2026 15:58
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.

2 participants