Skip to content

Remove GITHUB_TOKEN env from opencode workflow steps#189

Merged
Jordonbc merged 2 commits intoDevfrom
codex/fix-opencode-workflow-error-tkrg0h
Mar 19, 2026
Merged

Remove GITHUB_TOKEN env from opencode workflow steps#189
Jordonbc merged 2 commits intoDevfrom
codex/fix-opencode-workflow-error-tkrg0h

Conversation

@Jordonbc
Copy link
Collaborator

Motivation

  • Remove explicit GITHUB_TOKEN environment variable from the primary and fallback steps to avoid passing the token unnecessarily through the action environment and rely on configured job permissions instead.

Description

  • Deleted GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} from the primary step in .github/workflows/opencode.yml.
  • Deleted GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} from the fallback step in .github/workflows/opencode.yml and ensured the file ends with a newline.

Testing

  • No automated tests were run for this workflow-only change.

Codex Task

@Jordonbc Jordonbc merged commit 3bd5344 into Dev Mar 19, 2026
5 checks passed
@github-actions
Copy link

The PR described appears to already be merged — the current .github/workflows/opencode.yml doesn't contain any GITHUB_TOKEN env vars. The open PRs are #40 (dependency bumps) and #187.


For completeness, here's my review of what the workflow change would accomplish:

Quality: Good

  • Removes unnecessary token exposure from the action environment.
  • Relies on use_github_token: true input and job-level permissions instead, which is the cleaner pattern.

No bugs

  • The action handles token auth via its use_github_token input, so removing the explicit env var doesn't break anything.

Minor note

  • Line 52 is missing a trailing newline (common YAML convention). Not a functional issue but worth fixing for consistency.

New%20session%20-%202026-03-19T06%3A06%3A05.174Z
opencode session  |  github run

@Jordonbc Jordonbc deleted the codex/fix-opencode-workflow-error-tkrg0h branch March 19, 2026 17:47
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.

1 participant