akadmin/chatwoot
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

refactor: simplify not found case

Browse Source
This commit is contained in:
Shivam Mishra
2025-07-09 18:00:11 +05:30
parent 0674edf81b
commit c789ef555c
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
enterprise/lib/captain/tools/add_contact_note_tool.rb
+2 -3
View File
@@ -9,7 +9,7 @@ class Captain::Tools::AddContactNoteTool < Captain::Tools::BaseAgentTool
return 'Missing required parameters: contact_id and note are required' if note.blank? || contact_id.blank?
contact = find_contact(contact_id)
return contact if contact.is_a?(String) # Error message
return 'Contact not found' unless contact
create_contact_note(contact, note)
"Note added successfully to contact #{contact.name} (ID: #{contact.id})"
@@ -18,8 +18,7 @@ class Captain::Tools::AddContactNoteTool < Captain::Tools::BaseAgentTool
private
def find_contact(contact_id)
contact = account_scoped(::Contact).find_by(id: contact_id)
contact || 'Contact not found'
account_scoped(::Contact).find_by(id: contact_id)
end
def create_contact_note(contact, note)