[ENG-4178] chore: removes shouldCheckSecretPermission column

[gupta-piyush19]

Description 📣

removes shouldCheckSecretPermission column

Type ✨

• Bug fix
• New feature
• Improvement
• Breaking change
• Documentation

Tests 🛠️

# Here's some code block to paste some code snippets

---

• I have read the contributing guide, agreed and acknowledged the code of conduct. 📝

[maidul98]

✅ 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-apps]

Greptile Summary

• Removes unused shouldCheckSecretPermission column from SecretApprovalPolicy table after logic was refactored in September to check reviewer status directly
• Updates database schema, DAL queries, and API response validation to remove all references to the deprecated field

Confidence Score: 5/5

• This PR is safe to merge with no risk - it cleanly removes an unused database column
• The column was made obsolete by a September refactor that replaced the flag-based logic with direct reviewer status checks. All references properly removed across migration, schema, DAL, and API layers with no remaining usage
• No files require special attention

[greptile-apps]

_{4 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format}