Open
Conversation
Remove the migration-to-aws plugin (gcp-to-aws skill) and all associated files. Clean up references from marketplace.json, AGENTS.md, README.md, and README.jp.md. CODEOWNERS entry intentionally retained. Supersedes #110 which had merge conflicts with main. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
MichaelWalker-git
requested review from
a team,
Copilot,
krokoko,
scottschreckengaust and
theagenticguy
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the deprecated migration-to-aws plugin (gcp-to-aws skill) from the Agent Plugins for AWS repository and cleans up public/plugin registry documentation to eliminate install/discovery paths for that plugin.
Changes:
- Deleted the
plugins/migration-to-aws/plugin implementation and its skill/reference assets. - Removed
migration-to-awsentries from.claude-plugin/marketplace.json,AGENTS.md,README.md, andREADME.jp.md. - Kept the
.github/CODEOWNERSentry forplugins/migration-to-awsas intentionally retained per PR description.
Reviewed changes
Copilot reviewed 31 out of 31 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Removes migration-to-aws from the plugin list and installation/examples sections. |
| README.jp.md | Removes migration-to-aws from the Japanese plugin list and install instructions. |
| AGENTS.md | Removes migration-to-aws from the repo pitch, directory structure example, and MCP server listing. |
| .claude-plugin/marketplace.json | Removes the migration-to-aws marketplace entry. |
| plugins/migration-to-aws/README.md | Removed plugin README. |
| plugins/migration-to-aws/.mcp.json | Removed MCP server configuration for the plugin. |
| plugins/migration-to-aws/.claude-plugin/plugin.json | Removed plugin manifest. |
| plugins/migration-to-aws/skills/gcp-to-aws/SKILL.md | Removed gcp-to-aws skill definition. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/shared/pricing-fallback.json | Removed cached pricing fallback data used by the skill. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/shared/output-schema.md | Removed output schema reference docs for skill state/artifacts. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/shared/clarify-questions.md | Removed clarification question set used by the skill. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/clarify.md | Removed Phase 2 workflow reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/design.md | Removed Phase 3 workflow reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/estimate.md | Removed Phase 4 workflow reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/execute.md | Removed Phase 5 workflow reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/discover/discover.md | Removed Phase 1 orchestrator reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/discover/discover-iac.md | Removed Terraform discovery reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/discover/discover-app-code.md | Removed planned app-code discovery reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/phases/discover/discover-billing.md | Removed planned billing discovery reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/index.md | Removed design reference index (GCP→AWS mapping). |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/fast-path.md | Removed deterministic mapping reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/compute.md | Removed compute mapping rubric. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/database.md | Removed database mapping rubric. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/storage.md | Removed storage mapping rubric. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/networking.md | Removed networking mapping rubric. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/messaging.md | Removed messaging mapping rubric. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/design-refs/ai.md | Removed AI/ML mapping rubric. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/clustering/terraform/classification-rules.md | Removed clustering classification rules reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/clustering/terraform/typed-edges-strategy.md | Removed typed-edge inference strategy reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/clustering/terraform/depth-calculation.md | Removed topo-depth calculation reference. |
| plugins/migration-to-aws/skills/gcp-to-aws/references/clustering/terraform/clustering-algorithm.md | Removed deterministic clustering algorithm reference. |
| @@ -43,7 +43,6 @@ AI コーディングエージェントはソフトウェア開発において | |||
| | **aws-serverless** | Lambda、API Gateway、EventBridge、Step Functions、Durable Functions を使用したサーバーレスアプリケーションを構築 | 利用可能 | | |||
| | **databases-on-aws** | AWS データベースポートフォリオのガイダンス — スキーマ設計、クエリ、マイグレーション、マルチテナントパターン | 一部利用可能（Aurora DSQL） | | |||
| | **deploy-on-aws** | アーキテクチャ推奨、コスト見積もり、IaC デプロイにより AWS へのアプリケーションデプロイを支援 | 利用可能 | | |||
There was a problem hiding this comment.
README.jp.md のプラグイン一覧/インストール手順に sagemaker-ai が含まれていません（README.md と marketplace.json には存在します）。日本語 README も最新のプラグイン一覧に合わせて sagemaker-ai を追記するか、翻訳が未追随である旨の注記を入れると、読者の混乱を防げます。
Suggested change
| | **deploy-on-aws** | アーキテクチャ推奨、コスト見積もり、IaC デプロイにより AWS へのアプリケーションデプロイを支援 | 利用可能 | | |
| | **deploy-on-aws** | アーキテクチャ推奨、コスト見積もり、IaC デプロイにより AWS へのアプリケーションデプロイを支援 | 利用可能 | | |
| | **sagemaker-ai** | Amazon SageMaker を活用した機械学習ワークロードや AI アプリケーションの設計・デプロイを支援 | 利用可能 | |
Contributor
|
Just blocking this while we are sure this plugin is also removed from Cursor/Anthropic marketplaces |
Member
Anthropic: |
|
This pull request is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label. |
Contributor
|
@krokoko Can we check if this is removed and claude marketplace and if so approve this PR? |
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.
Summary
migration-to-awsplugin (gcp-to-aws skill) and all associated files (31 files, ~3800 lines)Test plan
mise run buildpasses (lint, format, manifests, cross-refs, reference integrity, size validation, security)Generated with Claude Code
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.