Skip to content

Add Claude Code GitHub Workflow#5

Closed
alex-jadecli wants to merge 2 commits intomainfrom
add-claude-github-actions-1776001622686
Closed

Add Claude Code GitHub Workflow#5
alex-jadecli wants to merge 2 commits intomainfrom
add-claude-github-actions-1776001622686

Conversation

@alex-jadecli
Copy link
Copy Markdown

🤖 Installing Claude Code GitHub App

This PR adds a GitHub Actions workflow that enables Claude Code integration in our repository.

What is Claude Code?

Claude Code is an AI coding agent that can help with:

  • Bug fixes and improvements
  • Documentation updates
  • Implementing new features
  • Code reviews and suggestions
  • Writing tests
  • And more!

How it works

Once this PR is merged, we'll be able to interact with Claude by mentioning @claude in a pull request or issue comment.
Once the workflow is triggered, Claude will analyze the comment and surrounding context, and execute on the request in a GitHub action.

Important Notes

  • This workflow won't take effect until this PR is merged
  • @claude mentions won't work until after the merge is complete
  • The workflow runs automatically whenever Claude is mentioned in PR or issue comments
  • Claude gets access to the entire PR or issue context including files, diffs, and previous comments

Security

  • Our Anthropic API key is securely stored as a GitHub Actions secret
  • Only users with write access to the repository can trigger the workflow
  • All Claude runs are stored in the GitHub Actions run history
  • Claude's default tools are limited to reading/writing files and interacting with our repo by creating comments, branches, and commits.
  • We can add more allowed tools by adding them to the workflow file like:
allowed_tools: Bash(npm install),Bash(npm run build),Bash(npm run lint),Bash(npm run test)

There's more information in the Claude Code action repo.

After merging this PR, let's try mentioning @claude in a comment on any PR to get started!

alex-jadecli pushed a commit that referenced this pull request Apr 12, 2026
Add claude.yml (@claude mention handler) and claude-code-review.yml
(automatic PR review with code-review plugin) from PR #5.
Remove inline claude-code-review job from ci.yml since the dedicated
workflow replaces it.

https://claude.ai/code/session_012Fo1zDkFscGkGriGfyXALE
alex-jadecli added a commit that referenced this pull request Apr 12, 2026
* fix: add pre-commit hooks, fix all lint and type errors

- Add .pre-commit-config.yaml with: trailing-whitespace, end-of-file-fixer,
  check-yaml, check-added-large-files, ruff (lint + format), mypy, pytest (pre-push)
- Fix 53 E501 line-too-long errors via per-file-ignores for skill scripts
- Fix F401 unused numpy import in embed_tools.py
- Fix E721 type comparison in schema_diff.py (type() != -> is not)
- Fix B017 blind Exception catches in test_cli.py (-> ValidationError)
- Fix all 21 mypy errors: add dict[str, Any] type params across 7 model files,
  fix spider Failure type narrowing with proper type: ignore comments
- Apply ruff-format to 20 files for consistent formatting
- CI workflow: replace separate lint/typecheck job with pre-commit/action
- SessionStart hook: run make install + make install-dev, auto-install pre-commit hooks
- Add pre-commit to dev dependencies
- Set line-length=120 in claude_code_models subpackage

https://claude.ai/code/session_012Fo1zDkFscGkGriGfyXALE

* fix(ci): remove uv cache that requires missing uv.lock

setup-uv enable-cache expects a uv.lock file for cache key generation.
This project uses pip install, not uv sync, so no lockfile exists.
Remove enable-cache to unblock all CI jobs.

https://claude.ai/code/session_012Fo1zDkFscGkGriGfyXALE

* feat: add dedicated Claude Code workflows from PR #5

Add claude.yml (@claude mention handler) and claude-code-review.yml
(automatic PR review with code-review plugin) from PR #5.
Remove inline claude-code-review job from ci.yml since the dedicated
workflow replaces it.

https://claude.ai/code/session_012Fo1zDkFscGkGriGfyXALE

---------

Co-authored-by: Claude <noreply@anthropic.com>
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