-
Notifications
You must be signed in to change notification settings - Fork 177
Fix windows claude cli #18
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -157,6 +157,27 @@ cd claude-code-security-review | |||||||||||||||||||||||||||
pytest claudecode -v | ||||||||||||||||||||||||||||
``` | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
## Windows setup | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
To run the Claude Code Security Review on Windows: | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
1. Install Node.js (v18 or later) and npm: | ||||||||||||||||||||||||||||
- Download from https://nodejs.org or use nvm-windows. | ||||||||||||||||||||||||||||
2. Install the Claude CLI:npm install -g @anthropic-ai/claude-code | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
3. Add npm global directory to PATH:$npmPath = npm config get prefix | ||||||||||||||||||||||||||||
[Environment]::SetEnvironmentVariable("Path", [Environment]::GetEnvironmentVariable("Path", "User") + ";$npmPath", "User") | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
4. Set PowerShell execution policy:Set-ExecutionPolicy -Scope CurrentUser -ExecutionPolicy RemoteSigned | ||||||||||||||||||||||||||||
5. Verify installation:claude --version | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The PowerShell commands are not properly formatted as code blocks and appear as continuous text. These should be separate code blocks with proper formatting.
Suggested change
Copilot uses AI. Check for mistakes. Positive FeedbackNegative Feedback There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing proper formatting for the PowerShell command. Should be in a code block for better readability.
Suggested change
Copilot uses AI. Check for mistakes. Positive FeedbackNegative Feedback There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The command should be formatted as a code block for consistency with documentation standards.
Suggested change
claude --version
Copilot uses AI. Check for mistakes. Positive FeedbackNegative Feedback |
||||||||||||||||||||||||||||
Expected output: 1.0.71 (Claude Code) or similar. | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
Troubleshooting | ||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing proper markdown heading formatting. Should be '### Troubleshooting' to maintain documentation hierarchy.
Suggested change
Copilot uses AI. Check for mistakes. Positive FeedbackNegative Feedback |
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
Claude CLI not found: Ensure claude.ps1 or claude.cmd is in PATH and execution policy is RemoteSigned. | ||||||||||||||||||||||||||||
Test failures: Verify npm is installed and check logs in github_action_audit.py for errors. | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
## Support | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
For issues or questions: | ||||||||||||||||||||||||||||
|
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.
The installation command is missing proper formatting and line breaks. It should be formatted as a code block for clarity.
Copilot uses AI. Check for mistakes.