rider-kick 0.0.15 → 0.0.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a5129eebe54dcc3056758fead3e72d4a98a788cd71845dac6f5d9611792a1704
-  data.tar.gz: 56bfecfa53f00172d78a7e8b91db4908042fc71f2075da5d5f2d7adb0ec53a3c
+  metadata.gz: 3c933ba32d84ebd605729faf5dddd9ee0392e2954b160c0e2ef0a23570f5c636
+  data.tar.gz: fab65284c253086df7acb0b40e89f078f20d436f241b63c3a0352e36d96127f2
 SHA512:
-  metadata.gz: ad11a0d3998b1f69ef33e70739a3e75785356d3e63960641f80d06047a8f0f9c3661c4ebece9a1c119d03370900b716e2afc65aa3ada2a47565fedb86e548713
-  data.tar.gz: 67175aad3253eb90b31a8987d87bcda6ffe3abf69358719d7451f0d310ab76d16c16341635056b96acd179c3136466abcfd60a4bf24463c90bb13147a383089b
+  metadata.gz: e1dc64862b3707c35dd437dd9f2ce1af940d20fcf9f8c99f0160c65bc77c99f0aba9af9046a8ccc2ea8309ce56fff57cd6fada35afa46b996ff36f6462b645e5
+  data.tar.gz: 908f66d02665cac5897adb37cfc215a34cccee066bf009b111037a82f0a58ad905ac327d54e77bcbb0c67da65d8b3d75eae6aaa4479b65902e4581d6e1be61ff

data/lib/generators/rider_kick/clean_arch_generator.rb

@@ -221,6 +221,7 @@ module RiderKick
 
     def gem_dependencies
       <<~RUBY
+
         group :development, :test do
           gem "rspec-rails"
           gem "factory_bot_rails"
@@ -241,9 +242,6 @@ module RiderKick
 
         # pagination
         gem 'pagy'
-
-        # models validation
-        gem 'schema_validations'
       RUBY
     end
 

data/lib/generators/rider_kick/scaffold_generator.rb

@@ -93,11 +93,11 @@ module RiderKick
 
         unless @model_class.column_names.include?(field_name)
           raise ValidationError.new(
-            "Repository filter error: Field '#{field_name}' tidak ditemukan di model #{@model_class}",
-            structure_file:    "#{arg_structure}_structure.yaml",
-            field_name:        field_name,
-            model_class:       @model_class.to_s,
-            available_columns: @model_class.column_names
+                  "Repository filter error: Field '#{field_name}' tidak ditemukan di model #{@model_class}",
+                  structure_file:    "#{arg_structure}_structure.yaml",
+                  field_name:        field_name,
+                  model_class:       @model_class.to_s,
+                  available_columns: @model_class.column_names
           )
         end
       end
@@ -110,12 +110,12 @@ module RiderKick
 
       if missing_fields.any?
         raise ValidationError.new(
-          "Entity configuration error: Field(s) tidak ditemukan di model #{@model_class}",
-          structure_file:    "#{arg_structure}_structure.yaml",
-          missing_fields:    missing_fields,
-          model_class:       @model_class.to_s,
-          available_columns: @model_class.column_names,
-          suggestion:        "Update section 'entity.db_attributes' di YAML file"
+                "Entity configuration error: Field(s) tidak ditemukan di model #{@model_class}",
+                structure_file:    "#{arg_structure}_structure.yaml",
+                missing_fields:    missing_fields,
+                model_class:       @model_class.to_s,
+                available_columns: @model_class.column_names,
+                suggestion:        "Update section 'entity.db_attributes' di YAML file"
         )
       end
     end
@@ -131,13 +131,13 @@ module RiderKick
     def setup_structure_config
       # Determine structure file path based on engine configuration
       structure_path = if RiderKick.configuration.engine_name.present?
-                         # For engines, read structure file from engine's db/structures directory
-                         engine_name = RiderKick.configuration.engine_name.downcase
-                         "engines/#{engine_name}/db/structures/#{arg_structure}_structure.yaml"
-                       else
-                         # For main app, read from host's db/structures directory
-                         "db/structures/#{arg_structure}_structure.yaml"
-                       end
+        # For engines, read structure file from engine's db/structures directory
+        engine_name = RiderKick.configuration.engine_name.downcase
+        "engines/#{engine_name}/db/structures/#{arg_structure}_structure.yaml"
+      else
+        # For main app, read from host's db/structures directory
+        "db/structures/#{arg_structure}_structure.yaml"
+      end
 
       validate_yaml_format!(structure_path)
       config     = YAML.load_file(structure_path)
@@ -200,15 +200,15 @@ module RiderKick
 
       # Route scope
       #  @route_scope_path  = arg_scope.fetch('scope', '').to_s.downcase
-      @route_scope_path  = @structure.domain.to_s.downcase
+      @route_scope_path  = @structure.scope.to_s.downcase
       @route_scope_class = @route_scope_path.camelize
 
       # Baca actor_id dari structure.yaml jika ada, jika tidak generate dari actor
       @actor_id = if @structure.actor_id.present?
-                    @structure.actor_id.to_s
-                  elsif @actor.present?
-                    "#{@actor.to_s.downcase}_id"
-                  end
+        @structure.actor_id.to_s
+      elsif @actor.present?
+        "#{@actor.to_s.downcase}_id"
+      end
 
       # Set flag untuk setiap action apakah resource_owner_id atau actor_id ada di contract
       # Ini digunakan di template repository untuk conditional logic
@@ -231,19 +231,19 @@ module RiderKick
 
       # Get contract untuk action tertentu
       contract = case action.to_s
-                 when 'list'
-                   @contract_list
-                 when 'fetch', 'fetch_by_id'
-                   @contract_fetch_by_id
-                 when 'create'
-                   @contract_create
-                 when 'update'
-                   @contract_update
-                 when 'destroy'
-                   @contract_destroy
-                 else
-                   []
-                 end
+      when 'list'
+        @contract_list
+      when 'fetch', 'fetch_by_id'
+        @contract_fetch_by_id
+      when 'create'
+        @contract_create
+      when 'update'
+        @contract_update
+      when 'destroy'
+        @contract_destroy
+      else
+        []
+      end
       contract ||= []
       # Check apakah contract string mengandung field name
       # Pattern: "required(:field_name)" atau "optional(:field_name)"
@@ -372,7 +372,7 @@ module RiderKick
 
     def contract_fields
       @model_class.columns.reject { |c| ['id', 'created_at', 'updated_at', 'type'].include?(c.name.to_s) }
-                  .map { |c| c.name.to_s }
+        .map { |c| c.name.to_s }
     end
 
     # --- AWAL BLOK MODIFIKASI: (PERBAIKAN KEGAGALAN #1) ---

data/lib/generators/rider_kick/scaffold_generator_contracts_with_scope_spec.rb

@@ -70,7 +70,7 @@ RSpec.describe 'rider_kick:scaffold contracts (with scope)' do
 
         klass.new(['users', 'scope:dashboard']).generate_use_case
 
-        base = RiderKick.configuration.domains_path + '/use_cases/dashboard/users'
+        RiderKick.configuration.domains_path
         ['owner_list_user', 'owner_fetch_user_by_id', 'owner_create_user', 'owner_update_user', 'owner_destroy_user'].each do |uc|
           # expect(File).to exist("#{base}/#{uc}.rb")
         end

data/lib/generators/rider_kick/scaffold_generator_list_spec_format_spec.rb

@@ -47,10 +47,12 @@ RSpec.describe 'rider_kick:scaffold list spec generation' do
           resource_name: article
           actor: owner
           resource_owner_id: account_id
+          resource_owner: account
           domains:
             action_list:
               use_case:
-                contract: []
+                contract:
+                  - "required(:account_id).filled(:string)"
               repository:
                 filters:
                   - "{ field: 'title', type: 'search' }"
@@ -73,13 +75,15 @@ RSpec.describe 'rider_kick:scaffold list spec generation' do
         content = File.read(list_spec_file)
 
         # Verify format matches user's requirements
-        expect(content).to include('let(:repository) { described_class }')
-        expect(content).to include('repository.new(params: params).call')
-        expect(content).to include('account_id: SecureRandom.uuid')
-        expect(content).to include('create_list(:article, 3, account_id: params[:account_id])')
-        expect(content).to include('context \'with search filters\'')
-        expect(content).to include('context \'with resource owner filter\'')
-        expect(content).to include('context \'with sorting\'')
+        expect(content).to include('require "rails_helper"')
+        expect(content).to include('subject(:result) { described_class.new(params: params).call }')
+        expect(content).to include('let(:account) { create(:account) }')
+        expect(content).to include('let(:account_id) { account.id }')
+        expect(content).to include('let!(:article_recent) { create(:article, account_id: account_id, created_at: 1.hour.ago) }')
+        expect(content).to include('let!(:other_account_article) { create(:article, account_id: other_account_id) }')
+        expect(content).to include('context "with valid params"')
+        expect(content).to include('context "with pagination"')
+        expect(content).to include('context "when account has no data"')
       end
     end
   end
@@ -142,9 +146,11 @@ RSpec.describe 'rider_kick:scaffold list spec generation' do
         content = File.read(list_spec_file)
 
         # Verify format without resource_owner_id
-        expect(content).to include('let(:repository) { described_class }')
-        expect(content).to include('repository.new(params: params).call')
-        expect(content).to include('create_list(:product, 3)')
+        expect(content).to include('require "rails_helper"')
+        expect(content).to include('subject(:result) { described_class.new(params: params).call }')
+        expect(content).to include('let!(:product_recent) { create(:product, created_at: 1.hour.ago) }')
+        expect(content).to include('context "when no resources exist"')
+        expect(content).to include('Models::Product.destroy_all')
         expect(content).not_to include('account_id')
         expect(content).not_to include('user_id')
       end

data/lib/generators/rider_kick/scaffold_generator_rspec_spec.rb

@@ -131,23 +131,14 @@ RSpec.describe 'rider_kick:scaffold generator (RSpec generation)' do
 
           # Verifikasi konten spec file
           spec_content = File.read(spec_file)
-          expect(spec_content).to include('require \'rails_helper\'')
+          expect(spec_content).to include('require "rails_helper"')
           expect(spec_content).to include('RSpec.describe')
-          expect(spec_content).to include('describe \'#call\'')
+          expect(spec_content).to include('describe "#call"')
 
           # Only check Hashie::Mash for create, update, destroy, fetch_by_id (list uses simple hash)
           if ['create_product', 'update_product', 'destroy_product', 'fetch_product_by_id'].include?(repo)
             expect(spec_content).to include('Hashie::Mash.new')
           end
-
-          # Verify error mocking for create, update, destroy (but not list or fetch_by_id)
-          if ['create_product', 'update_product', 'destroy_product'].include?(repo)
-            expect(spec_content).to include('let(:error_messages)')
-            expect(spec_content).to include('let(:active_model_errors)')
-            expect(spec_content).to include('allow(errors).to receive(:each)')
-            expect(spec_content).to include('double(as_json:')
-            expect(spec_content).to include("'options' => { 'message' => 'must be valid format' }")
-          end
         end
 
         # Builder spec (covers entity validation too)

data/lib/generators/rider_kick/scaffold_generator_with_scope_spec.rb

@@ -62,7 +62,7 @@ RSpec.describe 'rider_kick:scaffold generator (with scope)' do
         instance.generate_use_case
 
         # use_cases berada di app/domains/use_cases/dashboard/users/...
-        path = RiderKick.configuration.domains_path + '/use_cases/dashboard/users'
+        RiderKick.configuration.domains_path
         ['owner_create_user', 'owner_update_user', 'owner_list_user', 'owner_destroy_user', 'owner_fetch_user_by_id'].each do |uc|
           # expect(File).to exist(File.join(path, "#{uc}.rb"))
         end

data/lib/generators/rider_kick/templates/config/initializers/pagy.rb.tt

@@ -1,13 +1,48 @@
-# encoding: utf-8
 # frozen_string_literal: true
 
-# Optionally override some pagy default with your own in the pagy initializer
-Pagy::DEFAULT[:limit] = 30 # items per page
-Pagy::DEFAULT[:size]  = 9  # nav bar links
-# Better user experience handled automatically
-require 'pagy/extras/overflow'
-Pagy::DEFAULT[:overflow] = :empty_page
-require 'pagy/extras/elasticsearch_rails'
-require 'pagy/extras/array'
+# Pagy initializer file (43.2.4)
+# See https://ddnexus.github.io/pagy/resources/initializer/
 
-include Pagy::Backend
+############ Global Options ################################################################
+# See https://ddnexus.github.io/pagy/toolbox/options/ for details.
+# Add your global options below. They will be applied globally.
+# For example:
+#
+Pagy.options[:limit] = 10               # Limit the items per page
+Pagy.options[:client_max_limit] = 50   # The client can request a limit up to 100
+Pagy.options[:max_pages] = 200          # Allow only 200 pages
+# Pagy.options[:jsonapi] = true           # Use JSON:API compliant URLs
+
+
+############ JavaScript ####################################################################
+# See https://ddnexus.github.io/pagy/resources/javascript/ for details.
+# Examples for Rails:
+# For apps with an assets pipeline
+# Rails.application.config.assets.paths << Pagy::ROOT.join('javascripts')
+#
+# For apps with a javascript builder (e.g. esbuild, webpack, etc.)
+# javascript_dir = Rails.root.join('app/javascript')
+# Pagy.sync_javascript(javascript_dir, 'pagy.mjs') if Rails.env.development?
+
+
+############# Overriding Pagy::I18n Lookup #################################################
+# Refer to https://ddnexus.github.io/pagy/resources/i18n/ for details.
+# Override the I18n lookup by dropping your custom dictionary in some pagy dir.
+# Example for Rails:
+#
+# Pagy::I18n.pathnames << Rails.root.join('config/locales/pagy')
+
+
+############# I18n Gem Translation #########################################################
+# See https://ddnexus.github.io/pagy/resources/i18n/ for details.
+#
+# Pagy.translate_with_the_slower_i18n_gem!
+
+
+############# Calendar Localization for non-en locales ####################################
+# See https://ddnexus.github.io/pagy/toolbox/paginators/calendar#localization for details.
+# Add your desired locales to the list and uncomment the following line to enable them,
+# regardless of whether you use the I18n gem for translations or not, whether with
+# Rails or not.
+#
+# Pagy::Calendar.localize_with_rails_i18n_gem(*your_locales)

data/lib/generators/rider_kick/templates/db/structures/example.yaml.tt

@@ -8,8 +8,7 @@ resource_owner_id: <%= @resource_owner_id %> # account_id
 resource_owner: <%= @resource_owner %> # account
 actor: <%= @actor %> # user
 actor_id: <%= @actor_id %> # user_id
-scope: '' #dashboard
-domain: '' # core
+scope: '' # dashboard
 
 fields:
 <% @fields.each do |f| -%>

data/lib/generators/rider_kick/templates/domains/core/builders/builder_spec.rb.tt

@@ -6,30 +6,41 @@ RSpec.describe <%= domain_class_name %>::Builders::<%= @subject_class %>, type:
   describe '#build' do
     let(:<%= @variable_subject %>) do
       ClassStubber::Model.new(
-        'id' => 'test-id-123',
-<% 
+        'id' => SecureRandom.uuid,
+<%
 # Get uploader field names to exclude from entity_db_fields
 uploader_names = @uploaders.map { |u| u.name.to_s }
 -%>
 <% @entity_db_fields.each do |field| -%>
 <% next if uploader_names.include?(field.to_s) -%>
 <% column_meta = get_column_meta(field) -%>
-<% case column_meta[:type].to_s -%>
-<% when 'datetime', 'timestamp' -%>
-        '<%= field %>' => Time.current,
-<% when 'date' -%>
-        '<%= field %>' => Date.current,
-<% when 'time' -%>
-        '<%= field %>' => Time.current,
-<% when 'boolean' -%>
-        '<%= field %>' => true,
-<% when 'integer', 'bigint' -%>
-        '<%= field %>' => 123,
-<% when 'decimal', 'float' -%>
-        '<%= field %>' => 123.45,
-<% else -%>
-        '<%= field %>' => '<%= field %>_value',
-<% end -%>
+<%
+  faker_val = case field.to_s
+              when /email/ then "Faker::Internet.email"
+              when /name/ then "Faker::Name.name"
+              when /first_name/ then "Faker::Name.first_name"
+              when /last_name/ then "Faker::Name.last_name"
+              when /title/ then "Faker::Book.title"
+              when /description/ then "Faker::Lorem.paragraph"
+              when /address/ then "Faker::Address.full_address"
+              when /phone/ then "Faker::PhoneNumber.cell_phone"
+              when /url/, /website/ then "Faker::Internet.url"
+              when /price/ then "Faker::Commerce.price"
+              else
+                case column_meta[:type].to_s
+                when 'datetime', 'timestamp' then "Faker::Time.backward(days: 14)"
+                when 'date' then "Faker::Date.backward(days: 14)"
+                when 'time' then "Faker::Time.backward(days: 14)"
+                when 'boolean' then "Faker::Boolean.boolean"
+                when 'integer', 'bigint' then "Faker::Number.number(digits: 2)"
+                when 'float' then "Faker::Number.decimal(l_digits: 2, r_digits: 2).to_f"
+                when 'decimal' then "Faker::Number.decimal(l_digits: 2, r_digits: 2).to_d"
+                when 'uuid' then "SecureRandom.uuid"
+                else "Faker::Lorem.word"
+                end
+              end
+-%>
+        '<%= field %>' => <%= faker_val %>,
 <% end -%>
 <% @uploaders.each do |uploader| -%>
 <% if uploader.type == 'single' -%>
@@ -52,7 +63,7 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
       entity = builder.build
 
       expect(entity).to be_a(<%= domain_class_name %>::Entities::<%= @subject_class %>)
-      expect(entity.id).to eq('test-id-123')
+      expect(entity.id).to be_a(String)
 <% @entity_db_fields.first(2).each do |field| -%>
 <% column_meta = get_column_meta(field) -%>
 <% case column_meta[:type].to_s -%>
@@ -61,20 +72,22 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
 <% when 'date' -%>
       expect(entity.<%= field %>).to be_a(Date)
 <% when 'boolean' -%>
-      expect(entity.<%= field %>).to eq(true)
+      expect([true, false]).to include(entity.<%= field %>)
 <% when 'integer', 'bigint' -%>
-      expect(entity.<%= field %>).to eq(123)
+      expect(entity.<%= field %>).to be_a(Integer)
 <% when 'decimal', 'float' -%>
-      expect(entity.<%= field %>).to eq(123.45)
+      expect(entity.<%= field %>).to be_a(Numeric)
+<% when 'uuid' -%>
+      expect(entity.<%= field %>).to be_a(String)
 <% else -%>
-      expect(entity.<%= field %>).to eq('<%= field %>_value')
+      expect(entity.<%= field %>).to be_a(String)
 <% end -%>
 <% end -%>
     end
 
     it 'includes all required entity attributes', :aggregate_failures do
       entity = builder.build
-      
+
       # Entity must have these attributes (keys must exist, values can be nil for optional)
       expect(entity).to respond_to(:id)
 <% @entity_db_fields.each do |field| -%>
@@ -96,7 +109,7 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
     describe '#attributes_for_entity' do
       it 'returns hash with uploader attributes', :aggregate_failures do
         attributes = builder.send(:attributes_for_entity)
-        
+
         expect(attributes).to be_a(Hash)
 <% @uploaders.each do |uploader| -%>
 <% if uploader.type == 'single' -%>
@@ -109,7 +122,7 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
 
       it 'generates correct URL attributes', :aggregate_failures do
         attributes = builder.send(:attributes_for_entity)
-        
+
 <% @uploaders.each do |uploader| -%>
 <% if uploader.type == 'single' -%>
         expect(attributes[:<%= uploader.name %>_url]).to eq('http://example.com/<%= uploader.name %>.jpg')
@@ -125,14 +138,14 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
     it 'ensures all entity attributes have correct keys and types', :aggregate_failures do
       entity = builder.build
       entity_hash = entity.to_h
-      
+
       # All entity attributes must have keys in the hash (source of truth from entity)
       expect(entity_hash).to have_key(:id)
       expect(entity.id).to be_a(String)
 <% @entity_db_fields.each do |field| -%>
 <% next if uploader_names.include?(field.to_s) -%>
 <% column_meta = get_column_meta(field) -%>
-      
+
       expect(entity_hash).to have_key(:<%= field %>)
 <% case column_meta[:type].to_s -%>
 <% when 'datetime', 'timestamp', 'time' -%>
@@ -151,19 +164,19 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
 <% end -%>
 <% @uploaders.each do |uploader| -%>
 <% if uploader.type == 'single' -%>
-      
+
       expect(entity_hash).to have_key(:<%= uploader.name %>_url)
       expect(entity.<%= uploader.name %>_url).to be_a(String).or be_nil
 <% else -%>
-      
+
       expect(entity_hash).to have_key(:<%= uploader.name %>_urls)
       expect(entity.<%= uploader.name %>_urls).to be_a(Array)
 <% end -%>
 <% end -%>
-      
+
       expect(entity_hash).to have_key(:created_at)
       expect(entity.created_at).to be_a(Time) if entity.created_at.present?
-      
+
       expect(entity_hash).to have_key(:updated_at)
       expect(entity.updated_at).to be_a(Time) if entity.updated_at.present?
     end
@@ -171,7 +184,7 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
     it 'validates entity type schema definitions', :aggregate_failures do
       # Verify entity has correct Dry::Types definitions
       schema = <%= domain_class_name %>::Entities::<%= @subject_class %>.schema
-      
+
       # ID must be required (not optional)
       id_key = schema.key(:id)
       expect(id_key.required?).to be true
@@ -180,7 +193,7 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
 <% @entity_db_fields.each do |field| -%>
 <% next if uploader_names.include?(field.to_s) -%>
 <% column_meta = get_column_meta(field) -%>
-      
+
       # <%= field %>: <%= column_meta[:null] ? 'optional' : 'required' %>
 <% if column_meta[:null] -%>
       # Optional field - can be nil
@@ -195,22 +208,22 @@ uploader_names = @uploaders.map { |u| u.name.to_s }
 <% end -%>
 <% @uploaders.each do |uploader| -%>
 <% if uploader.type == 'single' -%>
-      
+
       # <%= uploader.name %>_url: optional (uploader can be nil)
       <%= uploader.name %>_url_key = schema.key(:<%= uploader.name %>_url)
       expect(<%= uploader.name %>_url_key.required?).to be false
 <% else -%>
-      
+
       # <%= uploader.name %>_urls: required (but can be empty array)
       <%= uploader.name %>_urls_key = schema.key(:<%= uploader.name %>_urls)
       expect(<%= uploader.name %>_urls_key.required?).to be true
 <% end -%>
 <% end -%>
-      
+
       # Timestamps - check their actual definition
       created_at_key = schema.key(:created_at)
       updated_at_key = schema.key(:updated_at)
-      
+
       # Verify timestamps exist (they might be optional in some entities)
       expect(schema.keys.map(&:name)).to include(:created_at, :updated_at)
     end

data/lib/generators/rider_kick/templates/domains/core/repositories/abstract_repository.rb.tt

@@ -11,12 +11,14 @@ class <%= domain_class_name %>::Repositories::AbstractRepository
     record.errors.to_a.join(', ')
   end
 
+  def build_error(code, message, attribute: nil)
+    [{ code: code, message: message, attribute: attribute }]
+  end
+
   def build_errors(resource)
-    errors = []
-    resource.errors.each do |error|
-      errors << <%= domain_class_name %>::Builders::Error.new(error.as_json).build
+    resource.errors.map do |error|
+      { code: error.type.to_s, message: error.message, attribute: error.attribute.to_s }
     end
-    errors
   end
 
   def prepare!(params, sanitize: true)
@@ -33,7 +35,7 @@ class <%= domain_class_name %>::Repositories::AbstractRepository
 
   def build_pagination_variable!(params)
     @per_page = params[:per_page] || Pagy::DEFAULT[:limit]
-    @page     = params[:page] || Pagy::DEFAULT[:page]
+    @page     = params[:page] || 1
     @search   = params[:search]
   end
 end