📝 docs(test-agent): ACP test-agent specification (acp-stack 01)#130
Merged
taras merged 1 commit intoJul 24, 2026
Merged
Conversation
Deterministic ACP test agent that stands in for a real coding agent for black-box testing of the agent integration. Contract for the test-agent implementation stack (engine, controller, worker, declarative vocabulary). Base of the test-agent stack; controller-launched worker only.
PR #130: 📝 docs(test-agent): ACP test-agent specification (acp-stack 01)1 files, +335 / -0 Scope✅ PR scope looks good. Structural✅ No structural bloat detected. Slop✅ Slop indicators look low. Static Analysis✅ Oxlint found no issues. CorrectnessNo extraneous code patterns detected. |
taras
marked this pull request as ready for review
July 24, 2026 17:02
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The test-agent stack begins with its contract. Landing the specification
first gives every implementation PR above it a fixed target to review
against, and lets the design be read on its own before any code exists.
What changes
Adds
specs/test-agent-spec.md: the deterministic ACP test agent thatstands in for a real coding agent so Executable.md's agent integration can
be black-box tested against scripted, repeatable responses.
Review guide
Read the spec top to bottom — it is organized in comprehension order:
purpose and the boundary diagram, then
<TestAgent>, behavior documents,<WhenPrompt>templates, scenario isolation, the controller/workerarchitecture, replay, the deterministic runtime, and failure/teardown.
What must stay true
xmd test-agentis a controller-launched worker only — it has no standalonebehavior-document mode. Behavior documents run under a deterministic profile
(no process/network, virtual filesystem, inline-only eval).
How to verify it
Both pass. This PR is documentation only.
Scope
Specification only — no implementation, no release wiring. The engine
(
acp-stack-02), controller (03), and worker (04) implement it inorder; the declarative
<TestAgent>vocabulary (13) completes it.Risks and limitations
None — text only.
Based on
agent/acp-stack-00-process-and-format(PR #129), notmain.Next:
agent/acp-stack-02-test-agent-behavior.