hanami/chat

@kaikuchn maybe is not needed:

require 'hanami'

class Signup
  include Hanami::Validations

  validations do
    required(:name)
  end
end

p Signup.new(name: nil).validate.success? # true

which means name can be nil without maybe and, to me, it is correct, because the only thing that required implies is a structure-level rule whether the key exists or not, regardless of its value.

So, to me, I believe that the p Signup.new.validate.success?should print false, as long as you only have plain required(:name).

Like when we use dry-validation here:

require 'dry-validation'

class Signup < Dry::Validation::Contract
  params do
    required(:name)
  end
end

contract = Signup.new


p contract.call({}).success? # false
p contract.call({name: 'Peter'}).success? # true 
p contract.call({name: nil}).success? # true

Am I wrong to thing that Hanami internally is supposed to be using dry-validation|dry-schema?