Repository navigation
docs(aws-startup-advisor): fix Stable Image Core/Ultra model IDs - #512
Merged
herosjourney merged 1 commit intoOct 28, 2026
Merged
Conversation
leon1418
reviewed
Oct 27, 2026
leon1418
left a comment
Contributor
There was a problem hiding this comment.
[🤖 AI review 🤖]
Reviewed all 10 changed files at 5b829874d4a8ca63d3848801e54659090058de25, including the vendored copies and unchanged model-ID consumers. No substantive findings. The current AWS model cards list both Core and Ultra as v1:1.
Local validation: 513 plugin tests and 562 agent-advisor tests passed, along with model-ID, lifecycle freshness, pricing coverage, artifact, vendored-parity, repository validation, and diff checks. No live Bedrock invocation was performed; this review does not establish whether the old IDs still resolve.
Approval recommended for the human reviewer. Merge readiness remains subject to the repository checks and required approvals.
leon1418
approved these changes
Oct 27, 2026
github-merge-queue
Bot
removed this pull request from the merge queue due to no response for status checks
Oct 27, 2026
6 of 40 tasks
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.
Problem
The plugin tells users to call Stability AI's Stable Image Core and Stable Image Ultra on Bedrock as
stability.stable-image-core-v1:0andstability.stable-image-ultra-v1:0. The current AWS model cards listv1:1for both.v1:0no longer appears on either card, on the Bedrock model-lifecycle pages, or in the Legacy/EOL tables. A user who copies the plugin's ID into code or a Terraform variable risks an invalid-model error at invoke time, and the generated migration report names an ID AWS no longer publishes.The wrong ID sits in the shared pricing cache and lifecycle registry, so every skill that vendors those files (
gcp-to-aws,azure-to-aws,agent-advisor) repeats it.Fix
Correct the two IDs to
v1:1everywhere the plugin emits them, and cite the source next to the pricing rows.skills/shared/ai/bedrock-pricing-cache.md: both IDs in the Stability image table, plus a one-line note citing the two model cards and the verification date.skills/shared/ai/ai-model-lifecycle.md: both IDs in the Active Replacement column.skills/gcp-to-aws/references/phases/discover/discover-preview.md: the example target ID.scripts/model-id-lint.py: a comment that quoted the old ID.fixtures/migration-report-reference.html: the reference report's Stability row.skills/{agent-advisor,azure-to-aws,gcp-to-aws}/references/vendored/ai/: regenerated withtools/sync-vendored.py, not hand-edited.Source of truth: the Programmatic Access tables on the Stable Image Core 1.0 and Stable Image Ultra 1.0 model cards, read 2026-10-27. Both:
bedrock-runtimemodel ID…-v1:1, in-Regionus-west-2only, no Geo or Global inference profile, lifecycle Active, EOL N/A.What this PR does NOT do
stability.sd3-5-large-v1:0matches its model card and stays.v1:0still resolves at runtime. The docs no longer mention it; nothing here was invoked against Bedrock.Scope
Skills:
shared/aicanonical files and their vendored twins ingcp-to-aws,azure-to-aws,agent-advisor; onegcp-to-awsphase doc. No new skill, no script behavior change.Files: 10 paths (0 added, 10 modified, 0 deleted), +25 / −17.
Base:
mainatd7857c85(includes #500). One commit,5b829874.Not touched: mapping guides, Clarify docs,
bma-runtime.py,model-id-lint.pylogic (comment only), any skill script.Heads-up for in-flight PRs: #483 and
feat/ai-mapping-guide-tierseditbedrock-pricing-cache.md/ai-model-lifecycle.mdand their twins. They should mergemainafter this lands and re-runtools/sync-vendored.pyso the twins stay byte-identical.Validation
Local gates, run from this branch against
mainatd7857c85, all exit 0:python3 plugins/aws-startup-advisor/tools/model-staleness.py --strict: pass.python3 plugins/aws-startup-advisor/scripts/model-id-lint.py: pass.python3 plugins/aws-startup-advisor/tools/sync-vendored.py --check: 59 vendored files byte-identical.mise -C plugins/aws-startup-advisor run ci: markdownlint 0 errors;lint:artifacts85 artifacts / 0 errors;lint:skill-refs1,404 refs / 0 missing; model-id, model-staleness, pricing, and consent-disclosure lints pass; plugin tests 513 passed; telemetry tests 279 passed; Heroku source-review tests 44 passed.python3 tools/validate.py: all validations passed.mise run validate: pass.gitleaksonupstream/main..HEAD: no leaks.These are documentation and fixture changes. Nothing here was executed against a live Bedrock endpoint.
Limitations
stability.stable-image-core-v1:0/-ultra-v1:0still resolve on Bedrock is not stated in AWS docs and was not tested. Users with the old ID in production should check their own invocations.us-west-2only. The pricing cache's Stability table is scoped to US East (N. Virginia) pricing; the price is the same, but the region note on that table is a separate follow-up.Type of Change
Checklist
python3 tools/validate.pyStartup Advisor review checklist
mise -C plugins/aws-startup-advisor run ciand all checks pass.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.