Implement auto-open feature for browser launch on server start
Motivation and Context
This eliminates the manual step of copying URLs and opening browsers, making the development workflow more efficient.
How Has This Been Tested?
Tested on macOS environments with various browsers (Chrome, Firefox, Safari). Verified that the feature respects user configuration and can be disabled via configuration options.
Breaking Changes
No breaking changes. The feature is enabled by default but can be disabled through configuration.
Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
- [ ] Documentation update
Checklist
- [x] I have read the [MCP Documentation](https://modelcontextprotocol.io)
- [x] My code follows the repository's style guidelines
- [x] New and existing tests pass locally
- [x] I have added appropriate error handling
- [x] I have added or updated documentation as needed
@cliffhall I've commited
This is a double negative that made me have to think about its validity for a second. Could it instead be "Enable Browser Auto-open" and default to true?
@cliffhall Review please
@cliffhall Review please
@kawakamidev It works fine for opening, but the UI configuration field doesn't change the behavior. If I set it to false then close the inspector, when I run the inspector again, it just auto launches.
@kawakamidev It works fine for opening, but the UI configuration field doesn't change the behavior. If I set it to false then close the inspector, when I run the inspector again, it just auto launches.
The problem here is that for this to work, the setting can't just be stored in the browser's local storage, it also has to write back to a locally stored configuration that will be consulted before the app launches next time. That's tough, since we don't currently have such a file and plumbing backwards from the client to write it would be a pain.
Perhaps we should trim this back to the place you originally had it. Just the auto-open, no UI affordance.
OK
@cliffhall I have reverted the changes!