Skip to content

docs: clarify Hyperliquid real-price setup and SSL fix#36

Merged
squaeragent merged 3 commits into
zero-intel:mainfrom
J4h5u5:docs/hyperliquid-real-prices-readme
May 6, 2026
Merged

docs: clarify Hyperliquid real-price setup and SSL fix#36
squaeragent merged 3 commits into
zero-intel:mainfrom
J4h5u5:docs/hyperliquid-real-prices-readme

Conversation

@J4h5u5
Copy link
Copy Markdown
Contributor

@J4h5u5 J4h5u5 commented May 5, 2026

Summary

Add a short troubleshooting note to engine/README.md under the --hyperliquid-live-prices section.

It explains what to do when /market/quote returns 503 with CERTIFICATE_VERIFY_FAILED, so users can read real Hyperliquid mids in paper mode.

Tests

Docs-only change. No runtime behavior changes.

Safety Impact

No execution, sizing, credentials, auth, risk gate, or operator command logic changed.

AI Assistance

  • No.
  • Yes, partially. I reviewed the diff and tests.
  • Yes, mostly. I reviewed the diff and tests.

Checklist

  • I ran the relevant tests.
  • I updated docs when behavior changed.
  • I did not add real credentials or private runtime data.
  • If AI-assisted, I included the relevant context packet and safety invariant in the PR body.

Context packet: user hit /market/quote 503 while using --hyperliquid-live-prices; API error showed Python SSL certificate validation failure (CERTIFICATE_VERIFY_FAILED).

Safety invariant: paper mode stays default and fail-closed; read-only market data setup must not weaken custody or enable live order placement.

Summary by CodeRabbit

Documentation

  • Documentation
    • Added comprehensive QA onboarding checklist with standardized validation workflows, regression test procedures, safety-critical invariant verification, privacy checks, and escalation guidelines for runtime quality assurance.
    • Added troubleshooting guide addressing SSL certificate verification failures in local Python development environments with dependency installation steps, configuration details, and service restart procedures.

J4h5u5 and others added 2 commits May 4, 2026 21:15
Add a practical onboarding checklist for technical/QA engineers covering baseline checks, safety regression slices, MCP/privacy verification, PR review flow, invariant audit points, and escalation rules.

Co-authored-by: Codex <codex@openai.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
@J4h5u5 J4h5u5 requested a review from squaeragent as a code owner May 5, 2026 13:47
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 77239cd9-8eaf-434c-9d75-7d4c4be1d997

📥 Commits

Reviewing files that changed from the base of the PR and between de206f9 and 5380cca.

📒 Files selected for processing (2)
  • docs/qa-onboarding-checklist.md
  • engine/README.md

📝 Walkthrough

Walkthrough

This PR adds two independent documentation updates: a new QA onboarding checklist for validating ZERO in a safety-critical runtime, and a troubleshooting note in the engine README for resolving certificate verification failures in /market/quote requests.

Changes

QA Onboarding Checklist

Layer / File(s) Summary
New Documentation
docs/qa-onboarding-checklist.md
Introduces a repeatable QA onboarding workflow with ordered steps for environment/bootstrap verification, core baseline tests, paper API smoke testing, safety-critical regression slices (live behavior, reconciliation, MCP, privacy, journal integrity, anchor cadence), MCP/privacy verification commands, PR review guidance, invariant fast-audit items, and escalation rules with required reporting details.

README Troubleshooting

Layer / File(s) Summary
Troubleshooting Guidance
engine/README.md
Adds instructions for resolving /market/quote failures returning 503 with CERTIFICATE_VERIFY_FAILED by installing certifi, exporting SSL_CERT_FILE environment variable, and restarting the zero-paper-api service.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A checklist hops in, so thorough and bright,
QA paths now cleared for rigorous might,
While README whispers of certs gone astray—
Two docs dance together to light up the way! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title does not accurately represent the main changes. The PR adds a QA onboarding checklist as the primary change, but the title only mentions Hyperliquid SSL fix and setup, which is secondary. Revise the title to reflect the primary change: 'docs: add QA onboarding checklist and Hyperliquid SSL troubleshooting' or similar to capture both significant additions.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@squaeragent squaeragent left a comment

Choose a reason for hiding this comment

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

Approved: docs-only contribution, no runtime or safety-boundary changes, all checks green.

@squaeragent squaeragent enabled auto-merge (squash) May 6, 2026 10:30
@squaeragent squaeragent merged commit 2813065 into zero-intel:main May 6, 2026
29 checks passed
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