Skip to content

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

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

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

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Agent Documentation Update

Findings

Area Issue File(s)
accuracy src/compile/extensions/container_runtime.rs (defines Mount, MountMode, McpgServerConfig used for typed Docker runtime config on MCPG stdio servers) is missing from the extensions/ subtree in the AGENTS.md architecture map AGENTS.md

Applied Fixes

  • Added a container_runtime.rs entry under src/compile/extensions/ in the architecture tree with a short description of its role

Notes

Verified by diffing every *.rs file under src/ (excluding the intentionally-collapsed ir/tasks/ directory) against files mentioned in AGENTS.md; container_runtime.rs was the only file present in the codebase but absent from the doc. Confirmed via mod.rs (mod container_runtime; pub use container_runtime::{...}) that it is an active, in-tree module, not dead code. No other drift found in the CLI reference, front-matter fields, safe-outputs tool matrix, or workflow-authoring prompts during this pass.


Created by the agent-documentation maintainer workflow.

Generated by Agent Documentation Maintainer · auto · 92.7 AIC · ⌖ 17.7 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