Skip to content

Add Claude AI security review for PRs#1919

Open
velo wants to merge 2 commits intomainfrom
add-claude-security-review
Open

Add Claude AI security review for PRs#1919
velo wants to merge 2 commits intomainfrom
add-claude-security-review

Conversation

@velo
Copy link
Collaborator

@velo velo commented Mar 4, 2026

Summary

  • Add Claude AI security review GitHub Action that runs on every PR
  • Uses claude-sonnet-4-6 to review code changes for security issues only
  • Tailored security prompt for this project's tech stack

Test plan

  • Verify workflow YAML syntax is valid
  • Open a test PR to confirm the action runs
  • Verify the action uses the correct model and prompt

Signed-off-by: Marvin Froeder <marvin@datasqrl.com>
@velo velo requested a review from ferenc-csaky March 4, 2026 23:52
@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 13.01%. Comparing base (9f71918) to head (7c16f4c).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1919   +/-   ##
=========================================
  Coverage     13.01%   13.01%           
  Complexity      752      752           
=========================================
  Files           585      585           
  Lines         16552    16552           
  Branches       1920     1920           
=========================================
  Hits           2154     2154           
  Misses        14199    14199           
  Partials        199      199           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ferenc-csaky
Copy link
Collaborator

I'm reluctant about this. We can try this, but if this will be as crappy as copilot it will be just annoying noise.

@velo
Copy link
Collaborator Author

velo commented Mar 5, 2026

I'm reluctant about this. We can try this, but if this will be as crappy as copilot it will be just annoying noise.

Absolutely with you. If becomes a copilot I will take it out.

Signed-off-by: Marvin Froeder <marvin@datasqrl.com>
Copy link
Collaborator

@ferenc-csaky ferenc-csaky left a comment

Choose a reason for hiding this comment

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

Assuming CI is green

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants