Skip to content

Add defaulter caching#884

Merged
3alpha merged 7 commits intomainfrom
add-defaulter-caching
Jan 12, 2026
Merged

Add defaulter caching#884
3alpha merged 7 commits intomainfrom
add-defaulter-caching

Conversation

@3alpha
Copy link
Copy Markdown
Collaborator

@3alpha 3alpha commented Nov 4, 2025 •
edited
Loading

Usage related changes

Load to user provided RPC endpoints should be lighter

Development related changes

Resolves #746. First implementation of caching, uses LRU cache with hardcoded capacity. By improving observability, that limit can be finetuned. For now, let's keep it fixed.

Checklist:

  • Checked out the contribution guidelines
  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/clippy_check.sh
  • No unused dependencies - ./scripts/check_unused_deps.sh
  • No spelling errors - ./scripts/check_spelling.sh
  • Performed code self-review
  • Rebased to the latest commit of the target branch (or merged it into my branch)
    • Once you make the PR reviewable, please avoid force-pushing
  • Updated the docs if needed - ./website/README.md
  • Linked the issues resolvable by this PR - linking info
  • Updated the tests if needed; all passing - execution info

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Nov 4, 2025 •
edited
Loading

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1) 
Validation error: Unrecognized key(s) in object: 'auto_resolve_threads'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

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.

@3alpha 3alpha force-pushed the add-defaulter-caching branch from 3101a35 to d2e177f Compare January 8, 2026 14:39
@3alpha 3alpha added enhancement New feature or request performance labels Jan 12, 2026
@3alpha 3alpha merged commit 351e3f8 into main Jan 12, 2026
4 checks passed
@3alpha 3alpha deleted the add-defaulter-caching branch January 12, 2026 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement New feature or request performance

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Forking could rely on caching

1 participant

@3alpha