- Notifications
You must be signed in to change notification settings - Fork3.2k
Commitbac6db2
refactor(gists): migrate gists.go to NewTool pattern (#1591)
* Migrate context_tools to new ServerTool patternConvert GetMe, GetTeams, and GetTeamMembers to use the new typeddependency injection pattern:- Functions now take only translations helper, return toolsets.ServerTool- Handler is generated lazily via deps.GetClient/deps.GetGQLClient- Tests updated to use serverTool.Handler(deps) pattern- Fixed error return pattern to return nil for Go error (via result.IsError)Co-authored-by: Adam Holt <oholt@github.com>* refactor(gists): migrate gists.go to NewTool patternConvert all gist tools (ListGists, GetGist, CreateGist, UpdateGist)to use the new NewTool helper with ToolDependencies injection.- Remove getClient parameter from function signatures- Use deps.GetClient(ctx) inside handlers- Standardize error handling with utils.NewToolResultErrorFromErr()- Update all tests to use serverTool.Handler(deps) patternCo-authored-by: Adam Holt <omgitsads@users.noreply.github.com>---------Co-authored-by: Adam Holt <oholt@github.com>Co-authored-by: Adam Holt <omgitsads@users.noreply.github.com>1 parentb863da9 commitbac6db2
3 files changed
+376
-375
lines changed0 commit comments
Comments
(0)