Skip to content

Comments

refactor(batch-snapshot): optimize batch processing and enhance error handling#148

Merged
QSchlegel merged 2 commits intomainfrom
feature/get-TVL
Oct 9, 2025
Merged

refactor(batch-snapshot): optimize batch processing and enhance error handling#148
QSchlegel merged 2 commits intomainfrom
feature/get-TVL

Conversation

@Andre-Diamond
Copy link
Collaborator

  • Reduced batch size from 10 to 5 for improved performance and reliability.
  • Introduced a new configurable request timeout to prevent hanging requests.
  • Enhanced input validation for batch parameters to ensure correct usage.
  • Updated error handling to include detailed wallet structure information in failure reports.
  • Improved documentation to reflect changes in batch processing and configuration options.

… handling

- Reduced batch size from 10 to 5 for improved performance and reliability.
- Introduced a new configurable request timeout to prevent hanging requests.
- Enhanced input validation for batch parameters to ensure correct usage.
- Updated error handling to include detailed wallet structure information in failure reports.
- Improved documentation to reflect changes in batch processing and configuration options.
@vercel
Copy link

vercel bot commented Oct 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
multisig Ready Ready Preview Comment Oct 9, 2025 5:51am

…h structures

- Removed hasClarityApiKey and clarityApiKeyLength fields from BatchProgress and BatchResults interfaces.
- Updated logging in BatchSnapshotOrchestrator to exclude Clarity API key information for improved clarity and security.
- Adjusted WalletFailure interface to remove Clarity API key references in the wallet structure.
@QSchlegel QSchlegel merged commit 17b088d into main Oct 9, 2025
4 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