From 222724638c58daf32ebe876994e66b9ff6496047 Mon Sep 17 00:00:00 2001 From: Phil Merrell Date: Tue, 23 Jun 2026 16:32:25 -0600 Subject: [PATCH] docs(infra): correct stale SSM comment on mcp-sandbox origin wiring The McpSandboxDistributionConstruct doc comment claimed it publishes the proxy origin to SSM at `/{prefix}/mcp-sandbox/origin`. That was true of the pre-#396 standalone McpSandboxStack, but the single-stack consolidation dropped the SSM publication: the origin is now exposed as `proxyOrigin` and threaded through PlatformComputeRefs straight into inference-api's `AGENTCORE_MCP_APPS_SANDBOX_ORIGIN` env var. The stale comment misleads anyone debugging the sandbox (a missing SSM param looks like a broken deploy when it is expected). Update the comment to match the code. Co-Authored-By: Claude Opus 4.8 --- .../mcp-sandbox/mcp-sandbox-distribution-construct.ts | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/infrastructure/lib/constructs/mcp-sandbox/mcp-sandbox-distribution-construct.ts b/infrastructure/lib/constructs/mcp-sandbox/mcp-sandbox-distribution-construct.ts index 6108847e0..27ef5315d 100644 --- a/infrastructure/lib/constructs/mcp-sandbox/mcp-sandbox-distribution-construct.ts +++ b/infrastructure/lib/constructs/mcp-sandbox/mcp-sandbox-distribution-construct.ts @@ -111,9 +111,11 @@ export interface McpSandboxDistributionConstructProps { * lets the construct still synthesize on the CloudFront default domain * for unit/synth tests and domain-less local stacks. * - * SSM publication: `/{prefix}/mcp-sandbox/origin` → - * `https://mcp-sandbox.{domainName}` (or the CloudFront default domain - * fallback when no custom domain is configured). + * Origin exposure: the resolved origin (`https://mcp-sandbox.{domainName}`, + * or the CloudFront default domain when no custom domain is configured) is + * surfaced as `proxyOrigin` and threaded through `PlatformComputeRefs` + * directly into inference-api's `AGENTCORE_MCP_APPS_SANDBOX_ORIGIN` env var. + * It is no longer published to SSM (pre-#396 the standalone stack did). */ export class McpSandboxDistributionConstruct extends Construct { public readonly distribution: cloudfront.Distribution;