Skip to content

docs: lead Quickstart with the indices file (fix stale PyPI description) - #20

Merged
CryptoFewka merged 4 commits into
mainfrom
docs/quickstart-indices-first
Jun 25, 2026
Merged

docs: lead Quickstart with the indices file (fix stale PyPI description)#20
CryptoFewka merged 4 commits into
mainfrom
docs/quickstart-indices-first

Conversation

@CryptoFewka

@CryptoFewka CryptoFewka commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

The README Quickstart renders as the PyPI project description, and it still framed the fastest path as a local beacon node (localhost:5052) + a /etc/optimum pubkeys file, implying keysync runs on the validator host.

Lead instead with KEYSYNC_INDICES_FILE (a JSON records file, no beacon needed) pointing at examples/indices.json, and present the BLS-pubkeys + beacon path as an alternative with a generic host. This matches the dashboard API-key boilerplate (#222).

Docs only. Note: PyPI descriptions are immutable per version, so this reaches pypi.org only via a new release (suggest cutting 1.1.1 after merge).

Written with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the quickstart instructions to better describe the simplest way to get started with a JSON file of records.
    • Clarified how validator tracking works, including support for using either a BLS pubkey or a bare index.
    • Added guidance that a beacon node can fill in the remaining validator details when needed.

The Quickstart (which renders as the PyPI project description) framed the
fastest path as a local beacon node + a /etc/optimum pubkeys file, implying
keysync runs on the validator host. Lead instead with KEYSYNC_INDICES_FILE (a
JSON records file, no beacon needed) pointing at the committed sample, and
present the BLS-pubkeys + beacon path as an alternative with a generic host.
Matches the dashboard's API-key boilerplate.
@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@CryptoFewka, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 44 minutes and 18 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository: getoptimum/coderabbit/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8c75c4f1-90fa-468e-a865-ce25e583dd9a

📥 Commits

Reviewing files that changed from the base of the PR and between 176bf3d and 58a9f1c.

⛔ Files ignored due to path filters (1)
  • LICENSE is excluded by none and included by none
📒 Files selected for processing (1)
  • src/optimum_keysync/__init__.py
📝 Walkthrough

Walkthrough

The README Quickstart was updated. It now describes starting keysync from a JSON records file, mentions the indices file environment variable, and adds guidance for tracking validators by BLS pubkey or bare index, including a beacon-node-based example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title is relevant, but it exceeds 72 chars and does not follow the required docs(<domain/pkg>): format. Use docs(readme): lead Quickstart with the indices file.
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Description check ✅ Passed The PR objectives clearly match the README-only Quickstart rewrite and PyPI-description note.
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.


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

pyproject declared the MIT license but the grant text wasn't in the repo.
Add a standard MIT LICENSE (Copyright (c) 2026 Optimum); setuptools includes
it in the sdist and wheel metadata (License-File: LICENSE).
@CryptoFewka
CryptoFewka merged commit b11e61b into main Jun 25, 2026
3 checks passed
@CryptoFewka
CryptoFewka deleted the docs/quickstart-indices-first branch June 25, 2026 21:04
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