claude-code - 💡(How to fix) Fix Built-in /security-review fails in jj-only repos (git denied) [1 comments, 2 participants]

TL;DR

Modify the /security-review to fall back to jj status or jj diff when git is unavailable due to permissions restrictions.

Guidance

• Investigate the permissions.deny configuration in jj-only environments to understand the specific restrictions on git commands.
• Consider adding a fallback mechanism to the /security-review implementation to use jj status or jj diff when git status is not allowed.
• Review the includeGitInstructions flag to determine if it can be used to disable git commands and use jj commands instead.
• Test the modified /security-review with different permissions.deny configurations to ensure it works as expected.

Example

No code snippet is provided as the issue does not contain sufficient information about the implementation details.

Notes

The solution may require changes to the /security-review implementation and the permissions.deny configuration. The exact changes will depend on the specific requirements and constraints of the jj-only environments.

Recommendation

Apply workaround: Modify the /security-review to use jj status or jj diff when git is unavailable, as this approach allows for a flexible fallback mechanism that respects the permissions.deny restrictions.