Skip to content

Remove RPC source configuration from indexer skills#1370

Merged
DZakh merged 1 commit into
mainfrom
claude/rpc-source-config-overview-6gqjgf
Jul 1, 2026
Merged

Remove RPC source configuration from indexer skills#1370
DZakh merged 1 commit into
mainfrom
claude/rpc-source-config-overview-6gqjgf

Conversation

@DZakh

@DZakh DZakh commented Jul 1, 2026

Copy link
Copy Markdown
Member

Removes the dedicated RPC-as-a-data-source documentation from the managed indexer skill set (packages/cli/templates/static/shared/.claude/skills/). RPC source configuration is no longer surfaced as guidance in the skills — except the WebSocket realtime section, kept by request.

Removed

  • indexer-performance — the ## RPC Tuning section (tuning params, for options table); RPC-tuning wording dropped from the frontmatter description. The WebSocket for Realtime Indexing section is kept (promoted from ### to ## now that its parent section is gone).
  • indexer-configuration — the ## RPC Configuration pointer section.
  • indexer-troubleshooting — the "rate limited / RPC tuning" error bullet; renamed RPC / HyperSync ErrorsHyperSync Errors; dropped RPC from the frontmatter description.

Intentionally left in place

These reference RPC but aren't RPC source-configuration docs:

  • indexer-external-calls / indexer-traces — handler-level RPC calls via the Effect API / viem (a different feature).
  • The ## Environment Variables sections in indexer-configuration and indexer-troubleshooting — they teach the required ENVIO_ prefix and only use an rpc: URL as the example.
  • The rpc_config deprecation note in indexer-configuration — a "do not use" warning.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated guidance for indexer-related skills to better reflect current optimization and troubleshooting topics.
    • Simplified one skill by removing an outdated configuration section.
    • Refined performance guidance to emphasize WebSocket-based realtime indexing and related optimization tips.
    • Streamlined troubleshooting content by focusing on HyperSync errors and removing RPC-specific references.

@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e1be36d3-96bf-4c75-8976-9f4536cf6a41

📥 Commits

Reviewing files that changed from the base of the PR and between fd0c3f9 and f6d20ec.

📒 Files selected for processing (3)
  • packages/cli/templates/static/shared/.claude/skills/indexer-configuration/SKILL.md
  • packages/cli/templates/static/shared/.claude/skills/indexer-performance/SKILL.md
  • packages/cli/templates/static/shared/.claude/skills/indexer-troubleshooting/SKILL.md
💤 Files with no reviewable changes (1)
  • packages/cli/templates/static/shared/.claude/skills/indexer-configuration/SKILL.md

📝 Walkthrough

Walkthrough

Documentation-only updates across three SKILL.md files remove and rename RPC-related sections in favor of HyperSync/WebSocket-focused guidance. Changes include deleting an RPC Configuration section, replacing RPC Tuning with WebSocket for Realtime Indexing content, and renaming headers/descriptions to drop explicit RPC references.

Changes

SKILL.md Documentation Updates

Layer / File(s) Summary
Remove RPC Configuration section
packages/cli/templates/static/shared/.claude/skills/indexer-configuration/SKILL.md
Removes the "## RPC Configuration" block that pointed to the indexer-performance skill.
Replace RPC Tuning with WebSocket guidance
packages/cli/templates/static/shared/.claude/skills/indexer-performance/SKILL.md
Rewrites the frontmatter description to cover HyperSync, batch size, WebSocket realtime config, and preload optimization; replaces the "RPC Tuning" section with a "WebSocket for Realtime Indexing" section.
Rename troubleshooting headers/descriptions
packages/cli/templates/static/shared/.claude/skills/indexer-troubleshooting/SKILL.md
Updates the skill description to drop the "RPC errors" mention and renames "RPC / HyperSync Errors" to "HyperSync Errors", removing the associated RPC rate-limit/timeout paragraph.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing RPC source configuration guidance from the indexer skills.
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.

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

Drop the dedicated RPC-as-a-data-source documentation from the managed
indexer skill set, keeping the WebSocket realtime section:

- indexer-performance: remove the RPC Tuning section (tuning params and
  `for` options); keep the WebSocket for Realtime Indexing section and drop
  RPC-tuning wording from the skill description
- indexer-configuration: remove the RPC Configuration pointer section
- indexer-troubleshooting: remove the RPC rate-limit/tuning error bullet,
  rename the section to HyperSync Errors, and drop RPC from the description

Handler-level RPC usage (Effect API / viem in indexer-external-calls and
indexer-traces), the ENVIO_ env-var examples, and the rpc_config
deprecation note are left intact.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014aVfvTqexSNr2V4i4MaBEB
@DZakh DZakh force-pushed the claude/rpc-source-config-overview-6gqjgf branch from e9d48b7 to f6d20ec Compare July 1, 2026 12:45
@DZakh DZakh merged commit 2185bd0 into main Jul 1, 2026
8 checks passed
@DZakh DZakh deleted the claude/rpc-source-config-overview-6gqjgf branch July 1, 2026 13:03
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