Skip to content

feat: support configurable filesystem roots - #2605

Open
joebasrawi wants to merge 3 commits into
ChromeDevTools:mainfrom
joebasrawi:feat/configurable-filesystem-roots
Open

feat: support configurable filesystem roots#2605
joebasrawi wants to merge 3 commits into
ChromeDevTools:mainfrom
joebasrawi:feat/configurable-filesystem-roots

Conversation

@joebasrawi

Copy link
Copy Markdown

Fixes #2598

This lets people set filesystem roots through the MCP server or the Chrome DevTools CLI. Multiple roots work together with any roots sent by the MCP client, and temp directory access stays the same.

I added tests for the flags and root handling.

@google-cla

google-cla Bot commented Aug 21, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@joebasrawi
joebasrawi marked this pull request as ready for review August 21, 2026 22:11
@OrKoN
OrKoN self-requested a review August 22, 2026 06:51
Comment thread src/config/mcp-options.ts
Comment thread src/index.ts
Comment thread src/config/mcp-options.ts
@OrKoN
OrKoN self-requested a review August 23, 2026 10:03
@cursor
cursor Bot force-pushed the feat/configurable-filesystem-roots branch 2 times, most recently from c49caf0 to 1d109cd Compare August 29, 2026 00:53
joebasrawi and others added 3 commits August 31, 2026 13:17
CLI now defaults --allow-unrestricted-paths on, so an explicit
--workspace must win instead of conflicting with that default.

Co-authored-by: Joseph B <joebasrawi@users.noreply.github.com>
@cursor
cursor Bot force-pushed the feat/configurable-filesystem-roots branch from 1d109cd to 717d274 Compare August 31, 2026 13:17
@joebasrawi

Copy link
Copy Markdown
Author

Rebased onto current main (conflicts resolved). The Aug 22 notes are still in: os.tmpdir() as the yargs default, CLI roots always included with client roots, and --allow-unrestricted-paths conflicts with an explicit filesystem root (deprecated in favor of --workspace=/).

@OrKoN ready for another look when you have a minute.

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.

Add a CLI flag to configure MCP roots

2 participants