Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Dec 15, 2025

Summary by cubic

Fix release CI by scoping permissions to jobs and enabling OIDC for publishing. This unblocks the publish step by granting id-token: write with least-privilege settings.

  • Bug Fixes
    • Add id-token: write and contents: read to the publish job.
    • Move permissions to job level: release-please job gets contents/pull-requests/issues write; publish job uses minimal permissions.

Written for commit 5bd9234. Summary will update automatically on new commits.

@ryoppippi ryoppippi requested a review from a team as a code owner December 15, 2025 13:10
Copilot AI review requested due to automatic review settings December 15, 2025 13:10
@ryoppippi ryoppippi enabled auto-merge (squash) December 15, 2025 13:10
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 15, 2025

Open in StackBlitz

npm i https://pkg.pr.new/StackOneHQ/stackone-ai-node/@stackone/ai@234

commit: 5bd9234

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors the release workflow to follow the principle of least privilege by moving permissions from workflow-level to job-level scope, and adds the id-token: write permission to enable OIDC token-based publishing to npm.

  • Moved permissions from workflow-level to individual job-level for better security isolation
  • Added id-token: write permission to the publish-package job for OIDC authentication

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ryoppippi ryoppippi requested a review from glebedel December 15, 2025 13:11
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Copy link
Contributor

@glebedel glebedel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ryoppippi ryoppippi merged commit dd5b894 into main Dec 15, 2025
17 checks passed
@ryoppippi ryoppippi deleted the fix-release-ci branch December 15, 2025 13:57
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.

3 participants