Нам нужны твои мозги

Хотите расти как разработчик и найти крутую работу? Не протирайте штаны — займитесь Open Source проектами. Так легче всего попасть в лучшие команды разработчиков и положить себе в резюме настоящий проект, вместо нелепых «примеров кода». Но найти подходящий проект для участия сложно. Начинаются лень и отговорки, а за ними — отсутствие профессионального роста, критики по-настоящему крутых программистов, уныние и застой.

На Cult of Martians мы собираем интересные задачи для современных веб-программистов. Можно выбрать подходящую по сложности, продолжительности и специализации. Задачи не выдуманы «из воздуха» — каждая решает насущную проблему, и решить ее можно через создание нового Open Source проекта или улучшение существующего. Решайте задачи, прокачивайтесь, присылайте решение на оценку. Лучших могут пригласить к себе на работу компании, программистам которых понравится ваше решение.

В работе 0Короткие 0Длинные 0Сделано 229
Помощь запрашивал
Дмитрий Цепелев
 Дмитрий Цепелев
Помог
Shpak Roman
 Shpak Roman

Бэк: GraphQL RuboCop: добавить коп для обнаружения полей, которые можно вынести в отдельный тип

Для новичков, задача на неделю

rubocop-graphql — Ruby gem, проверяющий соответствие стилю код, использующий DSL graphql-ruby.

Необходимо реализовать коп ExtractType, предлагающий вынести поля с одинаковым префиксом в отдельный тип.

Польза: научиться писать копы для rubocop, принести пользу Ruby-сообществу.

При проектировании GraphQL схемы считается хорошей практикой выносить связанные поля в отдельные типы даже в случаях, когда соответствующие данные хранятся в одной таблице с другими данными. Такие ситуации часто возникают, когда поддержка GraphQL добавляется в существующие большие приложения. К примеру, если контактные данные пользователя хранятся в табличке users, то тип может получиться таким:

class Types::UserType < Types::BaseObject
  field :registered_at, String, null: false
  field :contact_phone, String, null: false
  field :contact_first_name, String, null: false
  field :contact_last_name, String, null: false
end

Лучше вынести все контактные данные в отдельный тип:

class Types::ContactType < Types::BaseObject
  field :phone, String, null: false
  field :first_name, String, null: false
  field :last_name, String, null: false
end

class Types::UserType < Types::BaseObject
  field :registered_at, String, null: false
  field :contact, Types::ContactType, null: false

  def contact
    self
  end
end

Постановка задачи

Если внутри класса—типа объявлено несколько полей с одним и тем же префиксом, то нужно предлагать выделить их в отдельный тип:

class Types::UserType < Types::BaseObject
  field :registered_at, String, null: false
  field :contact_first_name, String, null: false
  field :contact_last_name, String, null: false
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Consider moving contact_first_name, contact_last_name to a new type and adding the `contact` field instead
end

Ошибку нужно добавлять к последнему из полей. Количество полей задаётся параметром MaxFields, при достижении которого срабатывает коп; значение по умолчанию — 2.

Советы по реализации

  1. Разобраться с node pattern — инструментом rubocop для работы с AST.
  2. Конфигурация по умолчанию находится в config/default.yml.
  3. Примеры работы с конфигурацией можно подсмотреть в rubocop-md или rubocop-rspec.

Инструкции по выполнению

  1. Обсудить задачу, если есть вопросы.
  2. Форкнуть проект rubocop-graphql на GitHub.
  3. Реализовать необходимый функционал и покрыть его тестами.
  4. Сделать Pull Request.
Посмотреть решение