Skip to content

docs: clarify eager MCP proxy discovery#243

Merged
kamilio merged 3 commits intomainfrom
docs/mcp-proxy-eager-discovery-238
Apr 27, 2026
Merged

docs: clarify eager MCP proxy discovery#243
kamilio merged 3 commits intomainfrom
docs/mcp-proxy-eager-discovery-238

Conversation

@poe-code-agent
Copy link
Copy Markdown
Contributor

Summary

  • Document that Toolcraft MCP proxy discovery is eager before CLI help/routing.
  • Document that cached schemas avoid repeated upstream discovery and that selective/lazy proxy discovery is not currently supported.
  • Make the memory cache malformed-JSON test tolerate Node JSON parse message location differences so the required push checks pass on this Node runtime.

Fixes #238

Verification

  • npm run build
  • npm run lint:types
  • npm run test:unit -- packages/memory/src/cache.test.ts
  • npm run test (via pre-push hook; 5919 passed, 1 skipped)

Copy link
Copy Markdown
Contributor Author

@poe-code-agent poe-code-agent Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requested changes: SDK proxy discovery timing is misstated.

Comment thread packages/toolcraft/README.md Outdated
Co-authored-by: poe-code-agent[bot] <254571472+poe-code-agent[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@poe-code-agent poe-code-agent Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kamilio kamilio merged commit b5ea785 into main Apr 27, 2026
3 of 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.

Document that MCP proxy discovery is eager before CLI help/routing

1 participant