mirror of
https://github.com/ChromeDevTools/chrome-devtools-mcp.git
synced 2026-11-04 05:02:31 +00:00
Fixes #2987 ### Summary - Remove `slim` from `getCliOptions()` in `src/config/mcp-options.ts` so `chrome-devtools start --slim` is rejected in CLI strict mode, since the generated CLI requires the full tool set. - Add regression test in `tests/cli.test.ts` verifying `getCliOptions()` omits `slim`.
This commit is contained in:
1 parent
2ab93a3c04
commit
ae3f61c5b4
2 files changed
+4
No files matched your search
@@ -291,6 +291,9 @@ export function getCliOptions(): Partial<
|
||||
// Missing CLI serialization.
|
||||
delete options.viewport;
|
||||
|
||||
// The CLI is generated for the full mode, not slim mode.
|
||||
delete options.slim;
|
||||
|
||||
// Change the defaults for the CLI.
|
||||
delete options.experimentalStructuredContent;
|
||||
delete options.experimentalInteropTools;
|
||||
|
||||
@@ -946,6 +946,7 @@ describe('cli args parsing', () => {
|
||||
const cliOptions = getCliOptions();
|
||||
|
||||
assert.strictEqual(cliOptions.viewport, undefined);
|
||||
assert.strictEqual(cliOptions.slim, undefined);
|
||||
assert.strictEqual(cliOptions.experimentalStructuredContent, undefined);
|
||||
assert.strictEqual(cliOptions.experimentalInteropTools, undefined);
|
||||
|
||||
|
||||
Reference in new issue
Block a user