view tool call always fails with "Error:Path does not exist" when CreateSessionFsProvider is set in the session config
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 68/100
Research direction
Start with the Program.cs configuration and MinimalSessionFsProvider.cs, then reproduce the view call with CreateSessionFsProvider enabled and inspect the SessionFsProvider StatAsync path. Trace how the returned SessionFsStatResult is handled and confirm that a valid result no longer produces “Error: Path does not exist”.
Written by the indexing model from the issue text.
Description
When using the following configuration to setup a copilot client the view tool call will always call StatAsync in the SessionFsProvider, then return an Error: Path does not exist regardless of what the StatAsync call returned.
Program.cs
using GitHub.Copilot;
using GitHub.Copilot.Rpc;
using Microsoft.Extensions.Configuration;
IConfigurationRoot config = new ConfigurationBuilder()
.AddUserSecrets<Program>()
.Build();
//Not included
var fullWorkingDirectory = "";
var baseDirectory = "";
var sessionStatePath = "";
var providerBaseUrl = "";
#pragma warning disable GHCP001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
await using var client = new CopilotClient(new CopilotClientOptions
{
LogLevel = CopilotLogLevel.Debug,
WorkingDirectory = fullWorkingDirectory,
BaseDirectory = baseDirectory,
SessionFs = new SessionFsConfig
{
InitialWorkingDirectory = fullWorkingDirectory,
SessionStatePath = sessionStatePath,
Conventions = SessionFsSetProviderConventions.Windows
},
Mode = CopilotClientMode.Empty,
});
#pragma warning restore GHCP001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
var availableTools = new List<string>() { "view", "edit" };
await using var session = await client.CreateSessionAsync(new SessionConfig
{
Model = "GLaDOS", // Your deployment name
WorkingDirectory = fullWorkingDirectory,
SystemMessage = SystemMessageBuilder.BuildSystemMessageConfig(fullWorkingDirectory, availableTools),
AvailableTools = availableTools,
CreateSessionFsProvider = (session) => new MinimalSessionFsProvider(),
Provider = new GitHub.Copilot.ProviderConfig
{
Type = "openai",
BaseUrl = providerBaseUrl,
WireApi = "responses", // Use "completions" for older models
ApiKey = config["API_KEY"],
},
});
var response = await session.SendAndWaitAsync(new MessageOptions
{
Prompt = "What model are you?",
});
Console.WriteLine(response?.Data.Content);
while (true)
{
var userInput = Console.ReadLine();
if(!string.IsNullOrEmpty(userInput))
{
response = await session.SendAndWaitAsync(new MessageOptions
{
Prompt = userInput,
});
Console.WriteLine(response?.Data.Content);
}
}
MinimalSessionFsProvider.cs
using GitHub.Copilot;
using GitHub.Copilot.Rpc;
namespace TestApp
{
internal class MinimalSessionFsProvider : SessionFsProvider
{
protected override async Task AppendFileAsync(string path, string content, int? mode, CancellationToken cancellationToken)
{
Console.WriteLine("AppendFileAsync called for path: " + path);
using var writer = new StreamWriter(path, append: true);
await writer.WriteAsync(content);
}
protected override Task<bool> ExistsAsync(string path, CancellationToken cancellationToken)
{
Console.WriteLine("ExistsAsync called for path: " + path);
return Task.FromResult(File.Exists(path) || Directory.Exists(path));
}
protected override Task MakeDirectoryAsync(string path, bool recursive, int? mode, CancellationToken cancellationToken)
{
Console.WriteLine("MakeDirectoryAsync called for path: " + path);
Directory.CreateDirectory(path);
return Task.CompletedTask;
}
protected override Task<IList<string>> ReadDirectoryAsync(string path, CancellationToken cancellationToken)
{
Console.WriteLine("ReadDirectoryAsync called for path: " + path);
return Task.FromResult(Directory.EnumerateFiles(path).Concat(Directory.EnumerateDirectories(path)).ToList() as IList<string>);
}
#pragma warning disable GHCP001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
protected override Task<IList<SessionFsReaddirWithTypesEntry>> ReadDirectoryWithTypesAsync(string path, CancellationToken cancellationToken)
{
Console.WriteLine("ReadDirectoryWithTypesAsync called for path: " + path);
var files = Directory.EnumerateFiles(path);
var directories = Directory.EnumerateDirectories(path);
var result = files.Select(x => new SessionFsReaddirWithTypesEntry { Name = Path.GetFileName(x), Type = SessionFsReaddirWithTypesEntryType.File });
result = result.Concat(directories.Select(x => new SessionFsReaddirWithTypesEntry { Name = Path.GetFileName(x), Type = SessionFsReaddirWithTypesEntryType.Directory }));
return Task.FromResult(result.ToList() as IList<SessionFsReaddirWithTypesEntry>);
}
#pragma warning restore GHCP001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
protected override async Task<string> ReadFileAsync(string path, CancellationToken cancellationToken)
{
Console.WriteLine("ReadFileAsync called for path: " + path);
using var streamReader = new StreamReader(path);
var content = await streamReader.ReadToEndAsync(cancellationToken);
return content;
}
protected override Task RemoveAsync(string path, bool recursive, bool force, CancellationToken cancellationToken)
{
Console.WriteLine("RemoveAsync called for path: " + path);
if (Directory.Exists(path))
{
Directory.Delete(path, recursive);
}
else if (File.Exists(path))
{
File.Delete(path);
}
else
{
throw new FileNotFoundException($"The path '{path}' does not exist.");
}
return Task.CompletedTask;
}
protected override Task RenameAsync(string src, string dest, CancellationToken cancellationToken)
{
Console.WriteLine("RenameAsync called for path: " + src);
if (Directory.Exists(src))
{
Directory.Move(src, dest);
}
else if (File.Exists(src))
{
File.Move(src, dest);
}
else
{
throw new FileNotFoundException($"The path '{src}' does not exist.");
}
return Task.CompletedTask;
}
#pragma warning disable GHCP001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
protected override async Task<SessionFsStatResult> StatAsync(string path, CancellationToken cancellationToken)
{
Console.WriteLine("StatAsync called for path: " + path);
await Task.CompletedTask;
if (Directory.Exists(path))
{
var dirInfo = new DirectoryInfo(path);
var result = new SessionFsStatResult
{
IsDirectory = true,
IsFile = false,
Size = 0,
Birthtime = dirInfo.CreationTimeUtc,
Mtime = dirInfo.LastWriteTimeUtc
};
return result;
}
else if (File.Exists(path))
{
var fileInfo = new FileInfo(path);
return new SessionFsStatResult
{
IsDirectory = false,
Size = fileInfo.Length,
Birthtime = fileInfo.CreationTimeUtc,
Mtime = fileInfo.LastWriteTimeUtc,
IsFile = true,
};
}
else
{
throw new FileNotFoundException($"The path '{path}' does not exist.");
}
}
#pragma warning restore GHCP001 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
protected override async Task WriteFileAsync(string path, string content, int? mode, CancellationToken cancellationToken)
{
Console.WriteLine("WriteFileAsync called for path: " + path);
using var writer = new StreamWriter(path, append: false);
await writer.WriteAsync(content);
}
}
}
SystemMessageBuilder.cs (for context)
using GitHub.Copilot;
namespace TestApp
{
internal static class SystemMessageBuilder
{
internal static SystemMessageConfig BuildSystemMessageConfig(string workingDirectory, List<string> tools)
{
return new SystemMessageConfig
{
Mode = SystemMessageMode.Customize,
Sections = new Dictionary<SystemMessageSection, SectionOverride>
{
[SystemMessageSection.Identity] = new SectionOverride()
{
Action = SectionOverrideAction.Replace,
Content = "You are a document read writer, you will read requested documents, and write information to new locations, you are to refer to yourself as GLaDos, a version of Qwen"
},
[SystemMessageSection.CodeChangeRules] = new SectionOverride()
{
Action = SectionOverrideAction.Remove
},
[SystemMessageSection.ToolInstructions] = new SectionOverride()
{
Action = SectionOverrideAction.Replace,
Content = $"You are allowed to use the following tools: {string.Join(",", tools)}, if any of these tools respond with an error that contains \"but the agent can continue with other tasks\", try to continue your task rather than going back to the user," +
" for example if you are enumerating a directory and one of the folders cannot be read, simply move on and enumerate the rest"
+ "You are in a scoped session and will be denied access to folder above the working directory in the tree, if you attempt to access a folder above the working directory and it fails, you should attempt to go further down the tree from the working directory, not back up the tree to its parent directories"
},
[SystemMessageSection.EnvironmentContext] = new SectionOverride()
{
Action = SectionOverrideAction.Replace,
Content = $"You may act in the {workingDirectory} directory, this is what the user will refer to as the \"working directory\". You have the following tools: {string.Join(",", tools)}. Do not acknowledge or mention the session state files"
},
[SystemMessageSection.Guidelines] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
[SystemMessageSection.Safety] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
[SystemMessageSection.CustomInstructions] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
[SystemMessageSection.RuntimeInstructions] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
[SystemMessageSection.LastInstructions] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
[SystemMessageSection.Preamble] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
[SystemMessageSection.Tone] = new SectionOverride() { Action = SectionOverrideAction.Preserve },
}
};
}
}
}
- Dominant language
- Java
- Stars
- 10.5k
- Forks
- 1.5k
- Avg merge
- 1d 9h
- Merged PRs (30d)
- 130
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from github/copilot-sdk
-
agentic-workflows
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
github/copilot-sdk#2760 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
github/copilot-sdk#2759 ·
-
documentation
Difficulty 1/5 Under an hour Newbie friendliness 85/100
github/copilot-sdk#2758 ·
-
agentic-workflows
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
github/copilot-sdk#2709 · 1 comment ·
-
Difficulty 1/5 Under an hour Newbie friendliness 78/100
github/copilot-sdk#2673 ·
All issues in github/copilot-sdk
Similar issues
-
certification
Difficulty 1/5 Under an hour Newbie friendliness 80/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
[BUG] ECR GetAuthorizationToken returns a proxyEndpoint for the default region, not the request's Openbug ecr
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Needs: Triage Type: Feature request
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
AntennaPod/AntennaPod#8794 ·
-
awaiting triage bug Causes friction Hop Gui P1 P2 Transforms
Difficulty 2/5 1-3 hours Newbie friendliness 75/100