✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Greptile Summary

Fixed a bug in the migration's raw SQL query where parameter binding was incorrect. The raw query template expected individual placeholders for each ID in the IN clause, but was receiving the entire array as a single parameter. Switched to knex query builder which handles array parameters correctly for whereIn() operations. Added @ts-expect-error directive to suppress TypeScript errors for the shouldCheckSecretPermission field that was removed in a later migration.

• Fixed parameter binding issue in UPDATE query with IN clause
• Replaced knex.raw() with standard knex query builder
• Added TypeScript error suppression for removed field

Confidence Score: 4/5

• This PR is safe to merge with minimal risk - it fixes a genuine bug in migration parameter binding
• The change correctly fixes a SQL parameter binding issue where the raw query was improperly handling array parameters. The migration remains idempotent (protected by hasColumn check) and the knex query builder is the appropriate solution. Score is 4 rather than 5 because migrations should ideally be tested in a staging environment before merging
• No files require special attention

Important Files Changed