test(skill): port the skill-references test to Rust

[jd]

Replaces mergify_cli/tests/queue/test_skill.py with a Rust
integration test in crates/mergify-cli/tests/skill_references.rs.

The test validates two artifacts with no Python in the picture:

• The skills/mergify-merge-queue/SKILL.md Markdown file
  (frontmatter shape, required sections).
• The Rust binary's --list-native-commands output (every
  mergify queue <cmd> reference in the skill must resolve to
  a native command).

Keeping the test in pytest meant carrying Python plumbing for a
language-agnostic concern. The Rust port:

• spawns the binary via CARGO_BIN_EXE_mergify (the artifact
  cargo test just built), so the test always exercises the
  current code rather than whatever mergify happens to be on
  PATH;
• reads the skill file via CARGO_MANIFEST_DIR-relative
  resolution, so it's robust to the cwd;
• adds two dev-deps to mergify-cli (regex for the
  frontmatter / reference patterns, serde_yaml_ng for
  frontmatter parsing — already used by mergify-ci).

The mergify_cli/tests/queue/ directory had no other content,
so it goes away entirely; the Python yaml dev-dep stays in
place for now (still used elsewhere).

4 tests, same coverage as before:

• skill_content_is_readable
• skill_has_valid_frontmatter
• skill_has_required_sections
• skill_references_valid_commands

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

[jd]

This pull request is part of a Mergify stack:

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

[mergify]

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.

• all of:
  • check-success=ci-gate

🟢 👀 Review Requirements

Wonderful, this rule succeeded.

• any of:
  • #approved-reviews-by>=2
  • author = dependabot[bot]
  • author = mergify-ci-bot
  • author = renovate[bot]

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

• title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.

• #changes-requested-reviews-by = 0
• #review-requested = 0
• #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.

• body ~= (?ms:.{48,})

[jd]

Revision history

#	Type	Changes	Reason	Date
1	initial	1c824a4		2026-05-13 08:25 UTC
2	rebase	1c824a4 → c051a4c (rebase only)		2026-05-13 08:25 UTC
3	rebase	c051a4c → c78e289 (rebase only)		2026-05-18 08:20 UTC
4	rebase	c78e289 → f9ee5e7 (rebase only)		2026-05-19 08:07 UTC
5	content	f9ee5e7 → 458fec9		2026-05-19 13:04 UTC
6	rebase	458fec9 → a0a4522 (rebase only)		2026-05-19 14:15 UTC
7	rebase	a0a4522 → 9b87b7a (rebase only)		2026-05-20 08:42 UTC
8	content	9b87b7a → c228d70 (raw)		2026-05-20 09:05 UTC

Pull request has been modified.

[mergify]

@jd this pull request is now in conflict 😩

[Copilot]

Pull request overview

Ports the merge-queue skill validation test from pytest to a Rust integration test, so the checks run directly against the freshly built mergify binary and the repository’s SKILL.md without requiring Python test plumbing.

Changes:

• Removed the Python test_skill.py and the now-empty mergify_cli/tests/queue/ test package.
• Added crates/mergify-cli/tests/skill_references.rs to validate SKILL frontmatter/sections and cross-check mergify queue <cmd> references against --list-native-commands.
• Added Rust dev-dependencies (regex, serde_yaml_ng) for frontmatter parsing and reference extraction.

Reviewed changes

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

Show a summary per file

File	Description
mergify_cli/tests/queue/test_skill.py	Removes the pytest-based skill validation test.
mergify_cli/tests/queue/init.py	Removes the now-unused Python test package marker (directory deleted).
crates/mergify-cli/tests/skill_references.rs	New Rust integration test that validates SKILL.md and command references via the built binary.
crates/mergify-cli/Cargo.toml	Adds dev-dependencies needed by the new Rust integration test.
Cargo.lock	Locks new transitive dependencies introduced by the Rust test dev-deps.

---

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

[mergify]

Merge Queue Status

• ✅ Entered queue — 2026-05-20 10:06 UTC · Rule: default
• ✅ Checks skipped · PR is already up-to-date
• ✅ Merged — 2026-05-20 10:07 UTC · at c228d7088582f479d5c2253490ca665c3998f71a · squash

This pull request spent 1 minute 3 seconds in the queue, including 5 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)(?:\(.+\))?:
    • test(skill): port the skill-references test to Rust #1414
• all of [🛡 Merge Protections rule 👀 Review Requirements]:
  • any of:
    • #approved-reviews-by>=2
      • test(skill): port the skill-references test to Rust #1414
    • author = dependabot[bot]
      • test(skill): port the skill-references test to Rust #1414
    • author = mergify-ci-bot
      • test(skill): port the skill-references test to Rust #1414
    • author = renovate[bot]
      • test(skill): port the skill-references test to Rust #1414
• all of [🛡 Merge Protections rule 📕 PR description]:
  • body ~= (?ms:.{48,})
    • test(skill): port the skill-references test to Rust #1414
• all of [🛡 Merge Protections rule 🔎 Reviews]:
  • #changes-requested-reviews-by = 0
    • test(skill): port the skill-references test to Rust #1414
  • #review-requested = 0
    • test(skill): port the skill-references test to Rust #1414
  • #review-threads-unresolved = 0
    • test(skill): port the skill-references test to Rust #1414
• all of [🛡 Merge Protections rule 🤖 Continuous Integration]:
  • all of:
    • check-success=ci-gate