Skip to content

refactor(discussions): migrate to NewTool pattern #1599

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
SamMorrowDrums merged 30 commits into from
Dec 16, 2025
Merged

refactor(discussions): migrate to NewTool pattern #1599

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
d8a485e
refactor(discussions): migrate to NewTool pattern
SamMorrowDrums Dec 13, 2025
10b9770
Refactor security_advisories tools to use NewTool pattern
SamMorrowDrums Dec 13, 2025
63e8b04
refactor: convert projects, labels, and dynamic_tools to NewTool pattern
SamMorrowDrums Dec 13, 2025
7cf759b
Add --features CLI flag for feature flag support
SamMorrowDrums Dec 13, 2025
b0faff0
Add validation tests for tools, resources, and prompts metadata
SamMorrowDrums Dec 13, 2025
dfba00d
Fix default toolsets behavior when not in dynamic mode
SamMorrowDrums Dec 13, 2025
ef6d3df
refactor: address PR review feedback for toolsets
SamMorrowDrums Dec 14, 2025
9eab4e1
refactor: Apply HandlerFunc pattern to resources for stateless NewToo…
SamMorrowDrums Dec 14, 2025
fbb7ff5
refactor: simplify ForMCPRequest switch cases
SamMorrowDrums Dec 14, 2025
ef96f56
refactor(generate_docs): use strings.Builder and AllTools() iteration
SamMorrowDrums Dec 14, 2025
dd40b67
feat(toolsets): add AvailableToolsets() with exclude filter
SamMorrowDrums Dec 14, 2025
5cc47c8
refactor(generate_docs): hoist success logging to generateAllDocs
SamMorrowDrums Dec 14, 2025
b899878
refactor: consolidate toolset validation into ToolsetGroup
SamMorrowDrums Dec 14, 2025
d9d3cc1
refactor: rename toolsets package to registry with builder pattern
SamMorrowDrums Dec 15, 2025
e239e07
fix: remove unnecessary type arguments in helper_test.go
SamMorrowDrums Dec 15, 2025
2edea7f
fix: restore correct behavior for --tools and --toolsets flags
SamMorrowDrums Dec 15, 2025
905b7c8
Move labels tools to issues toolset
SamMorrowDrums Dec 15, 2025
d6b49fb
Restore labels toolset with get_label in both issues and labels
SamMorrowDrums Dec 15, 2025
4406d02
Fix instruction generation and capability advertisement
SamMorrowDrums Dec 15, 2025
c430253
Add tests for dynamic toolset management tools
SamMorrowDrums Dec 15, 2025
bb15a86
Advertise all capabilities in dynamic toolsets mode
SamMorrowDrums Dec 15, 2025
00fb1d2
Improve conformance test with dynamic tool calls and JSON normalization
SamMorrowDrums Dec 15, 2025
6cfac94
Add conformance-report to .gitignore
SamMorrowDrums Dec 15, 2025
0a55eac
Add conformance test CI workflow
SamMorrowDrums Dec 15, 2025
d6809c8
Add map indexes for O(1) lookups in Registry
SamMorrowDrums Dec 15, 2025
9815310
perf(registry): O(1) HasToolset lookup via pre-computed set
SamMorrowDrums Dec 15, 2025
6155fb0
simplify: remove lazy toolsByName map - not needed for actual use cases
SamMorrowDrums Dec 15, 2025
cc954fe
Add generic tool filtering mechanisms to registry package
Copilot Dec 16, 2025
a04be15
docs: improve filter evaluation order and FilteredTools documentation
SamMorrowDrums Dec 16, 2025
447f02a
Refactor GenerateToolsetsHelp() to use strings.Builder pattern
Copilot Dec 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading