mathpix 0.1.0 → 0.1.2

data/lib/mathpix/mcp/middleware/cors_middleware.rb

@@ -19,18 +19,14 @@ module Mathpix
           origin = env['HTTP_ORIGIN']
 
           # Handle preflight OPTIONS request
-          if env['REQUEST_METHOD'] == 'OPTIONS'
-            return preflight_response(origin)
-          end
+          return preflight_response(origin) if env['REQUEST_METHOD'] == 'OPTIONS'
 
           # Call app and add CORS headers
           status, headers, body = @app.call(env)
 
           # For wildcard '*', always add CORS headers even without Origin
           # For specific origins, only add when Origin header is present and allowed
-          if @allowed_origins.include?('*') || (origin && origin_allowed?(origin))
-            add_cors_headers(headers, origin)
-          end
+          add_cors_headers(headers, origin) if @allowed_origins.include?('*') || (origin && origin_allowed?(origin))
 
           [status, headers, body]
         end

data/lib/mathpix/mcp/middleware/oauth_middleware.rb

@@ -18,9 +18,7 @@ module Mathpix
           # Extract and validate token
           token = extract_token(env)
 
-          if token.nil?
-            return unauthorized_response('missing_token')
-          end
+          return unauthorized_response('missing_token') if token.nil?
 
           begin
             payload = @oauth_provider.validate_token(token)
@@ -39,9 +37,7 @@ module Mathpix
         def extract_token(env)
           # Try Bearer token
           auth_header = env['HTTP_AUTHORIZATION']
-          if auth_header&.start_with?('Bearer ')
-            return auth_header.sub('Bearer ', '')
-          end
+          return auth_header.sub('Bearer ', '') if auth_header&.start_with?('Bearer ')
 
           # Try X-API-Key header
           env['HTTP_X_API_KEY']

data/lib/mathpix/mcp/middleware/rate_limiting_middleware.rb

@@ -27,15 +27,13 @@ module Mathpix
           @window = window
           @cleanup_thread = start_cleanup_thread unless ENV['RACK_ENV'] == 'test'
 
-          $stderr.puts "[RATE LIMIT] Middleware initialized: object_id=#{object_id}" if ENV['RACK_ENV'] == 'test'
+          warn "[RATE LIMIT] Middleware initialized: object_id=#{object_id}" if ENV['RACK_ENV'] == 'test'
         end
 
         def call(env)
           # Exempt /health endpoint from rate limiting (monitoring endpoint)
           request_path = env['PATH_INFO'] || env['REQUEST_PATH']
-          if request_path == '/health'
-            return @app.call(env)
-          end
+          return @app.call(env) if request_path == '/health'
 
           client_id = extract_client_id(env)
 
@@ -43,7 +41,9 @@ module Mathpix
           if rate_limited?(client_id)
             retry_after = time_until_reset(client_id)
             bucket = @@buckets[client_id]
-            $stderr.puts "[RATE LIMIT] LIMITING client #{client_id}, Count: #{bucket[:count]}/#{@limit}" if ENV['RACK_ENV'] == 'test'
+            if ENV['RACK_ENV'] == 'test'
+              warn "[RATE LIMIT] LIMITING client #{client_id}, Count: #{bucket[:count]}/#{@limit}"
+            end
             return rate_limit_response(retry_after)
           end
 
@@ -53,7 +53,7 @@ module Mathpix
           # Debug: show count AFTER incrementing
           if ENV['RACK_ENV'] == 'test'
             bucket = @@buckets[client_id]
-            $stderr.puts "[RATE LIMIT] Client: #{client_id}, Count: #{bucket[:count]}/#{@limit}"
+            warn "[RATE LIMIT] Client: #{client_id}, Count: #{bucket[:count]}/#{@limit}"
           end
 
           @app.call(env)
@@ -70,7 +70,8 @@ module Mathpix
 
         def rate_limited?(client_id)
           bucket = @@buckets[client_id]
-          return false unless bucket  # Not rate limited if no bucket yet
+          return false unless bucket # Not rate limited if no bucket yet
+
           bucket[:count] >= @limit
         end
 
@@ -83,7 +84,7 @@ module Mathpix
             bucket = @@buckets[client_id]
             old_count = bucket[:count]
             bucket[:count] += 1
-            $stderr.puts "[RATE LIMIT] record_request: #{old_count} -> #{bucket[:count]}" if ENV['RACK_ENV'] == 'test'
+            warn "[RATE LIMIT] record_request: #{old_count} -> #{bucket[:count]}" if ENV['RACK_ENV'] == 'test'
           end
         end
 
@@ -104,10 +105,10 @@ module Mathpix
               'X-RateLimit-Reset' => (Time.now + retry_after).to_i.to_s
             },
             [JSON.generate({
-              error: 'rate_limit_exceeded',
-              message: 'Too many requests',
-              retry_after: retry_after
-            })]
+                             error: 'rate_limit_exceeded',
+                             message: 'Too many requests',
+                             retry_after: retry_after
+                           })]
           ]
         end
 
@@ -126,12 +127,12 @@ module Mathpix
 
         def cleanup_expired_buckets
           now = Time.now
-          @buckets.each_pair do |client_id, bucket|
-            if bucket[:reset_at] <= now
-              # Reset the bucket instead of deleting
-              bucket[:count] = 0
-              bucket[:reset_at] = now + @window
-            end
+          @buckets.each_pair do |_client_id, bucket|
+            next unless bucket[:reset_at] <= now
+
+            # Reset the bucket instead of deleting
+            bucket[:count] = 0
+            bucket[:reset_at] = now + @window
           end
         end
       end

data/lib/mathpix/mcp/resources/formats_list_resource.rb

@@ -14,97 +14,97 @@ module Mathpix
         #
         # @param client [Mathpix::Client] optional client (not used, kept for consistency)
         # @return [Hash] resource content
-        def self.fetch(client = nil)
+        def self.fetch(_client = nil)
           formats = {
             image_formats: [
               {
-                name: "latex_styled",
-                description: "LaTeX with styling commands",
-                use_case: "Typesetting, display"
+                name: 'latex_styled',
+                description: 'LaTeX with styling commands',
+                use_case: 'Typesetting, display'
               },
               {
-                name: "text",
-                description: "Plain text representation",
-                use_case: "Simple text extraction"
+                name: 'text',
+                description: 'Plain text representation',
+                use_case: 'Simple text extraction'
               },
               {
-                name: "latex_list",
-                description: "Array of LaTeX expressions",
-                use_case: "Multiple equations"
+                name: 'latex_list',
+                description: 'Array of LaTeX expressions',
+                use_case: 'Multiple equations'
               },
               {
-                name: "mathml",
-                description: "MathML markup language",
-                use_case: "Web display, accessibility"
+                name: 'mathml',
+                description: 'MathML markup language',
+                use_case: 'Web display, accessibility'
               },
               {
-                name: "asciimath",
-                description: "AsciiMath notation",
-                use_case: "Simple math notation"
+                name: 'asciimath',
+                description: 'AsciiMath notation',
+                use_case: 'Simple math notation'
               },
               {
-                name: "text_display",
-                description: "Display-style text",
-                use_case: "Large equation display"
+                name: 'text_display',
+                description: 'Display-style text',
+                use_case: 'Large equation display'
               },
               {
-                name: "latex_simplified",
-                description: "Simplified LaTeX",
-                use_case: "Basic LaTeX output"
+                name: 'latex_simplified',
+                description: 'Simplified LaTeX',
+                use_case: 'Basic LaTeX output'
               },
               {
-                name: "data",
-                description: "Full response with metadata",
-                use_case: "Complete API response"
+                name: 'data',
+                description: 'Full response with metadata',
+                use_case: 'Complete API response'
               },
               {
-                name: "html",
-                description: "HTML markup",
-                use_case: "Web integration"
+                name: 'html',
+                description: 'HTML markup',
+                use_case: 'Web integration'
               }
             ],
             document_formats: [
               {
-                name: "markdown",
-                description: "Markdown document format",
-                use_case: "Note-taking, documentation"
+                name: 'markdown',
+                description: 'Markdown document format',
+                use_case: 'Note-taking, documentation'
               },
               {
-                name: "latex",
-                description: "LaTeX document format",
-                use_case: "Academic papers, typesetting"
+                name: 'latex',
+                description: 'LaTeX document format',
+                use_case: 'Academic papers, typesetting'
               },
               {
-                name: "html",
-                description: "HTML document format",
-                use_case: "Web publishing"
+                name: 'html',
+                description: 'HTML document format',
+                use_case: 'Web publishing'
               },
               {
-                name: "docx",
-                description: "Microsoft Word document",
-                use_case: "Word processing"
+                name: 'docx',
+                description: 'Microsoft Word document',
+                use_case: 'Word processing'
               },
               {
-                name: "tex.zip",
-                description: "LaTeX with figures (zipped)",
-                use_case: "Complete LaTeX projects"
+                name: 'tex.zip',
+                description: 'LaTeX with figures (zipped)',
+                use_case: 'Complete LaTeX projects'
               }
             ]
           }
 
           {
-            uri: "mathpix://formats/list",
-            mime_type: "application/json",
+            uri: 'mathpix://formats/list',
+            mime_type: 'application/json',
             content: JSON.pretty_generate({
-              success: true,
-              image_formats_count: formats[:image_formats].length,
-              document_formats_count: formats[:document_formats].length,
-              formats: formats,
-              usage: {
-                image_capture: "Use with Mathpix.snap() or ConvertImageTool",
-                document_conversion: "Use with Mathpix.document() or ConvertDocumentTool"
-              }
-            })
+                                            success: true,
+                                            image_formats_count: formats[:image_formats].length,
+                                            document_formats_count: formats[:document_formats].length,
+                                            formats: formats,
+                                            usage: {
+                                              image_capture: 'Use with Mathpix.snap() or ConvertImageTool',
+                                              document_conversion: 'Use with Mathpix.document() or ConvertDocumentTool'
+                                            }
+                                          })
           }
         end
       end

data/lib/mathpix/mcp/resources/hierarchical_router.rb

@@ -84,7 +84,7 @@ module Mathpix
           links[:first] = build_link(path, limit, 0, query_params)
 
           # Previous link (if not on first page)
-          if offset > 0
+          if offset.positive?
             prev_offset = [offset - limit, 0].max
             links[:prev] = build_link(path, limit, prev_offset, query_params)
           end
@@ -168,9 +168,7 @@ module Mathpix
           return false if value == 'false'
 
           # Array (comma-separated)
-          if value.include?(',')
-            return value.split(',').map(&:strip)
-          end
+          return value.split(',').map(&:strip) if value.include?(',')
 
           # String
           value
@@ -179,9 +177,7 @@ module Mathpix
         def validate_params(params)
           # Validate limit
           if params[:limit]
-            unless params[:limit].is_a?(Integer)
-              raise InvalidParameterError, 'limit must be a number'
-            end
+            raise InvalidParameterError, 'limit must be a number' unless params[:limit].is_a?(Integer)
 
             unless params[:limit].between?(MIN_LIMIT, MAX_LIMIT)
               raise InvalidParameterError, "limit must be between #{MIN_LIMIT} and #{MAX_LIMIT}"
@@ -189,18 +185,13 @@ module Mathpix
           end
 
           # Validate offset
-          if params[:offset]
-            unless params[:offset].is_a?(Integer)
-              raise InvalidParameterError, 'offset must be a number'
-            end
-          end
+          raise InvalidParameterError, 'offset must be a number' if params[:offset] && !params[:offset].is_a?(Integer)
 
           # Validate ID format (if present)
-          if params[:id]
-            unless params[:id] =~ /^[a-zA-Z0-9_-]+$/
-              raise InvalidParameterError, 'id must be alphanumeric'
-            end
-          end
+          return unless params[:id]
+          return if params[:id] =~ /^[a-zA-Z0-9_-]+$/
+
+          raise InvalidParameterError, 'id must be alphanumeric'
         end
 
         def build_link(path, limit, offset, query_params)
@@ -228,7 +219,7 @@ module Mathpix
           { id: params[:id], line_number: params[:line_number] }
         end
 
-        def list_snips(params)
+        def list_snips(_params)
           []
         end
       end

data/lib/mathpix/mcp/resources/latest_snip_resource.rb

@@ -19,39 +19,39 @@ module Mathpix
 
           if recent.empty?
             {
-              uri: "mathpix://snip/recent/latest",
-              mime_type: "application/json",
+              uri: 'mathpix://snip/recent/latest',
+              mime_type: 'application/json',
               content: JSON.pretty_generate({
-                success: false,
-                message: "No recent captures found"
-              })
+                                              success: false,
+                                              message: 'No recent captures found'
+                                            })
             }
           else
             result = recent.first
             {
-              uri: "mathpix://snip/recent/latest",
-              mime_type: "application/json",
+              uri: 'mathpix://snip/recent/latest',
+              mime_type: 'application/json',
               content: JSON.pretty_generate({
-                success: true,
-                id: result.request_id,
-                created_at: result.timestamp,
-                latex: result.latex,
-                text: result.text,
-                confidence: result.confidence,
-                is_printed: result.printed?,
-                is_handwritten: result.handwritten?,
-                position: result.position
-              })
+                                              success: true,
+                                              id: result.request_id,
+                                              created_at: result.timestamp,
+                                              latex: result.latex,
+                                              text: result.text,
+                                              confidence: result.confidence,
+                                              is_printed: result.printed?,
+                                              is_handwritten: result.handwritten?,
+                                              position: result.position
+                                            })
             }
           end
         rescue Mathpix::Error => e
           {
-            uri: "mathpix://snip/recent/latest",
-            mime_type: "application/json",
+            uri: 'mathpix://snip/recent/latest',
+            mime_type: 'application/json',
             content: JSON.pretty_generate({
-              success: false,
-              error: e.message
-            })
+                                            success: false,
+                                            error: e.message
+                                          })
           }
         end
       end

data/lib/mathpix/mcp/resources/recent_snips_resource.rb

@@ -40,22 +40,22 @@ module Mathpix
 
           {
             uri: "mathpix://snip/recent?limit=#{limit}",
-            mime_type: "application/json",
+            mime_type: 'application/json',
             content: JSON.pretty_generate({
-              success: true,
-              limit: limit,
-              count: results.length,
-              results: results
-            })
+                                            success: true,
+                                            limit: limit,
+                                            count: results.length,
+                                            results: results
+                                          })
           }
         rescue Mathpix::Error => e
           {
             uri: "mathpix://snip/recent?limit=#{limit}",
-            mime_type: "application/json",
+            mime_type: 'application/json',
             content: JSON.pretty_generate({
-              success: false,
-              error: e.message
-            })
+                                            success: false,
+                                            error: e.message
+                                          })
           }
         end
 
@@ -67,6 +67,7 @@ module Mathpix
         def self.truncate(text, max_length)
           return nil unless text
           return text if text.length <= max_length
+
           "#{text[0...max_length]}..."
         end
       end

data/lib/mathpix/mcp/resources/snip_stats_resource.rb

@@ -27,22 +27,22 @@ module Mathpix
           stats = compute_stats(recent)
 
           {
-            uri: "mathpix://snip/stats",
-            mime_type: "application/json",
+            uri: 'mathpix://snip/stats',
+            mime_type: 'application/json',
             content: JSON.pretty_generate({
-              success: true,
-              period: "last_100_captures",
-              stats: stats
-            })
+                                            success: true,
+                                            period: 'last_100_captures',
+                                            stats: stats
+                                          })
           }
         rescue Mathpix::Error => e
           {
-            uri: "mathpix://snip/stats",
-            mime_type: "application/json",
+            uri: 'mathpix://snip/stats',
+            mime_type: 'application/json',
             content: JSON.pretty_generate({
-              success: false,
-              error: e.message
-            })
+                                            success: false,
+                                            error: e.message
+                                          })
           }
         end
 
@@ -67,7 +67,9 @@ module Mathpix
             average_confidence: avg_confidence.round(3),
             confidence_distribution: {
               high: confidences.count { |c| c >= Mathpix::Configuration::CONFIDENCE_HIGH },
-              medium: confidences.count { |c| c >= Mathpix::Configuration::CONFIDENCE_MEDIUM && c < Mathpix::Configuration::CONFIDENCE_HIGH },
+              medium: confidences.count do |c|
+                c >= Mathpix::Configuration::CONFIDENCE_MEDIUM && c < Mathpix::Configuration::CONFIDENCE_HIGH
+              end,
               low: confidences.count { |c| c < Mathpix::Configuration::CONFIDENCE_MEDIUM }
             }
           }

data/lib/mathpix/mcp/server.rb

@@ -2,7 +2,7 @@
 
 begin
   require 'mcp'
-  require 'mcp/transports/stdio'  # Transport classes not auto-loaded
+  require 'mcp/transports/stdio' # Transport classes not auto-loaded
 rescue LoadError
   raise LoadError, <<~ERROR
     The 'mcp' gem is required for MCP server functionality.
@@ -52,7 +52,7 @@ module Mathpix
       # @param name [String] server name
       # @param version [String] server version
       # @param mathpix_client [Mathpix::Client] optional client instance
-      def initialize(name: "mathpix", version: Mathpix::VERSION, mathpix_client: nil)
+      def initialize(name: 'mathpix', version: Mathpix::VERSION, mathpix_client: nil)
         @name = name
         @version = version
         @mathpix_client = mathpix_client || Mathpix.client
@@ -144,28 +144,28 @@ module Mathpix
       def resource_specs
         [
           {
-            uri: "mathpix://snip/recent/latest",
-            name: "latest-snip",
-            description: "Most recent capture result",
-            mime_type: "application/json"
+            uri: 'mathpix://snip/recent/latest',
+            name: 'latest-snip',
+            description: 'Most recent capture result',
+            mime_type: 'application/json'
           },
           {
-            uri: "mathpix://snip/stats",
-            name: "snip-stats",
-            description: "Overall capture statistics",
-            mime_type: "application/json"
+            uri: 'mathpix://snip/stats',
+            name: 'snip-stats',
+            description: 'Overall capture statistics',
+            mime_type: 'application/json'
           },
           {
-            uri: "mathpix://snip/recent?limit=10",
-            name: "recent-snips",
-            description: "Recent capture results (default limit 10)",
-            mime_type: "application/json"
+            uri: 'mathpix://snip/recent?limit=10',
+            name: 'recent-snips',
+            description: 'Recent capture results (default limit 10)',
+            mime_type: 'application/json'
           },
           {
-            uri: "mathpix://formats/list",
-            name: "formats-list",
-            description: "Available output formats",
-            mime_type: "application/json"
+            uri: 'mathpix://formats/list',
+            name: 'formats-list',
+            description: 'Available output formats',
+            mime_type: 'application/json'
           }
         ]
       end