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

@@ -192,9 +192,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 Lithic::Models::AccountFinancialAccountType
+module Lithic::AccountFinancialAccountType
   # This alias aids language service driven navigation.
-  TaggedSymbol = T.type_alias { T.all(Symbol, Lithic::Models::AccountFinancialAccountType) }
+  TaggedSymbol = T.type_alias { T.all(Symbol, Lithic::AccountFinancialAccountType) }
 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/lithic.rb

@@ -320,6 +320,7 @@
 require_relative "lithic/models/transfer"
 require_relative "lithic/models/transfer_create_params"
 require_relative "lithic/models/verification_method"
+require_relative "lithic/models"
 require_relative "lithic/resources/account_holders"
 require_relative "lithic/resources/accounts"
 require_relative "lithic/resources/aggregate_balances"

lib/lithic/client.rb

@@ -104,16 +104,11 @@ class Client < Lithic::Internal::Transport::BaseClient
     #
     # @param request_options [Lithic::RequestOptions, Hash{Symbol=>Object}, nil]
     #
-    # @return [Lithic::Models::APIStatus]
+    # @return [Lithic::APIStatus]
     #
     # @see Lithic::Models::ClientAPIStatusParams
     def api_status(params = {})
-      request(
-        method: :get,
-        path: "v1/status",
-        model: Lithic::Models::APIStatus,
-        options: params[:request_options]
-      )
+      request(method: :get, path: "v1/status", model: Lithic::APIStatus, options: params[:request_options])
     end
 
     # @api private

lib/lithic/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/lithic/internal/type/enum.rb

@@ -17,11 +17,11 @@ module Type
       # values safely.
       #
       # @example
-      #   # `account_financial_account_type` is a `Lithic::Models::AccountFinancialAccountType`
+      #   # `account_financial_account_type` is a `Lithic::AccountFinancialAccountType`
       #   case account_financial_account_type
-      #   when Lithic::Models::AccountFinancialAccountType::ISSUING
+      #   when Lithic::AccountFinancialAccountType::ISSUING
       #     # ...
-      #   when Lithic::Models::AccountFinancialAccountType::OPERATING
+      #   when Lithic::AccountFinancialAccountType::OPERATING
       #     # ...
       #   else
       #     puts(account_financial_account_type)

lib/lithic/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 [Lithic::RequestOptions, Hash{Symbol=>Object}]
-        #   attr_accessor :request_options
+        # @!attribute request_options
+        # Options to specify HTTP behaviour for this request.
+        #
+        #   @return [Lithic::RequestOptions, Hash{Symbol=>Object}]
 
         # @param mod [Module]
         def self.included(mod)

lib/lithic/internal/util.rb

@@ -535,7 +535,7 @@ class << self
             filename = ERB::Util.url_encode(val.filename)
             y << "; filename=\"#{filename}\""
           in Pathname | IO
-            filename = ERB::Util.url_encode(File.basename(val.to_path))
+            filename = ERB::Util.url_encode(::File.basename(val.to_path))
             y << "; filename=\"#{filename}\""
           else
           end