Skip to content

fix(ci): remove id-token write perm (not needed)#809

Merged
jsbroks merged 1 commit intoctrlplanedev:mainfrom
zachspar:spar/hotfix/ci
Feb 18, 2026
Merged

fix(ci): remove id-token write perm (not needed)#809
jsbroks merged 1 commit intoctrlplanedev:mainfrom
zachspar:spar/hotfix/ci

Conversation

@zachspar
Copy link
Contributor

@zachspar zachspar commented Feb 18, 2026

Background

CI pipeline is failing because id-token: write permission isn't granted to the calling workflow. This permission isn't needed at all, so removing it.

What's Changed?

  • remove id-token: write workflow permission in ci

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated build workflow permissions to improve security configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 18, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The workflow's permissions are reduced by removing the id-token: write permission from the GitHub Actions build-image workflow, narrowing the token scope without modifying any control flow or steps.

Changes

Cohort / File(s) Summary
Workflow Permission Scope
.github/workflows/build-image.yaml
Removed id-token: write permission from workflow permissions, reducing the token scope for the workflow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A token grows light, permissions now tight,
One less scope to wield in the workflow's flight,
Security whispers through each passing test,
The rabbit hops forth—less is often the best!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@jsbroks jsbroks merged commit 77b98dc into ctrlplanedev:main Feb 18, 2026
4 of 5 checks passed
@zachspar zachspar deleted the spar/hotfix/ci branch February 18, 2026 17:41
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.

2 participants

Comments