post-for-me 0.1.0.pre.alpha.7 → 0.1.0.pre.alpha.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 951a66c1e45fb72722e5f71297ebd8dd34ff622e4062e4f0adecd585f870e758
-  data.tar.gz: ee4387745b56c7b48a0edd473264a8ab5b84088f538109daecdba4c04392f129
+  metadata.gz: 01ccb7d44dae77373838e7b5fe9d286727f29d5b0cb28c147c4d5f2db24f8755
+  data.tar.gz: 80cf5a0f6c198eb315344dfeec272c8987552d5ec63ac8054b10b6c99787d577
 SHA512:
-  metadata.gz: a318a9673892837459ed63ee5d5ff5049fd0ffe26018bc2a222cc48bca57830b3043180202072be1d5337e9aa7efa2863d2d621ab3f133f51772da0e16b75f3e
-  data.tar.gz: 7db485907c9350625649a64f9ce73c35bfb0f114b2613171972d01f1d14ba51e40097c357b01cc0e93d7590e2c2c43c36387f590182ebc188265906c6a6ffd19
+  metadata.gz: 3040c94c71317b8e81ec5b45fd1b06ed7214b291b158d9ae292d229ce1ad7404b359c6424294702923d4d2edec71dbe5c14542e4c5ae101f189ef07f2dd730ef
+  data.tar.gz: a294fa51eff7716dd53392b2defcc295b559c5fddbc3d857f4274d6bcfd52189b105c8d0e247c25820e601670f76ef5c8ab319c423ce777eebf74e1dc69aa14a

data/CHANGELOG.md

@@ -1,5 +1,43 @@
 # Changelog
 
+## 0.1.0-alpha.9 (2025-10-03)
+
+Full Changelog: [v0.1.0-alpha.8...v0.1.0-alpha.9](https://github.com/DayMoonDevelopment/post-for-me-ruby/compare/v0.1.0-alpha.8...v0.1.0-alpha.9)
+
+### Features
+
+* **api:** api update ([29166f7](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/29166f7dd2eebd6f9a49e99c553f4212233879e9))
+* **api:** api update ([c75db7a](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/c75db7ab2fcc16f9e831ed1722e837d4dc1fbd91))
+* **api:** api update ([11635c1](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/11635c16438bf92c6aec3d4c524f09768bf8f2cc))
+* expose response headers for both streams and errors ([c4a0864](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/c4a0864dab0421eca1ba510038376497ca4c3d1f))
+
+
+### Bug Fixes
+
+* coroutine leaks from connection pool ([5ca79a4](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/5ca79a4bbab523051dbecd1f48eb95ad5bf93af7))
+* shorten multipart boundary sep to less than RFC specificed max length ([e467fe7](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/e467fe77aacf67eeff2f4abd38177476c925c411))
+
+
+### Performance Improvements
+
+* faster code formatting ([58bee30](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/58bee30eb5b12ee89f3271e386d425d119641227))
+
+
+### Chores
+
+* allow fast-format to use bsd sed as well ([b753209](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/b7532090e9056a27ce382fe54965a90a1bf65cce))
+* do not install brew dependencies in ./scripts/bootstrap by default ([ba2ba3c](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/ba2ba3cc8fa0b6e756a840035bcd6f30054a1831))
+* **internal:** codegen related update ([11c589f](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/11c589fc01e8a1c650e271dffb5c9bd9ef2f4da3))
+* **internal:** codegen related update ([6e6de4f](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/6e6de4f6f1424334a37de60918ea43190593d522))
+
+## 0.1.0-alpha.8 (2025-09-06)
+
+Full Changelog: [v0.1.0-alpha.7...v0.1.0-alpha.8](https://github.com/DayMoonDevelopment/post-for-me-ruby/compare/v0.1.0-alpha.7...v0.1.0-alpha.8)
+
+### Features
+
+* **api:** api update ([b15e35d](https://github.com/DayMoonDevelopment/post-for-me-ruby/commit/b15e35d20fbac883c45ae3241b432b2fcbc8219e))
+
 ## 0.1.0-alpha.7 (2025-08-25)
 
 Full Changelog: [v0.1.0-alpha.6...v0.1.0-alpha.7](https://github.com/DayMoonDevelopment/post-for-me-ruby/compare/v0.1.0-alpha.6...v0.1.0-alpha.7)

data/README.md

@@ -17,7 +17,7 @@ To use this gem, install via Bundler by adding the following to your application
 <!-- x-release-please-start-version -->
 
 ```ruby
-gem "post-for-me", "~> 0.1.0.pre.alpha.7"
+gem "post-for-me", "~> 0.1.0.pre.alpha.9"
 ```
 
 <!-- x-release-please-end -->

data/lib/post_for_me/errors.rb

@@ -40,6 +40,9 @@ module PostForMe
       # @return [Integer, nil]
       attr_accessor :status
 
+      # @return [Hash{String=>String}, nil]
+      attr_accessor :headers
+
       # @return [Object, nil]
       attr_accessor :body
 
@@ -47,13 +50,15 @@ module PostForMe
       #
       # @param url [URI::Generic]
       # @param status [Integer, nil]
+      # @param headers [Hash{String=>String}, nil]
       # @param body [Object, nil]
       # @param request [nil]
       # @param response [nil]
       # @param message [String, nil]
-      def initialize(url:, status: nil, body: nil, request: nil, response: nil, message: nil)
+      def initialize(url:, status: nil, headers: nil, body: nil, request: nil, response: nil, message: nil)
         @url = url
         @status = status
+        @headers = headers
         @body = body
         @request = request
         @response = response
@@ -74,6 +79,7 @@ module PostForMe
       #
       # @param url [URI::Generic]
       # @param status [nil]
+      # @param headers [Hash{String=>String}, nil]
       # @param body [nil]
       # @param request [nil]
       # @param response [nil]
@@ -81,6 +87,7 @@ module PostForMe
       def initialize(
         url:,
         status: nil,
+        headers: nil,
         body: nil,
         request: nil,
         response: nil,
@@ -95,6 +102,7 @@ module PostForMe
       #
       # @param url [URI::Generic]
       # @param status [nil]
+      # @param headers [Hash{String=>String}, nil]
       # @param body [nil]
       # @param request [nil]
       # @param response [nil]
@@ -102,6 +110,7 @@ module PostForMe
       def initialize(
         url:,
         status: nil,
+        headers: nil,
         body: nil,
         request: nil,
         response: nil,
@@ -116,21 +125,24 @@ module PostForMe
       #
       # @param url [URI::Generic]
       # @param status [Integer]
+      # @param headers [Hash{String=>String}, nil]
       # @param body [Object, nil]
       # @param request [nil]
       # @param response [nil]
       # @param message [String, nil]
       #
       # @return [self]
-      def self.for(url:, status:, body:, request:, response:, message: nil)
-        kwargs = {
-          url: url,
-          status: status,
-          body: body,
-          request: request,
-          response: response,
-          message: message
-        }
+      def self.for(url:, status:, headers:, body:, request:, response:, message: nil)
+        kwargs =
+          {
+            url: url,
+            status: status,
+            headers: headers,
+            body: body,
+            request: request,
+            response: response,
+            message: message
+          }
 
         case status
         in 400
@@ -162,15 +174,17 @@ module PostForMe
       #
       # @param url [URI::Generic]
       # @param status [Integer]
+      # @param headers [Hash{String=>String}, nil]
       # @param body [Object, nil]
       # @param request [nil]
       # @param response [nil]
       # @param message [String, nil]
-      def initialize(url:, status:, body:, request:, response:, message: nil)
+      def initialize(url:, status:, headers:, body:, request:, response:, message: nil)
         message ||= {url: url.to_s, status: status, body: body}
         super(
           url: url,
           status: status,
+          headers: headers,
           body: body,
           request: request,
           response: response,

data/lib/post_for_me/file_part.rb

@@ -38,18 +38,21 @@ module PostForMe
     def to_yaml(*a) = read.to_yaml(*a)
 
     # @param content [Pathname, StringIO, IO, String]
-    # @param filename [String, nil]
+    # @param filename [Pathname, String, nil]
     # @param content_type [String, nil]
     def initialize(content, filename: nil, content_type: nil)
-      @content = content
+      @content_type = content_type
       @filename =
-        case content
-        in Pathname
-          filename.nil? ? content.basename.to_path : ::File.basename(filename)
+        case [filename, (@content = content)]
+        in [String | Pathname, _]
+          ::File.basename(filename)
+        in [nil, Pathname]
+          content.basename.to_path
+        in [nil, IO]
+          content.to_path
         else
-          filename.nil? ? nil : ::File.basename(filename)
+          filename
         end
-      @content_type = content_type
     end
   end
 end

data/lib/post_for_me/internal/transport/base_client.rb

@@ -47,7 +47,7 @@ module PostForMe
           # @api private
           #
           # @param status [Integer]
-          # @param headers [Hash{String=>String}, Net::HTTPHeader]
+          # @param headers [Hash{String=>String}]
           #
           # @return [Boolean]
           def should_retry?(status, headers:)
@@ -85,7 +85,7 @@ module PostForMe
           #
           # @param status [Integer]
           #
-          # @param response_headers [Hash{String=>String}, Net::HTTPHeader]
+          # @param response_headers [Hash{String=>String}]
           #
           # @return [Hash{Symbol=>Object}]
           def follow_redirect(request, status:, response_headers:)
@@ -378,6 +378,7 @@ module PostForMe
           rescue PostForMe::Errors::APIConnectionError => e
             status = e
           end
+          headers = PostForMe::Internal::Util.normalized_headers(response&.each_header&.to_h)
 
           case status
           in ..299
@@ -390,7 +391,7 @@ module PostForMe
           in 300..399
             self.class.reap_connection!(status, stream: stream)
 
-            request = self.class.follow_redirect(request, status: status, response_headers: response)
+            request = self.class.follow_redirect(request, status: status, response_headers: headers)
             send_request(
               request,
               redirect_count: redirect_count + 1,
@@ -399,9 +400,9 @@ module PostForMe
             )
           in PostForMe::Errors::APIConnectionError if retry_count >= max_retries
             raise status
-          in (400..) if retry_count >= max_retries || !self.class.should_retry?(status, headers: response)
+          in (400..) if retry_count >= max_retries || !self.class.should_retry?(status, headers: headers)
             decoded = Kernel.then do
-              PostForMe::Internal::Util.decode_content(response, stream: stream, suppress_error: true)
+              PostForMe::Internal::Util.decode_content(headers, stream: stream, suppress_error: true)
             ensure
               self.class.reap_connection!(status, stream: stream)
             end
@@ -409,6 +410,7 @@ module PostForMe
             raise PostForMe::Errors::APIStatusError.for(
               url: url,
               status: status,
+              headers: headers,
               body: decoded,
               request: nil,
               response: response
@@ -485,19 +487,21 @@ module PostForMe
             send_retry_header: send_retry_header
           )
 
-          decoded = PostForMe::Internal::Util.decode_content(response, stream: stream)
+          headers = PostForMe::Internal::Util.normalized_headers(response.each_header.to_h)
+          decoded = PostForMe::Internal::Util.decode_content(headers, stream: stream)
           case req
           in {stream: Class => st}
             st.new(
               model: model,
               url: url,
               status: status,
+              headers: headers,
               response: response,
               unwrap: unwrap,
               stream: decoded
             )
           in {page: Class => page}
-            page.new(client: self, req: req, headers: response, page_data: decoded)
+            page.new(client: self, req: req, headers: headers, page_data: decoded)
           else
             unwrapped = PostForMe::Internal::Util.dig(decoded, unwrap)
             PostForMe::Internal::Type::Converter.coerce(model, unwrapped)

data/lib/post_for_me/internal/transport/pooled_net_requester.rb

@@ -134,9 +134,9 @@ module PostForMe
 
           # rubocop:disable Metrics/BlockLength
           enum = Enumerator.new do |y|
-            with_pool(url, deadline: deadline) do |conn|
-              next if finished
+            next if finished
 
+            with_pool(url, deadline: deadline) do |conn|
               req, closing = self.class.build_request(request) do
                 self.class.calibrate_socket_timeout(conn, deadline)
               end
@@ -149,7 +149,7 @@ module PostForMe
 
               self.class.calibrate_socket_timeout(conn, deadline)
               conn.request(req) do |rsp|
-                y << [conn, req, rsp]
+                y << [req, rsp]
                 break if finished
 
                 rsp.read_body do |bytes|
@@ -160,6 +160,8 @@ module PostForMe
                 end
                 eof = true
               end
+            ensure
+              conn.finish if !eof && conn&.started?
             end
           rescue Timeout::Error
             raise PostForMe::Errors::APITimeoutError.new(url: url, request: req)
@@ -168,16 +170,11 @@ module PostForMe
           end
           # rubocop:enable Metrics/BlockLength
 
-          conn, _, response = enum.next
+          _, response = enum.next
           body = PostForMe::Internal::Util.fused_enum(enum, external: true) do
             finished = true
-            tap do
-              enum.next
-            rescue StopIteration
-              nil
-            end
+            loop { enum.next }
           ensure
-            conn.finish if !eof && conn&.started?
             closing&.call
           end
           [Integer(response.code), response, body]

data/lib/post_for_me/internal/type/base_page.rb

@@ -39,7 +39,7 @@ module PostForMe
         #
         # @param client [PostForMe::Internal::Transport::BaseClient]
         # @param req [Hash{Symbol=>Object}]
-        # @param headers [Hash{String=>String}, Net::HTTPHeader]
+        # @param headers [Hash{String=>String}]
         # @param page_data [Object]
         def initialize(client:, req:, headers:, page_data:)
           @client = client

data/lib/post_for_me/internal/type/file_input.rb

@@ -82,17 +82,20 @@ module PostForMe
           #
           # @return [Pathname, StringIO, IO, String, Object]
           def dump(value, state:)
-            # rubocop:disable Lint/DuplicateBranch
             case value
+            in StringIO | String
+              # https://datatracker.ietf.org/doc/html/rfc7578#section-4.2
+              # while not required, a filename is recommended, and in practice many servers do expect this
+              PostForMe::FilePart.new(value, filename: "upload")
             in IO
               state[:can_retry] = false
+              value.to_path.nil? ? PostForMe::FilePart.new(value, filename: "upload") : value
             in PostForMe::FilePart if value.content.is_a?(IO)
               state[:can_retry] = false
+              value
             else
+              value
             end
-            # rubocop:enable Lint/DuplicateBranch
-
-            value
           end
 
           # @api private

data/lib/post_for_me/internal/util.rb

@@ -566,7 +566,8 @@ module PostForMe
         #
         # @return [Array(String, Enumerable<String>)]
         private def encode_multipart_streaming(body)
-          boundary = SecureRandom.urlsafe_base64(60)
+          # RFC 1521 Section 7.2.1 says we should have 70 char maximum for boundary length
+          boundary = SecureRandom.urlsafe_base64(46)
 
           closing = []
           strio = writable_enum do |y|
@@ -647,7 +648,7 @@ module PostForMe
         #
         # Assumes each chunk in stream has `Encoding::BINARY`.
         #
-        # @param headers [Hash{String=>String}, Net::HTTPHeader]
+        # @param headers [Hash{String=>String}]
         # @param stream [Enumerable<String>]
         # @param suppress_error [Boolean]
         #

data/lib/post_for_me/models/bluesky_configuration_dto.rb

@@ -12,13 +12,128 @@ module PostForMe
       # @!attribute media
       #   Overrides the `media` from the post
       #
-      #   @return [Array<String>, nil]
-      optional :media, PostForMe::Internal::Type::ArrayOf[String], nil?: true
+      #   @return [Array<PostForMe::Models::BlueskyConfigurationDto::Media>, nil]
+      optional :media,
+               -> { PostForMe::Internal::Type::ArrayOf[PostForMe::BlueskyConfigurationDto::Media] },
+               nil?: true
 
       # @!method initialize(caption: nil, media: nil)
       #   @param caption [Object, nil] Overrides the `caption` from the post
       #
-      #   @param media [Array<String>, nil] Overrides the `media` from the post
+      #   @param media [Array<PostForMe::Models::BlueskyConfigurationDto::Media>, nil] Overrides the `media` from the post
+
+      class Media < PostForMe::Internal::Type::BaseModel
+        # @!attribute url
+        #   Public URL of the media
+        #
+        #   @return [String]
+        required :url, String
+
+        # @!attribute tags
+        #   List of tags to attach to the media
+        #
+        #   @return [Array<PostForMe::Models::BlueskyConfigurationDto::Media::Tag>, nil]
+        optional :tags,
+                 -> { PostForMe::Internal::Type::ArrayOf[PostForMe::BlueskyConfigurationDto::Media::Tag] },
+                 nil?: true
+
+        # @!attribute thumbnail_timestamp_ms
+        #   Timestamp in milliseconds of frame to use as thumbnail for the media
+        #
+        #   @return [Object, nil]
+        optional :thumbnail_timestamp_ms, PostForMe::Internal::Type::Unknown, nil?: true
+
+        # @!attribute thumbnail_url
+        #   Public URL of the thumbnail for the media
+        #
+        #   @return [Object, nil]
+        optional :thumbnail_url, PostForMe::Internal::Type::Unknown, nil?: true
+
+        # @!method initialize(url:, tags: nil, thumbnail_timestamp_ms: nil, thumbnail_url: nil)
+        #   @param url [String] Public URL of the media
+        #
+        #   @param tags [Array<PostForMe::Models::BlueskyConfigurationDto::Media::Tag>, nil] List of tags to attach to the media
+        #
+        #   @param thumbnail_timestamp_ms [Object, nil] Timestamp in milliseconds of frame to use as thumbnail for the media
+        #
+        #   @param thumbnail_url [Object, nil] Public URL of the thumbnail for the media
+
+        class Tag < PostForMe::Internal::Type::BaseModel
+          # @!attribute id
+          #   Facebook User ID, Instagram Username or Instagram product id to tag
+          #
+          #   @return [String]
+          required :id, String
+
+          # @!attribute platform
+          #   The platform for the tags
+          #
+          #   @return [Symbol, PostForMe::Models::BlueskyConfigurationDto::Media::Tag::Platform]
+          required :platform, enum: -> { PostForMe::BlueskyConfigurationDto::Media::Tag::Platform }
+
+          # @!attribute type
+          #   The type of tag, user to tag accounts, product to tag products (only supported
+          #   for instagram)
+          #
+          #   @return [Symbol, PostForMe::Models::BlueskyConfigurationDto::Media::Tag::Type]
+          required :type, enum: -> { PostForMe::BlueskyConfigurationDto::Media::Tag::Type }
+
+          # @!attribute x
+          #   Percentage distance from left edge of the image, Not required for videos or
+          #   stories
+          #
+          #   @return [Float, nil]
+          optional :x, Float
+
+          # @!attribute y_
+          #   Percentage distance from top edge of the image, Not required for videos or
+          #   stories
+          #
+          #   @return [Float, nil]
+          optional :y_, Float, api_name: :y
+
+          # @!method initialize(id:, platform:, type:, x: nil, y_: nil)
+          #   Some parameter documentations has been truncated, see
+          #   {PostForMe::Models::BlueskyConfigurationDto::Media::Tag} for more details.
+          #
+          #   @param id [String] Facebook User ID, Instagram Username or Instagram product id to tag
+          #
+          #   @param platform [Symbol, PostForMe::Models::BlueskyConfigurationDto::Media::Tag::Platform] The platform for the tags
+          #
+          #   @param type [Symbol, PostForMe::Models::BlueskyConfigurationDto::Media::Tag::Type] The type of tag, user to tag accounts, product to tag products (only supported f
+          #
+          #   @param x [Float] Percentage distance from left edge of the image, Not required for videos or stor
+          #
+          #   @param y_ [Float] Percentage distance from top edge of the image, Not required for videos or stori
+
+          # The platform for the tags
+          #
+          # @see PostForMe::Models::BlueskyConfigurationDto::Media::Tag#platform
+          module Platform
+            extend PostForMe::Internal::Type::Enum
+
+            FACEBOOK = :facebook
+            INSTAGRAM = :instagram
+
+            # @!method self.values
+            #   @return [Array<Symbol>]
+          end
+
+          # The type of tag, user to tag accounts, product to tag products (only supported
+          # for instagram)
+          #
+          # @see PostForMe::Models::BlueskyConfigurationDto::Media::Tag#type
+          module Type
+            extend PostForMe::Internal::Type::Enum
+
+            USER = :user
+            PRODUCT = :product
+
+            # @!method self.values
+            #   @return [Array<Symbol>]
+          end
+        end
+      end
     end
   end
 end