Skip to content

Add opaque metadata passthrough to SDK tool definitions #1864

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
belaltaher8 wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add opaque metadata passthrough to SDK tool definitions #1864

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0d575bc
Add opaque metadata passthrough to SDK tool definitions
belaltaher8 Jul 8, 2026
71c6fb4
Address review feedback on tool metadata passthrough
belaltaher8 Jul 10, 2026
addebe1
Merge remote-tracking branch 'origin/main' into belaltaher8/tool-meta…
belaltaher8 Jul 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions dotnet/src/Client.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2705,17 +2705,20 @@ internal record ToolDefinition(
JsonElement Parameters, /* JSON schema */
bool? OverridesBuiltInTool = null,
bool? SkipPermission = null,
CopilotToolDefer? Defer = null)
CopilotToolDefer? Defer = null,
[property: JsonPropertyName("metadata")] IDictionary<string, object>? Metadata = null)
Comment thread
belaltaher8 marked this conversation as resolved.
Outdated
{
public static ToolDefinition FromAIFunction(AIFunctionDeclaration function)
{
var overrides = function.AdditionalProperties.TryGetValue(CopilotTool.OverridesBuiltInToolKey, out var val) && val is true;
var skipPerm = function.AdditionalProperties.TryGetValue(CopilotTool.SkipPermissionKey, out var skipVal) && skipVal is true;
var defer = function.AdditionalProperties.TryGetValue(CopilotTool.DeferKey, out var deferVal) && deferVal is CopilotToolDefer d ? d : (CopilotToolDefer?)null;
var metadata = function.AdditionalProperties.TryGetValue(CopilotTool.MetadataKey, out var metaVal) && metaVal is IDictionary<string, object> m ? m : null;
return new ToolDefinition(function.Name, function.Description, function.JsonSchema,
overrides ? true : null,
skipPerm ? true : null,
defer);
defer,
metadata);
}
}

Expand Down
20 changes: 19 additions & 1 deletion dotnet/src/CopilotTool.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ public static class CopilotTool
/// <summary>The key used in <see cref="AITool.AdditionalProperties"/> to carry the tool's <see cref="CopilotToolDefer"/> deferral mode.</summary>
internal const string DeferKey = "defer";

/// <summary>The key used in <see cref="AITool.AdditionalProperties"/> to carry the tool's opaque host-defined metadata.</summary>
internal const string MetadataKey = "metadata";

/// <summary>
/// Defines a tool for use in a <see cref="CopilotSession"/>.
/// </summary>
Expand Down Expand Up @@ -87,7 +90,7 @@ static void ApplyToolInvocationBinding(AIFunctionFactoryOptions factoryOptions)

static void ApplyToolOptions(AIFunctionFactoryOptions factoryOptions, CopilotToolOptions? toolOptions)
{
if (toolOptions is not null && (toolOptions.OverridesBuiltInTool || toolOptions.SkipPermission || toolOptions.Defer is not null))
if (toolOptions is not null && (toolOptions.OverridesBuiltInTool || toolOptions.SkipPermission || toolOptions.Defer is not null || toolOptions.Metadata is not null))
{
Dictionary<string, object?> additionalProperties = new(StringComparer.Ordinal);
if (factoryOptions.AdditionalProperties is not null)
Expand All @@ -113,6 +116,11 @@ static void ApplyToolOptions(AIFunctionFactoryOptions factoryOptions, CopilotToo
additionalProperties[DeferKey] = defer;
}

if (toolOptions.Metadata is { } metadata)
{
additionalProperties[MetadataKey] = metadata;
}

factoryOptions.AdditionalProperties = additionalProperties;
}
}
Expand Down Expand Up @@ -151,6 +159,16 @@ public sealed class CopilotToolOptions
/// SDK forwards it to the CLI as the tool's <c>defer</c> mode. Defaults to "auto".
/// </remarks>
public CopilotToolDefer? Defer { get; set; }

/// <summary>
/// Gets or sets opaque, host-defined metadata associated with the tool definition.
/// </summary>
/// <remarks>
/// Keys are namespaced and not part of the stable public API. When set, the SDK forwards
/// the metadata verbatim to the CLI as the tool's <c>metadata</c> object, which the runtime
Comment thread
belaltaher8 marked this conversation as resolved.
Outdated
/// may recognize to inform host-specific behavior. Unknown keys are preserved.
/// </remarks>
public IDictionary<string, object>? Metadata { get; set; }
Comment thread
belaltaher8 marked this conversation as resolved.
Outdated
}

/// <summary>
Expand Down
28 changes: 28 additions & 0 deletions dotnet/test/Unit/CopilotToolTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,34 @@ public void DefineTool_Omits_Copilot_Metadata_When_Flags_Are_False()
Assert.False(function.AdditionalProperties.ContainsKey("defer"));
}

[Fact]
public void DefineTool_Sets_Metadata_In_Additional_Properties()
{
var metadata = new Dictionary<string, object>
{
["github.com/copilot:safeForTelemetry"] = new Dictionary<string, object>
{
["name"] = true,
["inputsNames"] = false
}
};

var function = CopilotTool.DefineTool(
ReturnsOk,
new CopilotToolOptions { Metadata = metadata });

Assert.True(function.AdditionalProperties.TryGetValue("metadata", out var value));
Assert.Same(metadata, value);
}

[Fact]
public void DefineTool_Omits_Metadata_When_Unset()
{
var function = CopilotTool.DefineTool(ReturnsOk);

Assert.False(function.AdditionalProperties.ContainsKey("metadata"));
}

[Fact]
public void DefineTool_Accepts_Lambda_Handlers_Without_Casts()
{
Expand Down
47 changes: 47 additions & 0 deletions go/client_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1160,6 +1160,53 @@ func TestToolDefer(t *testing.T) {
})
}

func TestToolMetadata(t *testing.T) {
t.Run("Metadata is serialized in tool definition", func(t *testing.T) {
tool := Tool{
Name: "my_tool",
Description: "A custom tool",
Metadata: map[string]any{
"github.com/copilot:safeForTelemetry": map[string]any{"name": true, "inputsNames": false},
},
Handler: func(_ ToolInvocation) (ToolResult, error) { return ToolResult{}, nil },
}
data, err := json.Marshal(tool)
if err != nil {
t.Fatalf("failed to marshal: %v", err)
}
var m map[string]any
if err := json.Unmarshal(data, &m); err != nil {
t.Fatalf("failed to unmarshal: %v", err)
}
meta, ok := m["metadata"].(map[string]any)
if !ok {
t.Fatalf("expected metadata object, got %v", m)
}
if _, ok := meta["github.com/copilot:safeForTelemetry"]; !ok {
t.Errorf("expected namespaced key preserved, got %v", meta)
}
})

t.Run("Metadata omitted when unset", func(t *testing.T) {
tool := Tool{
Name: "custom_tool",
Description: "A custom tool",
Handler: func(_ ToolInvocation) (ToolResult, error) { return ToolResult{}, nil },
}
data, err := json.Marshal(tool)
if err != nil {
t.Fatalf("failed to marshal: %v", err)
}
var m map[string]any
if err := json.Unmarshal(data, &m); err != nil {
t.Fatalf("failed to unmarshal: %v", err)
}
if _, ok := m["metadata"]; ok {
t.Errorf("expected metadata to be omitted, got %v", m)
}
})
}

func TestClient_CreateSession_AllowsMissingPermissionHandler(t *testing.T) {
t.Run("accepts nil config before connection validation", func(t *testing.T) {
client := NewClient(&ClientOptions{Connection: StdioConnection{Path: "/__nonexistent_copilot_binary__"}})
Expand Down
6 changes: 6 additions & 0 deletions go/types.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1269,6 +1269,12 @@ type Tool struct {
// Defer controls whether the tool may be deferred (loaded lazily via tool
// search) rather than always pre-loaded. When empty, the runtime decides.
Defer ToolDefer `json:"defer,omitempty"`
// Metadata is opaque, host-defined metadata associated with the tool
// definition. Keys are namespaced and not part of the stable public API;
// the SDK forwards them verbatim to the runtime, which may recognize
// specific keys to inform host-specific behavior. Unknown keys are
// preserved and round-tripped untouched.
Comment thread
belaltaher8 marked this conversation as resolved.
Outdated
Metadata map[string]any `json:"metadata,omitempty"`
// Handler is optional. When nil, the SDK exposes the tool declaration but does
// not automatically invoke it.
Handler ToolHandler `json:"-"`
Expand Down
82 changes: 64 additions & 18 deletions java/src/main/java/com/github/copilot/rpc/ToolDefinition.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,10 @@
* controls whether the tool may be deferred (loaded lazily via tool
* search) rather than always pre-loaded; {@code null} lets the
* runtime decide
* @param metadata
* opaque, host-defined metadata forwarded verbatim to the runtime;
* keys are namespaced and not part of the stable public API;
* {@code null} when unset
* @see SessionConfig#setTools(java.util.List)
* @see ToolHandler
* @since 1.0.0
Expand All @@ -83,7 +87,8 @@
public record ToolDefinition(@JsonProperty("name") String name, @JsonProperty("description") String description,
@JsonProperty("parameters") Object parameters, @JsonIgnore ToolHandler handler,
@JsonProperty("overridesBuiltInTool") Boolean overridesBuiltInTool,
@JsonProperty("skipPermission") Boolean skipPermission, @JsonProperty("defer") ToolDefer defer) {
@JsonProperty("skipPermission") Boolean skipPermission, @JsonProperty("defer") ToolDefer defer,
@JsonProperty("metadata") Map<String, Object> metadata) {

@SteveSandersonMS SteveSandersonMS Jul 9, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like this would be a breaking change.

@edburns What's the recommended way to handle this in Java? Should this be added purely as a setter and not a constructor parameter? Or should it be on a new constructor overload?

@belaltaher8 belaltaher8 Jul 10, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after looking into it more yea this would be a breaking change 😭 it seems like overloading the record wth multiple constructors to maintain backwards compatibility is the recommendation from googling, but ill defer to @edburns for guidance 🙇🏼

@edburns edburns Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @belaltaher8 , thanks for looping me in. Thankfully, even if its a breaking change, it's ok because we have the protection of the @CopilotExperimental annotation on some of the methods. But perhaps the impacted methods do not have that annotation on it. Will evaluate.

@edburns edburns Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this, @belaltaher8. From my perspective as the Java SME, these two items are the most important. I realize this may come across as a pain in the ass, but Microsoft has decided that language idiomatic SDK surface area is a key differentiator from our competition. So when we add new features, we need to "localize" them with as much fidelity as possible.

  1. ADR-005 gap: annotation-based tools still can’t express metadata

    The new property bag is available on ToolDefinition, but @CopilotTool users (ToolDefinition.fromObject(...) / fromClass(...)) still can’t set it. Today:

    • com.github.copilot.tool.CopilotTool has no metadata member.

    • com.github.copilot.tool.CopilotToolProcessor#writeToolDefinition(...) always emits null for metadata.

    So ADR-005 is functionally behind ADR-006 unless users manually post-process definitions.

    What to change:

    • java/src/main/java/com/github/copilot/tool/CopilotTool.java

      Add nested annotation types and a new member:

      • MetadataEntry[] metadata() default {};

      • nested @interface MetadataEntry { String key(); MetadataValue value(); }

      • nested @interface MetadataValue { MetadataFlag[] flags() default {}; boolean bool() default false; String str() default ""; }

      • nested @interface MetadataFlag { String name(); boolean value(); }

      This keeps everything in one source file and avoids adding multiple top-level annotation classes.

    • java/src/main/java/com/github/copilot/tool/CopilotToolProcessor.java

      In writeToolDefinition(...), replace hardcoded null metadata emission with generated map source:

      • when metadata().length == 0 emit null

      • otherwise emit Map.of(...) / nested Map.of(...) matching annotation values.

      Add helper(s) that convert CopilotTool.MetadataEntry[] to Java source literals.

    • java/src/test/java/com/github/copilot/tool/CopilotToolProcessorTest.java

      Add compile-time generation assertions for:

      • metadata present → generated new ToolDefinition(..., ..., ..., metadataMap) includes expected key(s) and nested flag map

      • metadata absent → generated constructor argument is null.

      • Docs/examples

      • Update ADR-005 example snippet (currently old ctor shape) to include metadata position or, better, use stable factory-style examples.

  2. Use an overloaded constructor for compatibility

    Adding metadata as a new record component changed constructor arity. To preserve compatibility for existing direct constructor call sites (including previously generated ADR-005 classes), add a backward-compatible overload.

    What to change:

    • java/src/main/java/com/github/copilot/rpc/ToolDefinition.java

    Keep canonical 8-arg record constructor and add:

    public ToolDefinition(String name, String description, Object parameters, ToolHandler handler,
        Boolean overridesBuiltInTool, Boolean skipPermission, ToolDefer defer) {
    this(name, description, parameters, handler, overridesBuiltInTool, skipPermission, defer, null);
}

    • Keep PR’s 8-arg updates in factory methods and processor output.

    • Keep fluent copy methods carrying metadata through (overridesBuiltInTool(...), skipPermission(...), defer(...)) and .metadata(...) setter-copy method.

Testing guidance (specific):

  • ToolDefinitionTest

  • 7-arg overload sets metadata == null

  • 8-arg/copy .metadata(...) serializes metadata

  • chaining flags before/after .metadata(...) preserves metadata

  • CopilotToolProcessorTest

  • nested-annotation metadata generates expected Map.of(...) source

  • no metadata generates null argument

  • combined flags + metadata all appear in generated constructor call

  • Generated fixture coverage

  • update $$CopilotToolMeta fixture files to include the final constructor shape and metadata emission behavior where applicable

  • Run

    • cd java && mvn verify to cover processor tests + runtime tests end-to-end.

    • cd java && mvn spotless:apply to ensure formatting rules are kept.

I will paste the AI generated sketch of the new inner-annotations to enable @CopilotTool availability for your useful new feature next.

@edburns edburns Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

/*---------------------------------------------------------------------------------------------
 *  Copyright (c) Microsoft Corporation. All rights reserved.
 *--------------------------------------------------------------------------------------------*/

package com.github.copilot.tool;

import java.lang.annotation.Documented;
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;

import com.github.copilot.CopilotExperimental;
import com.github.copilot.rpc.ToolDefer;

/**
 * Marks a method as a Copilot tool. The annotated method will be exposed to the
 * model as a callable tool during a session.
 */
@Documented
@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.METHOD)
@CopilotExperimental
public @interface CopilotTool {

    /** Tool description (sent to the model). */
    String value();

    /** Tool name. Defaults to method name converted to snake_case. */
    String name() default "";

    /** Whether this tool overrides a built-in tool. */
    boolean overridesBuiltInTool() default false;

    /** Whether to skip permission checks. */
    boolean skipPermission() default false;

    /** Defer configuration for this tool. */
    ToolDefer defer() default ToolDefer.NONE;

    /**
     * Optional metadata bag for host-defined properties.
     *
     * Example emitted shape:
     * Map.of("github.com/copilot:safeForTelemetry",
     *        Map.of("name", true, "inputsNames", false))
     */
    MetadataEntry[] metadata() default {};

    @Documented
    @Retention(RetentionPolicy.RUNTIME)
    @Target({})
    @interface MetadataEntry {
        String key();
        MetadataValue value();
    }

    @Documented
    @Retention(RetentionPolicy.RUNTIME)
    @Target({})
    @interface MetadataValue {
        // Scalar options for simple cases
        boolean bool() default false;
        String str() default "";

        // Object-like value for structured cases
        MetadataFlag[] flags() default {};
    }

    @Documented
    @Retention(RetentionPolicy.RUNTIME)
    @Target({})
    @interface MetadataFlag {
        String name();
        boolean value();
    }
}

Example usage:

@CopilotTool(
    value = "Reports phase",
    metadata = {
        @CopilotTool.MetadataEntry(
            key = "github.com/copilot:safeForTelemetry",
            value = @CopilotTool.MetadataValue(
                flags = {
                    @CopilotTool.MetadataFlag(name = "name", value = true),
                    @CopilotTool.MetadataFlag(name = "inputsNames", value = false)
                }
            )
        )
    }
)

public String reportPhase(String phase) {
    return phase;
}


/**
* Creates a tool definition with a JSON schema for parameters.
Expand All @@ -103,7 +108,7 @@ public record ToolDefinition(@JsonProperty("name") String name, @JsonProperty("d
*/
public static ToolDefinition create(String name, String description, Map<String, Object> schema,
ToolHandler handler) {
return new ToolDefinition(name, description, schema, handler, null, null, null);
return new ToolDefinition(name, description, schema, handler, null, null, null, null);
}

/**
Expand All @@ -127,7 +132,7 @@ public static ToolDefinition create(String name, String description, Map<String,
*/
public static ToolDefinition createOverride(String name, String description, Map<String, Object> schema,
ToolHandler handler) {
return new ToolDefinition(name, description, schema, handler, true, null, null);
return new ToolDefinition(name, description, schema, handler, true, null, null, null);
}

/**
Expand All @@ -150,7 +155,7 @@ public static ToolDefinition createOverride(String name, String description, Map
*/
public static ToolDefinition createSkipPermission(String name, String description, Map<String, Object> schema,
ToolHandler handler) {
return new ToolDefinition(name, description, schema, handler, null, true, null);
return new ToolDefinition(name, description, schema, handler, null, true, null, null);
}

/**
Expand All @@ -176,7 +181,32 @@ public static ToolDefinition createSkipPermission(String name, String descriptio
*/
public static ToolDefinition createWithDefer(String name, String description, Map<String, Object> schema,
ToolHandler handler, ToolDefer defer) {
return new ToolDefinition(name, description, schema, handler, null, null, defer);
return new ToolDefinition(name, description, schema, handler, null, null, defer, null);
}

/**
* Creates a tool definition with opaque, host-defined metadata.
* <p>
* Use this factory method to attach namespaced metadata that the SDK forwards
* verbatim to the runtime. The keys are not part of the stable public API; the
* runtime may recognize specific keys to inform host-specific behavior.
*
* @param name
* the unique name of the tool
* @param description
* a description of what the tool does
* @param schema
* the JSON Schema as a {@code Map}
* @param handler
* the handler function to execute when invoked
* @param metadata
* the opaque metadata map forwarded to the runtime
* @return a new tool definition with the metadata set
* @since 1.0.7
*/
public static ToolDefinition createWithMetadata(String name, String description, Map<String, Object> schema,
ToolHandler handler, Map<String, Object> metadata) {
return new ToolDefinition(name, description, schema, handler, null, null, null, metadata);
}

/**
Expand Down Expand Up @@ -247,7 +277,7 @@ public static List<ToolDefinition> fromClass(Class<?> clazz) {
*/
@CopilotExperimental
public ToolDefinition overridesBuiltInTool(boolean value) {
return new ToolDefinition(name, description, parameters, handler, value, skipPermission, defer);
return new ToolDefinition(name, description, parameters, handler, value, skipPermission, defer, metadata);
}

/**
Expand All @@ -261,7 +291,7 @@ public ToolDefinition overridesBuiltInTool(boolean value) {
*/
@CopilotExperimental
public ToolDefinition skipPermission(boolean value) {
return new ToolDefinition(name, description, parameters, handler, overridesBuiltInTool, value, defer);
return new ToolDefinition(name, description, parameters, handler, overridesBuiltInTool, value, defer, metadata);
}

/**
Expand All @@ -275,7 +305,23 @@ public ToolDefinition skipPermission(boolean value) {
*/
@CopilotExperimental
public ToolDefinition defer(ToolDefer value) {
return new ToolDefinition(name, description, parameters, handler, overridesBuiltInTool, skipPermission, value);
return new ToolDefinition(name, description, parameters, handler, overridesBuiltInTool, skipPermission, value,
metadata);
}

/**
* Returns a copy with the opaque {@code metadata} bag set.
*
* @param value
* the opaque, host-defined metadata forwarded verbatim to the
* runtime; keys are namespaced and not part of the stable public API
* @return a new {@code ToolDefinition} with the metadata applied
* @since 1.0.7
*/
@CopilotExperimental
public ToolDefinition metadata(Map<String, Object> value) {
return new ToolDefinition(name, description, parameters, handler, overridesBuiltInTool, skipPermission, defer,
value);
}

// ------------------------------------------------------------------
Expand Down Expand Up @@ -319,7 +365,7 @@ public static <R> ToolDefinition from(String name, String description, Supplier<
R result = handler.get();
return CompletableFuture.completedFuture(formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

/**
Expand Down Expand Up @@ -361,7 +407,7 @@ public static <T1, R> ToolDefinition from(String name, String description, Param
R result = handler.apply(arg1);
return CompletableFuture.completedFuture(formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

/**
Expand Down Expand Up @@ -409,7 +455,7 @@ public static <T1, T2, R> ToolDefinition from(String name, String description, P
R result = handler.apply(arg1, arg2);
return CompletableFuture.completedFuture(formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

// ------------------------------------------------------------------
Expand Down Expand Up @@ -458,7 +504,7 @@ public static <R> ToolDefinition fromAsync(String name, String description,
}
return future.thenApply(result -> formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

/**
Expand Down Expand Up @@ -506,7 +552,7 @@ public static <T1, R> ToolDefinition fromAsync(String name, String description,
}
return future.thenApply(result -> formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

/**
Expand Down Expand Up @@ -551,7 +597,7 @@ public static <T1, T2, R> ToolDefinition fromAsync(String name, String descripti
}
return future.thenApply(result -> formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

// ------------------------------------------------------------------
Expand Down Expand Up @@ -594,7 +640,7 @@ public static <R> ToolDefinition fromWithToolInvocation(String name, String desc
R result = handler.apply(invocation);
return CompletableFuture.completedFuture(formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

/**
Expand Down Expand Up @@ -640,7 +686,7 @@ public static <T1, R> ToolDefinition fromWithToolInvocation(String name, String
R result = handler.apply(arg1, invocation);
return CompletableFuture.completedFuture(formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

// ------------------------------------------------------------------
Expand Down Expand Up @@ -689,7 +735,7 @@ public static <R> ToolDefinition fromAsyncWithToolInvocation(String name, String
}
return future.thenApply(result -> formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

/**
Expand Down Expand Up @@ -741,7 +787,7 @@ public static <T1, R> ToolDefinition fromAsyncWithToolInvocation(String name, St
}
return future.thenApply(result -> formatResult(result, mapper));
};
return new ToolDefinition(name, description, schema, toolHandler, null, null, null);
return new ToolDefinition(name, description, schema, toolHandler, null, null, null, null);
}

// ------------------------------------------------------------------
Expand Down
3 changes: 2 additions & 1 deletion java/src/main/java/com/github/copilot/tool/CopilotToolProcessor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -276,7 +276,8 @@ private void writeToolDefinition(PrintWriter out, ExecutableElement method) {
out.println(" },");
out.println(" " + overridesArg + ",");
out.println(" " + skipPermArg + ",");
out.println(" " + deferArg);
out.println(" " + deferArg + ",");
out.println(" null");
Comment thread
belaltaher8 marked this conversation as resolved.
out.print(" )");
}

Expand Down
Loading
Loading