Commit e18c96c
authored
refactor(11/12): update manifests, config, docs, and example projects (#329)
## Summary
This is **PR 11 of 12** in a stacked PR series that decouples the rendering pipeline from MCP transport. Depends on PR 10 (boundary rewiring).
Updates all configuration files, YAML manifests, documentation, and example projects to reflect the rendering pipeline refactor. No behavioral code changes -- this is metadata, documentation, and project configuration.
### Manifest YAML updates (28 files)
All tool manifests updated to reflect the simplified handler contract. Changes are consistent across all manifests:
- Removed output format configuration that is now handled by the render session
- Updated parameter descriptions where they referenced the old rendering model
### New resource manifests
Added `manifests/resources/` directory with resource manifest definitions that were previously inline.
### Configuration
- `package.json` + `package-lock.json`: Dependency updates and script changes
- `knip.json`: Dead code analysis configuration for the new module structure
- `vitest.config.ts`: Minor updates for new test paths
- `vitest.flowdeck.config.ts` + `vitest.snapshot.config.ts`: New vitest configs for flowdeck integration tests and snapshot tests respectively
### Documentation
New developer documentation explaining the rendering pipeline architecture:
- `RENDERING_PIPELINE.md`: Architecture overview for contributors
- `RENDERING_PIPELINE_REFACTOR.md`: Migration guide and decision log
- `QUERY_TOOL_FORMAT_SPEC.md`: Specification for query tool output formatting
- `FIXTURE_DESIGNS.md`: Snapshot test fixture design documentation
- `STRUCTURED_XCODEBUILD_EVENTS_PLAN.md`: Design document for the xcodebuild event model
- Updated `ARCHITECTURE.md`, `TESTING.md`, `MANIFEST_FORMAT.md`, `TOOL_DISCOVERY_LOGIC.md`
### Example projects
Minor updates to example projects to work with the updated tool interfaces. Updated test files and project configuration.
### Other
- `AGENTS.md`: Updated project rules
- New test infrastructure: `test-helpers.ts`, `vitest-executor-safety.setup.ts`
- Build scripts updated: removed `copy-build-assets.js` (no longer needed), added benchmark and capture wrapper scripts
## Stack navigation
- PR 1-10/12: All code changes
- **PR 11/12** (this PR): Manifests, config, docs, examples
- PR 12/12: Snapshot test fixtures and benchmarks
## Test plan
- [ ] `npx vitest run` passes
- [ ] Manifest validation passes for all YAML files
- [ ] Documentation renders correctly in GitHub
- [ ] Example projects build successfully1 parent 2063cca commit e18c96c
68 files changed
Lines changed: 4130 additions & 720 deletions
File tree
- docs
- dev
- example_projects
- iOS_Calculator
- CalculatorAppPackage/Sources/CalculatorAppFeature
- CalculatorAppTests
- CalculatorApp
- manual-fixtures/CalculatorAppTests
- macOS
- MCPTest.xcodeproj
- MCPTestTests
- spm
- .xcodebuildmcp
- Sources/quick-task
- Tests/TestLibTests
- manifests
- resources
- tools
- scripts
- src/test-utils
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
| 49 | + | |
49 | 50 | | |
50 | 51 | | |
51 | 52 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
77 | 77 | | |
78 | 78 | | |
79 | 79 | | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
80 | 108 | | |
81 | 109 | | |
82 | 110 | | |
| |||
85 | 113 | | |
86 | 114 | | |
87 | 115 | | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
88 | 119 | | |
89 | 120 | | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
90 | 125 | | |
91 | 126 | | |
92 | 127 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
| 25 | + | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
41 | 40 | | |
42 | | - | |
43 | 41 | | |
44 | 42 | | |
45 | 43 | | |
46 | 44 | | |
47 | 45 | | |
48 | | - | |
| 46 | + | |
49 | 47 | | |
50 | 48 | | |
51 | 49 | | |
52 | | - | |
| 50 | + | |
53 | 51 | | |
54 | 52 | | |
55 | 53 | | |
56 | 54 | | |
57 | 55 | | |
58 | 56 | | |
59 | 57 | | |
60 | | - | |
61 | | - | |
| 58 | + | |
62 | 59 | | |
63 | 60 | | |
64 | 61 | | |
65 | 62 | | |
66 | 63 | | |
67 | 64 | | |
68 | 65 | | |
69 | | - | |
70 | 66 | | |
71 | | - | |
72 | 67 | | |
73 | 68 | | |
74 | 69 | | |
| |||
87 | 82 | | |
88 | 83 | | |
89 | 84 | | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | 85 | | |
101 | 86 | | |
102 | 87 | | |
| |||
200 | 185 | | |
201 | 186 | | |
202 | 187 | | |
203 | | - | |
204 | | - | |
205 | | - | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
206 | 191 | | |
207 | 192 | | |
208 | 193 | | |
209 | | - | |
| 194 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | | - | |
| 26 | + | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| |||
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
38 | | - | |
39 | 38 | | |
40 | | - | |
41 | 39 | | |
42 | 40 | | |
43 | 41 | | |
44 | 42 | | |
45 | 43 | | |
46 | | - | |
| 44 | + | |
47 | 45 | | |
48 | 46 | | |
49 | | - | |
| 47 | + | |
50 | 48 | | |
51 | 49 | | |
52 | 50 | | |
| |||
55 | 53 | | |
56 | 54 | | |
57 | 55 | | |
58 | | - | |
59 | | - | |
| 56 | + | |
60 | 57 | | |
61 | 58 | | |
62 | 59 | | |
63 | 60 | | |
64 | 61 | | |
65 | 62 | | |
66 | 63 | | |
67 | | - | |
68 | 64 | | |
69 | | - | |
70 | 65 | | |
71 | 66 | | |
72 | 67 | | |
| |||
85 | 80 | | |
86 | 81 | | |
87 | 82 | | |
88 | | - | |
89 | | - | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | 83 | | |
99 | 84 | | |
100 | 85 | | |
| |||
216 | 201 | | |
217 | 202 | | |
218 | 203 | | |
219 | | - | |
220 | | - | |
221 | | - | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
222 | 207 | | |
223 | 208 | | |
224 | 209 | | |
225 | | - | |
| 210 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
| |||
0 commit comments