Skip to content

Add lifecycle details to snapshot findings#79

Open
yti93 wants to merge 1 commit into
block:mainfrom
yti93:codex/snapshot-lifecycle-details
Open

Add lifecycle details to snapshot findings#79
yti93 wants to merge 1 commit into
block:mainfrom
yti93:codex/snapshot-lifecycle-details

Conversation

@yti93
Copy link
Copy Markdown

@yti93 yti93 commented May 12, 2026

🤖 Draft PR opened by Youssef's AI agent.

Summary

  • Add required finding.lifecycle_details metadata to snapshot findings.
  • Populate lifecycle details from the existing VersionLifecycle data during drift detection.
  • Document the snapshot wire shape and add detection/snapshot regression tests.

Test plan

  • env GOCACHE=/private/tmp/version-guard-go-build GOMODCACHE=/private/tmp/version-guard-go-mod go test ./...

Signed-off-by: Youssef <53960652+yti93@users.noreply.github.com>
@yti93 yti93 force-pushed the codex/snapshot-lifecycle-details branch from ce76e42 to 54ec420 Compare May 12, 2026 18:06
@yti93 yti93 marked this pull request as ready for review May 12, 2026 18:16
@yti93 yti93 requested a review from a team as a code owner May 12, 2026 18:16
@yti93
Copy link
Copy Markdown
Author

yti93 commented May 12, 2026

@bakayolo Docker Compose test completed and verified:

  • Built and started the collector stack with Docker Compose.
  • Used live Wiz credentials and the rds-mysql report ID.
  • Triggered an rds-mysql scan through POST /scan.
  • Collector fetched 36 live RDS MySQL rows from Wiz.
  • Collector wrote the snapshot to MinIO.
  • Snapshot had 36 rds-mysql findings.
  • All 36 had lifecycle_details.
  • No findings had the removed Recommendation field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant