emailfuse 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ed5308d363064b9bea74f6d008a55bde447dc2f956689a2d357e8ce8467775b9
-  data.tar.gz: 259675d6b858aba7da91e34b31e7a199ab5f98b70b81bbc0c64b8e72c959c01f
+  metadata.gz: d1b4f565a1eeaa3276ef82ece624d8fb7f0e2f04f1391b6c6ac3eac119a4e828
+  data.tar.gz: f1bf41d6a0c55dfe7c7c0a799effa3c2d0d666dff1a2e766a43209f97890ba65
 SHA512:
-  metadata.gz: eb5deeb16c7d71d1873557cbbdace95c73e3d3c913f4a157f8c55345ce3c06636d16abbbb69d5a268b461b4dbec81faa15d435db1a6e77f311f60183b4127027
-  data.tar.gz: cd86a4342f1994e0462f90537de60d36e29baea5b737fa383bd6d5cdb8bde5e943dc4910a3fd3332c4b50eeeba0c9d8132344c641bcb773373f8f319928c0bcf
+  metadata.gz: 48d156ee8fc31d249f1f89fa8f06407a2675ba243cb2955c3a82bf91ccd0b9b30ea5efc6294e38f9c32f3bbe8821cc1c16d343c9880960e2e9bfbfafb07bcf4f
+  data.tar.gz: 29a56256dbeabe5dcdcc09aa468495dab2bdb9c4fb62cbe1d39106bc82bd54cc4b4a925ff2b8025b21eef56ea5705bae2109cb7a41193ec4e94d68461e493104

data/lib/email_fuse/batch.rb

@@ -27,6 +27,9 @@ module EmailFuse
       #
       # https://resend.com/docs/api-reference/emails/send-batch-emails
       def send(params = [], options: {})
+        raise ArgumentError, "params must be an array of email hashes" unless params.is_a?(Array)
+        raise ArgumentError, "params cannot be empty" if params.empty?
+
         path = "emails/batch"
 
         EmailFuse::Request.new(path, params, "post", options: options).perform

data/lib/email_fuse/emails/attachments.rb

@@ -21,6 +21,9 @@ module EmailFuse
           attachment_id = params[:id]
           email_id = params[:email_id]
 
+          raise ArgumentError, ":email_id is required" if email_id.nil? || email_id.to_s.empty?
+          raise ArgumentError, ":id is required" if attachment_id.nil? || attachment_id.to_s.empty?
+
           path = "emails/#{email_id}/attachments/#{attachment_id}"
           EmailFuse::Request.new(path, {}, "get").perform
         end
@@ -53,6 +56,8 @@ module EmailFuse
         #   )
         def list(params = {})
           email_id = params[:email_id]
+          raise ArgumentError, ":email_id is required" if email_id.nil? || email_id.to_s.empty?
+
           base_path = "emails/#{email_id}/attachments"
 
           # Extract pagination parameters

data/lib/email_fuse/emails/receiving/attachments.rb

@@ -22,6 +22,9 @@ module EmailFuse
             attachment_id = params[:id]
             email_id = params[:email_id]
 
+            raise ArgumentError, ":email_id is required" if email_id.nil? || email_id.to_s.empty?
+            raise ArgumentError, ":id is required" if attachment_id.nil? || attachment_id.to_s.empty?
+
             path = "emails/receiving/#{email_id}/attachments/#{attachment_id}"
             EmailFuse::Request.new(path, {}, "get").perform
           end
@@ -54,6 +57,8 @@ module EmailFuse
           #   )
           def list(params = {})
             email_id = params[:email_id]
+            raise ArgumentError, ":email_id is required" if email_id.nil? || email_id.to_s.empty?
+
             base_path = "emails/receiving/#{email_id}/attachments"
 
             # Extract pagination parameters

data/lib/email_fuse/emails/receiving.rb

@@ -12,7 +12,9 @@ module EmailFuse
         #
         # @example
         #   EmailFuse::Emails::Receiving.get("4ef9a417-02e9-4d39-ad75-9611e0fcc33c")
-        def get(email_id = "")
+        def get(email_id)
+          raise ArgumentError, "email_id is required" if email_id.nil? || email_id.to_s.empty?
+
           path = "emails/receiving/#{email_id}"
           EmailFuse::Request.new(path, {}, "get").perform
         end

data/lib/email_fuse/emails.rb

@@ -5,40 +5,69 @@ module EmailFuse
   module Emails
     class << self
       # Sends or schedules an email.
-      # see more: https://resend.com/docs/api-reference/emails/send-email
+      #
+      # @param params [Hash] Email parameters
+      # @option params [String] :from The sender email address (required)
+      # @option params [Array<String>, String] :to The recipient email address(es) (required)
+      # @option params [String] :subject The email subject
+      # @option params [String] :html The HTML content of the email
+      # @option params [String] :text The plain text content of the email
+      #
+      # @return [EmailFuse::Response] The response containing the email ID
       def send(params, options: {})
+        raise ArgumentError, ":from is required" unless params[:from] || params["from"]
+        raise ArgumentError, ":to is required" unless params[:to] || params["to"]
+
         path = "emails"
         EmailFuse::Request.new(path, params, "post", options: options).perform
       end
 
       # Retrieve a single email.
-      # see more: https://resend.com/docs/api-reference/emails/retrieve-email
-      def get(email_id = "")
+      #
+      # @param email_id [String] The email ID (required)
+      #
+      # @return [EmailFuse::Response] The email object
+      def get(email_id)
+        raise ArgumentError, "email_id is required" if email_id.nil? || email_id.to_s.empty?
+
         path = "emails/#{email_id}"
         EmailFuse::Request.new(path, {}, "get").perform
       end
 
       # Update a scheduled email.
-      # see more: https://resend.com/docs/api-reference/emails/update-email
+      #
+      # @param params [Hash] Update parameters
+      # @option params [String] :email_id The email ID (required)
+      #
+      # @return [EmailFuse::Response] The updated email object
       def update(params)
-        path = "emails/#{params[:email_id]}"
+        email_id = params[:email_id] || params["email_id"]
+        raise ArgumentError, ":email_id is required" if email_id.nil? || email_id.to_s.empty?
+
+        path = "emails/#{email_id}"
         EmailFuse::Request.new(path, params, "patch").perform
       end
 
       # Cancel a scheduled email.
-      # see more: https://resend.com/docs/api-reference/emails/cancel-email
-      def cancel(email_id = "")
+      #
+      # @param email_id [String] The email ID (required)
+      #
+      # @return [EmailFuse::Response] Confirmation of cancellation
+      def cancel(email_id)
+        raise ArgumentError, "email_id is required" if email_id.nil? || email_id.to_s.empty?
+
         path = "emails/#{email_id}/cancel"
         EmailFuse::Request.new(path, {}, "post").perform
       end
 
       # List emails with optional pagination.
-      # see more: https://resend.com/docs/api-reference/emails/list-emails
       #
       # @param options [Hash] Optional parameters for pagination
       # @option options [Integer] :limit Maximum number of emails to return (1-100, default 20)
       # @option options [String] :after Cursor for pagination (newer emails)
       # @option options [String] :before Cursor for pagination (older emails)
+      #
+      # @return [EmailFuse::Response] Paginated list of emails
       def list(options = {})
         path = "emails"
 

data/lib/email_fuse/errors.rb

@@ -2,7 +2,6 @@
 
 module EmailFuse
   # Errors wrapper class
-  # For more info: https://resend.com/docs/api-reference/error-codes
   class Error < StandardError
     # 4xx HTTP status code
     ClientError = Class.new(self)
@@ -13,11 +12,14 @@ module EmailFuse
     # code 500
     InternalServerError = Class.new(ServerError)
 
-    # code 422
-    InvalidRequestError = Class.new(ServerError)
+    # code 400, 401, 404, 422 - these are client errors (4xx)
+    InvalidRequestError = Class.new(ClientError)
+
+    # code 404
+    NotFoundError = Class.new(ClientError)
 
     # code 429
-    class RateLimitExceededError < ServerError
+    class RateLimitExceededError < ClientError
       attr_reader :rate_limit_limit, :rate_limit_remaining, :rate_limit_reset, :retry_after
 
       def initialize(msg, code = nil, headers = {})
@@ -29,19 +31,16 @@ module EmailFuse
       end
     end
 
-    # code 404
-    NotFoundError = Class.new(ServerError)
-
     ERRORS = {
+      400 => EmailFuse::Error::InvalidRequestError,
       401 => EmailFuse::Error::InvalidRequestError,
-      404 => EmailFuse::Error::InvalidRequestError,
+      404 => EmailFuse::Error::NotFoundError,
       422 => EmailFuse::Error::InvalidRequestError,
       429 => EmailFuse::Error::RateLimitExceededError,
-      400 => EmailFuse::Error::InvalidRequestError,
       500 => EmailFuse::Error::InternalServerError
     }.freeze
 
-    attr_reader :headers
+    attr_reader :code, :headers
 
     def initialize(msg, code = nil, headers = {})
       super(msg)

data/lib/email_fuse/mailer.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "base64"
 require "email_fuse"
 
 module EmailFuse
@@ -225,15 +226,21 @@ module EmailFuse
 
     #
     # Handle attachments when present
+    # Uses base64 encoding for better API compatibility
     #
     # @return Array attachments array
     #
     def get_attachments(mail)
       attachments = []
       mail.attachments.each do |part|
+        # Get decoded content and ensure binary encoding for consistent base64 output
+        content = part.body.decoded.dup
+        content = content.force_encoding(Encoding::BINARY)
+
         attachment = {
           filename: part.filename,
-          content: part.body.decoded.bytes
+          content: Base64.strict_encode64(content),
+          content_type: part.content_type
         }
 
         # Rails uses the auto generated cid for inline attachments

data/lib/email_fuse/request.rb

@@ -9,7 +9,7 @@ module EmailFuse
     def initialize(path = "", body = {}, verb = "POST", options: {})
       # Allow api_key override via options, fall back to global config
       api_key = options[:api_key] || EmailFuse.api_key
-      raise if api_key.nil?
+      raise ArgumentError, "API key is required. Set via EmailFuse.api_key= or pass :api_key in options" if api_key.nil?
 
       api_key = api_key.call if api_key.is_a?(Proc)
 
@@ -58,6 +58,10 @@ module EmailFuse
     def build_request_options
       options = { headers: @headers }
 
+      # Set timeout from options, global config, or default
+      timeout = @options[:timeout] || EmailFuse.timeout || EmailFuse::DEFAULT_TIMEOUT
+      options[:timeout] = timeout
+
       if get_request_with_query?
         options[:query] = @body
       elsif !@body.empty?

data/lib/email_fuse/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module EmailFuse
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/lib/email_fuse/webhooks.rb

@@ -73,13 +73,15 @@ module EmailFuse
 
       # Retrieve a single webhook for the authenticated user
       #
-      # @param webhook_id [String] The webhook ID
+      # @param webhook_id [String] The webhook ID (required)
       #
       # @return [Hash] The webhook object with full details
       #
       # @example
       #   EmailFuse::Webhooks.get("4dd369bc-aa82-4ff3-97de-514ae3000ee0")
-      def get(webhook_id = "")
+      def get(webhook_id)
+        raise ArgumentError, "webhook_id is required" if webhook_id.nil? || webhook_id.to_s.empty?
+
         path = "webhooks/#{webhook_id}"
         EmailFuse::Request.new(path, {}, "get").perform
       end
@@ -102,20 +104,25 @@ module EmailFuse
       #     status: "enabled"
       #   )
       def update(params = {})
+        params = params.dup # Don't mutate caller's hash
         webhook_id = params.delete(:webhook_id)
+        raise ArgumentError, ":webhook_id is required" if webhook_id.nil? || webhook_id.to_s.empty?
+
         path = "webhooks/#{webhook_id}"
         EmailFuse::Request.new(path, params, "patch").perform
       end
 
       # Remove an existing webhook
       #
-      # @param webhook_id [String] The webhook ID
+      # @param webhook_id [String] The webhook ID (required)
       #
       # @return [Hash] Confirmation object with id, object type, and deleted status
       #
       # @example
       #   EmailFuse::Webhooks.remove("4dd369bc-aa82-4ff3-97de-514ae3000ee0")
-      def remove(webhook_id = "")
+      def remove(webhook_id)
+        raise ArgumentError, "webhook_id is required" if webhook_id.nil? || webhook_id.to_s.empty?
+
         path = "webhooks/#{webhook_id}"
         EmailFuse::Request.new(path, {}, "delete").perform
       end

data/lib/email_fuse.rb

@@ -26,8 +26,10 @@ require "email_fuse/railtie" if defined?(Rails) && defined?(ActionMailer)
 
 # Main EmailFuse module
 module EmailFuse
+  DEFAULT_TIMEOUT = 30
+
   class << self
-    attr_accessor :api_key
+    attr_accessor :api_key, :timeout
     attr_writer :base_url
 
     def configure

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: emailfuse
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Dean Perry
@@ -29,14 +29,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '7.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '7.2'
 email: dean@voupe.com
 executables: []
 extensions: []
@@ -70,7 +70,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
+      version: '3.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="