Skip to content

Commit 3c4fdbd

Browse files
committed
docs: add tool mode separation plan for UI vs normal mode architecture
Add comprehensive implementation plan for separating UI-mode (OpenAI) and normal-mode tool behavior into independent, self-contained modules. Key design decisions: - Actor Executor pattern for direct actor tool mode awareness - Three-layer architecture (core → registry → mode-specific tools) - Object.freeze for tool definition immutability (Skyfire safety) - Mode-aware category registry eliminating deep-clone hack - 5-phase chained PR strategy for incremental delivery
1 parent 587bdc7 commit 3c4fdbd

File tree

2 files changed

+713
-0
lines changed

2 files changed

+713
-0
lines changed

res/index.md

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,22 @@ Refactor plan for modularizing existing resource handling (no new resources).
4242
- Behavior-preserving steps and non-goals
4343
- **Use case**: Step-by-step guide for refactoring without behavior change
4444

45+
### [tool-mode-separation-plan.md](./tool-mode-separation-plan.md)
46+
Implementation plan for separating UI-mode (OpenAI) and normal-mode tool behavior into independent modules.
47+
48+
**Key approach:** Actor Executor pattern + separate tool definitions per mode + shared core logic layer.
49+
50+
**Estimated effort:** 6-10 developer days
51+
52+
- Design decisions table (actor-mcp passthrough, Skyfire freeze, task lifecycle, etc.)
53+
- Three-layer architecture (core → registry → mode-specific tools)
54+
- Actor Executor pattern for direct actor tools (`type: 'actor'`) mode awareness
55+
- Tool definition immutability via `Object.freeze` (Skyfire safety)
56+
- Mode-aware category registry eliminating deep-clone hack
57+
- 5-phase migration plan with chained PR strategy (7 PRs)
58+
- Directory structure and complete file manifest with PR assignments
59+
- **Use case**: Reference for implementing the UI/normal mode tool separation
60+
4561
### [patterns-for-simplification.md](./patterns-for-simplification.md)
4662
Analysis of patterns from the **official TypeScript MCP SDK** and **FastMCP** framework that could simplify the codebase.
4763

0 commit comments

Comments
 (0)