Skip to content

fix(dev-lead): grant statuses:read (fixes startup_failure)#370

Merged
don-petry merged 6 commits into
mainfrom
fix/dev-lead-align
Jun 8, 2026
Merged

fix(dev-lead): grant statuses:read (fixes startup_failure)#370
don-petry merged 6 commits into
mainfrom
fix/dev-lead-align

Conversation

@don-petry

@don-petry don-petry commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

The reusable workflow has requested statuses: read at the job level since #435, but this caller never granted it, so dispatch runs fail with startup_failure. Adds statuses: read to jobs.dev-lead.permissions.

Urgent: should merge before the 2026-06-07 04:30 UTC re-sweep (stuck PR #362).

Refs petry-projects/.github#402.

🤖 Generated with Claude Code

Summary by CodeRabbit

No user-facing changes in this release. Internal workflow permissions were updated to support development processes.

The reusable workflow has requested statuses:read at the job level since #435,
but this caller never granted it — so dispatch runs fail with startup_failure.
Adds statuses:read to jobs.dev-lead.permissions.

Refs petry-projects/.github#402

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 7, 2026 01:00
@don-petry don-petry requested a review from a team as a code owner June 7, 2026 01:00
@coderabbitai

coderabbitai Bot commented Jun 7, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1c486774-7706-453f-a970-6ed07a696b5a

📥 Commits

Reviewing files that changed from the base of the PR and between f3cde38 and 201dc63.

📒 Files selected for processing (1)
  • .github/workflows/dev-lead.yml

📝 Walkthrough

Walkthrough

The .github/workflows/dev-lead.yml workflow's job-level permissions are extended with statuses: read permission. This addition enables the workflow to function properly with the reusable dev-lead-reusable.yml workflow that requires this permission.

Changes

Dev-Lead Workflow Permission Update

Layer / File(s) Summary
Job permissions for reusable workflow compatibility
.github/workflows/dev-lead.yml
The job-level permissions are updated to include statuses: read, adding the required permission for the reusable dev-lead-reusable.yml workflow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 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 identifies the fix: granting the missing statuses:read permission to the dev-lead workflow and explains what it resolves (startup_failure).
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/dev-lead-align

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

Copilot AI 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.

Pull request overview

This PR fixes GitHub Actions startup_failure for repository_dispatch runs by granting the missing statuses: read permission to the dev-lead job, matching the permissions required by the referenced reusable workflow.

Changes:

  • Add statuses: read to jobs.dev-lead.permissions in the caller workflow.

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — review-changes (applied)

Changes committed and pushed.

@don-petry don-petry enabled auto-merge (squash) June 7, 2026 01:02
@sonarqubecloud

sonarqubecloud Bot commented Jun 7, 2026

Copy link
Copy Markdown

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — review-changes (no-changes)

No changes were needed for this PR.

@don-petry don-petry merged commit ba7a75a into main Jun 8, 2026
10 checks passed
@don-petry don-petry deleted the fix/dev-lead-align branch June 8, 2026 00:20
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

CI Failure: SonarCloud Code Analysis

Step: SonarCloud Quality Gate
Root cause: Env issue

SonarCloud Code Analysis failed as an external quality gate check on this PR. Because the details URL points directly to https://sonarcloud.io with no embedded GitHub Actions run ID, no step-level logs are available; the most common cause for this pattern is a missing or invalid SONAR_TOKEN repository secret, which prevents SonarCloud from authenticating and completing its analysis. The PR diff itself only adds a single statuses: read permission line to a workflow YAML file, so the failure is not related to source-code changes in this PR.

Suggested fix: Verify that the SONAR_TOKEN secret is correctly configured under Settings → Secrets and variables → Actions in the petry-projects/google-app-scripts repository, then re-run the SonarCloud check.

View run logs

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.

4 participants