Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #98 +/- ##
==========================================
+ Coverage 88.40% 88.83% +0.43%
==========================================
Files 21 36 +15
Lines 1069 1559 +490
==========================================
+ Hits 945 1385 +440
- Misses 61 119 +58
+ Partials 63 55 -8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR adds a new cage scan command for identifying ECR vulnerabilities in ECS service container images. The feature retrieves ECR image scan results and displays vulnerability counts by severity in a formatted table.
Changes:
- Adds new
scancommand that accepts either a service.json directory or explicit cluster/service flags - Implements ECR and ECS integration to retrieve container images and their vulnerability scan results
- Refactors application structure by extracting dependency injection and AWS client provisioning into a new
cageapppackage - Updates mock files to latest mockgen format (auto-generated changes)
Reviewed changes
Copilot reviewed 33 out of 34 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| cli/cage/commands/scan.go | New scan command implementation with argument parsing and scanner orchestration |
| cli/cage/scan/*.go | Core scanning logic: ECS image info retrieval, ECR manifest parsing, and scan findings formatting |
| cli/cage/cageapp/provider.go | Extracted dependency injection providers for reuse across commands |
| cli/cage/cageapp/flags.go | Moved flag definitions and App struct to new package |
| cli/cage/commands/*.go | Refactored existing commands to use new App struct and cageapp package |
| cli/cage/main.go | Simplified main by extracting providers to cageapp package |
| awsiface/iface.go | Added EcrClient interface for ECR operations |
| key/keys.go | Added EcrCli key for dependency injection |
| env/env.go | Removed CI field (moved to App struct) |
| cli/cage/prompt/prompt.go | Removed CI flag check (moved to command layer) |
| go.mod/go.sum | Updated AWS SDK dependencies and added ECR service |
| Makefile | Changed mockgen invocation and upgrade mock path |
| mocks/* | Auto-generated mock updates from newer mockgen version |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 43 out of 44 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 44 out of 45 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 45 out of 46 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 60 out of 61 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 60 out of 61 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
audit, for finding ECR vulnerabilities based on ECR Image Scan results.Usage
service.jsondir forauditcommand.--clusterand--serviceoptions for arbitrary your service.