Fix build issues: Add cross-platform support #208
Conversation
- Added platform detection to support both Windows and non-Windows environments - Made keysender dependency optional to avoid build failures on non-Windows platforms - Created a fallback mechanism using the clipboardy provider for clipboard operations - Added dummy implementations for unsupported operations with helpful error messages - Updated the README with cross-platform documentation - Fixed tests to work with the new platform-specific behavior 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
|
@MCPClaude are you there |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
12 similar comments
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
@MCPClaude are you there? |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
11 similar comments
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
@MCPClaude are you there? |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
8 similar comments
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
❌ Sorry @MCPClaude, only authorized users can trigger Claude commands. |
|
@MCPClaude are you there? |
|
❌ Sorry MCPControl, only authorized users can trigger Claude commands. |
|
@MCPClaude are you there? |
|
❌ An error occurred while processing your command. (Reference: err-iszeecpn, Time: 2025-05-20T15:02:26.231Z) Please check with an administrator to review the logs for more details. |
|
@MCPClaude are you there? |
|
❌ An error occurred while processing your command. (Reference: err-roy2ndif, Time: 2025-05-20T15:08:37.240Z) Please check with an administrator to review the logs for more details. |
|
@MCPClaude are you there? |
|
❌ An error occurred while processing your command. (Reference: err-xg9xkcyk, Time: 2025-05-20T15:12:48.454Z) Please check with an administrator to review the logs for more details. |
|
@MCPClaude are you there? |
1 similar comment
|
@MCPClaude are you there? |
|
I'm here and ready to assist you with the MCPControl repository. How can I help you with the codebase today? |
Summary
Test plan
up to date, audited 411 packages in 2s
122 packages are looking for funding
run
npm fundfor detailsfound 0 vulnerabilities on a non-Windows platform - should succeed without errors
RUN v3.1.3 /workspace/repo
✓ src/providers/registry.test.ts (5 tests) 5ms
✓ src/tools/screenshot.test.ts (3 tests) 7ms
✓ src/tools/screen.test.ts (11 tests) 11ms
✓ src/providers/clipboard/powershell/index.test.ts (11 tests) 48ms
✓ src/tools/keyboard.test.ts (10 tests) 10ms
✓ src/server.test.ts (4 tests) 10ms
✓ src/tools/mouse.test.ts (3 tests) 6ms
✓ src/tools/validation.zod.test.ts (16 tests) 22ms
↓ src/providers/factory.modular.test.ts (9 tests | 9 skipped)
✓ src/providers/keysender/index.test.ts (2 tests) 4ms
✓ src/providers/factory.test.ts (4 tests) 8ms
✓ src/handlers/tools.test.ts (13 tests) 23ms
✓ src/providers/autohotkey/index.test.ts (7 tests) 10ms
✓ src/providers/keysender/screen.test.ts (11 tests) 316ms
Test Files 13 passed | 1 skipped (14)
Tests 100 passed | 9 skipped (109)
Start at 03:02:06
Duration 740ms (transform 1.19s, setup 0ms, collect 2.58s, tests 480ms, environment 3ms, prepare 1.33s) - tests should pass (except platform-specific ones which are skipped)
🤖 Generated with Claude Code