Skip to content

fix(client): avoid invoking accessors during notification scan#23

Merged
kauandotnet merged 1 commit intomainfrom
codex/fix-mcp-client-accessor-scan
Apr 27, 2026
Merged

fix(client): avoid invoking accessors during notification scan#23
kauandotnet merged 1 commit intomainfrom
codex/fix-mcp-client-accessor-scan

Conversation

@kauandotnet
Copy link
Copy Markdown
Collaborator

@kauandotnet kauandotnet commented Apr 27, 2026

Summary

  • inspect prototype property descriptors while wiring @OnMcpNotification handlers
  • avoid invoking provider getters during bootstrap scanning
  • add a regression test that fails if scanning touches accessors
  • add a Changesets patch entry for @nest-mcp/client

Verification

  • pnpm --filter @nest-mcp/client test
  • pnpm --filter @nest-mcp/client typecheck
  • pnpm --filter @nest-mcp/client build
  • pnpm --filter @nest-mcp/client lint
  • pnpm changeset status --since main

@docs-page
Copy link
Copy Markdown

docs-page Bot commented Apr 27, 2026

To view this pull requests documentation preview, visit the following URL:

docs.page/btwld/nest-mcp~23

Documentation is deployed and generated using docs.page.

@kauandotnet kauandotnet self-assigned this Apr 27, 2026
@kauandotnet kauandotnet merged commit 8449edd into main Apr 27, 2026
3 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.

1 participant