core/api
8
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions

feat(cmd/api): dedupe sdk spec groups

Browse source
This commit is contained in:
Virgil 2026-04-01 23:18:19 +00:00
parent eb7e1e51cb
commit 14eedd7f91
2 changed files with 24 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
cmd/api/cmd_sdk.go
View file

@ -131,8 +131,11 @@ func sdkSpecBuilder(title, description, version, termsURL, contactName, contactU
}
func sdkSpecGroups() []goapi.RouteGroup {
bridge := goapi.NewToolBridge("/tools")
return append(goapi.RegisteredSpecGroups(), bridge)
groups := make([]goapi.RouteGroup, 0)
for group := range sdkSpecGroupsIter() {
groups = append(groups, group)
}
return groups
}
func sdkSpecGroupsIter() iter.Seq[goapi.RouteGroup] {

19
cmd/api/cmd_test.go
View file

@ -589,3 +589,22 @@ func TestAPISDKCmd_Good_TempSpecUsesMetadataFlags(t *testing.T) {
t.Fatalf("expected second server to be /, got %v", servers[1])
}
}
func TestAPISDKCmd_Good_SpecGroupsDeduplicateToolBridge(t *testing.T) {
snapshot := api.RegisteredSpecGroups()
api.ResetSpecGroups()
t.Cleanup(func() {
api.ResetSpecGroups()
api.RegisterSpecGroups(snapshot...)
})
api.RegisterSpecGroups(api.NewToolBridge("/tools"))
groups := sdkSpecGroups()
if len(groups) != 1 {
t.Fatalf("expected the built-in tools bridge to be deduplicated, got %d groups", len(groups))
}
if groups[0].BasePath() != "/tools" {
t.Fatalf("expected the remaining group to be /tools, got %s", groups[0].BasePath())
}
}