From b7dc1864d399b540de35d1e7d147992d01a9bc33 Mon Sep 17 00:00:00 2001 From: Mehdi ABAAKOUK Date: Tue, 24 Feb 2026 13:23:14 +0100 Subject: [PATCH] fix(ci): add warnings when MQ pull request body or metadata is missing Log warnings instead of silently returning None when the merge queue pull request has no body or no Mergify metadata in fenced blocks. Co-Authored-By: Claude Opus 4.6 Change-Id: I449dc4f833d56cc45b53ff16e6f3c035ce924775 --- mergify_cli/ci/queue/metadata.py | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/mergify_cli/ci/queue/metadata.py b/mergify_cli/ci/queue/metadata.py index aab0aae2..bec9650b 100644 --- a/mergify_cli/ci/queue/metadata.py +++ b/mergify_cli/ci/queue/metadata.py @@ -2,6 +2,7 @@ import typing +import click import yaml from mergify_cli import utils @@ -54,8 +55,20 @@ def extract_from_event(ev: dict[str, typing.Any]) -> MergeQueueMetadata | None: return None if not title.startswith("merge queue: "): return None - body = pr.get("body") or "" - return _yaml_docs_from_fenced_blocks(body) + body = pr.get("body") + if not body: + click.echo( + "WARNING: MQ pull request without body, skipping metadata extraction", + err=True, + ) + return None + ref = _yaml_docs_from_fenced_blocks(body) + if ref is None: + click.echo( + "WARNING: MQ pull request body without Mergify metadata, skipping metadata extraction", + err=True, + ) + return ref def detect() -> MergeQueueMetadata | None: