fix: update model name from claude-sonnet to sonnet in GitHub workflows

[luxsolari]

No description provided.

[Copilot]

Pull Request Overview

This PR updates the Claude model name in GitHub workflows from 'claude-sonnet' to 'sonnet' to align with current API naming conventions.

• Updates model parameter from --model claude-sonnet to --model sonnet in two workflow files
• Maintains all other configuration options and workflow functionality

Reviewed Changes

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

File	Description
.github/workflows/claude.yml	Updates model name in claude_args parameter
.github/workflows/claude-code-review.yml	Updates model name in claude_args parameter while preserving allowed-tools configuration

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[claude]

Code Review Summary

✅ LGTM - Clean Configuration Fix

This PR correctly updates the model name parameter in both GitHub workflow files from claude-sonnet to sonnet.

✅ Strengths

1. Targeted Fix: Changes are minimal and focused - only updating the model parameter where needed
2. Consistency: Both workflow files (.github/workflows/claude-code-review.yml and .github/workflows/claude.yml) are updated consistently
3. No Breaking Changes: The change maintains backward compatibility and follows current Claude model naming conventions
4. Clean Diff: No extraneous changes or formatting issues

📝 Code Quality Assessment

• Best Practices: ✅ Follows GitHub Actions best practices
• Syntax: ✅ YAML syntax is correct
• Consistency: ✅ Both files updated identically

🔒 Security Considerations

• Low Risk: Configuration-only change with no security implications
• No Secrets Exposed: No sensitive information in the changes

⚡ Performance Impact

• Neutral: No performance impact expected from model name change
• CI/CD: Should maintain same workflow execution performance

🧪 Testing Considerations

• Manual Testing: Recommend verifying the workflows execute successfully with the new model name
• No Unit Tests Needed: Configuration changes don't require additional test coverage

💡 Suggestions

This is a clean, straightforward fix that addresses the model naming update. No additional changes recommended.

Overall Assessment: This PR successfully resolves the model naming issue and is ready for merge. 🚀

[github-actions]

Qodana Community for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

• Or via our issue tracker: https://jb.gg/qodana-issue
• Or share your feedback: https://jb.gg/qodana-discussions