Skip to content

fix(ci): open the traceability refresh PR when only merged PRs exist - #544

Draft
nbarbettini wants to merge 1 commit into
modelcontextprotocol:mainfrom
nbarbettini:fix/traceability-refresh-pr-check
Draft

nbarbettini wants to merge 1 commit into
modelcontextprotocol:mainfrom
nbarbettini:fix/traceability-refresh-pr-check

Conversation

@nbarbettini

Copy link
Copy Markdown
Contributor

Fixes #543.

The "Open/update the rolling refresh PR" step guarded gh pr create with gh pr view "$branch", which also matches closed and merged PRs. Since #292 (head traceability-refresh) merged on 2026-05-19, every weekly run has force-pushed the branch and then skipped creating the PR. As a result, src/seps/traceability.json on main is still missing SEPs 1932, 1933, 2640 and 2663.

This changes the guard to count only open PRs for the branch:

[ "$(gh pr list --head "$branch" --state open --json number --jq length)" -gt 0 ] || gh pr create ...

Nothing else changes. The run / propose permission split is untouched, and this PR doesn't regenerate traceability.json.

After this merges, the next scheduled run (Monday 06:00 UTC) or a manual workflow_dispatch should open the refresh PR from the already-updated traceability-refresh branch.

Checked against the live repo: the new guard evaluates to "create" for traceability-refresh today, and to "skip" for a branch that has an open PR.

🤖 Generated with Claude Code

`gh pr view <branch>` also resolves to closed/merged PRs, so once modelcontextprotocol#292
(head `traceability-refresh`) merged, every scheduled run found it and
skipped `gh pr create`. Only skip creation when an open PR exists.

Fixes modelcontextprotocol#543

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pkg-pr-new

pkg-pr-new Bot commented Oct 5, 2026

Copy link
Copy Markdown

Open in StackBlitz

npx https://pkg.pr.new/@modelcontextprotocol/conformance@544

commit: 373d086

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Traceability refresh workflow stopped opening PRs after #292 merged

1 participant