feat: add fluent setters for Tool skip_permission and overrides_built_in_tool

Add ref-qualified with_skip_permission() and with_overrides_built_in_tool()
fluent setters enabling chaining on make_tool() and ToolBuilder results.

Closes #11

Co-authored-by: Tilak Patel <tilakny@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: 3 people

include/copilot/types.hpp

@@ -1291,6 +1291,14 @@ struct Tool
     /// When true, the tool can execute without a permission prompt.
     /// (Upstream v0.1.49+)
     bool skip_permission = false;
+
+    /// Fluent setter — mark this tool as not requiring permission prompts.
+    Tool& with_skip_permission(bool value = true) & { skip_permission = value; return *this; }
+    Tool  with_skip_permission(bool value = true) && { skip_permission = value; return std::move(*this); }
+
+    /// Fluent setter — mark this tool as replacing a built-in CLI tool.
+    Tool& with_overrides_built_in_tool(bool value = true) & { overrides_built_in_tool = value; return *this; }
+    Tool  with_overrides_built_in_tool(bool value = true) && { overrides_built_in_tool = value; return std::move(*this); }
 };
 
 // =============================================================================

tests/test_tool_builder.cpp

@@ -440,3 +440,62 @@ TEST(MakeToolTest, IntAndBoolParams)
     auto result = tool.handler(inv);
     EXPECT_EQ(result.text_result_for_llm, "port=8080,enabled=true");
 }
+
+// =============================================================================
+// Fluent Setter Tests (PR #11 contribution by @tilakpatell)
+// =============================================================================
+
+TEST(ToolFluentSetterTest, SkipPermissionOnLvalue)
+{
+    Tool tool;
+    tool.name = "safe_read";
+    tool.description = "Read-only lookup";
+    EXPECT_FALSE(tool.skip_permission);
+
+    tool.with_skip_permission();
+    EXPECT_TRUE(tool.skip_permission);
+
+    tool.with_skip_permission(false);
+    EXPECT_FALSE(tool.skip_permission);
+}
+
+TEST(ToolFluentSetterTest, OverridesBuiltInOnLvalue)
+{
+    Tool tool;
+    tool.name = "edit_file";
+    tool.description = "Custom editor";
+    EXPECT_FALSE(tool.overrides_built_in_tool);
+
+    tool.with_overrides_built_in_tool();
+    EXPECT_TRUE(tool.overrides_built_in_tool);
+
+    tool.with_overrides_built_in_tool(false);
+    EXPECT_FALSE(tool.overrides_built_in_tool);
+}
+
+TEST(ToolFluentSetterTest, ChainingOnMakeToolResult)
+{
+    auto tool = copilot::make_tool(
+        "safe_lookup", "Read-only lookup",
+        [](std::string query) { return "result: " + query; },
+        {"query"})
+        .with_skip_permission()
+        .with_overrides_built_in_tool();
+
+    EXPECT_EQ(tool.name, "safe_lookup");
+    EXPECT_TRUE(tool.skip_permission);
+    EXPECT_TRUE(tool.overrides_built_in_tool);
+}
+
+TEST(ToolFluentSetterTest, ChainingOnToolBuilder)
+{
+    auto tool = ToolBuilder("my_tool", "A tool")
+        .param<std::string>("input", "Input text")
+        .handler([](const std::string& s) { return s; })
+        .with_skip_permission()
+        .with_overrides_built_in_tool();
+
+    EXPECT_TRUE(tool.skip_permission);
+    EXPECT_TRUE(tool.overrides_built_in_tool);
+    EXPECT_EQ(tool.name, "my_tool");
+}