Skip to content

feat(change): PR preview labels for touched vs still-drifted apps - #71

Merged
YoungDan merged 2 commits into
mainfrom
feat/pr-preview-app-labeling
Sep 3, 2026
Merged

feat(change): PR preview labels for touched vs still-drifted apps#71
YoungDan merged 2 commits into
mainfrom
feat/pr-preview-app-labeling

Conversation

@SaikrishnaGundeti

Copy link
Copy Markdown
Member

Summary

  • Add Path A2 PR preview labeling (clarity only): CI outputs pr_touched_apps, also_still_drifted_apps, pr_preview_summary, plus GitHub step-summary section
  • Labels compare dry-run plan apps against changed paths (flag / DESLICER_CHANGED_PATHS / GitHub PR git range) and env YAML source_path mapping
  • Does not filter which apps the plan packs or executes — full desired-vs-observed reconcile unchanged
  • Docs: ci-outputs.md + multi-env matrix / lifecycle gate guidance in environments.md (notes Observer idempotent-index blocker for multi-group same SHA)

Test plan

  • cargo fmt --check
  • cargo clippy --lib -- -D warnings
  • cargo test --lib pr_preview
  • CI green on this PR

Made with Cursor

Emit pr_touched_apps / also_still_drifted_apps CI outputs and step-summary
sections for Path A2 plan clarity. Full reconcile execute semantics are
unchanged; labels use changed paths + env YAML mapping + dry-run app names.

Co-authored-by: Cursor <cursoragent@cursor.com>
@SaikrishnaGundeti
SaikrishnaGundeti requested a review from a team as a code owner September 3, 2026 11:32
Co-authored-by: Cursor <cursoragent@cursor.com>
@YoungDan
YoungDan merged commit 213ad8d into main Sep 3, 2026
11 checks passed
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.

2 participants