Skip to content

ci: test install.sh and the skills against the latest release - #1891

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/kozlek/public-shell/test-install-sh-skills-against-latest-release--1ec853ff
Oct 6, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
devs/kozlek/public-shell/test-install-sh-skills-against-latest-release--1ec853ff

Conversation

@kozlek

@kozlek kozlek commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

The CLI's source now lives in the private mergify-ci-integrations, where
it is built and released; this repository keeps what users fetch from it:
install.sh, the agent skills and the releases. Their checks can't rely on
building the binary here any more, so they move to the binary people
actually get, the latest release's:

  • skills: scripts/check-skills.py, ported from the CLI's
    skill_references.rs and widened from the merge-queue skill to all six:
    each skill's front matter names it and describes it, the merge-queue
    skill keeps its required sections, and every mergify <group> <subcommand> a skill mentions exists in --list-native-commands.
  • install-script-release: install.sh against the latest release's real
    binaries and SHA256SUMS, served from a local fixture, on Linux, macOS
    and Windows, plus the wrong-hash and malformed-line SHA256SUMS cases
    that must fail closed.

Both join ci-gate. They run next to the source-based jobs for now; those
go with the source in the next change.

Tested locally against the real releases: the skill check passes on
2026.10.1 and on the 2026.10.3 build, and fails on a made-up mergify stack frobnicate; the install job's steps install 2026.10.1 and refuse
both tampered SHA256SUMS.

Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com

🤖 Generated with Claude Code

The CLI's source now lives in the private mergify-ci-integrations, where
it is built and released; this repository keeps what users fetch from it:
install.sh, the agent skills and the releases. Their checks can't rely on
building the binary here any more, so they move to the binary people
actually get, the latest release's:

- `skills`: scripts/check-skills.py, ported from the CLI's
  skill_references.rs and widened from the merge-queue skill to all six:
  each skill's front matter names it and describes it, the merge-queue
  skill keeps its required sections, and every `mergify <group>
  <subcommand>` a skill mentions exists in `--list-native-commands`.
- `install-script-release`: install.sh against the latest release's real
  binaries and SHA256SUMS, served from a local fixture, on Linux, macOS
  and Windows, plus the wrong-hash and malformed-line SHA256SUMS cases
  that must fail closed.

Both join ci-gate. They run next to the source-based jobs for now; those
go with the source in the next change.

Tested locally against the real releases: the skill check passes on
2026.10.1 and on the 2026.10.3 build, and fails on a made-up `mergify
stack frobnicate`; the install job's steps install 2026.10.1 and refuse
both tampered SHA256SUMS.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Change-Id: I1ec853ffcd56b481a8141bfa6df617d80cf9fdac
Copilot AI balanced review requested due to automatic review settings October 6, 2026 13:20
@kozlek
kozlek added this pull request to stack #1893 October 6, 2026 13:20
@mergify
mergify Bot deployed to Mergify Merge Protections October 6, 2026 13:20 Active
@mergify

mergify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Show 6 satisfied protections

🟢 🤖 Continuous Integration

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

🟢 👀 Review Requirements

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

🟢 Enforce conventional commit

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

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

🟢 🔎 Reviews

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

🟢 📕 PR description

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

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mergify
mergify Bot requested a review from a team October 6, 2026 13:28
@kozlek
kozlek marked this pull request as ready for review October 6, 2026 14:04
@mergify
mergify Bot requested a review from a team October 6, 2026 14:14
@mergify

mergify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-10-06 14:50 UTC · Rule: default · triggered by merge protections
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-10-06 14:50 UTC · at 98794c47fb94e0e49185660f6a205950f674ae45 · squash

This pull request spent 19 seconds in the queue, including 2 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Oct 6, 2026
@mergify
mergify Bot merged commit 98794c4 into main Oct 6, 2026
26 of 28 checks passed
@mergify
mergify Bot deleted the devs/kozlek/public-shell/test-install-sh-skills-against-latest-release--1ec853ff branch October 6, 2026 14:50
@mergify mergify Bot removed the queued label Oct 6, 2026

This branch had an error being deployed

1 failed and 1 active deployments
func-tests-live — a85d2350 Deployed Oct 6, 2026 by kozlek via live-tests #1986
Mergify Merge Protections — a85d2350 Deployed Oct 6, 2026 by mergify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants