Skip to content

docs: fix stale safe-outputs.memory references and document McpOptions.enabled#327

Merged
jamesadevine merged 2 commits into
mainfrom
copilot/fix-documentation-drift-safe-outputs
Apr 27, 2026
Merged

docs: fix stale safe-outputs.memory references and document McpOptions.enabled#327
jamesadevine merged 2 commits into
mainfrom
copilot/fix-documentation-drift-safe-outputs

Conversation

Copilot AI commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Two documentation drift issues found by the weekly audit: stale references to the removed safe-outputs.memory key (memory moved to tools.cache-memory) and a completely undocumented enabled: false MCP toggle.

Stale clearMemory injection trigger (AGENTS.md)

Two places incorrectly attributed the auto-injected clearMemory parameter to safe-outputs.memory; corrected to tools.cache-memory to match the actual check in src/compile/common.rs.

Document McpOptions.enabled (AGENTS.md, README.md, prompts/create-ado-agentic-workflow.md)

The enabled: false field — which silently skips an MCP entry at compile time without requiring deletion — was absent from all user-facing docs. Added to the Configuration Properties section in AGENTS.md, as a comment in the README.md YAML example, and in Step 8 of the prompt template:

mcp-servers:
  my-tool:
    container: "node:20-slim"
    enabled: false   # temporarily disable without removing the entry
    allowed:
      - do_thing

Copilot AI changed the title [WIP] Fix documentation drift for safe-outputs.memory and MCP enabled field docs: fix stale safe-outputs.memory references and document McpOptions.enabled Apr 27, 2026
Copilot AI requested a review from jamesadevine April 27, 2026 13:36
@jamesadevine jamesadevine marked this pull request as ready for review April 27, 2026 13:37
@jamesadevine jamesadevine merged commit 0097428 into main Apr 27, 2026
@jamesadevine jamesadevine deleted the copilot/fix-documentation-drift-safe-outputs branch April 27, 2026 13:51
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.

📝 Documentation drift detected — safe-outputs.memory references and undocumented MCP enabled field

2 participants