Skip to content

fix(ci): unblock PR Test job and release sync PR creation#45

Merged
akirilyuk merged 3 commits into
mainfrom
fix/ci-pr-test-artifact-permissions
Jun 15, 2026
Merged

fix(ci): unblock PR Test job and release sync PR creation#45
akirilyuk merged 3 commits into
mainfrom
fix/ci-pr-test-artifact-permissions

Conversation

@akirilyuk

Copy link
Copy Markdown
Owner

Summary

  • Add actions: read to PR build.yml so reusable Test can download the compile-native artifact.
  • When integration tests run, force run_compile so TS-only PRs still upload bindings.
  • Release sync: optional REPO_SYNC_PAT; manual compare URL on failure.

Fixes Test / Test on PR #42 and #43.

PR Test failed in ~12s because build.yml lacked actions:read (artifact
download from compile-native) and TS-only PRs skipped native compile while
integration tests still required a .node.

Also document REPO_SYNC_PAT / repo Actions setting for post-release lockfile
PRs when GITHUB_TOKEN cannot create pull requests.

Co-authored-by: Cursor <cursoragent@cursor.com>
Reusable workflow_call jobs cannot download artifacts uploaded by caller
jobs in build.yml. Inline reusable-test steps so compile-native bindings
are available in the same workflow run.

Co-authored-by: Cursor <cursoragent@cursor.com>
Container compile-native artifacts are not reliably available to the
host Test job on self-hosted runners. Restore GHA cache first, then compile
inside ci-build via Docker when packages/bindings has no .node.

Co-authored-by: Cursor <cursoragent@cursor.com>
@akirilyuk akirilyuk merged commit 312554e into main Jun 15, 2026
7 checks passed
@akirilyuk akirilyuk deleted the fix/ci-pr-test-artifact-permissions branch June 15, 2026 09:51
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