Skip to content

🔒 Remove top-level GitHub token configuration#16398

Merged
dsyme merged 5 commits intomainfrom
simp8
Feb 17, 2026
Merged

🔒 Remove top-level GitHub token configuration#16398
dsyme merged 5 commits intomainfrom
simp8

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Feb 17, 2026

Summary

  • Removed references to top-level github-token configuration in workflow parsing
  • Simplified GitHub token precedence logic across various components
  • Updated token selection to prioritize custom and specific tokens

Changes

  • Deleted github-token field from workflow schemas
  • Removed GitHubToken field from WorkflowData struct
  • Updated token selection functions to remove top-level token fallback
  • Modified test cases to reflect new token resolution strategy

Rationale

Simplifying the GitHub token configuration reduces complexity and makes token selection more predictable. Now, tokens are selected based on specific tool or safe-output configurations with clear, direct precedence.

Copilot AI review requested due to automatic review settings February 17, 2026 16:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@dsyme dsyme merged commit a006fb0 into main Feb 17, 2026
56 checks passed
@dsyme dsyme deleted the simp8 branch February 17, 2026 20:03
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.

1 participant