Skip to content

feat: add MCP Client configuration to install script#22

Merged
cfc4n merged 1 commit intomasterfrom
failed-to-run-on-windows
Apr 7, 2025
Merged

feat: add MCP Client configuration to install script#22
cfc4n merged 1 commit intomasterfrom
failed-to-run-on-windows

Conversation

@cfc4n
Copy link
Copy Markdown
Member

@cfc4n cfc4n commented Apr 7, 2025

This pull request includes changes to the install/install.ps1 and services/file_system_windows.go files. The changes focus on adding MCP Client configuration steps and initializing the allowedDirsDefault variable in the Windows file system service.

Configuration and Initialization:

  • install/install.ps1: Added MCP Client configuration step by invoking moling.exe with the client --install command.
  • services/file_system_windows.go: Added a new initialization function to set the allowedDirsDefault variable using the user's cache directory or home directory. This change also includes adding a copyright notice and licensing information.

These changes ensure proper configuration of the MCP Client during installation and improve the initialization of the file system service in a Windows environment.

…system_windows.go for cache directory handling

Signed-off-by: CFC4N <cfc4n.cs@gmail.com>
@cfc4n cfc4n added bug Something isn't working enhancement New feature or request labels Apr 7, 2025
@cfc4n cfc4n requested a review from Copilot April 7, 2025 17:02
@cfc4n cfc4n linked an issue Apr 7, 2025 that may be closed by this pull request
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 7, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • install/install.ps1: Language not supported

@cfc4n cfc4n merged commit 43cce9e into master Apr 7, 2025
6 checks passed
@cfc4n cfc4n deleted the failed-to-run-on-windows branch April 7, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failed to run on Windows.

2 participants