Skip to content

chore(plugin): update service recommendation#123

Open
krokoko wants to merge 3 commits intoawslabs:mainfrom
krokoko:bgagent/01KNPV31Y8EHW8EJQWN1EGCVEF/in-the-deploy-to-aws-plugin-update-aws-a
Open

chore(plugin): update service recommendation#123
krokoko wants to merge 3 commits intoawslabs:mainfrom
krokoko:bgagent/01KNPV31Y8EHW8EJQWN1EGCVEF/in-the-deploy-to-aws-plugin-update-aws-a

Conversation

@krokoko
Copy link
Copy Markdown
Contributor

@krokoko krokoko commented Apr 8, 2026

Summary

  • AWS App Runner has entered maintenance mode — the plugin must not recommend it for new workloads.
  • Replaced every App Runner reference in the deploy-on-aws plugin with ECS Express Mode.
  • Added an explicit maintenance-mode warning in defaults.md and a hard principle in SKILL.md so the agent never falls back to App Runner based on general training data.

Changes

File What changed
skills/deploy/references/defaults.md Add a ⚠️ App Runner maintenance-mode warning block
skills/deploy/SKILL.md Added principle: "Never recommend AWS App Runner — use ECS Express mode instead"
skills/aws-architecture-diagram/references/example-complex-platform.drawio Swapped mxgraph.aws4.app_runner icon and label → mxgraph.aws4.ecs / "ECS Fargate"; updated legend text
skills/aws-architecture-diagram/references/aws4-shapes-services.md Annotated app_runner shape as (maintenance mode — use ecs/fargate instead)

Build & test results

mise run build  →  PASS
  lint:md           ✅  0 errors
  lint:manifests    ✅  all JSON schemas valid
  lint:cross-refs   ✅  0 errors / 0 warnings
  fmt:check         ✅  no formatting issues
  validate:refs     ✅  146/153 reachable (7 pre-existing orphans, unchanged)
  validate:size     ✅  22 skills, 0 over limit
  security:bandit   ✅  no issues
  security:semgrep  ✅  no issues
  security:gitleaks ✅  no leaks
  security:checkov  ✅  71 passed, 0 failed
  security:grype    ✅  no vulnerabilities

Decisions made

  • fix_icon_colors.py and aws4-shapes.json left unchangedapp_runner is still a valid AWS4 draw.io shape and may appear in existing customer diagrams. The python script and shape registry are purely declarative; leaving them avoids breaking any diagram that happens to already use the shape. The aws4-shapes-services.md annotation is sufficient to steer the agent away from generating new App Runner nodes.
  • ecs icon used in the diagram (not fargate) — the ECS service-level icon is in the Compute category with the same orange palette, making it a visually consistent drop-in replacement while also conveying the orchestration layer.

Agent notes

What went well:

  • App Runner references were well-isolated: one example diagram, one shape reference file, and the two skill instruction files. No logic code needed changing.
  • The mise run build pipeline is comprehensive and fast — a single command validates markdown, schemas, cross-refs, formatting, and security in one shot.

Difficulties:

  • aws4-shapes-services.md is a single long line per category, so the "maintenance mode" annotation had to be inserted inline rather than on a new line.
  • "ECS Express mode" is not a named AWS product; interpreted as "ECS Fargate in a simple / express configuration" — the most natural App Runner replacement.

Patterns discovered:

  • CLAUDE.md / AGENTS.md are symlinked — always edit AGENTS.md.
  • Orphaned .drawio files in references/ are expected and pre-existing (7 files); validate:refs reports them as warnings, not errors.
  • Commit messages follow conventional-commit format type(scope): description.

Suggestions for future tasks:

  • Consider adding an explicit "deprecated services" section to defaults.md as AWS services transition to maintenance mode (App Runner won't be the last).
  • The fix_icon_colors.py shape-colour map could be auto-generated from aws4-shapes.json to reduce duplication.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

bgagent and others added 3 commits April 8, 2026 15:31
… mode)

AWS App Runner has entered maintenance mode. Update the deploy-on-aws plugin
to redirect all App Runner recommendations to ECS Fargate + ALB, which
provides the same simple containerised deployment experience with full AWS
support and a clear long-term path.

- defaults.md: add App Runner maintenance-mode warning block and a new
  "Simple containerised app" row defaulting to ECS Fargate + ALB
- SKILL.md: add explicit principle never to recommend App Runner
- example-complex-platform.drawio: swap App Runner icon/label → ECS Fargate
- aws4-shapes-services.md: annotate app_runner shape as maintenance mode

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Task-Id: 01KNPV31Y8EHW8EJQWN1EGCVEF
Prompt-Version: 1c9c10e027a2
Updated the recommendation for App Runner to ECS Express Mode due to its maintenance status.

Signed-off-by: Alain Krok <[email protected]>
@krokoko krokoko requested review from a team, scottschreckengaust and theagenticguy April 8, 2026 16:05
@krokoko krokoko requested review from a team as code owners April 8, 2026 16:05
Copy link
Copy Markdown
Contributor

@scoropeza scoropeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@krokoko krokoko enabled auto-merge April 9, 2026 01:19
Sign up for free to 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