Skip to content

fix(ci): validate generated version PRs without fresh changesets - #33

Merged
canblmz1 merged 5 commits into
mainfrom
fix/validate-generated-version-prs
Oct 12, 2026
Merged

canblmz1 merged 5 commits into
mainfrom
fix/validate-generated-version-prs

Conversation

@canblmz1

Copy link
Copy Markdown
Owner

Fix the release-candidate CI loop

PR #32 correctly consumes its changesets, but the generic contributor gate rejects the resulting version-only diff. Keep ordinary contributor Changesets validation, and accept consumed-changeset candidates only when they exactly reproduce versioning from their comparison base.

  • Dependency-free preflight runs before pnpm setup/install in both readiness jobs.
  • Strict paths/modes and complete changeset consumption are required; core/CLI manifests must match the base except stable version fields.
  • After frozen installation, run the installed Changesets CLI directly in a detached base worktree and compare complete tracked Git trees.
  • No branch-name/author allowlist, failed-check fallback, empty changeset, or skipped quality checks.

Verification

81/81 dependency-free script regressions passed independently, including 34 gate tests; syntax, YAML ordering and base/blob checks passed. Actual installed Changesets replay was not run locally (the process test uses a fake CLI). Exact-head full CI, then real replay on the refreshed release PR, remain required.

Only five files change: readiness workflow, helper/tests, root test command and contribution guidance. No publish workflow, permissions, secret, dependency or version changes. This is validation hardening, not hostile-PR process isolation.

No release is authorized by this PR. Keep #32 draft pending this fix, fresh candidate review, all expected checks and explicit publication approval.

@canblmz1
canblmz1 marked this pull request as ready for review October 12, 2026 20:46
Copilot AI balanced review requested due to automatic review settings October 12, 2026 20:46
@canblmz1
canblmz1 merged commit eb0abc5 into main Oct 12, 2026
5 checks passed

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.

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.

2 participants