feat(appconfig): add AWS AppConfig integration for config versions and deployments#4926
feat(appconfig): add AWS AppConfig integration for config versions and deployments#4926TheodoreSpeaks wants to merge 3 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Eleven control-plane capabilities are wired end-to-end: create/get/list hosted versions, start/get/stop/list deployments, and list applications, environments, configuration profiles, and deployment strategies. Each operation follows the existing AWS tool stack—Zod contract, authenticated internal API route, Docs and discovery are updated with Reviewed by Cursor Bugbot for commit 1503711. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR adds AWS AppConfig integration following the existing AWS SDK-behind-internal-routes pattern (consistent with DynamoDB, SES, SQS). It contributes 11 control-plane operations across contracts, route handlers, tool definitions, a block config, and documentation.
Confidence Score: 5/5Safe to merge — all 11 operations correctly follow the established AWS internal-route pattern with proper auth, Zod validation, and SDK client cleanup. The implementation is structurally correct across all layers: contracts, routes, utils, tool definitions, and block config. No operations are missing required parameters, no credentials are leaked, and SDK client lifecycle is managed with No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant UI as Sim UI (Block)
participant Tool as Tool Layer (tools/appconfig)
participant Route as API Route (/api/tools/appconfig/*)
participant Utils as Utils (utils.ts)
participant AWS as AWS AppConfig SDK
UI->>Tool: params (region, accessKeyId, secretKey, op-specific fields)
Tool->>Route: "POST /api/tools/appconfig/{operation}"
Route->>Route: checkInternalAuth()
Route->>Route: parseToolRequest() [Zod validation + region check]
Route->>Utils: createAppConfigClient(config)
Utils-->>Route: AppConfigClient
Route->>Utils: operation(client, params)
Utils->>AWS: SDK Command (e.g. StartDeploymentCommand)
AWS-->>Utils: Response
Utils-->>Route: Mapped result object
Route-->>Tool: NextResponse.json(result)
Note over Route: client.destroy() in finally
Tool-->>UI: "transformResponse → { success, output }"
Reviews (3): Last reviewed commit: "Merge remote-tracking branch 'origin/sta..." | Re-trigger Greptile |
|
@greptile review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ff6926c. Configure here.
# Conflicts: # apps/sim/package.json # bun.lock
|
@greptile review |
|
Closing in favor of #4928 (waleedlatif1), which is a superset of this integration — it covers the same control-plane operations plus resource creation (applications/environments/profiles) and the live-config data plane. I ported this PR's one unique operation ( |

Summary
AppConfigClientutil,region+ access-key authappconfigblock with op dropdown, conditional fields, typed outputs, official AWS AppConfig icon, andAppConfigBlockMeta(7 templates); registered in tool + block registries; docs generatedType of Change
Testing
Tested manually.
bun run type-check, fullbun run lint, andbun run check:api-validation:strictall pass.Checklist