Remove file_path from build JSON request (#512)

* Remove file_path from build JSON request

* Updated unrelesed changelog

---------

Co-authored-by: Aaron de Mello <314152+AaronDDM@users.noreply.github.com>

Author: kraju3

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+### Unreleased
+* Removed `file_path` from `File` object to match the Send API schema
+
 ### 6.2.3 / 2025-01-23
 * Fixed issue where errors were not properly thrown due to an instance type of `String` instead of `Hash`
 

lib/nylas/utils/file_utils.rb

@@ -62,6 +62,7 @@ def self.build_json_request(attachments)
       opened_files = []
 
       attachments.each_with_index do |attachment, _index|
+        attachment.delete(:file_path)
         current_attachment = attachment[:content]
         next unless current_attachment
 

spec/nylas/utils/file_utils_spec.rb

@@ -165,6 +165,14 @@
       expect(opened_files).to include(mock_file)
     end
 
+    it "removed the file_path key from the attachment" do
+      attachments = [{ content: mock_file, file_path: "/path/to/file.txt" }]
+
+      result, _opened_files = described_class.build_json_request(attachments)
+
+      expect(result.first).not_to have_key(:file_path)
+    end
+
     it "skips attachments with no content" do
       attachments = [{ content: nil }]