view tool call always fails with "Error:Path does not exist" when CreateSessionFsProvider is set in the session config
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 68/100
Direzione di ricerca
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”.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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 },
}
};
}
}
}
- Lingua principale
- Java
- Stelle
- 10.5k
- Fork
- 1.5k
- Merge medio
- 1g 9h
- PR unite (30g)
- 130
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di github/copilot-sdk
-
agentic-workflows
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
github/copilot-sdk#2760 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
github/copilot-sdk#2759 ·
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
github/copilot-sdk#2758 ·
-
agentic-workflows
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
github/copilot-sdk#2709 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
github/copilot-sdk#2673 ·
Tutte le issue di github/copilot-sdk
Issue simili
-
executions.Query — startDate and timeRange filters are sent with inverted comparison operators Apertaarea/plugin
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
kestra-io/plugin-kestra#190 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
apache/rocketmq-dashboard#5064 ·
-
Consent portal: creating a duplicate Purpose shows a generic error instead of "already exists" Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
wso2/dpdp-accelerator#287 ·