tests: add coverage for loadToolsetsFromCfg

[goneri]

Co-Authored-By: Claude noreply@anthropic.com

---

Note

Adds comprehensive tests for toolset construction, covering aggregation into all, overlaps, invalid entries, order, deduplication, and property/reference preservation.

• Tests:
  • Add src/index.test.ts covering loadToolsetsFromCfg behavior.
    • Valid configs and overlaps; empty/non-existent tools handling.
    • Aggregation into all and override when all is user-defined.
    • Deduplication by tool name in all when duplicates exist.
    • Preservation of tool properties and object references.
    • Ensures toolset keys present and order maintained; mocks external modules.

^{Written by Cursor Bugbot for commit eadef9c. This will update automatically on new commits. Configure here.}

[cursor]

Bug: Test implementation differs from actual code behavior

The test recreates loadToolsetsFromCfg locally rather than importing it, and the test implementation is missing the deduplication logic that exists in the actual implementation in src/index.ts. The real function deduplicates tools by name in the all toolset using a reduce operation, but the test version just flattens all entries directly. This causes tests like "should handle overlapping toolsets correctly" to expect 5 tools in all when the actual implementation would return 3 deduplicated tools.

Additional Locations (2)

• src/index.test.ts#L184-L185
• src/index.test.ts#L318-L325