Skip to content

ci: fix release PR creation and CLI e2e pnpm matrix - #1158

Merged
AbhinRustagi merged 2 commits into
mainfrom
abhishek/fix-release-ci
Sep 11, 2026
Merged

ci: fix release PR creation and CLI e2e pnpm matrix#1158
AbhinRustagi merged 2 commits into
mainfrom
abhishek/fix-release-ci

Conversation

@abhithesys

Copy link
Copy Markdown
Contributor
  • release.yml: create the Version Packages PR with a token minted from the THESYS_PR_CREATOR GitHub App (vars.THESYS_PR_CREATOR_APP_ID + secrets.THESYS_PR_CREATOR_PRIVATE_KEY). GITHUB_TOKEN is not permitted to create PRs in this org, so every Release run has been failing at PR creation; app-made pushes and PRs also trigger workflows, giving the Version PR normal CI. Token goes to the action's github-token input (v2) and to env GITHUB_TOKEN for @changesets/changelog-github lookups; workflow permissions drop to contents:read + id-token:write.
  • cli-e2e.yml: the pnpm matrix step's explicit version collides with the root packageManager field (action-setup v6 hard-errors on the mismatch); point package_json_file at packages/openui-cli/package.json so the matrix version stays authoritative for template testing.

Requires the app var/secret to be configured on the repo before the next Release run can succeed.

- release.yml: create the Version Packages PR with a token minted from the
  THESYS_PR_CREATOR GitHub App (vars.THESYS_PR_CREATOR_APP_ID +
  secrets.THESYS_PR_CREATOR_PRIVATE_KEY). GITHUB_TOKEN is not permitted to
  create PRs in this org, so every Release run has been failing at PR
  creation; app-made pushes and PRs also trigger workflows, giving the
  Version PR normal CI. Token goes to the action's github-token input (v2)
  and to env GITHUB_TOKEN for @changesets/changelog-github lookups; workflow
  permissions drop to contents:read + id-token:write.
- cli-e2e.yml: the pnpm matrix step's explicit version collides with the
  root packageManager field (action-setup v6 hard-errors on the mismatch);
  point package_json_file at packages/openui-cli/package.json so the matrix
  version stays authoritative for template testing.

Requires the app var/secret to be configured on the repo before the next
Release run can succeed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
openui-docs Ready Ready Preview Sep 11, 2026 5:06am UTC

Request Review

Inside the checkout, pnpm >=10 self-switches to the root packageManager
version (pnpm@10.33.0) and reports it, failing the matrix expectation
(Expected pnpm 11.x, got 10.33.0). The e2e steps all run under runner.temp,
so verify there — measuring the pnpm the templates will actually use.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@AbhinRustagi
AbhinRustagi merged commit 36f551e into main Sep 11, 2026
32 checks passed
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