Skip to content

docs: add missing container_runtime.rs to AGENTS.md architecture tree - #2198

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
docs/agents-md-container-runtime-module-67e8307e1a5d4e61
Closed

github-actions[bot] wants to merge 1 commit into
mainfrom
docs/agents-md-container-runtime-module-67e8307e1a5d4e61

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Agent Documentation Update

Findings

Area Issue File(s)
accuracy src/compile/extensions/container_runtime.rs (typed Docker runtime configuration for MCPG stdio servers: mounts, network, user, add-host, tmpfs) exists in the codebase and is used by mod.rs (ContainerRuntimeConfig, Mount, Network, etc.), but was missing from the src/compile/extensions/ subtree in AGENTS.md's architecture map. AGENTS.md

Applied Fixes

  • Added a container_runtime.rs entry under src/compile/extensions/ in the AGENTS.md architecture tree, describing its role.

Notes

Verified by diffing 'find src -name .rs' against every file/module mentioned in AGENTS.md's architecture tree. All other directories (compile/ir/, compile/imports/, runtimes/, tools/, safe_outputs/, codemods/, top-level src/.rs) matched exactly. docs/cli.md, docs/front-matter.md, and docs/safe-outputs.md were spot-checked against src/main.rs's Commands enum, FrontMatter struct in src/compile/types.rs, and src/safe_outputs/.rs respectively, and found accurate. prompts/.md were checked against src/engine.rs (model defaults) and src/compile/types.rs (PrMode::Synthetic/Policy) and found accurate. Searched open PRs for overlapping documentation fixes; none found.


Created by the agent-documentation maintainer workflow.

Generated by Agent Documentation Maintainer · auto · 103.6 AIC · ⌖ 9.86 AIC · ⊞ 11.9K · ◷

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@jamesadevine

Copy link
Copy Markdown
Collaborator

The same container_runtime.rs architecture entry was already merged through #2189; #2238 records this duplicate disposition. Closing as superseded.

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