Skip to content

Add security review, specific to OpenFn#4476

Open
taylordowns2000 wants to merge 4 commits intomainfrom
security-reviewer
Open

Add security review, specific to OpenFn#4476
taylordowns2000 wants to merge 4 commits intomainfrom
security-reviewer

Conversation

@taylordowns2000
Copy link
Member

@taylordowns2000 taylordowns2000 commented Feb 27, 2026

This PR adds an OpenFn-specific security reviewer, and a Github action to call it on PRs that are ready for review

Closes #4521

AI Usage

Please disclose whether you've used AI anywhere in this PR (it's cool, we just
want to know!):

  • I have used Claude Code
  • I have used another model
  • I have not used AI

You can read more details in our
Responsible AI Policy

Pre-submission checklist

  • I have performed an AI review of my code (we recommend using /review
    with Claude Code)
  • I have implemented and tested all related authorization policies.
    (e.g., :owner, :admin, :editor, :viewer)
  • I have updated the changelog.
  • I have ticked a box in "AI usage" in this PR

@github-project-automation github-project-automation bot moved this to New Issues in Core Feb 27, 2026
@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.43%. Comparing base (e7c8d51) to head (86125f5).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4476      +/-   ##
==========================================
- Coverage   89.47%   89.43%   -0.05%     
==========================================
  Files         425      425              
  Lines       20212    20212              
==========================================
- Hits        18085    18076       -9     
- Misses       2127     2136       +9     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@taylordowns2000 taylordowns2000 changed the title security review, specific to OpenFn Add security review, specific to OpenFn Feb 27, 2026
@taylordowns2000 taylordowns2000 marked this pull request as ready for review February 27, 2026 09:06
@taylordowns2000 taylordowns2000 marked this pull request as draft February 27, 2026 10:35
@taylordowns2000 taylordowns2000 marked this pull request as ready for review February 27, 2026 10:35
Copy link
Contributor

@brandonjackson brandonjackson left a comment

Choose a reason for hiding this comment

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

Very cool! Will be interesting to keep an eye on false negatives.

@taylordowns2000 taylordowns2000 requested a review from stuartc March 10, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: New Issues

Development

Successfully merging this pull request may close these issues.

Run permissions & audit trail checks on PRs

3 participants