datarockets-style 1.3.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6270a40b4ee5b048f1e8df9b7771556a66cc0ff086b9e5198e1b2b2220d99113
-  data.tar.gz: e1d537d012304fdcf3d0439d8fa36625823da85930e31304ffa8f1ff4b839f91
+  metadata.gz: fae6be6c9b336722b260f0e5cfd4d650d95e5cd96e453756b5bcbbf3b74f13bd
+  data.tar.gz: 4bb202204deb99f738fad88e8b8b6aebfcb8ba1c59a4ead9a6c8ad39ab25e952
 SHA512:
-  metadata.gz: 67361be1894556117a361d9d2a7f5f82be094d61c5b16df15f2248fd5f41c7f669b6a864415d153d94cf3c94002f0bf936a3accb55c3fb765054a36c2a364c55
-  data.tar.gz: 5da0e3a48096c6189decf5692f0d3a258cd572310ccd2553edbc713c97a727e25bd785e4dec621f712a6fa048065fa79cf334815b3642e84acbb9961980cbd89
+  metadata.gz: 12f51702b7b03323daf313e476c1bb70e8daa6ffcea8d1d5c0b7ec2b01915cd31147a77c79c62b3bf876dddbe68fe221def1848c2cbb82d17242f7e1e018719e
+  data.tar.gz: e79c1b517947247f4c76f06e59e1c52816193080162b412b3f3beea3900d97d3f8b32749d99335dfadb1be51266bca04cb0cf385b515034a60def7341577ca39

data/.github/workflows/main.yml

@@ -13,25 +13,13 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v1
+    - uses: actions/checkout@v3
 
-    - uses: actions/cache@v1
+    - name: Set up Ruby 2.7
+      uses: ruby/setup-ruby@v1
       with:
-        path: vendor/bundle
-        key: ${{ runner.os }}-gem-${{ hashFiles('**/Gemfile.lock') }}
-        restore-keys: |
-          ${{ runner.os }}-gem-
-
-    - name: Set up Ruby 2.6
-      uses: actions/setup-ruby@v1
-      with:
-        ruby-version: 2.6.x
-
-    - name: Install and run bundler
-      run: |
-        gem install bundler
-        bundle config path vendor/bundle
-        bundle install --jobs 4 --retry 3
+        ruby-version: '2.7'
+        bundler-cache: true
 
     - name: Run linter
       run: make lint

data/.rubocop.yml

@@ -7,7 +7,7 @@ inherit_mode:
     - Exclude
 
 AllCops:
-  TargetRubyVersion: 2.6
+  TargetRubyVersion: 2.7
 
 # for checking cops with interpolation
 Lint/InterpolationCheck:

data/CHANGELOG.md

@@ -4,6 +4,28 @@ The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-
 
 ## master
 
+## 1.5.0 (2023-09-11)
+
+### Added
+
+* Added GraphQL cops. ([@Anastastasia-B][])
+
+### Changed
+
+* **(Breaking)** Support Ruby >= 2.7.0
+
+* Update rubocop to `1.56.3`. ([@Set27])
+* Update rubocop-rails to `2.21.0`. ([@Set27])
+* Update rubocop-spec to `2.24.0`. ([@Set27])
+
+## 1.4.0 (2023-02-11)
+
+### Changed
+
+* Update rubocop to `1.45.1`. ([@r.dubrovsky][])
+* Update rubocop-rails to `2.17.4`. ([@r.dubrovsky][])
+* Update rubocop-rspec to `2.18.1`. ([@r.dubrovsky][])
+
 ## 1.3.0 (2022-12-02)
 
 ### Added
@@ -264,3 +286,4 @@ The format is described in [Contributing notes](CONTRIBUTING.md#changelog-entry-
 [@a.branzeanu]: https://github.com/texpert
 [@nikitasakov]: https://github.com/nikitasakov
 [@paydaylight]: https://github.com/paydaylight
+[@Anastastasia-B]: https://github.com/Anastastasia-B

data/Gemfile

@@ -5,3 +5,7 @@ source "https://rubygems.org"
 git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 gemspec
+
+gem "pry-byebug"
+gem "rake", "~> 13.0"
+gem "rspec", "~> 3.12.0"

data/README.md

@@ -10,14 +10,14 @@ Add this line to your application's Gemfile:
 
 ```ruby
 group :test, :development do
-  gem 'datarockets-style', '~> 1.3.0'
+  gem 'datarockets-style', '~> 1.5.0'
 end
 ```
 
 Or, for a Ruby library, add this to your gemspec:
 
 ```ruby
-spec.add_development_dependency 'datarockets-style', '~> 1.3.0'
+spec.add_development_dependency 'datarockets-style', '~> 1.5.0'
 ```
 
 And then execute:
@@ -60,6 +60,16 @@ inherit_gem:
     - config/rails-locales.yml
 ```
 
+### GraphQL config
+
+To include specific rules for GraphQL, you can add the following config
+
+```yaml
+inherit_gem:
+  datarockets-style:
+    - config/graphql.yml
+```
+
 ### Rspec config
 
 For Rspec tests, you can add a special rubocop config

data/config/graphql.yml

@@ -0,0 +1,10 @@
+require: rubocop-graphql
+
+GraphQL/ExtractInputType:
+  Enabled: false
+
+GraphQL/ExtractType:
+  Enabled: false
+
+GraphQL/ResolverMethodLength:
+  Max: 10

data/datarockets-style.gemspec

@@ -14,7 +14,7 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/datarockets/datarockets-style"
   spec.license       = "MIT"
 
-  spec.required_ruby_version = ">= 2.6.0"
+  spec.required_ruby_version = ">= 2.7.0"
 
   # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
   # to allow pushing to a single host or delete this section to allow pushing to any host.
@@ -32,12 +32,10 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rubocop", "~> 1.39"
-  spec.add_dependency "rubocop-rails", "~> 2.17.3"
-  spec.add_dependency "rubocop-rspec", "~> 2.15"
+  spec.add_dependency "rubocop", "~> 1.56.3"
+  spec.add_dependency "rubocop-graphql", "~> 1.4"
+  spec.add_dependency "rubocop-rails", "~> 2.21"
+  spec.add_dependency "rubocop-rspec", "~> 2.24"
 
-  spec.add_development_dependency "pry-byebug"
-  spec.add_development_dependency "rake", "~> 13.0"
-  spec.add_development_dependency "rspec", "~> 3.10"
   spec.metadata["rubygems_mfa_required"] = "true"
 end

data/doc/STYLE_GUIDE_GRAPHQL.md

@@ -0,0 +1,486 @@
+# GraphQL Style Guide
+
+This style is based on the rules from [RuboCop::GraphQL](https://github.com/DmitryTsepelev/rubocop-graphql) gem with a few differences.
+
+## Table of contents
+
+* [Style](#Style)
+
+## Style
+
+* <a name="graphql-argument-description"></a>
+  Each argument should have a description.
+  <sup>[[link](#graphql-argument-description)]</sup>
+
+```ruby
+# bad
+class BanUser < BaseMutation
+  argument :uuid, ID, required: true
+end
+
+# good
+class BanUser < BaseMutation
+  argument :uuid, ID, required: true, description: "UUID of the user to ban"
+end
+```
+
+* <a name="graphql-argument-name"></a>
+  All argument names should be snake_case.
+  <sup>[[link](#graphql-argument-name)]</sup>
+
+```ruby
+# bad
+class BanUser < BaseMutation
+  argument :userId, ID, required: true
+end
+
+# good
+class BanUser < BaseMutation
+  argument :user_id, ID, required: true
+end
+```
+
+* <a name="graphql-argument-uniqueness"></a>
+  Avoid duplicate argument definitions.
+  <sup>[[link](#graphql-argument-uniqueness)]</sup>
+
+```ruby
+# bad
+class BanUser < BaseMutation
+  argument :user_id, ID, required: true
+  argument :user_id, ID, required: true
+end
+
+# good
+class BanUser < BaseMutation
+  argument :user_id, ID, required: true
+end
+```
+
+* <a name="graphql-field-definitions"></a>
+  All field definitions should be grouped together.
+  <sup>[[link](#graphql-field-definitions)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  field :first_name, String, null: true
+
+  def first_name
+    object.contact_data.first_name
+  end
+
+  field :last_name, String, null: true
+
+  def last_name
+    object.contact_data.last_name
+  end
+end
+
+# good
+class UserType < BaseType
+  field :first_name, String, null: true
+  field :last_name, String, null: true
+
+  def first_name
+    object.contact_data.first_name
+  end
+
+  def last_name
+    object.contact_data.last_name
+  end
+end
+```
+
+* <a name="graphql-field-description"></a>
+  Each field should have a description.
+  <sup>[[link](#graphql-field-description)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  field :name, String, null: true
+end
+
+# good
+class UserType < BaseType
+  field :name, String, "Name of the user", null: true
+end
+```
+
+* <a name="graphql-field-hash-key"></a>
+  Avoid unnecessary resolver methods in cases when :hash_key option can be used.
+  <sup>[[link](#graphql-field-hash-key)]</sup>
+
+```ruby
+# bad
+class Types::UserType < Types::BaseObject
+  field :phone, String, null: true
+   
+  def phone
+    object[:home_phone]
+  end
+end
+
+# good
+class Types::UserType < Types::BaseObject
+  field :phone, String, null: true, hash_key: :home_phone
+end
+```
+
+* <a name="graphql-field-mathod"></a>
+  Avoid unnecessary resolver methods in cases when :method option can be used.
+  <sup>[[link](#graphql-field-mathod)]</sup>
+
+```ruby
+# bad
+class Types::UserType < Types::BaseObject
+  field :phone, String, null: true
+   
+  def phone
+    object.home_phone
+  end
+end
+
+# good
+class Types::UserType < Types::BaseObject
+  field :phone, String, null: true, method: :home_phone
+end
+```
+
+* <a name="graphql-field-name"></a>
+  All field names should be snake_case.
+  <sup>[[link](#graphql-field-name)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  field :firstName, String, null: true
+end
+
+# good
+class UserType < BaseType
+  field :first_name, String, null: true
+end
+```
+
+* <a name="graphql-field-uniqueness"></a>
+  Avoid duplicate field definitions.
+  <sup>[[link](#graphql-field-uniqueness)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  field :name, String, null: true
+  field :phone, String, null: true
+  field :phone, String, null: true do
+    argument :something, String, required: false
+  end
+end
+
+# good
+class UserType < BaseType
+  field :name, String, null: true
+  field :phone, String, null: true do
+    argument :something, String, required: false
+  end
+end
+```
+
+* <a name="graphql-graphql-name"></a>
+  Types and mutations should have graphql_name configured only if it's different from the default name.
+  <sup>[[link](#graphql-graphql-name)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  graphql_name 'User'
+end
+
+# good
+class UserType < BaseType
+  graphql_name 'Viewer'
+end
+```
+
+* <a name="graphql-max-complexity-schema"></a>
+  Use max_complexity configuration in schema files.
+  <sup>[[link](#graphql-max-complexity-schema)]</sup>
+
+```ruby
+# good
+class AppSchema < BaseSchema
+  max_complexity 42
+end
+```
+
+* <a name="graphql-max-depth-schema"></a>
+  Use max_depth configuration in schema files.
+  <sup>[[link](#graphql-max-depth-schema)]</sup>
+
+```ruby
+# good
+class AppSchema < BaseSchema
+  max_depth 42
+end
+```
+
+* <a name="graphql-multiple-field-definitions"></a>
+  Fields with multiple definitions should be grouped together.
+  <sup>[[link](#graphql-multiple-field-definitions)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  field :first_name, String, null: true
+
+  def first_name
+    object.contact_data.first_name
+  end
+  field :first_name, Name, null: true
+end
+
+# good
+class UserType < BaseType
+  field :first_name, String, null: true
+  field :first_name, Name, null: true
+
+  def first_name
+    object.contact_data.first_name
+  end
+end
+```
+
+* <a name="graphql-not-authorized-node-type"></a>
+  Types that implement Node interface should have `.authorized?` check. Such types can be fetched by ID and therefore should have type level check to avoid accidental information exposure.
+  If `.authorized?` is defined in a parent class, you can add parent to the "SafeBaseClasses" to avoid offenses in children.
+  This cop also checks the `can_can_action` or `pundit_role` methods that can be used as part of the Ruby GraphQL Pro.
+  <sup>[[link](#graphql-not-authorized-node-type)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  implements GraphQL::Types::Relay::Node
+
+  field :uuid, ID, null: false
+end
+
+# good
+class UserType < BaseType
+  implements GraphQL::Types::Relay::Node
+
+  field :uuid, ID, null: false
+
+  def self.authorized?(object, context)
+    super && object.owner == context[:viewer]
+  end
+end
+
+# good
+class UserType < BaseType
+  implements GraphQL::Types::Relay::Node
+
+  pundit_role :staff
+
+  field :uuid, ID, null: false
+end
+
+# good
+class UserType < BaseType
+  implements GraphQL::Types::Relay::Node
+
+  can_can_action :staff
+
+  field :uuid, ID, null: false
+end
+```
+
+* <a name="graphql-object-description"></a>
+  All types (objects, inputs, interfaces, scalars, unions, mutations, subscriptions, and resolvers) should have a description.
+  <sup>[[link](#graphql-object-description)]</sup>
+
+```ruby
+# bad
+class Types::UserType < Types::BaseObject
+  ...
+end
+
+# good
+class Types::UserType < Types::BaseObject
+  description "Represents application user"
+  ...
+end
+```
+
+* <a name="graphql-ordered-arguments"></a>
+  Arguments should be alphabetically sorted within groups.
+  <sup>[[link](#graphql-ordered-arguments)]</sup>
+
+```ruby
+# bad
+class UpdateProfile < BaseMutation
+  argument :uuid, ID, required: true
+  argument :email, String, required: false
+  argument :name, String, required: false
+end
+
+# good
+class UpdateProfile < BaseMutation
+  argument :email, String, required: false
+  argument :name, String, required: false
+end
+
+# good
+class UpdateProfile < BaseMutation
+  argument :uuid, ID, required: true
+
+  argument :email, String, required: false
+  argument :name, String, required: false
+end
+```
+
+* <a name="graphql-ordered-fields"></a>
+  Fields should be alphabetically sorted within groups.
+  <sup>[[link](#graphql-ordered-fields)]</sup>
+
+```ruby
+# bad
+class UpdateProfile < BaseMutation
+  field :phone, String, null: true
+  field :name, String, null: true
+end
+
+# good
+class UpdateProfile < BaseMutation
+  field :name, String, null: true
+  field :phone, String, null: true
+end
+
+# good
+class UpdateProfile < BaseMutation
+  field :phone, String, null: true 
+
+  field :name, String, null: true
+end
+```
+
+* <a name="graphql-resolver-method-length"></a>
+  The length of a resolver method should not exceed 10 lines.
+  <sup>[[link](#graphql-resolver-method-length)]</sup>
+
+* <a name="graphql-unnecessary-argument-camelize"></a>
+  Avoid using :camelize option for arguments where it is unnecessary.
+  <sup>[[link](#graphql-unnecessary-argument-camelize)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  argument :filter, String, required: false, camelize: false
+end
+
+# good
+class UserType < BaseType
+  argument :filter, String, required: false
+end
+
+# good
+class UserType < BaseType
+  argument :email_filter, String, required: false, camelize: true
+end
+```
+
+* <a name="graphql-unnecessary-field-alias"></a>
+  Avoid defining an unnecessary alias, method, or resolver_method.
+  <sup>[[link](#graphql-unnecessary-field-alias)]</sup>
+
+```ruby
+# bad
+field :name, String, "Name of the user", null: true, alias: :name
+field :name, String, "Name of the user", null: true, method: :name
+field :name, String, "Name of the user", null: true, resolver_method: :name
+field :name, String, "Name of the user", null: true, hash_key: :name
+
+# good
+field :name, String, "Name of the user", null: true, alias: :real_name
+field :name, String, "Name of the user", null: true, method: :real_name
+field :name, String, "Name of the user", null: true, resolver_method: :real_name
+field :name, String, "Name of the user", null: true, hash_key: :real_name
+```
+
+* <a name="graphql-unnecessary-field-camelize"></a>
+  Avoid using :camelize option for fields where it is unnecessary.
+  <sup>[[link](#graphql-unnecessary-field-camelize)]</sup>
+
+```ruby
+# bad
+class UserType < BaseType
+  field :name, "Name of the user", String, null: true, camelize: true
+end
+
+# good
+class UserType < BaseType
+  field :name, String, "Name of the user", null: true
+end
+
+# good
+class UserType < BaseType
+  field :first_name, "Name of the user", String, null: true, camelize: true
+end
+```
+
+* <a name="graphql-unused-argument"></a>
+  Arguments should either be listed explicitly or **rest should be in the resolve signature (and similar methods, such as #authorized?).
+  <sup>[[link](#graphql-unused-argument)]</sup>
+
+```ruby
+# bad
+class SomeResolver < Resolvers::Base
+  type SomeType, null: false
+
+  argument :arg1, String, required: true
+  argument :arg2, String, required: true
+
+  def authorized?; end
+  def resolve(arg1:); end
+end
+
+# bad
+class SomeResolver < Resolvers::Base
+  type SomeType, null: false
+
+  argument :arg1, String, required: true
+  argument :arg2, String, required: true
+
+  def resolve; end
+end
+
+# good
+class SomeResolver < Resolvers::Base
+  argument :arg1, String, required: true
+  argument :user_id, String, required: true, loads: Types::UserType
+  argument :post_id, String, loads: Types::PostType, as: :article
+  argument :comment_ids, String, loads: Types::CommentType
+
+  def authorized?(arg1:, user:, article:, comments:); end
+  def resolve(arg1:, user:, article:, comments:); end
+end
+
+# good
+class SomeResolver < Resolvers::Base
+  argument :arg1, String, required: true
+  argument :user_id, String, required: true, loads: Types::UserType
+  argument :comment_ids, String, loads: Types::CommentType
+
+  def resolve(arg1:, **rest); end
+end
+
+# good
+class SomeResolver < Resolvers::Base
+  type SomeType, null: false
+
+  argument :arg1, String, required: true
+  argument :arg2, String, required: true
+
+  def resolve(args); end
+end
+```

data/lib/datarockets_style/formatter/todo_list_formatter.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require "pathname"
-require_relative "./todo_list_formatter/report_summary"
+require_relative "todo_list_formatter/report_summary"
 
 # This formatter works like default formattter (display dots for files with no offenses and
 # letters for files with problems in the them).

data/lib/datarockets_style/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DatarocketsStyle
-  VERSION = "1.3.0"
+  VERSION = "1.5.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: datarockets-style
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Roman Dubrovsky
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-12-02 00:00:00.000000000 Z
+date: 2023-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,85 +16,57 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.39'
+        version: 1.56.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.39'
+        version: 1.56.3
 - !ruby/object:Gem::Dependency
-  name: rubocop-rails
+  name: rubocop-graphql
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.17.3
+        version: '1.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.17.3
+        version: '1.4'
 - !ruby/object:Gem::Dependency
-  name: rubocop-rspec
+  name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.15'
+        version: '2.21'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.15'
-- !ruby/object:Gem::Dependency
-  name: pry-byebug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.21'
 - !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-- !ruby/object:Gem::Dependency
-  name: rspec
+  name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.10'
-  type: :development
+        version: '2.24'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.10'
-description: 
+        version: '2.24'
+description:
 email:
 - r.dubrovsky@datarockets.com
 executables: []
@@ -120,12 +92,14 @@ files:
 - bin/console
 - bin/setup
 - config/base.yml
+- config/graphql.yml
 - config/rails-locales.yml
 - config/rails.yml
 - config/rspec.yml
 - config/ruby.yml
 - datarockets-style.gemspec
 - doc/STYLE_GUIDE.md
+- doc/STYLE_GUIDE_GRAPHQL.md
 - doc/STYLE_GUIDE_RAILS.md
 - doc/STYLE_GUIDE_RSPEC.md
 - lib/datarockets_style.rb
@@ -142,7 +116,7 @@ licenses:
 metadata:
   allowed_push_host: https://rubygems.org
   rubygems_mfa_required: 'true'
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -150,15 +124,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.6.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: Datarockets style guides and shared style configs
 test_files: []