Skip to content

fix(filesystem): log effective allowed roots - #4913

Open
boss477 wants to merge 1 commit into
modelcontextprotocol:mainfrom
boss477:fix/filesystem-log-effective-roots
Open

boss477 wants to merge 1 commit into
modelcontextprotocol:mainfrom
boss477:fix/filesystem-log-effective-roots

Conversation

@boss477

@boss477 boss477 commented Sep 29, 2026

Copy link
Copy Markdown

Closes #4912

After client initialization, report the effective allowed directories and their source (server arguments or client-provided MCP Roots). When the client provides no usable roots, report that the current directories are retained and print the effective state, including when no directories are configured.

This keeps existing access-control behavior unchanged while making the final state easier for operators to diagnose.

A targeted filesystem test run is in progress; results will be added when available.

This branch has not been deployed

No deployments
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.

filesystem: clarify effective root state after initialization

1 participant