feat: retry tool discovery up to 3 times on connection failure#6
Open
Alex647648 wants to merge 1 commit intodimensionalOS:masterfrom
Open
feat: retry tool discovery up to 3 times on connection failure#6Alex647648 wants to merge 1 commit intodimensionalOS:masterfrom
Alex647648 wants to merge 1 commit intodimensionalOS:masterfrom
Conversation
If the DimOS MCP server is temporarily unavailable during plugin registration, the plugin would silently fail to load any tools. Now discoverToolsSync is retried up to 3 times with a 2-second delay between attempts, giving the server time to start. Each failure is logged with the attempt number. Only after all retries are exhausted does the plugin give up.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
discoverToolsSyncin a retry loop (max 3 attempts, 2s delay between retries)Motivation
When the DimOS MCP server starts slower than the OpenClaw plugin host (common in development), the single-attempt discovery fails silently and the plugin never loads tools. Users must manually restart. A simple retry loop handles this common race condition.
Test plan