1254 remove run fks

[rorymckinley]

Notes for the reviewer

This is a WIP and requires #1453 to be merged in before this PR can be completed. If #1453 is merged in, the shim migration can be reverted.

At the moment, I think all failing tests are due to #1453 not being merged in.

Related issue

Fixes #

Review checklist

• I have performed a self-review of my code
• I have verified that all appropriate authorization policies have been implemented and tested
• If needed, I have updated the changelog
• Product has QA'd this feature

[taylordowns2000]

@rorymckinley , @stuartc , can i close this?

[brandonjackson]

Closing due to inactivity @rorymckinley - please reopen if this is in error

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.71%. Comparing base (70a2023) to head (0979a95).
⚠️ Report is 1495 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1522   +/-   ##
=======================================
  Coverage   87.71%   87.71%           
=======================================
  Files         218      218           
  Lines        6593     6596    +3     
=======================================
+ Hits         5783     5786    +3     
  Misses        810      810           

☔ 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.