claude-code - 💡(How to fix) Fix hookify plugin: rule_engine.py _extract_field doesn't handle Write tool's content field [1 comments, 2 participants]

TL;DR

Update the _extract_field method in rule_engine.py to handle both new_string and content fields for the Write and Edit tools.

Guidance

• Verify that the issue is indeed caused by the mismatch between new_string and content fields in the Write and Edit tools.
• Update line 239 of rule_engine.py to use the suggested fix: return tool_input.get('new_string') or tool_input.get('content', '').
• Test the updated code with both Write and Edit tool operations to ensure that the new_text field works as expected.
• Review existing Hookify rules to ensure they are correctly configured to use the new_text field for file events.

Example

# Updated _extract_field method
def _extract_field(tool_input, field):
    if field == 'new_text':
        return tool_input.get('new_string') or tool_input.get('content', '')
    # ... rest of the method remains the same

Notes

This fix assumes that the content field is always present in the Write tool input when the new_string field is not. If this is not the case, additional error handling may be necessary.

Recommendation

Apply the suggested workaround by updating the _extract_field method, as it provides a universal solution for both Write and Edit tools.