Skip to content

ci: declare contents: read workflow permissions - #36

Merged
zen-agents[bot] merged 1 commit into
mainfrom
steward/ci-workflow-permissions
Aug 17, 2026
Merged

ci: declare contents: read workflow permissions#36
zen-agents[bot] merged 1 commit into
mainfrom
steward/ci-workflow-permissions

Conversation

@zen-agents

@zen-agents zen-agents Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Why

CodeQL actions/missing-workflow-permissions flags this workflow because it had no explicit permissions: block, so the default GITHUB_TOKEN was overly broad.

Change

Add a top-level least-privilege block:

permissions:
  contents: read

This matches the estate pattern already on agency-automations / creditscan-v2 / ZenSms.

Scope

  • Path: .github/workflows/ci.yml only
  • No pin, job, or step changes
  • Do not merge from this PR unless a later steward pass is assigned

Summary by CodeRabbit

  • Chores
    • Updated automated workflow permissions to use read-only access for repository contents.

CodeQL actions/missing-workflow-permissions flags CI workflows
without an explicit permissions block. Match the estate pattern
already on agency-automations / creditscan-v2 / ZenSms.
@greptile-apps

greptile-apps Bot commented Aug 17, 2026

Copy link
Copy Markdown

Required label not found on this PR.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: ZenSystemAI/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: bfc47f70-1998-4269-b999-75c8d2efa7a2

📥 Commits

Reviewing files that changed from the base of the PR and between 60b777e and 17b7d34.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Included review availability: 5 reviews are currently available. Based on recent review activity, included reviews refill at 6 per hour.


📝 Walkthrough

Walkthrough

The CI workflow now grants workflow-level read-only access to repository contents.

Changes

CI permissions

Layer / File(s) Summary
Configure CI repository access
.github/workflows/ci.yml
The workflow adds contents: read permissions for CI jobs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 17b7d

This change limits the workflow token to read-only repository contents and is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Suggested reviewers: skiipy11

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the workflow permission change made in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch steward/ci-workflow-permissions

Comment @coderabbitai help to get the list of available commands.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; the remaining evidence is a least-privilege contents: read addition to .github/workflows/ci.yml with no findings requiring human review. No reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@zen-agents zen-agents Bot added the review:standard One CodeRabbit review at the ready tip label Aug 17, 2026
@zen-agents
zen-agents Bot merged commit 10940b4 into main Aug 17, 2026
10 checks passed
@zen-agents
zen-agents Bot deleted the steward/ci-workflow-permissions branch August 17, 2026 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:standard One CodeRabbit review at the ready tip

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants