consolle 0.2.6 → 0.2.8

data/lib/consolle/server/request_broker.rb

@@ -1,8 +1,7 @@
 # frozen_string_literal: true
 
-require "thread"
-require "securerandom"
-require "logger"
+require 'securerandom'
+require 'logger'
 
 module Consolle
   module Server
@@ -21,53 +20,53 @@ module Consolle
 
       def start
         return if @running
-        
+
         @running = true
         @worker_thread = start_worker
-        logger.info "[RequestBroker] Started"
+        logger.info '[RequestBroker] Started'
       end
 
       def stop
         return unless @running
-        
+
         @running = false
-        
+
         # Push poison pill to wake up worker
         @queue.push(nil)
-        
+
         # Wait for worker to finish
         @worker_thread&.join(5)
-        
-        logger.info "[RequestBroker] Stopped"
+
+        logger.info '[RequestBroker] Stopped'
       end
 
       def process_request(request)
-        request_id = request["request_id"] || SecureRandom.uuid
-        
+        request_id = request['request_id'] || SecureRandom.uuid
+
         # Create future for response
         future = RequestFuture.new
-        
+
         # Store in map
         @mutex.synchronize do
           @request_map[request_id] = future
         end
-        
+
         # Queue request
         @queue.push({
-          id: request_id,
-          request: request,
-          timestamp: Time.now
-        })
-        
+                      id: request_id,
+                      request: request,
+                      timestamp: Time.now
+                    })
+
         # Wait for response (with timeout)
         begin
-          future.get(timeout: request["timeout"] || 30)
+          future.get(timeout: request['timeout'] || 30)
         rescue Timeout::Error
           {
-            "success" => false,
-            "error" => "RequestTimeout",
-            "message" => "Request timed out",
-            "request_id" => request_id
+            'success' => false,
+            'error' => 'RequestTimeout',
+            'message' => 'Request timed out',
+            'request_id' => request_id
           }
         ensure
           # Clean up
@@ -81,17 +80,17 @@ module Consolle
 
       def start_worker
         Thread.new do
-          logger.info "[RequestBroker] Worker thread started"
-          
+          logger.info '[RequestBroker] Worker thread started'
+
           while @running
             begin
               # Get next request
               item = @queue.pop
-              
+
               # Check for poison pill
               break if item.nil? && !@running
               next if item.nil?
-              
+
               # Process request
               process_item(item)
             rescue StandardError => e
@@ -99,120 +98,118 @@ module Consolle
               logger.error e.backtrace.join("\n")
             end
           end
-          
-          logger.info "[RequestBroker] Worker thread stopped"
+
+          logger.info '[RequestBroker] Worker thread stopped'
         end
       end
 
       def process_item(item)
         request_id = item[:id]
         request = item[:request]
-        
+
         logger.debug "[RequestBroker] Processing request: #{request_id}"
-        
+
         # Get future
         future = @mutex.synchronize { @request_map[request_id] }
         return unless future
-        
+
         # Process based on request type
-        response = case request["action"]
-        when "eval", "exec"
-          process_eval_request(request)
-        when "status"
-          process_status_request
-        when "restart"
-          process_restart_request
-        else
-          {
-            "success" => false,
-            "error" => "UnknownAction",
-            "message" => "Unknown action: #{request["action"]}"
-          }
-        end
-        
+        response = case request['action']
+                   when 'eval', 'exec'
+                     process_eval_request(request)
+                   when 'status'
+                     process_status_request
+                   when 'restart'
+                     process_restart_request
+                   else
+                     {
+                       'success' => false,
+                       'error' => 'UnknownAction',
+                       'message' => "Unknown action: #{request['action']}"
+                     }
+                   end
+
         # Add request_id to response
-        response["request_id"] = request_id
-        
+        response['request_id'] = request_id
+
         # Set future result
         future.set(response)
-        
+
         logger.debug "[RequestBroker] Request completed: #{request_id}"
       rescue StandardError => e
         logger.error "[RequestBroker] Error processing request #{request_id}: #{e.message}"
-        
+
         error_response = {
-          "success" => false,
-          "error" => e.class.name,
-          "message" => e.message,
-          "request_id" => request_id
+          'success' => false,
+          'error' => e.class.name,
+          'message' => e.message,
+          'request_id' => request_id
         }
-        
+
         future&.set(error_response)
       end
 
       def process_eval_request(request)
-        code = request["code"]
-        timeout = request["timeout"] || 30
-        
+        code = request['code']
+        timeout = request['timeout'] || 30
+
         unless code
           return {
-            "success" => false,
-            "error" => "MissingParameter",
-            "message" => "Missing required parameter: code"
+            'success' => false,
+            'error' => 'MissingParameter',
+            'message' => 'Missing required parameter: code'
           }
         end
-        
+
         # Execute through supervisor
         result = @supervisor.eval(code, timeout: timeout)
-        
+
         # Format response
         if result[:success]
           {
-            "success" => true,
-            "result" => result[:output],
-            "execution_time" => result[:execution_time]
+            'success' => true,
+            'result' => result[:output],
+            'execution_time' => result[:execution_time]
           }
         else
           {
-            "success" => false,
-            "error" => "ExecutionError",
-            "message" => result[:output]
+            'success' => false,
+            'error' => 'ExecutionError',
+            'message' => result[:output]
           }
         end
       end
 
       def process_status_request
         {
-          "success" => true,
-          "running" => @supervisor.running?,
-          "pid" => @supervisor.pid,
-          "rails_root" => @supervisor.rails_root,
-          "rails_env" => @supervisor.rails_env
+          'success' => true,
+          'running' => @supervisor.running?,
+          'pid' => @supervisor.pid,
+          'rails_root' => @supervisor.rails_root,
+          'rails_env' => @supervisor.rails_env
         }
       end
 
       def process_restart_request
-        begin
-          # Restart the Rails console subprocess
-          new_pid = @supervisor.restart
-          
-          {
-            "success" => true,
-            "message" => "Rails console subprocess restarted",
-            "pid" => new_pid,
-            "rails_root" => @supervisor.rails_root,
-            "rails_env" => @supervisor.rails_env
-          }
-        rescue StandardError => e
-          logger.error "[RequestBroker] Restart failed: #{e.message}"
-          logger.error e.backtrace.join("\n")
-          
-          {
-            "success" => false,
-            "error" => "RestartFailed",
-            "message" => "Failed to restart Rails console: #{e.message}"
-          }
-        end
+        # Restart the Rails console subprocess
+        new_pid = @supervisor.restart
+
+        {
+          'success' => true,
+          'message' => 'Rails console subprocess restarted',
+          'pid' => new_pid,
+          'rails_root' => @supervisor.rails_root,
+          'rails_env' => @supervisor.rails_env
+        }
+      rescue StandardError => e
+        logger.error "[RequestBroker] Restart failed: #{e.message}"
+        logger.error e.backtrace.join("\n")
+
+        {
+          'success' => false,
+          'error' => 'RestartFailed',
+          'message' => "Failed to restart Rails console: #{e.message}"
+        }
       end
 
       # Simple future implementation
@@ -236,7 +233,7 @@ module Consolle
           @mutex.synchronize do
             unless @set
               @condition.wait(@mutex, timeout)
-              raise Timeout::Error, "Future timed out" unless @set
+              raise Timeout::Error, 'Future timed out' unless @set
             end
             @value
           end
@@ -244,4 +241,4 @@ module Consolle
       end
     end
   end
-end
+end

data/lib/consolle/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Consolle
-  VERSION = File.read(File.expand_path("../../../.version", __FILE__)).strip
-end
+  VERSION = File.read(File.expand_path('../../.version', __dir__)).strip
+end

data/lib/consolle.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require_relative "consolle/version"
-require_relative "consolle/cli"
+require_relative 'consolle/version'
+require_relative 'consolle/cli'
 
 # Server components
-require_relative "consolle/server/console_socket_server"
-require_relative "consolle/server/console_supervisor"
-require_relative "consolle/server/request_broker"
+require_relative 'consolle/server/console_socket_server'
+require_relative 'consolle/server/console_supervisor'
+require_relative 'consolle/server/request_broker'
 
 module Consolle
   class Error < StandardError; end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: consolle
 version: !ruby/object:Gem::Version
-  version: 0.2.6
+  version: 0.2.8
 platform: ruby
 authors:
 - nacyot
@@ -10,7 +10,7 @@ cert_chain: []
 date: 2025-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: thor
+  name: logger
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -24,7 +24,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: logger
+  name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"