Fix agentic-atx-platform deployment + stale transform IDs; default to Claude Sonnet 4.5#46
Open
syepesaws wants to merge 3 commits into
Open
Conversation
…l prebuilt image useBaseInfra was hardcoded to always-true (=== 'true' || true), forcing hybrid mode and preventing the documented full-mode deploy. Normalize to accept string or boolean context. Add -c publicEcrImage to skip the slow local container build and use a prebuilt image.
…dentifiers Execute form and sample CSVs referenced 'AWS/early-access-comprehensive-codebase-analysis' and 'AWS/early-access-jfr-performance-optimization', which are not in the ATX registry and fail at exec. Use the current names (AWS/comprehensive-codebase-analysis, AWS/java-performance-optimization) and add the missing vue/angular/log4j transforms to match the registry and the rest of the repo.
…net 4.5 Update SAM template default, config template, code fallbacks and docs to us.anthropic.claude-sonnet-4-5-20250929-v1:0. deploy.sh now passes BedrockModelId only when explicitly set so the template default applies otherwise.
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
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.
Description
Fixes that unblock deploying the
agentic-atx-platformsample and align it with the current AWS Transform registry.1. CDK: honor
useBaseInfracontext + optional prebuilt image (cdk/bin/cdk.ts)useBaseInfrawas hardcoded as=== 'true' || true, which always evaluates totrue. This forced hybrid mode and meant the full-mode stacks (AtxContainerStack,AtxInfrastructureStack) documented in README Option A/B were never instantiated. Normalized to accept either a string'true'or a JSON booleantrue.-c publicEcrImage=<uri>context flag. When set, the local Docker build is skipped and the Batch job uses the prebuilt image (e.g.public.ecr.aws/b7y6j9m3/aws-transform-custom:latest). Default behavior is unchanged.2. Fix stale AWS-managed transformation identifiers (
ui/...)AWS/early-access-comprehensive-codebase-analysisandAWS/early-access-jfr-performance-optimization, which are not in the ATX registry and fail atatx custom def exec("Transformation not found").AWS/comprehensive-codebase-analysis,AWS/java-performance-optimization) and added the missingvue.js-version-upgrade,angular-version-upgrade, andearly-access-log4j-to-slf4j-migration, matchingfindtransform.py/TransformationList.jsx.3. Default orchestrator model to Claude Sonnet 4.5
us.anthropic.claude-sonnet-4-5-20250929-v1:0across the SAM template default,config.env.template, code fallbacks, and docs.sam/deploy.shnow passesBedrockModelIdonly when explicitly configured, so the SAM template default applies otherwise (no empty override).Testing
us-east-1: infrastructure, AgentCore runtime (READY), HTTP API, and CloudFront UI./orchestrateresponds, and a transformation submits to AWS Batch with a valid registry identifier.tsccompiles cleanly;bash -npasses ondeploy.sh.Notes / follow-ups
package-lock.jsonfiles are intentionally not included.Licensing
By submitting this PR, I confirm my contribution is made under the terms of the project's MIT-0 license.