Skip to content

fix: update allowed roles to include maintainer#1704

Merged
afarntrog merged 1 commit intostrands-agents:mainfrom
afarntrog:role_updates
Feb 16, 2026
Merged

fix: update allowed roles to include maintainer#1704
afarntrog merged 1 commit intostrands-agents:mainfrom
afarntrog:role_updates

Conversation

@afarntrog
Copy link
Contributor

@afarntrog afarntrog commented Feb 16, 2026

Description

Fixes the following to allow workflows to run for maintainers:

with:
    result-encoding: string
    script: const checkAuthorization = require('./_authorization-check/authorization-check/scripts/check-authorization.cjs');
  return await checkAuthorization(context, github, {
    username: 'afarntrog',
    allowedRoles: 'triage,write,admin'
  });
  
    github-token: ***
    debug: false
    user-agent: actions/github-script
    retries: 0
    retry-exempt-status-codes: 400,401,403,404,422
User afarntrog does not have write access to the repository (role: maintain, allowed: triage, write, admin)

Related Issues

Documentation PR

Type of Change

Bug fix
New feature
Breaking change
Documentation update
Other (please describe):

Testing

How have you tested the change? Verify that the changes do not break functionality or introduce warnings in consuming repositories: agents-docs, agents-tools, agents-cli

  • I ran hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@afarntrog
Copy link
Contributor Author

@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@afarntrog afarntrog merged commit 25c2aa4 into strands-agents:main Feb 16, 2026
17 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments