Skip to content

docs: fix documentation drift — undocumented create-pr config fields and missing MCP container fields#500

Merged
jamesadevine merged 1 commit into
mainfrom
docs/fix-documentation-drift-9fed47777950a289
May 11, 2026
Merged

docs: fix documentation drift — undocumented create-pr config fields and missing MCP container fields#500
jamesadevine merged 1 commit into
mainfrom
docs/fix-documentation-drift-9fed47777950a289

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Documentation Freshness Audit

This audit found the following inconsistencies between code and documentation:

Findings

Area Issue File(s)
create-pull-request config 7 configuration fields exist in CreatePrConfig struct but are undocumented docs/safe-outputs.md
MCP container config args: and mounts: fields missing from the primary workflow authoring guide prompts/create-ado-agentic-workflow.md

Details

1. Undocumented create-pull-request config fields (docs/safe-outputs.md)

The CreatePrConfig struct in src/safeoutputs/create_pr.rs defines these fields, but docs/safe-outputs.md didn't document them:

Field Default Description
title-prefix (none) Optional string prepended to all PR titles
if-no-changes "warn" Behavior when patch is empty: "warn", "error", "ignore"
max-files 100 Maximum files allowed in a single PR
protected-files "blocked" Whether manifest/CI files can be modified: "blocked" or "allowed"
excluded-files [] Glob patterns to strip from patch
allowed-labels [] (any) Allowlist of labels the agent can apply
fallback-record-branch true Record branch name in failure response when PR creation fails

2. Missing MCP container fields in workflow authoring prompt (prompts/create-ado-agentic-workflow.md)

The McpOptions struct has args and mounts fields that are documented in README.md and docs/mcp.md but were absent from the MCP container config example in the primary workflow authoring guide. AI agents using this prompt wouldn't know these fields exist when creating workflows that need Docker volume mounts or custom runtime arguments.

Applied Fixes

  • docs/safe-outputs.md: Added documentation for 7 missing create-pull-request configuration options
  • prompts/create-ado-agentic-workflow.md: Added args: and mounts: fields to the MCP container config example

This pull request was created by the automated documentation freshness check.

Generated by Documentation Freshness Check · ● 5.3M ·

…and missing MCP container fields

- docs/safe-outputs.md: add 7 undocumented create-pull-request configuration
  options: title-prefix, if-no-changes, max-files, protected-files,
  excluded-files, allowed-labels, fallback-record-branch
- prompts/create-ado-agentic-workflow.md: add args: and mounts: fields to
  MCP container config example (both are valid, documented in README and
  docs/mcp.md but missing from the primary workflow authoring guide)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine jamesadevine marked this pull request as ready for review May 11, 2026 06:10
@jamesadevine jamesadevine merged commit f9d3550 into main May 11, 2026
@jamesadevine jamesadevine deleted the docs/fix-documentation-drift-9fed47777950a289 branch May 11, 2026 06:10
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.

1 participant