Nicer error API

Both the combination of positional argument and naming does not feel
like Ruby.

This change makes `is_error` argument a keyword argument called `error`.

Thanks to this, the usage is more self-explanatory:

```ruby
response = Response.new([], error: true)
```

Author: pusewicz

lib/mcp/tool/response.rb

@@ -3,16 +3,18 @@
 module MCP
   class Tool
     class Response
-      attr_reader :content, :is_error
+      attr_reader :content, :error
 
-      def initialize(content, is_error = false)
+      def initialize(content, error: false)
         @content = content
-        @is_error = is_error
+        @error = error
       end
 
       def to_h
-        { content:, isError: is_error }.compact
+        { content:, isError: error? }.compact
       end
+
+      alias_method :error?, :error
     end
   end
 end

test/mcp/tool/response_test.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module MCP
+  class Tool
+    class ResponseTest < ActiveSupport::TestCase
+      test "#initialize with content and error flag" do
+        content = [{
+          type: "text",
+          text: "Unauthorized",
+        }]
+        response = Response.new(content, error: true)
+
+        assert_equal content, response.content
+        assert response.error?
+
+        response = Response.new(content, error: false)
+        assert_equal content, response.content
+        refute response.error?
+
+        response = Response.new(content)
+        assert_equal content, response.content
+        refute response.error?
+      end
+
+      test "#to_h" do
+        content = [{
+          type: "text",
+          text: "Unauthorized",
+        }]
+        response = Response.new(content)
+        actual = response.to_h
+
+        assert_equal [:content, :isError].sort, actual.keys.sort
+        assert_equal content, actual[:content]
+        refute actual[:isError]
+
+        response = Response.new(content, error: true)
+        actual = response.to_h
+        assert_equal [:content, :isError].sort, actual.keys.sort
+        assert_equal content, actual[:content]
+        assert actual[:isError]
+      end
+
+      test "#error?" do
+        response = Response.new(nil, error: true)
+        assert response.error?
+
+        response = Response.new(nil, error: false)
+        refute response.error?
+      end
+    end
+  end
+end

test/mcp/tool_test.rb

@@ -45,7 +45,7 @@ def call(message:, server_context: nil)
       tool = TestTool
       response = tool.call(message: "test")
       assert_equal response.content, [{ type: "text", content: "OK" }]
-      assert_equal response.is_error, false
+      refute response.error?
     end
 
     test "allows declarative definition of tools as classes" do
@@ -250,7 +250,7 @@ def call(message:, server_context: nil)
       tool = TypedTestTool
       response = tool.call(message: "test")
       assert_equal response.content, [{ type: "text", content: "OK" }]
-      assert_equal response.is_error, false
+      refute response.error?
     end
 
     class TestToolWithoutServerContext < Tool