feat(ci-insights): Add mergify tests show command#1382
Merged
mergify[bot] merged 1 commit intoMay 21, 2026
Merged
Conversation
remyduthu
had a problem deploying
to
func-tests-live
GitHub Actions
Failure
Contributor
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 👀 Review RequirementsWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 ReviewsWonderful, this rule succeeded.
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
remyduthu
changed the title
mergify ci tests lookup commandmergify ci tests lookup command
remyduthu
force-pushed
the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
from
6544154 to
57267f7
Compare
Contributor
Author
Revision history
|
remyduthu
had a problem deploying
to
func-tests-live
GitHub Actions
Failure
remyduthu
had a problem deploying
to
func-tests-live
GitHub Actions
Failure
There was a problem hiding this comment.
Pull request overview
Adds a new native Rust mergify ci tests lookup command to batch-resolve CI Insights test identities by name (glob-aware), fetch per-test health/metrics, and render either human output or a single JSON payload. This extends the Rust-native mergify ci surface area and introduces a reusable HTTP helper for ordered/repeated query parameters.
Changes:
- Add
mergify ci tests lookup NAME...command (CLI wiring, API calls, rendering, exit-code aggregation, and unit tests). - Add
HttpClient::get_with_queryto support ordered/repeated query params with proper encoding. - Update docs and add a live smoke test exercising the lookup endpoint.
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/mergify-ci/SKILL.md | Documents the new ci tests lookup command, options, and exit codes. |
| README.md | Mentions test health inspection under the mergify ci command group. |
| func-tests/test_live_smoke.py | Adds a live smoke test for the lookup endpoint (no-match path). |
| crates/mergify-core/src/http.rs | Introduces get_with_query plus unit tests for ordering/encoding behavior. |
| crates/mergify-cli/src/main.rs | Adds clap subcommand plumbing + native dispatch for ci tests lookup and JSON mode selection. |
| crates/mergify-ci/src/tests_lookup.rs | Implements lookup + details fan-out, rendering, JSON payload, exit code policy, and tests. |
| crates/mergify-ci/src/lib.rs | Exposes the new tests_lookup module. |
| crates/mergify-ci/src/detector.rs | Adds split_owner_repo helper with tests. |
| crates/mergify-ci/Cargo.toml | Adds Rust deps needed by the new command (chrono, futures). |
| Cargo.lock | Lockfile updates for newly introduced/updated dependencies. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
remyduthu
force-pushed
the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
from
57267f7 to
3d1d1d8
Compare
remyduthu
had a problem deploying
to
func-tests-live
GitHub Actions
Failure
remyduthu
had a problem deploying
to
func-tests-live
GitHub Actions
Failure
Contributor
|
@remyduthu this pull request is now in conflict 😩 |
remyduthu
commented
May 18, 2026
remyduthu
changed the title
mergify ci tests lookup commandmergify ci tests show command
remyduthu
force-pushed
the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
from
3d1d1d8 to
10479a6
Compare
jd
reviewed
May 20, 2026
remyduthu
force-pushed
the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
from
93627c6 to
a42941c
Compare
remyduthu
force-pushed
the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
from
a42941c to
fb35e47
Compare
Contributor
|
@remyduthu this pull request is now in conflict 😩 |
Wraps two CI Insights endpoints into a single batch command:
- `GET /v1/ci/{owner}/repositories/{repo}/search/tests` resolves test
identities by name (glob-aware) on the default branch. Filters
(`test_name`, `pipeline_name`, `job_name`, …) travel as repeated
query parameters; page size travels as `per_page`.
- `GET /v1/ci/{owner}/repositories/{repo}/tests/{test_id}` returns the
full health/metrics payload for one identity.
The search is a true batch API — pass one or more `<NAME>` positionals
(globs allowed) and one block per match is rendered. `--json` emits a
single `{"tests": [...]}` document; the human renderer hides metadata
lines for absent fields rather than printing placeholders. Exit code
reflects the worst health observed across results (0 = healthy or
unknown, 1 = any flaky, 6 = any broken).
`split_owner_repo` lives in `detector.rs` next to the existing
`owner/repo` validators so callers can interpolate the segments into
request paths without re-escaping. A new `HttpClient::get_with_query`
helper percent-encodes values and preserves repeated keys in caller
order, which the search endpoint relies on for `test_name` repetition.
Also adds a live-smoke case that exercises the search endpoint with a
guaranteed-nonexistent name so the round-trip stays independent of
canary repository state.
Fixes: MRGFY-7166
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Change-Id: Iafe286495b7842079e9c63b437a6305926fc22a5
remyduthu
force-pushed
the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
from
fb35e47 to
0419af5
Compare
JulianMaurin
approved these changes
May 21, 2026
kozlek
approved these changes
May 21, 2026
Contributor
Merge Queue Status
This pull request spent 17 minutes 59 seconds in the queue, including 17 minutes 24 seconds running CI. Required conditions to merge
|
36 tasks
mergify
Bot
deleted the
devs/remyduthu/tests-lookup-cli/add-mergify-ci-tests-lookup-cmd--afe28649
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Wraps two CI Insights endpoints into a single batch command:
GET /v1/ci/{owner}/repositories/{repo}/search/testsresolves testidentities by name (glob-aware) on the default branch. Filters
(
test_name,pipeline_name,job_name, …) travel as repeatedquery parameters; page size travels as
per_page.GET /v1/ci/{owner}/repositories/{repo}/tests/{test_id}returns thefull health/metrics payload for one identity.
The search is a true batch API — pass one or more
<NAME>positionals(globs allowed) and one block per match is rendered.
--jsonemits asingle
{"tests": [...]}document; the human renderer hides metadatalines for absent fields rather than printing placeholders. Exit code
reflects the worst health observed across results (0 = healthy or
unknown, 1 = any flaky, 6 = any broken).
split_owner_repolives indetector.rsnext to the existingowner/repovalidators so callers can interpolate the segments intorequest paths without re-escaping. A new
HttpClient::get_with_queryhelper percent-encodes values and preserves repeated keys in caller
order, which the search endpoint relies on for
test_namerepetition.Also adds a live-smoke case that exercises the search endpoint with a
guaranteed-nonexistent name so the round-trip stays independent of
canary repository state.
Fixes: MRGFY-7166
Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com