Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 19, 2025

  • Fix existing clippy warnings in test files
  • Abstract platform-specific code into traits
  • Implement Windows executor for process spawning
  • Add PowerShell and CMD interpreter support
  • Handle Windows path separators and environment variables
  • Test on Windows CI environment
  • Update documentation for Windows users

Initial analysis shows the repository already has some Windows-specific handling in src/process.rs using cfg!(target_os = "windows") to choose between cmd /C and sh -c. We need to abstract this into a proper trait-based architecture as outlined in the ROADMAP.md.

Current Windows handling:

  • Basic Windows command execution using cmd /C
  • Path handling in plugins (but Unix-specific separators in some areas)
  • Cross-platform testing setup in integration tests

Fixes #60.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

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.

Milestone 1.4: Windows Support (Week 7-8)
2 participants