Skip to content

fix: add eval timeout handling#5

Open
WhiteGiverMa wants to merge 1 commit into
tugcantopaloglu:mainfrom
WhiteGiverMa:fix/eval-timeout
Open

fix: add eval timeout handling#5
WhiteGiverMa wants to merge 1 commit into
tugcantopaloglu:mainfrom
WhiteGiverMa:fix/eval-timeout

Conversation

@WhiteGiverMa
Copy link
Copy Markdown

Summary

  • add runtime-side timeout handling for game_eval so long-running evals return a clear error instead of leaving the eval node hanging
  • expose an optional imeoutSeconds parameter in the MCP tool and forward it to Godot while keeping the client wait slightly longer than the runtime timeout
  • cover the new timeout mapping in handler tests

Verification

  • npm test
  • npm run build

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 9, 2026

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