8000 feat: add MCP Client configuration to install script by cfc4n · Pull Request #22 · gojue/moling · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: add MCP Client configuration to install script #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 7, 2025

Conversation

cfc4n
Copy link
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
@Copilot 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.
1 participant
0