Skip to content

fix(ci): add pull-requests write permission to GitHub App token in pre-commit autoupdate workflow - #88

Merged
ShoGinn merged 2 commits into
mainfrom
copilot/fix-with-copilot
Aug 2, 2026
Merged

fix(ci): add pull-requests write permission to GitHub App token in pre-commit autoupdate workflow#88
ShoGinn merged 2 commits into
mainfrom
copilot/fix-with-copilot

Conversation

Copilot AI commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

The pre-commit autoupdate workflow was failing with Resource not accessible by integration when peter-evans/create-pull-request attempted to open a PR. The GitHub App token was scoped to permission-contents: write only — pull-requests: write was missing, so the API call to create the PR was rejected.

Summary

  • Added permission-pull-requests: write to the actions/create-github-app-token step so the generated token carries the required permission for PR creation.
- name: Create release App token
  id: release-token
  uses: actions/create-github-app-token@... # v2
  with:
    app-id: ${{ vars.RELEASE_APP_ID }}
    private-key: ${{ secrets.RELEASE_APP_PRIVATE_KEY }}
    permission-contents: write
    permission-pull-requests: write   # added

Validation

  • just check (or equivalent targeted checks) was run successfully
  • Relevant unit/integration tests were added or updated
  • Examples, fixtures, screenshots, and logs use synthetic data and contain no credentials or personally identifiable information

Tax Logic Review Checklist

  • For tax-calculation numeric assertions, expected values come from IRS/SSA-backed fixtures or are explicitly derived from loaded tax models (not opaque literals)
  • Any intentionally synthetic numeric test data is labeled with a short rationale
  • Tax data or behavior changes are reflected in docs (README.md and/or docs/)
  • Authoritative IRS/SSA sources are linked for tax-data or calculation changes

Risk and Impact

CI-only change. No production code, tax logic, or user-facing behavior affected. Restores the automated weekly pre-commit hook update workflow.

@railway-app

railway-app Bot commented Aug 2, 2026

Copy link
Copy Markdown

🚅 Deployed to the tax-tracker-pr-88 environment in endearing-eagerness

Service Status Web Updated (UTC)
tax-tracker ⏭️ Skipped (View Logs) Web Aug 2, 2026 at 2:50 pm

@railway-app
railway-app Bot temporarily deployed to endearing-eagerness / tax-tracker-pr-88 August 2, 2026 14:48 Destroyed
@railway-app
railway-app Bot temporarily deployed to endearing-eagerness / tax-tracker-pr-88 August 2, 2026 14:50 Destroyed
Copilot AI changed the title [WIP] Fix issue with Copilot integration fix(ci): add pull-requests write permission to GitHub App token in pre-commit autoupdate workflow Aug 2, 2026
Copilot AI requested a review from ShoGinn August 2, 2026 14:51
@ShoGinn
ShoGinn marked this pull request as ready for review August 2, 2026 14:52
Copilot AI review requested due to automatic review settings August 2, 2026 14:52
@ShoGinn
ShoGinn merged commit 9771ac6 into main Aug 2, 2026
7 checks passed
@ShoGinn
ShoGinn deleted the copilot/fix-with-copilot branch August 2, 2026 14:52

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

Pull request overview

Updates the pre-commit autoupdate GitHub Actions workflow so the GitHub App token used by peter-evans/create-pull-request has sufficient permissions to open a pull request, resolving Resource not accessible by integration failures in the automation.

Changes:

  • Added permission-pull-requests: write to the actions/create-github-app-token step so the generated GitHub App token can create PRs.

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.

3 participants