Skip to content

Commit ef73918

Browse files
nhortonclaude
andcommitted
docs: fix stale documentation found during review
- Add deepschema skill to directory trees in architecture.md and CLAUDE.md - Add get_named_schemas tool (#9) to architecture.md MCP tools section - Update hooks README event mapping (add before_model, after_model, SubagentStop) - Update hooks README tool mapping (add web_fetch, web_search, task) - Add .deepreview to hooks README files table Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 1420d63 commit ef73918

2 files changed

Lines changed: 17 additions & 3 deletions

File tree

doc/architecture.md

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,8 @@ deepwork/ # DeepWork tool repository
104104
│ │ ├── skills/
105105
│ │ │ ├── deepwork/SKILL.md
106106
│ │ │ ├── review/SKILL.md
107-
│ │ │ └── configure_reviews/SKILL.md
107+
│ │ │ ├── configure_reviews/SKILL.md
108+
│ │ │ └── deepschema/SKILL.md
108109
│ │ ├── hooks/ # hooks.json, post_commit_reminder.sh, post_compact.sh, startup_context.sh, deepschema_write.sh
109110
│ │ └── .mcp.json # MCP server config
110111
│ └── gemini/ # Gemini CLI extension
@@ -818,7 +819,7 @@ All MCP server code lives in `src/deepwork/jobs/mcp/`.
818819
819820
The FastMCP server definition that:
820821
- Creates and configures the MCP server instance
821-
- Registers the workflow tools and review tools (`get_review_instructions`, `get_configured_reviews`, `mark_review_as_passed`)
822+
- Registers the workflow tools, review tools (`get_review_instructions`, `get_configured_reviews`, `mark_review_as_passed`), and DeepSchema tools (`get_named_schemas`)
822823
- Detects job definition issues at startup via `issues.py` and appends warnings to tool responses
823824
- Provides server instructions for agents
824825
@@ -914,6 +915,13 @@ Marks a review as passed so it is skipped on subsequent runs while the reviewed
914915
915916
**Returns**: Confirmation string or validation error.
916917
918+
#### 9. `get_named_schemas`
919+
Lists all named DeepSchemas discovered across all schema sources (project-local, standard, and env var). Returns each schema's name, summary, and matcher patterns.
920+
921+
**Parameters**: None.
922+
923+
**Returns**: List of `{name, summary, matchers}` dicts.
924+
917925
### Review Pass Caching
918926
919927
Each review task is assigned a deterministic `review_id` encoding the rule name, file paths, and a SHA-256 content hash (first 12 hex chars). When `get_review_instructions` generates instruction files, it names them `{review_id}.md` and checks for a corresponding `{review_id}.passed` marker. If the marker exists, the review is skipped.

src/deepwork/hooks/README.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,12 +84,14 @@ if __name__ == "__main__":
8484

8585
| DeepWork Normalized | Claude Code | Gemini CLI |
8686
|---------------------|-------------|------------|
87-
| `after_agent` | Stop | AfterAgent |
87+
| `after_agent` | Stop, SubagentStop | AfterAgent |
8888
| `before_tool` | PreToolUse | BeforeTool |
8989
| `after_tool` | PostToolUse | AfterTool |
9090
| `before_prompt` | UserPromptSubmit | BeforeAgent |
9191
| `session_start` | SessionStart | SessionStart |
9292
| `session_end` | SessionEnd | SessionEnd |
93+
| `before_model` || BeforeModel |
94+
| `after_model` || AfterModel |
9395

9496
## Tool Name Mapping
9597

@@ -101,6 +103,9 @@ if __name__ == "__main__":
101103
| `shell` | Bash | shell |
102104
| `glob` | Glob | glob |
103105
| `grep` | Grep | grep |
106+
| `web_fetch` | WebFetch | web_fetch |
107+
| `web_search` | WebSearch | web_search |
108+
| `task` | Task ||
104109

105110
## Decision Values
106111

@@ -136,3 +141,4 @@ pytest tests/shell_script_tests/test_hook_wrappers.py -v
136141
| `deepschema_write.py` | DeepSchema write-time validation hook |
137142
| `claude_hook.sh` | Shell wrapper for Claude Code |
138143
| `gemini_hook.sh` | Shell wrapper for Gemini CLI |
144+
| `.deepreview` | Review rule ensuring hooks use correct output routing (DW-REQ-006.6) |

0 commit comments

Comments
 (0)