mathpix 0.1.1 → 0.1.2

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

data/lib/mathpix/mcp/tools/batch_convert_tool.rb

@@ -12,33 +12,33 @@ module Mathpix
       #
       # The geodesic path: official SDK + batch iteration
       class BatchConvertTool < BaseTool
-        description "Convert multiple images in batch using Mathpix OCR"
+        description 'Convert multiple images in batch using Mathpix OCR'
 
         input_schema(
           properties: {
             image_paths: {
-              type: "array",
-              items: { type: "string" },
-              description: "Array of image paths or URLs to process"
+              type: 'array',
+              items: { type: 'string' },
+              description: 'Array of image paths or URLs to process'
             },
             formats: {
-              type: "array",
-              items: { type: "string" },
-              description: "Output formats for all images: latex, text, mathml, asciimath (default: latex_styled, text)"
+              type: 'array',
+              items: { type: 'string' },
+              description: 'Output formats for all images: latex, text, mathml, asciimath (default: latex_styled, text)'
             },
             parallel: {
-              type: "boolean",
-              description: "Process images in parallel (default: false)"
+              type: 'boolean',
+              description: 'Process images in parallel (default: false)'
             },
             max_parallel: {
-              type: "number",
-              description: "Maximum number of parallel requests (default: 3)"
+              type: 'number',
+              description: 'Maximum number of parallel requests (default: 3)'
             }
           },
-          required: ["image_paths"]
+          required: ['image_paths']
         )
 
-        def self.call(image_paths:, formats: nil, parallel: false, max_parallel: 3, server_context:)
+        def self.call(image_paths:, server_context:, formats: nil, parallel: false, max_parallel: 3)
           safe_execute do
             client = mathpix_client(server_context)
 
@@ -51,13 +51,11 @@ module Mathpix
             end
 
             # Process images
-            results = if parallel
+            if parallel
               # TODO: Implement parallel processing (requires threading)
               # For now, process sequentially
-              process_batch_sequential(client, normalized_paths, output_formats)
-            else
-              process_batch_sequential(client, normalized_paths, output_formats)
             end
+            results = process_batch_sequential(client, normalized_paths, output_formats)
 
             # Format response
             response_data = {
@@ -76,28 +74,24 @@ module Mathpix
           end
         end
 
-        private
-
         def self.process_batch_sequential(client, paths, formats)
           paths.map.with_index do |path, index|
-            begin
-              result = client.snap(path, formats: formats)
-              {
-                index: index,
-                image_path: path,
-                success: true,
-                latex: result.latex,
-                text: result.text,
-                confidence: result.confidence
-              }
-            rescue Mathpix::Error => e
-              {
-                index: index,
-                image_path: path,
-                success: false,
-                error: e.message
-              }
-            end
+            result = client.snap(path, formats: formats)
+            {
+              index: index,
+              image_path: path,
+              success: true,
+              latex: result.latex,
+              text: result.text,
+              confidence: result.confidence
+            }
+          rescue Mathpix::Error => e
+            {
+              index: index,
+              image_path: path,
+              success: false,
+              error: e.message
+            }
           end
         end
       end

data/lib/mathpix/mcp/tools/check_document_status_tool.rb

@@ -12,16 +12,16 @@ module Mathpix
       #
       # The geodesic path: official SDK + status polling
       class CheckDocumentStatusTool < BaseTool
-        description "Check the status of a document conversion (PDF, DOCX, PPTX)"
+        description 'Check the status of a document conversion (PDF, DOCX, PPTX)'
 
         input_schema(
           properties: {
             conversion_id: {
-              type: "string",
-              description: "Document conversion ID returned from convert_document"
+              type: 'string',
+              description: 'Document conversion ID returned from convert_document'
             }
           },
-          required: ["conversion_id"]
+          required: ['conversion_id']
         )
 
         def self.call(conversion_id:, server_context:)
@@ -52,7 +52,7 @@ module Mathpix
             end
 
             # Add error info if failed
-            if status_data['status'] == 'error' || status_data['status'] == 'failed'
+            if %w[error failed].include?(status_data['status'])
               response_data[:error] = status_data['error']
               response_data[:error_info] = status_data['error_info']
             end

data/lib/mathpix/mcp/tools/convert_document_tool.rb

@@ -12,36 +12,37 @@ module Mathpix
       #
       # The geodesic path: official SDK + Document class delegation
       class ConvertDocumentTool < BaseTool
-        description "Convert document (PDF, DOCX, PPTX) to Markdown, LaTeX, HTML, or other formats using Mathpix OCR"
+        description 'Convert document (PDF, DOCX, PPTX) to Markdown, LaTeX, HTML, or other formats using Mathpix OCR'
 
         input_schema(
           properties: {
             document_path: {
-              type: "string",
-              description: "Path to document file or URL (PDF, DOCX, or PPTX)"
+              type: 'string',
+              description: 'Path to document file or URL (PDF, DOCX, or PPTX)'
             },
             formats: {
-              type: "array",
-              items: { type: "string" },
-              description: "Output formats: markdown, latex, html, docx (default: markdown)"
+              type: 'array',
+              items: { type: 'string' },
+              description: 'Output formats: markdown, latex, html, docx (default: markdown)'
             },
             include_tables: {
-              type: "boolean",
-              description: "Include table extraction as HTML"
+              type: 'boolean',
+              description: 'Include table extraction as HTML'
             },
             max_wait: {
-              type: "number",
-              description: "Maximum wait time in seconds for conversion (default: 600)"
+              type: 'number',
+              description: 'Maximum wait time in seconds for conversion (default: 600)'
             },
             poll_interval: {
-              type: "number",
-              description: "Polling interval in seconds (default: 3.0)"
+              type: 'number',
+              description: 'Polling interval in seconds (default: 3.0)'
             }
           },
-          required: ["document_path"]
+          required: ['document_path']
         )
 
-        def self.call(document_path:, formats: nil, include_tables: false, max_wait: 600, poll_interval: 3.0, server_context:)
+        def self.call(document_path:, server_context:, formats: nil, include_tables: false, max_wait: 600,
+                      poll_interval: 3.0)
           safe_execute do
             client = mathpix_client(server_context)