jetski 0.2.8 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23c5e792f85839ddaf6326ac6307142d4bf73fca59aaa203cfe1b30186fc58b1
-  data.tar.gz: e82a214526ce082b8e2df362ebcc8b5418082cb079e5da6feea441d5556fc0dd
+  metadata.gz: 9695762a6c63b04e17d8477feff33bd1cf6e44a819a7112c17abed5c9411826b
+  data.tar.gz: 046a721b230e7fe80c5fe19a95a1a81d64d3e1956caff0163127a4752064a58a
 SHA512:
-  metadata.gz: bcfe67153f00c1b9dd4135c4e41d5f840ebc5c32ac9d74e2db6cda02fa0f3b01580df15f339df7cc19ebeaae4b7c8ad7b9effbc784e971e4e038cf0bec2e8f10
-  data.tar.gz: b68d5592f411737d82b54d29dda2f61b5b7c55c4bc7d7978ad3b7b1cdc557029cfc438c7a71edd51ae21af257d94821743e507c16a31a78c132b7645aa1d8889
+  metadata.gz: 3186f1f38ab5fd20ddbd24f6c3e9958f34e44161269333c83b1ae42208e64d4016ed03354458c4a7540c4e38784120a9ff8e971279465b2d66156d9909c0a7c3
+  data.tar.gz: 1b6674aaffba2a6054bd0d07e9923d5c0bf43df333572e4b81f9474c2d671d01c3ad63b987d0625a794012c227003b831c97ce3b956e7e3b655985f35677400c

data/bin/jetski

@@ -23,6 +23,15 @@ class JetskiCLI < Thor
     say "Jetski v#{Jetski::VERSION}"
   end
 
+  desc "routes", "shows the routes in your app."
+  def routes
+    compiled_routes = Jetski::Router::Parser.compile_routes
+    say "🌊 Jetski Routes 🏄‍♂️"
+    compiled_routes.each do |route_obj|
+      say "#{route_obj[:method]} #{route_obj[:url]} #{route_obj[:controller_name]}##{route_obj[:action_name]}"
+    end
+  end
+  
   def self.source_root
     File.join(File.dirname(__FILE__), '..', 'templates')
   end

data/lib/jetski/base_controller.rb

@@ -3,13 +3,33 @@ module Jetski
   class BaseController
     attr_accessor :action_name, :controller_name
     attr_reader :res
+    attr_reader :performed_render
+
+    class << self
+      def request_method(method)
+        # Really just a shell method since Im using File.readlines to use the logic in routes.
+      end
+
+      def root
+        # another shell method
+      end
+
+      def path(location)
+        # Another shell method
+      end
+    end
+
     def initialize(res)
       @res = res
+      @performed_render = false
     end
 
     # Method to render matching view with controller_name/action_name
     
     def render(**args)
+      @performed_render = true
+      request_status = args[:status] || 200
+      res.status = request_status
       if args[:text]
         res.content_type = "text/plain"
         res.body = "#{args[:text]}\n"
@@ -21,6 +41,7 @@ module Jetski
         res.body = args[:json].to_json
         return
       end
+      
       render_template_file
     end
 

data/lib/jetski/router/parser.rb

@@ -0,0 +1,93 @@
+module Jetski
+  class Router
+    module Parser
+      extend self
+      def compile_routes
+        auto_found_routes = []
+        controller_file_paths = Dir.glob([File.join(Jetski.app_root, 'app', 'controllers', '**', '*_controller.rb')])
+        controller_file_paths.each do |file_path| 
+          controller_file_name = file_path.split('app/controllers')[1]
+          controller_as_url = controller_file_name.gsub(/_controller.rb/, '')
+          controller_name = controller_as_url.split("/").last
+          controller_classname = controller_as_url.split("/").reject(&:empty?).map(&:capitalize).join("::") + "Controller"
+          controller_file_readlines = File.readlines(file_path)
+          controller_file_readlines.each.with_index do |line, idx| 
+            strp_line = line.strip 
+            if strp_line.start_with?('def')
+              action_name = strp_line.split("def").last.strip
+              base_opts = { 
+                controller_classname: controller_classname, 
+                controller_file_name: controller_file_name,
+                controller_name: controller_name 
+              }
+              case action_name
+              when "new"
+                auto_found_routes << base_opts.merge({
+                  url: controller_as_url + "/new",
+                  method: "GET",
+                  action_name: action_name,
+                })
+              when "create"
+                auto_found_routes << base_opts.merge({
+                  url: controller_as_url,
+                  method: "POST",
+                  action_name: action_name,
+                })
+              when "show"
+                auto_found_routes << base_opts.merge({
+                  url: controller_as_url + "/:id",
+                  method: "GET",
+                  action_name: action_name,
+                })
+              when "edit"
+                auto_found_routes << base_opts.merge({
+                  url: controller_as_url + "/:id/edit",
+                  method: "GET",
+                  action_name: action_name,
+                })
+              when "update"
+                auto_found_routes << base_opts.merge({
+                  url: controller_as_url + "/:id",
+                  method: "PUT",
+                  action_name: action_name,
+                })
+              when "destroy"
+                auto_found_routes << base_opts.merge({
+                  url: controller_as_url + "/:id",
+                  method: "DELETE",
+                  action_name: action_name,
+                })
+              else
+                method_route_options = controller_file_readlines[(idx - 2)..(idx - 1)].map(&:strip)
+                custom_request_method = method_route_options.find { |line| line.start_with? "request_method" }
+                custom_request_method = if custom_request_method
+                  custom_request_method.split(" ")[1].gsub('"', '').upcase
+                else
+                  "GET"
+                end
+                custom_path_option = method_route_options.find { |line| line.start_with? "path" }
+                check_root = controller_file_readlines[idx - 1].strip
+                url_to_use = if check_root.include?("root")
+                  "/"
+                else
+                  if custom_path_option
+                    custom_path_option.split(" ")[1].gsub('"', '')
+                  else
+                    url_friendly_action_name = action_name.split("_").join("-")
+                    controller_as_url + "/#{url_friendly_action_name}"
+                  end
+                end
+                auto_found_routes << base_opts.merge({
+                  url: url_to_use,
+                  method: custom_request_method,
+                  action_name: action_name,
+                })
+              end
+            end
+          end
+        end
+        auto_found_routes
+      end
+    end
+  end
+end

data/lib/jetski/router.rb

@@ -1,47 +1,48 @@
 module Jetski
   class Router
+    include Parser
     attr_reader :server
     def initialize(server)
       @server = server
     end
 
     def call
-      parse_routes && host_assets
+      host_routes && host_assets
     end
 
-    def parse_routes
-      # Convert routes file into render of correct controller and action
-      routes_file = File.join(Jetski.app_root, "config/routes")
+    def host_routes
+      routes = compile_routes
+      routes.each do |af_route|
+        served_url = af_route[:url]
+        request_method = af_route[:method]
+        controller_classname = af_route[:controller_classname]
+        controller_name = af_route[:controller_name]
+        action_name = af_route[:action_name]
+        controller_file_name = af_route[:controller_file_name]
 
-      File.readlines(routes_file, chomp: true).each do |line|
-        route_action, served_url, controller_name, action_name = line.split(" ")
         server.mount_proc served_url do |req, res|
           errors = []
-          if (route_action.upcase != req.request_method)
+          if (request_method!= req.request_method)
             errors << "Wrong request was performed"
           end
-          # TODO: Fix the fact that we are always setting res.body to something here. 
-          # Theres no way to return. We need to organize into case statement or if/else type
-          
           if errors.empty?
-            controller_name_formatted = controller_name.split("/").map(&:capitalize).join("::")
-            constantized_controller = "#{controller_name_formatted}Controller"
-            path_to_defined_controller = File.join(Jetski.app_root, "app/controllers/#{controller_name}_controller.rb")
+            path_to_defined_controller = File.join(Jetski.app_root, "app/controllers/#{controller_file_name}")
             require_relative path_to_defined_controller
             begin
-              controller_class = Object.const_get(constantized_controller)
+              controller_class = Object.const_get(controller_classname)
             rescue NameError
-              errors << "#{constantized_controller} is not defined. Please create a file app/controllers/#{controller_name}.rb"
+              errors << "#{controller_classname} is not defined. Please create a file app/controllers/#{controller_file_name}"
             end
           end
 
-          if errors.empty? # Continue unless error found
+          if errors.empty?
             controller = controller_class.new(res)
             controller.action_name = action_name
             controller.controller_name = controller_name
             controller.send(action_name)
-            # Render matching HTML template for GET requests only
-            controller.render if route_action.upcase == "GET"
+            if !controller.performed_render && (request_method.upcase == "GET")
+              controller.render
+            end
             # TODO: Need to setup redirects for other request types. POST/PUT/DELETE
           end
 

data/lib/jetski/version.rb

@@ -1,3 +1,3 @@
 module Jetski
-  VERSION = "0.2.8"
+  VERSION = "0.3.1"
 end

data/lib/jetski.rb

@@ -1,9 +1,11 @@
 require_relative './jetski/version'
+require_relative './jetski/base_controller'
 require_relative './jetski/server'
+require_relative './jetski/router/parser'
 require_relative './jetski/router'
-require_relative './jetski/base_controller'
 require "webrick"
 require "json"
+require "json"
 
 module Jetski
   # Debug stage add constants here for debugging.

data/templates/base/app/controllers/pages_controller.rb

@@ -1,4 +1,5 @@
 class PagesController < Jetski::BaseController
+  root
   def home
   end
 end

data/templates/base/app/views/layouts/application.html

@@ -5,6 +5,7 @@
   <meta name="viewport" content="width=device-width, initial-scale=1.0">
   <title> Jetski app </title>
   <link rel="stylesheet" href="/application.css">
+  <link rel="icon" type="image/x-icon" href="/jetski-logo.png">
   DYNAMIC_CSS
 </head>
 <body>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jetski
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.3.1
 platform: ruby
 authors:
 - Indigo Tech Tutorials
@@ -49,14 +49,15 @@ files:
 - lib/jetski.rb
 - lib/jetski/base_controller.rb
 - lib/jetski/router.rb
+- lib/jetski/router/parser.rb
 - lib/jetski/server.rb
 - lib/jetski/version.rb
 - templates/base/Gemfile
+- templates/base/app/assets/images/jetski-logo.png
 - templates/base/app/assets/stylesheets/application.css
 - templates/base/app/controllers/pages_controller.rb
 - templates/base/app/views/layouts/application.html
 - templates/base/app/views/pages/home.html
-- templates/base/config/routes
 - templates/base/start.rb
 homepage: https://rubygems.org/gems/jetski
 licenses:

data/templates/base/config/routes

@@ -1 +0,0 @@
-get / pages home