feat: aws-transform plugin 1.7.0 — continuous-modernization actionable errors - #254
Conversation
There was a problem hiding this comment.
Review: aws-transform 1.7.0 — continuous-modernization actionable errors
Decision: Comment (non-blocking).
The version bump (1.6.0 → 1.7.0) is complete and consistent across .claude-plugin/plugin.json, .codex-plugin/plugin.json, and the marketplace entry; the codex marketplace carries no version field for this plugin, so nothing is missed. All automated gates pass (dprint, markdownlint-cli2, validate-cross-refs, and ajv schema validation), and every cited atx ct command and cross-reference link resolves. The consolidated troubleshooting reference is a genuine improvement.
Three content-correctness issues are worth fixing (inline). They don't break the build or the plugin, and a couple are patterns synced from the internal source — hence Comment rather than Request changes — but each can misdirect a user or agent:
reposcope advice is GitHub-only but applied to GitLab/Bitbucket (401 row). Verified againstcontinuous-modernization-source.md:33-34(GitLab needsapi; Bitbucket needsread:repository:bitbucketet al.).atx ct setup security-agent(bare) is the admin CFN-provisioning command, not a read-only access check — the check is--status. Verified againstcontinuous-modernization-security-agent.md(line 25: requiresiam:CreateRole; line 27: "The agent MUST NOT execute these commands using agentic tools") and againstcontinuous-modernization-analysis.md:211, which already uses--statusfor the same intent.
Minor (not inlined): the connection-error / AccessDenied / empty-results block is duplicated near-verbatim across analysis/findings/guide/remediation/status in addition to the new canonical troubleshooting reference — a future correction to any item above must be made in ~6 places. Consider having callers link to troubleshooting.md rather than re-inline.
…e errors Sync continuous-modernization skill updates from ATXKiroPower (skill-1.7.0): - Actionable error guidance on failure paths (no silent empty results) - Bump aws-transform plugin version 1.6.0 -> 1.7.0 (plugin.json, codex plugin.json, marketplace entry)
8b7291b to
4bff7f8
Compare
scottschreckengaust
left a comment
scottschreckengaust
left a comment
There was a problem hiding this comment.
Approve — clean documentation sync (aws-transform → skill-1.7.0) with matching version bumps.
Verified:
- Version bump consistent across all three manifests (
.claude-plugin/marketplace.json,plugins/aws-transform/.claude-plugin/plugin.json,plugins/aws-transform/.codex-plugin/plugin.json), all now1.7.0. The Codex marketplace (.agents/plugins/marketplace.json) carries no per-plugin version field, so nothing was missed there. - The prior
ada credentials updatereference has been removed; noadareferences remain anywhere in the plugin. - All 16 relative markdown links in the changed files resolve to existing reference files, and the
SKILL.mdanchor link to#atx-ct-reports-unknown-command-ctstill resolves (the heading is preserved). - Every
atx ct …command cited in the new error/troubleshooting content matches the canonical commands documented elsewhere in the skill. fmt:check,lint:md,lint:manifests, andlint:cross-refsall pass (0 errors); the only warnings are pre-existing and unrelated to this PR.
One optional, non-blocking nit inline.
3 participants
Syncs the
aws-transformcontinuous-modernization skill updates from the internal source (release skill-1.7.0).Changes
atx ctcommands no longer surface as silent empty results.aws-transformplugin version 1.6.0 → 1.7.0 (.claude-plugin/plugin.json,.codex-plugin/plugin.json, and the marketplace entry).6
continuous-modernization-*.mdreference files updated; no manifest/schema changes beyond the version bump.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.