feat: generate RuboCop-clean Ruby SDK files

Emit Layout-clean Ruby from Twig templates so a fresh generation already
matches RuboCop 1.90.0 (4-space indent, Layout cops only). Ship .rubocop.yml
with the SDK and fail CI on formatter regressions.

Co-authored-by: Chirag Aggarwal <chiragaggarwal5k@gmail.com>

Author: cursoragent

.github/workflows/validation.yml

@@ -14,6 +14,7 @@ jobs:
       FLUTTER_VERSION: '3.35.7'
       KTLINT_VERSION: '1.8.0'
       KTLINT_SHA256: 'a3fd620207d5c40da6ca789b95e7f823c54e854b7fade7f613e91096a3706d75'
+      RUBOCOP_VERSION: '1.90.0'
       OSV_SCANNER_VERSION: 'v2.3.1'
     strategy:
       fail-fast: false
@@ -301,6 +302,13 @@ jobs:
               gem install bundler-audit
               bundle-audit update
               bundle-audit check --no-update
+              if [ ! -f .rubocop.yml ]; then
+                echo "Generated SDK is missing .rubocop.yml"
+                exit 1
+              fi
+              gem install rubocop --version "${RUBOCOP_VERSION}"
+              rubocop --version
+              rubocop
               ;;
             node)
               npm ci

AGENTS.md

@@ -120,6 +120,14 @@ When you change templates for a language below, regenerate and run that language
   (cd examples/python && python -m black --check setup.py)
   ```
 
+- **Ruby** — RuboCop 1.90.0 (Layout cops, 4-space indent)
+
+  ```bash
+  rm -rf examples/ruby
+  php example.php ruby server
+  (cd examples/ruby && rubocop)
+  ```
+
 - **PHP** — Pint, PHPStan, and Rector
 
   ```bash

src/SDK/Language/Ruby.php

@@ -122,6 +122,11 @@ public function getFiles(): array
                 'destination'   => 'Gemfile',
                 'template'      => 'ruby/Gemfile.twig',
             ],
+            [
+                'scope'         => 'copy',
+                'destination'   => '.rubocop.yml',
+                'template'      => 'ruby/.rubocop.yml',
+            ],
             [
                 'scope'         => 'default',
                 'destination'   => '{{ spec.info.title | caseDash }}.gemspec',
@@ -383,6 +388,14 @@ public function getFilters(): array
     {
         return [
             new TwigFilter('caseEnumKey', fn(string $value): string => $this->toUpperSnakeCase($value)),
+            new TwigFilter('rtrimLines', function (mixed $value): string {
+                $lines = explode("\n", str_replace("\r\n", "\n", (string) $value));
+                foreach ($lines as $index => $line) {
+                    $lines[$index] = rtrim($line);
+                }
+
+                return implode("\n", $lines);
+            }),
             new TwigFilter('enumExample', function (Schema|Parameter $param): string {
                 $schema = $this->getSchema($param);
                 $enumSchema = $this->getEnumSchema($param);

templates/ruby/.rubocop.yml

@@ -0,0 +1,26 @@
+# RuboCop layout for the generated Ruby SDK.
+# IndentationWidth 4 matches the published sdk-for-ruby house style.
+# Only Layout cops are enabled: this is a formatter check, not a full linter.
+AllCops:
+  NewCops: enable
+  TargetRubyVersion: 2.7
+  SuggestExtensions: false
+  DisabledByDefault: true
+  Exclude:
+    - '.github/**/*'
+    - 'vendor/**/*'
+    - '.bundle/**/*'
+
+Layout:
+  Enabled: true
+
+Layout/IndentationWidth:
+  Width: 4
+
+Layout/LineLength:
+  Max: 200
+  AllowedPatterns:
+    - '^\s*#'
+
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented

templates/ruby/Gemfile.twig

@@ -1,3 +1,3 @@
 source 'https://rubygems.org'
 
-gemspec
+gemspec

templates/ruby/base/params.twig

@@ -9,17 +9,26 @@
 {% for parameter in (method | parameters('all')) %}
 {% if parameter.required %}
             if {{ parameter.name | caseSnake | escapeKeyword }}.nil?
-              raise {{spec.info.title | caseUcfirst}}::Exception.new('Missing required parameter: "{{ parameter.name | caseCamel | escapeKeyword }}"')
+                raise {{spec.info.title | caseUcfirst}}::Exception.new('Missing required parameter: "{{ parameter.name | caseCamel | escapeKeyword }}"')
             end
 
 {% endif %}
 {% endfor %}
+{% set requestParams = (method | parameters('query')) | merge((method | parameters('body'))) %}
+{% if requestParams | length == 0 %}
+            api_params = {}
+{% else %}
             api_params = {
-{% for parameter in (method | parameters('query')) | merge((method | parameters('body')))  %}
+{% for parameter in requestParams %}
                 {{ parameter.name }}: {{ parameter.name | caseSnake | escapeKeyword }},
 {% endfor %}
             }
-            
+{% endif %}
+
+{% set headerEntries = (method | securityHeaders) | length + (method | parameters('header')) | length + (method | methodHeaders) | length %}
+{% if headerEntries == 0 %}
+            api_headers = {}
+{% else %}
             api_headers = {
             {%~ for key, security in (method | securityHeaders) %}
                 "{{ security.name }}": @client.get_config('{{ key | caseLower }}'),
@@ -31,3 +40,4 @@
                 "{{ key }}": '{{ header }}',
 {% endfor %}
             }
+{% endif %}

templates/ruby/base/requests/api.twig

@@ -23,7 +23,7 @@
 {% endfor %}
 
             raise Exception, "Unable to match response to any expected response model"
-{% else %}
+{%- else %}
             @client.call(
                 method: '{{ method.method.value | caseUpper }}',
                 path: api_path,
@@ -35,4 +35,4 @@
                 response_type: Models::{{ (method | responseModel) | caseUcfirst }}
                 {%~ endif %}
             )
-{% endif %}
+{%- endif %}

templates/ruby/base/requests/file.twig

@@ -31,7 +31,7 @@
 {% endfor %}
 
             raise Exception, "Unable to match response to any expected response model"
-{% else %}
+{%- else %}
             @client.chunked_upload(
                 path: api_path,
                 headers: api_headers,
@@ -43,4 +43,4 @@
                 response_type: Models::{{(method | responseModel) | caseUcfirst}}
 {% endif %}
             )
-{% endif %}
+{%- endif %}

templates/ruby/gemspec.twig

@@ -1,13 +1,12 @@
 Gem::Specification.new do |spec|
+    spec.name = '{{spec.info.title | caseLower | caseSnake}}'
+    spec.version = '{{ sdk.version }}'
+    spec.license = '{{ spec.info.license.name }}'
+    spec.summary = '{{ sdk.shortDescription }}'
+    spec.author = '{{ spec.info.contact.name }}'
+    spec.homepage = '{{ spec.info.contact.url }}'
+    spec.email = '{{ spec.info.contact.email }}'
+    spec.files = Dir['lib/**/*.rb']
 
-  spec.name = '{{spec.info.title | caseLower | caseSnake}}'
-  spec.version = '{{ sdk.version }}'
-  spec.license = '{{ spec.info.license.name }}'
-  spec.summary = '{{ sdk.shortDescription }}'
-  spec.author = '{{ spec.info.contact.name }}'
-  spec.homepage = '{{ spec.info.contact.url }}'
-  spec.email = '{{ spec.info.contact.email }}'
-  spec.files = Dir['lib/**/*.rb']
-
-  spec.add_dependency 'mime-types', '~> 3.4.1'
-end
+    spec.add_dependency 'mime-types', '~> 3.4.1'
+end

templates/ruby/lib/container/client.rb.twig

@@ -1,4 +1,4 @@
-#frozen_string_literal: true
+# frozen_string_literal: true
 
 require 'net/http'
 require 'uri'
@@ -7,15 +7,14 @@ require 'cgi'
 
 module {{ spec.info.title | caseUcfirst }}
     class Client
-
         def initialize
-            @chunk_size = 5*1024*1024
+            @chunk_size = 5 * 1024 * 1024
             @headers = {
                 'user-agent' => RUBY_PLATFORM + ':{{ language.name | caseLower }}-' + RUBY_VERSION,
-                'x-sdk-name'=> '{{ sdk.name }}',
-                'x-sdk-platform'=> '{{ sdk.platform }}',
-                'x-sdk-language'=> '{{ language.name | caseLower }}',
-                'x-sdk-version'=> '{{ sdk.version }}'{% if defaultHeaders | length > 0 %},{% endif %}
+                'x-sdk-name' => '{{ sdk.name }}',
+                'x-sdk-platform' => '{{ sdk.platform }}',
+                'x-sdk-language' => '{{ language.name | caseLower }}',
+                'x-sdk-version' => '{{ sdk.version }}'{% if defaultHeaders | length > 0 %},{% endif %}
 
 {% for key,header in defaultHeaders %}
                 '{{key}}' => '{{header}}'{% if not loop.last %},{% endif %}
@@ -225,13 +224,14 @@ module {{ spec.info.title | caseUcfirst }}
                 chunks_uploaded.to_i >= chunks_total.to_i
             end
 
-            on_progress.call({
+            progress = {
                 id: result['$id'],
-                progress: uploaded_size.to_f/size.to_f * 100.0,
+                progress: uploaded_size.to_f / size.to_f * 100.0,
                 size_uploaded: uploaded_size,
                 chunks_total: result['chunksTotal'] || total_chunks,
                 chunks_uploaded: result['chunksUploaded'] || completed_count
-            }) unless on_progress.nil?
+            }
+            on_progress.call(progress) unless on_progress.nil?
 
             mutex = Mutex.new
             queue = Queue.new
@@ -263,13 +263,14 @@ module {{ spec.info.title | caseUcfirst }}
                             uploaded_size += chunk[:ending] - chunk[:start]
                             last_result = chunk_result
                             completed_result = chunk_result if upload_complete.call(chunk_result)
-                            on_progress.call({
+                            progress = {
                                 id: upload_id,
-                                progress: uploaded_size.to_f/size.to_f * 100.0,
+                                progress: uploaded_size.to_f / size.to_f * 100.0,
                                 size_uploaded: uploaded_size,
                                 chunks_total: chunk_result['chunksTotal'] || total_chunks,
                                 chunks_uploaded: chunk_result['chunksUploaded'] || completed_count
-                            }) unless on_progress.nil?
+                            }
+                            on_progress.call(progress) unless on_progress.nil?
                         end
                     end
                 end
@@ -308,13 +309,13 @@ module {{ spec.info.title | caseUcfirst }}
             @boundary = "----A30#3ad1"
             if method != "GET"
                 case headers[:'content-type']
-                    when 'application/json'
-                        payload = params.to_json
-                    when 'multipart/form-data'
-                        payload = encode_form_data(params) + "--#{@boundary}--\r\n"
-                        headers[:'content-type'] = "multipart/form-data; boundary=#{@boundary}"
-                    else
-                        payload = encode(params)
+                when 'application/json'
+                    payload = params.to_json
+                when 'multipart/form-data'
+                    payload = encode_form_data(params) + "--#{@boundary}--\r\n"
+                    headers[:'content-type'] = "multipart/form-data; boundary=#{@boundary}"
+                else
+                    payload = encode(params)
                 end
             end
 
@@ -371,10 +372,10 @@ module {{ spec.info.title | caseUcfirst }}
             return response
         end
 
-        def encode_form_data(value, key=nil)
+        def encode_form_data(value, key = nil)
             case value
             when Hash
-                value.map { |k,v| encode_form_data(v,k) }.join
+                value.map { |k, v| encode_form_data(v, k) }.join
             when Array
                 value.map { |v| encode_form_data(v, "#{key}[]") }.join
             when nil
@@ -399,11 +400,11 @@ module {{ spec.info.title | caseUcfirst }}
 
         def encode(value, key = nil)
             case value
-            when Hash  then value.map { |k,v| encode(v, append_key(key,k)) }.join('&')
+            when Hash  then value.map { |k, v| encode(v, append_key(key, k)) }.join('&')
             when Array then value.map { |v| encode(v, "#{key}[]") }.join('&')
             when nil   then ''
             else
-            "#{key}=#{CGI.escape(value.to_s)}"
+                "#{key}=#{CGI.escape(value.to_s)}"
             end
         end