Skip to content

fix CI npm cache write denied under restricted GITHUB_TOKEN - #59

Merged
proxymesh merged 1 commit into
mainfrom
cursor/fix-publish-cache-permissions-91f9
Sep 1, 2026
Merged

fix CI npm cache write denied under restricted GITHUB_TOKEN#59
proxymesh merged 1 commit into
mainfrom
cursor/fix-publish-cache-permissions-91f9

Conversation

@proxymeshai

Copy link
Copy Markdown
Collaborator

Summary

Confirmed npm and JSR both successfully published 0.2.4 (publish job concluded success). The annotation on Post Setup Node was a cache-save warning, not a publish failure:

cache write denied: token has no writable scopes

Cause

publish.yml (and proxy_integration_tests.yml) set explicit least-privilege permissions with only contents: read. That allows cache restore but not reserve/write when actions/setup-node uses cache: npm.

Fix

Add actions: write so setup-node can save the npm cache:

  • .github/workflows/publish.yml
  • .github/workflows/proxy_integration_tests.yml

actionlint passes on the updated workflows.

Slack Thread

Open in Web Open in Cursor 

Publish and integration workflows restrict GITHUB_TOKEN to contents:read,
which can restore caches but cannot reserve/write them. That produced the
setup-node warning "cache write denied: token has no writable scopes" after
a successful 0.2.4 publish. Add actions:write so cache: npm can save.

Co-authored-by: ProxyMesh AI <proxymeshai@users.noreply.github.com>
@proxymeshai
proxymeshai marked this pull request as ready for review September 1, 2026 20:45
@cursor
cursor Bot requested a review from proxymesh September 1, 2026 20:51

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not approved: this is a CI workflow permission change, not a Dependabot version bump or security update, so it needs human review. Cursor Bugbot passed with no findings; Cursor Security Agent was not running. Assigned a reviewer.

Open in Web View Automation 

Sent by Cursor Approval Agent: dependabot approver

@proxymesh
proxymesh merged commit 5dd365b into main Sep 1, 2026
3 checks passed
@proxymesh
proxymesh deleted the cursor/fix-publish-cache-permissions-91f9 branch September 1, 2026 21:05
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