Skip to content

feature: windows tray support#74

Merged
Dumbris merged 3 commits into
smart-mcp-proxy:mainfrom
DarkGenius:feature/windows-tray-support
Oct 23, 2025
Merged

feature: windows tray support#74
Dumbris merged 3 commits into
smart-mcp-proxy:mainfrom
DarkGenius:feature/windows-tray-support

Conversation

@DarkGenius

Copy link
Copy Markdown
Contributor

This PR adds Windows support for the mcpproxy-tray application, enabling system tray functionality on Windows platforms.

Changes Made

  • Build System: Extended build tags from darwin to (darwin || windows) for tray components
  • Process Monitoring: Added Windows-specific process monitoring (process_windows.go) with proper exit code handling
  • Cross-platform UI: Made OpenWebUI work on both macOS and Windows using appropriate system calls
  • Binary Resolution: Fixed executable detection for Windows .exe files
  • Port Conflict Resolution: Added automatic port selection when default port is unavailable

@DarkGenius DarkGenius changed the title Feature/windows tray support feature: windows tray support Oct 12, 2025
@Dumbris

Dumbris commented Oct 12, 2025

Copy link
Copy Markdown
Member

Hey, @DarkGenius thank you for your contribution!
Could you rebase your PR branch on latest main? I have fixed e2e tests and pr-build workflows.

Previously, stderr monitoring started only after successful MCP initialize() handshake. This caused startup errors (like missing API keys or credentials) to be invisible in logs when initialization timed out.

Now stderr monitoring begins immediately after process start, before the initialize() call. This ensures all process output is captured and logged, making debugging connection failures much easier.
@DarkGenius DarkGenius force-pushed the feature/windows-tray-support branch from a5d8b46 to c3a3e41 Compare October 22, 2025 06:41
- Extend build tags to support Windows (darwin || windows)
- Add Windows-specific process monitoring (process_windows.go)
- Implement cross-platform OpenWebUI with Windows support
- Update binary resolution for .exe files on Windows
- Add automatic port conflict resolution
@DarkGenius DarkGenius force-pushed the feature/windows-tray-support branch from c3a3e41 to a64e289 Compare October 22, 2025 07:05

@Dumbris Dumbris left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a ton for this update!

@Dumbris Dumbris merged commit 83f0f25 into smart-mcp-proxy:main Oct 23, 2025
34 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.

2 participants