Skip to content

fix(ci): update existing schema-regen PRs with changeset#163

Open
benminer wants to merge 2 commits intomainfrom
feature/auto-changeset-fix
Open

fix(ci): update existing schema-regen PRs with changeset#163
benminer wants to merge 2 commits intomainfrom
feature/auto-changeset-fix

Conversation

@benminer
Copy link
Copy Markdown
Collaborator

Summary

  • When an open schema-regeneration PR already exists, the workflow previously skipped it entirely — PRs created before the changeset requirement (like chore: regenerate schemas from OpenAPI spec #156) are permanently blocked
  • Adds an "Update existing PR" step that checks out the existing branch, re-runs schema generation, adds a changeset if one is missing, and pushes
  • Enhances the existing-PR check to output branch name and URL (not just count)
  • Updates Slack notification to fire on both create and update paths

Test plan

When an open schema-regeneration PR already exists, the workflow
skipped it entirely. PRs created before the changeset requirement
(like #156) are permanently blocked because they lack a changeset
and the workflow never updates them.

Add an "Update existing PR" path that checks out the existing branch,
re-runs schema generation, adds a changeset if missing, and pushes.
@benminer benminer requested a review from a team April 27, 2026 19:46
- Use git checkout -B (force-reset) to tolerate pre-existing local branch
- Only set pr_url output when a commit is actually pushed
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.

1 participant