feat(rust): port queue show to native Rust by jd · Pull Request #1399 · Mergifyio/mergify-cli

jd · 2026-05-11T07:13:30Z

The Rust binary now serves mergify queue show <PR_NUMBER>
natively. The Python implementation
(mergify_cli/queue/cli.py:show plus the eight rendering
helpers it depended on, plus mergify_cli/queue/api.py) is
removed in the same PR — the port-and-delete rule keeps a single
live copy of every command.

This is the last command in the queue group, so the whole
mergify_cli/queue/ Python package goes away. Closes the gap
noted in #1380's commit message: mergify queue --help now
lists show alongside pause / unpause / status.

mergify queue show <PR_NUMBER> [-v] [--json] [-r REPO] [-t TOKEN] [-u URL]:

Resolves repository / token / API URL via the shared
mergify_core::auth resolver.
Fetches GET /v1/repos/<repo>/merge-queue/pull/<pr_number>
through the new HttpClient::get_if_exists helper. On 404
the command exits with MERGIFY_API_ERROR and the message
PR #<n> is not in the merge queue, matching the Python
implementation. Other 4xx/5xx surface as normal API errors.
With --json: pretty-prints the raw response. Schema is the
Mergify API contract, so unknown fields survive verbatim.
Without --json: renders the metadata block (position /
priority / queue rule / queued / ETA), then a CI-state line
and a checks section, then a conditions section.
--verbose switches the checks summary to a full table and
the conditions summary to a tree (├── └── │ ) instead
of the compact N/M met summary with bullet-listed
failures.

New plumbing in mergify-core::http:

Client::get_if_exists<T>(&path) -> Result<Option<T>, _> —
GET that returns None on 404. Mirrors
delete_if_exists for read-only endpoints where "not found"
is a meaningful caller branch rather than a server failure.
Reuses the same retry policy, bearer-auth injection, and
flavor-aware error mapping as get / post / put.

Tests:

7 new unit tests in crates/mergify-queue/src/show.rs:
compact metadata + checks summary + failing-conditions block;
verbose checks table + conditions tree; JSON passthrough with
a synthetic future_field to verify unknown fields survive;
404 → MergifyApi error with the right message; missing
mergeability_check falls through to "Waiting for
mergeability check..."; condition-group summarization (two
labels joined with or; truncation at 3+ labels);
aggregator recursion (any of / all of / not falls
through to the first leaf).

Wiring:

crates/mergify-cli/src/main.rs: adds Show(ShowCliArgs)
to QueueSubcommand, dispatch to
mergify_queue::show::run. Adds ("queue", "show") to
NATIVE_COMMANDS. ShowCliArgs carries the positional
pr_number: u64 plus --verbose and --json flags.

Python deletions:

mergify_cli/queue/__init__.py / cli.py / api.py:
removed entirely. The whole package goes away — all four
queue subcommands are now Rust-native.
mergify_cli/cli.py: drops the
from mergify_cli.queue import cli as queue_cli_mod import
and the cli.add_command(queue_cli_mod.queue) call.
mergify_cli/tests/queue/test_cli.py: deleted entirely (it
only covered _relative_time, which lives in
mergify-tui::time now).
mergify_cli/tests/queue/test_show.py: deleted entirely.
mergify_cli/tests/queue/test_skill.py: drops the click
import; the skill-reference check now consults the binary
alone (no parallel click-command list to merge).

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

Depends-On: #1408

jd · 2026-05-11T07:13:33Z

This pull request is part of a Mergify stack:

#	Pull Request	Link
1	test(queue): add live smoke test for queue show	#1408
2	feat(rust): port queue show to native Rust	#1399	👈
3	test(skill): port the skill-references test to Rust	#1414
4	test(freeze): add live smoke test for freeze list	#1434
5	feat(rust): port freeze list to native Rust	#1435
6	test(freeze): add live smoke test for freeze create/update/delete	#1436
7	feat(rust): port freeze create/update/delete to native Rust	#1437
8	refactor(rust): dedupe emit-helper boilerplate across command crates	#1438
9	refactor(rust): share test scaffolding via mergify-test-support crate	#1439
10	refactor(core): introduce CommandContext for the queue+freeze prelude	#1441
11	refactor(ci): consolidate the CI-env scrubber into a shared testing module	#1442
12	refactor: drop stale Phase X.Y doc markers and one inline color branch	#1443
13	refactor(tui): share StyledGlyph across queue show/status renderers	#1444
14	refactor(queue): drop indexmap, group_by_scope returns a Vec<(K, V)>	#1445
15	refactor(ci): swap uuid for getrandom in the GHA heredoc delimiter	#1446
16	refactor(config): standardize the workspace on serde_yaml_ng for YAML parsing	#1447

mergify · 2026-05-11T07:14:06Z

jd · 2026-05-11T11:46:46Z

Revision history

#	Type	Changes	Date
1	initial	`9b78873`	2026-05-11 11:46 UTC
2	rebase	`9b78873 → e00f387`	2026-05-11 11:46 UTC
3	rebase	`e00f387 → c783f0f`	2026-05-11 12:06 UTC
4	rebase	`c783f0f → d96bb5e`	2026-05-11 12:19 UTC
5	rebase	`d96bb5e → 47c76c0`	2026-05-11 14:08 UTC
6	rebase	`47c76c0 → 0d446e4`	2026-05-11 14:59 UTC
7	rebase	`0d446e4 → 40a75b9`	2026-05-11 15:05 UTC
8	rebase	`40a75b9 → d258905`	2026-05-11 20:59 UTC
9	content	`d258905 → b9688d1` (raw)	2026-05-12 07:44 UTC
10	content	`b9688d1 → 72c4bf3` (raw)	2026-05-12 08:54 UTC
11	rebase	`72c4bf3 → 9d67b7f` (rebase only)	2026-05-13 08:25 UTC
12	rebase	`9d67b7f → ceb14f3` (rebase only)	2026-05-18 08:20 UTC
13	rebase	`ceb14f3 → 43451ed` (rebase only)	2026-05-19 08:07 UTC
14	rebase	`43451ed → 5a75bc2` (rebase only)	2026-05-19 13:04 UTC
15	rebase	`5a75bc2 → fe67fc5` (rebase only)	2026-05-19 14:15 UTC

Pull request has been modified.

Pins the contract for ``mergify queue show`` 404 handling before the Rust port lands on top. Same test exercises Python at this PR's CI and Rust on the port commit's rebase. Uses a PR number far above the test repo's actual PR count to force the 404 path. That's robust against the test repo's queue state (PR #1 may or may not be queued at any given moment) and exercises the parts that would silently break on URL or schema drift: endpoint reachability, auth, and 404 → ``MERGIFY_API_ERROR`` exit code (6) mapping with the ``not in the merge queue`` message. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Change-Id: I944234ba2d22d99b410f4b8c91e56f0d8a49a9f7

The Rust binary now serves ``mergify queue show <PR_NUMBER>`` natively. The Python implementation (``mergify_cli/queue/cli.py:show`` plus the eight rendering helpers it depended on, plus ``mergify_cli/queue/api.py``) is removed in the same PR — the port-and-delete rule keeps a single live copy of every command. This is the last command in the ``queue`` group, so the whole ``mergify_cli/queue/`` Python package goes away. Closes the gap noted in #1380's commit message: ``mergify queue --help`` now lists ``show`` alongside ``pause`` / ``unpause`` / ``status``. ``mergify queue show <PR_NUMBER> [-v] [--json] [-r REPO] [-t TOKEN] [-u URL]``: 1. Resolves repository / token / API URL via the shared ``mergify_core::auth`` resolver. 2. Fetches ``GET /v1/repos/<repo>/merge-queue/pull/<pr_number>`` through the new ``HttpClient::get_if_exists`` helper. On 404 the command exits with ``MERGIFY_API_ERROR`` and the message ``PR #<n> is not in the merge queue``, matching the Python implementation. Other 4xx/5xx surface as normal API errors. 3. With ``--json``: pretty-prints the raw response. Schema is the Mergify API contract, so unknown fields survive verbatim. 4. Without ``--json``: renders the metadata block (position / priority / queue rule / queued / ETA), then a CI-state line and a checks section, then a conditions section. ``--verbose`` switches the checks summary to a full table and the conditions summary to a tree (``├── └── │ ``) instead of the compact ``N/M met`` summary with bullet-listed failures. New plumbing in ``mergify-core::http``: - ``Client::get_if_exists<T>(&path) -> Result<Option<T>, _>`` — GET that returns ``None`` on 404. Mirrors ``delete_if_exists`` for read-only endpoints where "not found" is a meaningful caller branch rather than a server failure. Reuses the same retry policy, bearer-auth injection, and flavor-aware error mapping as ``get`` / ``post`` / ``put``. Tests: - 7 new unit tests in ``crates/mergify-queue/src/show.rs``: compact metadata + checks summary + failing-conditions block; verbose checks table + conditions tree; JSON passthrough with a synthetic ``future_field`` to verify unknown fields survive; 404 → ``MergifyApi`` error with the right message; missing ``mergeability_check`` falls through to "Waiting for mergeability check..."; condition-group summarization (two labels joined with ``or``; truncation at 3+ labels); aggregator recursion (``any of`` / ``all of`` / ``not`` falls through to the first leaf). Wiring: - ``crates/mergify-cli/src/main.rs``: adds ``Show(ShowCliArgs)`` to ``QueueSubcommand``, dispatch to ``mergify_queue::show::run``. Adds ``("queue", "show")`` to ``NATIVE_COMMANDS``. ``ShowCliArgs`` carries the positional ``pr_number: u64`` plus ``--verbose`` and ``--json`` flags. Python deletions: - ``mergify_cli/queue/__init__.py`` / ``cli.py`` / ``api.py``: removed entirely. The whole package goes away — all four ``queue`` subcommands are now Rust-native. - ``mergify_cli/cli.py``: drops the ``from mergify_cli.queue import cli as queue_cli_mod`` import and the ``cli.add_command(queue_cli_mod.queue)`` call. - ``mergify_cli/tests/queue/test_cli.py``: deleted entirely (it only covered ``_relative_time``, which lives in ``mergify-tui::time`` now). - ``mergify_cli/tests/queue/test_show.py``: deleted entirely. - ``mergify_cli/tests/queue/test_skill.py``: drops the click import; the skill-reference check now consults the binary alone (no parallel click-command list to merge). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Change-Id: I6c265303a37642529dbbcef6f255eb429407a1d2

mergify · 2026-05-20T07:30:48Z

mergify Bot had a problem deploying to Mergify Merge Protections May 11, 2026 07:13 Failure

jd had a problem deploying to func-tests-live May 11, 2026 07:13 — with GitHub Actions Failure

mergify Bot requested a review from a team May 11, 2026 07:27

jd force-pushed the devs/jd/worktree-rust-port/port-queue-show-native-rust--6c265303 branch from 9b78873 to e00f387 Compare May 11, 2026 11:46

jd force-pushed the devs/jd/worktree-rust-port/derive-native-queue-cmds-binary-hardcoded-list--74502fe8 branch from da5a658 to 0e20a86 Compare May 11, 2026 11:46

jd had a problem deploying to func-tests-live May 11, 2026 11:46 — with GitHub Actions Failure

This was referenced May 11, 2026

feat(rust): port ci queue-info to native Rust #1407

Merged

test(ci): add live smoke test for ci git-refs #1404

Merged

feat(rust): port ci git-refs to native Rust #1405

Merged

test(ci): add live smoke test for ci queue-info #1406

Merged

mergify Bot had a problem deploying to Mergify Merge Protections May 11, 2026 11:46 Failure

jd force-pushed the devs/jd/worktree-rust-port/port-queue-show-native-rust--6c265303 branch from e00f387 to c783f0f Compare May 11, 2026 12:06

jd force-pushed the devs/jd/worktree-rust-port/derive-native-queue-cmds-binary-hardcoded-list--74502fe8 branch from 0e20a86 to 65e243f Compare May 11, 2026 12:06

jd changed the base branch from devs/jd/worktree-rust-port/derive-native-queue-cmds-binary-hardcoded-list--74502fe8 to devs/jd/worktree-rust-port/add-live-smoke-test-queue-show--944234ba May 11, 2026 12:06

jd had a problem deploying to func-tests-live May 11, 2026 12:06 — with GitHub Actions Failure

This was referenced May 11, 2026

test(queue): add live smoke test for queue status #1409

Merged

test(queue): add live smoke test for queue show #1408

Merged

mergify Bot had a problem deploying to Mergify Merge Protections May 11, 2026 12:06 Failure

jd force-pushed the devs/jd/worktree-rust-port/port-queue-show-native-rust--6c265303 branch from c783f0f to d96bb5e Compare May 11, 2026 12:19

mergify Bot had a problem deploying to Mergify Merge Protections May 18, 2026 08:20 Failure

jd force-pushed the devs/jd/worktree-rust-port/port-queue-show-native-rust--6c265303 branch from ceb14f3 to 43451ed Compare May 19, 2026 08:07

jd force-pushed the devs/jd/worktree-rust-port/add-live-smoke-test-queue-show--944234ba branch from 77ebbfc to f15a0d7 Compare May 19, 2026 08:07

This was referenced May 19, 2026

test(freeze): add live smoke test for freeze list #1434

Merged

feat(rust): port freeze list to native Rust #1435

Merged

jd temporarily deployed to func-tests-live May 19, 2026 08:07 — with GitHub Actions Inactive

mergify Bot had a problem deploying to Mergify Merge Protections May 19, 2026 08:07 Failure

jd force-pushed the devs/jd/worktree-rust-port/add-live-smoke-test-queue-show--944234ba branch from f15a0d7 to ede38fd Compare May 19, 2026 13:04

jd force-pushed the devs/jd/worktree-rust-port/port-queue-show-native-rust--6c265303 branch from 43451ed to 5a75bc2 Compare May 19, 2026 13:04

jd temporarily deployed to func-tests-live May 19, 2026 13:04 — with GitHub Actions Inactive

jd and others added 2 commits May 19, 2026 16:14

JulianMaurin approved these changes May 20, 2026

View reviewed changes

kozlek approved these changes May 20, 2026

View reviewed changes

mergify Bot mentioned this pull request May 20, 2026

merge queue: embarking main (8438ac4) and #1399 together #1449

Closed

38 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(rust): port queue show to native Rust#1399

feat(rust): port queue show to native Rust#1399
mergify[bot] merged 2 commits into
mainfrom
devs/jd/worktree-rust-port/port-queue-show-native-rust--6c265303

jd commented May 11, 2026 •

edited

Loading

Uh oh!

jd commented May 11, 2026 •

edited

Loading

Uh oh!

mergify Bot commented May 11, 2026 •

edited

Loading

Uh oh!

jd commented May 11, 2026 •

edited

Loading

Uh oh!

mergify Bot commented May 20, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

5 participants

Conversation

jd commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jd commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mergify Bot commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Protections

🟢 ⛓️ Depends-On Requirements

🟢 🤖 Continuous Integration

🟢 👀 Review Requirements

🟢 Enforce conventional commit

🟢 🔎 Reviews

🟢 📕 PR description

Uh oh!

jd commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Revision history

Uh oh!

mergify Bot commented May 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

5 participants

jd commented May 11, 2026 •

edited

Loading

jd commented May 11, 2026 •

edited

Loading

mergify Bot commented May 11, 2026 •

edited

Loading

jd commented May 11, 2026 •

edited

Loading

mergify Bot commented May 20, 2026 •

edited

Loading