fix(internal): ensure formatting always uses c.utf-8 locale

Author: stainless-app[bot]

.rubocop.yml

@@ -63,6 +63,8 @@ Layout/MultilineMethodParameterLineBreaks:
 # Prefer compact hash literals.
 Layout/SpaceInsideHashLiteralBraces:
   EnforcedStyle: no_space
+  Exclude:
+    - "**/*.rbi"
 
 Lint/BooleanSymbol:
   Enabled: false
@@ -90,6 +92,10 @@ Lint/MissingSuper:
   Exclude:
     - "**/*.rbi"
 
+Lint/SymbolConversion:
+  Exclude:
+    - "**/*.rbi"
+
 # Disabled for safety reasons, this option changes code semantics.
 Lint/UnusedMethodArgument:
   AutoCorrect: false
@@ -244,6 +250,10 @@ Style/RedundantInitialize:
   Exclude:
     - "**/*.rbi"
 
+Style/RedundantParentheses:
+  Exclude:
+    - "**/*.rbi"
+
 # Prefer slashes for regex literals.
 Style/RegexpLiteral:
   EnforcedStyle: slashes

Gemfile.lock

@@ -1,6 +1,6 @@
 GIT
   remote: https://github.com/stainless-api/syntax_tree-rbs.git
-  revision: 140eb3ba2ff4b959b345ac2a7927cd758a9f1284
+  revision: c30b50219918be7cfe3ef803a00b59d1e77fcada
   branch: main
   specs:
     syntax_tree-rbs (1.0.0)

README.md

@@ -191,9 +191,9 @@ Sorbet's typed enums require sub-classing of the [`T::Enum` class](https://sorbe
 Since this library does not depend on `sorbet-runtime`, it uses a [`T.all` intersection type](https://sorbet.org/docs/intersection-types) with a ruby primitive type to construct a "tagged alias" instead.
 
 ```ruby
-module FinchAPI::Models::ConnectionStatusType
+module FinchAPI::ConnectionStatusType
   # This alias aids language service driven navigation.
-  TaggedSymbol = T.type_alias { T.all(Symbol, FinchAPI::Models::ConnectionStatusType) }
+  TaggedSymbol = T.type_alias { T.all(Symbol, FinchAPI::ConnectionStatusType) }
 end
 ```
 

Rakefile

@@ -34,29 +34,40 @@ multitask(:test) do
   ruby(*%w[-w -e], rb, verbose: false) { fail unless _1 }
 end
 
-rubo_find = %w[find ./lib ./test ./rbi -type f -and ( -name *.rb -or -name *.rbi ) -print0]
 xargs = %w[xargs --no-run-if-empty --null --max-procs=0 --max-args=300 --]
+locale = {"LC_ALL" => "C.UTF-8"}
 
 desc("Lint `*.rb(i)`")
 multitask(:"lint:rubocop") do
+  find = %w[find ./lib ./test ./rbi -type f -and ( -name *.rb -or -name *.rbi ) -print0]
+
   rubocop = %w[rubocop --fail-level E]
   rubocop += %w[--format github] if ENV.key?("CI")
 
   # some lines cannot be shortened
   rubocop += %w[--except Lint/RedundantCopDisableDirective,Layout/LineLength]
 
   lint = xargs + rubocop
-  sh("#{rubo_find.shelljoin} | #{lint.shelljoin}")
+  sh("#{find.shelljoin} | #{lint.shelljoin}")
 end
 
-desc("Format `*.rb(i)`")
-multitask(:"format:rubocop") do
+desc("Format `*.rb`")
+multitask(:"format:rb") do
+  # while `syntax_tree` is much faster than `rubocop`, `rubocop` is the only formatter with full syntax support
+  find = %w[find ./lib ./test -type f -and -name *.rb -print0]
   fmt = xargs + %w[rubocop --fail-level F --autocorrect --format simple --]
-  sh("#{rubo_find.shelljoin} | #{fmt.shelljoin}")
+  sh("#{find.shelljoin} | #{fmt.shelljoin}")
+end
+
+desc("Format `*.rbi`")
+multitask(:"format:rbi") do
+  find = %w[find ./rbi -type f -and -name *.rbi -print0]
+  fmt = xargs + %w[stree write --]
+  sh(locale, "#{find.shelljoin} | #{fmt.shelljoin}")
 end
 
 desc("Format `*.rbs`")
-multitask(:"format:syntax_tree") do
+multitask(:"format:rbs") do
   find = %w[find ./sig -type f -name *.rbs -print0]
   inplace = /darwin|bsd/ =~ RUBY_PLATFORM ? ["-i", ""] : %w[-i]
   uuid = SecureRandom.uuid
@@ -88,7 +99,7 @@ multitask(:"format:syntax_tree") do
   # transform class aliases to type aliases, which syntax tree has no trouble with
   sh("#{find.shelljoin} | #{pre.shelljoin}")
   # run syntax tree to format `*.rbs` files
-  sh("#{find.shelljoin} | #{fmt.shelljoin}") do
+  sh(locale, "#{find.shelljoin} | #{fmt.shelljoin}") do
     success = _1
   end
   # transform type aliases back to class aliases
@@ -99,7 +110,7 @@ multitask(:"format:syntax_tree") do
 end
 
 desc("Format everything")
-multitask(format: [:"format:rubocop", :"format:syntax_tree"])
+multitask(format: [:"format:rb", :"format:rbi", :"format:rbs"])
 
 desc("Typecheck `*.rbs`")
 multitask(:"typecheck:steep") do

lib/finch_api.rb

@@ -174,6 +174,7 @@
 require_relative "finch_api/models/sandbox/payment_create_params"
 require_relative "finch_api/models/sandbox/payment_create_response"
 require_relative "finch_api/models/webhook_event"
+require_relative "finch_api/models"
 require_relative "finch_api/resources/access_tokens"
 require_relative "finch_api/resources/account"
 require_relative "finch_api/resources/connect"

lib/finch_api/file_part.rb

@@ -45,9 +45,9 @@ def initialize(content, filename: nil, content_type: nil)
       @filename =
         case content
         in Pathname
-          filename.nil? ? content.basename.to_path : File.basename(filename)
+          filename.nil? ? content.basename.to_path : ::File.basename(filename)
         else
-          filename.nil? ? nil : File.basename(filename)
+          filename.nil? ? nil : ::File.basename(filename)
         end
       @content_type = content_type
     end

lib/finch_api/internal/individuals_page.rb

@@ -19,7 +19,7 @@ class IndividualsPage
       # @return [Array<generic<Elem>>, nil]
       attr_accessor :individuals
 
-      # @return [FinchAPI::Models::Paging]
+      # @return [FinchAPI::Paging]
       attr_accessor :paging
 
       # @return [Boolean]
@@ -75,7 +75,7 @@ def initialize(client:, req:, headers:, page_data:)
         end
         case page_data
         in {paging: Hash | nil => paging}
-          @paging = FinchAPI::Internal::Type::Converter.coerce(FinchAPI::Models::Paging, paging)
+          @paging = FinchAPI::Internal::Type::Converter.coerce(FinchAPI::Paging, paging)
         else
         end
       end

lib/finch_api/internal/page.rb

@@ -19,7 +19,7 @@ class Page
       # @return [Array<generic<Elem>>, nil]
       attr_accessor :data
 
-      # @return [FinchAPI::Models::Paging]
+      # @return [FinchAPI::Paging]
       attr_accessor :paging
 
       # @return [Boolean]
@@ -75,7 +75,7 @@ def initialize(client:, req:, headers:, page_data:)
         end
         case page_data
         in {paging: Hash | nil => paging}
-          @paging = FinchAPI::Internal::Type::Converter.coerce(FinchAPI::Models::Paging, paging)
+          @paging = FinchAPI::Internal::Type::Converter.coerce(FinchAPI::Paging, paging)
         else
         end
       end

lib/finch_api/internal/type/enum.rb

@@ -17,13 +17,13 @@ module Type
       # values safely.
       #
       # @example
-      #   # `connection_status_type` is a `FinchAPI::Models::ConnectionStatusType`
+      #   # `connection_status_type` is a `FinchAPI::ConnectionStatusType`
       #   case connection_status_type
-      #   when FinchAPI::Models::ConnectionStatusType::PENDING
+      #   when FinchAPI::ConnectionStatusType::PENDING
       #     # ...
-      #   when FinchAPI::Models::ConnectionStatusType::PROCESSING
+      #   when FinchAPI::ConnectionStatusType::PROCESSING
       #     # ...
-      #   when FinchAPI::Models::ConnectionStatusType::CONNECTED
+      #   when FinchAPI::ConnectionStatusType::CONNECTED
       #     # ...
       #   else
       #     puts(connection_status_type)

lib/finch_api/internal/type/request_parameters.rb

@@ -5,10 +5,10 @@ module Internal
     module Type
       # @api private
       module RequestParameters
-        # @!parse
-        #   # Options to specify HTTP behaviour for this request.
-        #   # @return [FinchAPI::RequestOptions, Hash{Symbol=>Object}]
-        #   attr_accessor :request_options
+        # @!attribute request_options
+        # Options to specify HTTP behaviour for this request.
+        #
+        #   @return [FinchAPI::RequestOptions, Hash{Symbol=>Object}]
 
         # @param mod [Module]
         def self.included(mod)