fix(stack): restore push summary and capture git push output

[JulianMaurin]

stack push rendered only log_lines and discarded the planned/upserted data, so the "Stacked pull request plan" preview and per-PR [created] … <url> summary were missing. Port get_log_from_local_change / get_log_from_orphan_change as plain-text formatters and emit the plan, created, and orphan-deleted lines into log_lines in flow order (dry-run included).

push_branches ran git push with inherited stdio, leaking git's remote: ruleset-bypass and branch-creation narration. Capture it and surface stderr only on failure, matching run_git_silent.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

[mergify]

Merge Protections

🟢 All 5 merge protections satisfied — ready to merge.

[Copilot]

Pull request overview

Restores stack push user-facing output parity by reintroducing the plan preview and per-PR outcome/orphan-deletion summaries, and reduces noisy git push narration by capturing its output and only surfacing stderr on failure.

Changes:

• Capture git push output in push_branches and include stderr only when the push fails.
• Add a plan preview (including dry-run) and append per-PR outcome lines after upsert, plus orphan-deletion lines.
• Introduce plain-text change/orphan log formatters with unit tests.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 11 comments.

File	Description
crates/mergify-stack/src/notes_push.rs	Capture git push output instead of inheriting stdio; include stderr on failure only.
crates/mergify-stack/src/commands/push.rs	Emit plan preview and restore post-upsert per-PR summary + orphan-deletion log lines.
crates/mergify-stack/src/changes.rs	Add plain-text log formatters for local/orphan changes and unit tests for expected output.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[JulianMaurin]

Revision history

#	Type	Changes	Reason	Date
1	initial	bb98d3f		2026-06-16 17:37 UTC
2	content	bb98d3f → d67e94f (raw)	Address review: missing html_url now falls back to instead of an empty string (trailing-space/empty-URL).	2026-06-16 17:37 UTC
3	content	d67e94f → 6b9797e (raw)	cargo fmt: wrap the orphan-formatter title binding (CI fmt gate).	2026-06-16 18:24 UTC

[JulianMaurin]

This pull request is part of a Mergify stack:

#	Pull Request	Link
1	fix(stack): restore push summary and capture git push output	#1614	👈
2	fix(stack): strip raw Rich markup from push rebase narration	#1615
3	fix(stack): preserve rebase-conflict recovery guidance and exit code	#1616
4	fix(stack): restore full-stack action-plan previews	#1617
5	fix(stack): restore prefix-match error detail and not-found exit codes	#1618
6	fix(core): surface API error detail and the failing endpoint URL	#1619
7	fix(ci,queue,stack): restore minor output details lost in the port	#1620

[mergify]

Queued — the merge queue status continues in this comment ↓.

[mergify]

Merge Queue Status

• ✅ Entered queue — 2026-06-17 07:20 UTC · Rule: default
• ✅ Checks passed · on draft merge queue: queuing main (50a0fea) and #1614 together #1621
• ✅ Merged — 2026-06-17 07:26 UTC · at 6b9797e7241f3e2181c75d538cab727519f29ea3 · squash

This pull request spent 6 minutes 12 seconds in the queue, including 5 minutes 44 seconds running CI.

Required conditions to merge

• all of [🛡 Merge Protections rule Enforce conventional commit]:
  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:
    • fix(stack): restore push summary and capture git push output #1614
• all of [🛡 Merge Protections rule 👀 Review Requirements]:
  • any of:
    • #approved-reviews-by>=2
      • fix(stack): restore push summary and capture git push output #1614
    • author = dependabot[bot]
      • fix(stack): restore push summary and capture git push output #1614
    • author = mergify-ci-bot
      • fix(stack): restore push summary and capture git push output #1614
    • author = renovate[bot]
      • fix(stack): restore push summary and capture git push output #1614
• all of [🛡 Merge Protections rule 📕 PR description]:
  • body ~= (?ms:.{48,})
    • fix(stack): restore push summary and capture git push output #1614
• all of [🛡 Merge Protections rule 🔎 Reviews]:
  • #changes-requested-reviews-by = 0
    • fix(stack): restore push summary and capture git push output #1614
  • #review-requested = 0
    • fix(stack): restore push summary and capture git push output #1614
  • #review-threads-unresolved = 0
    • fix(stack): restore push summary and capture git push output #1614
• all of [🛡 Merge Protections rule 🤖 Continuous Integration]:
  • all of:
    • check-success=ci-gate