Repository navigation
feat(ec2): add Amazon EC2 integration - #9304
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
There was a problem hiding this comment.
All reported issues were addressed across 51 files
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
[High impact] The PR appears safe to merge; no blocking issues were found. SummaryAdds an Amazon EC2 integration with 34 operations for instances, networking, Elastic IPs, EBS, and tags.
|
… rule, and EBS delete operations
- Add describe_network_acls, describe_vpc_endpoints, describe_vpc_peering_connections,
describe_images, describe_key_pairs, authorize/revoke_security_group_ingress,
delete_volume, and delete_snapshot
- Regenerate block successor names (fixes the lint job's repo audit)
- Merge the two tag tables into one tags field so fixed tags work in the Agent editor
- Keep a __proto__ tag key by building records with Object.fromEntries
- Pass [{Key, Value}] tag arrays through the block instead of dropping them
- Use a filter example each describe operation actually accepts
- Report revoked: false when a revoke matched no rule; mark the ingress target clauses core
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 61 files
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
|
@greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 61 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
Summary
@aws-sdk/client-ec2as in-process tool operations (lib/internal/ec2), authenticated with an AWS access key like the other AWS blocksnextToken, and skip pagination when explicit IDs are given (EC2 rejectsMaxResultsalongside IDs)AllocateAddressandCreateSnapshotrun with SDK retries off, since neither takes a client token; snapshots default toOwnerIds: selffor list calls so public snapshots aren't returned[{Key, Value}], or the block's tag table (including on the Agent tool path)Type of Change
Testing
check-canvas-sentences --block=ec2(25/25),check-block-registry.ts origin/staging,check:tool-request-boundary,check:tool-metadata,check:docs,check:integration-catalog,check:deployment-config,check:bare-icons,check:source-conventions,check:utils,check:comment-hygiene,check:unused-exports,check:explicit-any, and related auditstools/request-transport.test.ts,lib/internal/tool-operations/registry.server.test.ts,blocks/blocks.test.tscheck:audits, and test suites run in CIChecklist
test-auditauthoring gate)🤖 Generated with Claude Code