Skip to content

Add claude GitHub actions 1778466254012 - #2142

Merged
diegosouzapw merged 2 commits into
mainfrom
add-claude-github-actions-1778466254012
May 11, 2026
Merged

diegosouzapw merged 2 commits into
mainfrom
add-claude-github-actions-1778466254012

Conversation

@diegosouzapw

Copy link
Copy Markdown
Owner

Summary

  • Describe the user-facing or operational change.

Related Issues

  • Closes #
  • Related to #

Validation

  • npm run lint
  • npm run test:unit
  • npm run test:coverage
  • Coverage is still >= 60% for statements, lines, functions, and branches
  • SonarQube PR analysis is green or any remaining issues are explicitly documented below

Tests Added Or Updated

  • List every changed or added automated test file.
  • If no production code changed, state that here.

Coverage Notes

  • If this PR changes src/, open-sse/, electron/, or bin/, explain which tests cover the change.
  • If coverage moved down in any touched file, explain why and what follow-up task will recover it.

Reviewer Notes

  • Call out any risky areas, migrations, feature flags, or manual validation that reviewers should know about.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@diegosouzapw
diegosouzapw merged commit 67af6c3 into main May 11, 2026
5 of 6 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ffecd43653

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +22 to +24
contents: read
pull-requests: read
issues: read

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Grant write scopes to Claude workflow token

This workflow is meant to let @claude respond on issues/PRs and potentially propose code changes, but contents, pull-requests, and issues are all set to read. With a read-only workflow token, Claude can inspect context but write operations (posting replies, updating threads, branch/PR updates) will fail with permission errors. These scopes should be write for this action mode.

Useful? React with 👍 / 👎.

id: claude-review
uses: anthropics/claude-code-action@v1
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Guard code-review job when fork PR secrets are unavailable

This job runs for all pull_request events but authenticates only via ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}. For PRs from forks (including Dependabot-style fork restrictions), repository secrets are not provided, so this input resolves to an empty string and the Claude action cannot authenticate, causing consistent failures on those PRs. Add a condition to skip fork PRs or another auth/event strategy that works without repository secrets.

Useful? React with 👍 / 👎.

@github-actions

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: cancelled
  • PR test policy: failure

Coverage artifact was not available for this run.

PR Test Policy

This PR changes production code in src/, open-sse/, electron/, or bin/ without accompanying automated tests.

diegosouzapw added a commit to rafacpti23/OmniRoute that referenced this pull request May 13, 2026
- Restore workflow files to match release/v3.8.0 (already merged in diegosouzapw#2142)
- Restore package-lock.json from release branch
- Fix trailing space in Brazilian WhatsApp group URL
@diegosouzapw
diegosouzapw deleted the add-claude-github-actions-1778466254012 branch May 14, 2026 18:03
HouMinXi pushed a commit to HouMinXi/OmniRoute that referenced this pull request Aug 2, 2026
* "Claude PR Assistant workflow"

* "Claude Code Review workflow"
Poid-ZA pushed a commit to Poid-ZA/OmniRoute that referenced this pull request Aug 5, 2026
* "Claude PR Assistant workflow"

* "Claude Code Review workflow"
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