Skip to content

test(prodstyle): MCP/A2A protocol ingress coverage + CI gate - #270

Merged
IanFrelinger merged 1 commit into
masterfrom
feat/mcp-a2a-prodstyle
Aug 13, 2026
Merged

test(prodstyle): MCP/A2A protocol ingress coverage + CI gate#270
IanFrelinger merged 1 commit into
masterfrom
feat/mcp-a2a-prodstyle

Conversation

@IanFrelinger

Copy link
Copy Markdown
Owner

Summary

Final PR of the MCP/A2A integration series (src chain #263/#268/#265/#266, application wiring #267 landed via #269): ProdStyle coverage for the protocol surfaces on the real Nexo.API pipeline, plus a dedicated CI gate.

Changes

  • src/Nexo.Tests.Infrastructure/Tests/VirtualProduction/McpA2AProtocolIngressProdStyleTests.cs (net9, real Program + full middleware chain): dark-by-default surfaces; all-verbs credential enforcement incl. the MCP GET SSE channel; initialize-with-key advertises the server; agent card credentialed by default with anonymity as explicit opt-in; a real A2A SDK clientmessage/send round trip completing against an echo agent transport; AirGapped profile refuses protocol enablement at boot.
  • .github/workflows/mcp-a2a-gate.yml (modeled on grpc-transport-gate.yml): path-filtered adapter suites (MCP server/client, A2A transport/server) + the ProdStyle protocol filter on net9.

Compose env-var rows for the portal/agent-server stacks remain deferred until chore/relocate-compose-files lands (avoids rename conflicts).

Testing

  • 6/6 ProdStyle tests passing locally; all four adapter suites green in prior PR runs (39+29+36+13)

Testing strategy (blast radius)

Test-only + CI workflow; the ProdStyle delta satisfies the strategy gate for the application wiring landed in #269.

  • make test-prod-style equivalent — the new suite itself

Release (only when this PR ships a versioned NuGet/GHCR release)

  • Not a versioned release — skip

🤖 Generated with Claude Code

@cursor

cursorBot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

- src/Nexo.Tests.Infrastructure Tests/VirtualProduction/
McpA2AProtocolIngressProdStyleTests (net9, real Program pipeline): dark-by-
default surfaces; all-verbs credential enforcement incl. the MCP GET SSE
channel; initialize with key advertises the server; agent-card credentialed
by default with anonymity as explicit opt-in; a real A2A SDK client
message/send round trip to an echo agent transport (task Completed with
artifact); AirGapped profile refuses protocol enablement at boot.
- .github/workflows/mcp-a2a-gate.yml (modeled on grpc-transport-gate): path-
filtered adapter suites + the ProdStyle protocol filter on net9 (dotnet 8+9
installed; TestHost 9 required by System.Text.Json 10).
Compose env-var rows for the portal/agent-server stacks are deferred until the
compose relocation (chore/relocate-compose-files) lands to avoid rename
conflicts; tracked in the integration summary.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@IanFrelinger
IanFrelingerforce-pushed the feat/mcp-a2a-prodstyle branch from a8fbb19 to c61a335CompareAugust 13, 2026 21:19
@IanFrelinger
IanFrelinger merged commit a0db4f6 into masterAug 13, 2026
15 checks passed
@IanFrelinger
IanFrelinger deleted the feat/mcp-a2a-prodstyle branch August 13, 2026 21:34
IanFrelinger pushed a commit that referenced this pull request Aug 17, 2026
…rograms historical; document spikes/
docs/DocsIndex.md: Start Here #1 is the tester quickstart, README #2,
GettingStarted #3; IntegratorGuide and consumer-template/CONSUMING.md are
listed for the first time; RELEASE.md, the setup-matrix scripts and the
bootstrap helpers move out of the numbered list into an "escape hatches"
paragraph (RELEASE stays under Operator); a new "Trust loop / certification
(experimental, hold-mode)" group makes docs/trust-loop/*, the evidence
ledger, governed-pipeline, SELF-EXTEND-AUDIT, the sample objective and
spikes/ reachable; "Planning & Roadmap" becomes "Planning history
(historical as of 2026-08-16)" because it described a finished program.
Stale status docs are date-stamped rather than deleted:
- ProtocolIntegration-MCP-A2A.md said the Nexo.API wiring was "planned"; it
landed in #269/#270 (Program.cs registers all four adapters and maps
/api/mcp and /api/a2a/{agentId}; IngressCatalog rows McpServer/A2AAgents).
- ci-pre-existing-failures.md said the Full Platform Readiness Gate was RED
as of 06-21; it went green with #317-#320/#335. Kept because
docs/planning/* still cite it.
- GapAnalysis.md and NorthStarGapAnalysis.md predate the trust loop; both get
a historical banner naming their still-open rows and pointing at the
ledger, ProjectTiers and CHANGELOG.
GettingStarted.md #5 contradicted DocsIndex on "installer" scripts (the
scripts/install/* helpers exist but bootstrap the container lane); it now
says so and points first-run readers at the quickstart. The pointer to a
README section "First Successful Pipeline Run" (which does not exist) now
names the sections that do. IntegratorGuide.md recommended `dotnet build
Nexo.sln` (the docs guard forbids it in README); it now builds
Nexo.Kernel.sln and the CLI project, with hello-brick as the smallest test
example, and the compatibility matrix names the solutions that actually
apply.
spikes/README.md: the 27 tracked files under spikes/ were cited as PASS
evidence by the ledger but undocumented. It says what each spike is, every
flag of the flight runner, which ledger rows cite each spike, and that none
of it is a supported entry point.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign up for freeto 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.

2 participants

@IanFrelinger@PlzTouchGrass