-
Notifications
You must be signed in to change notification settings - Fork 0
feat(mcp): automatically configure Zenable #55
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've got 3 comments for you to consider
The PR title and description are not accurate. Here are my suggestions:
Title: feat(mcp): add automatic uv and zenable-mcp installation in post-generation hook
Description: # Add Automatic Tool Installation During Project Initialization
Changes
- Added post-generation hook to automatically install uv package manager and zenable-mcp tool
- Implemented platform-specific installation logic for Linux/macOS and Windows
- Added fallback mechanisms and error handling for installation failures
Security Considerations
- Remote script execution without verification (curl | sh pattern)
- Unpinned package installations
- Consider adding verification steps or making installations optional
Testing
Manual testing required:
- Test project generation on Linux/macOS and Windows
- Verify uv and zenable-mcp are properly installed
- Test fallback behavior when installations fail
- Validate error handling and user feedback
Reviewed with 🤟 by Zenable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've got 2 comments for you to consider
Reviewed with 🤟 by Zenable
Contributor Comments
This automatically configures the free tier Zenable AI coding guardrails
Pull Request Checklist
Thank you for submitting a contribution!
Please address the following items: