taro 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5007f07dcb3230a1a45011138c19cc0f233e0a0e0fa81d64d554ee3d5cc4a82e
-  data.tar.gz: 99dde57ec71bb2724a29005e644a0bb23d642df11fe4555bd9203340169814df
+  metadata.gz: 41f14035882a39e03a7bae9c799a42c9fa63414e4201fbbab7f876d54d8d214b
+  data.tar.gz: 768b505506f8ad58792d0d86b6152f1aa60da98419aef111b9fc187ee74eab42
 SHA512:
-  metadata.gz: abb5fb481da01a4a50b19e891c3373b42372c97993788ed24c51b07146e087e25d040bc4ea077606a14013633b30f6254f6ced2e939ba9b039b341d95f918211
-  data.tar.gz: b5dd6c2222598ad3d8ee64c64fd0fc6c40299c3f071b08317d17aa29c3f786924ae4b064f587b1e548ba102e0cc802b8a5adf7ccbadf033795974bfdaae5329a
+  metadata.gz: 27607c95ed9c5a7ab2c4d579dec41357657f24c694f43eac06b15677a82e491fd5986c5cad8b7bf999ee23ee456d75cc2c90debee47e93d02cdc901b4e45f498
+  data.tar.gz: f99bfdfe66f7a09ac1746c23e2671bcec7f20cfee23e980db76561f14e1ea082bc2fcc59e4650af8732d7e608a8b7c012473dc0f862bb15ff9d89501a641c825

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 ## [Unreleased]
 
+## [1.3.0] - 2024-11-25
+
+### Added
+
+- Support for string patterns (on StringType children and in export)
+
+### Fixed
+
+- Fixed OpenAPI export of params with enum (inline & type-based)
+
 ## [1.2.0] - 2024-11-18
 
 ### Added

data/README.md

@@ -305,8 +305,7 @@ end
   - mixed enums
   - nullable enums
   - string format specifications (e.g. binary, int64, password ...)
-  - string pattern specifications
-  - string minLength and maxLength
+  - string minLength and maxLength (substitute: `self.pattern = /\A.{3,5}\z/`)
   - number minimum, exclusiveMinimum, maximum, multipleOf
   - readOnly, writeOnly
 

data/lib/taro/export/open_api_v3.rb

@@ -64,15 +64,22 @@ class Taro::Export::OpenAPIv3 < Taro::Export::Base # rubocop:disable Metrics/Cla
   end
 
   def export_parameter(field)
+    validate_path_or_query_parameter(field)
+
     {
       name: field.name,
       deprecated: field.deprecated,
       description: field.desc,
       required: !field.null,
-      schema: { type: field.openapi_type },
+      schema: export_field(field).except(:deprecated, :description),
     }.compact
   end
 
+  def validate_path_or_query_parameter(field)
+    [:array, :object].include?(field.type.openapi_type) &&
+      raise("Unexpected object as path/query param #{field.name}: #{field.type}")
+  end
+
   def request_body(declaration, route)
     return unless route.can_have_request_body?
 
@@ -87,7 +94,7 @@ class Taro::Export::OpenAPIv3 < Taro::Export::Base # rubocop:disable Metrics/Cla
     body_input_type.openapi_name = params.openapi_name
 
     # For polymorphic routes (more than one for the same declaration),
-    # we can't use refs because they request body might differ.
+    # we can't use refs because their request body might differ:
     # Different params might be in the path vs. in the request body.
     use_refs = !declaration.polymorphic_route?
     schema = request_body_schema(body_input_type, use_refs:)
@@ -115,13 +122,17 @@ class Taro::Export::OpenAPIv3 < Taro::Export::Base # rubocop:disable Metrics/Cla
   end
 
   def export_type(type)
-    if type < Taro::Types::ScalarType
+    if type < Taro::Types::ScalarType && !custom_scalar_type?(type)
       { type: type.openapi_type }
     else
       extract_component_ref(type)
     end
   end
 
+  def custom_scalar_type?(type)
+    type < Taro::Types::ScalarType && (type.openapi_pattern || type.deprecated)
+  end
+
   def export_field(field)
     if field.type < Taro::Types::ScalarType
       export_scalar_field(field)
@@ -174,6 +185,8 @@ class Taro::Export::OpenAPIv3 < Taro::Export::Base # rubocop:disable Metrics/Cla
       enum_type_details(type)
     elsif type < Taro::Types::ListType
       list_type_details(type)
+    elsif custom_scalar_type?(type)
+      custom_scalar_type_details(type)
     else
       raise NotImplementedError, "Unsupported type: #{type}"
     end
@@ -209,6 +222,15 @@ class Taro::Export::OpenAPIv3 < Taro::Export::Base # rubocop:disable Metrics/Cla
     }.compact
   end
 
+  def custom_scalar_type_details(scalar)
+    {
+      type: scalar.openapi_type,
+      deprecated: scalar.deprecated,
+      description: scalar.desc,
+      pattern: scalar.openapi_pattern,
+    }.compact
+  end
+
   def assert_unique_openapi_name(type)
     @name_to_type_map ||= {}
     if (prev = @name_to_type_map[type.openapi_name]) && type != prev

data/lib/taro/types/scalar/iso8601_date_type.rb

@@ -2,11 +2,10 @@ class Taro::Types::Scalar::ISO8601DateType < Taro::Types::ScalarType
   self.desc = 'Represents a time as Date in ISO8601 format.'
   self.openapi_name = 'ISO8601Date'
   self.openapi_type = :string
-
-  PATTERN = /\A\d{4}-(0[1-9]|1[0-2])-(0[1-9]|[12]\d|3[01])\z/
+  self.pattern = /\A\d{4}-(0[1-9]|1[0-2])-(0[1-9]|[12]\d|3[01])\z/
 
   def coerce_input
-    if object.instance_of?(String) && object.match?(PATTERN)
+    if object.instance_of?(String) && object.match?(pattern)
       Date.parse(object)
     else
       input_error("must be a ISO8601 formatted string")

data/lib/taro/types/scalar/iso8601_datetime_type.rb

@@ -2,11 +2,10 @@ class Taro::Types::Scalar::ISO8601DateTimeType < Taro::Types::ScalarType
   self.desc = 'Represents a time as DateTime in ISO8601 format.'
   self.openapi_name = 'ISO8601DateTime'
   self.openapi_type = :string
-
-  PATTERN = /\A\d{4}-(0[1-9]|1[0-2])-(0[1-9]|[12]\d|3[01])T([01]\d|2[0-3]):[0-5]\d:[0-5]\d(Z|[+-](0[0-9]|1[0-4]):[0-5]\d)\z/
+  self.pattern = /\A\d{4}-(0[1-9]|1[0-2])-(0[1-9]|[12]\d|3[01])T([01]\d|2[0-3]):[0-5]\d:[0-5]\d(Z|[+-](0[0-9]|1[0-4]):[0-5]\d)\z/
 
   def coerce_input
-    if object.instance_of?(String) && object.match?(PATTERN)
+    if object.instance_of?(String) && object.match?(pattern)
       DateTime.iso8601(object)
     else
       input_error("must be a ISO8601 formatted string")

data/lib/taro/types/scalar/string_type.rb

@@ -2,14 +2,25 @@ class Taro::Types::Scalar::StringType < Taro::Types::ScalarType
   self.openapi_type = :string
 
   def coerce_input
-    object.instance_of?(String) ? object : input_error('must be a String')
+    object.instance_of?(String) || input_error('must be a String')
+
+    pattern.nil? || pattern.match?(object) ||
+      input_error("must match pattern #{pattern.inspect}")
+
+    object
   end
 
   def coerce_response
-    case object
-    when String then object
-    when Symbol then object.to_s
-    else response_error('must be a String or Symbol')
-    end
+    str =
+      case object
+      when String then object
+      when Symbol then object.to_s
+      else response_error('must be a String or Symbol')
+      end
+
+    pattern.nil? || pattern.match?(str) ||
+      response_error("must match pattern #{pattern.inspect}")
+
+    str
   end
 end

data/lib/taro/types/scalar/uuid_v4_type.rb

@@ -1,23 +1,5 @@
-class Taro::Types::Scalar::UUIDv4Type < Taro::Types::ScalarType
+class Taro::Types::Scalar::UUIDv4Type < Taro::Types::Scalar::StringType
   self.desc = "A UUID v4 string"
   self.openapi_name = 'UUIDv4'
-  self.openapi_type = :string
-
-  PATTERN = /\A\h{8}-?(?:\h{4}-?){3}\h{12}\z/
-
-  def coerce_input
-    if object.is_a?(String) && object.match?(PATTERN)
-      object
-    else
-      input_error("must be a UUID v4 string")
-    end
-  end
-
-  def coerce_response
-    if object.is_a?(String) && object.match?(PATTERN)
-      object
-    else
-      response_error("must be a UUID v4 string")
-    end
-  end
+  self.pattern = /\A\h{8}-?(?:\h{4}-?){3}\h{12}\z/
 end

data/lib/taro/types/scalar_type.rb

@@ -1,5 +1,6 @@
 # Abstract base class for scalar types, i.e. types without fields.
 class Taro::Types::ScalarType < Taro::Types::BaseType
+  include Taro::Types::Shared::Pattern
 end
 
 module Taro::Types::Scalar

data/lib/taro/types/shared/custom_field_resolvers.rb

@@ -15,8 +15,8 @@ module Taro::Types::Shared::CustomFieldResolvers
     end
 
     def method_added(name)
-      if name == :object
-        raise(Taro::ArgumentError, '#object is a reserved, internally used method name')
+      if [:object, :pattern].include?(name)
+        raise(Taro::ArgumentError, "##{name} is a reserved, internally used method name")
       elsif ![:coerce_input, :coerce_response].include?(name) &&
             !self.name.to_s.start_with?('Taro::Types::')
         custom_resolvers[name] = true

data/lib/taro/types/shared/pattern.rb

@@ -0,0 +1,69 @@
+module Taro::Types::Shared::Pattern
+  def pattern
+    self.class.pattern
+  end
+
+  def self.included(base)
+    base.extend(ClassMethods)
+  end
+
+  module ClassMethods
+    attr_reader :pattern, :openapi_pattern
+
+    def pattern=(regexp)
+      openapi_type == :string ||
+        raise(Taro::RuntimeError, 'pattern requires openapi_type :string')
+
+      @pattern = regexp
+      @openapi_pattern = Taro::Types::Shared::Pattern.to_es262(regexp)
+    end
+  end
+
+  def self.to_es262(regexp)
+    validate(regexp).source.gsub(
+      /#{NOT_ESCAPED}\\[Ahz]/,
+      { '\\A' => '^', '\\h' => '[0-9a-fA-F]', '\\z' => '$' }
+    )
+  end
+
+  def self.validate(regexp)
+    validate_no_flags(regexp)
+    validate_not_empty(regexp)
+    validate_no_advanced_syntax(regexp)
+    regexp
+  end
+
+  def self.validate_no_flags(regexp)
+    (flags = regexp.inspect[%r{/\w+\z}]) &&
+      raise(Taro::ArgumentError, "pattern flags (#{flags}) are not supported")
+  end
+
+  def self.validate_not_empty(regexp)
+    regexp.source.empty? &&
+      raise(Taro::ArgumentError, 'pattern cannot be empty')
+  end
+
+  def self.validate_no_advanced_syntax(regexp)
+    return unless (match = regexp.source.match(ADVANCED_RUBY_REGEXP_SYNTAX_REGEXP))
+
+    feature = match.named_captures.find { |k, v| break k if v }
+    raise Taro::ArgumentError, <<~MSG
+      pattern uses non-JS syntax #{match} (#{feature}) at index #{match.begin(0)}
+    MSG
+  end
+
+  NOT_ESCAPED = /(?<!\\)(?:\\\\)*\K/
+
+  # This is not 100% accurate, e.g. /[?+]/ is a false positive, but it should be
+  # good enough so we don't need regexp_parser or js_regex as a dependency.
+  ADVANCED_RUBY_REGEXP_SYNTAX_REGEXP = /
+    #{NOT_ESCAPED}
+    (?:
+        (?<a special group or lookaround> \(\?[^:] )
+      | (?<a Ruby-specific escape> \\[a-zA-Z&&[^bBdDsSwWAzfhnrv]] )
+      | (?<an advanced quantifier> [?*+}][?+] )
+      | (?<a nested set> \[[^\]]*(?<!\\)\[ )
+      | (?<a set intersection> && )
+    )
+  /x
+end

data/lib/taro/version.rb

@@ -1,4 +1,4 @@
 # :nocov:
 module Taro
-  VERSION = "1.2.0"
+  VERSION = "1.3.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: taro
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Janosch Müller
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-11-18 00:00:00.000000000 Z
+date: 2024-11-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -99,6 +99,7 @@ files:
 - lib/taro/types/shared/object_coercion.rb
 - lib/taro/types/shared/openapi_name.rb
 - lib/taro/types/shared/openapi_type.rb
+- lib/taro/types/shared/pattern.rb
 - lib/taro/types/shared/rendering.rb
 - lib/taro/version.rb
 - tasks/benchmark.rake