rage-rb 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4570bfdf4f86125be1d6c35b9ccba08a956fe8a20dac392e5ae914db347ad396
-  data.tar.gz: '019e919339f97e641e65f9aecc91c02740875ebcbe71edf14a07144950e47f07'
+  metadata.gz: 7684c979952ee81bee165b9b667dea87199f3fa28dadadc4878b29a12211fdac
+  data.tar.gz: 2428443ee2456ef375990decfbf36a2952670ee517ac07cf4c994f5b9b7076f9
 SHA512:
-  metadata.gz: 9ad7b89eb46407831ae723c1c15b87ea12b898ae1784989374b07fcffcc3afcff2a237861c94565afe4c5f486b99edb15022b539ab9cad142778e4b2718b92ea
-  data.tar.gz: 471cb6bcbf294d9a4eb3e5fd69827f26661effafc1df20447b8e7eddd0816aff411266d22104988978c46318dc08c4fa753f1e1a4fa1327268a5f24d8851ac5a
+  metadata.gz: f59da6b77e69fa2b89761b30a497b4c0fa8f02cbabeed65c953b0f5e807b349e5b305bf31564bcc3eaeed4957215766e8781af5a5b8bbd9754e0461fa18f7b87
+  data.tar.gz: 733a15ef91abcbfc079a3bbbb83f82686038059b3c518c1c0a279e8000ad885c65cc5019cc691e2731fa1e59ca1f62e8fe95c8f454837ed0bdc65e181157ffc7

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 ## [Unreleased]
 
+## [0.3.0] - 2023-10-08
+
+### Added
+
+- CLI `routes` task.
+- CLI `console` task.
+- `:if` and `:unless` options in `before_action`.
+- Allow to set response headers.
+- Block version of `before_action`.
+
 ## [0.2.0] - 2023-09-27
 
 ### Added

data/Gemfile

@@ -11,6 +11,3 @@ gem "rspec", "~> 3.0"
 
 gem "pg"
 gem "mysql2"
-
-gem "benchmark-ips"
-gem "mustache"

data/README.md

@@ -142,7 +142,7 @@ end
 Version | Changes
 ------- |------------
 0.2 :white_check_mark: | ~~Gem configuration by env.<br>Add `skip_before_action`.<br>Add `rescue_from`.<br>Router updates:<br>&emsp;• make the `root` helper work correctly with `scope`;<br>&emsp;• support the `defaults` option;~~
-0.3 | CLI updates:<br>&emsp;• `routes` task;<br>&emsp;• `console` task;<br>Support the `:if` and `:unless` options in `before_action`.<br>Allow to set response headers.
+0.3 :white_check_mark: | ~~CLI updates:<br>&emsp;• `routes` task;<br>&emsp;• `console` task;<br>Support the `:if` and `:unless` options in `before_action`.<br>Allow to set response headers.~~
 0.4 | Expose the `params` object.<br>Support header authentication with `authenticate_with_http_token`.<br>Router updates:<br>&emsp;• add the `resources` route helper;<br>&emsp;• add the `namespace` route helper;<br>&emsp;• support regexp constraints;
 0.5 | Implement Iodine-based equivalent of `ActionController::Live`.<br>Use `ActionDispatch::RemoteIp`.
 0.6 | Expose the `cookies` object.<br>Expose the `send_data` and `send_file` methods.<br>Support conditional get with `etag` and `last_modified`.

data/lib/rage/application.rb

@@ -19,7 +19,7 @@ class Rage::Application
       end
 
     rescue => e
-      [500, {}, ["#{e.class}:#{e.message}\n\n#{e.backtrace.join("\n")}"]] # TODO: check Rage.env
+      [500, {}, ["#{e.class}:#{e.message}\n\n#{e.backtrace.join("\n")}"]]
 
     ensure
       # notify Iodine the request can now be served

data/lib/rage/cli.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
-
 require "thor"
 require "rage"
+require "irb"
 
 module Rage
   class CLI < Thor
@@ -25,11 +25,81 @@ module Rage
 
       ::Iodine.start
     end
+
+    desc 'routes', 'List all routes.'
+    option :grep, aliases: "-g", desc: "Filter routes by pattern"
+    def routes
+      # the result would be something like this:
+      # Action    Verb  Path  Controller#Action
+      # index     GET   /     application#index
+
+      # load config/application.rb
+      environment
+
+      routes = Rage.__router.routes
+
+      pattern = options[:grep]
+
+      if pattern
+        routes = routes.select do |route|
+          route[:path].match?(pattern) || route[:raw_handler].to_s.match?(pattern) || route[:method].match?(pattern)
+        end
+      end
+
+      return puts 'Action    Verb  Path  Controller#Action' if routes.empty?
+  
+      # construct a table
+      table = []
+
+      # longest_path is either the length of the longest path or 5
+      longest_path = routes.map { |route| route[:path].length }.max + 3
+      longest_path = longest_path > 5 ? longest_path : 5
+
+      longest_verb = routes.map { |route| route[:method].length }.max + 3
+      longest_verb = longest_verb > 4 ? longest_verb : 7
+
+      # longest_handler is either the length of the longest handler or 7, since DELETE is the longest HTTP method
+      longest_handler = routes.map { |route| route[:raw_handler].is_a?(Proc) ? 7 : route[:raw_handler].split('#').last.length }.max + 3
+      longest_handler = longest_handler > 7 ? longest_handler : 7
+
+      # longest_controller is either the length of the longest controller or 12, since Controller#{length} is the longest controller
+      longest_controller = routes.map { |route| route[:raw_handler].is_a?(Proc) ? 7 : route[:raw_handler].to_s.length }.max + 3
+      longest_controller = longest_controller > 12 ? longest_controller : 12
+
+      routes.each do |route|
+        table << [
+          format("%- #{longest_handler}s", route[:raw_handler].is_a?(Proc) ? 'Lambda' : route[:raw_handler].split('#').last),
+          format("%- #{longest_verb}s", route[:method]),
+          format("%- #{longest_path}s", route[:path]),
+          format("%- #{longest_controller}s", route[:raw_handler].is_a?(Proc) ? 'Lambda' : route[:raw_handler])
+        ]
+      end
+
+      table.unshift([format("%- #{longest_handler}s", 'Action'), format("%- #{longest_verb}s", 'Verb'), format("%- #{longest_path}s", 'Path'),
+                     format("%- #{longest_path}s", "Controller#Action\n")])
+      # print the table
+      table.each do |row|
+        # this should be changed to use the main logger when added
+        puts row.join
+      end
+    end
+
+    desc "c", "Start the app console."
+    def console
+      environment
+      ARGV.clear
+      IRB.start
+    end
+
+    private
+
+    def environment
+      require File.expand_path("config/application.rb", Dir.pwd)
+    end
   end
 
   class NewAppGenerator < Thor::Group
     include Thor::Actions
-
     argument :path, type: :string
 
     def self.source_root

data/lib/rage/controller/api.rb

@@ -12,13 +12,21 @@ class RageController::API
 
       before_actions_chunk = if @__before_actions
         filtered_before_actions = @__before_actions.select do |h|
-          (h[:only].nil? || h[:only].include?(action)) &&
-            (h[:except].nil? || !h[:except].include?(action))
+          (!h[:only] || h[:only].include?(action)) &&
+            (!h[:except] || !h[:except].include?(action))
         end
 
         lines = filtered_before_actions.map do |h|
+          condition = if h[:if] && h[:unless]
+            "if #{h[:if]} && !#{h[:unless]}"
+          elsif h[:if]
+            "if #{h[:if]}"
+          elsif h[:unless]
+            "unless #{h[:unless]}"
+          end
+
           <<-RUBY
-            #{h[:name]}
+            #{h[:name]} #{condition}
             return [@__status, @__headers, @__body] if @__rendered
           RUBY
         end
@@ -65,6 +73,16 @@ class RageController::API
       klass.__rescue_handlers = @__rescue_handlers.freeze
     end
 
+    # @private
+    @@__tmp_name_seed = ("a".."i").to_a.permutation
+
+    # @private
+    # define temporary method based on a block
+    def define_tmp_method(block)
+      name = @@__tmp_name_seed.next.join
+      define_method("__rage_tmp_#{name}", block)
+    end
+
     ############
     #
     # PUBLIC API
@@ -89,8 +107,7 @@ class RageController::API
     def rescue_from(*klasses, with: nil, &block)
       unless with
         if block_given?
-          name = ("a".."z").to_a.sample(15).join
-          with = define_method("__#{name}", &block)
+          with = define_tmp_method(block)
         else
           raise "No handler provided. Pass the `with` keyword argument or provide a block."
         end
@@ -107,21 +124,53 @@ class RageController::API
 
     # Register a new `before_action` hook. Calls with the same `action_name` will overwrite the previous ones.
     #
-    # @param action_name [String] the name of the callback to add
-    # @param only [Symbol, Array<Symbol>] restrict the callback to run only for specific actions
-    # @param except [Symbol, Array<Symbol>] restrict the callback to run for all actions except specified
+    # @param action_name [String, nil] the name of the callback to add
+    # @param [Hash] opts action options
+    # @option opts [Symbol, Array<Symbol>] :only restrict the callback to run only for specific actions
+    # @option opts [Symbol, Array<Symbol>] :except restrict the callback to run for all actions except specified
+    # @option opts [Symbol, Proc] :if only run the callback if the condition is true
+    # @option opts [Symbol, Proc] :unless only run the callback if the condition is false
     # @example
     #   before_action :find_photo, only: :show
     #
     #   def find_photo
     #     Photo.first
     #   end
-    def before_action(action_name, only: nil, except: nil)
+    # @example
+    #   before_action :require_user, unless: :logged_in?
+    # @example
+    #   before_action :set_locale, if: -> { params[:locale] != "en-US" }
+    # @example
+    #   before_action do
+    #     unless logged_in? # would be `controller.send(:logged_in?)` in Rails
+    #       head :unauthorized
+    #     end
+    #   end
+    # @note The block form doesn't receive an argument and is executed on the controller level as if it was a regular method.
+    def before_action(action_name = nil, **opts, &block)
+      if block_given?
+        action_name = define_tmp_method(block)
+      elsif action_name.nil?
+        raise "No handler provided. Pass the `action_name` parameter or provide a block."
+      end
+
+       _only, _except, _if, _unless = opts.values_at(:only, :except, :if, :unless)
+
       if @__before_actions && @__before_actions.frozen?
         @__before_actions = @__before_actions.dup
       end
 
-      action = { name: action_name, only: only && Array(only), except: except && Array(except) }
+      action = {
+        name: action_name,
+        only: _only && Array(_only),
+        except: _except && Array(_except),
+        if: _if,
+        unless: _unless
+      }
+      
+      action[:if] = define_tmp_method(action[:if]) if action[:if].is_a?(Proc)
+      action[:unless] = define_tmp_method(action[:unless]) if action[:unless].is_a?(Proc)
+
       if @__before_actions.nil?
         @__before_actions = [action]
       elsif i = @__before_actions.find_index { |a| a[:name] == action_name }
@@ -192,7 +241,7 @@ class RageController::API
       @__body << if json
         json.is_a?(String) ? json : json.to_json
       else
-        __set_header("content-type", "text/plain; charset=utf-8")
+        headers["content-type"] = "text/plain; charset=utf-8"
         plain.to_s
       end
 
@@ -225,11 +274,13 @@ class RageController::API
     end
   end
 
-  private
-
-  # copy-on-write implementation for the headers object
-  def __set_header(key, value)
-    @__headers = @__headers.dup if DEFAULT_HEADERS.equal?(@__headers)
-    @__headers[key] = value
+  # Set response headers.
+  #
+  # @example
+  #   headers["Content-Type"] = "application/pdf"
+  def headers
+    # copy-on-write implementation for the headers object
+    @__headers = {}.merge!(@__headers) if DEFAULT_HEADERS.equal?(@__headers)
+    @__headers
   end
 end

data/lib/rage/router/backend.rb

@@ -3,6 +3,8 @@
 require "uri"
 
 class Rage::Router::Backend
+  attr_reader :routes
+
   OPTIONAL_PARAM_REGEXP = /\/?\(\/?(:\w+)\/?\)/
   STRING_HANDLER_REGEXP = /^([a-z0-9_\/]+)#([a-z_]+)$/
 
@@ -13,6 +15,7 @@ class Rage::Router::Backend
   end
 
   def on(method, path, handler, constraints: {}, defaults: nil)
+    raw_handler = handler
     raise "Path could not be empty" if path&.empty?
 
     if match_index = (path =~ OPTIONAL_PARAM_REGEXP)
@@ -42,7 +45,7 @@ class Rage::Router::Backend
       handler = ->(env, _params) { orig_handler.call(env) }
     end
 
-    __on(method, path, handler, constraints, defaults)
+    __on(method, path, handler, raw_handler, constraints, defaults)
   end
 
   def lookup(env)
@@ -52,7 +55,7 @@ class Rage::Router::Backend
 
   private
 
-  def __on(method, path, handler, constraints, defaults)
+  def __on(method, path, handler, raw_handler, constraints, defaults)
     @constrainer.validate_constraints(constraints)
     # Let the constrainer know if any constraints are being used now
     @constrainer.note_usage(constraints)
@@ -159,7 +162,7 @@ class Rage::Router::Backend
       end
     end
 
-    route = { method: method, path: path, pattern: pattern, params: params, constraints: constraints, handler: handler, defaults: defaults }
+    route = { method: method, path: path, pattern: pattern, params: params, constraints: constraints, handler: handler, raw_handler: raw_handler, defaults: defaults }
     @routes << route
     current_node.add_route(route, @constrainer)
   end

data/lib/rage/setup.rb

@@ -1,7 +1,23 @@
 Iodine.patch_rack
 
-project_root = Pathname.new(".").expand_path
+require_relative "#{Rage.root}/config/environments/#{Rage.env}"
 
-require_relative "#{project_root}/config/environments/#{Rage.env}"
-Dir["#{project_root}/app/**/*.rb"].each { |path| require_relative path }
-require_relative "#{project_root}/config/routes"
+
+# load application files
+app, bad = Dir["#{Rage.root}/app/**/*.rb"], []
+
+loop do
+  path = app.shift
+  break if path.nil?
+
+  require_relative path
+
+# push the file to the end of the list in case it depends on another file that has not yet been required;
+# re-raise if only errored out files are left
+rescue NameError
+  raise if (app - bad).empty?
+  app << path
+  bad << path
+end
+
+require_relative "#{Rage.root}/config/routes"

data/lib/rage/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rage
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/lib/rage-rb.rb

@@ -3,6 +3,7 @@
 require "rack"
 require "json"
 require "iodine"
+require "pathname"
 
 module Rage
   def self.application
@@ -33,6 +34,10 @@ module Rage
     [:default, Rage.env.to_sym]
   end
 
+  def self.root
+    @root ||= Pathname.new(".").expand_path
+  end
+
   module Router
     module Strategies
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rage-rb
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Roman Samoilov
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-09-27 00:00:00.000000000 Z
+date: 2023-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor