Skip to content

Improve Claude GitHub Actions workflows#96

Open
a-simeshin wants to merge 1 commit intoai-forever:mainfrom
a-simeshin:main
Open

Improve Claude GitHub Actions workflows#96
a-simeshin wants to merge 1 commit intoai-forever:mainfrom
a-simeshin:main

Conversation

@a-simeshin
Copy link
Collaborator

@a-simeshin a-simeshin commented Feb 9, 2026

Summary

  • Убран триггер synchronize из автоматического ревью — Claude не спамит при каждом коммите
  • Добавлены write-права для интерактивного @claude — можно просить править код
  • Переход на OAuth токен (CLAUDE_CODE_OAUTH_TOKEN) вместо API ключа

Changes

Workflow Change
claude-code-review.yml Removed synchronize, switched to OAuth token
claude.yml Changed permissions from read to write

Setup

Секрет CLAUDE_CODE_OAUTH_TOKEN настроен через /install-github-app

🤖 Generated with Claude Code

@a-simeshin a-simeshin self-assigned this Feb 9, 2026
@a-simeshin a-simeshin requested a review from linarkou February 9, 2026 18:30
@a-simeshin a-simeshin added the enhancement New feature or request label Feb 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

LGTM ✓

Изменения корректны:

  • Удаление synchronize триггера предотвратит спам от Claude при каждом коммите
  • Добавление write прав безопасно, так как workflow активируется только при явном упоминании @claude

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

LGTM ✅

Изменения корректны:

  • Убран триггер synchronize — избегаем спама при каждом коммите
  • Переход на claude_code_oauth_token — правильный метод аутентификации
  • Права write необходимы для интерактивного режима

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

LGTM ✅

Изменения выглядят хорошо:

  • Убран триггер synchronize — меньше спама при каждом коммите
  • Переход на OAuth токен для лучшей аутентификации
  • Права write для интерактивного режима

Оба workflow файла корректно настроены.

- Remove synchronize trigger from auto-review (no spam on every commit)
- Add write permissions to interactive @claude (can modify code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

LGTM ✅

Изменения выглядят хорошо:

  • ✅ Правильное разделение на 3 независимых job'а (internal/fork/interactive)
  • ✅ Корректные условия для каждого триггера
  • ✅ Переиспользование env переменных для REVIEW_PROMPT и CLAUDE_TOOLS
  • ✅ Переход на OAuth токен вместо API ключа
  • ✅ Корректные права для интерактивного режима (write вместо read)
  • ✅ Удален триггер synchronize для уменьшения спама

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant