chore(ci): declare least-privilege permissions on the desktop e2e workflow - #6622
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Both jobs ( Reviewed by Cursor Bugbot for commit bf4241c. Configure here. |
Greptile SummaryThis PR applies least-privilege token access to the manually dispatched desktop E2E workflow.
Confidence Score: 5/5The PR appears safe to merge with no actionable issues identified. The workflow only needs read access to check out repository contents, while its remaining setup, test, packaging, and artifact operations do not require broader
|
| Filename | Overview |
|---|---|
| .github/workflows/desktop-e2e.yml | Adds an appropriate read-only repository permission without disrupting the workflow’s checkout, test, packaging, or artifact-upload steps. |
Reviews (1): Last reviewed commit: "chore(ci): declare least-privilege permi..." | Re-trigger Greptile
bf4241c to
a902a5b
Compare
Summary
permissions: contents: readblock to.github/workflows/desktop-e2e.yml— it was the only workflow in the repo without one, so both its jobs inherited the repo-defaultGITHUB_TOKENscopelatestcanary leg, which runsbun add -d electron@latestand pulls an unpinned dependency tree onto the runnerType of Change
Testing
Tested manually — the workflow is
workflow_dispatch-only and neither job readssecrets.GITHUB_TOKENbeyond checkout, so read-only is sufficientChecklist