simplecov-mcp 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5f733ad72b95b76dc7cea02f364c109de7a464d1cb49357b5b73efa86e8fffb
-  data.tar.gz: fd9267d251147bb54b224473e854450722499ac572d609fd1c5eef5ebbcf5e22
+  metadata.gz: 95c9d9edb894cafc89610f2d171e7452ec092a484666cfb15eac0314a1189963
+  data.tar.gz: bdb9d23aa1a53adfde9abe953fbeb21e1c998435a9fa63e7150f8ab7d26c8184
 SHA512:
-  metadata.gz: d8fd3956c7fd728b9e69e35c4dd4d63e5615e7391f0cac0cd602c5eb7ddd541e3c7e5c4c58d0c144b14473da16a685288c9474b4161e42c857cbab214df31f8f
-  data.tar.gz: d96a621426b18b1ae040984ec42909248556fe7a09ec7680936925057424f1491588ab20af2193a0f26e78e9f3e3d4eed85ff1d677ef624a3090ce937ac99a44
+  metadata.gz: 5f2c42c2f48e5fecf6c7e7392dd384b7255a2b3b48ccb5fcc89fb423918ff9f5028b57f6ce68b4c03f288c23665a23d08d7cf48f9e36eaebab753cd4f2aa7ec6
+  data.tar.gz: 4a96af4e01a259d194a902e9ea45c98058ffc06c5ed4313cf6785e7fc40929b3a186b6ccd9bbb078b9907f84173223096eafb278bbb696c3df414495b1e672d4

data/lib/simplecov_mcp/errors.rb

@@ -63,7 +63,26 @@ module SimpleCovMcp
   class FileNotFoundError < FileError; end
   class FilePermissionError < FileError; end
   class NotAFileError < FileError; end
-  class ResultsetNotFoundError < FileError; end
+
+  class ResultsetNotFoundError < FileError
+    def user_friendly_message
+      base = "File error: #{message}"
+
+      # Only add helpful tips in CLI and library modes, not MCP mode
+      unless SimpleCovMcp.context.mcp_mode?
+        base += <<~HELP
+
+
+          Try one of the following:
+            - cd to a directory containing coverage/.resultset.json
+            - Specify a resultset: simplecov-mcp -r PATH
+            - Use -h for help: simplecov-mcp -h
+        HELP
+      end
+
+      base
+    end
+  end
 
   # Coverage data related errors
   class CoverageDataError < Error

data/lib/simplecov_mcp/model.rb

@@ -49,7 +49,7 @@ module SimpleCovMcp
           timestamp: @cov_timestamp
         )
       rescue Errno::ENOENT => e
-        raise FileError.new("Coverage data not found at #{resultset || @root}")
+        raise ResultsetNotFoundError.new("Coverage data not found at #{resultset || @root}")
       rescue JSON::ParserError => e
         raise CoverageDataError.new("Invalid coverage data format: #{e.message}")
       rescue Errno::EACCES => e
@@ -62,7 +62,12 @@ module SimpleCovMcp
         raise CoverageDataError.new("Invalid path in coverage data: #{e.message}")
       rescue RuntimeError => e
         # RuntimeError from find_resultset or other operations
-        raise CoverageDataError.new("Failed to load coverage data: #{e.message}")
+        # Check if it's a resultset not found error
+        if e.message.downcase.include?('resultset')
+          raise ResultsetNotFoundError.new(e.message)
+        else
+          raise CoverageDataError.new("Failed to load coverage data: #{e.message}")
+        end
       end
     end
 

data/lib/simplecov_mcp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SimpleCovMcp
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

data/spec/integration_spec.rb

@@ -3,6 +3,9 @@
 require 'spec_helper'
 
 RSpec.describe 'SimpleCov MCP Integration Tests' do
+  # Timeout for MCP server operations (increased for JRuby compatibility)
+  MCP_TIMEOUT = 5
+
   let(:project_root) { (FIXTURES_DIR / 'project1').to_s }
   let(:coverage_dir) { File.join(project_root, 'coverage') }
   let(:resultset_path) { File.join(coverage_dir, '.resultset.json') }
@@ -257,7 +260,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
     it 'handles invalid resultset paths gracefully' do
       expect do
         SimpleCovMcp::CoverageModel.new(root: project_root, resultset: '/nonexistent/path')
-      end.to raise_error(SimpleCovMcp::CoverageDataError, /Failed to load coverage data/)
+      end.to raise_error(SimpleCovMcp::ResultsetNotFoundError, /Specified resultset not found/)
     end
 
     it 'provides helpful CLI error messages' do
@@ -332,7 +335,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
     end
 
     # Run the MCP executable with a single JSON-RPC request hash and return the captured streams.
-    def run_mcp_json(request_hash, env: default_env, timeout: 5)
+    def run_mcp_json(request_hash, env: default_env, timeout: MCP_TIMEOUT)
       Spec::Support::McpRunner.call_json(
         request_hash,
         **runner_args(env: env, timeout: timeout)
@@ -340,7 +343,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
     end
 
     # Run the MCP executable with a sequence of JSON-RPC requests (one per line).
-    def run_mcp_json_stream(request_hashes, env: default_env, timeout: 5)
+    def run_mcp_json_stream(request_hashes, env: default_env, timeout: MCP_TIMEOUT)
       Spec::Support::McpRunner.call_json_stream(
         request_hashes,
         **runner_args(env: env, timeout: timeout)
@@ -348,7 +351,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
     end
 
     # Run the MCP executable with a raw string payload (already encoded as needed).
-    def run_mcp_input(input, env: default_env, timeout: 5)
+    def run_mcp_input(input, env: default_env, timeout: MCP_TIMEOUT)
       Spec::Support::McpRunner.call(
         input: input,
         **runner_args(env: env, timeout: timeout)
@@ -594,7 +597,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
       malformed_request = "{'jsonrpc': '2.0', 'id': 999, 'method': 'invalid'}"
 
       env = { 'RUBY_LIB' => lib_path }
-      result = run_mcp_input(malformed_request, env: env, timeout: 3)
+      result = run_mcp_input(malformed_request, env: env)
 
       # Should handle gracefully without crashing
       # May return error response or empty output
@@ -615,8 +618,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
 
       result = run_mcp_json(
         request,
-        env: default_env.merge('SIMPLECOV_MCP_OPTS' => '--log-file stderr'),
-        timeout: 3
+        env: default_env.merge('SIMPLECOV_MCP_OPTS' => '--log-file stderr')
       )
 
       response = parse_jsonrpc_response(result[:stdout])
@@ -631,7 +633,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
         'SIMPLECOV_MCP_OPTS' => '--log-file stdout'
       }
 
-      result = run_mcp_input(nil, env: env, timeout: 3)
+      result = run_mcp_input(nil, env: env)
 
       combined_output = result[:stdout] + result[:stderr]
       expect(combined_output).to include('stdout').and include('not permitted')
@@ -645,7 +647,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
           params: { name: 'version_tool', arguments: {} } }
       ]
 
-      result = run_mcp_json_stream(requests, timeout: 5)
+      result = run_mcp_json_stream(requests)
 
       responses = result[:stdout].lines.map do |line|
         next if line.strip.empty?
@@ -811,7 +813,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
       it 'handles completely invalid JSON input' do
         invalid_json = 'this is not JSON at all'
 
-        result = run_mcp_input(invalid_json, env: default_env, timeout: 3)
+        result = run_mcp_input(invalid_json, env: default_env)
 
         # Should not crash with unhandled exception
         combined = result[:stdout] + result[:stderr]
@@ -825,7 +827,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
       end
 
       it 'handles empty input gracefully' do
-        result = run_mcp_input('', env: default_env, timeout: 2)
+        result = run_mcp_input('', env: default_env)
 
         # Empty input should be handled without crash
         expect(result[:stderr]).not_to include('NameError')
@@ -835,7 +837,7 @@ RSpec.describe 'SimpleCov MCP Integration Tests' do
       it 'handles partial JSON input' do
         partial_json = '{"jsonrpc": "2.0", "id": 300, "method":'
 
-        result = run_mcp_input(partial_json, env: default_env, timeout: 2)
+        result = run_mcp_input(partial_json, env: default_env)
 
         # Should handle gracefully without crashing
         expect(result[:stderr]).not_to include('uninitialized constant')

data/spec/model_error_handling_spec.rb

@@ -187,23 +187,21 @@ RSpec.describe SimpleCovMcp::CoverageModel, 'error handling' do
 
       expect do
         SimpleCovMcp::CoverageModel.new(root: root, resultset: '/nonexistent/path')
-      end.to raise_error(SimpleCovMcp::CoverageDataError) do |error|
-        expect(error.message).to include('Failed to load coverage data')
+      end.to raise_error(SimpleCovMcp::ResultsetNotFoundError) do |error|
         expect(error.message).to include('Specified resultset not found')
       end
     end
 
     it 'handles RuntimeError with generic messages' do
-      # Test RuntimeError with any generic message
+      # Test RuntimeError with any generic message that includes 'resultset'
       allow(SimpleCovMcp::CovUtil).to receive(:find_resultset).and_raise(
         RuntimeError.new('Something went wrong during resultset lookup')
       )
 
       expect do
         SimpleCovMcp::CoverageModel.new(root: root, resultset: 'coverage')
-      end.to raise_error(SimpleCovMcp::CoverageDataError) do |error|
-        expect(error.message).to include('Failed to load coverage data')
-        expect(error.message).to include('Something went wrong')
+      end.to raise_error(SimpleCovMcp::ResultsetNotFoundError) do |error|
+        expect(error.message).to include('Something went wrong during resultset lookup')
       end
     end
   end

data/spec/simplecov_mcp_model_spec.rb

@@ -19,10 +19,10 @@ RSpec.describe SimpleCovMcp::CoverageModel do
       end.to raise_error(SimpleCovMcp::FileError, /Coverage data not found/)
     end
 
-    it 'raises CoverageDataError when resultset file does not exist' do
+    it 'raises ResultsetNotFoundError when resultset file does not exist' do
       expect do
         described_class.new(root: root, resultset: '/nonexistent/path/.resultset.json')
-      end.to raise_error(SimpleCovMcp::CoverageDataError, /Failed to load coverage data/)
+      end.to raise_error(SimpleCovMcp::ResultsetNotFoundError, /Specified resultset not found/)
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simplecov-mcp
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Keith R. Bennett