Skip to content

Commit ad2fe9b

Browse files
committed
Mark TODO items done and trim MCP proposal
Update TODO.md: mark Phase 2 TUI enhancements, the `sap-devs` MCP server entry, and scratch/session context as DONE. Remove the detailed proposed-tools and architecture proposal for exposing `sap-devs` as an MCP server (tool list, architecture notes, and dependency comment) to reflect that the plan has been completed or consolidated.
1 parent 05f1c99 commit ad2fe9b

1 file changed

Lines changed: 3 additions & 23 deletions

File tree

TODO.md

Lines changed: 3 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ Interactive UI for editing and maintaining pack content YAML files (event-types.
200200
- ✅ Schema-driven forms: enum fields render as select dropdowns, URI/pattern fields get inline validation, required field highlighting
201201
- ✅ Layer resolution: all 4 layers (official, company, user, project) respected; user-layer overrides auto-created when editing from home directory
202202

203-
#### Phase 2 — TUI Enhancements
203+
#### Phase 2 — TUI Enhancements - DONE ✔️
204204

205205
- Undo/redo support within the editor session - DONE ✔️
206206
- Diff view showing changes against the lower-layer version before saving - DONE ✔️
@@ -287,32 +287,12 @@ A persistent system-tray (or menu-bar) icon that surfaces tool status, triggers
287287

288288
## AI Agent Influence
289289

290-
### `sap-devs` as an MCP server
290+
### `sap-devs` as an MCP server - DONE ✔️
291291

292292
Expose `sap-devs` as a live MCP server so AI agents can query it on demand during a conversation, instead of relying solely on static injected text.
293293

294294
**Problem:** Static injection pushes everything upfront and hopes the agent reads it. With an MCP server, the agent pulls specific context when it needs it — no token budget pressure, always fresh, and topically relevant to the current task.
295295

296-
**Proposed tools the server would expose:**
297-
298-
- `get_tip(pack, topic)` — return one actionable tip, optionally filtered by topic keyword
299-
- `search_resources(query, pack)` — return matching curated resources with URLs
300-
- `get_context(profile)` — return the full context block for a profile on demand
301-
- `get_recent_news()` — return latest SAP Developer News episodes
302-
- `list_packs()` — enumerate available packs so the agent knows what domains are covered
303-
- `get_known_errors(pattern)` — look up a SAP error message and return cause + fix (backing data: `known_errors.yaml` ✅)
304-
305-
**Architecture:**
306-
307-
- Implemented as a sub-command: `sap-devs mcp serve` starts the MCP server process
308-
- Registered in the tool's MCP config via `sap-devs mcp install sap-devs-server` (reusing the existing `mcp-wire` adapter mechanism)
309-
- Reads from the same pack content as `inject` — no duplicate data
310-
- Stateless: each tool call loads packs fresh (or from cache); no daemon required
311-
312-
**Why this matters:** This inverts the whole model. Instead of "push everything and hope", the agent fetches what it needs, when it needs it. Particularly powerful for tools like Claude Code where the agent can decide mid-task that it needs CAP-specific context.
313-
314-
**Dependency:** Requires the base MCP infrastructure in `cmd/mcp.go` to be extended to support self-hosting, not just wiring third-party servers.
315-
316296
---
317297

318298
### Behavioral rules / anti-patterns injection - DONE ✔️
@@ -339,7 +319,7 @@ Implemented: added `## sap-devs CLI Reference (for AI agents)` table to `content
339319

340320
---
341321

342-
### ~~Scratch/session context — `sap-devs context add`~~
322+
### ~~Scratch/session context — `sap-devs context add`~~ DONE ✔️
343323

344324
Implemented: `context add/list/clear` commands with `.sap-devs/scratch.yaml` storage and `## Current Context` injection in project-scope output.
345325

0 commit comments

Comments
 (0)