Skip to content

Conversation

@hifi-phil
Copy link
Contributor

Summary

  • Add tool slices for categorizing tools by operation type (CRUD, navigation, workflow, information, entity management, system)
  • Consolidate slice definitions into single source of truth (slice-registry.ts)
  • Add mode expansion system with base modes (content, media, translation, etc.) and compound modes (publisher, developer, admin, full)
  • Rename 'editor' compound mode to 'publisher' to avoid confusion with future Editor MCP
  • Migrate all tools from factory pattern to decorator pattern with explicit slice assignments
  • Update slice matching to AND logic (tools must have ALL requested slices)
  • Add comprehensive tests for tool factory covering readonly mode, slices, and modes
  • Fix indexer test to normalize dynamic field values

Test plan

  • All existing tests pass
  • New slice-matcher tests pass
  • New mode-expander tests pass
  • New collection-config-loader-modes tests pass
  • New tool-factory-integration tests pass
  • Manual verification of tool filtering with different slice combinations
  • Manual verification of mode expansion with publisher, developer, admin, full modes

🤖 Generated with Claude Code

- Add tool slices for categorizing tools by operation type (CRUD, navigation, workflow, etc.)
- Consolidate slice definitions into single source of truth (slice-registry.ts)
- Add mode expansion system with base modes and compound modes (publisher, developer, admin, full)
- Rename 'editor' compound mode to 'publisher' to avoid confusion with future Editor MCP
- Migrate all tools to decorator pattern with explicit slice assignments
- Update slice matching to AND logic (tools must have ALL requested slices)
- Add comprehensive tests for tool factory covering readonly mode, slices, and modes
- Fix indexer test to normalize dynamic field values

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants