opencode/yao
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Improve MCP loading process by verifying directory existence before file operations

Browse source 
- Added a check to confirm the existence of the 'mcps' directory prior to loading MCP files, enhancing error handling and preventing unnecessary attempts to load from a non-existent directory.
- Updated error handling to ensure that any issues encountered during the loading process are logged appropriately.
This commit is contained in:
Max 2025-11-28 19:56:16 +08:00
parent 36e3334058
commit 125c32a0b5
1 changed files with 21 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
mcp/mcp.go
View file

@ -19,24 +19,29 @@ import (
func Load(cfg config.Config) error {
messages := []string{}
// Load filesystem MCP clients
exts := []string{"*.mcp.yao", "*.mcp.json", "*.mcp.jsonc"}
err := application.App.Walk("mcps", func(root, file string, isdir bool) error {
if isdir {
return nil
}
_, err := mcp.LoadClient(file, share.ID(root, file))
if err != nil {
messages = append(messages, err.Error())
}
return err
}, exts...)
// Check if mcps directory exists
exists, err := application.App.Exists("mcps")
if len(messages) > 0 {
for _, message := range messages {
log.Error("Load filesystem MCP clients error: %s", message)
// Load filesystem MCP clients if directory exists
if err == nil && exists {
exts := []string{"*.mcp.yao", "*.mcp.json", "*.mcp.jsonc"}
err = application.App.Walk("mcps", func(root, file string, isdir bool) error {
if isdir {
return nil
}
_, err := mcp.LoadClient(file, share.ID(root, file))
if err != nil {
messages = append(messages, err.Error())
}
return err
}, exts...)
if len(messages) > 0 {
for _, message := range messages {
log.Error("Load filesystem MCP clients error: %s", message)
}
return fmt.Errorf("%s", strings.Join(messages, ";\n"))
}
return fmt.Errorf("%s", strings.Join(messages, ";\n"))
}
// Load database MCP clients (ignore error)